Skip redundant sync when configs already linked

package.json

@@ -1,68 +1,68 @@
 {
-  "name": "@donnes/syncode",
-  "version": "1.1.7",
-  "description": "Sync AI code agent configs (Claude Code, Cursor, Windsurf, OpenCode) across machines and projects.",
-  "private": false,
-  "type": "module",
-  "bin": {
-    "syncode": "./dist/index.js"
-  },
-  "main": "./dist/index.js",
-  "types": "./dist/index.d.ts",
-  "files": [
-    "dist",
-    "README.md",
-    "LICENSE"
-  ],
-  "scripts": {
-    "build": "tsup",
-    "dev": "tsx src/index.ts",
-    "test": "tsx scripts/execute-tests.ts",
-    "prepublishOnly": "npm run build",
-    "check:lint": "biome check . --diagnostic-level=error",
-    "check:unsafe": "biome check . --write --unsafe --diagnostic-level=error",
-    "check:types": "bunx tsc --noEmit"
-  },
-  "keywords": [
-    "ai-agents",
-    "claude-code",
-    "cursor",
-    "windsurf",
-    "opencode",
-    "vscode",
-    "ai-coding",
-    "code-agent",
-    "skill-conversion",
-    "syncode",
-    "dotfiles",
-    "configuration-management",
-    "developer-tools",
-    "ai-assistant",
-    "coding-assistant"
-  ],
-  "engines": {
-    "node": ">=20.0.0"
-  },
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/donnes/syncode.git"
-  },
-  "bugs": {
-    "url": "https://github.com/donnes/syncode/issues"
-  },
-  "homepage": "https://github.com/donnes/syncode#readme",
-  "author": "Donald Silveira",
-  "license": "MIT",
-  "dependencies": {
-    "@clack/prompts": "^0.9.1"
-  },
-  "devDependencies": {
-    "@biomejs/biome": "^2.3.11",
-    "@opencode-ai/sdk": "^1.1.41",
-    "@types/bun": "latest",
-    "@types/node": "^20.0.0",
-    "tsup": "^8.5.1",
-    "tsx": "^4.21.0",
-    "typescript": "^5.7.3"
-  }
+	"name": "@donnes/syncode",
+	"version": "1.1.7",
+	"description": "Sync AI code agent configs (Claude Code, Cursor, Windsurf, OpenCode) across machines and projects.",
+	"private": false,
+	"type": "module",
+	"bin": {
+		"syncode": "./dist/index.js"
+	},
+	"main": "./dist/index.js",
+	"types": "./dist/index.d.ts",
+	"files": [
+		"dist",
+		"README.md",
+		"LICENSE"
+	],
+	"scripts": {
+		"build": "tsup",
+		"dev": "tsx src/index.ts",
+		"test": "tsx scripts/execute-tests.ts",
+		"prepublishOnly": "npm run build",
+		"check:lint": "biome check . --diagnostic-level=error",
+		"check:unsafe": "biome check . --write --unsafe --diagnostic-level=error",
+		"check:types": "bunx tsc --noEmit"
+	},
+	"keywords": [
+		"ai-agents",
+		"claude-code",
+		"cursor",
+		"windsurf",
+		"opencode",
+		"vscode",
+		"ai-coding",
+		"code-agent",
+		"skill-conversion",
+		"syncode",
+		"dotfiles",
+		"configuration-management",
+		"developer-tools",
+		"ai-assistant",
+		"coding-assistant"
+	],
+	"engines": {
+		"node": ">=20.0.0"
+	},
+	"repository": {
+		"type": "git",
+		"url": "git+https://github.com/donnes/syncode.git"
+	},
+	"bugs": {
+		"url": "https://github.com/donnes/syncode/issues"
+	},
+	"homepage": "https://github.com/donnes/syncode#readme",
+	"author": "Donald Silveira",
+	"license": "MIT",
+	"dependencies": {
+		"@clack/prompts": "^0.9.1"
+	},
+	"devDependencies": {
+		"@biomejs/biome": "^2.3.11",
+		"@opencode-ai/sdk": "^1.1.41",
+		"@types/bun": "latest",
+		"@types/node": "^20.0.0",
+		"tsup": "^8.5.1",
+		"tsx": "^4.21.0",
+		"typescript": "^5.7.3"
+	}
 }

scripts/release.ts

@@ -52,6 +52,8 @@ pkg.version = newVersion;
 await writeFile("package.json", `${JSON.stringify(pkg, null, 2)}\n`);
 console.log("Updated package.json");
 
+await $`bun check:unsafe`;
+
 // 5. Git operations
 const tag = `v${newVersion}`;
 let output = `version=${newVersion}\ntag=${tag}\n`;

src/adapters/amp.ts

@@ -9,6 +9,7 @@ import {
   createSymlink,
   ensureDir,
   exists,
+  getSymlinkTarget,
   isSymlink,
   removeDir,
 } from "../utils/fs";
@@ -52,7 +53,10 @@ export class AmpAdapter implements AgentAdapter {
   }
 
   isLinked(systemPath: string, repoPath: string): boolean {
-    return exists(systemPath) && exists(repoPath) && isSymlink(systemPath);
+    if (!exists(systemPath) || !exists(repoPath) || !isSymlink(systemPath)) {
+      return false;
+    }
+    return getSymlinkTarget(systemPath) === repoPath;
   }
 
   async import(systemPath: string, repoPath: string): Promise<ImportResult> {
@@ -63,6 +67,20 @@ export class AmpAdapter implements AgentAdapter {
       };
     }
 
+    if (isSymlink(systemPath)) {
+      return {
+        success: true,
+        message: "Already linked to repo - no import needed",
+      };
+    }
+
+    if (exists(repoPath)) {
+      return {
+        success: true,
+        message: "Configs already in repo - no import needed",
+      };
+    }
+
     ensureDir(repoPath);
     copyDir(systemPath, repoPath);
 
