Skip to content

πŸ“ [docs] pr-body: Door and Blast radius in Risk, before/after in AC proof - #75

Merged
EricTechPro merged 3 commits into
mainfrom
risk-door-blast-radius
Oct 23, 2026
Merged

EricTechPro merged 3 commits into
mainfrom
risk-door-blast-radius

Conversation

@EricTechPro

Copy link
Copy Markdown
Owner

Summary

Folds merge danger and before/after proof into the blocks the PR body already has. No new marker block, and super-board-pr-body.sh is unchanged.

 <!-- sb:risk -->
 ## Risk
 🟒 **Low** · <one line>
+- **Door:** <two-way | one-way> Β· <how to undo>
+- **Blast radius:** <one word> Β· <who breaks>
 <!-- /sb:risk -->
  • writing-standard.md: the Risk rule and example, and the AC proof line now shows before and after
  • run.md: the PR template
  • super-build: fills Door and Blast radius when it opens the PR
  • super-review: checks them against the diff
  • super-qa preamble: names them in its Risk line
  • tests/test_writing_format.py: new test_risk_names_door_and_blast_radius

Evidence

  • Before: the template has no Door or Blast radius, so the new test fails on main
  • After: python3 tests/test_writing_format.py passes 15/15 and bash tests/run-safety.sh passes all 62 suites

Risk

🟒 Low · docs and templates only. Scripts and the marker list are unchanged.

  • Door: two-way Β· reverting the PR restores the old Risk shape
  • Blast radius: PR bodies Β· new board PRs get two more Risk lines

πŸ€– Generated with Claude Code

…roof

- writing-standard.md: Risk adds Door (two-way/one-way) and Blast radius lines; a one-way door is never 🟒
- writing-standard.md: an AC proof line shows before and after (failed on base, passes on head)
- run.md: PR template Risk block carries the two new lines
- super-build opens the PR with them filled; super-review checks them against the diff
- super-qa iteration preamble names them in its Risk line
- tests/test_writing_format.py pins Door and Blast radius in both templates

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more β†’

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account β†’

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us β†’

EricTechPro and others added 2 commits October 20, 2026 16:16
…way door

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@EricTechPro
EricTechPro merged commit a2809b6 into main Oct 23, 2026
13 checks passed
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