claude-dev-env 1.8.0 → 1.8.2

package/bin/install.mjs

@@ -28,6 +28,13 @@ const INSTALL_GROUPS = {
     prompts: {
         description: 'Prompt engineering tools',
         skills: ['prompt-generator', 'agent-prompt'],
+        includeHookFiles: [
+            'blocking/agent-execution-intent-gate.py',
+            'blocking/prompt_workflow_gate_core.py',
+            'blocking/prompt-workflow-stop-guard.py',
+            'HOOK_SPECS_PROMPT_WORKFLOW.md',
+        ],
+        includeRules: ['prompt-workflow-context-controls.md'],
     },
     journal: {
         description: 'Session logging and memory',
@@ -162,20 +169,45 @@ function install(selectedGroups) {
     const allowedDirectories = selectedGroups
         ? new Set(selectedGroups.flatMap(groupName => INSTALL_GROUPS[groupName].includeDirectories || []))
         : null;
-    const shouldInstallHooks = selectedGroups
+    const shouldInstallAllHooks = selectedGroups
         ? selectedGroups.some(groupName => INSTALL_GROUPS[groupName].includeAllHooks)
         : true;
+    const allowedHookFiles = selectedGroups
+        ? new Set(selectedGroups.flatMap(groupName => INSTALL_GROUPS[groupName].includeHookFiles || []))
+        : null;
+    const allowedRules = selectedGroups
+        ? new Set(selectedGroups.flatMap(groupName => INSTALL_GROUPS[groupName].includeRules || []))
+        : null;
 
     const allInstalledFiles = [];
     const summary = {};
     for (const directory of CONTENT_DIRECTORIES) {
-        if (allowedDirectories && !allowedDirectories.has(directory)) continue;
+        const hasFullAccess = !allowedDirectories || allowedDirectories.has(directory);
+        const hasPartialRules = directory === 'rules' && allowedRules && allowedRules.size > 0;
+        if (!hasFullAccess && !hasPartialRules) continue;
         const sourceDir = join(PACKAGE_ROOT, directory);
         if (!existsSync(sourceDir)) continue;
         const destDir = join(CLAUDE_HOME, directory);
-        const stats = copyTree(sourceDir, destDir);
-        summary[directory] = stats;
-        allInstalledFiles.push(...stats.paths);
+        if (hasFullAccess) {
+            const stats = copyTree(sourceDir, destDir);
+            summary[directory] = stats;
+            allInstalledFiles.push(...stats.paths);
+        } else if (hasPartialRules) {
+            let rulesCreated = 0;
+            let rulesUpdated = 0;
+            for (const ruleFile of allowedRules) {
+                const sourcePath = join(sourceDir, ruleFile);
+                if (!existsSync(sourcePath)) continue;
+                const destPath = join(destDir, ruleFile);
+                mkdirSync(dirname(destPath), { recursive: true });
+                const existed = existsSync(destPath);
+                copyFileSync(sourcePath, destPath);
+                allInstalledFiles.push(destPath);
+                if (existed) { rulesUpdated++; } else { rulesCreated++; }
+                console.log(`  ${existed ? '\u21bb' : '\u2713'} ${join(directory, ruleFile)} (${existed ? 'updated' : 'new'})`);
+            }
+            summary[directory] = { created: rulesCreated, updated: rulesUpdated, paths: [] };
+        }
     }
     const skillsSource = join(PACKAGE_ROOT, 'skills');
     if (existsSync(skillsSource)) {
@@ -193,11 +225,18 @@ function install(selectedGroups) {
         summary.skills = { created: skillsCreated, updated: skillsUpdated, paths: skillPaths };
         allInstalledFiles.push(...skillPaths);
     }
-    if (shouldInstallHooks) {
+    const shouldInstallAnyHooks = shouldInstallAllHooks || (allowedHookFiles && allowedHookFiles.size > 0);
+    if (shouldInstallAnyHooks) {
         const hooksSource = join(PACKAGE_ROOT, 'hooks');
         if (existsSync(hooksSource)) {
             const hooksDestination = join(CLAUDE_HOME, 'hooks');
-            const filesToCopy = collectFiles(hooksSource).filter(file => !file.endsWith('hooks.json'));
+            const filesToCopy = collectFiles(hooksSource)
+                .filter(file => !file.endsWith('hooks.json'))
+                .filter(file => {
+                    if (shouldInstallAllHooks) return true;
+                    const relativePath = relative(hooksSource, file).replace(/\\/g, '/');
+                    return allowedHookFiles.has(relativePath);
+                });
             let hooksCreated = 0;
             let hooksUpdated = 0;
             for (const sourceFile of filesToCopy) {

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.0",
+    "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.