Skip to content

docs(manifesto): no stringly-typed discriminators (from #147 feedback loop)#149

Merged
hadamrd merged 1 commit into
trunkfrom
docs/quality-manifesto-enum-rule
May 28, 2026
Merged

docs(manifesto): no stringly-typed discriminators (from #147 feedback loop)#149
hadamrd merged 1 commit into
trunkfrom
docs/quality-manifesto-enum-rule

Conversation

@hadamrd
Copy link
Copy Markdown
Owner

@hadamrd hadamrd commented May 28, 2026

Adds the rule the CTO called out — every bug becomes a permanent gate. Critic (#141) reads the manifesto and blocks sev1 violations.

…loop)

Closes the feedback loop the CTO described: every bug we fix becomes a
permanent gate. Today's PR #147 (critic SDK event-capture mismatch)
exposed a 4-PR train of bugs with the same shape — #97, #120, #128,
#147 — all driven by string-literal discriminators that didn't match
across module boundaries.

The critic (PR #141) reads the quality manifesto + flags sev1
violations. This rule + the critic infrastructure together mean the
next worker that writes ``event["type"] == "result"`` (or similar
cross-module string-comparison) gets the PR auto-blocked with the
manifesto rationale.
@hadamrd hadamrd merged commit 687fba6 into trunk May 28, 2026
1 check passed
@hadamrd hadamrd deleted the docs/quality-manifesto-enum-rule branch May 28, 2026 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant