@wix/himalaya-cli 0.860.0 → 0.862.0

package/guides/mobile-ux.md

@@ -115,8 +115,10 @@ errors block readiness; record tool-coverage limits separately.
 
 ## Evidence
 
-- Render compact (320/360), phone, and expanded widths. Record whether a one-column expanded
-  layout is intentional or the screen adapts.
+- Render each screen at a phone width, at the narrowest width you support, and again at a width
+  past the 600px breakpoint. `himi render <screen> --width 320` then `--width 834` writes both
+  into `.himi/ux-audit.json`, which is what this marker grades. Record whether a one-column
+  layout at the wider size is intentional or the screen adapts.
 - Exercise typed input, submit, completion, cancel/back, and any destructive action; record
   confirmation, undo, or recovery.
 - Drive iOS back/cancel, keyboard submit, Dynamic Type, and VoiceOver; drive Android system Back,
@@ -143,8 +145,11 @@ itself the evidence — see the strict audit contract.
 
 **Two commands read this contract.** `himi validate` reports a gap as a **warning**, so a rewrite
 that drops something the gate needs is caught in the loop you are already running rather than at
-the end of it. `himi validate --strict` and `himi ux audit --strict` both promote that warning to a
-**release blocker**. Both read the same markers. The two HEADINGS are matched verbatim — other
+the end of it. `himi ux audit --strict` promotes every gap to a **release blocker**.
+`himi validate --strict` promotes the structural ones — the two headings and the one-CTA rule — but
+deliberately leaves **width evidence** a warning: a package that has not rendered yet would
+otherwise fail strict validation the moment it is scaffolded, so that gap is the evidence gate's to
+refuse. Both read the same markers. The two HEADINGS are matched verbatim — other
 tooling addresses the document by them, so a heading really is the thing itself:
 
 - The heading `## Implementation acceptance`.
