Created
Jun 24, 17:56
Started
Jun 24, 17:57
Completed
Jun 24, 18:59
DevOps handoff
Type
Feature
Shape
backend
Worktree Slug
dor-check-post-deploy-gate
Repositories
mcritchie-studio
Release Train
—
Branch
feat/dor-check-post-deploy-gate
Local URL
—
QA URL
—
Production URL
—
Acceptance Criteria
Expected Test Plan
Checks Run
Agent Context
Session retro improvement #1 — a near-miss prod incident. merge-docs-reviewer-into-alex shipped with post_deploy_cmd='bin/rails db:seed', which would have seeded demo News/Content/Tasks into PRODUCTION and also aborted on a non-idempotent seed (db/seeds/54). Not caught because reviewers read the code diff, not the deploy metadata; surfaced only when QA aborted. Add a bin/dor-check gate that REJECTS a post_deploy_cmd matching a bare full-suite seed (bin/rails db:seed / rails db:seed / db:seed:replant — db/seeds.rb loads ALL db/seeds/*.rb incl demo data + any non-idempotent file) and REQUIRES a narrow command — a scoped runner like rails runner 'load Rails.root.join("db/seeds/NN_x.rb").to_s' or a dedicated rake task. The failure message must explain WHY (prod demo-data + non-idempotent abort risk) and show the narrow pattern. Gate only when devops.post_deploy_cmd is present. See memory feedback_post_deploy_cmd_for_seeds. DOCS: devops-cycle-design.md + the devops-task-board module; also note in AGENTS.md/parallel-agent-devops that reviewers must review post_deploy_cmd, not just the diff.
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.
Post-merge review sign-off (cascade finished after the parallel conductor had already merged this onto rel-20260624-6f6638): carl[heavy] APPROVE (full regex probe — BARE_SEED_TASK has no hole; lookbehind/lookahead kill mydb:seed/db:seed_fund), shannon[light] APPROVE. Non-blocking follow-ups: (1) exempt-kind + zero-diff task hits exit 0 before the gate — move the post_deploy check ABOVE the exempt early-exit to close it; (2) db:setup/db:reset/db:prepare also seed but are out of this task's scope.
Sealed-bid sizing
Edit →Alex (PM)
—
Avi (PO)
—
Dev
—
Actual
—
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.