claude-dev-env 1.94.0 → 2.0.0

package/_shared/advisor/scripts/tier_model_ids.py

@@ -4,16 +4,16 @@
 
     resolve_cli_model_id("Opus")   # ok: "opus"
     resolve_cli_model_id("fable")  # ok: "fable" (any letter case)
-    resolve_cli_model_id("Grok")   # ok: "grok" (Grok host single tier)
+    resolve_cli_model_id("ThirdParty")  # ok: "third-party" (third-party host tier)
     resolve_cli_model_id(" Opus ") # ok: "opus" (leading/trailing whitespace)
     resolve_cli_model_id("Titan")  # flag: ValueError
 
-    detect_host_profile(setting_by_name={"GROK_BUILD": "1"})  # ok: "Grok"
+    detect_host_profile(setting_by_name={"THIRD_PARTY": "1"})  # ok: "ThirdParty"
     detect_host_profile(
         setting_by_name={"ADVISOR_HOST_PROFILE": "Claude"}
     )  # ok: "Claude"
 
-Values are stable short aliases (``opus``, ``sonnet``, ``grok``), not dated
+Values are stable short aliases (``opus``, ``sonnet``, ``third-party``), not dated
 full model IDs such as ``claude-opus-4-…``. The map lives in
 ``advisor_scripts_constants`` so protocol text, skills, and tests share one
 source of truth for the aliases the Agent tool ``model:`` field and CLI
@@ -35,13 +35,13 @@ if _config_directory not in sys.path:
 
 from advisor_scripts_constants.model_tier_run_validator_constants import (  # noqa: E402
     ALL_CLI_MODEL_ID_BY_TIER,
-    ALL_GROK_BUILD_TRUTHY_VALUES,
     ALL_HOST_PROFILES,
     ALL_KNOWN_TIER_NAMES,
-    GROK_BUILD_ENV_VAR,
+    ALL_THIRD_PARTY_TRUTHY_VALUES,
     HOST_PROFILE_CLAUDE,
     HOST_PROFILE_ENV_VAR,
-    HOST_PROFILE_GROK,
+    HOST_PROFILE_THIRD_PARTY,
+    THIRD_PARTY_ENV_VAR,
     UNKNOWN_HOST_PROFILE_ERROR,
     UNKNOWN_LADDER_NAME_ERROR,
 )
@@ -53,13 +53,13 @@ def canonical_tier_name(tier_name: str) -> str | None:
     ::
 
         canonical_tier_name("opus")   # ok: "Opus"
-        canonical_tier_name("grok")   # ok: "Grok"
+        canonical_tier_name("thirdparty")  # ok: "ThirdParty"
         canonical_tier_name(" Opus ") # ok: "Opus"
         canonical_tier_name("")       # ok: None
         canonical_tier_name("Titan")  # ok: None
 
     Strips leading and trailing whitespace, then matches any letter case
-    against ``ALL_KNOWN_TIER_NAMES`` (Claude ladder plus the Grok host tier).
+    against ``ALL_KNOWN_TIER_NAMES`` (Claude ladder plus the third-party host tier).
 
     Args:
         tier_name: Raw tier text from a spawn log, protocol walk, or caller.
@@ -83,7 +83,7 @@ def resolve_cli_model_id(tier: str) -> str:
 
         resolve_cli_model_id("Sonnet")  # ok: "sonnet"
         resolve_cli_model_id("HAIKU")   # ok: "haiku"
-        resolve_cli_model_id("Grok")    # ok: "grok"
+        resolve_cli_model_id("ThirdParty")  # ok: "third-party"
         resolve_cli_model_id(" Opus ")  # ok: "opus"
         resolve_cli_model_id("Titan")   # flag: ValueError
 
@@ -93,7 +93,7 @@ def resolve_cli_model_id(tier: str) -> str:
 
     Args:
         tier: Ladder tier name (``Fable``, ``Opus``, ``Sonnet``, ``Haiku``,
-            or ``Grok``).
+            or ``ThirdParty``).
 
     Returns:
         The short model alias for that tier (for example ``"opus"``).
