Skip to content

docs(rules): spec-isolation decided up front; UI changes ship with a reviewed screenshot - #216

Merged
alicoding merged 1 commit into
mainfrom
testing-rules-additions
Aug 18, 2026
Merged

docs(rules): spec-isolation decided up front; UI changes ship with a reviewed screenshot#216
alicoding merged 1 commit into
mainfrom
testing-rules-additions

Conversation

@alicoding

Copy link
Copy Markdown
Owner

Two standing rules distilled from today's failure classes: (1) a spec asserting global app state declares dedicated-server isolation when written, never discovered as a CI cohabitation flake — and no test depends on another test's leftovers, same file included; (2) every UI change attaches a screenshot reviewed against the design contract before the PR opens — the layer where pipeline-green-but-visually-wrong gets caught.

🤖 Generated with Claude Code

https://claude.ai/code/session_01FW5GkkAG8du7tNdYLk2zSd

…anges ship with a reviewed screenshot

Both rules from the same day's failure classes: cohabitation/order
dependence bit twice (atlas session bleed, guardrail file-order
history), and two pipeline-green visual defects were caught only by
eyes (selection invisibility, sticky ring).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FW5GkkAG8du7tNdYLk2zSd
@alicoding
alicoding enabled auto-merge (squash) August 18, 2026 02:00
@alicoding
alicoding merged commit f044326 into main Aug 18, 2026
20 checks passed
@alicoding
alicoding deleted the testing-rules-additions branch August 18, 2026 02:01
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