Skip to content

docs(CLAUDE.md): add write/read symmetry convention (#174, #175) - #176

Merged
jphein merged 1 commit into
mainfrom
docs/write-read-symmetry-convention
May 29, 2026
Merged

docs(CLAUDE.md): add write/read symmetry convention (#174, #175)#176
jphein merged 1 commit into
mainfrom
docs/write-read-symmetry-convention

Conversation

@jphein

@jphein jphein commented May 29, 2026

Copy link
Copy Markdown
Collaborator

This session found two bugs of the same shape — write canonicalizes the value, read doesn't — at the room (#174) and wing (#175) layers. The lesson generalizes: any time a write normalizes/validates input into a canonical form, every read surface must apply the same normalization/validation, or you get silent empty-result bugs.

Codifies the discovery pattern so future code review catches it proactively rather than after-the-fact. Sits next to the silent-exception convention (#169) and the library-version awareness section as the third recurring pattern this session converged on.

This session found two bugs of the same shape — write canonicalizes
the value, read doesn't — at the room (#174) and wing (#175) layers.

The lesson generalizes: any time a write normalizes/validates input
into a canonical form, every read surface must apply the same
normalization/validation, or you get silent empty-result bugs where
the same data is unreachable by a slightly-different query.

Codify the discovery pattern so future code review catches it
proactively rather than after-the-fact:
  "when you find a write that canonicalizes a value, grep for every
   endpoint that reads that value and verify they share the
   canonicalization."

Sits next to the silent-exception convention (#169) and the library-
version awareness section as the third recurring pattern this
session's autonomous loop converged on.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 29, 2026 00:41
@gemini-code-assist

Copy link
Copy Markdown

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@jphein
jphein merged commit e7b4082 into main May 29, 2026
@jphein
jphein deleted the docs/write-read-symmetry-convention branch May 29, 2026 00:41

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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.

2 participants