fvtt-world-cli 1.1.0 → 1.1.1

package/README.md

@@ -100,8 +100,9 @@ The package ships `foundry-world-editor`, an installable [Agent Skill](docs/skil
 an agent to drive the CLI safely. It follows the open Agent Skills standard, so the same skill
 works in Claude Code, Codex, and other agents that read `SKILL.md` files.
 
-An installed skill stays current on its own: updating the package refreshes it. A copy you have
-edited locally is never replaced automatically, so your own instructions survive updates.
+An installed skill stays current on its own: after a package update, the bridge daemon refreshes
+it at startup. A copy you have edited locally is never replaced automatically, so your own
+instructions survive updates.
 
 ## Built to be trusted with a live world
 

package/dist/index.js

@@ -10,7 +10,7 @@ import { readFileSync as readFileSync2 } from "node:fs";
 import { createHash as createHash2, randomBytes as randomBytes2, randomUUID as randomUUID2, timingSafeEqual } from "node:crypto";
 
 // ../protocol/src/constants.js
-var PROTOCOL_VERSION = "1.1.0";
+var PROTOCOL_VERSION = "1.1.1";
 var PROTOCOL_COMPONENTS = Object.freeze({
   MODULE: "module",
   CLI_DAEMON: "cli-daemon",
@@ -14699,10 +14699,12 @@ function syncInstalledSkillCopies(dependencies) {
   try {
     const sourceDirectory = resolveSkillSourceDirectory();
     if (!sourceDirectory) return;
+    let installedAnywhere = false;
     for (const root of knownSkillRoots(dependencies)) {
       try {
         const destination = join2(root, SKILL_NAME);
         const state = inspectInstalledSkill(destination, sourceDirectory);
+        if (state !== "missing") installedAnywhere = true;
         if (state === "pristine") {
           rmSync(destination, { recursive: true, force: true });
           cpSync(sourceDirectory, destination, { recursive: true });
@@ -14722,6 +14724,13 @@ function syncInstalledSkillCopies(dependencies) {
         continue;
       }
     }
+    if (!installedAnywhere) {
+      write(
+        dependencies.stderr,
+        `The ${SKILL_NAME} agent skill is not installed. If AI agents use this CLI, consider running \`fvtt-world-cli skill install\` to give them its operating manual.
+`
+      );
+    }
   } catch {
     return;
   }

package/docs/skill.md

@@ -32,12 +32,16 @@ fvtt-world-cli skill install --to <skills-directory>
 copy participates in updates later. `--link` symlinks instead of copying, which keeps the installed
 skill permanently identical to the CLI it came from.
 
+A bridge daemon started while no copy is installed anywhere prints a reminder with the
+installation command, so a missing skill does not go unnoticed.
+
 ## Staying up to date
 
 The skill is versioned together with the CLI, and the CLI keeps installed copies current on its
-own: updating the npm package refreshes them, and the daemon performs the same check at startup for
-installations updated by other means, such as a git checkout. For updates, a copy is one
-of two kinds:
+own: the bridge daemon checks every known copy at startup and refreshes outdated ones, so a package
+update takes effect the next time the daemon starts. The check runs inside the CLI itself; nothing
+about installation or updates relies on npm lifecycle scripts, which npm blocks by default. For
+updates, a copy is one of two kinds:
 
 - An **unmodified** copy, exactly what some version of the CLI shipped, is replaced silently with
   the current version.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fvtt-world-cli",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "description": "Safe local CLI and authenticated bridge for automating live Foundry VTT worlds.",
   "license": "MIT",
   "author": "DroiD16",
@@ -20,8 +20,8 @@
   },
   "type": "module",
   "bin": {
-    "fvtt-world-cli": "./bin/fvtt-world-cli.js",
-    "worldctl": "./bin/fvtt-world-cli.js"
+    "fvtt-world-cli": "bin/fvtt-world-cli.js",
+    "worldctl": "bin/fvtt-world-cli.js"
   },
   "files": [
     "bin/",
@@ -34,8 +34,7 @@
     "build": "esbuild src/index.ts --bundle --platform=node --format=esm --target=node20 --external:commander --external:pino --external:tsx --external:ws --external:zod --outfile=dist/index.js",
     "typecheck": "tsc --project ../../tsconfig.json",
     "prepack": "npm run build && node ./bin/prepare-package-assets.mjs",
-    "postpack": "node ./bin/prepare-package-assets.mjs --clean",
-    "postinstall": "node ./bin/sync-installed-skill.mjs"
+    "postpack": "node ./bin/prepare-package-assets.mjs --clean"
   },
   "dependencies": {
     "commander": "^13.1.0",
@@ -45,6 +44,6 @@
     "zod": "^3.24.3"
   },
   "devDependencies": {
-    "@fvtt-world-cli/protocol": "1.1.0"
+    "@fvtt-world-cli/protocol": "1.1.1"
   }
 }

package/bin/sync-installed-skill.mjs

@@ -1,89 +0,0 @@
-import { createHash } from "node:crypto";
-import { cpSync, lstatSync, readFileSync, rmSync, statSync } from "node:fs";
-import { homedir, platform } from "node:os";
-import { dirname, join, resolve } from "node:path";
-import { fileURLToPath } from "node:url";
-
-const SKILL_NAME = "foundry-world-editor";
-
-try {
-  const binDirectory = dirname(fileURLToPath(import.meta.url));
-  const sourceDirectory = [resolve(binDirectory, "../skills"), resolve(binDirectory, "../../../skills")]
-    .map((candidate) => resolve(candidate, SKILL_NAME))
-    .find((candidate) => {
-      try {
-        return statSync(join(candidate, "SKILL.md")).isFile();
-      } catch {
-        return false;
-      }
-    });
-  if (!sourceDirectory) process.exit(0);
-  const packaged = readFileSync(join(sourceDirectory, "SKILL.md"), "utf8");
-
-  const home = process.env.HOME?.trim() || homedir();
-  const xdgHome = process.env.XDG_CONFIG_HOME?.trim();
-  const configPath = xdgHome
-    ? join(xdgHome, "fvtt-world-cli", "config.json")
-    : platform() === "darwin"
-      ? join(home, "Library", "Application Support", "fvtt-world-cli", "config.json")
-      : platform() === "win32"
-        ? join(
-            process.env.APPDATA?.trim() || join(home, "AppData", "Roaming"),
-            "fvtt-world-cli",
-            "config.json"
-          )
-        : join(home, ".config", "fvtt-world-cli", "config.json");
-
-  const roots = new Set([join(home, ".agents", "skills")]);
-  try {
-    const config = JSON.parse(readFileSync(configPath, "utf8"));
-    if (Array.isArray(config?.skillInstalls)) {
-      for (const root of config.skillInstalls) {
-        if (typeof root === "string" && root.length > 0) roots.add(root);
-      }
-    }
-  } catch {
-    // no or unreadable config: the standard location is still checked
-  }
-
-  for (const root of roots) {
-    try {
-      const destination = join(root, SKILL_NAME);
-      const existing = (() => {
-        try {
-          return lstatSync(destination);
-        } catch {
-          return null;
-        }
-      })();
-      if (!existing || existing.isSymbolicLink()) continue;
-
-      const installed = readFileSync(join(destination, "SKILL.md"), "utf8");
-      if (installed === packaged) continue;
-
-      const recorded = (() => {
-        try {
-          return readFileSync(join(destination, "SKILL.md.sha256"), "utf8").trim();
-        } catch {
-          return null;
-        }
-      })();
-
-      if (recorded === createHash("sha256").update(installed).digest("hex")) {
-        rmSync(destination, { recursive: true, force: true });
-        cpSync(sourceDirectory, destination, { recursive: true });
-        console.error(
-          `fvtt-world-cli: updated the installed agent skill at ${destination} to the packaged version.`
-        );
-      } else {
-        console.error(
-          `fvtt-world-cli: the installed agent skill at ${destination} has local modifications and was NOT updated. Compare them, then run \`fvtt-world-cli skill update --force\` to replace it.`
-        );
-      }
-    } catch {
-      continue;
-    }
-  }
-} catch {
-  process.exit(0);
-}