McRitchie Studio

Agent Context

From conductor retro 2026-06-25 (ship of rel-20260624-b2f18e). FIX #1 (true ship-blocker): bin/release record_post_deploy_check (bin/release ~L375) builds a heroku-run rails-runner via conductor() that string-interpolates cmd.inspect into the shell; a post_deploy_cmd containing parens (seed-54: bin/rails runner "load Rails.root.join(%q(...)).to_s") produced /bin/bash: syntax error near unexpected token '(' -> conductor() abort! at ~L253 (record op returned no JSON) -> prepare aborted BEFORE assemble! (release stuck 'assembling') and would abort ship after prod-deploy+gem-publish. Make the conductor() rails-runner payload shell-safe (pass Ruby via stdin/heredoc or Shellwords.escape the whole -e arg) so ANY cmd records. Add an integration test feeding a paren/quote-laden cmd. FIX #2: the ship-preflight (Release::ShipSequence / repo_git_state parsing git status --porcelain) counts uncommitted generated artifacts as dirty and blocks the ff every ship: docs/agents/audits/retro-rel-*.md (bin/release retro output) + docs/agents/maintenance/delete-later.md (agent-worktree ledger). Ignore those known-generated paths (allowlist) OR --untracked-files=no for them; keep counting real code dirt. FIX #1b (dor-check): canonical post_deploy_cmd is 'bin/rails runner <path>' (rails runner takes a file path directly; paren-free). dor-check should SUGGEST/warn toward that form (not hard-reject existing valid cmds). Test tiers: unit for the quoting helper + dor-check matcher; integration for prepare-reaches-assemble with a paren cmd.