feat(signals): classify automation and CI toolchain config in path-matchers - #1261
Conversation
…tchers Extend the deterministic config matcher (JSONbored#561) so slop classification recognizes common dependency-automation and hosted-CI files: - renovate.json, dependabot.yml, asdf .tool-versions, mise.toml - lefthook and pre-commit hook config - GitLab/Azure pipeline basenames and buf codegen config - .github/workflows/*.yml and .circleci/config.yml Pure path-only matchers with unit coverage and negative cases. Co-authored-by: Cursor <cursoragent@cursor.com>
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1261 +/- ##
=======================================
Coverage 95.39% 95.39%
=======================================
Files 193 193
Lines 20929 20932 +3
Branches 7564 7566 +2
=======================================
+ Hits 19966 19969 +3
Misses 383 383
Partials 580 580
🚀 New features to boost your workflow:
|
JSONbored
left a comment
There was a problem hiding this comment.
This is the earlier of the two path-matcher PRs and it's correct — workflow/circleci/dependabot regexes plus the common toolchain basenames, with a real "renovate-like source names aren't config" guard. It wins the tiebreak over the later #1422 by being the original, not because either is "better." It's a jaso0n0818 fork so I won't touch it — your call to take it; if you do, #1422's extra lockfile/config entries can be folded in after.
Summary
Extends
isConfigFile/classifyChangedFileso slop signals stop miscounting common automation and hosted-CI config as uncategorized source effort (#561).New config recognition:
renovate.json,dependabot.yml.tool-versions,mise.tomllefthook.yml/.lefthook.yaml,.pre-commit-config.yaml.gitlab-ci.yml,azure-pipelines.yml,.github/workflows/*.yml,.circleci/config.ymlbuf.yaml,buf.gen.yamlPath-only, deterministic, full unit coverage + negative cases.
Test plan
npm test -- test/unit/path-matchers.test.ts