fix: include vercel-ai-exporter in build pipeline and workspace#1179
fix: include vercel-ai-exporter in build pipeline and workspace#1179serhiizghama wants to merge 1 commit intoVoltAgent:mainfrom
Conversation
The package was excluded from pnpm-workspace.yaml and the build:all script, causing it to be published without a compiled dist/ directory. Consumers received an empty package after install. Re-add it to: - pnpm-workspace.yaml (workspace member) - root build / build:all scripts (remove --ignore flag) Note: CI test matrix (.github/workflows/) requires workflow scope and is left as a follow-up. Fixes VoltAgent#1024
|
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
1 participant
Problem
The
@voltagent/vercel-ai-exporterpackage is published to npm without a builtdist/directory, making it unusable. Importing from it fails because./dist/index.mjsand./dist/index.jsdon't exist.Root cause: commit 9c81f46 removed the package from
pnpm-workspace.yamland added--ignore @voltagent/vercel-ai-exporterto the root build scripts. As a result,pnpm build:all— which runs during the release workflow — never compiles the package before it is published.Fixes #1024
What changed
pnpm-workspace.yaml: removed the!packages/vercel-ai-exporterexclusion so pnpm treats it as a workspace member againpackage.json(root): removed--ignore @voltagent/vercel-ai-exporterfrom thebuildandbuild:allscripts so the package is compiled duringpnpm build:allHow to test
pnpm install pnpm build:all ls packages/vercel-ai-exporter/dist # should contain index.js, index.mjs, index.d.ts, index.d.mtsAfter the fix,
pnpm pack --dry-runfrompackages/vercel-ai-exportershould include thedist/*files.Note: CI test matrix entries for
vercel-ai-exporterin the workflow YAMLs requireworkflowscope on the token — I'll add those in a follow-up if helpful, or maintainers can add them directly.Summary by cubic
Restores building and publishing of
@voltagent/vercel-ai-exporterso itsdist/files are included and imports resolve. Addresses #1024 by putting the package back into the workspace and build pipeline.!packages/vercel-ai-exporterexclusion to re-add the package to the workspace.package.json: removed--ignore @voltagent/vercel-ai-exporterfrombuildandbuild:allso the package compiles during releases.Written for commit a3af453. Summary will update on new commits.