claude-dev-env 8.92.2 → 8.92.3

package/hooks/advisory/owned_pr_review_loop.py

@@ -53,7 +53,10 @@ try:
         ALL_PRESET_REPOSITORY_DIGESTS,
         ALL_SHELL_COMMAND_INPUT_KEYS,
         ALL_SHELL_TOOL_NAMES,
+        ALL_PYTHON_SCRIPTLESS_OPTIONS,
         ALREADY_EXISTS_MARKER,
+        BACKSLASH,
+        DOUBLED_BACKSLASH,
         GH_API_POST_METHOD,
         GH_API_PULLS_ENDPOINT_PATTERN,
         GH_API_SUBCOMMAND,
@@ -68,14 +71,28 @@ try:
         MCP_TOOL_NAME_PREFIX,
         NAME_DIGEST_TEXT_ENCODING,
         PULL_REQUEST_URL_PATTERN,
+        PYTHON_OPTION_AND_VALUE_WORD_COUNT,
+        PYTHON_OPTION_PREFIX,
+        PYTHON_SHORT_OPTION_LENGTH,
         REVIEW_LOOP_CONTEXT_TEMPLATE,
         SHELL_ARGUMENT_SEPARATOR,
     )
     from hooks_constants.post_tool_use_context import (
         write_post_tool_use_context_to_stdout,
     )
+    from hooks_constants.pr_lifecycle_skill_gate_constants import PYTHON_OPTIONS_WITH_VALUE
     from hooks_constants.pre_tool_use_stdin import read_hook_input_dictionary_from_stdin
