forgedock 1.0.11 → 1.0.13

package/bin/forgedock.mjs

@@ -9,6 +9,7 @@ import {
   readFileSync,
   writeFileSync,
   renameSync,
+  copyFileSync,
 } from "fs";
 import { execSync } from "child_process";
 import readline from "readline";
@@ -36,6 +37,42 @@ const YELLOW = "\x1b[33m";
 const CYAN = "\x1b[36m";
 const RED = "\x1b[31m";
 const RESET = "\x1b[0m";
+const DIM = "\x1b[2m";
+
+// ---------------------------------------------------------------------------
+// Splash — the ForgeDock wordmark shown at startup
+// ---------------------------------------------------------------------------
+
+const LOGO_LINES = [
+  "  ╔═╗╔═╗╦═╗╔═╗╔═╗╔╦╗╔═╗╔═╗╦╔═",
+  "  ╠╣ ║ ║╠╦╝║ ╦║╣  ║║║ ║║  ╠╩╗",
+  "  ╚  ╚═╝╩╚═╚═╝╚═╝═╩╝╚═╝╚═╝╩ ╩",
+];
+
+function getVersion() {
+  try {
+    const pkg = JSON.parse(
+      readFileSync(join(FORGE_HOME, "package.json"), "utf-8"),
+    );
+    return pkg.version || "";
+  } catch {
+    return "";
+  }
+}
+
+function splash() {
+  const version = getVersion();
+  console.log("");
+  for (const line of LOGO_LINES) {
+    console.log(`${BOLD}${CYAN}${line}${RESET}`);
+  }
+  console.log("");
+  console.log(`  ${DIM}GitHub as a knowledge graph for AI agents${RESET}`);
+  if (version) {
+    console.log(`  ${DIM}v${version}${RESET}`);
+  }
+  console.log("");
+}
 
 /**
  * Prompt the user for a yes/no confirmation.
@@ -55,11 +92,34 @@ async function confirmOverwrite(question) {
     });
     rl.question(question, (answer) => {
       rl.close();
-      resolve(answer.trim().toLowerCase() === "y" || answer.trim().toLowerCase() === "yes");
+      resolve(
+        answer.trim().toLowerCase() === "y" ||
+          answer.trim().toLowerCase() === "yes",
+      );
     });
   });
 }
 
+/**
+ * Returns true if `dir` is inside a git work tree. Used to avoid generating
+ * forge.yaml in non-project directories — e.g. when npx defaults cwd to a
+ * system folder (C:\Windows) because the launching shell sat on an
+ * unsupported UNC/WSL path. The command is a fixed string with no
+ * interpolated input, so there is no shell-injection surface (ref #151).
+ */
+function isGitWorkTree(dir) {
+  try {
+    const out = execSync("git rev-parse --is-inside-work-tree", {
+      cwd: dir,
+      stdio: ["ignore", "pipe", "ignore"],
+      encoding: "utf-8",
+    });
+    return out.trim() === "true";
+  } catch {
+    return false;
+  }
+}
+
 async function findMarkdownFiles(dir) {
   const results = [];
   const entries = await readdir(dir, { withFileTypes: true });
@@ -110,10 +170,17 @@ async function install() {
           updated++;
         }
       } else {
-        console.log(
-          `  ${YELLOW}WARNING${RESET}: ${rel} is a regular file — skipping (remove it manually to let ForgeDock manage it)`,
-        );
-        skipped++;
+        // Existing regular file — a ForgeDock-shipped command copied by a prior
+        // install (common on Windows, where symlinks aren't permitted). It is
+        // ForgeDock-managed (we only iterate shipped command names), so refresh
+        // it: skip if already current, otherwise overwrite with a fresh copy.
+        if (readFileSync(file, "utf-8") === readFileSync(target, "utf-8")) {
+          skipped++;
+        } else {
+          copyFileSync(file, target);
+          console.log(`  ${YELLOW}Updated${RESET}: ${rel}`);
+          updated++;
+        }
       }
     } catch (err) {
       if (err.code !== "ENOENT") {
@@ -122,9 +189,20 @@ async function install() {
         );
         throw err;
       }
-      // Doesn't exist — create symlink
-      await symlink(file, target);
-      console.log(`  ${GREEN}Installed${RESET}: ${rel}`);
+      // Doesn't exist — create it. Prefer a symlink; fall back to a copy on
+      // systems where symlink creation is not permitted (e.g. Windows without
+      // Developer Mode / admin), which would otherwise throw EPERM. See #587.
+      try {
+        await symlink(file, target);
+        console.log(`  ${GREEN}Installed${RESET}: ${rel}`);
+      } catch (linkErr) {
+        if (linkErr.code === "EPERM" || linkErr.code === "EACCES") {
+          copyFileSync(file, target);
+          console.log(`  ${GREEN}Installed${RESET} (copy): ${rel}`);
+        } else {
+          throw linkErr;
+        }
+      }
       installed++;
     }
   }
