fix(signals): classify vue/svelte/astro source maps as generated#3357
Conversation
Vite/Rollup emit .vue.map/.svelte.map/.astro.map siblings for framework sources; slop already treated .ts/.mts maps as generated but missed these spellings after vue/svelte/astro became code. 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 04:45:36 UTC
⏸️ Suggested Action - Manual Review Review summary Nits — 4 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 #3357 +/- ##
=======================================
Coverage 93.88% 93.88%
=======================================
Files 280 280
Lines 30562 30562
Branches 11132 11132
=======================================
Hits 28694 28694
Misses 1211 1211
Partials 657 657
🚀 New features to boost your workflow:
|
Summary
isGeneratedFileinpath-matchers.tsto treat.vue.map,.svelte.map, and.astro.mapas generated output, matching the existing.ts.map/.mts.maphandling (fix(signals): classify .mts and .cts source maps as generated #3347).classifyChangedFile.Scope
CONTRIBUTING.md.Validation
git diff --checknpm run test:cion Node 22npm audit --audit-level=moderate— 0 vulnerabilities.vue.map,.svelte.map,.astro.mapSafety
UI Evidence
N/A — backend path classifier only.
Notes
Conflict avoidance: Touches only
src/signals/path-matchers.tsandtest/unit/path-matchers.test.ts. Zero overlap with open PRs (#3316 dart, #3304 review holds, #3350 api auth, #3340 engine, #3339 enrichment, #3314 miner, #3305 enrichment-wire, #3281 grafana). Merges cleanly after any of those land without rebase.Made with Cursor