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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

1 participant