Document Brain cleanup, the built-in reviewer, and the Brain HTTP API - #932
Merged
Merged
Conversation
Nightly docs audit. Status Events was the scheduled deep-dive and came back clean, so this is diff-driven work from d561a3b..9319fca. - automations.tsx: `brain_delete_events` was missing from the job-agent Brain tool list, which claims parity with standard agents; and the Brains-tab paragraph never mentioned that entries can be deleted at all. Documents per-card delete, the per-section Delete all scoped to the selected collection or the whole project, Clear collection, and Clear project — plus that the confirmation quotes scope totals rather than the capped/filtered rows on screen. - tools.tsx: events are append-only but not permanent — `brain_delete_events` prunes by id or by filter within a collection, with a `dryRun` count first. The Brains tab is for cleanup, not just inspection. - 03-api-spec.md: adds the Brain section (15 routes, all read/delete — writes go through MCP), including the bulk delete-by-entry-type rules (`collection` XOR `allCollections`, single-value keys, cascade). - 03-api-spec.md: `note` on POST /agents/:id/launch-review, and GET /personas now lists built-ins after the repo's own. - README / CLAUDE.md / AGENTS.md: personas no longer require repo setup. - tips.ts: ambient tip for the built-in reviewer, anchored to the new personas#built-in-reviewer heading. Verified unchanged: the Status Events section holds on every claim — the dispatch_event enum, the meanings injected by buildStartupRules, the activity monitor's 3-minute stale window and its two correction messages, the sidebar card's label/timestamp/repo line, the Activity page charts, and the done/waiting_user/blocked notification set. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Uh oh!
There was an error while loading. Please reload this page.
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 freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Nightly docs audit (
d561a3b1..9319fcaa).Deep-dive: Status Events — clean, no changes. Verified every claim in
docs-sections/events.tsxagainst source: thedispatch_eventenum (agent-lifecycle-tools.ts), the event-type meanings injected at launch (buildLaunchGuidanceintmux/command-builder.ts), the activity monitor's 3-minute stale window and its two correction messages plus the conditional-write rule (activity-monitor.ts,writeLatestEventIfCurrent), the sidebar card's label/timestamp/repo line and expanded message (agent-card-status.tsx), the Activity page's heatmap / active-hours / working-time / status-breakdown surfaces (activity-pane.tsx), and thedone/waiting_user/blockednotification set (slack.ts,web-notifications.ts).Diff-driven fixes:
docs-sections/automations.tsx—brain_delete_events(Add brain_delete_events MCP tool #928) was missing from the job-agent Brain tool list, which claims parity with standard agents. The Brains-tab paragraph also never mentioned deletion at all: now covers per-card delete, the per-section Delete all scoped to the selected collection or the whole project (Add bulk delete by entry type to the Brain UI #929), Clear collection, Clear project, and that confirmations quote real scope totals rather than the capped/filtered rows on screen.docs-sections/tools.tsx— events are append-only but not permanent:brain_delete_eventsprunes by id or by filter within a collection,dryRunpreviews the count. The Brains tab is for cleanup, not just inspection.docs/03-api-spec.md— new Brain (Shared Memory) section: all 15 routes (read/delete only; writes go through MCP), plus the bulk delete-by-entry-type rules (collectionXORallCollections, single-value query keys, list-item cascade). Closes a long-standing backlog item.docs/03-api-spec.md—notefield onPOST /agents/:id/launch-review(Ship a built-in generic review persona and a launch focus note #923: 2,000 chars, collapsed to one line, quote/marker-stripped), andGET /personasnow appends built-ins after the repo's own.README.md,CLAUDE.md,AGENTS.md— persona review no longer requires repo setup; the built-incode-reviewgeneralist ships with Dispatch.lib/tips/tips.ts— new ambient tip "Review With No Setup" (since 0.33.6) deep-linking the newpersonas#built-in-revieweranchor.Deferred to a future run: whether to add a caveat that Codex resume (#927) recovers its session id from
~/.codexrollout logs, so pruned logs or a different machine silently start a fresh conversation — filed in the audit backlog.Next run's focus: Worktrees section deep-dive (shortest core section, untouched since #798).
🤖 Generated with Claude Code