docks-kit 0.1.3 → 0.1.5

package/SoT/toolchain.json

@@ -3,6 +3,7 @@
   "tools": {
     "jq":     { "kind": "required", "note": "deployed Claude hooks/statusline assets" },
     "curl":   { "kind": "required", "note": "installers + usage fetch" },
+    "git":    { "kind": "check", "note": "plugin marketplaces (claude/codex clone them) + kit checkout updates" },
     "node":   { "kind": "check", "note": "hosts npm globals (agent-browser, LSP servers)" },
     "npm":    { "kind": "check", "note": "npm-global installer" },
     "claude": { "kind": "check", "floor": "2.1.170", "note": "kit floor — `best` alias + Fable 5 need >=2.1.170 (mirrors settings minimumVersion)" },

package/cli/src/engine-native/claudeSync.ts

@@ -520,6 +520,11 @@ function syncPlugins(ctx: Ctx, claudeDir: string): void {
     warn("claude CLI not in PATH — skipping plugin reconcile (run /plugin marketplace add + /plugin install manually)")
     return
   }
+  if (!commandExists("git")) {
+    const hint = process.platform === "win32" ? "winget install Git.Git (then open a new terminal)" : "install git via your package manager"
+    warn(`git not found — plugin marketplaces are git repos, so every plugin operation would fail. Skipping plugin passes. Install: ${hint}, then re-run sync`)
+    return
+  }
 
   const repoSettings = readJsonFile(repoSettingsFile)
   const repoObj = repoSettings !== undefined && isObject(repoSettings) ? repoSettings : {}

package/cli/src/engine-native/codexSync.ts

@@ -397,6 +397,11 @@ function syncPlugins(ctx: Ctx, sotConfig: string): void {
     )
     return
   }
+  if (!commandExists("git")) {
+    const hint = process.platform === "win32" ? "winget install Git.Git (then open a new terminal)" : "install git via your package manager"
+    warn(`git not found — Codex plugin marketplaces are git repos, so plugin refresh would fail. Skipping. Install: ${hint}, then re-run sync`)
+    return
+  }
 
   let refreshed = 0
   let failed = 0

package/cli/src/engine-native/skillsSync.ts

@@ -5,7 +5,7 @@
  * and the snapshot write.
  */
 import { spawnSync } from "node:child_process"
-import { cpSync, existsSync, lstatSync, mkdirSync, readFileSync, readlinkSync, rmSync, statSync, symlinkSync, writeFileSync } from "node:fs"
+import { cpSync, existsSync, lstatSync, mkdirSync, readFileSync, readlinkSync, realpathSync, rmdirSync, rmSync, statSync, symlinkSync, unlinkSync, writeFileSync } from "node:fs"
 import { tmpdir } from "node:os"
 import { capture, commandExists, isExecutable, p, which } from "./exec"
 import type { Ctx } from "./index"
@@ -136,11 +136,17 @@ function healClaudeSymlink(ctx: Ctx, skillsDir: string, base: string): boolean {
   if (linkStat?.isSymbolicLink() === true) {
     const current = safeReadlink(claudeLink)
     if (current === relTarget) return false
+    // win32: `npx skills add` creates absolute symlinks/junctions — any link
+    // that RESOLVES to the canonical dir is healthy, not stale.
+    if (process.platform === "win32" && realpathEquals(claudeLink, canonical)) return false
     if (ctx.dryRun) {
       echo(`[dry-run] would replace stale Claude symlink: ~/.claude/skills/${base} -> ${current}  (correct: ${relTarget})`)
       return true
     }
-    rmSync(claudeLink, { force: true })
+    if (!removeLink(claudeLink)) {
+      warn(`could not remove stale link ~/.claude/skills/${base} — remove it manually, then re-run sync`)
+      return false
+    }
   } else if (linkStat !== undefined) {
     warn(`~/.claude/skills/${base} exists as a real path (not a symlink) — leaving alone; remove manually if it's stale`)
     return false
@@ -169,9 +175,34 @@ function safeReadlink(path: string): string {
   }
 }
 
+function realpathEquals(a: string, b: string): boolean {
+  try {
+    return realpathSync(a) === realpathSync(b)
+  } catch {
+    return false
+  }
+}
+
+/**
+ * Remove a symlink/junction without crashing: Bun's rmSync throws EFAULT on
+ * win32 directory symlinks. unlink handles file symlinks, rmdir handles
+ * dir symlinks/junctions, rmSync is the POSIX catch-all.
+ */
+function removeLink(path: string): boolean {
+  for (const rm of [() => unlinkSync(path), () => rmdirSync(path), () => rmSync(path, { force: true })]) {
+    try {
+      rm()
+      return true
+    } catch {
+      // try the next removal shape
+    }
+  }
+  return lstat(path) === undefined
+}
+
 /** skills::_link_or_copy — real symlink preferred, copy fallback (Windows). */
 export function linkOrCopy(target: string, link: string): boolean {
-  rmSync(link, { force: true })
+  removeLink(link)
   try {
     symlinkSync(target, link, process.platform === "win32" ? "dir" : undefined)
   } catch {

package/cli/src/engine-native/toolchain.ts

@@ -72,6 +72,8 @@ export function installedVersion(ctx: Ctx, tool: string): string {
       const m = /effect-solutions@([0-9][0-9.]*)/.exec(capture(bunbin, ["pm", "-g", "ls"]))
       return m?.[1] ?? ""
     }
+    case "git":
+      return firstLineField(capture("git", ["--version"]), 2)
     case "node":
       return capture("node", ["--version"]).replace(/^v/, "")
     case "npm":

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "docks-kit",
-  "version": "0.1.3",
+  "version": "0.1.5",
   "description": "Portable AI coding agent config kit — SoT sync engine + typed CLI for Claude Code, Codex, and universal agent skills",
   "type": "module",
   "license": "MIT",