plan: adapt concurrency restructure to builder/adversary loop protocol (gates M1/M2, phase-namespaced state)

cc-ci-plan/concurrency-restructure-full-plan.md

@@ -151,29 +151,46 @@ ABRA_DIR isolation:
 
 Each test must clean up helper processes in `finally`/fixtures (no leaked children in the test VM).
 
-## Roles & flow
+## Roles, gates & Definition of Done (loop protocol — plan.md §6.1 applies)
 
-1. **Builder** (fable — main-loop model, no override): implements P1→P5 + test suite on branch
+This phase runs as Builder/Adversary loops with phase-namespaced state files
-   `restructure/concurrency` in a fresh clone; one commit per phase; runs
+(STATUS-conc.md, REVIEW-conc.md, BACKLOG-conc.md, JOURNAL-conc.md).
-   `pytest tests/unit -q`, `pytest tests/concurrency -q`, `scripts/lint.sh` before each commit;
+
-   pushes the branch (never main). Commit author `autonomic-bot <autonomic-bot@noreply.git.autonomic.zone>`.
+**Builder**: implements P1→P5 + the tests/concurrency suite on branch `restructure/concurrency`
-2. **Adversary** (opus): adversarial review of the full branch diff — hunts races (probe vs
+in YOUR clone; one commit per phase; before each commit ALL of:
-   acquire ordering, signal-handler reentrancy, teardown-during-teardown), deleted-code fallout
+`pytest tests/unit -q`, `pytest tests/concurrency -q` (once it exists), `scripts/lint.sh` —
-   (grep for dangling references to registry symbols), gate integrity (recipe tests/`RUN_APP_RE`/
+green. Push the branch (NOT main — main merge is gated below). Claim gates via `claim(conc): ...`
-   warm apps untouched), and test-suite blind spots. Produces findings list; default-skeptical.
+commits + STATUS-conc.md per protocol.
-3. Builder addresses findings; adversary re-checks. Orchestrator merges to main only when:
+
-   adversary pass + both test suites green + lint green + branch push build green.
+**Adversary**: cold-verify from your own clone. For M1: check out the branch, run both suites +
-4. **Live verification** (orchestrator, post-merge):
+lint yourself, then adversarially review the full diff — hunt races (probe vs acquire ordering,
-   a. trigger an immich !testme, cancel mid-run via drone API → assert harness pid dies (no leak),
+signal-handler reentrancy, teardown-during-teardown), deleted-code fallout (grep for dangling
-      lock released, next janitor reaps the app;
+references to registry symbols: register_run_app, unregister_run_app, _run_owner_state,
-   b. two parallel !testme runs (immich + plausible) → both green, zero leakage;
+ACTIVE_RUN_DIR, CCCI_JANITOR_MAX_AGE, acquire_recipe_lock), gate integrity (recipe tests /
-   c. double-!testme same PR → second blocks on app lock, then runs;
+RUN_APP_RE / warm apps untouched), and test-suite blind spots vs the 19 cases above.
-   d. confirm a full green run end-to-end (the standing regression canary).
+Default-skeptical; findings to REVIEW-conc.md.
+
+**Gates:**
+- **M1 — implementation verified.** Branch complete (P1–P5 + tests), both suites + lint green
+  on the Adversary's cold clone, adversarial diff review PASS in REVIEW-conc.md.
+- **M2 — merged + live-verified.** After M1 PASS only, Builder merges the branch to main
+  (merge commit, never force) and confirms the push build green. Then live verification on the
+  real CI host (evidence in STATUS-conc.md, Adversary re-checks):
+  a. trigger an immich !testme, cancel mid-run via drone API → harness pid dies (no leaked
+     python), lock released, next janitor reaps the app — zero leakage;
+  b. two parallel !testme runs (immich PR#2 + plausible PR#3) → both green, zero leakage;
+  c. double-!testme same PR → second blocks on the app lock (visible in its drone log), then runs;
+  d. one full green run end-to-end.
+
+**## DONE in STATUS-conc.md** only when M1 and M2 both show a fresh Adversary PASS in
+REVIEW-conc.md. NOTE for both loops: recipe-mirror PRs (immich#2, plausible#3) are used as
+!testme targets only — NEVER merge or push to recipe mirror repos.
 
 ## Guardrails (builder + adversary MUST honor)
 
 - NEVER weaken recipe-test gates or touch `tests/<recipe>/` content.
-- NEVER push cc-ci main; branch only. Never force-push.
+- cc-ci main is touched ONLY by the M2 merge after M1 PASS; all other work stays on the branch.
+  Never force-push. NEVER merge/push recipe mirror repos.
 - No secrets in commits; reference `.testenv` / `/run/secrets` locations only.
 - Don't touch `services_converged()` / paused-is-settled logic except where the plan says.
 - Match repo commit style (`feat(...)`/`fix(harness)`/`test(...)`/`docs:`).