claude-dev-env 8.23.0 → 8.23.2

package/hooks/hooks_constants/skill_loaded_reminder_constants.py

@@ -9,12 +9,12 @@ __all__ = [
     "COMPACTION_REMINDER",
     "COMPACTION_SOURCE",
     "NOT_LOADED_REMINDER",
-    "POTETO_MODE_SKILL_NAME",
+    "ALL_POTETO_MODE_SKILL_NAMES",
     "PRE_TOOL_USE_EVENT_NAME",
     "PROMPT_SEPARATOR",
     "SESSION_START_EVENT_NAME",
     "SKILL_TOOL_NAME",
-    "SLASH_COMMAND_MARKER",
+    "ALL_SLASH_COMMAND_MARKERS",
     "SUBAGENT_START_EVENT_NAME",
     "CLAUDE_SUBAGENT_PROMPT_PREFIX",
     "CODEX_SUBAGENT_PROMPT_PREFIX",
@@ -34,9 +34,12 @@ WORKFLOW_SUBAGENT_TYPE = "workflow-subagent"
 
 ALL_SELF_LOADING_SUBAGENT_TYPES = frozenset({"pstack:poteto-agent", "poteto-agent"})
 
-POTETO_MODE_SKILL_NAME = "pstack:poteto-mode"
+ALL_POTETO_MODE_SKILL_NAMES = ("poteto-mode", "pstack:poteto-mode")
 SKILL_TOOL_NAME = "Skill"
-SLASH_COMMAND_MARKER = "<command-name>/pstack:poteto-mode</command-name>"
+ALL_SLASH_COMMAND_MARKERS = (
+    "<command-name>/poteto-mode</command-name>",
+    "<command-name>/pstack:poteto-mode</command-name>",
+)
 PROMPT_SEPARATOR = "\n\n"
 
 ASSISTANT_ENTRY_TYPE = "assistant"
@@ -45,11 +48,11 @@ TOOL_USE_BLOCK_TYPE = "tool_use"
 COMPACT_BOUNDARY_SUBTYPE = "compact_boundary"
 
 CLAUDE_SUBAGENT_PROMPT_PREFIX = (
-    "Before any other work, invoke the pstack:poteto-mode skill with the Skill tool. "
+    "Before any other work, invoke the poteto-mode skill with the Skill tool. "
     "Every prompt you write for a subagent opens with the same instruction."
 )
 
-CODEX_SUBAGENT_PROMPT_PREFIX = "$pstack:poteto-mode"
+CODEX_SUBAGENT_PROMPT_PREFIX = "$poteto-mode"
 
 ALL_SPAWN_PROMPT_FIELDS_AND_PREFIXES_BY_TOOL_NAME = {
     "Agent": ("prompt", CLAUDE_SUBAGENT_PROMPT_PREFIX),
@@ -58,13 +61,13 @@ ALL_SPAWN_PROMPT_FIELDS_AND_PREFIXES_BY_TOOL_NAME = {
 }
 
 NOT_LOADED_REMINDER = (
-    "The pstack:poteto-mode skill is not loaded in this context. "
-    "Your next tool call is the Skill tool with skill pstack:poteto-mode, before any other "
+    "The poteto-mode skill is not loaded in this context. "
+    "Your next tool call is the Skill tool with skill poteto-mode, before any other "
     "tool call."
 )
 
 COMPACTION_REMINDER = (
-    "The context was just compacted and the pstack:poteto-mode skill rules were dropped. "
-    "Your next tool call is the Skill tool with skill pstack:poteto-mode, before any other "
+    "The context was just compacted and the poteto-mode skill rules may have been dropped. "
+    "Your next tool call is the Skill tool with skill poteto-mode, before any other "
     "tool call. Then re-read the request that started this session and continue."
 )

package/hooks/session/skill_loaded_reminder.py

@@ -3,10 +3,10 @@
 
 ::
 
-    Agent or Codex spawn_agent, skill not named -> prompt opens with "invoke pstack:poteto-mode"
-    Workflow script helper starts               -> "invoke pstack:poteto-mode now"
-    context compacted mid-run                    -> "invoke pstack:poteto-mode again"
-    user turn, skill not loaded since compacting -> "invoke pstack:poteto-mode now"
+    Agent or Codex spawn_agent, skill not named -> prompt opens with "invoke poteto-mode"
+    Workflow script helper starts               -> "invoke poteto-mode now"
+    context compacted mid-run                    -> "invoke poteto-mode again"
+    user turn, skill not loaded since compacting -> "invoke poteto-mode now"
     user turn, skill already loaded              -> nothing
 
 A session that loaded the skill hears nothing more until a compaction drops it.
@@ -31,12 +31,12 @@ from hooks_constants.skill_loaded_reminder_constants import (
     COMPACTION_REMINDER,
     COMPACTION_SOURCE,
     NOT_LOADED_REMINDER,
-    POTETO_MODE_SKILL_NAME,
+    ALL_POTETO_MODE_SKILL_NAMES,
     PRE_TOOL_USE_EVENT_NAME,
     PROMPT_SEPARATOR,
     SESSION_START_EVENT_NAME,
     SKILL_TOOL_NAME,
-    SLASH_COMMAND_MARKER,
+    ALL_SLASH_COMMAND_MARKERS,
     SUBAGENT_START_EVENT_NAME,
     ALL_SPAWN_PROMPT_FIELDS_AND_PREFIXES_BY_TOOL_NAME,
     TOOL_USE_BLOCK_TYPE,
@@ -58,9 +58,9 @@ def subagent_input_with_poteto_mode(
     ::
 
         Agent        {"prompt": "Reply leaf."}   -> {"prompt": "Before any ...\\n\\nReply leaf."}
-        spawn_agent  {"message": "Fix it."}      -> {"message": "$pstack:poteto-mode\\n\\nFix it."}
-        Agent        {"prompt": "Invoke pstack:poteto-mode, then ..."}  -> None
-        Agent        {"subagent_type": "pstack:poteto-agent", ...}      -> None, it loads the skill
+        spawn_agent  {"message": "Fix it."}      -> {"message": "$poteto-mode\\n\\nFix it."}
+        Agent        {"prompt": "Invoke poteto-mode, then ..."}  -> None
+        Agent        {"subagent_type": "poteto-agent", ...}      -> None, it loads the skill
 
     Claude Code spawns through Agent or Task and Codex through spawn_agent, so
     each tool name carries its own prompt field and invocation text.
@@ -71,7 +71,7 @@ def subagent_input_with_poteto_mode(
     """
     field_name, invocation_prefix = ALL_SPAWN_PROMPT_FIELDS_AND_PREFIXES_BY_TOOL_NAME[tool_name]
     prompt = all_tool_input_fields.get(field_name)
-    if not isinstance(prompt, str) or POTETO_MODE_SKILL_NAME in prompt:
+    if not isinstance(prompt, str) or ALL_POTETO_MODE_SKILL_NAMES[0] in prompt:
         return None
     if all_tool_input_fields.get("subagent_type") in ALL_SELF_LOADING_SUBAGENT_TYPES:
         return None
@@ -84,7 +84,7 @@ def _invokes_poteto_mode(all_entry_fields: dict[str, object]) -> bool:
         return False
     content_blocks = message.get("content")
     if all_entry_fields.get("type") == USER_ENTRY_TYPE and isinstance(content_blocks, str):
-        return SLASH_COMMAND_MARKER in content_blocks
+        return any(each_marker in content_blocks for each_marker in ALL_SLASH_COMMAND_MARKERS)
     if all_entry_fields.get("type") != ASSISTANT_ENTRY_TYPE or not isinstance(content_blocks, list):
         return False
     return any(
@@ -92,7 +92,7 @@ def _invokes_poteto_mode(all_entry_fields: dict[str, object]) -> bool:
         and each_block.get("type") == TOOL_USE_BLOCK_TYPE
         and each_block.get("name") == SKILL_TOOL_NAME
         and isinstance(each_block.get("input"), dict)
-        and each_block["input"].get("skill") == POTETO_MODE_SKILL_NAME
+        and each_block["input"].get("skill") in ALL_POTETO_MODE_SKILL_NAMES
         for each_block in content_blocks
     )
 
@@ -100,7 +100,7 @@ def _invokes_poteto_mode(all_entry_fields: dict[str, object]) -> bool:
 def _marker_entry(transcript_line: str) -> dict[str, object] | None:
     if (
         COMPACT_BOUNDARY_SUBTYPE not in transcript_line
-        and POTETO_MODE_SKILL_NAME not in transcript_line
+        and ALL_POTETO_MODE_SKILL_NAMES[0] not in transcript_line
     ):
         return None
     try:
@@ -121,9 +121,9 @@ def is_poteto_mode_loaded(all_transcript_lines: Iterable[str]) -> bool:
 
     ::
 
-        Skill(pstack:poteto-mode) ... Read ... Edit               -> True
-        /pstack:poteto-mode typed as a command ... Read           -> True
-        Skill(pstack:poteto-mode) ... compact_boundary ... Read   -> False
+        Skill(poteto-mode) ... Read ... Edit               -> True
+        /poteto-mode typed as a command ... Read           -> True
+        Skill(poteto-mode) ... compact_boundary ... Read   -> False
         Read ... Edit                                             -> False
 
     Only lines naming the skill or a compaction are parsed, so a long transcript

package/hooks/session/test_skill_loaded_reminder.py

@@ -27,7 +27,7 @@ SKILL_CALL_LINE = json.dumps(
                 {
                     "type": "tool_use",
                     "name": "Skill",
-                    "input": {"skill": "pstack:poteto-mode"},
+                    "input": {"skill": "poteto-mode"},
                 }
             ],
         },
@@ -38,7 +38,7 @@ TYPED_COMMAND_LINE = json.dumps(
         "type": "user",
         "message": {
             "role": "user",
-            "content": "<command-name>/pstack:poteto-mode</command-name>",
+            "content": "<command-name>/poteto-mode</command-name>",
         },
     }
 )
@@ -58,7 +58,7 @@ TOOL_RESULT_QUOTING_THE_SKILL_LINE = json.dumps(
             "content": [
                 {
                     "type": "tool_result",
-                    "content": 'SKILL = "pstack:poteto-mode" and <command-name>/pstack:poteto-mode</command-name>',
+                    "content": 'SKILL = "poteto-mode" and <command-name>/poteto-mode</command-name>',
                 }
             ]
         },
@@ -107,7 +107,7 @@ class TestSubagentSpawn:
         rewritten_input = hook_output["updatedInput"]
         assert hook_output["permissionDecision"] == "allow"
         assert rewritten_input["prompt"].startswith(
-            "Before any other work, invoke the pstack:poteto-mode skill with the Skill tool."
+            "Before any other work, invoke the poteto-mode skill with the Skill tool."
         )
         assert rewritten_input["prompt"].endswith("\n\nList every caller of main.")
         assert rewritten_input["subagent_type"] == "Explore"
@@ -115,10 +115,10 @@ class TestSubagentSpawn:
 
     def test_the_legacy_task_tool_name_is_rewritten_too(self) -> None:
         emitted = json.loads(_run_main(_agent_call({"prompt": "Go."}, tool_name="Task")))
-        assert "pstack:poteto-mode" in emitted["hookSpecificOutput"]["updatedInput"]["prompt"]
+        assert "poteto-mode" in emitted["hookSpecificOutput"]["updatedInput"]["prompt"]
 
     def test_a_prompt_that_already_names_the_skill_is_left_alone(self) -> None:
-        assert _run_main(_agent_call({"prompt": "Invoke pstack:poteto-mode first. Then go."})) == ""
+        assert _run_main(_agent_call({"prompt": "Invoke poteto-mode first. Then go."})) == ""
 
     def test_a_poteto_agent_is_left_alone_because_it_loads_the_skill_itself(self) -> None:
         assert (
@@ -161,14 +161,14 @@ class TestCodexSpawn:
             )
         )
         rewritten_input = emitted["hookSpecificOutput"]["updatedInput"]
-        assert rewritten_input["message"] == "$pstack:poteto-mode\n\nFix the failing test."
+        assert rewritten_input["message"] == "$poteto-mode\n\nFix the failing test."
         assert rewritten_input["model"] == "gpt-6-luna"
         assert "prompt" not in rewritten_input
 
     def test_a_codex_spawn_message_that_already_mentions_the_skill_is_left_alone(self) -> None:
         assert (
             _run_main(
-                _agent_call({"message": "$pstack:poteto-mode\n\nFix it."}, tool_name="spawn_agent")
+                _agent_call({"message": "$poteto-mode\n\nFix it."}, tool_name="spawn_agent")
             )
             == ""
         )
@@ -181,6 +181,14 @@ class TestIsPotetoModeLoaded:
     def test_a_typed_slash_command_loads_it(self) -> None:
         assert reminder.is_poteto_mode_loaded([TYPED_COMMAND_LINE, READ_CALL_LINE])
 
+    def test_a_prefixed_skill_call_loads_it(self) -> None:
+        prefixed_skill_call_line = SKILL_CALL_LINE.replace("poteto-mode", "pstack:poteto-mode")
+        assert reminder.is_poteto_mode_loaded([prefixed_skill_call_line])
+
+    def test_a_prefixed_typed_command_loads_it(self) -> None:
+        prefixed_command_line = TYPED_COMMAND_LINE.replace("/poteto-mode", "/pstack:poteto-mode")
+        assert reminder.is_poteto_mode_loaded([prefixed_command_line])
+
     def test_a_compaction_after_the_skill_call_drops_it(self) -> None:
         assert not reminder.is_poteto_mode_loaded(
             [SKILL_CALL_LINE, COMPACT_BOUNDARY_LINE, READ_CALL_LINE]
@@ -193,7 +201,7 @@ class TestIsPotetoModeLoaded:
 
     def test_a_plain_user_prompt_does_not_load_it(self) -> None:
         plain_prompt_line = json.dumps(
-            {"type": "user", "message": {"content": "Read pstack:poteto-mode docs."}}
+            {"type": "user", "message": {"content": "Read poteto-mode docs."}}
         )
         assert not reminder.is_poteto_mode_loaded([plain_prompt_line])
 
@@ -210,7 +218,7 @@ class TestReminderFor:
         hook_output = emitted["hookSpecificOutput"]
         assert hook_output["hookEventName"] == "UserPromptSubmit"
         assert hook_output["additionalContext"] == NOT_LOADED_REMINDER
-        assert "pstack:poteto-mode" in NOT_LOADED_REMINDER
+        assert "poteto-mode" in NOT_LOADED_REMINDER
 
     def test_a_user_turn_in_a_session_that_loaded_the_skill_prints_nothing(
         self, tmp_path: Path
@@ -229,7 +237,7 @@ class TestReminderFor:
         hook_output = emitted["hookSpecificOutput"]
         assert hook_output["hookEventName"] == "SessionStart"
         assert hook_output["additionalContext"] == COMPACTION_REMINDER
-        assert "pstack:poteto-mode" in COMPACTION_REMINDER
+        assert "poteto-mode" in COMPACTION_REMINDER
 
     def test_a_resumed_session_keeps_its_context_and_prints_nothing(self) -> None:
         assert _run_main({"hook_event_name": "SessionStart", "source": "resume"}) == ""

package/package.json

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

package/rules/correction-lens.md

@@ -63,10 +63,13 @@ Count the repetitions.
 
 ## Where the control lands
 
-A control lands in this package, so every session that loads this environment
-carries it, on any machine, under any model. A control written into one
-project's instructions reaches that project alone, and a control written into a
-chat reply reaches that conversation alone.
+A control lands in the repository whose code, CI, or pipeline it guards. A
+control for one repository's workflows, review bots, pipelines, or agents lands
+in that repository. This package takes only repo-agnostic controls, the
+environment and development pieces any repository could use, so every session
+that loads this environment carries them. A control written into one project's
+instructions reaches that project alone, and a control written into a chat reply
+reaches that conversation alone.
 
 Memory holds the decision. The control holds the behavior. A correction that
 produced a memory file and nothing else has been recorded and never encoded, so
@@ -120,8 +123,9 @@ Effort rules out no layer; a missing capability does, and you name which one.
 The same correction arriving twice means the layer was too low. Move it up one
 layer and say so.
 
-Land the control in the shared environment package, so every session carries
-it. A memory file records the decision and encodes nothing.
+Land the control in the repository whose code, CI, or pipeline it guards. The
+shared environment package takes only repo-agnostic controls that any
+repository could use. A memory file records the decision and encodes nothing.
 ```
 
 ## Sibling rules