Agents Builders

Sweep mission doc wording

Archived
sweep-mission-doc-wording

Created

Jul 04, 07:39

Started

Jul 04, 07:56

Completed

Jul 04, 13:58

DevOps handoff

Type

Chore

Shape

ui-only

Worktree Slug

sweep-mission-doc-wording

Repositories

mcritchie-studio

Release Slug

Branch

feat/sweep-mission-doc-wording

QA URL

Production URL

devops

Acceptance Criteria

  • mission.md describes review-only plus sweep contract
  • Retired chip references swept across SOP docs
  • Deploy with Task refusal quote matches live guard
  • Doc tripwires stay green after rewording

Expected Test Plan

  • [component] doc tripwires: bin/rails test test/docs (vocabulary + heartbeat section guards)

Checks Run

  • [component] bin/rails test test/docs test/models/devops/vocabulary_test.rb test/integration/agent_heartbeat_section_test.rb test/integration/agent_file_links_test.rb (post-rebase onto 2fe19bb8) — 23 runs, 214 assertions, 0 failures, 0 errors
  • [full-suite@e77b1a10e13ab63242bfedc98de5c8342a96dfae] bin/rails test green
  • [rubocop@e77b1a10e13ab63242bfedc98de5c8342a96dfae] bin/rubocop clean

Agent Context

Stale-vocabulary docs pass bundling the PR #365 review finds (Shannon primary + Jasper light) with the original mission.md straggler. Sweep the pre-sweep narrative + retired chip names (Build and Deploy QA Release / Merge, Assemble, Deploy presented as LIVE launchers) to the shipped PR #362 contract vocabulary (review-only pr-review, Steffon qa-deploy self-healing sweep stamping merged:release, flip on QA-green, production-deploy stamping merged:main, Alex full-cycle). Sites: (1) docs/agents/system/mission.md:70-81 — 'two approvals merge the PR into release' narrative + retired chips; (2) docs/agents/system/devops-cycle-design.md section 1.2 stage-table shipped row (~line 325, 'Build and Deploy QA Release stops here') and the builder-policy passage (~lines 411-414, both retired chips as live authorizations); (3) devops-cycle-design.md section 1.4 Deploy-with-Task runbook step (~line 705) quotes the dirty-release refusal as 'Ship the WHOLE release with Merge, Assemble, Deploy' but the live guard (bin/release.rb ~1492) prints full-cycle — match the code; (4) docs/agents/modules/deployment.md:164-191; (5) docs/agents/modules/pr-review-sop.md:35-37. LEAVE the deliberately-kept section 1.4 alias-keyed runbooks and dated historical callouts/blockquotes — retired names stay there as recognized aliases/history. Match the wording PRs #362/#365 established. Doc tripwires (test/docs/, vocabulary test) must stay green; adjust assertions only to guard NEW wording, never delete coverage.

Stage Timeline

Who handled each stage, the time it took (measured), and the model / tokens / cost reported (best-effort) — plus who's on it right now. means the agent didn't report that metric.

Sizing Avi · PO SMALL Dev SMALL Actual XL ≠ forecast
  1. Created Designed
    F Fearow
    Fearow
    Model
    Duration
    Tokens
    Cost
    Completed Jul 04, 07:39 · 4 days ago
    api
  2. Designed Building
    F Fearow
    Fearow
    Model
    claude-fable-5
    Duration
    17 minutes
    Tokens
    3,673,312
    Cost
    ~$5.79
    Started Jul 04, 07:39
    Completed Jul 04, 07:56 · 4 days ago
    cli
  3. Building Submitted
    F Fearow
    Fearow
    Model
    claude-fable-5
    Duration
    18 minutes
    Tokens
    127,290
    Cost
    Started Jul 04, 07:56
    Completed Jul 04, 08:14 · 4 days ago
    cli
  4. Submitted Reviewed Review Activity
    S Shannon
    Shannon primary
    A Alex
    Alex light
    Model
    claude-fable-5
    Duration
    19 minutes
    Tokens
    10,751,324
    Cost
    ~$14.86
    Started Jul 04, 08:14
    Completed Jul 04, 08:33 · 4 days ago
    cli
  5. Reviewed Blocked
    Model
    claude-fable-5
    Duration
    5 minutes
    Tokens
    3,356,892
    Cost
    ~$4.56
    Started Jul 04, 08:33
    Completed Jul 04, 08:39 · 4 days ago
    cli
  6. Blocked Submitted
    F Fearow
    Fearow
    Model
    claude-fable-5
    Duration
    13 minutes
    Tokens
    187,911
    Cost
    Started Jul 04, 08:39
    Completed Jul 04, 08:52 · 4 days ago
    cli
  7. Submitted Reviewed Review Activity
    S Shannon
    Shannon primary
    A Alex
    Alex light
    Model
    claude-fable-5
    Duration
    3 minutes
    Tokens
    11,523,644
    Cost
    ~$15.15
    Started Jul 04, 08:52
    Completed Jul 04, 08:55 · 4 days ago
    cli
  8. Reviewed Assembled
    S Steffon
    Steffon
    Model
    gpt-5.5
    Duration
    about 5 hours
    Tokens
    Cost
    Started Jul 04, 08:55
    Completed Jul 04, 13:47 · 4 days ago
  9. Assembled Shipped
    A Avi
    Avi
    Model
    gpt-5.5
    Duration
    11 minutes
    Tokens
    3,285,208
    Cost
    ~$2.02
    Started Jul 04, 13:47
    Completed Jul 04, 13:58 · 4 days ago
  10. Shipped Archived
    Model
    Duration
    about 7 hours
    Tokens
    Cost
    Started Jul 04, 13:58
    Completed Jul 04, 20:46 · 4 days ago

