@indigoai-us/hq-cli 5.49.0 → 5.50.0

package/dist/index.js

@@ -3,7 +3,7 @@
  * HQ CLI - Module management, package management, and cloud sync for HQ
  */
 
-!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="60f911e0-ddd5-5a2d-91c7-18ba47457b57")}catch(e){}}();
+!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="d57c1cc3-6dd3-5c7f-8eef-c7692d41b836")}catch(e){}}();
 import { Command } from "commander";
 import { initSentry, Sentry } from "./sentry.js";
 import { registerAddCommand } from "./commands/add.js";
@@ -45,6 +45,7 @@ import { registerSignalsCommand } from "./commands/signals.js";
 import { registerReindexCommand } from "./commands/reindex.js";
 import { registerRescueCommand } from "./commands/rescue.js";
 import { sanitizeArgv } from "./utils/feedback-diagnostics.js";
+import { environmentalFsErrorMessage } from "./utils/environmental-error.js";
 import { maybeWarnNewVersion, refreshVersionCache, } from "./utils/version-check.js";
 import { enforceVersionGate, shouldSkipGate, } from "./utils/version-gate.js";
 import { CLI_VERSION } from "./cli-version.js";
@@ -176,7 +177,18 @@ registerRescueCommand(program);
         await program.parseAsync();
     }
     catch (err) {
-        Sentry.captureException(err);
+        // A full disk / exhausted quota / read-only filesystem is the user's
+        // machine, not an HQ code defect. Surface a clear, actionable message and
+        // skip Sentry capture so one full disk doesn't flood the tracker with
+        // identical, unfixable crash reports (HQ-CLI-2). Genuine errors still go
+        // to Sentry and still exit 1.
+        const envMsg = environmentalFsErrorMessage(err);
+        if (envMsg) {
+            process.stderr.write(`hq: ${envMsg}\n`);
+        }
+        else {
+            Sentry.captureException(err);
+        }
         process.exitCode = 1;
     }
     finally {
@@ -186,4 +198,4 @@ registerRescueCommand(program);
     }
 })();
 //# sourceMappingURL=index.js.map
-//# debugId=60f911e0-ddd5-5a2d-91c7-18ba47457b57
+//# debugId=d57c1cc3-6dd3-5c7f-8eef-c7692d41b836

package/dist/utils/environmental-error.d.ts

@@ -0,0 +1,10 @@
+/**
+ * If `err` is an environmental disk/quota/read-only filesystem error, return a
+ * short, user-facing message explaining it; otherwise return `null`.
+ *
+ * A non-null result means the caller should print the message and SKIP Sentry
+ * capture — the condition is the user's machine, not a bug HQ can fix. A null
+ * result means "this is a normal error; handle it as usual (capture to Sentry)".
+ */
+export declare function environmentalFsErrorMessage(err: unknown): string | null;
+//# sourceMappingURL=environmental-error.d.ts.map

package/dist/utils/environmental-error.js