@@ -118,9 +118,9 @@ def detect_host_profile(
 
     ::
 
-        detect_host_profile(setting_by_name={"ADVISOR_HOST_PROFILE": "Grok"})
-        # ok: "Grok"
-        detect_host_profile(setting_by_name={"GROK_BUILD": "1"})  # ok: "Grok"
+        detect_host_profile(setting_by_name={"ADVISOR_HOST_PROFILE": "ThirdParty"})
+        # ok: "ThirdParty"
+        detect_host_profile(setting_by_name={"THIRD_PARTY": "1"})  # ok: "ThirdParty"
         detect_host_profile(setting_by_name={})                   # ok: "Claude"
         detect_host_profile(setting_by_name={"ADVISOR_HOST_PROFILE": "X"})
         # flag: ValueError
@@ -128,14 +128,14 @@ def detect_host_profile(
     Order:
 
     1. ``ADVISOR_HOST_PROFILE`` when set (must be a known profile name).
-    2. ``GROK_BUILD`` when truthy (``1``, ``true``, ``yes``, ``on``).
+    2. ``THIRD_PARTY`` when truthy (``1``, ``true``, ``yes``, ``on``).
     3. Default ``Claude``.
 
     Args:
         setting_by_name: Env name → setting text (defaults to ``os.environ``).
 
     Returns:
-        ``HOST_PROFILE_GROK`` or ``HOST_PROFILE_CLAUDE``.
+        ``HOST_PROFILE_THIRD_PARTY`` or ``HOST_PROFILE_CLAUDE``.
 
     Raises:
         ValueError: When ``ADVISOR_HOST_PROFILE`` is set to an unknown name.
@@ -157,8 +157,8 @@ def detect_host_profile(
             raise ValueError(UNKNOWN_HOST_PROFILE_ERROR.format(explicit_host_profile))
         return maybe_canonical_host
     raw_harness_marker = resolved_setting_by_name.get(
-        GROK_BUILD_ENV_VAR, ""
+        THIRD_PARTY_ENV_VAR, ""
     ).strip().lower()
-    if raw_harness_marker in ALL_GROK_BUILD_TRUTHY_VALUES:
-        return HOST_PROFILE_GROK
+    if raw_harness_marker in ALL_THIRD_PARTY_TRUTHY_VALUES:
+        return HOST_PROFILE_THIRD_PARTY
     return HOST_PROFILE_CLAUDE

package/_shared/pr-loop/CLAUDE.md

@@ -14,6 +14,7 @@ Runtime documents and scripts shared by every PR-loop skill. Changes here affect
 | `state-schema.md` | Fields each PR-loop workflow tracks across iterations; documents common fields and per-skill extensions |
 | `code-rules-gate.md` | Reference for the CODE_RULES pre-commit gate check; describes what the gate blocks and when it runs |
 | `precatch-rubric.md` | Shared pre-catch lane checklist — deterministic sweep, doc-vs-code parity, test-assertion completeness, PR-description parity, adversarial audit — that autoconverge's lenses and pr-converge's CODE_REVIEW step read on demand |
+| `worker-spawn.md` | Worker-spawn tier protocol and Claude-only slash-step host routing: three tiers, CLI/API inputs, JSON result shape, host detection, code-review modes via `detect_host_profile` + `invoke_code_review`, leader-socket rule, cloud setup; documents `scripts/resolve_worker_spawn.py` |
 
 ## Subdirectory
 

package/_shared/pr-loop/scripts/CLAUDE.md

@@ -13,7 +13,7 @@ Python scripts invoked at runtime by the PR-loop skills. Each script is a standa
 | `revoke_project_claude_permissions.py` | Removes the allow-rules and entries that `grant_project_claude_permissions.py` wrote; safe to run when no prior grant exists |
 | `code_rules_gate.py` | Pre-commit gate that runs `code_rules_enforcer` checks on staged Python files before a fix commit lands, and the terminology sweep over the staged diff |
 | `terminology_sweep.py` | Flags a prose term that near-misses an identifier introduced on added code lines of a unified diff (shared leading word, divergent tail) |
-| `reviews_disabled.py` | Shared helper for the reviewer opt-out and opt-in gates; parses `CLAUDE_REVIEWS_DISABLED` and `CLAUDE_REVIEWS_ENABLED` tokens to decide which reviewers run (bugbot is off by default and runs only when `CLAUDE_REVIEWS_ENABLED` lists it) |
+| `reviews_disabled.py` | Shared helper for the reviewer opt-out and opt-in gates; parses `CLAUDE_REVIEWS_DISABLED` tokens `bugteam`, `bugbot`, `copilot`, and `codex`, plus `CLAUDE_REVIEWS_ENABLED` for bugbot opt-in (bugbot is off by default and runs only when `CLAUDE_REVIEWS_ENABLED` lists it) |
 | `copilot_quota.py` | Copilot premium-request quota pre-check: resolves a configured GitHub account, reads its remaining `premium_interactions` quota via `gh api copilot_internal/user`, and exits 0 (run Copilot) or non-zero (skip: out of quota, API down, or no account configured) |
 | `reviewer_availability.py` | Unified reviewer-availability entry point for Copilot and Bugbot: reuses `copilot_quota.py` and `reviews_disabled.py` and exits 0 when the named `--reviewer` may be spawned, non-zero when it is opted out or (for Copilot) out of quota |
 | `fix_hookspath.py` | Repairs a malformed `core.hooksPath` global git config entry |
@@ -23,6 +23,7 @@ Python scripts invoked at runtime by the PR-loop skills. Each script is a standa
 
 | Entry | Description |
 |---|---|
+| `code_rules_gate_parts/` | The decomposed modules `code_rules_gate.py` wires together: enforcer loading, git file sets, blob readers, added-line maps, violation scoping, wrapper plumb-through, gate running, staged-test running, and argument parsing |
 | `pr_loop_shared_constants/` | Named constants used by the scripts above |
 | `tests/` | pytest suite for all scripts in this directory |
 

package/_shared/pr-loop/scripts/README.md

@@ -8,6 +8,7 @@ Runnable helpers used by **bugteam**, **qbug**, **pr-converge**, and related ski
 | --- | --- |
 | `preflight.py` | Local checks before a PR-loop run (pytest discovery, optional pre-commit, hooksPath sanity). |
 | `reviewer_availability.py` | Unified availability check for Copilot and Bugbot; reuses `copilot_quota.py` and `reviews_disabled.py` and exits 0 when the named `--reviewer` may be spawned. |
+| `reviews_disabled.py` | Opt-out and opt-in gate for reviewer tokens `bugteam`, `bugbot`, `copilot`, and `codex` via `CLAUDE_REVIEWS_DISABLED` / `CLAUDE_REVIEWS_ENABLED`. |
 | `code_rules_gate.py` | CODE_RULES gate over PR-scoped diffs (`--base`, staged-only, path filters). |
 | `fix_hookspath.py` | Repair `core.hooksPath` when it does not point at the packaged git-hooks tree. Single home for unexpected non-empty git stderr (hard-fail `RuntimeError`); skill wrappers may call it later. |
 | `grant_project_claude_permissions.py` / `revoke_project_claude_permissions.py` | Claude Code permission JSON helpers used during publish-style flows. |