claude-dev-env 1.8.1 → 1.8.2

package/hooks/blocking/agent-execution-intent-gate.py

@@ -4,7 +4,6 @@
 from __future__ import annotations
 
 import json
-import os
 import sys
 
 from prompt_workflow_gate_core import (
@@ -41,30 +40,11 @@ def main() -> None:
     combined_text = f"{description}\n{prompt_text}"
 
     if not has_structured_execution_intent(tool_input):
-        allow_text_fallback = os.getenv(
-            "PROMPT_WORKFLOW_ALLOW_TEXT_INTENT_FALLBACK", ""
-        ).strip().lower() in {"1", "true", "yes"}
-        text_intent_detected = has_explicit_execution_intent(combined_text)
-        if allow_text_fallback and text_intent_detected:
-            print(
-                "PROMPT-WORKFLOW GATE: compatibility text-intent fallback used; "
-                "structured execution intent contract should be provided.",
-                file=sys.stderr,
-            )
-        else:
-            fallback_note = ""
-            if text_intent_detected:
-                print(
-                    "PROMPT-WORKFLOW GATE: text intent marker detected without structured "
-                    "execution intent contract.",
-                    file=sys.stderr,
-                )
-                fallback_note = " Legacy text marker was detected but is not sufficient."
+        if not has_explicit_execution_intent(combined_text):
             _deny(
                 "BLOCKED: Missing structured execution intent signal for Agent/Task launch. "
                 "Provide `tool_input.execution_intent: explicit` or "
                 "`tool_input.execution_intent_explicit: true`."
-                + fallback_note
             )
             sys.exit(0)
 

package/hooks/blocking/test_agent_execution_intent_gate.py

@@ -1,7 +1,6 @@
 """Tests for agent-execution-intent-gate hook."""
 
 import json
-import os
 import subprocess
 import sys
 from pathlib import Path
@@ -31,18 +30,23 @@ def test_denies_task_without_explicit_intent_marker() -> None:
     assert "structured execution intent signal" in response["hookSpecificOutput"]["permissionDecisionReason"]
 
 
-def test_denies_phrase_marker_without_structured_intent_contract() -> None:
+def test_allows_phrase_marker_with_scope_anchors() -> None:
     payload = {
         "tool_name": "Task",
         "tool_input": {
-            "prompt": "execution_intent: explicit and delegate now",
+            "prompt": (
+                "execution_intent: explicit\n"
+                "target_local_roots\n"
+                "target_canonical_roots\n"
+                "target_file_globs\n"
+                "comparison_basis\n"
+                "completion_boundary\n"
+            ),
             "description": "explicit delegation intent",
         },
     }
     result = _run_hook(payload)
-    response = json.loads(result.stdout)
-    assert response["hookSpecificOutput"]["permissionDecision"] == "deny"
-    assert "Missing structured execution intent signal" in response["hookSpecificOutput"]["permissionDecisionReason"]
+    assert result.stdout.strip() == ""
 
 
 def test_denies_when_scope_anchors_missing() -> None:
@@ -78,29 +82,3 @@ def test_allows_when_intent_and_scope_anchors_present() -> None:
     result = _run_hook(payload)
     assert result.stdout.strip() == ""
 
-
-def test_text_intent_fallback_is_logged_and_allowed_when_enabled() -> None:
-    payload = {
-        "tool_name": "Task",
-        "tool_input": {
-            "description": "delegate now",
-            "prompt": (
-                "execution_intent: explicit\n"
-                "target_local_roots\n"
-                "target_canonical_roots\n"
-                "target_file_globs\n"
-                "comparison_basis\n"
-                "completion_boundary\n"
-            ),
-        },
-    }
-    result = subprocess.run(
-        [sys.executable, str(SCRIPT_PATH)],
-        input=json.dumps(payload),
-        text=True,
-        capture_output=True,
-        check=False,
-        env={**os.environ, "PROMPT_WORKFLOW_ALLOW_TEXT_INTENT_FALLBACK": "1"},
-    )
-    assert result.stdout.strip() == ""
-    assert "compatibility text-intent fallback used" in result.stderr

package/package.json

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

package/skills/prompt-generator/SKILL.md

@@ -178,12 +178,36 @@ Required section list is immutable for this pipeline: `role`, `context`, `instru
 
 ### 11. Internal refinement object format (required by default mode)
 
-When step 10 is active (default), build this object internally to drive refinement and audit.
+When step 10 is active (default), build the refinement and audit state internally. Present the final merged prompt and a compact audit summary to the user. Keep the full internal object private unless the user explicitly asks for debug details.
 
-Present the final merged prompt and audit result to the user.
+**Default user-facing audit output (compact table):**
 
-Share this raw object only when the user explicitly asks for debug details.
-Do not expose the raw internal object by default.
+```
+**Audit: <overall_status>** | checklist_results: <pass_count>/14
+
+| Check | Status |
+|-------|--------|
+| structured_scoped_instructions | pass |
+| sequential_steps_present | pass |
+| positive_framing | pass |
+| acceptance_criteria_defined | pass |
+| safety_reversibility_language | pass |
+| no_destructive_shortcuts_guidance | pass |
+| concrete_output_contract | pass |
+| scope_boundary_present | pass |
+| explicit_scope_anchors_present | pass |
+| all_instructions_artifact_bound | pass |
+| no_ambiguous_scope_terms | pass |
+| completion_boundary_measurable | pass |
+| citation_grounding_policy_present | pass |
+| source_priority_rules_present | pass |
+```
+
+Replace `<overall_status>` with `pass` or `fail`. Replace `<pass_count>` with the actual count. Replace each row's `pass` with `fail` where applicable.
+
+**Debug mode (full JSON, shown only when user requests debug details):**
+
+When the user explicitly asks for debug details ("show debug", "show internal", "raw internal object", "pipeline object"), output the full internal object:
 
 ```json
 {
@@ -213,13 +237,21 @@ Do not expose the raw internal object by default.
       "corrective_edits",
       "retry_count"
     ]
+  },
+  "checklist_results": {
+    "<row_name>": {
+      "status": "pass|fail",
+      "evidence_quote": "exact quote used for verification",
+      "source_ref": "URL or local path",
+      "fix_if_fail": "concrete edit text (empty only if pass)"
+    }
   }
 }
 ```
 
 ### 12. Deterministic compliance checklist fields (audit reports all)
 
-If step 10 is active (default), the final audit report must include all fields below with `pass|fail`, direct quote evidence, and source reference:
+If step 10 is active (default), the audit must evaluate all 14 fields below. Each row name must appear as a literal substring in the user-facing output (the compact table satisfies this).
 
 - `structured_scoped_instructions`
 - `sequential_steps_present`
@@ -236,12 +268,14 @@ If step 10 is active (default), the final audit report must include all fields b
 - `citation_grounding_policy_present`
 - `source_priority_rules_present`
 
-For each checklist row, require:
+For each checklist row, maintain internally:
 - `status`: `pass|fail`
 - `evidence_quote`: exact quote used for verification
 - `source_ref`: URL or local path
 - `fix_if_fail`: concrete edit text (empty only if pass)
 
+The compact table (step 11) shows `status` per row. The `evidence_quote`, `source_ref`, and `fix_if_fail` fields are internal-only and appear only in debug mode.
+
 Scope quality rule for generated prompts:
 - Bind every major instruction to explicit artifacts from the scope block.
 - Prefer concrete references (paths/globs/comparisons) over context-relative wording.