Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(go): Added /util/generate + filled feature gaps in Generate API. #1818

Merged
merged 20 commits into from
Feb 8, 2025

Conversation

apascal07
Copy link
Collaborator

@apascal07 apascal07 commented Feb 4, 2025

A lot of things happening here because there was a lot busted while trying to add GenerateAction and make the parity tests pass:

  • Updated Go schema from JS schema.
  • Updated parts to support the correct tool request/response schemas.
  • Updated model.Generate() to execute multiple + parallel tool requests.
  • Added support for interrupts in tools (didn't need to be here but... it made it easier to consider the handleToolRequests() refactor).
  • Added GenerateAction under /util/generate.
  • Loosened the restrictions on tool schemas (can be anything now, including any, instead of map[string]any).
  • Added APIs to control features like max turns, return tool requests, and tool choice (currently unused, plugins need to be updated).

Checklist (if applicable):

@github-actions github-actions bot added docs Improvements or additions to documentation sample root labels Feb 5, 2025
@apascal07 apascal07 requested a review from pavelgj February 5, 2025 21:38
@apascal07 apascal07 marked this pull request as ready for review February 5, 2025 21:38
go/ai/tools.go Outdated Show resolved Hide resolved
@apascal07 apascal07 requested a review from pavelgj February 8, 2025 01:00
@apascal07 apascal07 merged commit 8dc9ed8 into main Feb 8, 2025
7 checks passed
@apascal07 apascal07 deleted the ap-go-generate branch February 8, 2025 02:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants