Commit 26a3dca
committed
fix(stderr): route diagnostics to stderr, keep stdout for data
`vers run --json` previously emitted 'Warning: vers.toml not found...' to
stdout, contaminating the JSON output and breaking `vers run --json | jq`.
Same shape elsewhere \u2014 six diagnostics across runconfig, presenters, and
handlers were using `fmt.Println`/`Printf` (stdout) instead of stderr.
Fixed:
- internal/runconfig/config.go 'vers.toml not found' warning
- internal/presenters/run_presenter.go '.vers directory not found' warning
- internal/presenters/branch_presenter.go 'no VM IDs returned' error + HEAD update warning
- internal/handlers/upgrade.go 'skipping checksum verification' warning
- internal/handlers/kill.go 'about to delete VM' confirmation header
After this, `vers run --json 2>/dev/null` produces clean JSON that
`jq` can parse. Addresses agent-native CLI principle 2 (data on stdout,
diagnostics on stderr).1 parent e9ff14c commit 26a3dca
5 files changed
Lines changed: 9 additions & 6 deletions
File tree
- internal
- handlers
- presenters
- runconfig
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
3 | 3 | | |
4 | 4 | | |
5 | 5 | | |
| 6 | + | |
6 | 7 | | |
7 | 8 | | |
8 | 9 | | |
| |||
30 | 31 | | |
31 | 32 | | |
32 | 33 | | |
33 | | - | |
| 34 | + | |
34 | 35 | | |
35 | 36 | | |
36 | 37 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
107 | 107 | | |
108 | 108 | | |
109 | 109 | | |
110 | | - | |
| 110 | + | |
111 | 111 | | |
112 | 112 | | |
113 | 113 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
| 5 | + | |
5 | 6 | | |
6 | 7 | | |
7 | 8 | | |
| |||
12 | 13 | | |
13 | 14 | | |
14 | 15 | | |
15 | | - | |
| 16 | + | |
16 | 17 | | |
17 | 18 | | |
18 | 19 | | |
| |||
50 | 51 | | |
51 | 52 | | |
52 | 53 | | |
53 | | - | |
| 54 | + | |
54 | 55 | | |
55 | 56 | | |
56 | 57 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
| 5 | + | |
5 | 6 | | |
6 | 7 | | |
7 | 8 | | |
| |||
18 | 19 | | |
19 | 20 | | |
20 | 21 | | |
21 | | - | |
| 22 | + | |
22 | 23 | | |
23 | 24 | | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
55 | 55 | | |
56 | 56 | | |
57 | 57 | | |
58 | | - | |
| 58 | + | |
59 | 59 | | |
60 | 60 | | |
61 | 61 | | |
| |||
0 commit comments