chore: replace tooling repo - #655
Conversation
🦋 Changeset detectedLatest commit: 6b79036 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #655 +/- ##
=======================================
Coverage 98.14% 98.14%
=======================================
Files 49 49
Lines 10018 10020 +2
=======================================
+ Hits 9832 9834 +2
Misses 186 186
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Bun migrates `package-lock.json` when no bun lockfile is present. npm records the `tooling` git dependency with a `git+ssh://` resolved URL and an npm-computed integrity hash, but Bun fetches the GitHub tarball instead, so the hash never matches and the install fails with `IntegrityCheckFailed extracting tarball from tooling`. Remove `package-lock.json` in the runner checkout so Bun resolves from `package.json` directly. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Merging this PR will degrade performance by 97.29%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Memory | extensions-many: 6-extension list (warm) |
2.8 KB | 102.9 KB | -97.29% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing bun-install-ci (6b79036) with main (e21cfb8)
`tooling` reaches for the TypeScript compiler API, which the native port no longer exposes from the `typescript` entry point, and its `git+ssh` dependency entry is what broke `bun install` on the cross-runtime job. Generate the declarations with `tsc` instead. They are emitted from the JSDoc in `lib/` into `types/`, replacing the single bundled `types.d.ts`, and `scripts/check-types.js` takes over the check that the committed output is up to date. `CachedInputFileSystem` becomes a named class declaration so that the public surface references it by name instead of inlining its private fields. With the git dependency gone the bun job installs without a workaround.
evenstensberg
changed the title
1 participant
What kind of change does this PR introduce?
ci fix
Did you add tests for your changes?
N/A
Does this PR introduce a breaking change?
No
If relevant, what needs to be documented once your changes are merged or what have you already documented?
N/A
Use of AI
Me and my good friend Claude