claude-multi 0.4.2 → 0.4.3

package/CHANGELOG.md

@@ -5,6 +5,33 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.4.3] - 2026-01-26
+
+### Added
+- **Broken Symlink Detection**: Automatically detect and fix broken symlinks
+  - New `isBrokenSymlink()` and `detectBrokenSymlinks()` functions
+  - Auto-fix broken symlinks when creating or updating instances
+  - Handles edge cases: broken symlinks, existing directories, incorrect symlink targets
+- **`fix-symlinks` Command**: Fix broken symlinks for existing instances
+  - Interactive instance selection with multi-select support
+  - `--all` flag to fix all instances at once
+  - Auto-fixes instances with auto-sync enabled, warns for manual mode
+  - Clear status indicators for broken symlinks vs. valid symlinks
+- **Interactive Re-sync**: Added "Re-sync symlinks" option to interactive mode menu
+  - Quick access to symlink diagnosis and repair from interactive UI
+- **Test Suite**: Comprehensive tests for symlink detection and fixing
+  - Tests for broken symlink detection
+  - Tests for valid symlink handling
+  - Tests for mixed scenarios (broken + valid symlinks)
+  - Error handling tests for non-existent paths
+
+### Fixed
+- Symlink creation now uses `lstatSync` for proper broken symlink detection
+- Previously, `existsSync()` returned `false` for broken symlinks, preventing detection
+- Now correctly identifies and repairs broken symlinks in all scenarios
+
+[0.4.3]: https://github.com/hmziqrs/claude-multi/compare/v0.4.2...v0.4.3
+
 ## [0.4.2] - 2026-01-26
 
 ### Fixed

package/dist/cli.js

@@ -7286,7 +7286,7 @@ import { homedir as homedir2 } from "os";
 import { join as join3 } from "path";
 
 // src/config.ts
-import { existsSync, mkdirSync, readdirSync, statSync, unlinkSync, readlinkSync, rmSync } from "fs";
+import { existsSync, mkdirSync, readdirSync, statSync, readlinkSync, rmSync, lstatSync } from "fs";
 import { readFile, writeFile, copyFile, mkdir, symlink } from "fs/promises";
 import { homedir } from "os";
 import { join } from "path";
@@ -7420,6 +7420,41 @@ function getDefaultClaudeDir() {
   }
   return join(homedir(), ".claude");
 }
