Skip to content

fix(templates): make review projections scenario-first - #355

Merged
johnlanni merged 2 commits into
mainfrom
codex/scenario-first-human-review-projections
Jul 28, 2026
Merged

fix(templates): make review projections scenario-first#355
johnlanni merged 2 commits into
mainfrom
codex/scenario-first-human-review-projections

Conversation

@johnlanni

@johnlanni johnlanni commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • define decision integrity as the hard constraint, then optimize content, wording, order, interaction, and visual form for the least cognitive effort needed to reach the right decision
  • make human-review projections lead with the affected person/operator, their goal, and the observable outcome before technical structure
  • require one concrete trigger-to-outcome walkthrough plus a meaningful failure or boundary case
  • map each case step to what the person sees, what the system does, and what the reviewer should verify
  • keep component details, artifact IDs, and DAGs as progressive technical explanations of the case
  • update Proposal, Design, and Implement recipes, the HTML skeleton, issue scaffolds, generated assets, and focused assertions

The guidance forbids simplifying away material facts, uncertainty, tradeoffs, risks, alternatives, or decision consequences. This preserves the coverage-complete, statusless, non-authoritative projection contract from #350.

Testing

  • go test ./internal/templates ./internal/commands -count=1
  • git diff --check
  • forward-tested the generated Design guidance against an idempotent-webhook Design; the result led with the operator retry case and normal/retry/crash walkthrough before architecture
  • forward-tested a Proposal with ambiguous writes and competing stakeholder preferences; the result remained easy to follow while preserving the split evidence, sample weakness, credible options, and unresolved decision

Signed-off-by: zty98751 <zty98751@alibaba-inc.com>
Signed-off-by: zty98751 <zty98751@alibaba-inc.com>
@johnlanni
johnlanni merged commit 7ee8474 into main Jul 28, 2026
6 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