fix(signals): classify WASM source maps as generated - #3523
Conversation
Extend isGeneratedFile source-map parity to bundler .wasm.map output with classifyChangedFile hot-path coverage. Co-authored-by: Cursor <cursoragent@cursor.com>
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
|
Warning 🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨 ⏸️ Gittensory review result - manual review recommendedReview updated: 2026-07-05 15:38:37 UTC
⏸️ Suggested Action - Manual Review Review summary Nits — 1 non-blocking
Review context
Contributor next steps
Signal definitions
🟩 Safe / merged · 🟦 Advisory · 🟨 Held for review · 🟥 Blocked / closed 💰 Earn for open-source contributions like this. Gittensor lets GitHub contributors earn for the work they already do — register to start earning →. Checked by Gittensory, a quiet PR intelligence layer for OSS maintainers.
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3523 +/- ##
=======================================
Coverage 93.09% 93.09%
=======================================
Files 301 301
Lines 31450 31450
Branches 11483 11483
=======================================
Hits 29279 29279
Misses 1517 1517
Partials 654 654
🚀 New features to boost your workflow:
|
Summary
Extend
isGeneratedFilesource-map parity so bundler output likedist/pkg.wasm.mapis classified as generated, matching existing JS/TS and front-end.maphandling. Includes directclassifyChangedFileassertions for the hot-path category behavior.Fixes #561
Scope
type(scope): short summaryConventional Commit format, for examplefix(api): restore profile access checks.CONTRIBUTING.mdand does not reintroduce GitHub Pages, VitePress,site/, orCNAME.Validation
git diff --checknpm run actionlintnpm run typechecknpm run test:coveragelocally;codecov/patchrequires ≥99% coverage of the lines AND branches you changed (aim for 100% on your diff so CI variance does not fail near the threshold). Global coverage is a non-blocking trend with a loose 90% backstop, not the gate.npm run test:workersnpm run build:mcpnpm run test:mcp-packnpm run ui:openapi:checknpm run ui:lintnpm run ui:typechecknpm run ui:buildnpm audit --audit-level=moderateIf any required check was skipped, explain why:
Safety
UI Evidencesection below with JPG/JPEG or PNG screenshots arranged as organized, captioned, clickable thumbnails. SVG screenshots are not used as review evidence. Review-only screenshots or recordings are not committed to the repository.UI Evidence
N/A — signals-only change with no visible UI.
Notes
Incremental maintenance parity for the path-matcher slop signals from #561. Supports generated-file classification work tracked in #2279.
Made with Cursor