Skip to content

[appkit plugin] Add pnpm workspace for JS packages#383

Open
hubertzub-db wants to merge 1 commit intodatabricks:mainfrom
hubertzub-db:stack/agent-plugin-p1
Open

[appkit plugin] Add pnpm workspace for JS packages#383
hubertzub-db wants to merge 1 commit intodatabricks:mainfrom
hubertzub-db:stack/agent-plugin-p1

Conversation

@hubertzub-db
Copy link

@hubertzub-db hubertzub-db commented Mar 19, 2026

🥞 Stacked PR


Summary

Introduces pnpm workspaces to manage the JS/TypeScript packages in this repo alongside the existing Python packages.

  • Added pnpm-workspace.yaml registering integrations/ai-sdk-provider and integrations/langchainjs
  • Added root package.json (private, workspace scripts, pnpm.onlyBuiltDependencies)
  • Replaced per-package package-lock.json files with a single pnpm-lock.yaml
  • @databricks/langchainjs now depends on @databricks/ai-sdk-provider via workspace:^ (resolved to a real version range at publish time)
  • Updated all JS CI workflows (langchainjs.yml, ai-sdk-provider.yml and their release counterparts) to use pnpm/action-setup@v4, cache: "pnpm", pnpm install --frozen-lockfile, and pnpm publish --no-git-checks
  • Added node_modules/ to .gitignore

Python tooling (uv, pyproject.toml, for-each-project) is untouched — the two ecosystems coexist independently.

@hubertzub-db hubertzub-db changed the title Test [appkit] Add pnpm workspace for JS packages Mar 19, 2026
@hubertzub-db hubertzub-db changed the title [appkit] Add pnpm workspace for JS packages [appkit plugin] Add pnpm workspace for JS packages Mar 19, 2026
plan.md Outdated
@@ -0,0 +1,256 @@
# Plan: npm Package Management in databricks-ai-bridge
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we remove this plan.md?

Copy link
Collaborator

@smurching smurching left a comment

Choose a reason for hiding this comment

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

LGTM! Just one comment on removing plan.md

@hubertzub-db hubertzub-db force-pushed the stack/agent-plugin-p1 branch from 50e789a to 472e162 Compare March 20, 2026 09:25
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.

2 participants