Skip to content

Commit ac3ac07

Browse files
committed
chore: add the empty changeset this PR needs
Check Changeset requires one on every PR; an empty-frontmatter changeset is the sanctioned form for a change that releases nothing, and the gate's own error message says so. Second time this session — #4009 was a comment-only PR that failed the same way. I fixed that one and did not draw the rule, which is that "this releases nothing" is a reason to write an EMPTY changeset, not a reason to skip the file. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CYbS3kS8xzsHNXFTzp4e2z
1 parent 2bfdcba commit ac3ac07

1 file changed

Lines changed: 17 additions & 0 deletions

File tree

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
---
2+
---
3+
4+
`domain-handler-registry.test.ts` now uses `envelopeViolations` instead of its own
5+
copy of the rule. Deliberately empty frontmatter: test-only, this releases nothing.
6+
7+
That test hand-rolled "no key beside the envelope's own may hold the payload" for
8+
#4038. #4090 promoted the rule into the spec so it would stop having two
9+
definitions; leaving the local copy would have recreated the exact failure this
10+
line has been closing — one rule, two places, only one updated next time.
11+
12+
The two were also not equivalent. The local set allowed any body whose top-level
13+
keys were `success` / `data` / `meta`, so it passed a success body with no `data`
14+
at all and one carrying an `error` beside `success: true`. The shared predicate
15+
rejects both, which makes this a strict tightening rather than a refactor:
16+
injecting `{ success: true }` into the producer now fails the suite, and used to
17+
pass it.

0 commit comments

Comments
 (0)