-    from hooks_constants.shell_command_segments import command_tokens, split_into_segments
+    from hooks_constants.pull_request_proof_constants import (
+        ALL_PYTHON_PROGRAM_NAMES,
+        PULL_REQUEST_SCRIPT_CREATE_WORD,
+        PULL_REQUEST_SCRIPT_NAME,
+    )
+    from hooks_constants.pytest_invocation import unquoted_token
+    from hooks_constants.shell_command_segments import (
+        command_tokens,
+        split_into_segments,
+        token_basename,
+    )
     from hooks_constants.shell_command_wrappers import (
         all_wrapped_command_texts,
         segment_program_and_arguments,
@@ -127,8 +144,34 @@ def _is_gh_api_pulls_post(all_arguments: list[str]) -> bool:
     return sends_body
 
 
+def _python_script_index(all_arguments: list[str]) -> int | None:
+    index = 0
+    while index < len(all_arguments) and all_arguments[index].startswith(PYTHON_OPTION_PREFIX):
+        if all_arguments[index][:PYTHON_SHORT_OPTION_LENGTH] in ALL_PYTHON_SCRIPTLESS_OPTIONS:
+            return None
+        is_option_with_value = all_arguments[index] in PYTHON_OPTIONS_WITH_VALUE
+        index += PYTHON_OPTION_AND_VALUE_WORD_COUNT if is_option_with_value else 1
+    return index if index < len(all_arguments) else None
+
+
+def _is_pull_request_script_create(program_name: str, all_arguments: list[str]) -> bool:
+    all_script_arguments = all_arguments
+    if program_name != PULL_REQUEST_SCRIPT_NAME:
+        script_index = (
+            _python_script_index(all_arguments) if program_name in ALL_PYTHON_PROGRAM_NAMES else None
+        )
+        if script_index is None:
+            return False
+        if token_basename(unquoted_token(all_arguments[script_index])) != PULL_REQUEST_SCRIPT_NAME:
+            return False
+        all_script_arguments = all_arguments[script_index + 1 :]
+    return all_script_arguments[:1] == [PULL_REQUEST_SCRIPT_CREATE_WORD]
+
+
 def _segment_opens_pull_request(all_segment_tokens: list[str]) -> bool:
     program_name, all_arguments = segment_program_and_arguments(all_segment_tokens)
+    if _is_pull_request_script_create(program_name, all_arguments):
+        return True
     return program_name == GH_PROGRAM_NAME and (
         _is_gh_pr_create(all_arguments) or _is_gh_api_pulls_post(all_arguments)
     )
@@ -137,11 +180,18 @@ def _segment_opens_pull_request(all_segment_tokens: list[str]) -> bool:
 def command_creates_pull_request(command: str) -> bool:
     """Return True when any segment of the shell command opens a pull request.
 
-    Three shapes count::
+    Four shapes count::
 
         gh pr create --fill
         gh pr new --fill
         gh api repos/acme/widgets/pulls --input pr.json
+        python ~/.agents/skills/pull-request/scripts/pull_request.py create --repo acme/widgets
+
+    The publisher counts when ``python``, ``python3``, or ``py`` runs a
+    script named ``pull_request.py`` with ``create``, or the script runs
+    alone. Interpreter options such as ``-X utf8`` before the script are
+    skipped, and an unquoted Windows path such as
+    ``C:\\tools\\pull_request.py`` keeps its separators.
 
     A ``gh api`` call counts only when it targets the pulls collection. An
     explicit method (``-X POST``, ``-XPOST``, ``--method=POST``) decides
@@ -157,7 +207,11 @@ def command_creates_pull_request(command: str) -> bool:
     return any(
         _segment_opens_pull_request(each_segment)
         for each_command_text in all_wrapped_command_texts(command)
-        for each_segment in split_into_segments(command_tokens(each_command_text))
+        for each_command_tokens in (
+            command_tokens(each_command_text),
+            command_tokens(each_command_text.replace(BACKSLASH, DOUBLED_BACKSLASH)),
+        )
+        for each_segment in split_into_segments(each_command_tokens)
     )
 
 

package/hooks/advisory/test_owned_pr_review_loop.py

@@ -85,6 +85,13 @@ def _context_of(stdout: str) -> str:
         "gh api --method=POST repos/acme/widgets/pulls",
         "gh api -XPOST repos/acme/widgets/pulls",
         "gh api repos/acme/widgets/pulls -Ftitle=t -fhead=x",
+        'python "C:/Users/me/.agents/skills/pull-request/scripts/pull_request.py" create --repo acme/widgets',
+        "python3 ~/.agents/skills/pull-request/scripts/pull_request.py create --title t",
+        "py pull_request.py create --fill",
+        "git push && python pull_request.py create --repo acme/widgets 2>&1 | tail -5",
+        "python -X utf8 ~/.agents/skills/pull-request/scripts/pull_request.py create --fill",
+        "python -I -W ignore pull_request.py create",
+        "python C:\\Users\\me\\.agents\\skills\\pull-request\\scripts\\pull_request.py create",
     ],
 )
 def test_should_detect_gh_pr_create(command: str) -> None:
@@ -103,6 +110,12 @@ def test_should_detect_gh_pr_create(command: str) -> None:
         "gh api repos/acme/widgets/issues --input issue.json",
         "gh api --method GET repos/acme/widgets/pulls -f state=open",
         "gh api -X GET repos/acme/widgets/pulls -f state=open",
+        "python pull_request.py edit --repo acme/widgets --number 42",
+        "python other_script.py create",
+        "python pull_request.py",
+        "python -m pull_request.py create",
+        "python -c pull_request.py create",
+        "python C:\\Users\\me\\scripts\\my_pull_request.py create",
     ],
 )
 def test_should_not_detect_other_commands(command: str) -> None:
@@ -325,6 +338,16 @@ def test_main_should_emit_the_loop_after_a_gh_api_pulls_post() -> None:
     assert "jl-cmd-projects/claude-dev-env#42" in _context_of(stdout)
 
 
+def test_main_should_emit_the_loop_after_the_pull_request_publisher_creates() -> None:
+    publisher_command = (
+        'python "C:/Users/me/.agents/skills/pull-request/scripts/pull_request.py" create'
+        " --repo jl-cmd-projects/claude-dev-env --base main --head fix/x --title t"
+        " --body-file body.md 2>&1 | tail -5"
+    )
+    stdout = _run_hook(_shell_payload(publisher_command, f"{OWNED_PR_URL}\n"))
+    assert "jl-cmd-projects/claude-dev-env#42" in _context_of(stdout)
+
+
 def test_main_should_emit_the_loop_for_a_codex_shell_call() -> None:
     stdout = _run_hook(
         _shell_payload(

package/hooks/features/bash-dispatchers.md

@@ -23,7 +23,7 @@ This family rewrites a Git Bash command when path conversion would change a revi
 - `blocking/cloud_graphql_gate.py` runs inside that dispatcher on `PreToolUse`, matcher `Bash`, timeout `60` seconds, for the Bash and PowerShell tools, when `CLAUDE_CODE_REMOTE` is `true`.
 - `blocking/bash_post_call_dispatcher.py` runs on `PostToolUse`, matcher `Bash|PowerShell`, timeout `60` seconds in `hooks.json`.
 - `advisory/pr_done_reminder.py` runs inside that dispatcher on `PostToolUse`, matcher `Bash|PowerShell`, timeout `60` seconds. `ALL_BASH_POST_TOOL_USE_HOSTED_HOOK_ENTRIES` selects `Bash` and `PowerShell`.
-- `advisory/owned_pr_review_loop.py` runs on `PostToolUse`, matcher `Bash|PowerShell|exec_command|shell_command|mcp__.*__(create_pull_request|github_create_pull_request)`, timeout `10` seconds in `hooks.json`. It wakes on `gh pr create`, `gh pr new`, a POST to `gh api repos/<owner>/<repo>/pulls`, and an MCP pull request creation. The installer also registers it on Codex `PostToolUse` through `ALL_CODEX_NATIVE_POST_TOOL_USE_HOOK_SCRIPT_NAMES`.
+- `advisory/owned_pr_review_loop.py` runs on `PostToolUse`, matcher `Bash|PowerShell|exec_command|shell_command|mcp__.*__(create_pull_request|github_create_pull_request)`, timeout `10` seconds in `hooks.json`. It wakes on `gh pr create`, `gh pr new`, a POST to `gh api repos/<owner>/<repo>/pulls`, a `pull_request.py create` call to the pull request publisher, and an MCP pull request creation. The installer also registers it on Codex `PostToolUse` through `ALL_CODEX_NATIVE_POST_TOOL_USE_HOOK_SCRIPT_NAMES`.
 - `blocking/owned_pr_merge_stop_gate.py` runs on `Stop`, timeout `60` seconds in `hooks.json`. The installer also registers it on Codex `Stop` through `ALL_CODEX_NATIVE_STOP_HOOK_SCRIPT_NAMES`.
 
 ## Proving it

package/hooks/hooks_constants/owned_pr_review_loop_constants.py

@@ -39,6 +39,12 @@ __all__ = [
     "GH_SHORT_OPTION_LENGTH",
     "ALL_GH_API_SHORT_OPTIONS",
     "GH_OPTION_VALUE_SEPARATOR",
+    "PYTHON_OPTION_PREFIX",
+    "PYTHON_SHORT_OPTION_LENGTH",
+    "PYTHON_OPTION_AND_VALUE_WORD_COUNT",
+    "ALL_PYTHON_SCRIPTLESS_OPTIONS",
+    "BACKSLASH",
+    "DOUBLED_BACKSLASH",
     "MCP_TOOL_NAME_PREFIX",
     "ALL_MCP_CREATE_PULL_REQUEST_TOOL_SUFFIXES",
     "MCP_OWNER_INPUT_KEY",
@@ -85,6 +91,12 @@ GH_LONG_OPTION_PREFIX: str = "--"
 GH_SHORT_OPTION_LENGTH: int = 2
 ALL_GH_API_SHORT_OPTIONS: frozenset[str] = frozenset({"-f", "-F", "-X"})
 GH_OPTION_VALUE_SEPARATOR: str = "="
+PYTHON_OPTION_PREFIX: str = "-"
+PYTHON_SHORT_OPTION_LENGTH: int = 2
+PYTHON_OPTION_AND_VALUE_WORD_COUNT: int = 2
+ALL_PYTHON_SCRIPTLESS_OPTIONS: frozenset[str] = frozenset({"-m", "-c"})
+BACKSLASH: str = "\\"
+DOUBLED_BACKSLASH: str = "\\\\"
 
 MCP_TOOL_NAME_PREFIX: str = "mcp__"
 ALL_MCP_CREATE_PULL_REQUEST_TOOL_SUFFIXES: tuple[str, ...] = (

package/package.json

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