@datarails-ai/dr-appspace-cli 0.10.0 → 0.10.1

package/dist/cli.js

@@ -13,7 +13,7 @@ import { fileURLToPath as fileURLToPath6 } from "url";
 // package.json
 var package_default = {
   name: "@datarails-ai/dr-appspace-cli",
-  version: "0.10.0",
+  version: "0.10.1",
   description: "Datarails App Space CLI \u2014 build, try, push and publish App Space apps",
   type: "module",
   engines: {
@@ -57,7 +57,7 @@ var package_default = {
 };
 
 // src/buildInfo.ts
-var BUILD_TIME = "2026-09-30T12:30:07.259Z";
+var BUILD_TIME = "2026-09-30T13:19:16.568Z";
 function formatBuildTime(iso) {
   if (iso === "") return "";
   const d = new Date(iso);
@@ -24703,6 +24703,11 @@ function judgeInteraction(interaction, calls) {
       `${call.method} ${call.url} \u2014 ${call.failure} (the flow made this call; the UI runtime catches it and paints a banner, so nothing reaches the console)`
     );
   }
+  const askedForReason = mine.some((call) => call.needsReason);
+  const reasonCovered = askedForReason && mine.some((call) => call.reasonSupplied === true && call.needsReason !== true && call.failure === void 0 && call.error === void 0 && call.status !== void 0 && call.status >= 200 && call.status < 300);
+  if (askedForReason && !reasonCovered) {
+    problems.push(`${interaction.handler} needs a reason \u2014 no successful retry with a reason was observed.`);
+  }
   for (const field of interaction.fields) {
     if (field.unknownColumn === true) {
       problems.push(
@@ -24719,6 +24724,7 @@ function judgeInteraction(interaction, calls) {
     calls,
     problems
   };
+  if (reasonCovered) outcome.reasonCovered = true;
   if (interaction.id !== void 0) outcome.id = interaction.id;
   if (interaction.entity !== void 0) outcome.entity = interaction.entity;
   const attached = interaction.fields.flatMap(
@@ -25162,6 +25168,7 @@ function movedBetween(before, after, param, echoes = []) {
 }
 
 // src/commands/appspace/_uiSmoke.ts
+var SMOKE_REASON = "App Space smoke test: exercising the required reason flow.";
 var NULL_TEXT_CONTEXT = 80;
 var NULL_TEXT_LIMIT = 10;
 var NULL_LITERAL = /(?<![A-Za-z0-9_])(?:(?:null|undefined|NaN)+|\[object Object\])(?![A-Za-z0-9_])/g;
@@ -25336,7 +25343,13 @@ function answerOf(res) {
   }
   if (typeof parsed !== "object" || parsed === null || Array.isArray(parsed)) return {};
   const out = { object: true };
-  const { rows: rows2, error } = parsed;
+  const { rows: rows2, error, needs } = parsed;
+  if (needs && typeof needs === "object" && !Array.isArray(needs)) {
+    const reason = needs.reason;
+    if (reason === true || reason && typeof reason === "object" && !Array.isArray(reason)) {
+      out.needsReason = true;
+    }
+  }
   if (Array.isArray(rows2)) {
     out.rows = rows2.length;
     const { keys: keys2, numeric } = rowKeys(rows2);
@@ -25866,6 +25879,13 @@ async function runUiSmoke(opts) {
   let lastActivity = Date.now();
   async function record2(req, via, refuse) {
     const call = { method: req.method, url: maskToken(req.url), via };
+    if (via === "fetch" && req.body) {
+      try {
+        const payload = JSON.parse(req.body);
+        if (typeof payload?.reason === "string" && payload.reason.trim()) call.reasonSupplied = true;
+      } catch {
+      }
+    }
     const auth = req.headers["authorization"];
     if (auth !== void 0) call.authorization = summariseAuthorization(auth);
     calls.push(call);
@@ -25893,12 +25913,13 @@ async function runUiSmoke(opts) {
       call.status = res.status;
       if (res.status >= 400) call.failure = `HTTP ${String(res.status)}`;
       else if (via === "fetch") {
-        const { rows: rows2, error, keys: keys2, numeric, object } = answerOf(res);
+        const { rows: rows2, error, keys: keys2, numeric, object, needsReason } = answerOf(res);
         if (rows2 !== void 0) call.rows = rows2;
         if (error !== void 0) call.error = error;
         if (keys2 !== void 0) call.keys = keys2;
         if (numeric !== void 0) call.numeric = numeric;
         if (object !== void 0) call.object = object;
+        if (needsReason) call.needsReason = true;
       }
       return res;
     } catch (err) {
@@ -25981,9 +26002,10 @@ async function runUiSmoke(opts) {
   const displaced = process.listeners("unhandledRejection");
   process.removeAllListeners("unhandledRejection");
   process.on("unhandledRejection", onUnhandled);
-  async function settle(deadline, since = calls.length, graceMs = 0) {
+  async function settle(deadline, since = calls.length, graceMs = 0, drivePrompt) {
     const graceUntil = Date.now() + graceMs;
     while (Date.now() < deadline) {
+      if (drivePrompt?.()) lastActivity = Date.now();
       const parsed = dom.window.document.readyState === "complete";
       const started = calls.length > since || inFlight > 0;
       if (!started && Date.now() < graceUntil) {
@@ -26213,7 +26235,24 @@ async function runUiSmoke(opts) {
       } catch (err) {
         notes = [`firing it threw: ${describeRejection(err)}`];
       }
-      await settle(Math.min(Date.now() + perInteraction, budgetUntil), since, graceMs);
+      const answeredPrompts = /* @__PURE__ */ new Set();
+      await settle(Math.min(Date.now() + perInteraction, budgetUntil), since, graceMs, () => {
+        const pending = calls.slice(since).filter((call) => callsHandler(call.url, interaction.handler)).at(-1);
+        if (!pending?.needsReason || answeredPrompts.has(pending) || answeredPrompts.size >= 3) return false;
+        const dialog = dom.window.document.querySelector('[data-test="confirmDialog"]');
+        const reason = dialog?.querySelector('textarea[data-test="confirmReason"]');
+        const confirm = dialog?.querySelector('button[data-test="confirmOk"]');
+        if (!(reason instanceof dom.window.HTMLTextAreaElement) || !(confirm instanceof dom.window.HTMLButtonElement)) return false;
+        if (reason.value !== SMOKE_REASON) {
+          Object.getOwnPropertyDescriptor(dom.window.HTMLTextAreaElement.prototype, "value").set.call(reason, SMOKE_REASON);
+          reason.dispatchEvent(new dom.window.Event("input", { bubbles: true }));
+          return true;
+        }
+        if (confirm.disabled) return false;
+        answeredPrompts.add(pending);
+        confirm.click();
+        return true;
+      });
       const outcome = judgeInteraction(
         interaction,
         calls.slice(since).map((call) => ({ ...call }))
@@ -27870,6 +27909,26 @@ function cardTitleFindings(contract) {
     see: SEE.labels
   }));
 }
+function uxFindings(contract) {
+  if (contract === null || !isAppContract(contract)) return [];
+  const codes = {
+    "A3.23": "create-form-inline",
+    "A3.24": "row-no-detail",
+    "A3.25": "dialog-no-trigger"
+  };
+  const { violations } = checkUx({}, contract, null);
+  return violations.flatMap((found) => {
+    if (!(found.rule in codes)) return [];
+    return [{
+      code: codes[found.rule],
+      where: found.where,
+      message: found.message,
+      fix: "Apply the kit guidance above to the named element in contract.json and recompile",
+      see: `App kit ${found.rule} (checks/rules/ux.mjs)`,
+      severity: "warning"
+    }];
+  });
+}
 function collectFindings(input) {
   const levers = input.bundle === void 0 ? [] : leverFindings(input.contract, input.bundle, input.handlerCode ?? {});
   const unread = new Set(levers.filter((one) => one.code === "param-unread").map((one) => one.where));
@@ -27895,7 +27954,8 @@ function collectFindings(input) {
     ...input.contract === null ? [] : auditFindings(input.contract),
     ...input.declared === null ? [] : unreachedFindings(input.declared, input.cases, input.contract, input.runs),
     // Last: said and never counted, so they never push a counted finding down the list.
-    ...workflow.filter((one) => one.severity === "warning")
+    ...workflow.filter((one) => one.severity === "warning"),
+    ...uxFindings(input.contract)
   ];
 }
 function formatFindings(findings) {
@@ -28390,7 +28450,8 @@ function renderInteractions(report) {
     if (outcome.problems.length === 0) {
       const answered = outcome.calls.find((call) => callsHandler(call.url, outcome.handler));
       const status = answered?.status === void 0 ? "" : ` (HTTP ${String(answered.status)})`;
-      out += `  ok   ${head}${status}
+      const reason = outcome.reasonCovered ? " \u2014 needs a reason: covered (canned smoke-test reason)" : "";
+      out += `  ok   ${head}${status}${reason}
 ${attached}`;
       continue;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@datarails-ai/dr-appspace-cli",
-  "version": "0.10.0",
+  "version": "0.10.1",
   "description": "Datarails App Space CLI — build, try, push and publish App Space apps",
   "type": "module",
   "engines": {