@datarails-ai/dr-appspace-cli 0.12.4 → 0.12.6

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.12.4",
+  version: "0.12.6",
   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-10-12T12:52:52.625Z";
+var BUILD_TIME = "2026-10-12T20:27:28.392Z";
 function formatBuildTime(iso) {
   if (iso === "") return "";
   const d = new Date(iso);
@@ -17277,7 +17277,7 @@ var contract_schema_default = {
         module: { type: "string", description: "dr-widgets, dr-insights, dr-fmt, echarts, ui-kit" },
         version: { type: "string", description: "1.1.1 for a vendor module, v3 for the ui-kit" },
         file: { type: "string" },
-        path: { type: "string", description: "the platform-root path, e.g. vendor/dr-widgets@1.1.1/widgets.js" },
+        path: { type: "string", description: "the platform-root path, e.g. vendor/dr-widgets@1.2.1/widgets.js" },
         rel: { enum: ["script", "stylesheet"] },
         pages: { type: "array", items: { type: "string" } },
         order: { type: "integer", minimum: 0, description: "load order: where this module is first reached across the bundle" }
@@ -18034,6 +18034,7 @@ var LITERAL_OK = /* @__PURE__ */ new Set([
 ]);
 function validateContract(contract, catalog) {
   const problems = [];
+  const warnings = [];
   const doc = isObject4(contract) ? contract : {};
   const app = obj(doc["app"]);
   const kind = kindOf(doc);
@@ -18051,6 +18052,7 @@ function validateContract(contract, catalog) {
     checkPeriodOverride(doc, problems);
     checkKinds(doc, problems);
     checkDrillTargets(doc, problems);
+    checkAggOrAccountType(doc, warnings);
     checkPairing(doc, problems);
     checkTransforms(doc, problems);
   } else {
@@ -18063,7 +18065,7 @@ function validateContract(contract, catalog) {
     checkCustomDeclares(doc, problems);
     skipChartColumns(doc, skipped);
   }
-  return { problems: dedupe(problems), skipped: dedupeSkipped(skipped) };
+  return { problems: dedupe(problems), skipped: dedupeSkipped(skipped), warnings: dedupe(warnings) };
 }
 function checkCustomDeclares(doc, problems) {
   for (const problem5 of declaresProblems2(doc)) {
@@ -18883,7 +18885,7 @@ function checkDrillTargets(doc, problems) {
     const row = raw;
     if (row["data"] !== void 0 && row["data"] !== null) continue;
     const drill = list2(row["drill"]).filter((name) => typeof name === "string");
-    if (drill.length === 0) continue;
+    if (drill.length === 0 || !menuless(row)) continue;
     const offered = new Set(drill.map((name) => name.toLowerCase()));
     const listed = JSON.stringify(drill);
     if (row["kind"] === "breakdown") {
@@ -18893,7 +18895,7 @@ function checkDrillTargets(doc, problems) {
       problems.push({
         code: "drill-needs-dimension",
         pointer: pointer("data", "widgets", id, "drill"),
-        message: `this breakdown is cut by ${JSON.stringify(dimension)} and its drill list ${listed} does not name it, so no legend row answers a click \u2014 the card looks drillable and is not.`,
+        message: `this breakdown gets no point menu, so a click on it is the drill itself; it is cut by ${JSON.stringify(dimension)} and its drill list ${listed} does not name it, so no legend row answers a click \u2014 the card looks drillable and is not.`,
         fix: `Put ${JSON.stringify(dimension)} first in drill \u2014 ${JSON.stringify([dimension, ...drill])}: the clicked slice is a value of the card's own dimension, and the levels after it are where the grid goes next.`
       });
       continue;
@@ -18905,11 +18907,43 @@ function checkDrillTargets(doc, problems) {
     problems.push({
       code: "drill-needs-dimension",
       pointer: pointer("data", "widgets", id, "drill"),
-      message: `this table's drill list ${listed} names none of its columns (${keys2.map((key) => JSON.stringify(key)).join(", ")}), and a cell drills only when its column key is in drill \u2014 so no cell answers a click.`,
+      message: `this table gets no point menu, so a click on it is the drill itself; its drill list ${listed} names none of its columns (${keys2.map((key) => JSON.stringify(key)).join(", ")}), and a cell drills only when its column key is in drill \u2014 so no cell answers a click.`,
       fix: `Add the column a reader clicks to drill \u2014 e.g. ${JSON.stringify([keys2[0], ...drill])} \u2014 naming the grid's own dimension column first.`
     });
   }
 }
+function menuless(row) {
+  if (row["compute"] || row["transform"] !== void 0 && row["transform"] !== null) return true;
+  const source = row["source"];
+  if (!isObject4(source) || "app_table" in source || "call" in source) return true;
+  if ("metric_id" in source) return Boolean(source["drill_rows"]) && typeof row["value_field"] !== "string";
+  if (!addressesTable(source)) return true;
+  const measures = list2(source["measures"]).length > 0 ? list2(source["measures"]) : source["measure"] ? [source["measure"]] : [];
+  const fields = measures.map((one) => obj(one)["field"]);
+  const named12 = typeof row["value_field"] === "string" && fields.includes(row["value_field"]);
+  return !(named12 || fields.length === 1);
+}
+function addressesTable(source) {
+  return source["address"] === "id" || source["address"] === "alias" || Boolean(source["table_alias"]) || source["table_id"] !== void 0 && source["table_id"] !== null;
+}
+function checkAggOrAccountType(doc, warnings) {
+  const widgets = obj(obj(doc["data"])["widgets"]);
+  const declared = (value) => typeof value === "string" && value.trim() !== "";
+  for (const [id, raw] of Object.entries(widgets)) {
+    if (id.startsWith("_") || !isObject4(raw)) continue;
+    const row = raw;
+    if (row["kind"] !== "kpi" && row["kind"] !== "series") continue;
+    if (!isObject4(row["source"])) continue;
+    if (["data", "compute", "transform", "shape"].some((key) => row[key] !== void 0 && row[key] !== null)) continue;
+    if (declared(row["agg"]) || declared(row["account_type"])) continue;
+    warnings.push({
+      code: "agg-or-account-type",
+      pointer: pointer("data", "widgets", id),
+      message: `this ${row["kind"]} declares neither "agg" nor "account_type", so its figure sums its months \u2014 a quarter or a year of a balance (cash, assets, liabilities, equity, receivables, payables, inventory) would show three or twelve balances added up.`,
+      fix: 'If it is a balance, add "account_type": "asset" (or "liability", "equity"), or "agg": "last", so a quarter or year shows the closing month; if it is a flow (revenue, expenses), add "account_type": "revenue" (or "expense") to say so and silence this.'
+    });
+  }
+}
 function checkPairing(doc, problems) {
   const data = doc["data"];
   if (!isObject4(data) || !isObject4(data["widgets"])) return;
@@ -27102,7 +27136,7 @@ var kit_default = {
   fmt: "dr-fmt@1.0.0",
   markdown: "dr-md@1.0.0",
   uiKit: "ui-kit/v3",
-  widgets: "dr-widgets@1.1.2"
+  widgets: "dr-widgets@1.2.1"
 };
 
 // ../flows/appspace-builder/kit/kit.json
@@ -27263,10 +27297,11 @@ async function runCompile(dir, opts) {
   } finally {
     client?.cleanup();
   }
-  const { problems, skipped } = validateContract(contract, catalog);
+  const { problems, skipped, warnings } = validateContract(contract, catalog);
   report.validate = {
     problems,
     checks_skipped: skipped,
+    warnings,
     catalog: {
       source: catalog.source,
       server: catalog.server ?? null,
@@ -27365,6 +27400,7 @@ async function afterWrite(dir, target, kind, contract, built, report, kit, skipp
   }
   report.verify = runVerify(target, kind, report.app_id, kit);
   report.verify.warnings = [
+    ...(report.validate.warnings ?? []).map((one) => `${one.code} ${one.pointer}: ${one.message} ${one.fix}`),
     ...(built.warnings ?? []).map((one) => `${one.code} ${one.pointer ?? "/"}: ${one.message}`),
     ...report.verify.warnings ?? []
   ];

package/package.json

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