Agents Builders

Harden QA release SOP

Archived
harden-qa-release-sop

Created

Jun 23, 02:41

Started

Jun 23, 02:41

Completed

Jun 23, 03:03

DevOps handoff

Type

Docs

Shape

Worktree Slug

harden-qa-release-sop

Repositories

mcritchie-studio

Release Train

Branch

feat/harden-qa-release-sop

Local URL

QA URL

Production URL

docs devops

Acceptance Criteria

  • SOP specifies --yes on prepare ship merge
  • Merge step covers draft-ready and base-retarget
  • Cold-start note clarifies conductor not feature agent

Expected Test Plan

No expected checks recorded.

Checks Run

No completed checks recorded.

Agent Context

Docs-only hardening of the Build and Deploy QA Release SOP in docs/agents/system/devops-cycle-design.md, capturing gotchas a cold non-interactive agent hits. Full spec in the dispatch prompt. No code, docs-only (dor-exempt).

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.

  1. Shipped Archived
    Model
    Duration
    Tokens
    Cost
    Completed Jun 23, 17:13 · 4 days ago

Conversation

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

QA Feedback 4 days ago

Strong PR — cold-start framing, pre-merge checklist (gh pr ready / gh pr edit --base release), and /up 000 note are all accurate, and the ship gate is handled correctly (no --yes on ship; meta-trigger stops at the operator human gate). One accuracy defect in the --yes callout (lines 289-297): it claims prepare/ship/merge/archive 'all prompt for confirmation' and 'silently no-op' without --yes. Verified against bin/release: confirm() is called ONLY by prepare (line 378, 'return unless confirm' = true silent no-op, the dangerous case you nailed), ship (line 881) and archive (line 994). Both ship and archive use 'abort! ... unless confirm' = LOUD abort, not a silent no-op. merge (def at line 320) has NO confirm call at all — it never prompts, so --yes is harmless/ignored there, not required. Net: the commands shown are all safe, but the rationale over-reaches. Fix: tighten the callout so prepare is the silent-no-op case, ship/archive abort loudly without --yes, and merge takes --yes only defensively/for future-proofing (no prompt today). Docs-only and dor-check (docs-exempt) both confirmed.

Sealed-bid sizing

Edit →

Alex (PM)

Avi (PO)

Dev

Actual