@@ -80,6 +98,17 @@ export class AmpAdapter implements AgentAdapter {
       };
     }
 
+    if (isSymlink(systemPath)) {
+      const target = getSymlinkTarget(systemPath);
+      if (target === repoPath) {
+        return {
+          success: true,
+          message: "Already linked to repo - no export needed",
+          linkedTo: repoPath,
+        };
+      }
+    }
+
     // Remove existing (symlink or directory)
     if (exists(systemPath)) {
       if (isSymlink(systemPath)) {

src/adapters/antigravity.ts

@@ -9,6 +9,7 @@ import {
   createSymlink,
   ensureDir,
   exists,
+  getSymlinkTarget,
   isSymlink,
   removeDir,
 } from "../utils/fs";
@@ -54,7 +55,10 @@ export class AntigravityAdapter implements AgentAdapter {
   }
 
   isLinked(systemPath: string, repoPath: string): boolean {
-    return exists(systemPath) && exists(repoPath) && isSymlink(systemPath);
+    if (!exists(systemPath) || !exists(repoPath) || !isSymlink(systemPath)) {
+      return false;
+    }
+    return getSymlinkTarget(systemPath) === repoPath;
   }
 
   async import(systemPath: string, repoPath: string): Promise<ImportResult> {
@@ -65,6 +69,20 @@ export class AntigravityAdapter implements AgentAdapter {
       };
     }
 
+    if (isSymlink(systemPath)) {
+      return {
+        success: true,
+        message: "Already linked to repo - no import needed",
+      };
+    }
+
+    if (exists(repoPath)) {
+      return {
+        success: true,
+        message: "Configs already in repo - no import needed",
+      };
+    }
+
     ensureDir(repoPath);
     copyDir(systemPath, repoPath);
 
@@ -82,6 +100,17 @@ export class AntigravityAdapter implements AgentAdapter {
       };
     }
 
+    if (isSymlink(systemPath)) {
+      const target = getSymlinkTarget(systemPath);
+      if (target === repoPath) {
+        return {
+          success: true,
+          message: "Already linked to repo - no export needed",
+          linkedTo: repoPath,
+        };
+      }
+    }
+
     // Remove existing (symlink or directory)
     if (exists(systemPath)) {
       if (isSymlink(systemPath)) {

src/adapters/claude.ts

@@ -2,14 +2,14 @@
  * Claude Code adapter
  */
 
-import { copyFileSync, existsSync, statSync, unlinkSync } from "node:fs";
+import { copyFileSync, existsSync, statSync } from "node:fs";
 import { dirname, join } from "node:path";
 import {
   copyDir,
   ensureDir,
   exists,
   isDirectory,
-  removeDir,
+  isSymlink,
 } from "../utils/fs";
 import { contractHome } from "../utils/paths";
 import type {
@@ -89,6 +89,10 @@ export class ClaudeAdapter implements AgentAdapter {
 
       if (!existsSync(srcPath)) continue;
 
+      if (isSymlink(srcPath)) continue;
+
+      if (existsSync(destPath)) continue;
+
       if (statSync(srcPath).isDirectory()) {
         copyDir(srcPath, destPath);
         filesImported.push(`${pattern}/`);
@@ -134,14 +138,7 @@ export class ClaudeAdapter implements AgentAdapter {
 
       if (!existsSync(srcPath)) continue;
 
-      // Remove existing and copy fresh
-      if (existsSync(destPath)) {
-        if (statSync(destPath).isDirectory()) {
-          removeDir(destPath);
-        } else {
-          unlinkSync(destPath);
-        }
-      }
+      if (existsSync(destPath)) continue;
 
       if (statSync(srcPath).isDirectory()) {
         copyDir(srcPath, destPath);

src/adapters/clawdbot.ts

@@ -9,6 +9,7 @@ import {
   createSymlink,
   ensureDir,
   exists,
+  getSymlinkTarget,
   isSymlink,
   removeDir,
 } from "../utils/fs";
@@ -62,7 +63,10 @@ export class ClawdbotAdapter implements AgentAdapter {
   }
 
   isLinked(systemPath: string, repoPath: string): boolean {
-    return exists(systemPath) && exists(repoPath) && isSymlink(systemPath);
+    if (!exists(systemPath) || !exists(repoPath) || !isSymlink(systemPath)) {
+      return false;
+    }
+    return getSymlinkTarget(systemPath) === repoPath;
   }
 
   async import(systemPath: string, repoPath: string): Promise<ImportResult> {
@@ -73,6 +77,20 @@ export class ClawdbotAdapter implements AgentAdapter {
       };
     }
 
+    if (isSymlink(systemPath)) {
+      return {
+        success: true,
+        message: "Already linked to repo - no import needed",
+      };
+    }
+
+    if (exists(repoPath)) {
+      return {
+        success: true,
+        message: "Configs already in repo - no import needed",
+      };
+    }
+
     ensureDir(repoPath);
     copyDir(systemPath, repoPath);
 
@@ -110,6 +128,17 @@ export class ClawdbotAdapter implements AgentAdapter {
     repoPath: string,
     systemPath: string,
   ): Promise<ExportResult> {
+    if (isSymlink(systemPath)) {
+      const target = getSymlinkTarget(systemPath);
+      if (target === repoPath) {
+        return {
+          success: true,
+          message: "Already linked to repo - no export needed",
+          linkedTo: repoPath,
+        };
+      }
+    }
+
     if (exists(systemPath) && !isSymlink(systemPath)) {
       const backupPath = `${systemPath}.backup`;
       if (exists(backupPath)) {