Skip to content

Commit ea51df2

Browse files
committed
test(pm): quote the paths the comment-mask cases name, so they can fail (#8478)
Reverse-verifying the comment half — removing the mask and watching the diagnostics — showed two of the new cases staying GREEN through the ablation. The scan only ever reads QUOTED spans, and those two fixtures named their paths bare inside the comment, so they asserted nothing either way. Quoted, the same ablation reddens all three comment cases. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_018WuTtyckQa1VcXwgd52JpN
1 parent 760caf8 commit ea51df2

1 file changed

Lines changed: 8 additions & 2 deletions

File tree

scripts/pm/dispatch-gates.mjs

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -899,11 +899,17 @@ function selfTest() {
899899
// expensive one — a false lead is pasted into every dispatch prompt whose
900900
// surface brushes it — but a narrowing that also drops the real inputs would
901901
// just move the dishonesty.
902+
// Every path a comment case names is QUOTED inside that comment, because the
903+
// scan only ever reads quoted spans: an unquoted path in prose is invisible
904+
// to it with or without masking, so a fixture spelling one bare would assert
905+
// nothing and pass forever. Measured — the first draft of the two cases below
906+
// did exactly that, and only the reverse-verification run that removed the
907+
// comment mask showed them staying green through it.
902908
const commented = [
903909
'/**',
904-
' * Reads `packages/spec/src` and .changeset/x.md — prose, not inputs.',
910+
' * Reads `packages/spec/src` and `.changeset/x.md` — prose, not inputs.',
905911
' */',
906-
"const REAL = 'packages/rest/src'; // twin of packages/core/src, says the comment",
912+
"const REAL = 'packages/rest/src'; // twin of 'packages/core/src', says the comment",
907913
"const U = 'https://example.com/a/b'; const AFTER_URL = 'packages/metadata/src';",
908914
"const RE = /['\"`]/;",
909915
"const AFTER_REGEX = 'packages/client/src';",

0 commit comments

Comments
 (0)