fix(rag): index TypeScript .mts and .cts module sources#3341
Conversation
RAG skipped .mts/.cts while isCodeFile already counts them as source, so PRs changing only module TypeScript got no code context. Align CODE_EXT_RE with local-branch classifiers. 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:19:53 UTC
⏸️ Suggested Action - Manual Review Review summary Nits — 3 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 #3341 +/- ##
=======================================
Coverage 93.88% 93.88%
=======================================
Files 280 280
Lines 30534 30534
Branches 11120 11120
=======================================
Hits 28666 28666
Misses 1211 1211
Partials 657 657
🚀 New features to boost your workflow:
|
Summary
.mtsand.ctstoCODE_EXT_REinsrc/review/rag.tsso RAG indexes TypeScript module sources instead of skipping them.isCodeFileinsignals/local-branch.tsalready treats these as source; RAG was the outlier, so PRs touching only.mts/.ctsfiles got no retrieved code context.Scope
type(scope): short summaryConventional Commit format.CONTRIBUTING.md.Validation
git diff --checknpm run test:ci(includes actionlint, typecheck, test:coverage, workers, MCP, UI checks)npm audit --audit-level=moderate.mts/.ctsclassification,filePriority, andisIndexablePathIf any required check was skipped, explain why:
npm run test:cion Node 22 before push.Safety
UI Evidence
N/A — no visible UI change.
Notes
Conflict avoidance: Touches only
src/review/rag.tsandtest/unit/rag.test.ts. Zero file overlap with open PRs (#3316 dart classifiers, #3304 mega, #3340 engine, #3339/#3338/#3337/#3333 enrichment, #3315/#3313/#3310 selfhost, #3314 miner, #3305 enrichment-wire, #3281 grafana, #3255 queue). Should merge cleanly after any of those land without rebase.Made with Cursor