-
Notifications
You must be signed in to change notification settings - Fork 4.1k
Continue CLI support provider Ask Sage #9719
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
Continue CLI support provider Ask Sage #9719
Conversation
With a custom config.yaml file, a user can now specify models to use the Ask Sage provider (specifically, provider name is specified as 'askSage' in the provider parameter). Users can now use the CLI tool along with the VSCode extension, and can specify what tools/capabilities they would like the models to have within the CLI, as long as that model has the ability to use tools.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2 issues found across 4 files
Prompt for AI agents (all issues)
Check if these issues are valid — if so, understand the root cause of each and fix them.
<file name="packages/openai-adapters/src/apis/AskSage.ts">
<violation number="1" location="packages/openai-adapters/src/apis/AskSage.ts:155">
P1: Token retrieval failures are cached for an hour, preventing retries after a transient error.</violation>
<violation number="2" location="packages/openai-adapters/src/apis/AskSage.ts:364">
P2: Streaming tool call deltas reuse index 0 for every call, collapsing multiple tool calls into one when merging streamed responses.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
…re being squashed into one index, but now each tool call will have its own index, and fixed a token caching issue where if the token creation failed, the promise isn't saved in cache
RomneyDa
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Addition of Ask Sage to open ai adapters.
@alex-mcgraw-askSage note since this package is a dependency of core you could deduplicate the AskSageResponse, AskSageRequestArgs, etc. types between the two by importing them from @continuedev/openai-adapters. This is non-blocking.
|
🎉 This PR is included in version 1.8.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Description
With a custom config.yaml file, a user can now specify models to use the Ask Sage provider (specifically, provider name is specified as 'askSage' in the provider parameter). Users can now use the CLI tool along with the VSCode extension, and can specify what tools/capabilities they would like the models to have within the CLI, as long as that model has the ability to use tools.
AI Code Review
@continue-reviewChecklist
Screen recording or screenshot
Tests
11 tests were created in "packages/openai-adapters/src/test/asksage-adapter.vitest.ts".
Continue Tasks:▶️ 2 queued — View all
Summary by cubic
Add AskSage provider to the CLI so you can run AskSage models with tools via config.yaml. This aligns CLI behavior with the VS Code extension.
New Features
Migration
Written for commit b4d3844. Summary will update on new commits.