-
-
Notifications
You must be signed in to change notification settings - Fork 43
feat(enrichment): ReDoS scanner on added/changed regex #1503
Copy link
Copy link
Closed
Labels
gittensor:featureGittensor-scored feature linked to a feature issue - worth 1.25x multiplier.Gittensor-scored feature linked to a feature issue - worth 1.25x multiplier.gittensor:priorityMaintainer-selected Gittensor priority bonus - worth 1.75x multiplier.Maintainer-selected Gittensor priority bonus - worth 1.75x multiplier.
Description
Metadata
Metadata
Assignees
Labels
gittensor:featureGittensor-scored feature linked to a feature issue - worth 1.25x multiplier.Gittensor-scored feature linked to a feature issue - worth 1.25x multiplier.gittensor:priorityMaintainer-selected Gittensor priority bonus - worth 1.75x multiplier.Maintainer-selected Gittensor priority bonus - worth 1.75x multiplier.
Projects
StatusShow more project fields
Done
Context
A REES (review-enrichment service) analyzer. Tier: high-value.
Detects: A regex introduced/modified by the PR vulnerable to catastrophic backtracking (nested/overlapping quantifiers, ambiguous alternation) — DoS on attacker-controlled input.
Data source: Bundled recheck (npm) or redos-detector run over regex literals extracted from added (+) diff lines. Pure CPU, no network. Exact-or-fuzz verdict.
This is heavy/external/historical analysis the no-checkout headless
claude --printreviewer cannot do; the REES returns it as a brief block the engine splices into the review (additive + fail-safe).Implementation (established pattern, all inside
review-enrichment/)BriefFindingskey insrc/types.tssrc/analyzers/<name>.ts— pure, injectfetchfor testssrc/brief.tsANALYZERS registrysrc/render.tsnode:testunits againstdist/+ a live smoke against the real data sourceDeliverables
Parent: #1499