cc-dev-template 0.1.91 → 0.1.93

package/bin/install.js

@@ -252,7 +252,6 @@ const settingsFile = path.join(CLAUDE_DIR, 'settings.json');
 
 if (fs.existsSync(mergeSettingsPath)) {
   const configs = [
-    { file: 'read-guard-hook.json', name: 'Context guard for large reads' },
     { file: 'task-output-guard-hook.json', name: 'TaskOutput context guard' },
     { file: 'statusline-config.json', name: 'Custom status line' },
     // Spinner verbs - choose one (Star Trek or Factorio)
@@ -344,7 +343,9 @@ const deprecatedFiles = [
   path.join(CLAUDE_DIR, 'hooks', 'bash-overflow-guard.sh'),
   path.join(CLAUDE_DIR, 'scripts', 'bash-overflow-hook.json'),
   path.join(CLAUDE_DIR, 'scripts', 'env-config.json'),
-  path.join(CLAUDE_DIR, 'scripts', 'spinner-verbs-helldivers.json')
+  path.join(CLAUDE_DIR, 'scripts', 'spinner-verbs-helldivers.json'),
+  path.join(CLAUDE_DIR, 'scripts', 'read-guard.js'),
+  path.join(CLAUDE_DIR, 'scripts', 'read-guard-hook.json')
 ];
 
 deprecatedFiles.forEach(file => {
@@ -388,6 +389,24 @@ if (fs.existsSync(settingsFile)) {
       });
     }
 
+    // Remove read-guard hooks from settings
+    if (settings.hooks) {
+      ['PreToolUse'].forEach(hookType => {
+        if (settings.hooks[hookType] && Array.isArray(settings.hooks[hookType])) {
+          const originalLength = settings.hooks[hookType].length;
+          settings.hooks[hookType] = settings.hooks[hookType].filter(hook => {
+            const command = hook.hooks?.[0]?.command || '';
+            return !command.includes('read-guard');
+          });
+          if (settings.hooks[hookType].length < originalLength) {
+            console.log(`✓ Removed deprecated read-guard hook from ${hookType}`);
+            settingsModified = true;
+            cleanupPerformed = true;
+          }
+        }
+      });
+    }
+
     // Remove bash-overflow-guard hooks from settings
     if (settings.hooks) {
       ['PostToolUse'].forEach(hookType => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cc-dev-template",
-  "version": "0.1.91",
+  "version": "0.1.93",
   "description": "Structured AI-assisted development framework for Claude Code",
   "bin": {
     "cc-dev-template": "./bin/install.js"

package/src/skills/done/SKILL.md

@@ -14,11 +14,25 @@ This requires full conversation context. Handle it yourself rather than delegati
 
 ## Steps
 
-**1. Commit your work**
+**1. Apply session learnings**
 
-Write clear commit messages that explain what was accomplished. This IS your record of completed work.
+Review the session for anything that passes the "had I known this ahead of time" test — things that, known upfront, would have prevented mistakes or saved real time.
+
+If anything qualifies, read `references/apply-learnings.md` and follow its hierarchy to put the fix in the right place.
+
+If nothing qualifies (most sessions), skip this step entirely.
+
+**2. Prune memory**
+
+Read your MEMORY.md file. Check every entry against `references/memory-rules.md`.
+
+- Entries that are project-specific tribal knowledge (not personal) — relocate using the hierarchy from step 1
+- Entries that are stale, obvious, or discoverable from code — delete
+- Entries that duplicate CLAUDE.md or rules files — delete
+
+If MEMORY.md exceeds 200 lines, stop and deeply reflect. Something is wrong — you're storing things that don't belong.
 
-**2. Update `docs/CURRENT_WORK.md`**
+**3. Update `docs/CURRENT_WORK.md`**
 
 This file is forward-looking. Review it holistically and ensure it contains ONLY:
 
@@ -35,23 +49,9 @@ This file is forward-looking. Review it holistically and ensure it contains ONLY
 
 The file should be scannable in 30 seconds. If it takes longer, it's too long.
 
-**3. Apply session learnings**
+**4. Commit everything**
 
-Review the session for anything that passes the "had I known this ahead of time" test — things that, known upfront, would have prevented mistakes or saved real time.
-
-If anything qualifies, read `references/apply-learnings.md` and follow its hierarchy to put the fix in the right place.
-
-If nothing qualifies (most sessions), skip this step entirely.
-
-**4. Prune memory**
-
-Read your MEMORY.md file. Check every entry against `references/memory-rules.md`.
-
-- Entries that are project-specific tribal knowledge (not personal) — relocate using the hierarchy from step 3
-- Entries that are stale, obvious, or discoverable from code — delete
-- Entries that duplicate CLAUDE.md or rules files — delete
-
-If MEMORY.md exceeds 200 lines, stop and deeply reflect. Something is wrong — you're storing things that don't belong.
+Commit all session work plus any changes from steps 1-3 (rules, CLAUDE.md updates, memory relocations, CURRENT_WORK.md). Write clear commit messages that explain what was accomplished.
 
 **5. Push**
 

package/src/scripts/read-guard-hook.json

@@ -1,15 +0,0 @@
-{
-  "hooks": {
-    "PreToolUse": [
-      {
-        "matcher": "Read",
-        "hooks": [
-          {
-            "type": "command",
-            "command": "node ~/.claude/scripts/read-guard.js"
-          }
-        ]
-      }
-    ]
-  }
-}

package/src/scripts/read-guard.js

@@ -1,146 +0,0 @@
-#!/usr/bin/env node
-
-/**
- * read-guard.js - Block large file reads to preserve context
- *
- * This script intercepts Read tool calls and blocks reads of large files that would
- * consume excessive context. It forces Claude to use more targeted approaches like
- * Grep to find relevant sections, then Read with offset/limit parameters.
- *
- * Usage:
- *   echo '{"tool_input":{"file_path":"/path/to/file.js"}}' | node read-guard.js
- *
- * Input format (JSON via stdin):
- *   {
- *     "tool_input": {
- *       "file_path": "/absolute/path/to/file.js",
- *       "limit": 100,  // optional - if present, allow the read
- *       "offset": 0    // optional
- *     }
- *   }
- *
- * Exit codes:
- *   0 - Allow the read (outputs nothing or JSON to allow/deny)
- *
- * Environment variables:
- *   CONTEXT_GUARD_TOKEN_LIMIT - Token threshold (default: 5000)
- *
- * Token estimation:
- *   Rough heuristic: file_size_bytes / 4 ≈ tokens
- *   This catches obvious context bloat without needing a tokenizer.
- */
-
-const fs = require('fs');
-
-const TOKEN_THRESHOLD = parseInt(process.env.CONTEXT_GUARD_TOKEN_LIMIT || '5000', 10);
-
-/**
- * Read JSON from stdin
- * @returns {Promise<object | null>} - Parsed JSON or null if invalid
- */
-async function readStdin() {
-  return new Promise((resolve) => {
-    let data = '';
-
-    process.stdin.setEncoding('utf8');
-    process.stdin.on('data', chunk => {
-      data += chunk;
-    });
-
-    process.stdin.on('end', () => {
-      try {
-        resolve(JSON.parse(data));
-      } catch {
-        resolve(null);
-      }
-    });
-
-    process.stdin.on('error', () => {
-      resolve(null);
-    });
-  });
-}
-
-/**
- * Format file size for display
- * @param {number} bytes - File size in bytes
- * @returns {string} - Human-readable size
- */
-function formatSize(bytes) {
-  if (bytes < 1024) return `${bytes}B`;
-  if (bytes < 1024 * 1024) return `${(bytes / 1024).toFixed(1)}KB`;
-  return `${(bytes / (1024 * 1024)).toFixed(1)}MB`;
-}
-
-/**
- * Main execution
- */
-async function main() {
-  const inputData = await readStdin();
-
-  // If we can't parse input, allow the read
-  if (!inputData) {
-    process.exit(0);
-  }
-
-  const toolInput = inputData.tool_input || {};
-  const filePath = toolInput.file_path;
-  const limit = toolInput.limit;
-
-  // If limit is already specified, Claude is being intentional - allow it
-  if (limit !== undefined) {
-    process.exit(0);
-  }
-
-  // Image files: token consumption based on dimensions, not file size
-  const imageExtensions = ['.png', '.jpg', '.jpeg', '.gif', '.webp', '.svg', '.bmp', '.ico', '.tiff', '.tif'];
-  const ext = filePath.toLowerCase().slice(filePath.lastIndexOf('.'));
-  if (imageExtensions.includes(ext)) {
-    process.exit(0);
-  }
-
-  // If file doesn't exist or no path, let Read tool handle the error
-  if (!filePath || !fs.existsSync(filePath)) {
-    process.exit(0);
-  }
-
-  // Get file stats
-  let stats;
-  try {
-    stats = fs.statSync(filePath);
-  } catch {
-    // Can't stat file, let Read tool handle it
-    process.exit(0);
-  }
-
-  // Skip directories
-  if (stats.isDirectory()) {
-    process.exit(0);
-  }
-
-  // Estimate tokens from file size
-  const fileSizeBytes = stats.size;
-  const estimatedTokens = Math.ceil(fileSizeBytes / 4);
-
-  // If under threshold, allow the read
-  if (estimatedTokens <= TOKEN_THRESHOLD) {
-    process.exit(0);
-  }
-
-  // Block the read with guidance
-  const output = {
-    hookSpecificOutput: {
-      hookEventName: "PreToolUse",
-      permissionDecision: "deny",
-      permissionDecisionReason: `This file (~${estimatedTokens.toLocaleString()} tokens, ${formatSize(fileSizeBytes)}) would consume significant context. Use Grep to find relevant sections first, then Read with offset/limit parameters to read specific portions. If you need the full file, use multiple parallel Read calls with offset/limit to read it in chunks.`
-    }
-  };
-
-  console.log(JSON.stringify(output));
-  process.exit(0);
-}
-
-main().catch(() => {
-  // If anything goes wrong, allow the read to avoid disrupting workflow
-  process.exit(0);
-});