@tokenoftrust/cli 2.1.0 → 2.1.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tokenoftrust/cli",
-  "version": "2.1.0",
+  "version": "2.1.2",
   "description": "Token of Trust developer CLI — clone a tenant store, run it locally with save→reload, and submit it for preview. Installs the `tot` command.",
   "license": "Apache-2.0",
   "author": "Token of Trust",

package/src/capability-registry.mjs

@@ -64,7 +64,7 @@ export const CAPABILITIES = [
       'Declared old->new paths materialize into the PUBLISHED artifact and are served by the edge '
       + 'function on the public path. They correctly 404 on the preview/test channel — that is by '
       + 'design and is not a missing declaration.',
-    implementedBy: ['scripts/site-migration/validate-site.mjs'],
+    implementedBy: ['agency-kit/tools/validate-site.mjs'],
     populatedBy: 'content/redirects.json in the tenant repo',
     verifiedBy: 'Step 9 post-cutover crawl (not provable pre-cutover)',
     appliesTo: ['commerce', 'marketing'],
@@ -105,7 +105,7 @@ export const CAPABILITIES = [
       'The merchant\'s own pre-migration scores, captured before cutover, so parity can be judged '
       + 'and improvement proven. NOT re-derivable once the hostname moves.',
     implementedBy: [
-      'scripts/site-migration/capture-source-baseline.mjs',
+      'agency-kit/tools/capture-source-baseline.mjs',
       'docs/devkit-staging/tools/lib/lighthouse-parity.mjs',
     ],
     populatedBy: 'Step 8 — capture-source-baseline.mjs, before cutover',
@@ -202,7 +202,7 @@ export const CAPABILITIES = [
       + 'category, so the score is visible and the dimension is offerable as an add-on — but a '
       + 'good score needs the surface built.',
     implementedBy: [
-      'scripts/site-migration/capture-source-baseline.mjs',
+      'agency-kit/tools/capture-source-baseline.mjs',
       'docs/devkit-staging/tools/validate-page-browser.mjs',
       'apps/storefront/src/lib/diag/evidence.ts',
     ],

package/src/commands/report.mjs

@@ -1,7 +1,7 @@
 /**
  * `tot report go-live` — the merchant-facing go-live readiness report for one store.
  *
- *   tot report go-live --pages <dir> [--style <json>] [--workspace <dir>] [--out <md>]
+ *   tot report go-live --pages <dir> [--style <json>] [--workspace <dir>] [--out <md>] [--json]
  *
  * It is generated FROM measurements, never authored:
  *
@@ -15,7 +15,8 @@
  * `.tot/lighthouse-source-baseline.json` (the source site's pre-migration scores).
  *
  * Writes <out> (default <pages>/go-live-report.md) and its implementor companion
- * <out>.internal.md. No network, no credentials: reproducible from the same inputs.
+ * <out>.internal.md. --json also prints the readiness checks it reported on to stdout.
+ * No network, no credentials: reproducible from the same inputs.
  * Exit 0 written; 2 usage; 3 refused (no measurements to report on).
  */
 import { existsSync, mkdirSync, readdirSync, readFileSync, writeFileSync } from "node:fs";
@@ -34,6 +35,7 @@ export const USAGE = `tot report go-live — the go-live readiness report for yo
       [--style <json>]                 the style-concept audit's JSON (default <pages>/style.json)
       [--workspace <dir>]              the store checkout (default: this checkout)
       [--out <md>]                     where to write it (default <pages>/go-live-report.md)
+      [--json]                         also print the readiness checks as JSON
 
 Save each page's JSON from the devkit page checks into <dir> first, e.g.
   validate-page.sh <base> / > <dir>/home.json
@@ -58,6 +60,7 @@ export function parseArgs(argv) {
   for (let i = 0; i < argv.length; i += 1) {
     const t = argv[i];
     if (t === "--help" || t === "-h") { a.help = true; continue; }
+    if (t === "--json") { a.json = true; continue; }
     if (!t.startsWith("--")) {
       if (a.sub === null) { a.sub = t; continue; }
       return { ...a, error: `unexpected argument: ${t}` };
@@ -164,10 +167,22 @@ export function evidenceFromDir({ pagesDir, stylePath = null, lint = null }) {
   };
 
   if (summary) {
+    // A site-wide run that recorded its checks is reported as recorded; one that did not has them
+    // derived here from its own aggregate, by the same derivation as single-page results.
+    const checks = Array.isArray(summary.readiness?.checks)
+      ? summary.readiness.checks
+      : [
+          storeConfigCheck(lint),
+          ...buildReadinessChecks({
+            aggregate: { ...summary, totals: { ...summary.totals, issues: summary.totals?.issues ?? { lowRiskOpen: 0 } } },
+            styleJson: styleJson ?? (summary.style?.gates ? { gates: summary.style.gates } : null),
+            args: { manifest: summary.manifest ?? "(none supplied to the style-concept audit)" },
+          }),
+        ];
     return {
       measured: true,
       generatedAt: summary.generatedAt ?? null,
-      checks: summary.readiness?.checks ?? [],
+      checks,
       totals: summary.totals ?? {},
       pages,
       ...styleFacts,
@@ -303,6 +318,9 @@ export function run(argv, ctx, io = {}) {
     pages: evidence.pages,
     commerceSync,
   }));
+  if (args.json) {
+    out(`${JSON.stringify({ tenant, report: outPath, internal: internalPath, totals: evidence.totals, checks: evidence.checks }, null, 2)}\n`);
+  }
   err(`wrote ${outPath}\nwrote ${internalPath}\n${TEASER}\n`);
   return 0;
 }

package/src/commands/start.mjs

@@ -267,10 +267,18 @@ export async function run(argv, ctx) {
       noLogin: false, noOpen: args.noOpen, docker: args.docker,
     });
     const runtime = /** @type {{ useDocker: boolean, runnerDir: any }} */ ({ useDocker: args.docker, runnerDir: null });
-    await Promise.all([
+    await settleCheckoutAndRuntime(
       ensureCheckout(client, tenant, dir, env),
       prefetchRuntime(client, devArgs, env, runtime, ctx),
-    ]);
+      {
+        onRuntimeFailedAfterCheckout: () => {
+          printCheckoutLocation(dir);
+          console.log(
+            `     Once the problem below is fixed, run \`tot start\` again from ${resolve(dir, "..")} — it reuses ./${tenant}.`,
+          );
+        },
+      },
+    );
 
     // Say EXACTLY where the code landed + THE one next step, so a freshly
     // checked-out dev is never left wondering what to open. Absolute path (works
