Skip to content

fix(vscode): compilation + binary bundling#74

Merged
avrabe merged 1 commit intomainfrom
fix/vscode-build
Mar 22, 2026
Merged

fix(vscode): compilation + binary bundling#74
avrabe merged 1 commit intomainfrom
fix/vscode-build

Conversation

@avrabe
Copy link
Copy Markdown
Contributor

@avrabe avrabe commented Mar 22, 2026

VSIX builds correctly with bundled rivet binary. 🤖

- Fix Mocha import (default import for ESM compat)
- Fix glob import (named import for v11)
- Fix workspaceRoot undefined reference in WebView bridge
- Include bin/ in VSIX (was excluded by .vscodeignore)
- VSIX now bundles rivet binary for self-contained install

Trace: skip
@avrabe avrabe merged commit b8030f1 into main Mar 22, 2026
@avrabe avrabe deleted the fix/vscode-build branch March 22, 2026 06:22
Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'Rivet Criterion Benchmarks'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.20.

Benchmark suite Current: 7af4d77 Previous: 1d83426 Ratio
store_lookup/100 2158 ns/iter (± 26) 1722 ns/iter (± 12) 1.25
store_lookup/1000 24749 ns/iter (± 346) 19334 ns/iter (± 343) 1.28
traceability_matrix/1000 59354 ns/iter (± 576) 40479 ns/iter (± 119) 1.47
query/100 818 ns/iter (± 2) 638 ns/iter (± 1) 1.28
query/1000 7163 ns/iter (± 21) 5179 ns/iter (± 17) 1.38

This comment was automatically generated by workflow using github-action-benchmark.

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 22, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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.

1 participant