create-zudo-doc 5.0.0 → 5.1.0

package/dist/scaffold.d.ts

@@ -32,5 +32,5 @@ export declare function deriveDocSkillName(projectName: string): string;
  *
  * Bumped in lockstep by scripts/release-create-zudo-doc.sh.
  */
-export declare const ZUDO_DOC_PIN = "^5.0.0";
+export declare const ZUDO_DOC_PIN = "^5.1.0";
 export declare function scaffold(choices: UserChoices): Promise<void>;

package/dist/scaffold.js

@@ -43,7 +43,7 @@ export function deriveDocSkillName(projectName) {
  *
  * Bumped in lockstep by scripts/release-create-zudo-doc.sh.
  */
-export const ZUDO_DOC_PIN = "^5.0.0";
+export const ZUDO_DOC_PIN = "^5.1.0";
 /**
  * Files in `templates/base/**` that must not be copied by the unconditional
  * base mirror. Each entry is matched against the path relative to
@@ -703,7 +703,7 @@ function generatePackageJson(choices) {
         // `/exclude` at module scope from the always-bundled chrome graph; #3110
         // moved compileExclude into @takazudo/zudo-doc, so docHistory-OFF projects
         // no longer need the package at all.
-        deps["@takazudo/zudo-doc-history-server"] = "^5.0.0";
+        deps["@takazudo/zudo-doc-history-server"] = "^5.1.0";
         // tsx is no longer needed here: the relocated package plugin imports the
         // runner directly (no `tsx -e` spawn) since the package ships compiled
         // dist/ — package-first migration #2321 (#2337).

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "create-zudo-doc",
-  "version": "5.0.0",
+  "version": "5.1.0",
   "description": "Create a new zudo-doc documentation site",
   "license": "MIT",
   "author": "Takeshi Takatsudo",

package/templates/base/scripts/setup-doc-skill.sh

@@ -201,6 +201,26 @@ if [ ! -d "$DOCS_DIR" ]; then
   exit 1
 fi
 
+# Helper: physical (symlink-resolved) form of a directory path; echoes the
+# input unchanged when it is not a directory. Stock macOS ships no realpath(1)
+# and BSD readlink has no -f, so `cd` + `pwd -P` is the only resolver available
+# on every platform this script targets (it is also copied verbatim into
+# downstream projects by create-zudo-doc, so it must stay portable).
+physical_dir() {
+  local resolved
+  # Fall back to the input on ANY resolution failure -- notably an unreadable
+  # directory, where `cd` fails and the subshell yields "". Without the
+  # non-empty guard, two different unreadable paths would both collapse to ""
+  # and compare EQUAL, silently skipping a link that should have been created.
+  # stderr is suppressed so a permission-denied probe stays quiet.
+  if [ -d "$1" ] && resolved="$(cd "$1" 2>/dev/null && pwd -P)" &&
+    [ -n "$resolved" ]; then
+    printf '%s\n' "$resolved"
+  else
+    printf '%s\n' "$1"
+  fi
+}
+
 # Helper: replace a symlink or file at the given path
 ensure_symlink() {
   local link_path="$1"
@@ -228,7 +248,17 @@ safe_link_tracked_skill() {
   if [ -L "$link_path" ]; then
     local current_target
     current_target="$(readlink "$link_path")"
-    if [ "$current_target" = "$link_target" ]; then
+    # Compare where the link actually LANDS, not the literal string it stores.
+    # link_target is built from MAIN_PROJECT_DIR, which comes from
+    # `git worktree list` and is therefore always PHYSICAL, while an existing
+    # link may hold an unresolved path for the same directory. A raw string
+    # compare then mis-reports our own correct link as foreign whenever the
+    # project or its parents sit behind a symlink (macOS $TMPDIR: /var ->
+    # /private/var; also a symlinked $HOME or repo checkout), so the link was
+    # skipped with a spurious "already links to ..." warning instead of
+    # no-opping (zudolab/zudo-doc#3156 D4).
+    if [ "$current_target" = "$link_target" ] ||
+      [ "$(physical_dir "$link_path")" = "$(physical_dir "$link_target")" ]; then
       return 0 # already correct -> no-op
     fi
     if [ -e "$link_path" ]; then