@buildinternet/uploads 0.42.1 → 0.42.2

package/dist/hooks-install.d.ts

@@ -2,12 +2,20 @@
  * Install thin user-global hook manifests for Grok and Cursor.
  *
  * Claude and Codex ship the same PreToolUse hook via their plugins
- * (`hooks/hooks.json` → `uploads hook pre-pr-screenshot`). Do not also write
+ * (`hooks/hooks.json` → HOOK_COMMAND). Do not also write
  * ~/.codex/hooks.json here, or the reminder would fire twice.
  *
  * Idempotent: skip when our command string is already present.
+ * An older bare `uploads hook …` entry is upgraded in place.
  */
-export declare const HOOK_COMMAND = "uploads hook pre-pr-screenshot";
+/** What the harness actually runs once `uploads` is on PATH. */
+export declare const HOOK_INVOCATION = "uploads hook pre-pr-screenshot";
+/**
+ * Fail-open shell used in every harness manifest.
+ * A missing `uploads` binary is a silent no-op (exit 0, no stderr) so plugin
+ * users without the CLI do not get a hook-failure annotation on every shell tool.
+ */
+export declare const HOOK_COMMAND = "if command -v uploads >/dev/null 2>&1; then uploads hook pre-pr-screenshot; fi";
 export type HookWriteResult = {
     path: string;
     action: "wrote" | "merged" | "skipped" | "would-write" | "would-merge";

package/dist/hooks-install.js

@@ -2,15 +2,23 @@
  * Install thin user-global hook manifests for Grok and Cursor.
  *
  * Claude and Codex ship the same PreToolUse hook via their plugins
- * (`hooks/hooks.json` → `uploads hook pre-pr-screenshot`). Do not also write
+ * (`hooks/hooks.json` → HOOK_COMMAND). Do not also write
  * ~/.codex/hooks.json here, or the reminder would fire twice.
  *
  * Idempotent: skip when our command string is already present.
+ * An older bare `uploads hook …` entry is upgraded in place.
  */
 import fs from "node:fs";
 import os from "node:os";
 import path from "node:path";
-export const HOOK_COMMAND = "uploads hook pre-pr-screenshot";
+/** What the harness actually runs once `uploads` is on PATH. */
+export const HOOK_INVOCATION = "uploads hook pre-pr-screenshot";
+/**
+ * Fail-open shell used in every harness manifest.
+ * A missing `uploads` binary is a silent no-op (exit 0, no stderr) so plugin
+ * users without the CLI do not get a hook-failure annotation on every shell tool.
+ */
+export const HOOK_COMMAND = `if command -v uploads >/dev/null 2>&1; then ${HOOK_INVOCATION}; fi`;
 const TIMEOUT_SEC = 15;
 const GROK_PAYLOAD = {
     description: "Advisory reminder to stage screenshots on uploads.sh before opening a PR that touches UI files.",
@@ -38,6 +46,16 @@ function containsCommand(filePath) {
         return false;
     }
 }
+function hookCommandOf(entry) {
+    if (!entry || typeof entry !== "object")
+        return undefined;
+    const command = entry.command;
+    return typeof command === "string" ? command : undefined;
+}
+function isOurHookEntry(entry) {
+    const command = hookCommandOf(entry);
+    return Boolean(command?.includes(HOOK_INVOCATION));
+}
 function writeJson(filePath, value) {
     fs.mkdirSync(path.dirname(filePath), { recursive: true });
     fs.writeFileSync(filePath, `${JSON.stringify(value, null, 2)}\n`, "utf8");
@@ -79,7 +97,17 @@ function mergeCursor(home, dryRun) {
         ? { ...existing.hooks }
         : {};
     const list = Array.isArray(hooks.beforeShellExecution) ? [...hooks.beforeShellExecution] : [];
-    list.push({ command: HOOK_COMMAND, timeout: TIMEOUT_SEC });
+    const ours = list.findIndex(isOurHookEntry);
+    const nextEntry = { command: HOOK_COMMAND, timeout: TIMEOUT_SEC };
+    if (ours >= 0) {
+        if (hookCommandOf(list[ours]) === HOOK_COMMAND) {
+            return { path: filePath, action: "skipped" };
+        }
+        list[ours] = { ...list[ours], ...nextEntry };
+    }
+    else {
+        list.push(nextEntry);
+    }
     hooks.beforeShellExecution = list;
     existing.hooks = hooks;
     if (existing.version === undefined)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@buildinternet/uploads",
-  "version": "0.42.1",
+  "version": "0.42.2",
   "description": "CLI and client for uploads.sh — workspace-scoped image hosting for GitHub embeds",
   "type": "module",
   "sideEffects": false,