@@ -159,10 +237,24 @@ async function install() {
   );
   console.log("");
 
-  // Auto-generate forge.yaml if missing — no second command needed
-  const forgeYamlPath = join(process.cwd(), "forge.yaml");
+  // Auto-generate forge.yaml if missing — no second command needed.
+  // Only do this inside a real git project; otherwise cwd may be a system or
+  // read-only directory (e.g. C:\Windows when npx is launched from a WSL path),
+  // and writing forge.yaml there would crash. See #585.
+  const cwd = process.cwd();
+  const forgeYamlPath = join(cwd, "forge.yaml");
   if (!existsSync(forgeYamlPath)) {
-    await init(true);
+    if (isGitWorkTree(cwd)) {
+      await init(true);
+    } else {
+      console.log(
+        `  ${YELLOW}Skipping forge.yaml generation${RESET} — current directory is not a git project.`,
+      );
+      console.log(
+        `  Run ${CYAN}npx forgedock init${RESET} inside your project directory to generate it.`,
+      );
+      console.log("");
+    }
   }
 }
 
@@ -180,14 +272,23 @@ async function uninstall() {
 
     try {
       const stats = await lstat(target);
+      const { unlink } = await import("fs/promises");
       if (stats.isSymbolicLink()) {
         const current = await readlink(target);
         if (current === file) {
-          const { unlink } = await import("fs/promises");
           await unlink(target);
           console.log(`  ${RED}Removed${RESET}: ${rel}`);
           removed++;
         }
+      } else if (
+        readFileSync(file, "utf-8") === readFileSync(target, "utf-8")
+      ) {
+        // Regular file installed by ForgeDock in copy mode (content matches the
+        // shipped command). Safe to remove. Content-differing files are left
+        // in place in case the user edited them.
+        await unlink(target);
+        console.log(`  ${RED}Removed${RESET}: ${rel}`);
+        removed++;
       }
     } catch (err) {
       if (err.code !== "ENOENT") {
@@ -511,15 +612,15 @@ project:
   name: "${projectName}"
   owner: "${owner}"
   repo: "${repo}"
-  description: "${description.replace(/\\/g, '\\\\').replace(/"/g, '\\"')}"
+  description: "${description.replace(/\\/g, "\\\\").replace(/"/g, '\\"')}"
 
 # =============================================================================
 # PATHS (REQUIRED)
 # =============================================================================
 
 paths:
-  root: "${cwd.replace(/\\/g, '\\\\').replace(/"/g, '\\"')}"
-  worktree_base: "${worktreeBase.replace(/\\/g, '\\\\').replace(/"/g, '\\"')}"
+  root: "${cwd.replace(/\\/g, "\\\\").replace(/"/g, '\\"')}"
+  worktree_base: "${worktreeBase.replace(/\\/g, "\\\\").replace(/"/g, '\\"')}"
 
 # =============================================================================
 # BRANCHES (REQUIRED)
@@ -583,7 +684,18 @@ branches:
 #     - '"status": "ok"'
 `;
 
-  writeFileSync(outputPath, content, "utf-8");
+  try {
+    writeFileSync(outputPath, content, "utf-8");
+  } catch (err) {
+    console.error("");
+    console.error(
+      `  ${RED}Error${RESET}: Could not write forge.yaml to ${outputPath} — ${err.code ?? err.message}`,
+    );
+    console.error(
+      `  Run ${CYAN}npx forgedock init${RESET} from a writable project directory.`,
+    );
+    process.exit(1);
+  }
 
   console.log(`  ${GREEN}Created${RESET}: forge.yaml`);
   console.log("");
@@ -641,6 +753,8 @@ function help() {
   console.log("");
 }
 
+splash();
+
 switch (command) {
   case "install":
     await install();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "forgedock",
-  "version": "1.0.11",
+  "version": "1.0.13",
   "description": "GitHub as a knowledge graph for AI agents. Autonomous dev pipeline for Claude Code.",
   "bin": {
     "forgedock": "bin/forgedock.mjs"