@yagni-app/code-staging 1.1.6-staging.1443.1 → 1.1.6-staging.1445.1

package/dist/extension/pipeline/goCommand.js

@@ -362,6 +362,9 @@ function formatSummaryForChat(result, costText) {
 const UNTRACKED_NOTICE = {
     disabled: "Run tracking is off for this workspace, so this /go run won't appear on the Work page.",
     unauthorized: "Run tracking is unauthorized. Run `yagni login` to refresh your token so runs appear on the Work page.",
+    // A refusal always arrives with the server's own sentence; this is the line
+    // for the rare body that carries none.
+    refused: "YAGNI declined to record this run, so it won't appear on the Work page. The pipeline ran anyway.",
     error: "Couldn't reach YAGNI to record this run, so it won't appear on the Work page. The pipeline ran anyway.",
 };
 /** Toast severity for an outcome: clean is good, round_cap warns, abort/fail are errors. */
@@ -737,9 +740,11 @@ export function registerGoCommand(pi, deps = {}) {
                     }
                     return { runId: started.runId, workItemId: started.workItemId };
                 }
-                // Recording skipped (flag off) or failed (unauthorized / unreachable):
-                // tell the user the run won't show on the Work page rather than swallowing it.
-                notify(UNTRACKED_NOTICE[started.reason], "warning");
+                // Recording skipped (flag off) or failed (unauthorized / refused /
+                // unreachable): tell the user the run won't show on the Work page
+                // rather than swallowing it. A refusal carries the server's own
+                // sentence, which says what to do about it, so it wins over our copy.
+                notify(started.message ?? UNTRACKED_NOTICE[started.reason], "warning");
                 return {};
             };
             let resumeFrom;

package/dist/extension/pipeline/runSession.d.ts

