build: drop tooling and maintain the type declarations by hand - #656
build: drop tooling and maintain the type declarations by hand#656evenstensberg wants to merge 1 commit into
Conversation
`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 breaks `bun install` on the cross-runtime job: npm records an integrity hash over its own packed tarball, so the tarball bun fetches instead can never match it. `types.d.ts` becomes the source of truth rather than generated output. `lint:types-public` checks that it compiles, and `lint:types-test` already checks it against the test suite, which consumes the package through these declarations and so fails when they drift from `lib/`.
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #656 +/- ##
=======================================
Coverage 98.14% 98.14%
=======================================
Files 49 49
Lines 10018 10018
=======================================
Hits 9832 9832
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:
|
Merging this PR will degrade performance by 74.21%
|
| Mode | Benchmark | BASE |
HEAD |
Efficiency | |
|---|---|---|---|---|---|
| ❌ | Memory | extensions-many: 6-extension list (warm) |
2.8 KB | 102.9 KB | -97.29% |
| ❌ | Memory | node-compare: node require.resolve x 1000 |
22 KB | 27.9 KB | -21.06% |
| ❌ | Memory | restrictions: path prefix + regex (warm) |
6 KB | 7.5 KB | -19.71% |
Tip
Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.
Comparing bun2 (622c3b4) with main (e21cfb8)
1 participant
No description provided.