@@ -0,0 +1,40 @@
+// src/utils/environmental-error.ts
+//
+// Classify errors that stem from the user's machine/filesystem state rather
+// than an HQ code defect. These are unactionable from our side: a full disk, an
+// exhausted quota, or a read-only filesystem. The CLI surfaces a clear,
+// actionable message and does NOT report them to Sentry — otherwise a single
+// full disk floods the issue tracker with identical, unfixable crash reports.
+//
+// HQ-CLI-2: `hq reindex` hit ENOSPC in the operation-lock temp-file write
+// (`fs.openSync` → "ENOSPC: no space left on device, open") and the raw error
+// propagated uncaught to the CLI's top-level handler, which captured it to
+// Sentry and exited silently — 5 stack-trace crashes in 6 seconds, with no
+// message telling the user their disk was full.
+/**
+ * Node errno codes for "the filesystem cannot accept this write" — purely
+ * environmental, never a code bug. Mapped to the message shown to the user.
+ */
+
+!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="03976a9c-d4c3-57dc-bd1d-40c9d025193d")}catch(e){}}();
+const ENVIRONMENTAL_FS_CODES = {
+    ENOSPC: "No space left on device. Free up disk space and try again.",
+    EDQUOT: "Disk quota exceeded. Free up space (or raise your quota) and try again.",
+    EROFS: "The filesystem is read-only, so HQ can't write here. Check the mount/permissions and try again.",
+};
+/**
+ * If `err` is an environmental disk/quota/read-only filesystem error, return a
+ * short, user-facing message explaining it; otherwise return `null`.
+ *
+ * A non-null result means the caller should print the message and SKIP Sentry
+ * capture — the condition is the user's machine, not a bug HQ can fix. A null
+ * result means "this is a normal error; handle it as usual (capture to Sentry)".
+ */
+export function environmentalFsErrorMessage(err) {
+    const code = err?.code;
+    if (typeof code !== "string")
+        return null;
+    return ENVIRONMENTAL_FS_CODES[code] ?? null;
+}
+//# sourceMappingURL=environmental-error.js.map
+//# debugId=03976a9c-d4c3-57dc-bd1d-40c9d025193d

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@indigoai-us/hq-cli",
-  "version": "5.49.0",
+  "version": "5.50.0",
   "description": "HQ by Indigo management CLI — modules and cloud sync",
   "main": "dist/index.js",
   "bin": {
@@ -18,7 +18,7 @@
     "clean": "rm -rf dist"
   },
   "dependencies": {
-    "@indigoai-us/hq-cloud": "^6.11.14",
+    "@indigoai-us/hq-cloud": "^6.12.0",
     "@indigoai-us/hq-onboarding": "^0.1.0",
     "@sentry/node": "^10.49.0",
     "chalk": "^5.3.0",

package/src/index.ts

@@ -45,6 +45,7 @@ import { registerSignalsCommand } from "./commands/signals.js";
 import { registerReindexCommand } from "./commands/reindex.js";
 import { registerRescueCommand } from "./commands/rescue.js";
 import { sanitizeArgv } from "./utils/feedback-diagnostics.js";
+import { environmentalFsErrorMessage } from "./utils/environmental-error.js";
 import {
   maybeWarnNewVersion,
   refreshVersionCache,
@@ -215,7 +216,17 @@ registerRescueCommand(program);
     }
     await program.parseAsync();
   } catch (err) {
-    Sentry.captureException(err);
+    // A full disk / exhausted quota / read-only filesystem is the user's
+    // machine, not an HQ code defect. Surface a clear, actionable message and
+    // skip Sentry capture so one full disk doesn't flood the tracker with
+    // identical, unfixable crash reports (HQ-CLI-2). Genuine errors still go
+    // to Sentry and still exit 1.
+    const envMsg = environmentalFsErrorMessage(err);
+    if (envMsg) {
+      process.stderr.write(`hq: ${envMsg}\n`);
+    } else {
+      Sentry.captureException(err);
+    }
     process.exitCode = 1;
   } finally {
     // Release health: finalize the per-run session before the flush.

package/src/utils/environmental-error.test.ts

@@ -0,0 +1,45 @@
+import { describe, it, expect } from "vitest";
+import { environmentalFsErrorMessage } from "./environmental-error.js";
+
+/** Build a Node-style errno error with a `.code`, as fs.*Sync throws. */
+function errnoError(code: string, message: string): NodeJS.ErrnoException {
+  const err = new Error(message) as NodeJS.ErrnoException;
+  err.code = code;
+  return err;
+}
+
+describe("environmentalFsErrorMessage", () => {
+  // HQ-CLI-2: the exact error that crashed `hq reindex` and flooded Sentry.
+  it("classifies ENOSPC (no space left on device) as environmental", () => {
+    const err = errnoError("ENOSPC", "ENOSPC: no space left on device, open '/x'");
+    const msg = environmentalFsErrorMessage(err);
+    expect(msg).not.toBeNull();
+    expect(msg).toMatch(/no space left on device/i);
+  });
+
+  it("classifies EDQUOT (quota exceeded) as environmental", () => {
+    const msg = environmentalFsErrorMessage(errnoError("EDQUOT", "EDQUOT: disk quota exceeded"));
+    expect(msg).toMatch(/quota/i);
+  });
+
+  it("classifies EROFS (read-only filesystem) as environmental", () => {
+    const msg = environmentalFsErrorMessage(errnoError("EROFS", "EROFS: read-only file system"));
+    expect(msg).toMatch(/read-only/i);
+  });
+
+  // A genuine code bug must still reach Sentry — only the disk-class codes are
+  // diverted, so we never silently swallow real defects.
+  it("returns null for a code-bug error (ENOENT) so it is still captured", () => {
+    expect(environmentalFsErrorMessage(errnoError("ENOENT", "ENOENT: not found"))).toBeNull();
+  });
+
+  it("returns null for a plain Error with no code", () => {
+    expect(environmentalFsErrorMessage(new Error("boom"))).toBeNull();
+  });
+
+  it("returns null for non-error values (null/undefined/string)", () => {
+    expect(environmentalFsErrorMessage(null)).toBeNull();
+    expect(environmentalFsErrorMessage(undefined)).toBeNull();
+    expect(environmentalFsErrorMessage("ENOSPC")).toBeNull();
+  });
+});

package/src/utils/environmental-error.ts

@@ -0,0 +1,39 @@
+// src/utils/environmental-error.ts
+//
+// Classify errors that stem from the user's machine/filesystem state rather
+// than an HQ code defect. These are unactionable from our side: a full disk, an
+// exhausted quota, or a read-only filesystem. The CLI surfaces a clear,
+// actionable message and does NOT report them to Sentry — otherwise a single
+// full disk floods the issue tracker with identical, unfixable crash reports.
+//
+// HQ-CLI-2: `hq reindex` hit ENOSPC in the operation-lock temp-file write
+// (`fs.openSync` → "ENOSPC: no space left on device, open") and the raw error
+// propagated uncaught to the CLI's top-level handler, which captured it to
+// Sentry and exited silently — 5 stack-trace crashes in 6 seconds, with no
+// message telling the user their disk was full.
+
+/**
+ * Node errno codes for "the filesystem cannot accept this write" — purely
+ * environmental, never a code bug. Mapped to the message shown to the user.
+ */
+const ENVIRONMENTAL_FS_CODES: Record<string, string> = {
+  ENOSPC: "No space left on device. Free up disk space and try again.",
+  EDQUOT:
+    "Disk quota exceeded. Free up space (or raise your quota) and try again.",
+  EROFS:
+    "The filesystem is read-only, so HQ can't write here. Check the mount/permissions and try again.",
+};
+
+/**
+ * If `err` is an environmental disk/quota/read-only filesystem error, return a
+ * short, user-facing message explaining it; otherwise return `null`.
+ *
+ * A non-null result means the caller should print the message and SKIP Sentry
+ * capture — the condition is the user's machine, not a bug HQ can fix. A null
+ * result means "this is a normal error; handle it as usual (capture to Sentry)".
+ */
+export function environmentalFsErrorMessage(err: unknown): string | null {
+  const code = (err as NodeJS.ErrnoException | null | undefined)?.code;
+  if (typeof code !== "string") return null;
+  return ENVIRONMENTAL_FS_CODES[code] ?? null;
+}