McRitchie Studio
Say Hi 👋
Agents Builders
← All Tasks
Sizing

Reviewer roles primary light

Shipped
reviewer-roles-primary-light

Created

Jun 26, 22:28

Started

Jun 27, 00:41

Completed

Jun 27, 01:22

DevOps handoff

Type

Chore

Shape

backend

Worktree Slug

reviewer-roles-primary-light

Repositories

mcritchie-studio

Release Train

—

Branch

feat/reviewer-roles-primary-light

Pull Request

https://github.com/amcritchie/mcritchie-studio/pull/251

Local URL

—

QA URL

—

Production URL

—

devops,agent-ops

Acceptance Criteria

  • reviewer-select outputs primary and light roles
  • Recorded review intents use primary not heavy
  • Avatars UI and docs reflect primary light
  • Vocabulary reviewer_roles is the single source

Expected Test Plan

  • [unit] ReviewerSelector returns primary+light; Vocabulary.reviewer_roles is the single source; StageAgent legacy heavy treated as primary
  • [integration] reviewer-select records a primary review intent; timeline renders a legacy heavy record as Primary

Checks Run

  • [unit] bin/rails test test/services/reviewer_selector_test.rb test/models/devops/vocabulary_test.rb test/helpers/stage_agents_helper_test.rb — primary+light roles, vocab single-source, legacy heavy→primary
  • [integration] bin/rails test test/views/consolidated_timeline_test.rb test/lib/reviewer_select_test.rb test/controllers/tasks_controller_test.rb — recorded intent uses primary; legacy heavy renders Primary
  • [full-suite@b9fc008a9e77c50e54814ae6b99e261ec2202712] bin/rails test green
  • [rubocop@b9fc008a9e77c50e54814ae6b99e261ec2202712] bin/rubocop clean

Agent Context

Adopted 2026-06-26 with the DevOps SOP single-source work (config/devops_vocabulary.yml). Rename the reviewer roles HEAVY -> PRIMARY (keep LIGHT) everywhere: ReviewerSelector + bin/reviewer-select output (HEAVY/LIGHT labels), the TaskEvent metadata['reviewers'] role values (role: heavy -> primary), the /deployments + task-timeline avatars UI that reads those roles, and devops-cycle-design.md §1.2 prose. Source the canonical role NAMES from config/devops_vocabulary.yml -> reviewer_roles (primary/light) so the SOP infographic, the tooling, and the docs all read one definition. 'Primary' generalizes as the accountable owner of a step (may apply to other stages later). Pure rename + keep behavior identical; add/adjust ReviewerSelector tests.

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 MEDIUM Dev MEDIUM Actual XL ≠ forecast
  1. Created → Designed
    M Machoke
    Machoke
    Model
    —
    Duration
    —
    Tokens
    —
    Cost
    —
    Completed Jun 26, 22:28 · about 11 hours ago
    api
  2. Designed → Building
    M Machoke
    Machoke
    Model
    claude-opus-4-8
    Duration
    about 1 hour
    Tokens
    25,320,330
    Cost
    ~$25.74
    Started Jun 26, 22:28
    Completed Jun 26, 23:22 · about 11 hours ago
    cli
  3. Building → Submitted
    M Machoke
    Machoke
    Model
    claude-opus-4-8
    Duration
    24 minutes
    Tokens
    —
    Cost
    —
    Started Jun 26, 23:22
    Completed Jun 26, 23:46 · about 10 hours ago
    cli
  4. Submitted → Reviewed
    J Jasper
    Jasper primary
    A Alex
    Alex light
    Model
    claude-opus-4-8
    Duration
    22 minutes
    Tokens
    27,460,229
    Cost
    ~$17.98
    Started Jun 26, 23:46
    Completed Jun 27, 00:08 · about 10 hours ago
    cli
  5. Reviewed → Blocked
    —
    Model
    —
    Duration
    26 minutes
    Tokens
    —
    Cost
    —
    Started Jun 27, 00:08
    Completed Jun 27, 00:34 · about 9 hours ago
    api
  6. Blocked → Building
    M Machoke
    Machoke
    Model
    claude-opus-4-8
    Duration
    7 minutes
    Tokens
    19,465,147
    Cost
    ~$13.19
    Started Jun 27, 00:34
    Completed Jun 27, 00:41 · about 9 hours ago
    cli
  7. Building → Submitted
    M Machoke
    Machoke
    Model
    claude-opus-4-8
    Duration
    5 minutes
    Tokens
    4,338,859
    Cost
    ~$2.93
    Started Jun 27, 00:41
    Completed Jun 27, 00:47 · about 9 hours ago
    cli
  8. Submitted → Reviewed
    J Jasper
    Jasper primary
    A Alex
    Alex light
    Model
    gpt-5.5
    Duration
    17 minutes
    Tokens
    1,043,357
    Cost
    ~$1.60
    Started Jun 27, 00:47
    Completed Jun 27, 01:04 · about 9 hours ago
    cli
  9. Reviewed → Assembled
    S Steffon
    Steffon
    Model
    gpt-5.5
    Duration
    1 minute
    Tokens
    258,019
    Cost
    ~$0.16
    Started Jun 27, 01:04
    Completed Jun 27, 01:05 · about 9 hours ago
  10. Assembled → Shipped
    A Avi
    Avi
    Model
    gpt-5.5
    Duration
    17 minutes
    Tokens
    —
    Cost
    —
    Started Jun 27, 01:05
    Completed Jun 27, 01:22 · about 9 hours ago

Conversation

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

QA Feedback about 9 hours ago

PR #251 now conflicts with release after PRs #252 and #253 merged. Rebase feat/reviewer-roles-primary-light on origin/release, resolve conflicts in stage agent/task timeline/docs test files, rerun the required checks, then resubmit for release merge.

Sealed-bid sizing

Edit →

Alex (PM)

—

Avi (PO)

MEDIUM

Dev

MEDIUM

Actual

XL