claude-dev-env 2.21.2 → 2.22.1

package/.agents/skills/orchestrator/SKILL.md

@@ -6,6 +6,7 @@ description: >-
   correction, plan, or stop. Triggers: '/orchestrator', 'orchestrator
   strategy', 'run with an orchestrator', 'executor-advisor mode',
   'orchestrator enforcement', 'agent routing', 'orchestrate'.
+disable-model-invocation: true
 ---
 
 # Orchestrator Strategy

package/.agents/skills/pr-cleanup/SKILL.md

@@ -48,6 +48,7 @@ with the user.
 | `pr-name-by-capability` | Find driver or motive words on reusable capability surfaces | Naming findings and rename directions |
 | `pr-small-cl` | Choose one coherent pull request or an ordered replacement stack | Focused boundary and dependencies |
 | `source-command-sr-loop` | Run `e-simplify`, then `e-code-review low --fix` until clean | Review passes, fixes, and validation |
+| `pr-summarizer` (repo-local) | When the repository under cleanup ships `.claude/skills/pr-summarizer/`, run it after Ready and post the secret-gist preview comment | Preview URL and comment confirmation |
 
 ## Task seeding
 
@@ -89,7 +90,12 @@ review passes, fixes, skips, tests, and commit SHAs.
 
 ### 6. Promote and report
 
-After the applicable gate passes, complete the [Finish report](#finish-report).
+After the applicable gate passes, when the repository under cleanup includes
+`.claude/skills/pr-summarizer/SKILL.md`, load and run that skill for each pull
+request promoted to Ready in this cleanup run. Post the secret gist preview as a
+pull request comment before completing the [Finish report](#finish-report).
+
+When the skill is absent, record `N/A` for the summary step and continue.
 
 ## Promotion gates
 
@@ -110,6 +116,9 @@ Reapply every relevant fix to the child. Rerun child tests, `e-simplify`, and
 `e-code-review` after the merge. Record the new child head and every validation
 result before promoting the child to Ready.
 
+When the repository ships `pr-summarizer`, run it on the child after child Ready
+promotion if the child is in scope for this cleanup run.
+
 Use `reference/task-seeds.md` and `reference/process-inventory.md` to record promotion evidence.
 
 ## Finish report
@@ -118,6 +127,7 @@ Use `reference/task-seeds.md` and `reference/process-inventory.md` to record pro
 - `pr-refinement` outcome.
 - `source-command-sr-loop` passes, commits, and validation results.
 - Parent-ready and child-ready SHAs when a child is promoted.
+- `pr-summarizer` preview URLs and comment confirmation for each Ready pull request, or `N/A` when the repo does not ship the skill.
 - Remaining hard block, or `null`.
 
 ## File index
@@ -125,5 +135,5 @@ Use `reference/task-seeds.md` and `reference/process-inventory.md` to record pro
 | Path | Purpose |
 |---|---|
 | `SKILL.md` | Hub for refinement, cleanup convergence, promotion gates, and reporting |
-| `reference/task-seeds.md` | Ordered session tasks for audits, delivery, validation, and promotion |
+| `reference/task-seeds.md` | Ordered session tasks for audits, delivery, validation, promotion, and summary |
 | `reference/process-inventory.md` | Process classes, evidence homes, and paired task checks |

package/.agents/skills/pr-cleanup/reference/process-inventory.md

@@ -8,8 +8,10 @@
 | Apply or disposition findings | judgment | `SKILL.md` and `task-seed:reference/task-seeds.md` | Changed diff or exact disposition | task-tool |
 | Validate parent head | deterministic | `task-seed:reference/task-seeds.md` | Scoped test and confirmation results | task-tool |
 | Promote parent and record exact Ready SHA | deterministic | `task-seed:reference/task-seeds.md` | Remote parent Ready state and SHA | task-tool |
+| Post parent PR summary gist comment | deterministic | `SKILL.md` and `task-seed:reference/task-seeds.md` | Preview URL, comment confirmation, or `N/A` | task-tool |
 | Merge exact parent SHA into child | deterministic | `task-seed:reference/task-seeds.md` | Child merge commit and parent SHA | task-tool |
 | Prove parent SHA ancestry | deterministic | `task-seed:reference/task-seeds.md` | `git merge-base --is-ancestor` exit code `0` | task-tool |
 | Reapply relevant fixes to child | judgment | `SKILL.md` and `task-seed:reference/task-seeds.md` | Child diff and reapplication record | task-tool |
 | Validate child and rerun confirmations | deterministic | `task-seed:reference/task-seeds.md` | New child-head tests, simplify, and review results | task-tool |
 | Promote child and report | deterministic | `task-seed:reference/task-seeds.md` | Child Ready state and finish report | task-tool |
+| Post child PR summary gist comment | deterministic | `SKILL.md` and `task-seed:reference/task-seeds.md` | Preview URL, comment confirmation, or `N/A` | task-tool |

package/.agents/skills/pr-cleanup/reference/task-seeds.md

@@ -11,10 +11,12 @@ each task with evidence from the stated gate.
 6. Select, apply, validate, or exactly disposition every actionable preflight finding in the one-agent parent cleanup worktree.
 7. Run parent scoped tests and cleanup/review confirmation checks.
 8. Promote the parent to Ready and record the exact remote `parent_ready_sha`.
-9. Create the child from its intended pre-parent base and merge the exact `parent_ready_sha`.
-10. Prove `parent_ready_sha` is an ancestor of the child head and record the exit code.
-11. Reapply every relevant preflight fix to the child.
-12. Run child scoped tests on the new child head.
-13. Rerun `e-simplify` and `e-code-review` on the new child head.
-14. Promote the child to Ready after all child evidence passes.
-15. Write the finish report with SHAs, evidence, states, and hard blocks.
+9. When `.claude/skills/pr-summarizer/SKILL.md` exists in the repository under cleanup, run that skill on the parent and post the gist preview comment; otherwise record `N/A`.
+10. Create the child from its intended pre-parent base and merge the exact `parent_ready_sha`.
+11. Prove `parent_ready_sha` is an ancestor of the child head and record the exit code.
+12. Reapply every relevant preflight fix to the child.
+13. Run child scoped tests on the new child head.
+14. Rerun `e-simplify` and `e-code-review` on the new child head.
+15. Promote the child to Ready after all child evidence passes.
+16. When the repository ships `pr-summarizer` and the child is in scope, run it on the child and post the gist preview comment; otherwise record `N/A`.
+17. Write the finish report with SHAs, evidence, summary preview URLs, states, and hard blocks.

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

@@ -5,6 +5,7 @@ description: >-
   it before substantive work, completion, commits, or when stuck. Triggers:
   'team-advisor', 'team advisor', 'second opinion', 'advisor', 'consult',
   'verify', 'validate', 'commit', 'push'.
+disable-model-invocation: true
 ---
 
 # Team Advisor
@@ -29,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/AGENTS.md

@@ -1,4 +1,4 @@
-# Scope
+Avoid all redundant text for all output. IE: instead of "the review found 2 real issues", simply state "the review found 2 issues".
 
 ## User-facing language hierarchy
 

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.21.2",
+    "version": "2.22.1",
     "description": "Claude Code development standards — rules, hooks, agents, commands, and skills",
     "type": "module",
     "bin": {