@fabricorg/cli 0.3.3 → 0.3.5

package/dist/fabric.js

@@ -57,7 +57,7 @@ import { Command as Command20 } from "commander";
 // package.json
 var package_default = {
   name: "@fabricorg/cli",
-  version: "0.3.3",
+  version: "0.3.5",
   description: "CLI for the Fabric AI platform",
   keywords: [
     "fabric",
@@ -1371,7 +1371,10 @@ var DRIVE_LETTER = /^[A-Za-z]:/;
 var RESERVED_DEVICE_NAME = /^(?:CON|PRN|AUX|NUL|COM[1-9]|LPT[1-9])(?:\.|$)/i;
 var FORBIDDEN_NAME_CHARACTER = /[<>:"|?*]/;
 var RESERVED_ROOTS = [".git", ".fabric"];
-var RESERVED_EXACT_PATHS = [".claude/settings.local.json"];
+var RESERVED_EXACT_PATHS = [
+  ".claude/settings.local.json",
+  ".codex/hooks.json"
+];
 function hasControlCharacter(input) {
   for (let index = 0; index < input.length; index++) {
     const code = input.charCodeAt(index);
@@ -1492,6 +1495,7 @@ function describeRejection(rejection) {
 
 // src/lib/instructions/safe-write.ts
 import { createHash, randomBytes } from "crypto";
+import { constants } from "fs";
 import {
   lstat,
   mkdir,
@@ -1587,24 +1591,79 @@ function assertWritableTarget(root, relativePath) {
     return check;
   });
 }
-function readFileSafely(root, relativePath) {
-  return __async(this, null, function* () {
+function readFileSafely(_0, _1) {
+  return __async(this, arguments, function* (root, relativePath, options = {}) {
     const check = yield assertWritableTarget(root, relativePath);
     if (!check.ok) {
       throw new Error(`Refusing to sync: ${describeRejection(check)}.`);
     }
-    const stats = yield lstatOrNull(check.path);
-    if (stats === null) {
-      return null;
-    }
+    let handle;
     try {
-      return { bytes: yield readFile(check.path), mode: stats.mode };
+      handle = yield open(
+        check.path,
+        constants.O_RDONLY | constants.O_NOFOLLOW | constants.O_NONBLOCK
+      );
     } catch (error) {
       if (error.code === "ENOENT") {
         return null;
       }
       throw error;
     }
+    try {
+      const opened = yield handle.stat();
+      if (!opened.isFile()) {
+        throw new Error(
+          `Refusing to sync: ${relativePath} is not a regular file.`
+        );
+      }
+      const afterOpen = yield assertWritableTarget(root, relativePath);
+      if (!afterOpen.ok) {
+        throw new Error(
+          `Refusing to sync: ${describeRejection(afterOpen)}.`
+        );
+      }
+      const current = yield lstatOrNull(afterOpen.path);
+      if (current === null || !current.isFile() || current.dev !== opened.dev || current.ino !== opened.ino) {
+        throw new Error(
+          `Refusing to sync: ${relativePath} changed while it was being opened.`
+        );
+      }
+      if (options.maxBytes === void 0) {
+        return { bytes: yield handle.readFile(), mode: opened.mode };
+      }
+      if (!Number.isSafeInteger(options.maxBytes) || options.maxBytes < 0) {
+        throw new Error(
+          "A safe file-read limit must be a non-negative integer."
+        );
+      }
+      if (opened.size > options.maxBytes) {
+        throw new Error(
+          `${relativePath} is too large to read safely (maximum ${options.maxBytes} bytes).`
+        );
+      }
+      const buffer = Buffer.allocUnsafe(options.maxBytes + 1);
+      let total = 0;
+      while (total <= options.maxBytes) {
+        const { bytesRead } = yield handle.read(
+          buffer,
+          total,
+          options.maxBytes + 1 - total,
+          total
+        );
+        if (bytesRead === 0) {
+          break;
+        }
+        total += bytesRead;
+      }
+      if (total > options.maxBytes) {
+        throw new Error(
+          `${relativePath} is too large to read safely (maximum ${options.maxBytes} bytes).`
+        );
+      }
+      return { bytes: buffer.subarray(0, total), mode: opened.mode };
+    } finally {
+      yield handle.close();
+    }
   });
 }
 function resolveExistingRoot(destination) {
@@ -2022,14 +2081,16 @@ function extractBundle(archive, wanted) {
 }
 
 // src/lib/instructions/hook.ts
-import { readFile as readFile2 } from "fs/promises";
 import path2 from "path";
 var CLAUDE_SETTINGS_RELATIVE_PATH = path2.join(
   ".claude",
   "settings.local.json"
 );
+var CODEX_HOOKS_RELATIVE_PATH = path2.join(".codex", "hooks.json");
 var CLAUDE_SETTINGS_POSIX_PATH = ".claude/settings.local.json";
+var CODEX_HOOKS_POSIX_PATH = ".codex/hooks.json";
 var HOOK_TIMEOUT_SECONDS = 15;
+var MAX_HOOK_CONFIG_BYTES = 1024 * 1024;
 var HOOK_SUBCOMMANDS = /* @__PURE__ */ new Set(["check", "sync"]);
 function buildHookCommand(projectId, apply, org) {
   const verb = apply ? "sync" : "check";
@@ -2047,17 +2108,25 @@ function assertKeyStaysOutside(destination, configPath) {
 }
 function mergeSessionStartHook(input) {
   return __async(this, null, function* () {
-    const settingsPath = path2.join(input.root, CLAUDE_SETTINGS_RELATIVE_PATH);
-    let existingRaw = null;
+    var _a2;
+    const hookTarget = hookTargetFor((_a2 = input.tool) != null ? _a2 : "claude-code");
+    const settingsPath = path2.join(input.root, hookTarget.relativePath);
+    let existingRaw;
     try {
-      existingRaw = yield readFile2(settingsPath, "utf8");
+      const existing = yield readFileSafely(
+        input.root,
+        hookTarget.posixPath,
+        { maxBytes: MAX_HOOK_CONFIG_BYTES }
+      );
+      existingRaw = existing === null ? null : new TextDecoder().decode(existing.bytes);
     } catch (error) {
-      if (error.code !== "ENOENT") {
-        throw error;
-      }
+      const message = error instanceof Error ? error.message : String(error);
+      throw new Error(
+        `${settingsPath} could not be read safely: ${message}. It was left untouched.`
+      );
     }
     let settings = {};
-    if (existingRaw !== null && existingRaw.trim().length > 0) {
+    if (existingRaw !== null) {
       let parsed;
       try {
         parsed = JSON.parse(existingRaw);
@@ -2124,7 +2193,7 @@ function mergeSessionStartHook(input) {
     });
     yield writeFileSafely({
       root: input.root,
-      relativePath: CLAUDE_SETTINGS_POSIX_PATH,
+      relativePath: hookTarget.posixPath,
       bytes: new TextEncoder().encode(
         `${JSON.stringify(next, null, 2).replace(/\r\n/g, "\n")}
 `
@@ -2138,6 +2207,15 @@ function mergeSessionStartHook(input) {
     };
   });
 }
+function hookTargetFor(tool) {
+  return tool === "codex" ? {
+    relativePath: CODEX_HOOKS_RELATIVE_PATH,
+    posixPath: CODEX_HOOKS_POSIX_PATH
+  } : {
+    relativePath: CLAUDE_SETTINGS_RELATIVE_PATH,
+    posixPath: CLAUDE_SETTINGS_POSIX_PATH
+  };
+}
 function isFabricHookFor(hook, projectId) {
   if (!isRecord(hook) || typeof hook.command !== "string") {
     return false;
@@ -2179,7 +2257,7 @@ function isRecord(value) {
 }
 
 // src/lib/instructions/lock.ts
-import { readFile as readFile3 } from "fs/promises";
+import { readFile as readFile2 } from "fs/promises";
 import path3 from "path";
 var LOCK_DIRECTORY = ".fabric";
 var LOCK_FILENAME = "instructions.lock";
@@ -2194,7 +2272,7 @@ function readLock(destination) {
     const file = lockPath(destination);
     let raw;
     try {
-      raw = yield readFile3(file, "utf8");
+      raw = yield readFile2(file, "utf8");
     } catch (error) {
       if (error.code === "ENOENT") {
         return null;
@@ -2766,12 +2844,15 @@ function buildInstructionsCommand() {
     });
   });
   instructions.command("push").description(
-    "Suggest this checkout's edits to the project's coding instructions"
+    "Propose this checkout's edits to the project's coding instructions, or publish them with --publish"
   ).requiredOption("--project <id>", "Project ID").option("--dest <dir>", "Destination directory (default: cwd)").option("--org <slug>", "Organization context").option(
     "--add <path>",
     "Also send a file the last sync did not write (repeatable)",
     collectAdded,
     []
+  ).option(
+    "--publish",
+    "Publish the change as a new version instead of proposing it (needs a key with instructions:publish)"
   ).option("--dry-run", "Print the change set and send nothing").option("--format <format>", "Output format: text|json").action(function(opts) {
     return __async(this, null, function* () {
       yield run(
@@ -2785,7 +2866,7 @@ function buildInstructionsCommand() {
     "Write a SessionStart hook for a coding tool and take the first copy"
   ).requiredOption("--project <id>", "Project ID").requiredOption(
     "--tool <tool>",
-    "Coding tool to configure: claude-code"
+    "Coding tool to configure: claude-code|codex"
   ).option("--dest <dir>", "Destination directory (default: cwd)").option("--org <slug>", "Organization context").option(
     "--apply",
     "Let the hook apply changes instead of only reporting them"
@@ -3269,6 +3350,7 @@ function runPush(opts, format) {
       projectId: opts.project,
       destination: root,
       dryRun: Boolean(opts.dryRun),
+      publish: Boolean(opts.publish),
       baseSnapshotId: lock.snapshotId,
       baseVersion: lock.snapshotVersion,
       put: plan.entries.filter((entry) => entry.action === "put").map((entry) => entry.path),
@@ -3277,11 +3359,17 @@ function runPush(opts, format) {
       snapshotId: null,
       version: null,
       proposalStatus: null,
-      status: null
+      status: null,
+      published: null
     };
     if (!opts.dryRun) {
       try {
-        const submitted = yield client.instructions.submitChange(
+        const submitted = opts.publish ? yield client.instructions.publishChange(
+          opts.project,
+          lock.snapshotId,
+          plan.changes,
+          { org: orgSlugFor(opts) }
+        ) : yield client.instructions.submitChange(
           opts.project,
           lock.snapshotId,
           plan.changes,
@@ -3291,6 +3379,7 @@ function runPush(opts, format) {
         outcome.version = submitted.version;
         outcome.proposalStatus = submitted.proposalStatus;
         outcome.status = submitted.status;
+        outcome.published = submitted.published;
       } catch (error) {
         throw asPushFailure(error);
       }
@@ -3330,6 +3419,16 @@ function asPushFailure(error) {
 }
 function pushVerdict(outcome) {
   var _a2;
+  if (outcome.publish) {
+    if (outcome.published) {
+      return `Published version ${outcome.version}. Run \`fabric instructions sync\` to bring this checkout onto it.`;
+    }
+    if (outcome.status === "FAILED" || outcome.status === "REJECTED") {
+      return `Version ${outcome.version} did not pass its checks (${outcome.status}), so nothing was published; open the project's Coding Instructions tab to see why.`;
+    }
+    const sent = `Sent as version ${outcome.version}. It publishes on its own once its checks pass (${outcome.status}) \u2014 then run \`fabric instructions sync\` to bring this checkout onto it.`;
+    return outcome.status === "READY" ? `${sent} It has passed its checks; its publication state is shown in the project's Coding Instructions tab.` : sent;
+  }
   if (outcome.proposalStatus === "PENDING" && outcome.status === "RECEIVING") {
     return `An earlier attempt is still sending this change; if version ${outcome.version} stays unfinished, cancel it in the project's Coding Instructions tab, or leave it and it is closed out automatically after six hours.`;
   }
@@ -3356,23 +3455,24 @@ function reportPush(outcome) {
   listPaths("deleted", outcome.deleted);
   if (outcome.dryRun) {
     line(
-      "Nothing was sent. Without --dry-run this would open a proposal for review."
+      outcome.publish ? "Nothing was sent. Without --dry-run this would create a new version that publishes on its own once its checks pass, with no review." : "Nothing was sent. Without --dry-run this would open a proposal for review."
     );
     return;
   }
   line(pushVerdict(outcome));
   line(
-    "Your lock was not changed: it still names the published version, which is what `fabric instructions sync` compares against."
+    outcome.publish ? `Your lock was not changed: it still names version ${outcome.baseVersion}, which is what \`fabric instructions sync\` compares against.` : "Your lock was not changed: it still names the published version, which is what `fabric instructions sync` compares against."
   );
 }
 function runInit(opts, format) {
   return __async(this, null, function* () {
-    if (opts.tool !== "claude-code") {
+    if (!isInstructionsHookTool(opts.tool)) {
       throw new CliFailure(
-        `Unsupported tool "${opts.tool}". Only --tool claude-code is supported today; other tools follow.`,
+        `Unsupported tool "${opts.tool}". Use --tool claude-code or --tool codex.`,
         2
       );
     }
+    const tool = opts.tool;
     const root = yield resolveDestinationRoot(destinationOf(opts));
     yield readLockForProject(root, opts.project);
     try {
@@ -3403,7 +3503,8 @@ function runInit(opts, format) {
     const merged = yield mergeSessionStartHook({
       root,
       projectId: opts.project,
-      command
+      command,
+      tool
     });
     if (format === "json") {
       printOutput(
@@ -3427,6 +3528,11 @@ function runInit(opts, format) {
     line(
       opts.apply ? "  It applies changes at session start." : "  It only reports changes; add --apply to init to have it apply them."
     );
+    if (tool === "codex") {
+      line(
+        "  Start Codex in this checkout, then use `/hooks` to review and trust the project hook."
+      );
+    }
     if (outcome === null) {
       line(
         "This project has nothing published yet, so there is nothing to copy \u2014 the hook will pick it up once there is."
@@ -3437,10 +3543,16 @@ function runInit(opts, format) {
     }
     line("");
     line(
-      `${CLAUDE_SETTINGS_RELATIVE_PATH} and ${LOCK_DIRECTORY}/ are local to this machine. If this repository does not ignore them already, add them to your own ignore rules \u2014 this command does not edit .gitignore.`
+      `${hookPathFor(tool)} and ${LOCK_DIRECTORY}/ are local to this machine. If this repository does not ignore them already, add them to your own ignore rules \u2014 this command does not edit .gitignore.`
     );
   });
 }
+function isInstructionsHookTool(tool) {
+  return tool === "claude-code" || tool === "codex";
+}
+function hookPathFor(tool) {
+  return tool === "codex" ? CODEX_HOOKS_RELATIVE_PATH : CLAUDE_SETTINGS_RELATIVE_PATH;
+}
 
 // src/commands/mcp/index.ts
 import { Command as Command12 } from "commander";