Repository navigation
Supervisor threads: follow-ups - #219
Conversation
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…s, and a failed help is never kept Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…r it was on main Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…the file uses, and the port layout says the real rule Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… soon as the page's data says so Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…and is kept so the lens stays read-only Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… per process, so it never hit Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… each line is Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
CI note: 🤖 Generated with Claude Code |
|
Reviewed at 665ac99:
Locally on Node 24, the kit tests pass 82 of 82 and the dashboard tests 144 of 144. On CI, kit (22, rest) failed once, in the focus tests. Its rerun passed, and so did the same tests locally, so it was the known load flake. CI now passes 9 of 9. Merging. |
Small follow-ups left after supervisor threads (#185/#194/#205/#215). The plan and the spec were reviewed adversarially by Fable, and so was this diff.
Changes
git log --raw -t --no-renames). Before this, any tree with anartifact.jsoncould be served under any slug. Only a yes is remembered. Spec §8 is updated.drawOrHoldredraws the open card. The test covers the stale-read race with held reads, not a timer.run/supervisor-pane-rolealong withrun/supervisor-pane./api/viewgets a test for the 500 it answers when describe fails (a folder it can't read; skipped as root, and CI doesn't run as root).test/lib/ports.mjsnow states the real rule for scans and the layout for each file.Considered and dropped after review
DEVTEAM_PORT_SPANsetting. Those tests pin their port, so the hub never scans.kitHasRecovermemo. Every caller asks once per repo, so it would never hit. Commit 2ad06ed adds it and the next commit reverts it.Tests (Node 24.21.0, targeted):
test/machine.test.mjsandtest/recovery.test.mjstest/gateway.test.mjs: 67/67🤖 Generated with Claude Code