synap 0.5.0 → 0.5.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "synap",
-  "version": "0.5.0",
+  "version": "0.5.2",
   "description": "A CLI for externalizing your working memory",
   "main": "src/cli.js",
   "bin": {

package/src/cli.js

@@ -1835,10 +1835,11 @@ async function main() {
             skillResult = { ...skillResult, ...(await skillInstaller.install()) };
             if (skillResult.installed) {
               console.log(`      ${chalk.green('✓')} Skill installed at ~/.claude/skills/synap-assistant/`);
+              if (skillResult.backupFile) {
+                console.log(`      ${chalk.yellow('•')} Your modifications backed up to ${skillResult.backupFile}`);
+              }
             } else if (skillResult.skipped) {
               console.log(`      ${chalk.yellow('•')} Skill already up to date`);
-            } else if (skillResult.needsForce) {
-              console.log(`      ${chalk.yellow('•')} Skill modified. Use ${chalk.cyan('synap install-skill --force')}`);
             }
           } catch (err) {
             console.log(`      ${chalk.red('✗')} Skill install failed: ${err.message}`);
@@ -2040,7 +2041,6 @@ async function main() {
     .command('install-skill')
     .description('Install Claude Code skill')
     .option('--uninstall', 'Remove the skill')
-    .option('--force', 'Override ownership check')
     .action(async (options) => {
       const skillInstaller = require('./skill-installer');
 
@@ -2048,13 +2048,14 @@ async function main() {
         await skillInstaller.uninstall();
         console.log(chalk.green('Skill uninstalled'));
       } else {
-        const result = await skillInstaller.install({ force: options.force });
+        const result = await skillInstaller.install();
         if (result.installed) {
           console.log(chalk.green('Skill installed to ~/.claude/skills/synap-assistant/'));
+          if (result.backupFile) {
+            console.log(chalk.yellow(`Your modifications were backed up to ${result.backupFile}`));
+          }
         } else if (result.skipped) {
           console.log(chalk.yellow('Skill already up to date'));
-        } else if (result.needsForce) {
-          console.log(chalk.yellow('Skill was modified by user. Use --force to overwrite.'));
         }
       }
     });

package/src/skill-installer.js

@@ -116,6 +116,8 @@ async function install(options = {}) {
     fs.mkdirSync(TARGET_SKILL_DIR, { recursive: true });
   }
 
+  let backupFile = null;
+
   // Check if target exists
   if (fs.existsSync(TARGET_SKILL_FILE)) {
     const targetContent = fs.readFileSync(TARGET_SKILL_FILE, 'utf8');
@@ -124,32 +126,33 @@ async function install(options = {}) {
     const canonicalTargetHash = getHash(canonicalTarget);
     const targetMatchesSource = canonicalTargetHash === sourceHash;
 
-    if (targetSource !== SKILL_SOURCE && !options.force) {
-      return { installed: false, needsForce: true };
+    // Already up to date
+    if (targetMatchesSource && targetContent === normalizedSourceContent) {
+      return { installed: false, skipped: true };
     }
 
-    if (targetSource === SKILL_SOURCE && !options.force) {
-      if (targetMatchesSource && targetContent === normalizedSourceContent) {
-        return { installed: false, skipped: true };
-      }
-
-      if (targetHash) {
-        if (targetHash !== canonicalTargetHash) {
-          return { installed: false, needsForce: true };
-        }
-      } else if (!targetMatchesSource) {
-        return { installed: false, needsForce: true };
+    // Check if user modified the file (hash mismatch or different source)
+    const userModified = targetSource !== SKILL_SOURCE ||
+      (targetHash && targetHash !== canonicalTargetHash) ||
+      (!targetHash && !targetMatchesSource);
+
+    if (userModified) {
+      // Create timestamped backup
+      const timestamp = new Date().toISOString().slice(0, 10).replace(/-/g, '');
+      backupFile = `${TARGET_SKILL_FILE}.backup.${timestamp}`;
+      // If backup already exists today, add time
+      if (fs.existsSync(backupFile)) {
+        const time = new Date().toISOString().slice(11, 16).replace(':', '');
+        backupFile = `${TARGET_SKILL_FILE}.backup.${timestamp}-${time}`;
       }
+      fs.writeFileSync(backupFile, targetContent);
     }
-
-    const backupFile = TARGET_SKILL_FILE + '.backup';
-    fs.writeFileSync(backupFile, targetContent);
   }
 
   // Install
   fs.writeFileSync(TARGET_SKILL_FILE, normalizedSourceContent);
 
-  return { installed: true };
+  return { installed: true, backupFile };
 }
 
 /**