claude-dev-env 8.35.3 → 8.36.1

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

@@ -52,6 +52,7 @@ For Claude Projects facts or reported coordinator mechanisms, read [platform evi
 ### Orient and retain the goals
 
 Read the current user message and applicable project instructions.
+Read `~/.claude/rules/long-horizon-autonomy.md` and `~/.claude/rules/workers-done-before-complete.md` before the first dispatch. Invoking this skill does not load those two rules, so this read does.
 At startup or after context loss, inspect `.orchestrator/active-runs/` under the supplied project directory.
 Use an alternate registry only when loaded instructions or the runtime provide its exact locator.
 Verify the startup loader pointer described in run state before claiming recovery from a cold session.

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

@@ -28,7 +28,7 @@ Keep current authorization and configured model routing when restoring work.
 ## Process
 
 1. Read the current message and loaded instructions. Load the orchestrator entrypoint.
-2. Resolve the run from its supplied locator or `.orchestrator/active-runs/` under the project directory.
+   Read `~/.claude/rules/long-horizon-autonomy.md` and `~/.claude/rules/workers-done-before-complete.md`. Invoking this skill does not load those two rules, so this read does.2. Resolve the run from its supplied locator or `.orchestrator/active-runs/` under the project directory.
    Read the run record's owner and optional wake metadata first.
    For a recorded one-shot firing, confirm both the invocation's wake identity and current root ownership.
    Immediately run `begin-firing` with its explicit `--status-file` and `--run-slug` through [optional scheduling](../orchestrator/reference/scheduling.md).

package/hooks/hooks_constants/test_piped_pytest_blocker_constants.py

@@ -0,0 +1,14 @@
+from hooks_constants import piped_pytest_blocker_constants as constants
+
+
+def test_pytest_programs_and_interpreters_cover_supported_invocations() -> None:
+    assert {"pytest", "pytest.exe", "py.test"} <= constants.ALL_PYTEST_PROGRAM_BASENAMES
+    assert constants.PYTHON_INTERPRETER_BASENAME_PATTERN.fullmatch("python3.12.exe")
+    assert (constants.MODULE_RUN_FLAG, constants.PYTEST_MODULE_NAME) == ("-m", "pytest")
+
+
+def test_pipe_and_wrapper_tokens_preserve_command_boundaries() -> None:
+    assert {"|", "|&"} <= constants.ALL_PIPE_OPERATOR_TOKENS
+    assert ";;" in constants.ALL_SEGMENT_RESET_OPERATOR_TOKENS
+    assert "-u" in constants.ALL_WRAPPER_OPTION_GRAMMARS_BY_NAME["sudo"].all_value_taking_options
+    assert constants.ALL_WRAPPER_OPTION_GRAMMARS_BY_NAME["timeout"].leading_operand_count == 1

package/hooks/hooks_constants/test_verify_before_acting_constants.py

@@ -0,0 +1,18 @@
+from hooks_constants import verify_before_acting_constants as constants
+
+
+def test_mutating_tool_and_mcp_names_are_classified() -> None:
+    assert {"Write", "Edit"} <= constants.ALL_ALWAYS_MUTATING_TOOL_NAMES
+    assert {"Bash", "PowerShell"} == constants.ALL_SHELL_TOOL_NAMES
+    assert constants.MCP_ACTION_WORD_SPLIT_PATTERN.split("create_pull_request") == [
+        "create",
+        "pull",
+        "request",
+    ]
+    assert "create" in constants.ALL_MCP_MUTATING_VERBS
+
+
+def test_hedge_pattern_matches_a_hedged_claim_only() -> None:
+    assert constants.HEDGE_PATTERN.search("Perhaps the command writes a file.")
+    assert constants.HEDGE_PATTERN.search("I might be mistaken.")
+    assert constants.HEDGE_PATTERN.search("I checked the command output.") is None

package/package.json

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

package/rules/archiving-agent-config.md

@@ -1,3 +1,14 @@
+---
+paths:
+  - "**/rules/**"
+  - "**/rules-archived/**"
+  - "**/skills/**"
+  - "**/skills-archived/**"
+  - "**/agents/**"
+  - "**/commands/**"
+  - "**/ever-shipped-skills.mjs"
+---
+
 # Archiving a Rule, Skill, Agent, or Command
 
 **When this applies:** Pruning agent configuration in this package — moving a

package/rules/bdd.md

@@ -1,3 +1,12 @@
+---
+paths:
+  - "**/*.py"
+  - "**/*.mjs"
+  - "**/*.js"
+  - "**/*.ts"
+  - "**/*.tsx"
+---
+
 # BDD (discovery-driven development)
 
 **Canonical detail:** `~/.claude/system-prompts/software-engineer.xml` → `<behavior_protocol>`.

package/rules/build-needs-a-user.md

@@ -1,3 +1,14 @@
+---
+paths:
+  - "**/scripts/**"
+  - "**/hooks/**"
+  - "**/bin/**"
+  - "**/ci/**"
+  - "**/tools/**"
+  - "**/skills/**"
+  - "**/commands/**"
+---
+
 # Build Needs a User
 
 **When this applies:** Before you build a new tool, script, hook, gate, skill,

package/rules/code-standards.md

@@ -1,3 +1,13 @@
+---
+paths:
+  - "**/*.py"
+  - "**/*.mjs"
+  - "**/*.js"
+  - "**/*.ts"
+  - "**/*.tsx"
+  - "**/*.ps1"
+---
+
 # Code Standards
 
 > **Canonical review contract:** [`CODE_RULES.md`](../docs/CODE_RULES.md) — the human and AI review contract for code quality, loaded on demand.

package/rules/failure-blast-radius.md

@@ -1,3 +1,12 @@
+---
+paths:
+  - "**/*.py"
+  - "**/*.mjs"
+  - "**/*.js"
+  - "**/*.ts"
+  - "**/*.ps1"
+---
+
 # Failure Blast Radius
 
 **When this applies:** Batch code that processes assets, rows, accounts, messages, or files where one member can fail while the others are fine.

package/rules/flag-non-breaking-findings.md

@@ -1,3 +1,14 @@
+---
+paths:
+  - "**/hooks/**"
+  - "**/policy_lint/**"
+  - "**/repository_checks/**"
+  - "**/.pre-commit-config.yaml"
+  - "**/.githooks/**"
+  - "**/.husky/**"
+  - "**/.github/workflows/**"
+---
+
 # Flag Non-Breaking Findings
 
 **When this applies:** Writing or changing a local gate — a git hook, a

package/rules/long-horizon-autonomy.md

@@ -1,3 +1,9 @@
+---
+paths:
+  - "**/skills/orchestrator/**"
+  - "**/skills/orchestrator-refresh/**"
+---
+
 # Long-Horizon Autonomy
 
 Source: [Anthropic - Prompting Claude Fable 5](https://platform.claude.com/docs/en/build-with-claude/prompt-engineering/prompting-claude-fable-5)

package/rules/prefer-existing-tools.md

@@ -1,3 +1,16 @@
+---
+paths:
+  - "**/scripts/**"
+  - "**/hooks/**"
+  - "**/bin/**"
+  - "**/ci/**"
+  - "**/tools/**"
+  - "**/skills/**"
+  - "**/package.json"
+  - "**/pyproject.toml"
+  - "**/requirements*.txt"
+---
+
 # Prefer Existing Tools
 
 **When this applies:** Before you build a tool, check, scanner, script, or library of your own.

package/rules/workers-done-before-complete.md

@@ -1,3 +1,9 @@
+---
+paths:
+  - "**/skills/orchestrator/**"
+  - "**/skills/orchestrator-refresh/**"
+---
+
 # Workers Done Before Complete
 
 Before marking a task `completed` that spawned workers — subagents, workflow agents, or background shells — confirm both: every worker has finished, and each worker's result is merged into run state (`state.json`, `pr-converge-state.json`, the task list, or whatever record the task keeps). A worker still running, or one whose output never landed in run state, keeps the task `in_progress`: list the live workers, report any dead or hung one as a finding rather than dropping it in silence, and schedule a wakeup so the run picks the workers back up before the task closes.

package/scripts/tests/test_rule_load_scopes.py

@@ -0,0 +1,77 @@
+"""Which rules load in every session, and how skill-scoped rules still reach their skill."""
+
+from pathlib import Path
+
+PACKAGE_ROOT = Path(__file__).resolve().parents[2]
+RULES_DIRECTORY = PACKAGE_ROOT / "rules"
+SKILLS_DIRECTORY = PACKAGE_ROOT / ".agents" / "skills"
+
+ALWAYS_ON_RULE_NAMES = frozenset(
+    {
+        "AGENTS.md",
+        "CLAUDE.md",
+        "agent-merges-its-own-green-pull-request.md",
+        "asd-ste100-language.md",
+        "ci-owns-the-gate.md",
+        "cleanup-temp-files.md",
+        "correction-lens.md",
+        "destructive-commands.md",
+        "durable-post-artifacts.md",
+        "explore-thoroughly.md",
+        "filesystem-search.md",
+        "gh-cli-conventions.md",
+        "git-workflow.md",
+        "memory-stores-durable-facts.md",
+        "no-contrast-framing.md",
+        "question-presentation.md",
+        "re-stage-before-commit.md",
+        "research-mode.md",
+        "review-closure-is-a-check.md",
+        "shell-invocation.md",
+        "verify-before-asking.md",
+        "verify-runtime-state.md",
+    }
+)
+
+SKILL_SCOPED_RULES = {
+    "long-horizon-autonomy.md": ("orchestrator", "orchestrator-refresh"),
+    "workers-done-before-complete.md": ("orchestrator", "orchestrator-refresh"),
+}
+
+
+def _declares_paths(rule_path: Path) -> bool:
+    lines = rule_path.read_text(encoding="utf-8").splitlines()
+    if not lines or lines[0] != "---":
+        return False
+    for each_line in lines[1:]:
+        if each_line == "---":
+            return False
+        if each_line.startswith("paths:"):
+            return True
+    return False
+
+
+def test_only_allowlisted_rules_load_in_every_session() -> None:
+    unscoped_rule_names = {
+        each_rule.name
+        for each_rule in RULES_DIRECTORY.glob("*.md")
+        if not _declares_paths(each_rule)
+    }
+    assert unscoped_rule_names == ALWAYS_ON_RULE_NAMES
+
+
+def test_skill_scoped_rules_carry_paths_on_their_skill_directories() -> None:
+    for each_rule_name, all_skill_names in SKILL_SCOPED_RULES.items():
+        rule_text = (RULES_DIRECTORY / each_rule_name).read_text(encoding="utf-8")
+        assert _declares_paths(RULES_DIRECTORY / each_rule_name)
+        for each_skill_name in all_skill_names:
+            assert f'"**/skills/{each_skill_name}/**"' in rule_text
+
+
+def test_each_scoping_skill_tells_the_agent_to_read_its_rules() -> None:
+    for each_rule_name, all_skill_names in SKILL_SCOPED_RULES.items():
+        for each_skill_name in all_skill_names:
+            skill_text = (SKILLS_DIRECTORY / each_skill_name / "SKILL.md").read_text(
+                encoding="utf-8"
+            )
+            assert f"~/.claude/rules/{each_rule_name}" in skill_text