Skip to content

ci: build the workspace before packaging the VSIX - #248

Merged
oratis merged 1 commit into
mainfrom
fix/vsix-build-order
Aug 8, 2026
Merged

ci: build the workspace before packaging the VSIX#248
oratis merged 1 commit into
mainfrom
fix/vsix-build-order

Conversation

@oratis

@oratis oratis commented Aug 8, 2026

Copy link
Copy Markdown
Owner

Summary

Third latent release.yml bug, uncovered once #247 got build-vscode past its version step.

✘ [ERROR] Could not resolve "@deepcode/core/skills"
✘ [ERROR] Could not resolve "@deepcode/core/tools"
✘ [ERROR] Could not resolve "@deepcode/core/dist/providers/deepseek.js"
    ../server/node_modules/@deepcode/core/package.json

build-vscode stamps core's VERSION into packages/core/src/index.ts, then goes straight to packaging. But the VSIX bundles the app-server, which bundles core, and core's package exports point at dist/ — which doesn't exist yet. Two consequences, not one: esbuild can't resolve the subpaths, and the version it just stamped would never have reached the bundle, since nothing compiled it.

pnpm release:check has always been pnpm build && … — which is why the gate passes locally and in validate. publish-cli already has the step at line 134. build-vscode was the one path that skipped it.

Progress on the pipeline

Attempt validate build-vscode Cause
1 No bubblewrap → bwrap ENOENT (#246)
2 ❌ step 7 npm version + workspace:* (#247)
3 ❌ step 8 This PR

Each fix moved the failure forward by one step, which is the expected shape when a pipeline has never run.

Test plan

  • pnpm release:check passes locally with the same build-then-package ordering this adds
  • Confirmed publish-cli already had pnpm build after its stamp step — this makes the two jobs consistent
  • Verified by the next tag run

Release notes label

  • release-notes:internal

Related

Follows #246 and #247. Blocks the v0.3.0 release (#245).

🤖 Generated with Claude Code

build-vscode stamped core's VERSION into packages/core/src/index.ts and went
straight to packaging. The VSIX bundles the app-server, which bundles core, and
core's package exports point at dist/ — so esbuild could not resolve
@deepcode/core/skills, /tools, or /dist/providers/deepseek.js, and the stamped
version would never have reached the bundle either.

`pnpm release:check` has always built first, which is why the gate passes
locally, and publish-cli already had the step. build-vscode was the one path
that skipped it.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@oratis
oratis merged commit b8d1402 into main Aug 8, 2026
5 checks passed
@oratis
oratis deleted the fix/vsix-build-order branch August 8, 2026 14:07
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.

1 participant