Claude and Connector SDK experiment blog #81
Conversation
|
alsnkline
requested review from
varundhall,
ameetfivetran and
fivetran-sadiemartin
|
Not sure I like the folder name I came up with. |
ai_and_connector_sdk/CLAUDE.md
Outdated
There was a problem hiding this comment.
I'm wondering whether this will work or not, as the file in inside a sub-directory.
@alsnkline can you please test once whether we need this file at the root level or it will work in sub-dirs?
There was a problem hiding this comment.
claude_test_20250228 doesn't sound very good, lets rename this directory?
There was a problem hiding this comment.
we don't keep this file in this repo, maybe remove it?
There was a problem hiding this comment.
I was thinking it would allow someone to explore a warehouse.db without having to regenerate.
There was a problem hiding this comment.
I was thinking it would allow someone to explore a warehouse.db without having to regenerate.
maybe I should add a readme.txt to explain these files roles
| if __name__ == "__main__": | ||
| connector.debug() |
There was a problem hiding this comment.
add the standard config.json file opening case here?
| Chat GPT can be effectively used to help you create working connector SDK code. This was achieved by passing it examples from this repo as well as the full [technical reference](https://fivetran.com/docs/connector-sdk/technical-reference) documentation. Chat GPT when tested required some manual adjustments of code. | ||
|
|
||
| # Claude | ||
| Claude proved capable of creating a connector that ran first time and populated a 4 table schema. There were extra unnecessary requirements included in the requirements.txt file and the `__main__` didn't include the code needed to run the connector using my IDE run button. We [created a blog](link_to_blog) about our experiences and include the code as Claude generated it. We have also included the `warehouse.db` and `state.json` files so you can easily review the output of the first sync even though these files are only temp files created during testing and are not required for a connector SDK. |
There was a problem hiding this comment.
don't forget to link the blog before merging
There was a problem hiding this comment.
This looks good to me, but we should definitely have @5tran-alexil review the language (not sure it's best practice to refer to ourselves in the first person?). This wouldn't be a blocker for me in terms of getting it out, though.
Files associated with my Claude and Connector SDK experiment.
Referenced in a blog draft available Connector SDK Creation with Anthropic's Claude
Publish date still to be discussed - DO NOT MERGE until agreed.