@@ -690,10 +698,32 @@ export function formatElapsed(ms) {
  * Developer Cockpit URL, the one next step is framed as "edit a line → see it in
  * your cockpit"; otherwise it's the same edit-to-see-it-reload action.
  */
-/** @param {string} dir @param {string|null} [cockpitUrl] */
-function printCheckoutLanding(dir, cockpitUrl = null) {
+/** @param {string} dir */
+function printCheckoutLocation(dir) {
   console.log("");
   console.log(`  📁 Your store code is at:  ${dir}`);
+}
+
+/**
+ * Run the checkout and the runner prefetch together, but never let a failed prefetch hide
+ * a checkout that succeeded: the code is already on disk, so say where before re-throwing
+ * the prefetch error. A failed checkout wins (nothing landed, nothing to point at).
+ * @param {Promise<unknown>} checkout
+ * @param {Promise<unknown>} runtime
+ * @param {{ onRuntimeFailedAfterCheckout: () => void }} hooks
+ */
+export async function settleCheckoutAndRuntime(checkout, runtime, { onRuntimeFailedAfterCheckout }) {
+  const [checkedOut, prepared] = await Promise.allSettled([checkout, runtime]);
+  if (checkedOut.status === "rejected") throw checkedOut.reason;
+  if (prepared.status === "rejected") {
+    onRuntimeFailedAfterCheckout();
+    throw prepared.reason;
+  }
+}
+
+/** @param {string} dir @param {string|null} [cockpitUrl] */
+function printCheckoutLanding(dir, cockpitUrl = null) {
+  printCheckoutLocation(dir);
   const where = cockpitUrl ? `your ${DEVELOPER_COCKPIT}` : "your browser";
   console.log(`     Next: open it in your editor and change one line in`);
   console.log(`     content/home.html — you'll see it reflected in ${where}.`);