-
Notifications
You must be signed in to change notification settings - Fork 3.9k
Add comprehensive tests for AI SDK v5 migration in openai-adapters #9223
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
base: dependabot/npm_and_yarn/packages/openai-adapters/multi-146a0ff17e
Are you sure you want to change the base?
Conversation
- Created comprehensive documentation for PostHog integration covering analytics, feature flags, and data-driven development workflows - Created Atlassian integration docs for Jira and Confluence automation including issue management, sprint planning, and documentation sync - Created Netlify integration docs for deployment automation, environment management, and site monitoring - Updated integrations overview page to include all three new integrations - Added new integration pages to docs.json navigation All three integrations follow the established documentation style and include: - Overview with key capabilities - Detailed setup instructions - Multiple use case examples with specific agent tasks - Manual and automated workflow guidance - GitHub integration examples - Comprehensive troubleshooting section - Links to relevant cookbook guides Co-authored-by: bekah-hawrot-weigel <[email protected]>
- Added Info callouts to cookbook guides referencing Mission Control integrations - Follows same pattern as Sentry integration documentation - Creates clear path for users from CLI workflows to Mission Control automation - PostHog: Links to product analytics integration - Atlassian: Links to Jira/Confluence integration - Netlify: Links to deployment automation integration Co-authored-by: nate <[email protected]> Generated with [Continue](https://continue.dev) Co-Authored-By: Continue <[email protected]>
…lamov/continue into feat/cli-git-ai-integration
Remove test expectation for credentials.json which no longer matches after removing credentials.* pattern. Add new tests to verify legitimate source files (tokens.py, credentials.go, etc.) are not blocked. Co-authored-by: nate <[email protected]>
Remove credentials.* and token.* patterns from IntelliJ security ignore list to match the TypeScript core changes. This ensures consistent behavior across both IDE extensions. Co-authored-by: nate <[email protected]>
Snyk has created this PR to upgrade mocha from 11.7.1 to 11.7.5. See this package in npm: mocha See this project in Snyk: https://app.snyk.io/org/continue-dev-inc.-default/project/7ff64b12-9373-49d8-a6a5-70de1609223c?utm_source=github&utm_medium=referral&page=upgrade-pr
Bumps [actions/cache](https://github.com/actions/cache) from 3 to 5. - [Release notes](https://github.com/actions/cache/releases) - [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md) - [Commits](actions/cache@v3...v5) --- updated-dependencies: - dependency-name: actions/cache dependency-version: '5' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]>
Bumps [JetBrains/qodana-action](https://github.com/jetbrains/qodana-action) from 2025.2.3 to 2025.2.4. - [Release notes](https://github.com/jetbrains/qodana-action/releases) - [Commits](JetBrains/qodana-action@v2025.2.3...v2025.2.4) --- updated-dependencies: - dependency-name: JetBrains/qodana-action dependency-version: 2025.2.4 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]>
Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 6 to 7. - [Release notes](https://github.com/actions/download-artifact/releases) - [Commits](actions/download-artifact@v6...v7) --- updated-dependencies: - dependency-name: actions/download-artifact dependency-version: '7' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]>
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 5 to 6. - [Release notes](https://github.com/actions/upload-artifact/releases) - [Commits](actions/upload-artifact@v5...v6) --- updated-dependencies: - dependency-name: actions/upload-artifact dependency-version: '6' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]>
Add a Tip callout to the models documentation page explaining how users can choose models when creating agents in Mission Control or workflows, and that Continue uses a default agent with recommended model when none is specified. Co-authored-by: bekah-hawrot-weigel <[email protected]> Generated with [Continue](https://continue.dev) Co-Authored-By: Continue <[email protected]>
Removed image and added tips for model selection.
move updated telemetry page to correct location
docs: add tip callout about agent model selection
Add PostHog, Atlassian, and Netlify integration documentation
…ctions/upload-artifact-6 chore(deps): bump actions/upload-artifact from 5 to 6
…ctions/download-artifact-7 chore(deps): bump actions/download-artifact from 6 to 7
…etBrains/qodana-action-2025.2.4 chore(deps): bump JetBrains/qodana-action from 2025.2.3 to 2025.2.4
…ctions/cache-5 chore(deps): bump actions/cache from 3 to 5
fix: overzealous 'token' and 'credential' security regexps
…de0b8d109421a2aa54 [Snyk] Upgrade mocha from 11.7.1 to 11.7.5
feat: Continue CLI Git AI integration for tracking AI-authored code
Added text at the bottom of the Integrations section pointing to Mission Control Integrations hub and docs page. Generated with [Continue](https://continue.dev) Co-Authored-By: Continue <[email protected]> Co-authored-by: bekah-hawrot-weigel <[email protected]>
Add integrations reference links to docs
* Test changelog * Remove redirect * Clean up language * Add note icon * Add links * Update YT link to include embed
- Update convertToolsToVercel test to verify inputSchema field (v5 change) - Add ai-sdk-v5-migration.test.ts: 20 tests for tool conversion edge cases - Add ai-sdk-v5-usage.test.ts: 17 tests for usage field mapping (inputTokens/outputTokens) - Add ai-sdk-v5-tool-calls.test.ts: 24 tests for tool call input field mapping - Add ai-sdk-v5-model-init.test.ts: 21 tests for model initialization changes (provider.chat) Total: 82 new test cases covering v5 migration changes including: - Tool definition format (parameters → inputSchema) - Usage tracking (promptTokens → inputTokens, completionTokens → outputTokens) - Tool calls (args → input) - Model initialization (provider(model) → provider.chat(model)) - Parameter renaming (maxTokens → maxOutputTokens) - Anthropic caching (promptTokensDetails → inputTokensDetails) Generated with [Continue](https://continue.dev) Co-authored-by: peter-parker <[email protected]> Co-Authored-By: Continue <[email protected]>
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.
No issues found across 61 files
Documentation ReviewI've reviewed PR #9223 for necessary documentation updates. Conclusion: No documentation updates needed. Reasoning:
The tests ensure correctness of the internal migration, but this doesn't require user-facing documentation. |
Summary
This PR adds comprehensive test coverage for the AI SDK v4 to v5 migration changes introduced in #9115.
Changes
Updated Existing Tests
New Test Suites (82 tests total)
1. ai-sdk-v5-migration.test.ts (20 tests)
Tests for tool conversion changes:
parameters→inputSchema)2. ai-sdk-v5-usage.test.ts (17 tests)
Tests for usage field mapping:
promptTokens→inputTokens,completionTokens→outputTokenspromptTokensDetails→inputTokensDetails3. ai-sdk-v5-tool-calls.test.ts (24 tests)
Tests for tool call structure changes:
args→input4. ai-sdk-v5-model-init.test.ts (21 tests)
Tests for model initialization pattern changes:
provider(modelName)→ v5 pattern:provider.chat(modelName)maxTokens→maxOutputTokensTest Coverage
All tests pass (82/82):
Key Migration Changes Tested
parameters→inputSchemapromptTokens→inputTokenscompletionTokens→outputTokenspromptTokensDetails→inputTokensDetails(Anthropic)args→inputprovider(model)→provider.chat(model)maxTokens→maxOutputTokensRelated
This agent session was co-authored by peter-parker and Continue.