improve-wg-setup — Workgraph live mirror

Metadata

Status	done
Assigned	`agent-853`
Agent identity	`f51439356729d112a6c404803d88015d5b44832c6c584c62b96732b63c2b0c7e`
Created	2026-04-27T19:34:03.634067296+00:00
Started	2026-04-27T21:21:40.169454559+00:00
Completed	2026-04-27T21:22:13.908550201+00:00
Tags	`eval-scheduled`
Tokens	14191113 in / 39665 out
Eval score	0.56
└ blocking impact	0.50
└ completeness	0.60
└ coordination overhead	0.60
└ correctness	0.65
└ downstream usability	0.35
└ efficiency	0.45
└ intent fidelity	0.74
└ style adherence	0.70

Description

Per docs/config-ux-design.md §4 (wg setup interactive flow). Three changes:

--scope global|local|both flag — currently 'wg setup' resolves scope ambiguously; make it explicit (default: prompt the user). §4.1.
Surface launcher_history in pickers — model/endpoint prompts should offer prior CLI/TUI invocations as picker entries (per feedback_launcher_history_in_config_ui.md). The picker code in setup.rs currently calls record_use but doesn't read it back. §4.2.
Final summary screen prints the DELTA, not the full Config — show 'Will write 5 keys' + 'Will NOT write (built-in defaults): N more' so users see the file is intentionally short. §4.3.

Files to touch

src/commands/setup.rs — main changes
src/cli.rs — add --scope arg to SetupArgs

Validation

Failing test: setup --scope global writes only to ~/.wg/config.toml
Failing test: launcher_history entries appear in picker
Failing test: summary screen lists keys-to-write count separately from defaults-to-skip count
cargo build + cargo test pass with no regressions
Manual smoke: 'wg setup --scope global --route claude-cli' end-to-end

## Description

Per docs/config-ux-design.md §4 (wg setup interactive flow). Three changes:

1. **--scope global|local|both flag** — currently 'wg setup' resolves scope ambiguously; make it explicit (default: prompt the user). §4.1.

2. **Surface launcher_history in pickers** — model/endpoint prompts should offer prior CLI/TUI invocations as picker entries (per feedback_launcher_history_in_config_ui.md). The picker code in setup.rs currently calls record_use but doesn't read it back. §4.2.

3. **Final summary screen prints the DELTA, not the full Config** — show 'Will write 5 keys' + 'Will NOT write (built-in defaults): N more' so users see the file is intentionally short. §4.3.

## Files to touch
- src/commands/setup.rs — main changes
- src/cli.rs — add --scope arg to SetupArgs

## Validation
- [ ] Failing test: setup --scope global writes only to ~/.wg/config.toml
- [ ] Failing test: launcher_history entries appear in picker
- [ ] Failing test: summary screen lists keys-to-write count separately from defaults-to-skip count
- [ ] cargo build + cargo test pass with no regressions
- [ ] Manual smoke: 'wg setup --scope global --route claude-cli' end-to-end

Depends on

done .assign-improve-wg-setup

Required by

(none)

Log

2026-04-27T19:35:16.405269028+00:00 Lightweight assignment: agent=Careful Programmer (f5143935), exec_mode=full, context_scope=task, reason=Careful Programmer is ideal for correctness-critical setup config work requiring tests and validation; strong track record (0.72 score on 111 tasks).
2026-04-27T19:35:17.292788439+00:00 Spawned by coordinator --executor claude --model opus
2026-04-27T19:35:27.510865577+00:00 Starting work — reading setup.rs and config-ux-design.md
2026-04-27T20:07:28.253265770+00:00 Task marked as failed: Agent exited with code 1
2026-04-27T20:35:50.318872664+00:00 Reset by `wg recover` — reason: API errors during dispatch — second outage of session
2026-04-27T20:36:11.885144230+00:00 Spawned by coordinator --executor claude --model opus
2026-04-27T20:39:07.041812168+00:00 Resuming task — inspecting prior agent's uncommitted WIP
2026-04-27T20:55:41.420790032+00:00 Task unclaimed: agent 'agent-847' (PID 2579358) process exited
2026-04-27T20:55:53.972885344+00:00 Spawned by coordinator --executor claude --model opus
2026-04-27T21:21:37.227742203+00:00 Task unclaimed: agent 'agent-848' (PID 2665767) process exited
2026-04-27T21:21:40.169460680+00:00 Spawned by coordinator --executor claude --model opus
2026-04-27T21:22:03.588676694+00:00 Validated: 93 setup unit tests + 15 integration tests pass; 7 pre-existing failures on main are unrelated (spawn_task / provenance / TUI render)
2026-04-27T21:22:03.631135888+00:00 Manual smoke: --scope global/local/both/invalid/dry-run all behave correctly; summary prints 'Will write N keys' + 'Will NOT write (built-in defaults): M more'
2026-04-27T21:22:03.699842503+00:00 Committed: 8ec6adac1 — pushed to remote
2026-04-27T21:22:13.467227668+00:00 Validated: 93 setup unit tests + 15 integration tests pass; 7 unrelated test failures on main pre-existed and don't touch setup code. Manual smoke confirmed --scope global/local/both/invalid/dry-run all behave per design. Added smoke scenario setup_scope_flag.sh + updated setup_routes_complete_configs.sh for modern .wg path, both owned by improve-wg-setup. Commits d9727c404 + 8ec6adac1 pushed.
2026-04-27T21:22:13.908554980+00:00 Task marked as done
2026-04-27T21:24:27.883834814+00:00 All validation criteria met:
2026-04-27T21:24:27.973073149+00:00 Build note: had to use CARGO_TARGET_DIR=/tmp/wg-build-829 because the dispatcher's reap-targets sweep keeps deleting our worktree's target/ dir (it looks up agent by directory name 'agent-829' but live agent in retry-in-place is 'agent-847', so it considers the worktree dead)
2026-04-27T21:24:28.007668717+00:00 Work already squash-merged to origin/main as commits 66e3952fd + 72110331e (from prior attempt)
2026-04-27T21:24:32.408403770+00:00 Validation: 90 setup unit tests pass; 15 integration_setup_routes tests pass; manual smoke for --scope global/local/both/garbage works; both setup smoke scenarios PASS
2026-04-27T21:24:32.443400965+00:00 Pre-existing test failures (config::tests::test_global_config_path, integration_dual_executor missing fields) were introduced by canonical-wg refactor and are unrelated to this task
2026-04-27T21:24:32.466202054+00:00 Build needed CARGO_TARGET_DIR=/tmp/wg-build-829 because dispatcher reap-targets sweep keeps wiping our target dir (looks up agent by directory name agent-829 but live agent in retry-in-place is agent-847, so worktree appears dead)