feat: Added agent support to SDK #54
Conversation
| @@ -125,6 +125,86 @@ def to_dict(self) -> dict: | |||
| } | |||
|
|
|||
|
|
|||
| @dataclass(frozen=True) | |||
| class LDAIAgent: | |||
There was a problem hiding this comment.
@keelerm84 you might chime in on this but I know we talked about dropping the LD prefix from class names if the language supports namespaces. Do we want to make this adjustment to the AI SDK before we release 1.0?
There was a problem hiding this comment.
Yes, I think we should probably do that, though in another task at this point. Is that something you want to do, or would you rather I take it?
|
@keelerm84 and @jsonbailey I've pushed some update to the branch, please take a look when you get a chance. Python isnt one of my strong languages so a close review will be appreciated - Cursor agents and I wrote this code together 😄 |
| @@ -125,6 +125,86 @@ def to_dict(self) -> dict: | |||
| } | |||
|
|
|||
|
|
|||
| @dataclass(frozen=True) | |||
| class LDAIAgent: | |||
There was a problem hiding this comment.
Yes, I think we should probably do that, though in another task at this point. Is that something you want to do, or would you rather I take it?
Requirements
Related issues
Provide links to any issues in this repository or elsewhere relating to this pull request.
Describe the solution you've provided
Provide a clear and concise description of what you expect to happen.
Describe alternatives you've considered
Provide a clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context about the pull request here.