happy paths after the senior-dev wave: telemetry --help exits 0, --help's six groups in the manual, wall tile's first activity line - #1533
Merged
Conversation
`codeaf telemetry --help` took the flag for a sixth verb and left with 1, and each verb's own --help answered Go's `flag: help requested` and 1, while the manual says asking for help is never a failure. The verbs now parse through the binary's shared help seam and the group answers the gesture with its usage line. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
#1488 added a Hand it a whole task heading for the programs a build carries, so the page that said five headings was wrong on every build that carries senior-dev. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
The wall's first reading of a conversation that had already finished took no new lines, so it set no fresh time, and the tile drew its spend beside a blank activity line. The manual promises one dim line saying what the conversation is doing now or when it last moved. The first reading now takes the transcript's own last write, read off the loop beside its contents, and later readings age from there. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
A happy-path pass over dev
c34a3a76d, after the merge wave that brought in senior-dev (#1488), the teams manager and wall (#1429, #1494, #1514, #1516), the one task page (#1485), the crew (#1436, #1518) and the first-run names (#1519). The real binary was driven in tmux against a real model across 40 documented flows: core chat on both roads, senior-dev, tasks and the worker harness, teams and the manager, and the shell doors and places. These three things did not match the manual:codeaf telemetry --helpexited 1. The group took--helpfor a sixth verb, and each verb's own--helpprinted Go'sflag: help requestedand exited 1. The manual says--helpon any verb exits 0. The verbs now go through the binary's shared help seam.TestAskingForHelpIsNotAFailurecovers the group and all five verbs, and a new test proves thatoff --helpchanges nothing.--helphas printed six groups since senior-dev: a coding agent built into codeaf takes a whole task, in the folder itself #1488, but the manual still said five. The page now names all six headings in order and says that a build with no carried program prints five.updated 5m ago". The first read now takes the transcript's last write time.TestWallFirstReadShowsConversationFileActivityfails on the old code and passes with the fix.Everything else in the pass held: first run, a reply, a tool edit, slash commands, the places bar, Home,
/new, quit and resume,/rewindand--no-host; senior-dev from the shell, in a plain folder, from the chat, and stopped;/tasklanding, the task room, a proposal's countdown,/stop,/crewand reopening after a restart; teams, the manager, delegation with Traffic,@mentions, closing and reopening a team, and a restart;exec --json, the belt's shell hands,connect,update --check, every place on a seeded home, and a fresh first run.Not in this PR:
memory is offon the default engine road. This predates the wave (it is in v0.4.1), filed as memory place says 'memory is off' on the default engine road while memory is on #1535.endedcard wording are senior-dev: the side row hides its clock while its page is open, and the manual says what the row and card show #1534.Local gates:
make test-quickgreen;make test-lawsgreen; focusedcmd/codeaf,internal/tui3andinternal/manualtests green.make pr-readyhitcmd/codeaf's 15-minute package ceiling while three other test runs loaded the box. The test running at the deadline had been going for 6 seconds, so this was load, not a hang, and CI's touched-packages run is the check.🤖 Generated with Claude Code