Repository navigation
Docs: rewrite board.md and split out a Dispatch page (T297) - #224
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This branch was successfully deployed
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.
Task A of the docs cleanup (T297).
board.mdis rewritten under the agreed headings. The Agents section moves to a newdispatch.md, rewritten rather than pasted. The new page sits in the sidebar after Board.Words
site/src/content/docs/docs/board.mdsite/src/content/docs/docs/dispatch.md(new)Both pages pass a length check: no paragraph over 60 words, no sentence over 25.
Other files
site/astro.config.mjs: sidebar Dispatch after Board.site/src/lib/agent-docs.mjs:dispatchadded toSIDEBAR_SLUGS, sollms-full.txtincludes the page.site/test/agent-surface.test.mjs:Dispatchadded toSIDEBAR_TITLES. Task E may also edit this file; my hunk is one line.tests/docs_parity.rs: palette test only. It learns a new condition,A task list has focus. "Always" rows must now also be offered over an open task page, so the old table (group/sort/fold under Always) fails this test. Task C edits the keys functions in this file; the hunks are separate.Moved facts
#board,#cards,#esc-and-quit)#blocked#configuration#starter-tasksReal task and PR numbers (T169/T203/T204, #190/#193/#194), repo paths, store field names (
past_reviews), gh flags and history are gone. Examples use T12/T202/T205 and PR #42. One name for each thing: Projects view, and workspace for what cleanup closes. The quoted cleanup card says "agent pane", so the page explains that once.Corrected claims
ron another row saysnot blockedsrc/ui/board/apply.rs:1388not blocked or in reviewsrc/ui/board/commands.rs:135,model.rs:6021ssrfindsset status: reviewcommands.rs:224subsequence match;ssrfirst matchesset status: readyrev, which matches only reviewunassigned(Mouse)apply.rs:2665-2697unassignedlistedctrl+csrc/ui/input.rs:2436,:2553,:2663ctrl+cquits from the board, a task page, and Helpsrc/app.rs:4527-4564(refused witha bulk dispatch is still running …)src/ui/board/draw.rs:357-375Dispatch N tasks?(N = launches),Start N tasks?when nothing launchescore.longpathsoff: dispatch says so oncesrc/dispatch.rs:3647-3651(per dispatch)src/agents.rs:409-462(refused in the pane after the workspace opens)model.rs:4266-4272[board]defaults apply to every viewmodel.rs:6816-6829@nameviews start at none/recentmodel.rs:3641-3644src/ui/queue.rs:1131-1158model.rs:5317-5318(the task is left out of the list)src/ui/queue.rs:1206-1207,:1273(ready sorts oldest pick first; open keeps capture order)model.rs:4180-4191@nameview searches tasksSuspected bugs (code not changed)
src/dispatch.rs:3641,run_in_paneerror returns without removing them)..cmd/.batrecords the dispatch and sets started before the pane refuses. The task shows◉with no agent (src/agents.rs:409-462).□(elsewhere) glyph (src/ui/render.rs:1337-1339).ctrl+b/ctrl+rrefuse a done task. In a marked set, done tasks go into the block or review card with the rest (apply.rs:2862,:2895).Inbound links for other tasks
These pages link old board.md anchors that no longer exist. They belong to other tasks, so I didn't touch them:
/docs/board/#base-branch/docs/dispatch/#base-branch#block-with-a-reason,#reply-to-a-running-agent/docs/board/#block-review-reply#dispatch/docs/dispatch/#dispatch#dispatch-on-windows-preview/docs/dispatch/#windows-preview#agents/docs/dispatch/#complete-and-clean-up/docs/dispatch/#complete-and-clean-up#wide-stage-slider/docs/board/#wide-layout#archive/docs/board/#done-drawer-and-archive#merge-the-pull-request/docs/dispatch/#merge-the-pull-request#base-branch,#dispatch(×2),#relaunch/docs/dispatch/#reply-to-a-running-agent,#review-with-what-was-done/docs/board/#block-review-reply#status/docs/board/#sections-and-status#after,#group-and-sort,#save-failuresand#delete-and-undostill resolve. The AGENTS.md product-reference row should namedispatch.md(task F). The llms.txt docs list needs the Dispatch page (task E).These links in board.md and dispatch.md point only at agreed headings on other pages, e.g. keys
#cardsand#esc-and-quit, storage#configurationand#starter-tasks. Those anchors appear once task C and task D land.Checks
cargo fmt --check && cargo clippy --all-targets -- -D warnings && cargo test: green, 2150 passed.cargo test --test integration -- docs_parity:: cli_discovery::: 5 passed. Restoring the old Always row makes the palette test fail ("group by…" is documented as always available but vanishes over a task page).cd site && npm ci && npm test && npm run build: 52 passed, build clean.src/not touched, so no Windows clippy.npm run test:paritynot run (task G owns the port). No live Herdr smoke: docs only.