Skip to content

fix(engine): pin web-tree-sitter back to ^0.20.8, unblocking the engine build - #9234

Merged
JSONbored merged 1 commit into
mainfrom
fix/pin-web-tree-sitter
Jul 27, 2026
Merged

fix(engine): pin web-tree-sitter back to ^0.20.8, unblocking the engine build#9234
JSONbored merged 1 commit into
mainfrom
fix/pin-web-tree-sitter

Conversation

@JSONbored

Copy link
Copy Markdown
Owner

Summary

The npm dependency bump in #9232 jumped web-tree-sitter from ^0.20.8 to ^0.26.11 — a breaking major-version range change. Parser.init() and the SyntaxNode export moved in the new major version, breaking packages/loopover-engine/src/miner/repo-map.ts's tsc build. This has been failing the engine build (and any CI job that builds it) for every PR based on main since that merge — confirmed reproducing on a fresh origin/main checkout.

This PR pins web-tree-sitter back to ^0.20.8 (the previously-working range) rather than migrating repo-map.ts to the new major-version API, to unblock CI immediately. A proper migration to 0.26.x (updating repo-map.ts for the new Parser.init()/SyntaxNode shape) can follow as a separate, deliberate PR.

Test plan

  • npm run build --workspace @loopover/engine — clean (was failing before this pin)
  • npx tsc --noEmit -p tsconfig.json --incremental false — clean across the whole repo

…ne build

The dependency bump in #9232 jumped web-tree-sitter 0.20.8 -> 0.26.11, a
breaking major-version range change: Parser.init() and the SyntaxNode
export moved, breaking src/miner/repo-map.ts's tsc build for every PR
based on main since that merge.
@superagent-security

Copy link
Copy Markdown
Contributor

Superagent didn't find any vulnerabilities or security issues in this PR.

@JSONbored JSONbored self-assigned this Jul 27, 2026
@JSONbored
JSONbored merged commit 03b6418 into main Jul 27, 2026
2 of 5 checks passed
@JSONbored
JSONbored deleted the fix/pin-web-tree-sitter branch July 27, 2026 08:25
@loopover-orb loopover-orb Bot added the gittensor:bug Gittensor-scored bug fix — scores a 0.05x multiplier. label Jul 27, 2026
@codecov

codecov Bot commented Jul 27, 2026

Copy link
Copy Markdown

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
22708 1 22707 21
View the top 1 failed test(s) by shortest run time
test/unit/repo-map.test.ts > test/unit/repo-map.test.ts
Stack Traces | 0s run time
Error: Cannot find package 'web-tree-sitter' imported from .../test/unit/repo-map.test.ts
 ❯ test/unit/repo-map.test.ts:3:1

⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯
Serialized Error: { code: 'ERR_MODULE_NOT_FOUND' }

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

JSONbored added a commit that referenced this pull request Jul 27, 2026
…endencies (#9242)

test/unit/repo-map.test.ts imports both directly from the repo root, but they
were only declared under packages/loopover-engine/package.json. Pinning
web-tree-sitter back to ^0.20.8 (#9234) changed npm's hoisting decision and
left both packages nested under packages/loopover-engine/node_modules/ only,
breaking that root-level test's module resolution in CI. Declaring them as
root devDependencies (matching the engine's pinned version) makes them
resolvable from root-level tests regardless of hoisting.
@github-actions github-actions Bot mentioned this pull request Jul 27, 2026
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gittensor:bug Gittensor-scored bug fix — scores a 0.05x multiplier.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant