claude-dev-env 2.22.0 → 2.22.1

package/.agents/skills/team-advisor/SKILL.md

@@ -30,6 +30,8 @@ One warm advisor at the strongest tier this session can reach. This session is t
 5. Skip the multi-consumer "who you are" opener — sole consumer.
 6. When the bind or reply path fails, fail closed and report to the user. On a third-party host, only the bound Claude advisor issues ENDORSE / CORRECTION / PLAN / STOP.
 
+**GOTCHA (Cursor / ThirdParty + Sol):** when the walk reaches Sol or the user asks for Sol, first tool call is `python ~/.claude/_shared/advisor/scripts/codex_sol_advisor.py --bind --enable-sol --cwd <repo-root>` with the charter on stdin — not Agent/Task, not a probe-path search. Details: [`third-party-bind.md`](../../_shared/advisor/reference/third-party-bind.md) GOTCHA and [`sol-rung.md`](../../_shared/advisor/reference/sol-rung.md).
+
 Full walk, charter, consult packet, Sol routing, and drift re-bind live in the protocol read map and its authoritative `reference/` leaves.
 
 ## Consult

package/_shared/advisor/advisor-protocol.md

@@ -76,6 +76,7 @@ Assemble and paste each executor's Advisor block per the **Advisor block** secti
 On a third-party (non-Claude, non-Codex) harness, the shared CLI Claude-chain is the one path to a Claude advisor: bind Fable through it, per [`reference/third-party-bind.md`](reference/third-party-bind.md).
 The bound Claude session is the advisor; this third-party session stays the executor.
 Walk `candidate_tiers = ["Fable"]`. When Fable is out of usage, the sol rung binds after Fable (`candidate_tiers = ["Fable", "Sol"]`).
+**Cursor Sol first shot:** when the walk reaches Sol (or the user asks for Sol), bind through `codex_sol_advisor.py --bind --enable-sol` on the first tool call (see the GOTCHA in [`reference/third-party-bind.md`](reference/third-party-bind.md)) — not the Agent tool, and not a probe-path hunt.
 **Fail closed:** when every candidate fails, set `selected_tier = null` and a `fallback_reason`, report that the advisor is unreachable, and **stop** — ENDORSE / CORRECTION / PLAN / STOP come only from a bound advisor.
 Executors report to the orchestrating session; that session consults the bound advisor and relays the four-signal reply.
 

package/_shared/advisor/reference/sol-rung.md

@@ -24,9 +24,9 @@ Flag on: run the Codex preflight first —
 python ~/.claude/_shared/pr-loop/scripts/codex_usage_probe.py
 ```
 
-Repo home: `packages/claude-dev-env/_shared/pr-loop/scripts/`.
+**GOTCHA — probe path:** that installed path is the only one. `codex_sol_advisor.resolve_usage_probe_path` builds `~/.claude/_shared/pr-loop/scripts/codex_usage_probe.py` and stops. Do not hunt `skills/codex-review/`, Codex worktrees under `C:\dev\.codex\worktrees\`, or other copies. Repo home: `packages/claude-dev-env/_shared/pr-loop/scripts/`.
 
-The shared entry point is `~/.claude/_shared/advisor/scripts/codex_sol_advisor.py`; it calls the installed probe and owns Sol bind or resume parsing. Bind with `python ~/.claude/_shared/advisor/scripts/codex_sol_advisor.py --bind --cwd <repo-root>` and pipe the charter on stdin. Resume with `--resume <session_id>` and pipe the delta consult on stdin.
+The shared entry point is `~/.claude/_shared/advisor/scripts/codex_sol_advisor.py`; it calls the installed probe and owns Sol bind or resume parsing. Bind with `python ~/.claude/_shared/advisor/scripts/codex_sol_advisor.py --bind --enable-sol --cwd <repo-root>` and pipe the charter on stdin. Resume with `--resume <session_id>` and pipe the delta consult on stdin.
 
 The gate passes only when the probe exits 0, `percent_left` is finite numeric data, and `percent_left` is strictly greater than `WEEKLY_USAGE_GATE_THRESHOLD_PERCENT` from the existing probe. The exact-threshold case fails closed when Fable did not bind.
 

package/_shared/advisor/reference/third-party-bind.md

@@ -3,6 +3,18 @@
 Detail behind the **Host profiles → Third-party host** section of [`advisor-protocol.md`](../advisor-protocol.md).
 Open this when binding or re-binding the advisor from a third-party (non-Claude, non-Codex) harness.
 
+## GOTCHA — Cursor Sol first shot
+
+Cursor is a third-party host. When the walk reaches Sol (`ADVISOR_SOL=1` / `--enable-sol` after Fable is out of usage, or the user asks for Sol), bind Sol on the **first** tool call through the headless Codex helper. Do not use the Agent or Task tool, do not spawn Grok as a substitute, and do not search the filesystem for `codex_usage_probe.py`.
+
+```
+python "$HOME/.claude/_shared/advisor/scripts/codex_sol_advisor.py" --bind --enable-sol --cwd <repo-root>
+```
+
+Pipe the standing-reviewer charter on stdin. Persist `session_id` from the JSON reply. Later consults: `--resume <session_id>` with the delta on stdin. Capture helper / `codex` stdout as UTF-8 on Windows (`encoding="utf-8"`, `errors="replace"`). Probe path and preflight: [`sol-rung.md`](sol-rung.md).
+
+When the Sol flag is off, follow the Claude-chain steps below.
+
 ## Bind steps
 
 1. Name the session identity first (protocol **Host profiles**). This path is for a ThirdParty profile.

package/_shared/advisor/scripts/codex_sol_advisor.py

@@ -34,6 +34,10 @@ from advisor_scripts_constants.sol_advisor_constants import (  # noqa: E402
     CODEX_REASONING_CONFIG_TEMPLATE,
     CODEX_RESUME_SUBCOMMAND,
     CODEX_SANDBOX_FLAG,
+    USAGE_PROBE_FILENAME,
+    USAGE_PROBE_PACKAGE_DIRECTORY_NAME,
+    USAGE_PROBE_SCRIPTS_DIRECTORY_NAME,
+    USAGE_PROBE_SHARED_DIRECTORY_NAME,
     SOL_BIND_FAILURE_REASON,
     SOL_CODEX_TIMEOUT_REASON,
     SOL_CODEX_TIMEOUT_SECONDS,
@@ -201,6 +205,14 @@ def resolve_advisor_effort(
 def resolve_usage_probe_path(home_directory: Path) -> Path:
     """Return the installed Codex weekly usage probe path.
 
+    ::
+
+        resolve_usage_probe_path(Path.home())
+            -> ~/.claude/_shared/pr-loop/scripts/codex_usage_probe.py
+
+    This is the only probe path the Sol helper uses. Do not search worktrees
+    or archived ``skills/codex-review`` trees for a second copy.
+
     Args:
         home_directory: Home directory used to construct the path.
 
@@ -210,10 +222,10 @@ def resolve_usage_probe_path(home_directory: Path) -> Path:
     return (
         home_directory
         / CLAUDE_CONFIG_DIRECTORY_NAME
-        / "skills"
-        / "codex-review"
-        / "scripts"
-        / "codex_usage_probe.py"
+        / USAGE_PROBE_SHARED_DIRECTORY_NAME
+        / USAGE_PROBE_PACKAGE_DIRECTORY_NAME
+        / USAGE_PROBE_SCRIPTS_DIRECTORY_NAME
+        / USAGE_PROBE_FILENAME
     )
 
 

package/_shared/advisor/scripts/config/advisor_scripts_constants/sol_advisor_constants.py

@@ -14,6 +14,10 @@ CODEX_EXEC_SUBCOMMAND: str = "exec"
 CODEX_RESUME_SUBCOMMAND: str = "resume"
 CODEX_SANDBOX_FLAG: str = "--sandbox"
 CLAUDE_CONFIG_DIRECTORY_NAME: str = ".claude"
+USAGE_PROBE_SHARED_DIRECTORY_NAME: str = "_shared"
+USAGE_PROBE_PACKAGE_DIRECTORY_NAME: str = "pr-loop"
+USAGE_PROBE_SCRIPTS_DIRECTORY_NAME: str = "scripts"
+USAGE_PROBE_FILENAME: str = "codex_usage_probe.py"
 SOL_SESSION_ID_METAVAR: str = "SESSION_ID"
 SOL_CODEX_TIMEOUT_SECONDS: float = 120.0
 SOL_USAGE_PROBE_TIMEOUT_SECONDS: float = 30.0

package/_shared/advisor/scripts/tests/test_codex_sol_advisor.py

@@ -31,9 +31,8 @@ def _load_sol_module() -> ModuleType:
 sol_advisor = _load_sol_module()
 SCRIPTS_ROOT = Path(__file__).parent.parent
 USAGE_PROBE_PATH = (
-    SCRIPTS_ROOT.parents[2]
-    / "skills"
-    / "codex-review"
+    SCRIPTS_ROOT.parents[1]
+    / "pr-loop"
     / "scripts"
     / "codex_usage_probe.py"
 )
@@ -134,8 +133,8 @@ def test_resolve_usage_probe_path_uses_supplied_home_directory(tmp_path: Path) -
     assert probe_path == (
         tmp_path
         / ".claude"
-        / "skills"
-        / "codex-review"
+        / "_shared"
+        / "pr-loop"
         / "scripts"
         / "codex_usage_probe.py"
     )

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "claude-dev-env",
-    "version": "2.22.0",
+    "version": "2.22.1",
     "description": "Claude Code development standards — rules, hooks, agents, commands, and skills",
     "type": "module",
     "bin": {