Skip to content

feat(cli): add --keep-workspaces flag and improve workspace summary#1070

Merged
christso merged 1 commit intomainfrom
feat/1066-workspace-ux
Apr 13, 2026
Merged

feat(cli): add --keep-workspaces flag and improve workspace summary#1070
christso merged 1 commit intomainfrom
feat/1066-workspace-ux

Conversation

@christso
Copy link
Copy Markdown
Collaborator

Summary

  • Add --keep-workspaces CLI flag to agentv eval run, wiring the existing orchestrator keepWorkspaces plumbing through the CLI
  • Normalize with CLI > YAML config > TS config precedence (same pattern as --verbose)
  • Improve end-of-run workspace summary: show preserved paths for all tests when --keep-workspaces is set, show hint when workspaces used without the flag
  • Fix orchestrator bug: return workspacePath on result for successfully preserved workspaces (previously only failures included the path)

Red/Green UAT

Red (main):

  • --keep-workspacesUnknown arguments error
  • Successful workspace paths never shown (orchestrator bug: workspace kept but path not in result)

Green (branch):

  • --keep-workspaces accepted, workspaces preserved, paths shown for all tests
  • Without --keep-workspaces: failed workspace paths shown + hint Use --keep-workspaces to preserve all workspaces for inspection.
  • No hint when workspaces aren't used at all

Test plan

  • Build passes
  • All 2117 unit tests pass (0 failures)
  • --keep-workspaces appears in --help output
  • Red test: flag rejected on main
  • Green test: flag accepted, workspace paths displayed, hint shown

Closes #1066

🤖 Generated with Claude Code

…1066)

Wire the existing keepWorkspaces orchestrator plumbing through the CLI:
- Add --keep-workspaces flag to `agentv eval run`
- Normalize with CLI > YAML config > TS config precedence
- Show preserved workspace paths for all tests (not just failures)
- Print hint when workspaces are used without --keep-workspaces
- Fix orchestrator bug: return workspacePath on successful preserved workspaces

Closes #1066

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@christso christso merged commit 83c3db7 into main Apr 13, 2026
1 check was pending
@christso christso deleted the feat/1066-workspace-ux branch April 13, 2026 03:08
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.

agentv eval does not clearly surface per-test workspace paths, and --keep-workspaces is undocumented in help

1 participant