feat: save SVG screenshots as artifacts on every TUI test flow

Author: Hweinstock

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

@@ -35,13 +35,16 @@ For each flow in the test spec:
 3. Follow the flow steps: use `tui_action` (preferred — combines send + wait + read in one call) or `tui_wait_for` +
    `tui_send_keys` for multi-step interactions
 4. Verify each expectation against the screen content
-5. On **pass**: record the flow name as passed
-6. On **failure**: use `tui_screenshot` to capture the terminal state, record the flow name, expected behavior, actual
-   behavior, and the screenshot text
+5. On **pass**: use `tui_screenshot` with `format: "svg"` and `savePath: "/tmp/tui-screenshots/<flow-name>-pass.svg"` to
+   capture the final state, then record the flow name as passed
+6. On **failure**: use `tui_screenshot` with `format: "svg"` and `savePath: "/tmp/tui-screenshots/<flow-name>-fail.svg"`
+   to capture the terminal state, also take a text screenshot for the PR comment, record the flow name, expected
+   behavior, actual behavior, and the screenshot text
 7. Always `tui_close` the session when done, even on failure
 
 **Constraints:**
 
+- Create `/tmp/tui-screenshots/` at the start before running any flows
 - Use `timeoutMs: 10000` (10 seconds) minimum for all `tui_wait_for` and `tui_action` pattern waits
 - Use small terminal dimensions: `cols: 100, rows: 24`
 - If a wait times out, retry once before declaring failure

.github/workflows/strands-command.yml

@@ -141,6 +141,14 @@ jobs:
           S3_SESSION_BUCKET: ${{ secrets.AGENT_SESSIONS_BUCKET }}
           BRANCH_NAME: ${{ steps.process-inputs.outputs.branch_name }}
 
+      - name: Upload TUI screenshots
+        if: always() && steps.process-inputs.outputs.mode == 'tester'
+        uses: actions/upload-artifact@v4
+        with:
+          name: tui-screenshots
+          path: /tmp/tui-screenshots/
+          if-no-files-found: ignore
+
       - name: Remove strands-running label
         if: always()
         uses: actions/github-script@v8