claude-dev-env 1.95.0 → 2.0.1

package/skills/pr-loop-cloud-transport/SKILL.md

@@ -20,6 +20,8 @@ Three checks, in order; the first failure routes to the cloud transport:
 
 A read-scoped account passes checks 1 and 2 and fails check 3 — the PR-loop skills push fix commits, so `push: false` means the local path stalls at its first push while the MCP transport carries the run.
 
+GitHub MCP substitution covers `gh` operations only. It does not make Claude-only slash commands (for example `/code-review`) runnable. Those steps go through the chain-spawn helper documented in [`worker-spawn.md`](../../_shared/pr-loop/worker-spawn.md#code-review-host-routing) (`detect_host_profile` + `invoke_code_review`). Transport (`gh` vs MCP) and harness (Claude vs third-party) are separate axes.
+
 ## Cloud transport workflow
 
 Copy this checklist and check off items as you complete them:

package/skills/reviewer-gates/SKILL.md

@@ -20,23 +20,24 @@ Every invocation carries the caller's identity (`--skill <caller>`, e.g. `pr-con
 
 ## Gate 1: Reviewer opt-out (`CLAUDE_REVIEWS_DISABLED`)
 
-The `CLAUDE_REVIEWS_DISABLED` environment variable is a comma-separated token list (case-insensitive, whitespace-tolerant). Three tokens exist:
+The `CLAUDE_REVIEWS_DISABLED` environment variable is a comma-separated token list (case-insensitive, whitespace-tolerant). Four tokens exist:
 
 | Token | Disables |
 |---|---|
 | `bugteam` | The whole bug-audit family: `/bugteam`, `/qbug`, `/findbugs`, and audit dispatch in `/monitor-open-prs` |
 | `bugbot` | Cursor Bugbot triggering and polling |
 | `copilot` | GitHub Copilot review requests and polling |
+| `codex` | Codex reviewer requests and polling |
 
 Cursor Bugbot is off by default: it runs only when `CLAUDE_REVIEWS_ENABLED` lists `bugbot`, and a `bugbot` token in `CLAUDE_REVIEWS_DISABLED` keeps it off even when the opt-in lists it.
 
 Run the shared parser — never an inline shell parse:
 
 ```bash
-python "$HOME/.claude/_shared/pr-loop/scripts/reviews_disabled.py" --reviewer <bugbot|bugteam|copilot>
+python "$HOME/.claude/_shared/pr-loop/scripts/reviews_disabled.py" --reviewer <bugbot|bugteam|copilot|codex>
 ```
 
-- **Exit 0** — the named reviewer is disabled for this run (opted out, or, for Bugbot, off by default without the `CLAUDE_REVIEWS_ENABLED` opt-in). A skill whose whole run depends on that reviewer responds with its refusal line and stops. A copilot- or bugteam-dependent caller names the opt-out: `/<caller> is disabled via CLAUDE_REVIEWS_DISABLED.` A Bugbot-dependent caller off by default names the opt-in the run needs: `/<caller> is disabled: Cursor Bugbot needs a bugbot token in CLAUDE_REVIEWS_ENABLED.` A loop that merely includes the reviewer as one gate marks the reviewer down (`bugbot_down = true`, `copilot_down = true`) and continues on its remaining signals.
+- **Exit 0** — the named reviewer is disabled for this run (opted out, or, for Bugbot, off by default without the `CLAUDE_REVIEWS_ENABLED` opt-in). A skill whose whole run depends on that reviewer responds with its refusal line and stops. A copilot- or bugteam-dependent caller names the opt-out: `/<caller> is disabled via CLAUDE_REVIEWS_DISABLED.` A Bugbot-dependent caller off by default names the opt-in the run needs: `/<caller> is disabled: Cursor Bugbot needs a bugbot token in CLAUDE_REVIEWS_ENABLED.` A loop that merely includes the reviewer as one gate marks the reviewer down (`bugbot_down = true`, `copilot_down = true`, `codex_down = true`) and continues on its remaining signals.
 - **Exit 1** — the reviewer is available; continue.
 
 ## Gate 2: Copilot quota pre-check (once per run)
@@ -93,11 +94,12 @@ When a reviewer is down, `check_convergence.py` must skip that reviewer's gate s
 |---|---|---|
 | Bugbot down or off | `--bugbot-down` | `bugbot` |
 | Copilot down or opted out | `--copilot-down` | `copilot` |
+| Codex down or opted out | `--codex-down` | `codex` |
 | Bugteam CLEAN post blocked | `--bugteam-post-blocked` | `bugteam` |
 
-The caller passes the flag on its own `check_convergence.py` run. The flag alone does not reach the independent mark-ready blocker hook, which re-runs `check_convergence.py` with no flags on `gh pr ready`. Each flag also honors its token: `check_convergence.py` reads the same bypass from `CLAUDE_REVIEWS_DISABLED`. So export the matching token in the same shell before `gh pr ready`, and the hook's no-flag re-check inherits the bypass through the env. Combine tokens with commas when more than one reviewer is down (`CLAUDE_REVIEWS_DISABLED="copilot,bugteam"`).
+The caller passes the flag on its own `check_convergence.py` run. The flag alone does not reach the independent mark-ready blocker hook, which re-runs `check_convergence.py` with no flags on `gh pr ready`. Each flag also honors its token: `check_convergence.py` reads the same bypass from `CLAUDE_REVIEWS_DISABLED`. So export the matching token in the same shell before `gh pr ready`, and the hook's no-flag re-check inherits the bypass through the env. Combine tokens with commas when more than one reviewer is down (`CLAUDE_REVIEWS_DISABLED="copilot,bugteam,codex"`).
 
-The `bugteam` token here means the bugteam CLEAN-review gate is skipped for this convergence check. It does not disable the bugbot or copilot gates: each token is scoped to its own reviewer.
+The `bugteam` token here means the bugteam CLEAN-review gate is skipped for this convergence check. It does not disable the bugbot, copilot, or codex gates: each token is scoped to its own reviewer.
 
 ## Gotchas
 

package/skills/team-advisor/SKILL.md

@@ -17,7 +17,7 @@ One warm, addressable advisor available at the strongest model tier the session
 
 **Detect the host profile first** (Host profiles in
 [`_shared/advisor/advisor-protocol.md`](../../_shared/advisor/advisor-protocol.md)
-— e.g. `ADVISOR_HOST_PROFILE` or `GROK_BUILD`). Do not start a model-floor
+— e.g. `ADVISOR_HOST_PROFILE` or `THIRD_PARTY`). Do not start a model-floor
 walk until the host is known.
 
 This session is the shared advisor's sole consumer, so its model floor is
