fullstackgtm 0.50.0 → 0.50.1

package/src/cli/schedule.ts

@@ -9,6 +9,7 @@ import { computeMissedFirings, createFileScheduleRunStore, createFileScheduleSto
 import { runCli } from "../cli.ts";
 import { isOptionValue, numericOption, option } from "./shared.ts";
 import { unknownSubcommandError } from "./suggest.ts";
+import { colorEnabled, paint, table } from "./ui.ts";
 
 
 /**
@@ -120,12 +121,19 @@ trigger: manual. status shows next firing and surfaces missed firings
       console.log('No schedules. Add one with `fullstackgtm schedule add "<command>" --cron "<expr>"`.');
       return;
     }
-    for (const entry of withNext) {
-      console.log(
-        `${entry.id}  ${entry.enabled ? "on " : "off"}  ${entry.cron.padEnd(14)} next ${entry.nextFiring ?? "—"}  ${entry.label}: ${entry.argv.join(" ")}`,
-      );
-    }
-    console.log("\nEntries are declarative — `schedule install` materializes enabled ones into the crontab.");
+    const p = paint(colorEnabled(process.stdout));
+    const verbose = rest.includes("--verbose");
+    const rows = [
+      ["STATE", "SCHEDULE", "NEXT", ...(verbose ? ["CRON", "COMMAND"] : [])],
+      ...withNext.map((entry) => [
+        entry.enabled ? "on" : "off",
+        `${entry.label} · ${entry.id}`,
+        entry.nextFiring ?? "—",
+        ...(verbose ? [entry.cron, entry.argv.join(" ")] : []),
+      ]),
+    ];
+    console.log(table(rows, [p.dim, null, null]).join("\n"));
+    console.log("\nDeclarative only · run `fullstackgtm schedule install` after changes.");
     return;
   }
 
@@ -272,21 +280,19 @@ trigger: manual. status shows next firing and surfaces missed firings
       const last = entry.lastRun as { firedAt: string; trigger: string; exitCode: number; noopReason?: string; artifacts: { planIds: string[]; runLabels: string[] } } | null;
       const streak = entry.streak as { outcome: string; length: number } | null;
       const missed = entry.missedFirings as string[];
-      console.log(`${entry.id}  ${entry.enabled ? "on " : "off"}  ${entry.cron}  ${entry.label}: ${(entry.argv as string[]).join(" ")}`);
-      console.log(`  next: ${entry.nextFiring ?? "— (disabled)"}`);
+      const verbose = rest.includes("--verbose");
+      const outcome = !last ? "never run" : last.exitCode === 0 ? (last.noopReason ? `no-op · ${last.noopReason}` : "healthy") : `failed · exit ${last.exitCode}`;
+      console.log(`${entry.enabled ? "●" : "○"} ${entry.label} · ${outcome}`);
+      console.log(`  ${entry.id} · next ${entry.nextFiring ?? "— (disabled)"}`);
+      if (verbose) console.log(`  ${(entry.argv as string[]).join(" ")} · cron ${entry.cron}`);
       if (last) {
         const artifacts = [
           ...last.artifacts.planIds.map((planId) => `plan ${planId}`),
           ...last.artifacts.runLabels.map((runLabel) => `run ${runLabel}`),
         ];
-        console.log(
-          `  last: ${last.firedAt} (${last.trigger}) exit ${last.exitCode}` +
-            (last.noopReason ? ` — no-op: ${last.noopReason}` : "") +
-            (artifacts.length ? ` — ${artifacts.join(", ")}` : ""),
-        );
-        console.log(`  streak: ${streak!.length} ${streak!.outcome}(s)`);
+        console.log(`  last ${last.firedAt} · ${last.trigger} · ${streak!.length} ${streak!.outcome}${streak!.length === 1 ? "" : "s"}${artifacts.length ? ` · ${artifacts.join(", ")}` : ""}`);
       } else {
-        console.log("  last: never fired");
+        console.log("  last never fired");
       }
       if (missed.length > 0) {
         console.log(

package/src/cli/signals.ts

@@ -27,6 +27,21 @@ function resolveSignalsConfig(args: string[]): SignalsConfig {
 /** A watchlist entry: an account domain plus optional per-source board tokens. */
 type WatchlistAccount = { domain: string; boards?: Partial<Record<AtsBoardSource, string>> };
 
+function concise(value: string, max = 72): string {
+  const oneLine = value.replace(/\s+/g, " ").trim();
+  return oneLine.length <= max ? oneLine : `${oneLine.slice(0, max - 1)}…`;
+}
+
+function renderSignals(signals: Signal[]): string {
+  if (signals.length === 0) return "No signals matched.";
+  const lines = [`Fresh signals (${signals.length})`, ""];
+  for (const signal of signals) {
+    lines.push(`${signal.weight.toFixed(2).padStart(5)}  ${signal.accountDomain}  · ${signal.bucket}`);
+    lines.push(`       ${concise(signal.trigger)}`);
+  }
+  return lines.join("\n");
+}
+
 /**
  * Resolve the watchlist of accounts to scan. Sources, in precedence order:
  *  - a --watchlist file (JSON array of {domain, boards?} or bare domain strings),
@@ -228,7 +243,7 @@ from the credential ladder, never argv; --connector-opt carries non-secret knobs
     const ranked = [...fresh].sort((a, b) => b.weight - a.weight || a.accountDomain.localeCompare(b.accountDomain));
 
     // Ranked fresh signals to stdout; guidance to stderr.
-    console.log(JSON.stringify(ranked, null, 2));
+    console.log(rest.includes("--json") || rest.includes("--verbose") ? JSON.stringify(ranked, null, 2) : renderSignals(ranked));
     console.error(
       `Fetched ${fetched} candidate signal(s); ${fresh.length} fresh, ${deduped.length} deduped ` +
         `(window ${config.dedupWindowDays}d). NO plan emitted — signals are read-only re: CRM.`,
@@ -275,7 +290,7 @@ from the credential ladder, never argv; --connector-opt carries non-secret knobs
       return true;
     });
     const ranked = filtered.sort((a, b) => b.weight - a.weight || a.accountDomain.localeCompare(b.accountDomain));
-    console.log(JSON.stringify(ranked, null, 2));
+    console.log(rest.includes("--json") || rest.includes("--verbose") ? JSON.stringify(ranked, null, 2) : renderSignals(ranked));
     console.error(`${ranked.length} signal(s)${unjudgedOnly ? " (unjudged)" : ""}.`);
     return;
   }
@@ -305,7 +320,11 @@ from the credential ladder, never argv; --connector-opt carries non-secret knobs
     const outcomes = await store.listOutcomes();
     const signalsById = new Map((await store.allSignals()).map((s) => [s.id, s]));
     const weights = computeWeights(config, outcomes, signalsById);
-    console.log(JSON.stringify(weights, null, 2));
+    if (rest.includes("--json") || rest.includes("--verbose")) {
+      console.log(JSON.stringify(weights, null, 2));
+    } else {
+      console.log(["Learned signal weights", "", ...SIGNAL_BUCKETS.map((bucket) => `${bucket.padEnd(10)} ${weights[bucket].toFixed(4)}`)].join("\n"));
+    }
     if (rest.includes("--explain")) {
       // Per-bucket config default vs learned + booked/total over credited signals.
       const booked = new Map<SignalBucket, number>();

package/src/cli/tam.ts

@@ -275,8 +275,15 @@ RevOps universe, with real names. --source explorium is a firmographic count onl
     if (out) {
       writeFileSync(resolve(process.cwd(), out), md);
       console.log(`Wrote ${out}.`);
-    } else {
+    } else if (rest.includes("--verbose")) {
       console.log(md);
+    } else if (timeline.length > 0) {
+      console.log(coverageToText(model, timeline.at(-1)!, eta));
+      console.log("\nUse --verbose for assumptions, cross-checks, and the full coverage history.");
+    } else {
+      console.log(`TAM "${model.name}" · ${model.universe.accounts.toLocaleString()} accounts · ${model.universe.contacts.toLocaleString()} buyers · $${Math.round(model.tamUsd).toLocaleString()}`);
+      console.log(`ICP ${model.icpName} · ${model.acv.basis}-basis ACV $${Math.round(model.acv.valueUsd).toLocaleString()} (${model.acv.source})`);
+      console.log("No coverage readings yet. Run `fullstackgtm tam status --save` to establish one; use --verbose for the full assumptions report.");
     }
     return;
   }

package/src/cli/ui.ts

@@ -324,8 +324,12 @@ export function createChecklist(
       const label = entry.state === "pending" ? p.dim(item.label.padEnd(labelWidth)) : item.label.padEnd(labelWidth);
       return `  ${glyph} ${label}${note}`;
     });
-    const up = painted > 0 ? `\u001b[${painted}A` : "";
-    stream.write(`${up}${lines.map((line) => `\u001b[2K${line}`).join("\n")}\n`);
+    // Keep the cursor on the board's last row while it is live. A trailing
+    // newline on every repaint can scroll the old top row into permanent
+    // history when the board sits at the bottom of the terminal, producing
+    // apparent duplicate/errored frames in terminal captures.
+    const up = painted > 1 ? `\u001b[${painted - 1}A` : "";
+    stream.write(`${up}${lines.map((line) => `\u001b[2K${line}`).join("\n")}`);
     painted = lines.length;
   };
 
@@ -353,14 +357,19 @@ export function createChecklist(
       timer = null;
       if (options.persist) {
         // The caller's final state update already painted the completed board.
-        // Freeze that frame in scrollback; repainting here can leave both the
-        // last live frame and the final frame visible in some terminals.
+        // Advance exactly once so subsequent output begins below it.
+        if (painted > 0) stream.write("\n");
         painted = 0;
         return;
       }
       if (painted > 0) {
-        // Erase the board: cursor up over every painted line, clear each.
-        stream.write(`\u001b[${painted}A${"\u001b[2K\n".repeat(painted)}\u001b[${painted}A`);
+        // Erase in place without linefeeds, which could themselves scroll.
+        const up = painted > 1 ? `\u001b[${painted - 1}A` : "";
+        const clear = Array.from({ length: painted }, (_, index) =>
+          `\u001b[2K${index < painted - 1 ? "\u001b[1B" : ""}`,
+        ).join("");
+        const restore = painted > 1 ? `\u001b[${painted - 1}A` : "";
+        stream.write(`${up}${clear}${restore}`);
         painted = 0;
       }
     },

package/src/connectors/hubspot.ts

@@ -42,6 +42,10 @@ export type HubspotConnectorOptions = {
    * alongside the legacy `onProgress` callback; both are presentation-only.
    */
   progress?: ProgressEmitter;
+  /** Maximum retries for HubSpot 429/5xx responses (default 5). */
+  maxRetries?: number;
+  /** Injectable delay for deterministic retry tests. */
+  sleep?: (milliseconds: number) => Promise<void>;
 };
 
 const OBJECT_PATHS: Partial<Record<GtmObjectType, string>> = {
@@ -76,6 +80,8 @@ const PULL_STAGE_BY_TYPE: Record<SnapshotProgress["objectType"], (typeof SNAPSHO
 export function createHubspotConnector(options: HubspotConnectorOptions): HubspotWritableConnector {
   const baseUrl = (options.apiBaseUrl ?? DEFAULT_API_BASE_URL).replace(/\/$/, "");
   const fetchImpl = options.fetchImpl ?? fetch;
+  const maxRetries = options.maxRetries ?? 5;
+  const sleep = options.sleep ?? ((milliseconds: number) => new Promise<void>((resolve) => setTimeout(resolve, milliseconds)));
   const mappings = options.fieldMappings;
   // create:<Name> dedup within one connector lifetime (one apply run): the
   // search API is eventually consistent, so a just-created company is
@@ -113,27 +119,45 @@ export function createHubspotConnector(options: HubspotConnectorOptions): Hubspo
   };
 
   async function request(path: string, init: RequestInit = {}): Promise<any> {
-    const token = await options.getAccessToken();
-    let response: Response;
-    try {
-      response = await fetchImpl(`${baseUrl}${path}`, {
-        ...init,
-        headers: {
-          Authorization: `Bearer ${token}`,
-          "Content-Type": "application/json",
-          ...(init.headers ?? {}),
-        },
-      });
-    } catch (error) {
-      const cause = error instanceof Error && error.cause instanceof Error ? `: ${error.cause.message}` : "";
-      throw new Error(`Cannot reach HubSpot at ${baseUrl}${cause}. Check network access.`);
+    let response: Response | undefined;
+    for (let attempt = 0; attempt <= maxRetries; attempt += 1) {
+      const token = await options.getAccessToken();
+      try {
+        response = await fetchImpl(`${baseUrl}${path}`, {
+          ...init,
+          headers: {
+            Authorization: `Bearer ${token}`,
+            "Content-Type": "application/json",
+            ...(init.headers ?? {}),
+          },
+        });
+      } catch (error) {
+        const cause = error instanceof Error && error.cause instanceof Error ? `: ${error.cause.message}` : "";
+        throw new Error(`Cannot reach HubSpot at ${baseUrl}${cause}. Check network access.`);
+      }
+      const retryable = response.status === 429 || response.status >= 500;
+      if (!retryable || attempt === maxRetries) break;
+      await response.text().catch(() => undefined);
+      const retryAfter = response.headers.get("Retry-After");
+      const seconds = retryAfter === null ? NaN : Number(retryAfter);
+      const dateDelay = retryAfter && !Number.isFinite(seconds) ? Date.parse(retryAfter) - Date.now() : NaN;
+      const delayMs = Math.min(
+        30_000,
+        Math.max(0, Number.isFinite(seconds) ? seconds * 1_000 : Number.isFinite(dateDelay) ? dateDelay : 1_000 * 2 ** attempt),
+      );
+      const reason = response.status === 429 ? "rate limited" : `temporarily unavailable (${response.status})`;
+      options.progress?.note(`HubSpot ${reason}; retrying in ${Math.ceil(delayMs / 1_000)}s (${attempt + 1}/${maxRetries})`);
+      await sleep(delayMs);
     }
-    if (!response.ok) {
+    if (!response || !response.ok) {
       // Status line only — HubSpot 4xx bodies echo submitted property values
       // (contact emails, company domains) and the request payload, and these
       // errors are persisted into scheduled-run records. Never interpolate it.
-      await response.text().catch(() => undefined);
-      throw new Error(`HubSpot API error ${response.status}. Check the token scopes and request.`);
+      await response?.text().catch(() => undefined);
+      if (response?.status === 429) {
+        throw new Error(`HubSpot rate limit (429) persisted after ${maxRetries} retries. Wait for the portal limit to reset, then retry; no CRM writes were made.`);
+      }
+      throw new Error(`HubSpot API error ${response?.status ?? "unknown"}. Check the token scopes and request.`);
     }
     // DELETE and some association writes return 204 with an empty body.
     const text = await response.text();

package/src/hostedPatchPlan.ts

@@ -207,7 +207,12 @@ export async function reportHostedPlanLifecycle(stored: StoredPlan, options: Opt
   if (!paired) return { status: "unpaired" };
   const status = stored.status === "applied" ? "applied" : stored.status === "rejected" ? "rejected" : "approved";
   const runRecord = status === "applied" ? stored.runs.at(-1) : undefined;
-  const operationReceipts = runRecord?.results.map((result) => ({
+  // The hosted terminal transition is authority-bound to the exact approved
+  // subset. Local runs also record excluded operations as `skipped` for a
+  // complete human audit trail; those are not execution receipts and must not
+  // be echoed as though they were authorized provider work.
+  const approved = new Set(stored.approvedOperationIds);
+  const operationReceipts = runRecord?.results.filter((result) => approved.has(result.operationId)).map((result) => ({
     packageOpId: result.operationId,
     status: result.status,
     ...(result.detail ? { error: result.detail.slice(0, 1000) } : {}),