test: split package smoke verification#67
Conversation
Move the expensive install-pack-install package smoke path behind a dedicated test:package script so default bun test keeps only fast package metadata coverage. Run the package smoke test explicitly in CI and document the full packaging verification command.
Greptile SummaryThis PR splits the expensive end-to-end package smoke test (install → build → pack → consumer-install → ESM/CJS execution) into a dedicated
Confidence Score: 5/5Safe to merge — this is a pure test-infrastructure reorganisation with no changes to production code or package exports. All production code is untouched. The smoke test logic is identical to what was in package.test.ts; the only meaningful change is the 15s to 60s timeout correction and the file split. CI covers both test paths. No regressions are expected. No files require special attention.
|
| Filename | Overview |
|---|---|
| test/package-smoke.ts | New smoke test file extracted from package.test.ts; timeout corrected to 60s; structure and logic unchanged from prior code |
| test/package.test.ts | Heavy install/build/pack smoke test removed; replaced with fast package.json metadata and exports-shape assertions that are safe to run in the default suite |
| .github/workflows/test.yml | Added a dedicated CI step that runs bun run test:package after the existing bun test step |
| package.json | Added test:package script that explicitly targets the smoke test file via bun test ./test/package-smoke.ts |
| docs/package-exports.md | Added a Verification section documenting the two-tier test commands |
| .changeset/quiet-maps-begin.md | Patch-level changeset added as required by project convention |
Reviews (2): Last reviewed commit: "test: relax package smoke timeout" | Re-trigger Greptile
Increase the dedicated package smoke test timeout to avoid flaky CI failures when temporary workspaces perform cold installs and builds.
Summary
bun run test:package.bun testsuite.bun run test:packageexplicitly in CI and document the command in package export docs.Closes #56
Verification
bun fmtbun lint:fixbun testbun run test:packagebun typecheck