Skip to content

itk wasm dist - #1585

Merged
thewtex merged 2 commits into
mainfrom
itk-wasm-dist
Aug 12, 2026
Merged

itk wasm dist#1585
thewtex merged 2 commits into
mainfrom
itk-wasm-dist

Conversation

@thewtex

@thewtex thewtex commented Aug 11, 2026

Copy link
Copy Markdown
Member
  • fix(packaging): declare files so pnpm 11 ships dist
  • chore(itk-wasm): bump version to 1.0.0-b.201

thewtex and others added 2 commits August 11, 2026 15:54
pnpm 11 re-anchors workspace root .gitignore patterns to the package being
packed, so the root `dist/*` rule (intended for the documentation site) matched
`<package>/dist/*` and stripped the build output from the tarball. itk-wasm
1.0.0-b.200 published with no dist/ at all, leaving every module, types, and
exports entry pointing at a missing file.

Bindgen-generated packages were unaffected because they ship an .npmignore,
which stops the walk up to the workspace root. Declare "files" on the three
hand-maintained core TypeScript packages instead, which takes precedence over
ignore files entirely. Anchoring the root rule as `/dist/*` does not help,
since pnpm re-anchors it either way.

Verified with pnpm 11.21.0: itk-wasm packs 363 dist files (matching b.199),
@itk-wasm/demo-app 2052, and create-itk-wasm 84. Nothing referenced the
previously packed test/, tsconfig.json, or playwright.config.js.

Add an ava regression test covering both the manifest declaration and the
actual pnpm pack output.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@thewtex
thewtex merged commit cbae0d1 into main Aug 12, 2026
68 checks passed
@thewtex
thewtex deleted the itk-wasm-dist branch August 12, 2026 20:05
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