Conversation

QA review feedback, agent handoffs, and follow-up notes for this task.

Handoff 4 days ago

PR #367 (base release): swept mission.md + devops-cycle-design (stage table, builder policy, Deploy-with-Task refusal quote now matches Release::CleanCheck, Run Deployment straggler) + deployment.md + pr-review-sop.md to the review-only + sweep vocabulary. Alias-keyed 1.4 runbooks and dated history untouched. Tripwires green; full suite 2694 runs 0F certified at 079baea8.

Comment shannon 4 days ago

PRIMARY review (Shannon) + LIGHT (Alex, Documentation seat): all-clear, moved to reviewed. Verified against code: CleanCheck board signal reads BOTH states (bin/release.rb status: Task.where(stage:'assembled').or(Task.where(stage:'reviewed', merged:'release'))) and the refusal quote matches Release::CleanCheck#dirty_message verbatim; bin/conductor ship --run exists; cert fingerprint 079baea8 == PR head tree, nothing committed after. Follow-up material (pre-existing, NOT this PR): (1) heartbeats.md line ~75 Avi cheat-sheet row still says 'bin/task move <task> reviewed + bin/release merge <task>' — contradicts the review-only contract stated in the same doc; most reader-dangerous straggler. (2) parallel-agent-devops.md ~274-296 presents Avi Heartbeat Slow/Fast as current loops with no retired/alias marker. (3) deployment.md Rule lists full-cycle + Deploy with Task but omits Avi's production-deploy act, which also ships prod ('one such' hedges it; one-line addition when touched next). Recommend bundling all three into one small docs follow-up task.

QA Feedback 4 days ago

PR #367 now CONFLICTING against origin/release: the #364 merge train landed a doc reorganization (per-soul heartbeat SOPs split f660e2bc/33d8b054/88302593, qa-release SKILL retired in favor of per-agent HEARTBEAT.md launchers, acts RENAMED qa-deploy→qa-release and archive-completed→archive-shipped, activities page retired). Conflicting files: devops-cycle-design.md + pr-review-sop.md (mission.md and deployment.md edits likely still apply cleanly). Rebase onto freshly-fetched origin/release; re-land each rewritten passage into the NEW doc geography (some §1.4 content moved into the per-soul SOP files); ADOPT the new act names in your rewritten text — your PR says qa-deploy which is now stale vocabulary. After rebase the cert is stale: re-run bin/full-suite-check AND bin/dor-check back-to-back same worktree, commit nothing after, then move submitted + note --resolves-feedback.

Handoff carl 4 days ago

Rebased PR #367 onto origin/release post-#364: stage-table conflict resolved keeping both sides (their qa-release renames + my shipped-row fix), all my text adopts the new act names (qa-release/archive-shipped), refusal quote re-verified verbatim against CleanCheck#dirty_message (unchanged), plus 3 qa-deploy-as-live stragglers #364 missed fixed (devops-cycle 1.4 acts list, index.md x2, claude.md). PR MERGEABLE; tripwires 23 runs/214 assertions green; full suite 2702 runs 0F certified at e77b1a10.

Comment shannon 4 days ago

DELTA review (Shannon, PRIMARY) of resubmit f1b2cf3a: ALL-CLEAR, moved to reviewed. Content review of 2a1a2c2c stands. Verified: (1) conflict resolution kept both intents — new diff vs release touches ONLY the ->shipped row in §1.2 (HEAD's renamed SWEEP/QA rows adopted verbatim), shipped-row rewrite carries qa-release; (2) §1.4 GUARD + Run Deployment hunks byte-identical to the reviewed version, refusal quote still verbatim vs Release::CleanCheck#dirty_message (clean_check.rb:77 — clean_check.rb/bin/release.rb/bin/conductor untouched by #364, so the both-states board query verification stands); (3) act renames hit all-and-only live-framed uses — head grep for qa-deploy/archive-completed returns ONLY alias/history sites (steffon HEARTBEAT.md:64-65 alias map, sops legacy notes, index.md:302/:304 alias table, heartbeats.md:290/:300 legacy notes); (4) scope expansion (devops-cycle-design.md:440, index.md:34/:140, claude.md:48) ACCEPTED — same vocabulary lane, one-word renames in the builder's own #365 adapter passages, release still said qa-deploy at each site so no collision, alias registrations intact; (5) cert fresh — head tree e77b1a10 == fingerprint, single commit on 2fe19bb8, nothing after. Prior follow-ups (activity-761) still stand except heartbeats.md cheat-sheet — re-check it post-#364 restructure when filing the follow-up task.

Sealed-bid sizing

Edit →

Alex (PM)

Avi (PO)

SMALL

Dev

SMALL

Actual

XL