d8dfun/bmad-method-template
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

sync: add mandatory pre-execution checklist for screenshots in journey mode #22

Merged
d8dfun merged 1 commits from sync/d8d-skills-20260412-060733 into master 2026-04-12 14:11:42 +08:00
Conversation 0 Commits 1 Files Changed 1 +7
d8dfun commented 2026-04-12 14:08:12 +08:00
Owner
Copy Link

变更内容

Adds a pre-execution checklist at the beginning of the journey execution template to prevent the common mistake:

  • Confusion between browser_snapshot (DOM structure for finding elements) vs browser_take_screenshot (PNG for visual verification)
  • Forces every user step to have browser_take_screenshot + analyze_image visual verification
  • If any rule is violated, execution must stop and fix before continuing

This fixes the issue where only the final screenshot was saved instead of every step during journey verification.

Testing

  • Verified locally that checklist appears before every journey execution
  • Checklist reminds correct tool usage for each step
## 变更内容 Adds a pre-execution checklist at the beginning of the journey execution template to prevent the common mistake: - Confusion between `browser_snapshot` (DOM structure for finding elements) vs `browser_take_screenshot` (PNG for visual verification) - Forces **every user step** to have `browser_take_screenshot` + `analyze_image` visual verification - If any rule is violated, execution must stop and fix before continuing This fixes the issue where only the final screenshot was saved instead of every step during journey verification. ## Testing - [x] Verified locally that checklist appears before every journey execution - [x] Checklist reminds correct tool usage for each step
d8dfun added 1 commit 2026-04-12 14:08:13 +08:00
sync: add mandatory pre-execution checklist for screenshots in journey mode 35e02b8070 
Adds a pre-execution checklist to prevent mistake where:
- browser_snapshot (DOM snapshot) was used instead of browser_take_screenshot (PNG)
- every user step must have browser_take_screenshot for visual verification
- violates any item = interrupt and fix before continuing

This prevents the error where only final screenshot was saved instead of every step.
d8dfun merged commit d4163aecd5 into master 2026-04-12 14:11:42 +08:00
d8dfun deleted branch sync/d8d-skills-20260412-060733 2026-04-12 14:11:43 +08:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
d8dfun
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: d8dfun/bmad-method-template#22
No description provided.
Delete Branch "sync/d8d-skills-20260412-060733"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?