+function isBrokenSymlink(path) {
+  try {
+    const stats = lstatSync(path);
+    if (!stats.isSymbolicLink())
+      return false;
+    const target = readlinkSync(path);
+    if (!target)
+      return true;
+    const { resolve, dirname, isAbsolute } = __require("path");
+    let targetPath;
+    if (isAbsolute(target)) {
+      targetPath = target;
+    } else {
+      targetPath = resolve(dirname(path), target);
+    }
+    return !existsSync(targetPath);
+  } catch {
+    return false;
+  }
+}
+function detectBrokenSymlinks(configDir) {
+  const symlinkDirs = ["plugins", "skills"];
+  const result = { broken: [], all: [] };
+  for (const dir of symlinkDirs) {
+    const targetPath = join(configDir, dir);
+    try {
+      const stats = lstatSync(targetPath);
+      result.all.push(dir);
+      if (stats.isSymbolicLink() && isBrokenSymlink(targetPath)) {
+        result.broken.push(dir);
+      }
+    } catch {}
+  }
+  return result;
+}
 function hasDefaultClaudeConfig() {
   const defaultDir = getDefaultClaudeDir();
   const settingsFile = join(defaultDir, "settings.json");
@@ -7485,12 +7520,19 @@ async function copyAllFromDefault(targetConfigDir, autoSync = true) {
       if (stat.isDirectory()) {
         if (autoSync && symlinkDirs.includes(entry)) {
           if (existsSync(targetPath)) {
-            try {
-              readlinkSync(targetPath);
-              unlinkSync(targetPath);
-            } catch {
-              rmSync(targetPath, { force: true, recursive: true });
+            const absoluteSource2 = join(homedir(), ".claude", entry);
+            const stat2 = lstatSync(targetPath);
+            if (stat2.isSymbolicLink()) {
+              const currentTarget = readlinkSync(targetPath);
+              if (currentTarget === absoluteSource2) {
+                console.log(`  \u2713 Already symlinked: ${entry}`);
+                continue;
+              }
+              console.log(source_default.yellow(`  \u26A0 Replacing incorrect symlink: ${entry}`));
+            } else {
+              console.log(source_default.yellow(`  \u26A0 Removing existing directory: ${entry}`));
             }
+            rmSync(targetPath, { force: true, recursive: true });
           }
           const absoluteSource = join(homedir(), ".claude", entry);
           await symlink(absoluteSource, targetPath, "dir");
@@ -8160,6 +8202,9 @@ program2.command("auto-sync <name> <status>").description("Toggle auto-sync for
     process.exit(1);
   }
 });
+program2.command("fix-symlinks [name...]").description("Fix broken symlinks for instances (auto-detects and repairs)").option("-a, --all", "Fix all instances").action(async (names, options) => {
+  await handleFixSymlinks(names, options.all);
+});
 program2.command("plugins").description("Manage enabled plugins for instances").argument("[action]", "Action to perform (list, enable, disable, copy)", "list").argument("[instance]", "Instance name (for list/enable/disable)", "").argument("[plugins...]", "Plugin IDs to enable/disable", []).action(async (action = "list", instanceName = "", plugins = []) => {
   try {
     switch (action) {
@@ -8317,6 +8362,55 @@ async function handlePluginsCopy(instanceName) {
   const enabledCount = Object.values(defaultPlugins).filter((v) => v === true).length;
   console.log(source_default.green(`\u2713 Copied ${enabledCount} enabled plugins to '${instanceName}'`));
 }
+async function handleFixSymlinks(names, fixAll) {
+  const instances = await listInstances();
+  if (instances.length === 0) {
+    console.log(source_default.yellow("No instances found."));
+    return;
+  }
+  let instancesToFix;
+  if (fixAll) {
+    instancesToFix = instances;
+  } else if (names.length > 0) {
+    instancesToFix = names.map((name) => instances.find((i) => i.name === name)).filter(Boolean);
+  } else {
+    const { selected } = await import_prompts.default({
+      type: "multiselect",
+      name: "selected",
+      message: "Select instances to fix:",
+      choices: instances.map((i) => ({
+        title: `${i.name} ${i.autoSync ? "(auto-sync)" : "(manual)"}`,
+        value: i.name
+      }))
+    });
+    instancesToFix = selected.map((name) => instances.find((i) => i.name === name)).filter(Boolean);
+  }
+  for (const instance of instancesToFix) {
+    const diagnosis = detectBrokenSymlinks(instance.configDir);
+    const needsFix = diagnosis.broken.length > 0;
+    console.log(source_default.bold(`
+\uD83D\uDD0D ${instance.name}`));
+    console.log(`  Config: ${instance.configDir}`);
+    console.log(`  Auto-sync: ${instance.autoSync ? "enabled" : "disabled"}`);
+    if (diagnosis.all.length === 0) {
+      console.log(source_default.gray("  No symlinks found (manual mode)"));
+      continue;
+    }
+    if (needsFix) {
+      console.log(source_default.red(`  \u274C Broken: ${diagnosis.broken.join(", ")}`));
+      if (instance.autoSync) {
+        await syncPluginsAndSkills(instance.configDir);
+        console.log(source_default.green(`  \u2705 Fixed: ${diagnosis.broken.join(", ")}`));
+      } else {
+        console.log(source_default.yellow("  \u26A0 Auto-sync is disabled. Enable it first."));
+      }
+    } else {
+      console.log(source_default.green(`  \u2705 All symlinks OK: ${diagnosis.all.join(", ")}`));
+    }
+  }
+  console.log(source_default.bold(`
+\u2728 Done!`));
+}
 program2.command("interactive").alias("i").description("Launch interactive mode for managing Claude Code instances").action(async () => {
   try {
     await runInteractiveMode();
@@ -8348,6 +8442,7 @@ async function runInteractiveMode() {
             { title: "\u2139\uFE0F  Show instance details", value: "info" },
             { title: "\uD83D\uDD0C Manage plugins", value: "plugins" },
             { title: "\uD83D\uDD04 Toggle auto-sync", value: "autosync" },
+            { title: "\uD83D\uDD04 Re-sync symlinks", value: "resync" },
             { title: "\uD83D\uDDD1\uFE0F  Remove instance", value: "remove" }
           ] : [],
           { title: "\uD83D\uDEAA Exit", value: "exit" }
@@ -8375,6 +8470,9 @@ async function runInteractiveMode() {
         case "autosync":
           await handleToggleAutoSync(instances);
           break;
+        case "resync":
+          await handleFixSymlinks([], false);
+          break;
         case "remove":
           await handleRemoveInstance(instances);
           break;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-multi",
-  "version": "0.4.2",
+  "version": "0.4.3",
   "description": "Manage multiple Claude Code instances with different aliases and configurations",
   "type": "module",
   "bin": {