Skip to content

Conversation

@eLysgaard
Copy link
Contributor

@eLysgaard eLysgaard commented Jan 16, 2026

Replace local ai-sdk-providers implementation with the one we just published on https://www.npmjs.com/package/@databricks/ai-sdk-provider, which is based on the work intially spearheaded in this project. This removes ~3300 lines of local provider code while keeping only providers-server.ts for server-side auth handling.

Changes:

  • Install @databricks/ai-sdk-provider in server, client, and core
  • Update zod to v4.3.5 across all packages
  • Delete local databricks-provider/ directory and mcp-approval-utils.ts
  • Update all imports to use @databricks/ai-sdk-provider directly
  • Fix test fixture for stream format change (finishReason removed)
  • Remove internal implementation test file

Tested and manually verified

  • Build passes (npm run build)
  • Lint passes (npm run lint)
  • Type checks pass
  • Unit tests pass (47 tests)
  • Routes tests pass (8 passed, 6 flaky due to timing)
  • Manually tested using FMAPI and KA

Replace local ai-sdk-providers implementation with the official
@databricks/ai-sdk-provider package (v0.1.0). This removes ~3300 lines
of local provider code while keeping only providers-server.ts for
server-side auth handling.

Changes:
- Install @databricks/ai-sdk-provider in server, client, and core
- Update zod to v4.3.5 across all packages
- Delete local databricks-provider/ directory and mcp-approval-utils.ts
- Update all imports to use @databricks/ai-sdk-provider directly
- Fix test fixture for stream format change (finishReason removed)
- Remove internal implementation test file

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@eLysgaard eLysgaard self-assigned this Jan 16, 2026
Copy link
Member

@simonfaltum simonfaltum left a comment

Choose a reason for hiding this comment

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

Looks good

@bbqiu bbqiu self-requested a review January 16, 2026 19:17
Copy link
Contributor

@bbqiu bbqiu left a comment

Choose a reason for hiding this comment

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

lgtm awesome to see LoC go down :D

i also tested manually with the agent template, thanks for doing this!

@eLysgaard eLysgaard merged commit 527ef01 into databricks:main Jan 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants