fix(enrichment): classify workflow paths case-insensitively in analysis-context#2525
Conversation
…is-context The actionPin scheduler gate used case-sensitive workflow path matching, so mixed-case workflow files were skipped even after the analyzer learned to scan them. Co-authored-by: Cursor <cursoragent@cursor.com>
|
Warning 🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨 ⏸️ Gittensory review result - manual review recommendedReview updated: 2026-07-02 08:22:54 UTC
⏸️ Suggested Action - Manual Review Review summary Nits — 5 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.
|
…lyzer Extract isWorkflowPath so analysis-context categorization and actionPin use the same case-insensitive rule, with a scheduler regression test. Co-authored-by: Cursor <cursoragent@cursor.com>
Summary
actions-pinto scan mixed-case workflow paths, but the REES scheduler still gatedactionPinonfile.category === "workflow"fromcategorizeFile, which used case-sensitive.github/workflows/matching..github/Workflows/CI.YMLwere classified as non-workflow files, soactionPinwas skipped withno_workfloweven though the analyzer would scan them if invoked.actions-pin.ts.Scope
type(scope): short summaryConventional Commit format.CONTRIBUTING.mdand does not reintroduce GitHub Pages, VitePress,site/, orCNAME.Validation
git diff --checkreview-enrichmentbuild +node --test test/analysis-context.test.ts— 12/12 passing (includes mixed-case workflow path regression)Safety
Notes
Made with Cursor