@@ -67,21 +67,27 @@ export interface RunStartResult extends RunSessionHandle {
  *                     workspace (the write endpoints 404 to hide themselves).
  *  - `unauthorized` — 401/403: the device token is missing/expired or lacks the
  *                     `yagni_code:write` scope (re-run `yagni login`).
+ *  - `refused`      — 400: the server declined for a reason a person can act on
+ *                     and said so in one sentence (this repository belongs to
+ *                     no Team). The sentence rides `message` and is what the
+ *                     caller shows.
  *  - `error`        — anything else: an over-budget 402, a 5xx, a 2xx with an
  *                     unrecognized body, or an unreachable backend / thrown
  *                     network error.
  */
-export type RunTrackingFailure = "disabled" | "unauthorized" | "error";
+export type RunTrackingFailure = "disabled" | "unauthorized" | "refused" | "error";
 /**
  * The outcome of opening a run. `tracked: true` carries the handle + gate
  * verdict; `tracked: false` carries the reason so `/go` can warn that this run
- * won't appear on the Work page. `start` never returns null and never throws.
+ * won't appear on the Work page, plus the server's own sentence when it sent
+ * one. `start` never returns null and never throws.
  */
 export type RunStartOutcome = ({
     tracked: true;
 } & RunStartResult) | {
     tracked: false;
     reason: RunTrackingFailure;
+    message?: string;
 };
 /** Arguments to open a run. */
 export interface StartArgs {

package/dist/extension/pipeline/runSession.js

@@ -25,27 +25,45 @@ function failureReason(status) {
         return "disabled";
     if (status === 401 || status === 403)
         return "unauthorized";
+    if (status === 400)
+        return "refused";
     return "error";
 }
 /** Cap a debug body excerpt so a giant HTML/JSON error never floods the channel. */
 const MAX_DEBUG_BODY = 200;
+/** Cap the sentence a refusal is allowed to print, so a stray body stays one line. */
+const MAX_REFUSAL_MESSAGE = 240;
 /**
- * A token-free, capped `: <body>` suffix for a failed-response debug line. Reads
- * the body only on the `!res.ok` branch (where `post` never reads it otherwise,
- * so there is no double-consume with the ok-path `res.json()`), and swallows any
- * read error so observability never breaks the fail-soft contract.
+ * The server's one-sentence refusal, if the body is the shape our own routes
+ * send (`{ error }`). Anything else, including HTML from a proxy, reads as no
+ * sentence at all, so the caller falls back to its own copy. Newlines are
+ * folded so a notice is never a stack.
  */
-async function bodyTail(res) {
+function refusalMessage(body) {
     try {
-        const t = (await res.text()).trim();
-        if (!t)
-            return "";
-        return `: ${t.length > MAX_DEBUG_BODY ? `${t.slice(0, MAX_DEBUG_BODY)} ...` : t}`;
+        const parsed = JSON.parse(body);
+        const text = typeof parsed.error === "string" ? parsed.error.trim() : "";
+        if (!text)
+            return undefined;
+        const oneLine = text.replace(/\s+/g, " ");
+        return oneLine.length > MAX_REFUSAL_MESSAGE
+            ? `${oneLine.slice(0, MAX_REFUSAL_MESSAGE)}...`
+            : oneLine;
     }
     catch {
-        return "";
+        return undefined;
     }
 }
+/**
+ * A token-free, capped `: <body>` suffix for a failed-response debug line, from
+ * a body already read once by the `!res.ok` branch.
+ */
+function bodyTail(body) {
+    const t = body.trim();
+    if (!t)
+        return "";
+    return `: ${t.length > MAX_DEBUG_BODY ? `${t.slice(0, MAX_DEBUG_BODY)} ...` : t}`;
+}
 export function makeRunSession(opts) {
     const f = opts.fetchImpl ?? fetch;
     let handle = null;
@@ -94,9 +112,21 @@ export function makeRunSession(opts) {
                 ...(opts.random ? { random: opts.random } : {}),
             });
             if (!res.ok) {
+                // Read the body ONCE on the failure branch (the ok path reads
+                // `res.json()` instead, so there is no double-consume): the debug line
+                // and the refusal sentence both come from this one read, and a body
+                // that cannot be read costs neither.
+                let body = "";
+                try {
+                    body = await res.text();
+                }
+                catch {
+                    /* a body we cannot read is a body we do not quote */
+                }
                 if (debugOn)
-                    debug(`[yagni-code] run-record ${path} -> HTTP ${res.status}${await bodyTail(res)}`);
-                return { ok: false, status: res.status };
+                    debug(`[yagni-code] run-record ${path} -> HTTP ${res.status}${bodyTail(body)}`);
+                const message = refusalMessage(body);
+                return { ok: false, status: res.status, ...(message ? { failureText: message } : {}) };
             }
             return { ok: true, data: (await res.json()) };
         }
@@ -119,8 +149,13 @@ export function makeRunSession(opts) {
             // deduped by the server (startCodeRun.findRunByIdempotencyKey) instead of
             // opening a second run + a second free-text work item.
             const r = await post("runs", { ...b, idempotencyKey: randomUUID() }, { retry: true });
-            if (!r.ok)
-                return { tracked: false, reason: failureReason(r.status) };
+            if (!r.ok) {
+                return {
+                    tracked: false,
+                    reason: failureReason(r.status),
+                    ...(r.failureText ? { message: r.failureText } : {}),
+                };
+            }
             const data = r.data;
             if (typeof data.runId === "string" && typeof data.workItemId === "string") {
                 handle = { runId: data.runId, workItemId: data.workItemId };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yagni-app/code-staging",
-  "version": "1.1.6-staging.1443.1",
+  "version": "1.1.6-staging.1445.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)",
@@ -58,5 +58,5 @@
     "turndown": "^7.2.4",
     "typebox": "^1.3.15"
   },
-  "yagniSourceSha": "8e549e888319699160837efefdfa8f1d835a84b5"
+  "yagniSourceSha": "df1d49933fa6a71d3b18754a183b5e400a45053a"
 }