Skip to content

Commit f2419b2

Browse files
committed
fix: use tui_read_screen text in PR comments, add verification section to SOP
1 parent 3da9f34 commit f2419b2

1 file changed

Lines changed: 19 additions & 2 deletions

File tree

.github/agent-sops/task-tester.sop.md

Lines changed: 19 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,15 +66,32 @@ Post a single PR comment:
6666
**Expected:** what should have happened **Actual:** what happened
6767

6868
<details>
69-
<summary>Screenshot</summary>
69+
<summary>Terminal output</summary>
70+
```
71+
72+
(paste tui_read_screen text output here)
7073

71-
(paste screen text here)
74+
```
7275
7376
</details>
7477
```
7578

7679
If all flows pass, omit the Failed section.
7780

81+
For failures, use `tui_read_screen` to capture the terminal text and paste it in the comment. SVG screenshots are
82+
uploaded as workflow artifacts separately — do not try to embed them in the comment.
83+
84+
## Verification
85+
86+
After each flow completes, verify the side effects — not just the TUI output:
87+
88+
- If a project was created: use `shell` to check the directory exists and contains expected files (e.g.
89+
`agentcore.json`)
90+
- If a resource was added: use `shell` to check the config file was updated
91+
- If a command produced output: verify the output matches expectations
92+
93+
Do not rely solely on what the TUI displays. Confirm the CLI actually did what it claimed.
94+
7895
## Forbidden Actions
7996

8097
- Do NOT modify, create, or delete source files

0 commit comments

Comments
 (0)