Skip to content

fix: replace all Anti-Pattern terminology with Failure Mode#636

Merged
notque merged 1 commit into
mainfrom
fix/joy-check-deep-anti-pattern-cleanup
May 10, 2026
Merged

fix: replace all Anti-Pattern terminology with Failure Mode#636
notque merged 1 commit into
mainfrom
fix/joy-check-deep-anti-pattern-cleanup

Conversation

@notque
Copy link
Copy Markdown
Owner

@notque notque commented May 10, 2026

Summary

  • Fleet-wide deep cleanup: 302 Anti-Pattern occurrences → 0 across 178 files
  • Previous PR fix: positive-frame all references for 100% joy-check compliance #635 only caught heading-level patterns (7 regex rules)
  • This PR catches everything the headings missed: table column headers, bold labels, inline references, blockquote scopes
  • Validator regex expanded from heading-only (^\s*#{1,6}.*Anti-Pattern) to all prose positions (Anti-Patterns?)
  • 27 instances in fenced code blocks preserved (legitimate Go/Python examples)
  • All changes are mechanical s/Anti-Pattern/Failure Mode/ — zero semantic edits

Test plan

  • python3 scripts/validate_positive_instruction_docs.py returns count: 0
  • pytest scripts/tests/test_joy_check_instruction_mode.py — all pass
  • CI joy-check job passes
  • Spot-check: fenced code blocks still contain // Anti-pattern: (preserved)

Extends joy-check enforcement beyond headings to all prose contexts:
table column headers, bold labels, inline references, blockquote scopes.

302 occurrences replaced across 179 files. 27 instances inside fenced
code blocks preserved (legitimate Go/Python examples showing patterns
to detect).

Validator expanded: Anti-Pattern regex now matches all prose positions,
not just markdown headings. Catches future regressions in table cells,
inline text, and reference descriptions.

Exclusions preserved: fenced code blocks, allowlisted files
(PHILOSOPHY.md, instruction-rubric.md, voice corpus).
@notque notque merged commit c7c313f into main May 10, 2026
6 checks passed
@notque notque deleted the fix/joy-check-deep-anti-pattern-cleanup branch May 10, 2026 19:52
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