Skip to content

fix: align visual pr and proof workflows - #3

Merged
sunwrobert merged 3 commits into
mainfrom
codex/sync-create-pr-skill
Sep 10, 2026
Merged

fix: align visual pr and proof workflows#3
sunwrobert merged 3 commits into
mainfrom
codex/sync-create-pr-skill

Conversation

@sunwrobert

@sunwrobert sunwrobert commented Sep 10, 2026

Copy link
Copy Markdown
Owner

Align create-pr and ship-with-proof with the updated personal workflow: publish a complete inline visual explanation, embed diagram PNGs when rendering and hosting are available, and keep browser verification optional.

Show me

 compose inline visual explanation
 create HTML explainer
- require diagram PNG hosting and browser verification
+ embed diagram PNGs when rendering and supported hosting are available
+ otherwise publish the inline body and report the limitation
+ prefer API/CLI; do not launch a browser solely for diagram verification
 retain tests, end-to-end proof, thermo review, CI, and approvals

ship-with-proof still prohibits committing generated evidence and still treats missing actual recording/backend proof as a blocked gate. Only explainer-image delivery is optional.

Validation: skill validator, lint/format, and typecheck passed. The unit-test command passed with no tests present. Both updated skills match their personal copies. No generated artifacts or browser verification were needed for this instruction-file change.

@sunwrobert sunwrobert changed the title fix: sync personal create-pr workflow fix: align visual pr and proof workflows Sep 10, 2026
@sunwrobert
sunwrobert merged commit 6124ea1 into main Sep 10, 2026
1 check 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