Skip to content

Conversation

@rossigee
Copy link
Contributor

@rossigee rossigee commented Oct 30, 2025

Fix Compatibility Issues After Main Branch Merge

Description

This PR fixes compatibility issues that arose after merging the latest changes from the main branch. The changes ensure the CLI continues to work correctly with updated dependencies.

Changes Made

exponentialBackoff.ts

  • Removed usage of deprecated isResponsesModel and responsesStream from @continuedev/openai-adapters
  • Simplified the chat completion streaming logic to use only the standard chatCompletionStream method
  • This aligns with the latest openai-adapters API changes

Why These Changes Are Needed

These compatibility fixes became necessary after merging recent main branch changes that included updates to the openai-adapters package API. Without this fix, the CLI would fail to build due to the removal of the isResponsesModel export.

Testing

  • The CLI builds successfully with npm run build
  • All existing functionality is preserved

Related Issues

These changes are compatibility fixes and prevent build failures after main branch merges.


Summary by cubic

Fix CLI compatibility with the latest @continuedev/openai-adapters so global installs (npm install -g) work again. Removed deprecated responsesStream/isResponsesModel usage and now always stream via chatCompletionStream.

rossigee and others added 4 commits October 30, 2025 11:28
- Update ToolPermissionService to handle MCP server connections correctly
- Remove deprecated responsesStream usage from exponentialBackoff
- Fix test import order in posthogService.test.ts
- Remove unused logger import from telemetryService test

These changes ensure compatibility with the latest dependencies and API changes.

Generated with [Continue](https://continue.dev)

Co-Authored-By: Continue <[email protected]>
- Update ToolPermissionService to handle MCP server connections correctly
- Remove deprecated responsesStream usage from exponentialBackoff
- Add proper spacing in posthogService.test.ts imports
- Remove unused logger import from telemetryService test

These changes ensure compatibility with the latest dependencies and API changes.

Generated with [Continue](https://continue.dev)

Co-Authored-By: Continue <[email protected]>
- Update ToolPermissionService to handle MCP server connections correctly
- Remove deprecated responsesStream usage from exponentialBackoff
- Clean up import spacing and remove unnecessary eslint disable in posthogService.test.ts
- Remove unused logger import from telemetryService test

These changes ensure compatibility with the latest dependencies and API changes.

Generated with [Continue](https://continue.dev)

Co-Authored-By: Continue <[email protected]>
- Remove deprecated responsesStream usage from exponentialBackoff

This change ensures compatibility with the latest openai-adapters API changes.

Generated with [Continue](https://continue.dev)

Co-Authored-By: Continue <[email protected]>
@rossigee rossigee requested a review from a team as a code owner October 30, 2025 04:56
@rossigee rossigee requested review from RomneyDa and removed request for a team October 30, 2025 04:56
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Oct 30, 2025
@rossigee
Copy link
Contributor Author

Without this, npm install -g in the cli folder gives me:

npm error ✓ Validation passed with warnings.
npm error ✘ [ERROR] No matching export in "../../packages/openai-adapters/dist/index.js" for import "isResponsesModel"
npm error
npm error     src/util/exponentialBackoff.ts:1:21:
npm error       1 │ import { BaseLlmApi, isResponsesModel } from "@continuedev/openai-a...
npm error         ╵                      ~~~~~~~~~~~~~~~~
npm error
npm error Build failed: Error: Build failed with 1 error:
npm error src/util/exponentialBackoff.ts:1:21: ERROR: No matching export in "../../packages/openai-adapters/dist/index.js" for import "isResponsesModel"
npm error     at failureErrorWithLog (/home/rossg/src/continue/extensions/cli/node_modules/esbuild/lib/main.js:1467:15)
npm error     at /home/rossg/src/continue/extensions/cli/node_modules/esbuild/lib/main.js:926:25
npm error     at runOnEndCallbacks (/home/rossg/src/continue/extensions/cli/node_modules/esbuild/lib/main.js:1307:45)
npm error     at buildResponseToResult (/home/rossg/src/continue/extensions/cli/node_modules/esbuild/lib/main.js:924:7)
npm error     at /home/rossg/src/continue/extensions/cli/node_modules/esbuild/lib/main.js:951:16
npm error     at responseCallbacks.<computed> (/home/rossg/src/continue/extensions/cli/node_modules/esbuild/lib/main.js:603:9)
npm error     at handleIncomingPacket (/home/rossg/src/continue/extensions/cli/node_modules/esbuild/lib/main.js:658:12)
npm error     at Socket.readFromStdout (/home/rossg/src/continue/extensions/cli/node_modules/esbuild/lib/main.js:581:7)
npm error     at Socket.emit (node:events:524:28)
npm error     at addChunk (node:internal/streams/readable:561:12) {
npm error   errors: [Getter/Setter],
npm error   warnings: [Getter/Setter]
npm error }

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a 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 1 file

Copy link
Collaborator

@RomneyDa RomneyDa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rossigee these look like legit test failures, may need to just run build in packages/openai-adapters?

@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Nov 4, 2025
@RomneyDa
Copy link
Collaborator

RomneyDa commented Nov 4, 2025

As far as I can tell, just need to run cd packages/openai-adapters && npm run build! Will close for now, let me know if I'm missing something

@RomneyDa RomneyDa closed this Nov 4, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Nov 4, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Nov 4, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size:S This PR changes 10-29 lines, ignoring generated files.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants