From 4b2de40f90b006df2e2f0315d76a2dd86ccfefba Mon Sep 17 00:00:00 2001 From: Tom Boucher Date: Fri, 15 May 2026 08:14:28 -0400 Subject: [PATCH] fix(3536): silence source-grep lint on config-schema parity test MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The SDK side of the parity test asserts the source shape of sdk/src/query/config-schema.ts (must re-export from ../configuration/index.js; must NOT contain inline `new Set([...])` literals). Runtime/IR comparison cannot distinguish a re-export from a redeclared Set with identical contents — only source inspection catches drift back to inline literals. Adds the documented `// allow-test-rule:` annotation explaining why the three `src.includes()` calls are structurally necessary. Test behavior unchanged; all 6 tests still pass; lint-no-source-grep now reports 0 violations across 514 test files. --- tests/config-schema-sdk-parity.test.cjs | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/tests/config-schema-sdk-parity.test.cjs b/tests/config-schema-sdk-parity.test.cjs index 9bd3dc3bc..569897717 100644 --- a/tests/config-schema-sdk-parity.test.cjs +++ b/tests/config-schema-sdk-parity.test.cjs @@ -1,5 +1,11 @@ 'use strict'; +// allow-test-rule: structural source assertion — verifies that sdk/src/query/config-schema.ts +// is a re-export shell (no inline literals) rather than a redeclared Set. Runtime/IR comparison +// cannot distinguish a re-export from a redeclared `new Set([...])` with identical contents +// because both would deep-equal the manifest set; only source-shape inspection catches drift +// back to inline literals. See test "SDK config-schema.ts re-exports from configuration module". + /** * Manifest-as-source-of-truth guard (Phase 2, Cycle 5, #3536). *