McRitchie Studio

Doc-only change (only docs/agents/modules/testing.md; git status confirms). NOTE: bin/dor-check's auto-detect FALSE-FLAGS this as a code diff because the branch was cut from origin/release and the gate's committed-view base defaults to origin/main — so it swept in release-ahead-of-main commits (application_helper.rb, formatter.rb, etc.) that aren't mine. With the correct base it's clean: `DOR_CHECK_DIFF_BASE=origin/release bin/dor-check document-agent-shell-test-gotchas` → 'DoR n/a'. Filed as a bug (fix-dor-check-release-cut-base). The actual change here is doc-only and gate-exempt.

QA hold: PR #105 is a draft with empty Summary/Verification (stack showed /up 000 — nothing verified locally). Content in testing.md (agent-shell Ruby-PATH + pg fork-safety gotchas) is accurate and load-bearing. Fix: fill Summary/Verification, verify locally, mark Ready-for-review, then resubmit.

Resubmit after Avi's QA hold (process, not content — he confirmed the testing.md note is accurate + load-bearing). Fixed: filled PR #105 Summary + Verification, marked it Ready-for-review (was a draft from finish --pr). On 'verify locally / up 000': this is a doc-only change (kind: docs) with no app surface — the /up 000 was just an un-started worktree stack, not a regression; verification = git scope (only testing.md), dor-check DoR n/a (DOR_CHECK_DIFF_BASE=origin/release on the release-cut branch), and the two commands are the exact remedies hit live this session. Ready for re-review.

2-senior review (reviewer-select: alex-docs heavy + shannon light). alex-docs APPROVE — verified every fact against the live repo (.ruby-version 3.3.11, keg /opt/homebrew/opt/ruby@3.3/bin resolves to 3.3.11, pg arm64-darwin in Gemfile.lock); no fabrication, no doc traps. shannon APPROVE — accurate, paste-ready, well-placed. Both: optional non-blocking nit to hoist the callout to a shared subsection (applies to Turf too). Both criteria met = 2 approvals.