From 10ad91dafbdf7c3a136a946ec2918b11e702bc82 Mon Sep 17 00:00:00 2001 From: sim Date: Tue, 8 Sep 2026 18:38:47 -0400 Subject: [PATCH] fix(#4460): drop the superfluous allow-test-rule marker local/no-source-grep's looksLikeSourcePath only matches readFileSync targets ending in .cjs/.cts/.js/.mjs/.mts/.ts -- WORKFLOW_PATH here points at code-review.md, so the rule can never fire regardless of the marker. Confirmed by reading eslint-rules/no-source-grep.cjs directly before removing it, not assumed. Caught by an independent code-review pass on the sibling #4466 fix, which copied this same now-unnecessary marker pattern -- fixed there too. Co-Authored-By: Claude Sonnet 5 --- tests/code-review-tier3-files-override-scoping.test.cjs | 4 ---- 1 file changed, 4 deletions(-) diff --git a/tests/code-review-tier3-files-override-scoping.test.cjs b/tests/code-review-tier3-files-override-scoping.test.cjs index ee70fb1e6..5c8800581 100644 --- a/tests/code-review-tier3-files-override-scoping.test.cjs +++ b/tests/code-review-tier3-files-override-scoping.test.cjs @@ -1,7 +1,3 @@ -// allow-test-rule: source-text-is-the-product (see #4460) -// Workflow markdown is the installed orchestration contract — this file's -// text IS what the reviewer flow runs at runtime. - 'use strict'; /**