McRitchie Studio
Say Hi 👋
Agents Builders
← All Tasks
Sizing

Clarify task-board DevOps handoff contract

Archived avi
task-552b68ce294d

Follow-up pass on the task-board DevOps workflow that landed in McRitchie Studio main. Worktree slug: task-board-devops-contract Scope: - Keep Task.slug as the immutable random app id. - Document devops["worktree_slug"] as the place to record the worktree slug. - Make production https://mcritchie.studio the canonical durable task-board target. - Add devops["checks_run"] for actual feature-agent checks while keeping devops["test_plan"] for expected checks. - Make release_train optional unless multiple tasks need to move together. - Update docs, UI/API/model helpers, focused tests, and regenerate /Users/alex/projects/AGENTS.md.

Created

Jun 17, 21:34

Started

Jun 17, 21:34

Completed

Jun 19, 16:15

DevOps handoff

Type

Chore

Shape

—

Worktree Slug

task-board-devops-contract

Repositories

mcritchie-studio

Release Train

—

Branch

feat/task-board-devops-contract

Pull Request

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

Local URL

http://localhost:3008

QA URL

https://qa.mcritchie.studio/tasks/task-552b68ce294d

Production URL

https://mcritchie.studio/tasks/task-552b68ce294d
docs devops task-board agents api

Acceptance Criteria

  • Docs no longer imply Task.slug must match the worktree slug.
  • Docs identify devops["worktree_slug"] as the worktree slug field.
  • Docs state production https://mcritchie.studio is the durable task-board target.
  • Docs state local/QA task boards are only for testing task-board behavior and production must be backfilled before PR handoff.
  • Task devops metadata supports expected test_plan separately from actual checks_run.
  • UI/API/model helpers allow agents to write and read checks_run.
  • release_train is documented as optional except when related tasks move through QA/release together.
  • Focused model/API tests cover worktree_slug and checks_run normalization/helpers.
  • /Users/alex/projects/AGENTS.md is regenerated from docs/agents/index.md.

Expected Test Plan

  • bin/rails test test/models/task_test.rb test/controllers/api/v1/tasks_controller_test.rb test/controllers/tasks_controller_test.rb
  • bin/install-agent-docs check
  • git diff --check

Checks Run

  • PATH="/opt/homebrew/opt/ruby@3.3/bin:$PATH" bundle exec rails tailwindcss:build
  • PATH="/opt/homebrew/opt/ruby@3.3/bin:$PATH" bundle exec rails test test/models/task_test.rb test/controllers/api/v1/tasks_controller_test.rb test/controllers/tasks_controller_test.rb (48 runs
  • 163 assertions
  • 0 failures)
  • bin/install-agent-docs
  • bin/install-agent-docs check
  • git diff --check

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.

No stage changes recorded yet.

Conversation

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

QA Feedback avi 9 days ago

Avi review paused: PR #27 now conflicts with current main after the task conversation feedback release. Please rebase feat/task-board-devops-contract on origin/main, resolve the app/model/view/doc overlaps, rerun the documented task/API tests plus bin/install-agent-docs and bin/install-agent-docs check, then return to PR REVIEW.

QA Feedback avi 9 days ago

Still blocked after today’s merges: PR #27 remains draft/stale and must be rebased onto current main dd8af08 before Avi can review. Please resolve doc/app overlaps with #34, rerun the documented checks, regenerate docs if needed, and return to PR REVIEW.

QA Feedback avi 9 days ago

Avi scout update for McRitchie Studio PR #27: not ready but not superseded. Functional fields devops.worktree_slug and devops.checks_run are still relevant, but branch is draft/conflicting and 9 commits behind main. Rebase on current main, preserve newer task conversation/API docs, add new keys to task-board API docs, rerun checks, regenerate AGENTS if needed.

QA Feedback avi 9 days ago

Avi conductor update: still blocked before review/merge. Current qa-intake marks PR #27 as needs-agent: PR is draft; merge state is dirty; branch is 15 commits behind origin/main after PR #46. Rebase feat/task-board-devops-contract onto current origin/main; reconcile against newer task-board/devops docs and model changes; rerun the task API/docs checks; update the task handoff; undraft before re-requesting Avi.

QA Feedback avi 8 days ago

Avi accepted management of this PR-review task. Current status: linked PR #27 is still draft and merge-conflicting, so it is not ready for merge or QA deployment. Required next action: rebase feat/task-board-devops-contract on current origin/main, reconcile against the already-merged task-board API/CLI/docs changes, rerun the relevant checks, and undraft only once the branch is clean and mergeable. Avi will re-review after that update.

Handoff avi 8 days ago

QA handoff: PR #27 merged as 4af95f8 and deployed to McRitchie Studio QA v38. QA URL: https://qa.mcritchie.studio/tasks/task-552b68ce294d. Checks: focused task-board suite 55 runs / 195 assertions; full Rails suite 761 runs / 2720 assertions; bin/rubocop inspected 404 files; GitHub CI lint/test/scan_js/scan_ruby passed; QA /up 200 and web + worker dynos are up. Awaiting Mr. McRitchie QA review; no production deploy performed.

Handoff avi 8 days ago

QA accepted for production-ready promotion: deployed on McRitchie Studio QA at https://qa.mcritchie.studio/tasks/task-552b68ce294d with /up 200, full Rails and RuboCop green; included in current production deployment batch.

Handoff avi 8 days ago

Production shipped: McRitchie Studio v72 / 4af95f8. Production URL: https://mcritchie.studio/tasks/task-552b68ce294d. Verification included production health checks, relevant route smokes, dyno status, and release notes posted to Discord. Task marked done.

Sealed-bid sizing

Edit →

Alex (PM)

—

Avi (PO)

—

Dev

—

Actual

—