Skip to content

fix(package): include wasm artifacts - #25

Closed
ubugeeei wants to merge 1 commit into
mainfrom
codex/fix-wasm-packlist
Closed

fix(package): include wasm artifacts#25
ubugeeei wants to merge 1 commit into
mainfrom
codex/fix-wasm-packlist

Conversation

@ubugeeei

Copy link
Copy Markdown
Member

Summary

  • remove wasm-pack generated ignore rules after browser artifact builds
  • add an npm pack dry-run assertion for required browser wasm artifacts
  • make the fastqr package check target ignore generated wasm output explicitly

Closes #16

Validation

  • pnpm install --frozen-lockfile
  • vp run fastqr#build-browser
  • node npm/fastqr/scripts/check-package-artifacts.mjs --browser
  • npm pack --dry-run --json includes wasm/fastqr_wasm.js, wasm/fastqr_wasm_bg.wasm, and declarations

@ubugeeei

Copy link
Copy Markdown
Member Author

Landed via #27 after local vp run ci and final GitHub Actions CI passed. Closing this issue-scoped PR to keep the queue and PR list clean.

@ubugeeei ubugeeei closed this May 18, 2026
@ubugeeei
ubugeeei deleted the codex/fix-wasm-packlist branch May 18, 2026 04:49
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.

fix(package): include wasm browser artifacts in npm tarballs

1 participant