Created
Jul 06, 16:55
Started
Jul 06, 16:56
Completed
Jul 06, 21:05
DevOps handoff
Type
Feature
Shape
ui-only
Worktree Slug
blocked-card-crew-layout
Repositories
mcritchie-studio
Release Slug
—
Branch
feat/blocked-card-crew-layout
Acceptance Criteria
Expected Test Plan
Checks Run
Agent Context
Operator request (screenshots): on a BLOCKED card show design pokemon (slot 1), building pokemon (slot 2), and the blocking agent + red X (slot 3). Root cause: crew_columns routes blocked cards to blocked_build_step_columns ONLY on the Build board (board==:build). The Deploy board (/deployments, board==:deploy — main production board) instead hits blocked_build_crew which fills slot 1 with the MASCOT (not the blocker) and leaves review/assembled empty => mascot+X + two dashed circles (matches operator screenshot). Fix: route ALL blocked tasks through blocked_build_step_columns (drop the board==:build guard on line 213), remove now-dead blocked_build_crew + its line-217 call, rename to a board-agnostic name, and update the partial/helper comments + the existing test at stage_agents_helper_test.rb:440 which asserts the OLD deploy behavior (build/review/assembled lanes).
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.
Conversation
QA review feedback, agent handoffs, and follow-up notes for this task.
Scout report: merge-ready - LIGHT read: all 3 acceptance criteria hold; blocked_build_crew fully dead; full-suite+rubocop certified to PR tree (dor-check STALE is a false positive from running on main). Reporter: carl Outcome: merge-ready Findings: - blocked_stage_columns body is byte-identical to old blocked_build_step_columns (only comment+def renamed) — build board blocked layout unchanged (acceptance #3). - Deploy board blocked path now returns designed/building/blocked; partial threads agents into crew_columns so slot 3 resolves the REAL blocker, not the mascot (acceptance #1/#2). - git grep on PR branch: zero blocked_build_crew and zero blocked_build_step_columns callers; rename clean; comments all updated. Questions: - none Checks: - cert fingerprint aa16cc32 == PR HEAD tree; full-suite + rubocop green; CI 4/4 green
Scout report: merge-ready - UI verified: blocked deploy card renders design·building·blocker with red X on slot 3; build board layout byte-identical (renamed method, dead blocked_build_crew fully removed); dor-check green from worktree Reporter: shannon Outcome: merge-ready Findings: - crew_columns drops the board==:build guard so BOTH boards route blocked -> blocked_stage_columns (renamed from blocked_build_step_columns, body unchanged) => build board unchanged, deploy board now design+building mascot + real blocker slot 3 - blocked_build_crew and its by_lane[:build] call fully removed; git grep confirms zero remaining references to old names - dor-check STALE from primary was a false positive (primary on main); recorded cert fingerprint aa16cc32 == branch tree hash exactly; dor-check PASSES from the worktree Questions: - none Checks: - unit: crew_columns blocked deploy -> %i[designed building blocked], slots 1/2 mascot, slot 3 = real blocker (avi), durations 420/2160 - component: rendered blocked card = 3 crew-cluster, 0 crew-empty, crew-blocked red X present
Avi review approved; ready for Steffon's qa-release sweep.
Sealed-bid sizing
Edit →Alex (PM)
—
Avi (PO)
SMALL
Dev
SMALL
Actual
XL
We emailed a one-tap sign-in link to . It expires shortly and can only be used once.
No email? Check spam, or close this and try again.