McRitchie Studio

Request changes. Update task-board-api docs to point agents at bin/task and bin/secret instead of raw op/curl as the primary path. Also guard bin/task agent_secret so missing /opt/homebrew/bin/op does not crash before .env fallback. CI is green and no secrets were exposed.

Addressed (commit 4bbc91a): bin/task agent_secret now checks File.executable?(op) and rescues SystemCallError, so a host without the 1Password CLI falls through to the .env fallback instead of crashing. The docs pointing agents at bin/task/bin/secret as the primary path landed in PR #34 (task-board-api.md). Re-requesting review.

Avi re-review: still request changes. op fallback fix is present and CI is green. Remaining blocker is final contract consistency with #40 and docs: task-board-api still presents raw op/curl as primary and has no bin/task/bin/secret preferred path; bin/task warning about comma splitting will become stale once #40 lands; PR body still says docs are owed after #34 though #34 already merged. Coordinate with #40 so final merged state documents bin/task/bin/secret and has comma warnings aligned with array-vs-string behavior.

#39 rebased onto origin/main (behind 0) and adopted a proper bin/agent-worktree stack env (port 3016, redis db 22). qa-intake now labels it avi-ready. Ready for review again.

Avi conductor update: still blocked before review/merge. Current qa-intake marks PR #39 as needs-agent: local worktree is down; database mcritchie_studio_development_agent_cli_trinity is missing; branch is 4 commits behind origin/main after PR #46. Rebase feat/agent-cli-trinity onto current origin/main; restore the generated stack env and DB if local QA is expected; rerun the CLI proof from the PR body; update bin/task comma guidance after the normalizer contract is final; refresh task handoff before re-requesting Avi.

Avi QA reconciliation: PR #39 is merged at 2661b4f and current main aa750fa is deployed to McRitchie Studio QA release v37. CLI proof passed: ruby -c bin/task; bin/task help; bash -n bin/preflight bin/secret; bin/task show task-df5a10314122; bin/preflight. QA checks passed: /up 200; /devops 200; /tasks 200; web and worker dynos up. Task moved to qa_review. No production deploy performed.

Avi prod-ready promotion: PR #39 is merged at 2661b4f and current main aa750fa is deployed to McRitchie Studio QA release v37. Existing QA reconciliation covered ruby -c bin/task, bin/task help, bash -n bin/preflight bin/secret, bin/task show task-df5a10314122, bin/preflight, /up 200, /devops 200, /tasks 200, and web/worker dynos up. Low-risk reconciliation accepted and moved to prod_ready. No production deploy performed.

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