@@ -229,8 +234,11 @@ tried in this order:
   icon-834.png` pass. The marker is permissive there on purpose. A gate that rejects real work
   costs build loops; a permissive one costs a line of prose, and a card or drawer width always
   passed anyway.
-- Or keep the scaffold's own sentence, `compact (320/360)` followed later by `expanded`, which is
-  still accepted as-is. The two **may wrap across lines** — the check is not line-based — but
+- Or, in a package scaffolded before #2590, keep the legacy scaffold sentence,
+  `compact (320/360)` followed later by `expanded`, which is still accepted as-is. The template
+  stopped shipping that sentence in #2590 — it satisfied this marker on a package that had
+  rendered nothing — so a newer scaffold does not carry it, and adding it by hand claims evidence
+  you may not have. The two **may wrap across lines** — the check is not line-based — but
   `expanded` must follow within about 400 characters, so keep them in the same sentence or bullet
   rather than in separate sections.
 

package/guides/owner-lite-site-binding.md

@@ -23,6 +23,11 @@ Source of truth: `apps/himi-owner-lite/config.ts`. A management app that rendere
 knowing *which owner* is looking at it would be showing someone else's business, which is why
 the gate fails closed rather than falling back to a visitor.
 
+The Sites List is the Tier-5 `himi-owner-lite-sites` worker. Its `pick` action calls
+`site.setActiveSite(msid)` and waits for the site-scoped signed instance before the selected
+business is used by subsequent management screens. The worker also clears the Stores catalog
+version around the switch so a catalog resolved for one business is not reused for another.
+
 ## What owner-lite *does* read
 
 Verified against `apps/himi-owner-lite/config.ts` — if a variable is not in this table, owner-lite

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wix/himalaya-cli",
-  "version": "0.860.0",
+  "version": "0.862.0",
   "type": "module",
   "description": "himi — the Himalaya authoring CLI. Build a Tier-5 content package on your own machine, push it to the production serve as a draft, preview it on web + Himi Player, and read worker logs back — the remote dev loop. Wraps @himalaya/serve-cli for transport.",
   "keywords": [
@@ -63,5 +63,5 @@
       "artifactId": "himalaya-cli"
     }
   },
-  "falconPackageHash": "6e5648ad5145fd23a5016b4c194db40fb00d71c22c7a0abaa2ed5020"
+  "falconPackageHash": "988c32c342c4858fe88297637a696b7452fae42ee373a75995e64c36"
 }

package/templates/beauty-salon-owner/tier5-src/himi-owner-lite-analytics/report.ts

@@ -181,11 +181,22 @@ function subjectLabelOf(m: SemanticModelSummary): string {
  * renders as a dash rather than as 0. `formatNumber` and not `toLocaleString`,
  * which is ECMA-402 and absent on QuickJS-ng (watchOS/Wear OS).
  */
+/**
+ * The one placeholder a cell falls back to when there is nothing to render.
+ *
+ * Named, because two things depend on it meaning exactly one thing: `cellText`
+ * produces it, and `runQuery` asks whether the server's total came back as it —
+ * which is how "the owner asked for a total and got a dash" is told apart from
+ * "the server answered". Spelling it twice let those two drift apart once
+ * already.
+ */
+const DASH = "\u2014";
+
 function cellText(v: SemanticValue | undefined): string {
-  if (!v) return "—";
+  if (!v) return DASH;
   if (v.formatted !== undefined && v.formatted !== "") return v.formatted;
   const raw = v.value;
-  if (raw === null || raw === undefined) return "—";
+  if (raw === null || raw === undefined) return DASH;
   if (typeof raw === "number") return formatNumber(raw);
   if (typeof raw === "boolean") return raw ? "Yes" : "No";
   if (typeof raw === "string") return raw;
@@ -387,6 +398,23 @@ async function runQuery(s: ReportState, days: number): Promise<ReportState | Dec
       ? copyParam("ownerReport.missingFields", "No data returned for: ") + result.missingFields.join(", ")
       : "";
 
+    // Did the SERVER actually answer with a total for the measure asked for?
+    // This is NOT the same question as "is a totals row on screen": the row is
+    // rendered as a dash when the owner asked and nothing came back, and a dash
+    // is a placeholder, not an answer. Keeping the two apart is what stops a
+    // placeholder from counting as content — see `hideTable`/`noEmpty` below.
+    //
+    // A zero is unaffected: `cellText` formats it as "0", not as the dash, and a
+    // zero total is a real answer that must keep the table up.
+    // Asked of `cellText` rather than re-derived: a `SemanticValue` is an OBJECT
+    // even when it carries `value: null`, so a nullish check on the entry itself
+    // calls a valueless total an answer. `cellText` already owns the decision of
+    // what counts as showable, down to `formatted: ""`, so the question is simply
+    // whether it produced the placeholder.
+    const serverTotal = result.totals?.[s.measureName];
+    const totalText = cellText(serverTotal);
+    const answeredTotal = totalText !== DASH;
+
     // `noTotals` is the owner's REQUEST and stays theirs; only they change it.
     // When they asked and the server sent no total, that is reported as a dash —
     // an absent number the owner asked for must be visible, not silently dropped
@@ -395,9 +423,12 @@ async function runQuery(s: ReportState, days: number): Promise<ReportState | Dec
       ? null
       : {
           dimension: copyParam("ownerReport.totalsLabel", "Total"),
-          measure: result.totals ? cellText(result.totals[s.measureName]) : "—",
+          measure: totalText,
         };
 
+    // Is there anything to READ? Rows, or a total the server actually sent.
+    const hasContent = rows.length > 0 || answeredTotal;
+
     const out: ReportState = {
       ...s,
       status: "ready",
@@ -405,16 +436,22 @@ async function runQuery(s: ReportState, days: number): Promise<ReportState | Dec
         + `${s.measureLabel} ${copyParam("ownerReport.by", "by")} ${s.dimensionLabel}`,
       rows,
       totals,
-      rowCount: rows.length > 0
-        ? `${result.paging.count || rows.length} ${copyParam("ownerReport.rows", "rows")}`
-        : "",
+      // A zero count is stated, not left blank. Silence beside a caption reads
+      // as "still loading" — the ambiguity this screen removes everywhere else —
+      // so "0 rows" says the query ran and came back with nothing.
+      rowCount: `${rows.length > 0 ? result.paging.count || rows.length : 0} ${copyParam("ownerReport.rows", "rows")}`,
       missingFields: gap,
       noGap: gap === "",
-      // Rows OR totals. The API can answer `results: []` WITH a `totals` object
-      // — the SDK has a test for exactly that shape — and gating on rows alone
-      // hid the server's total and put "Nothing to show" over it. Opting into
-      // Show total would then have thrown away the only answer that came back.
-      hideTable: rows.length === 0 && !totals,
+      // Rows OR a total the SERVER sent. The API can answer `results: []` WITH a
+      // `totals` object — the SDK has a test for exactly that shape — and gating
+      // on rows alone hid the server's total and put "Nothing to show" over it.
+      //
+      // But it must be `answeredTotal`, not `totals`: `totals` is non-null for
+      // the whole time the owner has Show total on, INCLUDING when the answer is
+      // the "—" placeholder. Gating on it meant an empty result with the toggle
+      // on rendered a table whose only row was `Total —`, and the empty state
+      // could never fire while the toggle was on. Two states each correct alone.
+      hideTable: !hasContent,
       notLoading: true,
       notErrored: true,
       hideContent: false,
@@ -423,10 +460,10 @@ async function runQuery(s: ReportState, days: number): Promise<ReportState | Dec
       // It is NOT conflated with a dropped field: the SDK reports no
       // `missingFields` when nothing came back at all, precisely because the two
       // are indistinguishable there, so this arm never accuses a field.
-      noEmpty: rows.length > 0 || totals !== null,
-      emptyText: rows.length === 0 && !totals
-        ? copyParam("ownerReport.empty", "No data for this range.")
-        : "",
+      noEmpty: hasContent,
+      emptyText: hasContent
+        ? ""
+        : copyParam("ownerReport.empty", "No data for this range."),
       // Carried through untouched: see the totals comment above.
       noTotals: s.noTotals,
       error: "",

package/templates/community-club-owner/tier5-src/himi-owner-lite-analytics/report.ts

@@ -181,11 +181,22 @@ function subjectLabelOf(m: SemanticModelSummary): string {
  * renders as a dash rather than as 0. `formatNumber` and not `toLocaleString`,
  * which is ECMA-402 and absent on QuickJS-ng (watchOS/Wear OS).
  */
+/**
+ * The one placeholder a cell falls back to when there is nothing to render.
+ *
+ * Named, because two things depend on it meaning exactly one thing: `cellText`
+ * produces it, and `runQuery` asks whether the server's total came back as it —
+ * which is how "the owner asked for a total and got a dash" is told apart from
+ * "the server answered". Spelling it twice let those two drift apart once
+ * already.
+ */
+const DASH = "\u2014";
+
 function cellText(v: SemanticValue | undefined): string {
-  if (!v) return "—";
+  if (!v) return DASH;
   if (v.formatted !== undefined && v.formatted !== "") return v.formatted;
   const raw = v.value;
-  if (raw === null || raw === undefined) return "—";
+  if (raw === null || raw === undefined) return DASH;
   if (typeof raw === "number") return formatNumber(raw);
   if (typeof raw === "boolean") return raw ? "Yes" : "No";
   if (typeof raw === "string") return raw;
@@ -387,6 +398,23 @@ async function runQuery(s: ReportState, days: number): Promise<ReportState | Dec
       ? copyParam("ownerReport.missingFields", "No data returned for: ") + result.missingFields.join(", ")
       : "";
 
+    // Did the SERVER actually answer with a total for the measure asked for?
+    // This is NOT the same question as "is a totals row on screen": the row is
+    // rendered as a dash when the owner asked and nothing came back, and a dash
+    // is a placeholder, not an answer. Keeping the two apart is what stops a
+    // placeholder from counting as content — see `hideTable`/`noEmpty` below.
+    //
+    // A zero is unaffected: `cellText` formats it as "0", not as the dash, and a
+    // zero total is a real answer that must keep the table up.
+    // Asked of `cellText` rather than re-derived: a `SemanticValue` is an OBJECT
+    // even when it carries `value: null`, so a nullish check on the entry itself
+    // calls a valueless total an answer. `cellText` already owns the decision of
+    // what counts as showable, down to `formatted: ""`, so the question is simply
+    // whether it produced the placeholder.
+    const serverTotal = result.totals?.[s.measureName];
+    const totalText = cellText(serverTotal);
+    const answeredTotal = totalText !== DASH;
+
     // `noTotals` is the owner's REQUEST and stays theirs; only they change it.
     // When they asked and the server sent no total, that is reported as a dash —
     // an absent number the owner asked for must be visible, not silently dropped
@@ -395,9 +423,12 @@ async function runQuery(s: ReportState, days: number): Promise<ReportState | Dec
       ? null
       : {
           dimension: copyParam("ownerReport.totalsLabel", "Total"),
-          measure: result.totals ? cellText(result.totals[s.measureName]) : "—",
+          measure: totalText,
         };
 
+    // Is there anything to READ? Rows, or a total the server actually sent.
+    const hasContent = rows.length > 0 || answeredTotal;
+
     const out: ReportState = {
       ...s,
       status: "ready",
@@ -405,16 +436,22 @@ async function runQuery(s: ReportState, days: number): Promise<ReportState | Dec
         + `${s.measureLabel} ${copyParam("ownerReport.by", "by")} ${s.dimensionLabel}`,
       rows,
       totals,
-      rowCount: rows.length > 0
-        ? `${result.paging.count || rows.length} ${copyParam("ownerReport.rows", "rows")}`
-        : "",
+      // A zero count is stated, not left blank. Silence beside a caption reads
+      // as "still loading" — the ambiguity this screen removes everywhere else —
+      // so "0 rows" says the query ran and came back with nothing.
+      rowCount: `${rows.length > 0 ? result.paging.count || rows.length : 0} ${copyParam("ownerReport.rows", "rows")}`,
       missingFields: gap,
       noGap: gap === "",
-      // Rows OR totals. The API can answer `results: []` WITH a `totals` object
-      // — the SDK has a test for exactly that shape — and gating on rows alone
-      // hid the server's total and put "Nothing to show" over it. Opting into
-      // Show total would then have thrown away the only answer that came back.
-      hideTable: rows.length === 0 && !totals,
+      // Rows OR a total the SERVER sent. The API can answer `results: []` WITH a
+      // `totals` object — the SDK has a test for exactly that shape — and gating
+      // on rows alone hid the server's total and put "Nothing to show" over it.
+      //
+      // But it must be `answeredTotal`, not `totals`: `totals` is non-null for
+      // the whole time the owner has Show total on, INCLUDING when the answer is
+      // the "—" placeholder. Gating on it meant an empty result with the toggle
+      // on rendered a table whose only row was `Total —`, and the empty state
+      // could never fire while the toggle was on. Two states each correct alone.
+      hideTable: !hasContent,
       notLoading: true,
       notErrored: true,
       hideContent: false,
@@ -423,10 +460,10 @@ async function runQuery(s: ReportState, days: number): Promise<ReportState | Dec
       // It is NOT conflated with a dropped field: the SDK reports no
       // `missingFields` when nothing came back at all, precisely because the two
       // are indistinguishable there, so this arm never accuses a field.
-      noEmpty: rows.length > 0 || totals !== null,
-      emptyText: rows.length === 0 && !totals
-        ? copyParam("ownerReport.empty", "No data for this range.")
-        : "",
+      noEmpty: hasContent,
+      emptyText: hasContent
+        ? ""
+        : copyParam("ownerReport.empty", "No data for this range."),
       // Carried through untouched: see the totals comment above.
       noTotals: s.noTotals,
       error: "",

package/templates/events-meetups-owner/tier5-src/himi-owner-lite-analytics/report.ts

@@ -181,11 +181,22 @@ function subjectLabelOf(m: SemanticModelSummary): string {
  * renders as a dash rather than as 0. `formatNumber` and not `toLocaleString`,
  * which is ECMA-402 and absent on QuickJS-ng (watchOS/Wear OS).
  */
+/**
+ * The one placeholder a cell falls back to when there is nothing to render.
+ *
+ * Named, because two things depend on it meaning exactly one thing: `cellText`
+ * produces it, and `runQuery` asks whether the server's total came back as it —
+ * which is how "the owner asked for a total and got a dash" is told apart from
+ * "the server answered". Spelling it twice let those two drift apart once
+ * already.
+ */
+const DASH = "\u2014";
+
 function cellText(v: SemanticValue | undefined): string {
-  if (!v) return "—";
+  if (!v) return DASH;
   if (v.formatted !== undefined && v.formatted !== "") return v.formatted;
   const raw = v.value;
-  if (raw === null || raw === undefined) return "—";
+  if (raw === null || raw === undefined) return DASH;
   if (typeof raw === "number") return formatNumber(raw);
   if (typeof raw === "boolean") return raw ? "Yes" : "No";
   if (typeof raw === "string") return raw;
@@ -387,6 +398,23 @@ async function runQuery(s: ReportState, days: number): Promise<ReportState | Dec
       ? copyParam("ownerReport.missingFields", "No data returned for: ") + result.missingFields.join(", ")
       : "";
 
+    // Did the SERVER actually answer with a total for the measure asked for?
+    // This is NOT the same question as "is a totals row on screen": the row is
+    // rendered as a dash when the owner asked and nothing came back, and a dash
+    // is a placeholder, not an answer. Keeping the two apart is what stops a
+    // placeholder from counting as content — see `hideTable`/`noEmpty` below.
+    //
+    // A zero is unaffected: `cellText` formats it as "0", not as the dash, and a
+    // zero total is a real answer that must keep the table up.
+    // Asked of `cellText` rather than re-derived: a `SemanticValue` is an OBJECT
+    // even when it carries `value: null`, so a nullish check on the entry itself
+    // calls a valueless total an answer. `cellText` already owns the decision of
+    // what counts as showable, down to `formatted: ""`, so the question is simply
+    // whether it produced the placeholder.
+    const serverTotal = result.totals?.[s.measureName];
+    const totalText = cellText(serverTotal);
+    const answeredTotal = totalText !== DASH;
+
     // `noTotals` is the owner's REQUEST and stays theirs; only they change it.
     // When they asked and the server sent no total, that is reported as a dash —
     // an absent number the owner asked for must be visible, not silently dropped
@@ -395,9 +423,12 @@ async function runQuery(s: ReportState, days: number): Promise<ReportState | Dec
       ? null
       : {
           dimension: copyParam("ownerReport.totalsLabel", "Total"),
-          measure: result.totals ? cellText(result.totals[s.measureName]) : "—",
+          measure: totalText,
         };
 
+    // Is there anything to READ? Rows, or a total the server actually sent.
+    const hasContent = rows.length > 0 || answeredTotal;
+
     const out: ReportState = {
       ...s,
       status: "ready",
@@ -405,16 +436,22 @@ async function runQuery(s: ReportState, days: number): Promise<ReportState | Dec
         + `${s.measureLabel} ${copyParam("ownerReport.by", "by")} ${s.dimensionLabel}`,
       rows,
       totals,
-      rowCount: rows.length > 0
-        ? `${result.paging.count || rows.length} ${copyParam("ownerReport.rows", "rows")}`
-        : "",
+      // A zero count is stated, not left blank. Silence beside a caption reads
+      // as "still loading" — the ambiguity this screen removes everywhere else —
+      // so "0 rows" says the query ran and came back with nothing.
+      rowCount: `${rows.length > 0 ? result.paging.count || rows.length : 0} ${copyParam("ownerReport.rows", "rows")}`,
       missingFields: gap,
       noGap: gap === "",
-      // Rows OR totals. The API can answer `results: []` WITH a `totals` object
-      // — the SDK has a test for exactly that shape — and gating on rows alone
-      // hid the server's total and put "Nothing to show" over it. Opting into
-      // Show total would then have thrown away the only answer that came back.
-      hideTable: rows.length === 0 && !totals,
+      // Rows OR a total the SERVER sent. The API can answer `results: []` WITH a
+      // `totals` object — the SDK has a test for exactly that shape — and gating
+      // on rows alone hid the server's total and put "Nothing to show" over it.
+      //
+      // But it must be `answeredTotal`, not `totals`: `totals` is non-null for
+      // the whole time the owner has Show total on, INCLUDING when the answer is
+      // the "—" placeholder. Gating on it meant an empty result with the toggle
+      // on rendered a table whose only row was `Total —`, and the empty state
+      // could never fire while the toggle was on. Two states each correct alone.
+      hideTable: !hasContent,
       notLoading: true,
       notErrored: true,
       hideContent: false,
@@ -423,10 +460,10 @@ async function runQuery(s: ReportState, days: number): Promise<ReportState | Dec
       // It is NOT conflated with a dropped field: the SDK reports no
       // `missingFields` when nothing came back at all, precisely because the two
       // are indistinguishable there, so this arm never accuses a field.
-      noEmpty: rows.length > 0 || totals !== null,
-      emptyText: rows.length === 0 && !totals
-        ? copyParam("ownerReport.empty", "No data for this range.")
-        : "",
+      noEmpty: hasContent,
+      emptyText: hasContent
+        ? ""
+        : copyParam("ownerReport.empty", "No data for this range."),
       // Carried through untouched: see the totals comment above.
       noTotals: s.noTotals,
       error: "",

package/templates/gym-fitness-owner/tier5-src/himi-owner-lite-analytics/report.ts

@@ -181,11 +181,22 @@ function subjectLabelOf(m: SemanticModelSummary): string {
  * renders as a dash rather than as 0. `formatNumber` and not `toLocaleString`,
  * which is ECMA-402 and absent on QuickJS-ng (watchOS/Wear OS).
  */
+/**
+ * The one placeholder a cell falls back to when there is nothing to render.
+ *
+ * Named, because two things depend on it meaning exactly one thing: `cellText`
+ * produces it, and `runQuery` asks whether the server's total came back as it —
+ * which is how "the owner asked for a total and got a dash" is told apart from
+ * "the server answered". Spelling it twice let those two drift apart once
+ * already.
+ */
+const DASH = "\u2014";
+
 function cellText(v: SemanticValue | undefined): string {
-  if (!v) return "—";
+  if (!v) return DASH;
   if (v.formatted !== undefined && v.formatted !== "") return v.formatted;
   const raw = v.value;
-  if (raw === null || raw === undefined) return "—";
+  if (raw === null || raw === undefined) return DASH;
   if (typeof raw === "number") return formatNumber(raw);
   if (typeof raw === "boolean") return raw ? "Yes" : "No";
   if (typeof raw === "string") return raw;
@@ -387,6 +398,23 @@ async function runQuery(s: ReportState, days: number): Promise<ReportState | Dec
       ? copyParam("ownerReport.missingFields", "No data returned for: ") + result.missingFields.join(", ")
       : "";
 
+    // Did the SERVER actually answer with a total for the measure asked for?
+    // This is NOT the same question as "is a totals row on screen": the row is
+    // rendered as a dash when the owner asked and nothing came back, and a dash
+    // is a placeholder, not an answer. Keeping the two apart is what stops a
+    // placeholder from counting as content — see `hideTable`/`noEmpty` below.
+    //
+    // A zero is unaffected: `cellText` formats it as "0", not as the dash, and a
+    // zero total is a real answer that must keep the table up.
+    // Asked of `cellText` rather than re-derived: a `SemanticValue` is an OBJECT
+    // even when it carries `value: null`, so a nullish check on the entry itself
+    // calls a valueless total an answer. `cellText` already owns the decision of
+    // what counts as showable, down to `formatted: ""`, so the question is simply
+    // whether it produced the placeholder.
+    const serverTotal = result.totals?.[s.measureName];
+    const totalText = cellText(serverTotal);
+    const answeredTotal = totalText !== DASH;
+
     // `noTotals` is the owner's REQUEST and stays theirs; only they change it.
     // When they asked and the server sent no total, that is reported as a dash —
     // an absent number the owner asked for must be visible, not silently dropped
@@ -395,9 +423,12 @@ async function runQuery(s: ReportState, days: number): Promise<ReportState | Dec
       ? null
       : {
           dimension: copyParam("ownerReport.totalsLabel", "Total"),
-          measure: result.totals ? cellText(result.totals[s.measureName]) : "—",
+          measure: totalText,
         };
 
+    // Is there anything to READ? Rows, or a total the server actually sent.
+    const hasContent = rows.length > 0 || answeredTotal;
+
     const out: ReportState = {
       ...s,
       status: "ready",
@@ -405,16 +436,22 @@ async function runQuery(s: ReportState, days: number): Promise<ReportState | Dec
         + `${s.measureLabel} ${copyParam("ownerReport.by", "by")} ${s.dimensionLabel}`,
       rows,
       totals,
-      rowCount: rows.length > 0
-        ? `${result.paging.count || rows.length} ${copyParam("ownerReport.rows", "rows")}`
-        : "",
+      // A zero count is stated, not left blank. Silence beside a caption reads
+      // as "still loading" — the ambiguity this screen removes everywhere else —
+      // so "0 rows" says the query ran and came back with nothing.
+      rowCount: `${rows.length > 0 ? result.paging.count || rows.length : 0} ${copyParam("ownerReport.rows", "rows")}`,
       missingFields: gap,
       noGap: gap === "",
-      // Rows OR totals. The API can answer `results: []` WITH a `totals` object
-      // — the SDK has a test for exactly that shape — and gating on rows alone
-      // hid the server's total and put "Nothing to show" over it. Opting into
-      // Show total would then have thrown away the only answer that came back.
-      hideTable: rows.length === 0 && !totals,
+      // Rows OR a total the SERVER sent. The API can answer `results: []` WITH a
+      // `totals` object — the SDK has a test for exactly that shape — and gating
+      // on rows alone hid the server's total and put "Nothing to show" over it.
+      //
+      // But it must be `answeredTotal`, not `totals`: `totals` is non-null for
+      // the whole time the owner has Show total on, INCLUDING when the answer is
+      // the "—" placeholder. Gating on it meant an empty result with the toggle
+      // on rendered a table whose only row was `Total —`, and the empty state
+      // could never fire while the toggle was on. Two states each correct alone.
+      hideTable: !hasContent,
       notLoading: true,
       notErrored: true,
       hideContent: false,
@@ -423,10 +460,10 @@ async function runQuery(s: ReportState, days: number): Promise<ReportState | Dec
       // It is NOT conflated with a dropped field: the SDK reports no
       // `missingFields` when nothing came back at all, precisely because the two
       // are indistinguishable there, so this arm never accuses a field.
-      noEmpty: rows.length > 0 || totals !== null,
-      emptyText: rows.length === 0 && !totals
-        ? copyParam("ownerReport.empty", "No data for this range.")
-        : "",
+      noEmpty: hasContent,
+      emptyText: hasContent
+        ? ""
+        : copyParam("ownerReport.empty", "No data for this range."),
       // Carried through untouched: see the totals comment above.
       noTotals: s.noTotals,
       error: "",

package/templates/nonprofit-cause-owner/tier5-src/himi-owner-lite-analytics/report.ts

@@ -181,11 +181,22 @@ function subjectLabelOf(m: SemanticModelSummary): string {
  * renders as a dash rather than as 0. `formatNumber` and not `toLocaleString`,
  * which is ECMA-402 and absent on QuickJS-ng (watchOS/Wear OS).
  */
+/**
+ * The one placeholder a cell falls back to when there is nothing to render.
+ *
+ * Named, because two things depend on it meaning exactly one thing: `cellText`
+ * produces it, and `runQuery` asks whether the server's total came back as it —
+ * which is how "the owner asked for a total and got a dash" is told apart from
+ * "the server answered". Spelling it twice let those two drift apart once
+ * already.
+ */
+const DASH = "\u2014";
+
 function cellText(v: SemanticValue | undefined): string {
-  if (!v) return "—";
+  if (!v) return DASH;
   if (v.formatted !== undefined && v.formatted !== "") return v.formatted;
   const raw = v.value;
-  if (raw === null || raw === undefined) return "—";
+  if (raw === null || raw === undefined) return DASH;
   if (typeof raw === "number") return formatNumber(raw);
   if (typeof raw === "boolean") return raw ? "Yes" : "No";
   if (typeof raw === "string") return raw;
@@ -387,6 +398,23 @@ async function runQuery(s: ReportState, days: number): Promise<ReportState | Dec
       ? copyParam("ownerReport.missingFields", "No data returned for: ") + result.missingFields.join(", ")
       : "";
 
+    // Did the SERVER actually answer with a total for the measure asked for?
+    // This is NOT the same question as "is a totals row on screen": the row is
+    // rendered as a dash when the owner asked and nothing came back, and a dash
+    // is a placeholder, not an answer. Keeping the two apart is what stops a
+    // placeholder from counting as content — see `hideTable`/`noEmpty` below.
+    //
+    // A zero is unaffected: `cellText` formats it as "0", not as the dash, and a
+    // zero total is a real answer that must keep the table up.
+    // Asked of `cellText` rather than re-derived: a `SemanticValue` is an OBJECT
+    // even when it carries `value: null`, so a nullish check on the entry itself
+    // calls a valueless total an answer. `cellText` already owns the decision of
+    // what counts as showable, down to `formatted: ""`, so the question is simply
+    // whether it produced the placeholder.
+    const serverTotal = result.totals?.[s.measureName];
+    const totalText = cellText(serverTotal);
+    const answeredTotal = totalText !== DASH;
+
     // `noTotals` is the owner's REQUEST and stays theirs; only they change it.
     // When they asked and the server sent no total, that is reported as a dash —
     // an absent number the owner asked for must be visible, not silently dropped
@@ -395,9 +423,12 @@ async function runQuery(s: ReportState, days: number): Promise<ReportState | Dec
       ? null
       : {
           dimension: copyParam("ownerReport.totalsLabel", "Total"),
-          measure: result.totals ? cellText(result.totals[s.measureName]) : "—",
+          measure: totalText,
         };
 
+    // Is there anything to READ? Rows, or a total the server actually sent.
+    const hasContent = rows.length > 0 || answeredTotal;
+
     const out: ReportState = {
       ...s,
       status: "ready",
@@ -405,16 +436,22 @@ async function runQuery(s: ReportState, days: number): Promise<ReportState | Dec
         + `${s.measureLabel} ${copyParam("ownerReport.by", "by")} ${s.dimensionLabel}`,
       rows,
       totals,
-      rowCount: rows.length > 0
-        ? `${result.paging.count || rows.length} ${copyParam("ownerReport.rows", "rows")}`
-        : "",
+      // A zero count is stated, not left blank. Silence beside a caption reads
+      // as "still loading" — the ambiguity this screen removes everywhere else —
+      // so "0 rows" says the query ran and came back with nothing.
+      rowCount: `${rows.length > 0 ? result.paging.count || rows.length : 0} ${copyParam("ownerReport.rows", "rows")}`,
       missingFields: gap,
       noGap: gap === "",
-      // Rows OR totals. The API can answer `results: []` WITH a `totals` object
-      // — the SDK has a test for exactly that shape — and gating on rows alone
-      // hid the server's total and put "Nothing to show" over it. Opting into
-      // Show total would then have thrown away the only answer that came back.
-      hideTable: rows.length === 0 && !totals,
+      // Rows OR a total the SERVER sent. The API can answer `results: []` WITH a
+      // `totals` object — the SDK has a test for exactly that shape — and gating
+      // on rows alone hid the server's total and put "Nothing to show" over it.
+      //
+      // But it must be `answeredTotal`, not `totals`: `totals` is non-null for
+      // the whole time the owner has Show total on, INCLUDING when the answer is
+      // the "—" placeholder. Gating on it meant an empty result with the toggle
+      // on rendered a table whose only row was `Total —`, and the empty state
+      // could never fire while the toggle was on. Two states each correct alone.
+      hideTable: !hasContent,
       notLoading: true,
       notErrored: true,
       hideContent: false,
@@ -423,10 +460,10 @@ async function runQuery(s: ReportState, days: number): Promise<ReportState | Dec
       // It is NOT conflated with a dropped field: the SDK reports no
       // `missingFields` when nothing came back at all, precisely because the two
       // are indistinguishable there, so this arm never accuses a field.
-      noEmpty: rows.length > 0 || totals !== null,
-      emptyText: rows.length === 0 && !totals
-        ? copyParam("ownerReport.empty", "No data for this range.")
-        : "",
+      noEmpty: hasContent,
+      emptyText: hasContent
+        ? ""
+        : copyParam("ownerReport.empty", "No data for this range."),
       // Carried through untouched: see the totals comment above.
       noTotals: s.noTotals,
       error: "",