@@ -29,23 +29,23 @@ the CLI fallback — using `team-advisor-agent` as the name and this session as
 the only consumer (skip the "who you are and your assignment" opener in each
 consult; a single-consumer session doesn't need it).
 
-**Grok host:** bind a max-tier Claude advisor through the shared CLI Claude-chain
-in the protocol (Fable high, then Opus max; `claude_chain_runner.py` walks
+**Third-party host:** bind a max-tier Claude advisor through the shared CLI Claude-chain
+in the protocol (Fable max, then Opus max; `claude_chain_runner.py` walks
 `~/.claude/claude-chain.json` for account usage failover). Consult via
 `--resume <session_id>` on that bind. This session is the sole consumer of that
 CLI advisor; skip the multi-consumer opener. When the chain cannot bind or
 reply, fail closed and report to the user — do **not** answer ENDORSE /
-CORRECTION / PLAN / STOP as this Grok session.
+CORRECTION / PLAN / STOP as this third-party session.
 
 ## Constraints
 
 - One advisor bind per session (`team-advisor-agent` on Claude; one CLI
-  `session_id` on Grok), owned by this session for its whole lifecycle
+  `session_id` on a third-party host), owned by this session for its whole lifecycle
   (spawn or CLI bind, drift re-bind, shutdown) — see
   [`_shared/advisor/advisor-protocol.md`](../../_shared/advisor/advisor-protocol.md).
 - Never bind the advisor, or its CLI path, at a tier below the protocol floor
-  for this host (Claude: this session's own tier; Grok: Opus floor with Fable
-  first).
+  for this host (Claude: this session's own tier; a third-party host: Opus floor
+  with Fable first).
 - The advisor only answers. It never edits a file, never runs a build or
   test, and never posts anything on the session's behalf.