@torka/claude-qol 0.2.1 → 0.3.0

package/install.js

@@ -13,11 +13,17 @@ const colors = {
   green: '\x1b[32m',
   yellow: '\x1b[33m',
   blue: '\x1b[34m',
+  cyan: '\x1b[36m',
   red: '\x1b[31m',
   reset: '\x1b[0m',
   bold: '\x1b[1m',
 };
 
+// Files that should not be overwritten if user has customized them
+const PROTECTED_FILES = [
+  'auto_approve_safe.rules.json'
+];
+
 function log(message, color = 'reset') {
   console.log(`${colors[color]}${message}${colors.reset}`);
 }
@@ -34,21 +40,16 @@ function logError(message) {
   log(`  ✗ ${message}`, 'red');
 }
 
-function logUpdated(message) {
+function logUpdate(message) {
   log(`  ↻ ${message}`, 'blue');
 }
 
-/**
- * Check if two files have identical content
- */
-function filesAreIdentical(file1, file2) {
-  try {
-    const content1 = fs.readFileSync(file1);
-    const content2 = fs.readFileSync(file2);
-    return content1.equals(content2);
-  } catch {
-    return false;
-  }
+function logBackup(message) {
+  log(`  ⤷ ${message}`, 'cyan');
+}
+
+function logPreserve(message) {
+  log(`  ★ ${message}`, 'yellow');
 }
 
 /**
@@ -87,6 +88,32 @@ function getTargetBase() {
   return path.join(process.env.INIT_CWD || process.cwd(), '.claude');
 }
 
+/**
+ * Ensure entries exist in a .gitignore file (append if missing)
+ */
+function ensureGitignoreEntries(gitignorePath, entries, header) {
+  let existingContent = '';
+  if (fs.existsSync(gitignorePath)) {
+    existingContent = fs.readFileSync(gitignorePath, 'utf8');
+  }
+
+  const existingLines = new Set(
+    existingContent.split('\n').map(line => line.trim()).filter(Boolean)
+  );
+
+  const missingEntries = entries.filter(entry => !existingLines.has(entry));
+
+  if (missingEntries.length > 0) {
+    const newContent = existingContent.trimEnd() +
+      (existingContent ? '\n\n' : '') +
+      `# ${header}\n` +
+      missingEntries.join('\n') + '\n';
+    fs.writeFileSync(gitignorePath, newContent);
+    return missingEntries.length;
+  }
+  return 0;
+}
+
 /**
  * Recursively copy directory contents
  */
@@ -110,17 +137,31 @@ function copyDirRecursive(src, dest, stats) {
       copyDirRecursive(srcPath, destPath, stats);
     } else {
       if (fs.existsSync(destPath)) {
-        // Check if files are identical
-        if (filesAreIdentical(srcPath, destPath)) {
-          stats.skipped.push(destPath);
+        // Check if this is a protected user config file
+        if (PROTECTED_FILES.includes(entry.name)) {
+          stats.preserved.push(destPath);
+          logPreserve(`Preserved (user config): ${path.relative(stats.targetBase, destPath)}`);
+          continue;
+        }
+
+        // Check if files are identical (inline comparison)
+        const srcContent = fs.readFileSync(srcPath);
+        const destContent = fs.readFileSync(destPath);
+        const filesAreIdentical = srcContent.equals(destContent);
+
+        if (filesAreIdentical) {
+          stats.unchanged.push(destPath);
           logSkip(`Unchanged: ${path.relative(stats.targetBase, destPath)}`);
         } else {
           // Backup existing file, then replace
-          const backupPath = destPath + '.bak';
+          const backupPath = destPath + '.backup';
           fs.copyFileSync(destPath, backupPath);
+          stats.backups.push(backupPath);
+          logBackup(`Backup: ${path.relative(stats.targetBase, backupPath)}`);
+
           fs.copyFileSync(srcPath, destPath);
           stats.updated.push(destPath);
-          logUpdated(`Updated (backup: .bak): ${path.relative(stats.targetBase, destPath)}`);
+          logUpdate(`Updated: ${path.relative(stats.targetBase, destPath)}`);
         }
       } else {
         fs.copyFileSync(srcPath, destPath);
@@ -148,10 +189,34 @@ function install() {
     fs.mkdirSync(targetBase, { recursive: true });
   }
 
+  // Ensure gitignore entries for package-installed files
+  const gitignorePath = path.join(targetBase, '.gitignore');
+  const gitignoreEntries = [
+    'scripts/auto_approve_safe.py',
+    'scripts/auto_approve_safe.rules.json',
+    'scripts/context-monitor.py',
+    'scripts/__pycache__/',
+    'commands/optimize-auto-approve-hook.md',
+    'skills/nash/',
+    'auto_approve_safe.decisions.jsonl',
+    'auto_approve_safe.decisions.archived.jsonl',
+    '*.backup',
+  ];
+  const addedCount = ensureGitignoreEntries(
+    gitignorePath,
+    gitignoreEntries,
+    'Installed by @torka/claude-qol'
+  );
+  if (addedCount > 0) {
+    log(`   Updated .gitignore (added ${addedCount} entries)`, 'green');
+  }
+
   const stats = {
     copied: [],
     updated: [],
-    skipped: [],
+    unchanged: [],
+    preserved: [],
+    backups: [],
     targetBase,
   };
 
@@ -176,8 +241,10 @@ function install() {
   // Summary
   log('\n' + colors.bold + '📊 Installation Summary' + colors.reset);
   log(`   Files copied: ${stats.copied.length}`, 'green');
-  log(`   Files updated (backups created): ${stats.updated.length}`, 'blue');
-  log(`   Files unchanged: ${stats.skipped.length}`, 'yellow');
+  log(`   Files updated: ${stats.updated.length}`, 'blue');
+  log(`   Backups created: ${stats.backups.length}`, 'cyan');
+  log(`   Files unchanged: ${stats.unchanged.length}`, 'yellow');
+  log(`   Files preserved (user config): ${stats.preserved.length}`, 'yellow');
 
   // Post-install instructions
   log('\n' + colors.bold + '📝 Configuration Required' + colors.reset);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@torka/claude-qol",
-  "version": "0.2.1",
+  "version": "0.3.0",
   "description": "Claude Code quality-of-life improvements: auto-approve hooks, context monitoring, and status line enhancements",
   "keywords": [
     "claude-code",