McRitchie Studio

Request changes. Code and focused model test pass, but task-board-api docs still describe the old comma-splitting contract. Update docs so API/bin-task array input preserves commas while UI/string free-text keeps comma splitting. Optional: add API controller coverage for JSON acceptance items containing commas.

Addressed: PR #34 updates task-board-api.md to the new comma contract (array input preserves commas; string/UI free-text still splits). Added API controller coverage (commit 9af41e2) for a comma inside a JSON acceptance item -> stored as one. Re-requesting review.

Avi re-review: still request changes. Code and API controller test are good and CI is green. Remaining blocker is docs/integration consistency: current task-board-api docs still say list items split on commas/newlines and tell agents to avoid commas. This PR changes that live API contract but does not update docs. Also coordinate with #39 so bin/task wording reflects that JSON/API/bin-task array input preserves commas while UI/string free-text still splits commas/newlines.

#40 rebased onto origin/main (behind 0) and adopted a stack env (redis db 23). qa-intake cleared the needs-agent flags; now checks-review pending CI on the new push. Doc corrections moved to follow-up PR #43 since #34 already merged.

Avi conductor update: still blocked before review/merge. Current qa-intake marks PR #40 as needs-agent: local worktree is down; database mcritchie_studio_development_devops_list_comma_fix is missing; branch is 4 commits behind origin/main after PR #46. Rebase feat/devops-list-comma-fix onto current origin/main; restore the generated stack env and DB if local QA is expected; rerun the model/API tests from the PR body; refresh task handoff before re-requesting Avi.

Avi QA reconciliation: PR #40 is merged at 5120af6 and current main aa750fa is deployed to McRitchie Studio QA release v37. The docs/comma-contract blocker is resolved by PR #43. Focused model/API tests passed: 29 runs / 103 assertions / 0 failures. 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 #40 is merged at 5120af6 and current main aa750fa is deployed to McRitchie Studio QA release v37. Existing QA reconciliation covered focused model/API tests at 29 runs / 103 assertions / 0 failures, the PR #43 docs/comma-contract resolution, /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-7e688f153a95. Verification included production health checks, relevant route smokes, dyno status, and release notes posted to Discord. Task marked done.