McRitchie Studio

Agent Context

Session retro improvement #3. Two routing misses this session: (a) reviewer-select picked carl to review the PR carl BUILT because task built_by was unset — I passed --builder carl by hand every time; (b) it assigned agents mid-build/mid-review on OTHER in-flight tasks (jasper while building geo; carl while reworking the conductor PR). FIX (a): when a task moves to 'building' via bin/task, stamp metadata.devops.built_by with the acting agent (the session/persona doing the build); reviewer-select (app/services/reviewer_selector.rb + bin/reviewer-select) should read task built_by and auto-exclude it by default (it already has a --builder OVERRIDE — make the recorded built_by the default source). FIX (b): give reviewer-select a --busy a,b,c flag (and/or have it query the board for agents currently on in-flight tasks: stage=building/claimed) and exclude them, but NEVER shrink the candidate pool below what's needed to pick a pair (keep-rather-than-starve, like the existing builder-keep logic). DOCS: devops-cycle-design.md reviewer-select section + the parallel-agent-devops module.