@yagni-app/code-staging 0.3.0-staging.1099.1 → 0.3.1-staging.1103.1

package/dist/crashReport.d.ts

@@ -29,6 +29,18 @@ export declare const MAX_CRASH_STACK_FRAMES = 40;
 export declare const MAX_CRASH_PAYLOAD_BYTES = 16384;
 /** Same truthiness rule as updateChecksDisabled: set and not "" / "0". */
 export declare function crashReportsDisabled(env?: NodeJS.ProcessEnv): boolean;
+/**
+ * True when this process is a test run. A crash reporter that phones home
+ * from a test suite files FICTIONAL crashes against real production
+ * telemetry — see the twin comment in `pi-extension-yagni/src/crashReport.ts`
+ * (Sentry YAGNI-BACKEND-2P / -2Q). Kept in sync with that copy.
+ *
+ * Deliberately broad: a false positive costs one unreported crash on a
+ * developer machine, a false negative pollutes production.
+ */
+export declare function runningUnderTest(env?: NodeJS.ProcessEnv): boolean;
+/** Reporting is off when the user disabled it OR this is a test process. */
+export declare function crashReportsSuppressed(env?: NodeJS.ProcessEnv): boolean;
 export interface SanitizeCrashOptions {
     /** Environment whose values get redacted (defaults to process.env). */
     env?: NodeJS.ProcessEnv;

package/dist/crashReport.js

@@ -33,6 +33,33 @@ export function crashReportsDisabled(env = process.env) {
     const value = env[CRASH_REPORT_DISABLE_ENV];
     return value !== undefined && value !== "" && value !== "0";
 }
+/**
+ * True when this process is a test run. A crash reporter that phones home
+ * from a test suite files FICTIONAL crashes against real production
+ * telemetry — see the twin comment in `pi-extension-yagni/src/crashReport.ts`
+ * (Sentry YAGNI-BACKEND-2P / -2Q). Kept in sync with that copy.
+ *
+ * Deliberately broad: a false positive costs one unreported crash on a
+ * developer machine, a false negative pollutes production.
+ */
+export function runningUnderTest(env = process.env) {
+    if (env.NODE_ENV === "test")
+        return true;
+    // node:test sets this in every spawned test-file process.
+    if (env.NODE_TEST_CONTEXT)
+        return true;
+    if (env.VITEST || env.JEST_WORKER_ID)
+        return true;
+    // npm/pnpm export the script name being run (`pnpm test`, `pnpm test:file`).
+    const script = env.npm_lifecycle_event;
+    if (script === "test" || (script !== undefined && script.startsWith("test:")))
+        return true;
+    return false;
+}
+/** Reporting is off when the user disabled it OR this is a test process. */
+export function crashReportsSuppressed(env = process.env) {
+    return crashReportsDisabled(env) || runningUnderTest(env);
+}
 // Mirrors scrubSecrets (backend yagniCode/scrubSecrets.ts and
 // pi-extension-yagni pipeline/scrubSecrets.ts) — keep in sync.
 const SECRET_PATTERNS = [
@@ -147,7 +174,7 @@ export function sanitizeCrashError(err, opts = {}) {
 export async function sendCrashReport(input) {
     try {
         const env = input.env ?? process.env;
-        if (crashReportsDisabled(env))
+        if (crashReportsSuppressed(env))
             return;
         const opts = { env, repoRoot: input.repoRoot };
         const stack = input.stack !== undefined

package/dist/extension/crashReport.d.ts

@@ -33,6 +33,24 @@ export declare const MAX_CRASH_STACK_FRAMES = 40;
 export declare const MAX_CRASH_PAYLOAD_BYTES = 16384;
 /** Same truthiness rule as the CLI's YAGNI_DISABLE_* family. */
 export declare function crashReportsDisabled(env?: NodeJS.ProcessEnv): boolean;
+/**
+ * True when this process is a test run.
+ *
+ * A crash reporter that phones home from a test suite reports FICTIONAL
+ * crashes against real production telemetry. That is not theoretical: the
+ * `/go` terminal catch builds a live reporter whenever its caller doesn't
+ * inject one, so `goCommand.test.ts`'s deliberate `throw new Error("boom")`
+ * POSTed to https://yagni.app on every run — 915 events across Sentry
+ * YAGNI-BACKEND-2P / -2Q, from a team that had never witnessed a crash.
+ *
+ * Injecting a stub at each call site fixes one test; this makes the whole
+ * class impossible, so a future test that forgets cannot re-open the leak.
+ * Detection is deliberately broad — a false positive costs one unreported
+ * crash on a developer machine, a false negative pollutes production.
+ */
+export declare function runningUnderTest(env?: NodeJS.ProcessEnv): boolean;
+/** Reporting is off when the user disabled it OR this is a test process. */
+export declare function crashReportsSuppressed(env?: NodeJS.ProcessEnv): boolean;
 export interface SanitizeCrashOptions {
     env?: NodeJS.ProcessEnv;
     repoRoot?: string;

package/dist/extension/crashReport.js

@@ -40,6 +40,39 @@ export function crashReportsDisabled(env = process.env) {
     const value = env[CRASH_REPORT_DISABLE_ENV];
     return value !== undefined && value !== "" && value !== "0";
 }
+/**
+ * True when this process is a test run.
+ *
+ * A crash reporter that phones home from a test suite reports FICTIONAL
+ * crashes against real production telemetry. That is not theoretical: the
+ * `/go` terminal catch builds a live reporter whenever its caller doesn't
+ * inject one, so `goCommand.test.ts`'s deliberate `throw new Error("boom")`
+ * POSTed to https://yagni.app on every run — 915 events across Sentry
+ * YAGNI-BACKEND-2P / -2Q, from a team that had never witnessed a crash.
+ *
+ * Injecting a stub at each call site fixes one test; this makes the whole
+ * class impossible, so a future test that forgets cannot re-open the leak.
+ * Detection is deliberately broad — a false positive costs one unreported
+ * crash on a developer machine, a false negative pollutes production.
+ */
+export function runningUnderTest(env = process.env) {
+    if (env.NODE_ENV === "test")
+        return true;
+    // node:test sets this in every spawned test-file process.
+    if (env.NODE_TEST_CONTEXT)
+        return true;
+    if (env.VITEST || env.JEST_WORKER_ID)
+        return true;
+    // npm/pnpm export the script name being run (`pnpm test`, `pnpm test:file`).
+    const script = env.npm_lifecycle_event;
+    if (script === "test" || (script !== undefined && script.startsWith("test:")))
+        return true;
+    return false;
+}
+/** Reporting is off when the user disabled it OR this is a test process. */
+export function crashReportsSuppressed(env = process.env) {
+    return crashReportsDisabled(env) || runningUnderTest(env);
+}
 const HOME_DIR_RE = /(?:\/(?:Users|home)\/|[A-Za-z]:\\Users\\)[^\s/\\]+/g;
 // Spaces deliberately allowed inside the token (real directories contain
 // them); prose after a path may fold into the kept basename — over-redacts
@@ -122,7 +155,7 @@ export function makeCrashReporter(opts) {
     return async (error, context, repoRoot) => {
         try {
             const env = opts.env ?? process.env;
-            if (crashReportsDisabled(env))
+            if (crashReportsSuppressed(env))
                 return;
             const token = opts.getToken();
             const sanitized = sanitizeCrashError(error, { env, repoRoot });
@@ -191,7 +224,7 @@ const DETACHED_SENDER_SRC = [
 export function reportFatalCrash(error, opts, context) {
     try {
         const env = opts.env ?? process.env;
-        if (crashReportsDisabled(env))
+        if (crashReportsSuppressed(env))
             return;
         const token = opts.getToken();
         if (!token)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yagni-app/code-staging",
-  "version": "0.3.0-staging.1099.1",
+  "version": "0.3.1-staging.1103.1",
   "description": "YAGNI Code: a terminal coding agent that already knows your company. One YAGNI login routes the model and grounds the agent in your team's context.",
   "license": "SEE LICENSE IN LICENSE.md",
   "author": "YAGNI, Inc. <jack@yagni.app> (https://yagni.app)",
@@ -39,5 +39,5 @@
     "smol-toml": "^1.8.0",
     "typebox": "^1.3.11"
   },
-  "yagniSourceSha": "c94b255f76f60efa95f4c2c016b1c533bbeae49f"
+  "yagniSourceSha": "5df84aa9651fc7d6dd30b38bb062e8ee3c71156d"
 }