@bongos/core 1.19.455 → 1.19.457

package/modules/hall-ui/public/sessions.states.json

@@ -0,0 +1,58 @@
+{
+  "_": "Sessions — the session ledger behind requireGovernmentPage. sessions.js pages SERVER-SIDE by offset and is the one sanctioned hand-rolled pager in the hall; this file measures it, it does not ask it to change. Written with the re-skin (task 1003307) so this page is MEASURED at three widths in both modes rather than looked at once.",
+  "page": "/builders/sessions",
+  "surface": "hall-ui",
+  "stub": {
+    "prefix": "/builders/"
+  },
+  "modeQuery": false,
+  "_feeds": "The kit's own stub carries no feed for this page, so against it the page renders its EMPTY and SKELETON states — which is the point: those are states the floors must hold in. Against the hall-preview harness the same states render populated. Every declared 404 below is a fixture the harness does not carry, named rather than hidden.",
+  "ignoreRequests": [
+    "/api/(gds|bongos)/me$",
+    "/api/(gds|bongos)/me/",
+    "/api/(gds|bongos)/goals",
+    "/api/(gds|bongos)/inbox/",
+    "/api/(gds|bongos)/versions",
+    "/api/(gds|bongos)/sessions"
+  ],
+  "ignoreConsole": [
+    "/api/(gds|bongos)/",
+    "\\b401\\b",
+    "\\b403\\b",
+    "\\b404\\b",
+    "\\b501\\b",
+    "Unauthorized",
+    "Forbidden",
+    "Not Found",
+    "Not Implemented",
+    "load failed"
+  ],
+  "states": {
+    "ledger": {
+      "auth": true,
+      "actions": [
+        [
+          "wait",
+          700
+        ]
+      ],
+      "expect": {
+        "visible": [
+          "#stats-scroll"
+        ]
+      }
+    },
+    "out": {
+      "auth": false
+    }
+  },
+  "reducedMotion": {
+    "auth": true,
+    "actions": [
+      [
+        "wait",
+        600
+      ]
+    ]
+  }
+}

package/modules/hall-ui/public/style.css

@@ -222,6 +222,11 @@
   --z-sidebar: 40;
   --z-scrim: 140;
   --z-drawer: 150;
+  /* A confirm dialog is the one thing that must sit above the drawer: it is
+     asking a question the drawer's content cannot answer. Added by task 1003307
+     because harbor and gate had each guessed a bare number for exactly this
+     (200 and 50), which is what a missing layer looks like. */
+  --z-modal: 160;
   --z-diagram-full: 9999;
 
   /* ---- the hall's own plate: the declared seam, deferred by the owner
@@ -1088,7 +1093,14 @@ main:focus { outline: none; }
 .next-up__foot a,
 .setup__steps a,
 .setup__hint a,
-.setup__foot a {
+.setup__foot a,
+/* task 1003307 — the two empty-state voices. .empty-state is the kit's own and
+   was simply missing from this list; .cb-empty is the same component under
+   collab's older name (Collab's "set yours in Settings" link measured 51x18).
+   Unifying the two names is worth doing, but renaming a class is not a
+   re-skin's business. */
+.empty-state p a,
+.cb-empty a {
   display: inline-block;
   min-width: 24px;
   min-height: 24px;
@@ -1144,14 +1156,20 @@ main:focus { outline: none; }
   gap: 8px;
 }
 .stat-tile:first-child { border-left: 0; }
-.stat-tile__label {
+.stat-tile__label,
+.gate-stat__label,
+.harbor-stat__label,
+.watch-stat__label {
   font-family: var(--label-face);
   font-size: var(--label-size);
   letter-spacing: var(--label-space);
   text-transform: var(--label-case);
   color: var(--ink-faint);
 }
-.stat-tile__num {
+.stat-tile__num,
+.gate-stat__value,
+.harbor-stat__value,
+.watch-stat__num {
   font-family: var(--font-display);
   font-size: clamp(26px, 2.6vw, 34px);
   font-weight: 300;
@@ -1415,7 +1433,19 @@ main [id] { scroll-margin-top: calc(var(--topbar-h) + 16px); }
 .ledger tbody tr:hover td { background: var(--glass); }
 /* task 1467: a builder name that links to their profile page reads as plain
    text in the dense leaderboard/roster, revealing itself as a link on hover. */
-.lb-name { color: inherit; }
+.lb-name {
+  color: inherit;
+  /* A link is a control and owes the 24px floor on BOTH axes. This one is plain
+     text in a dense ledger, so it had no box of its own and measured 83x18 on
+     the Watch board (task 1003307). The vertical padding buys the height; the
+     equal negative margin gives the space back so the row keeps its rhythm. */
+  display: inline-block;
+  min-height: 24px;
+  min-width: 24px;
+  padding: 3px 4px;
+  margin: 0 -4px;
+  line-height: 18px;
+}
 .lb-name:hover { color: var(--accent-ink); text-decoration: underline; text-underline-offset: 3px; }
 .ledger__empty {
   color: var(--ink-faint);
@@ -2815,6 +2845,30 @@ main [id] { scroll-margin-top: calc(var(--topbar-h) + 16px); }
 .kit-tabs .setup__tab .count-pill { margin-left: 6px; }
 
 /* empty state — one short line + at most one action link */
+/* ── four classes that were referenced and defined nowhere (task 1003307) ────
+   Each one rendered its text at the browser's default, which is why nobody
+   noticed: an unstyled 16px serif-ish line next to a styled one reads as "a bit
+   off", not as a bug. Found by auditing the pages against the archetype. */
+
+/* collab.html:38,51 — the section note under a .scroll__h. The shell's name for
+   this is .scroll__lede; both collab sections asked for .scroll__note. Rather
+   than edit the markup, the name joins the lede it meant. */
+.scroll__note { margin: 0 0 14px; font-size: 13.5px; color: var(--ink-soft); max-width: var(--measure); }
+
+/* collab.js:73-74 — the settle buttons asked for a small variant of .btn that
+   does not exist, so "Someone helped" rendered full-size on every row. */
+.btn--sm { min-height: 26px; padding: 0 11px; font-size: 12.5px; }
+
+/* people.js:45 — the builder's name cell. The only definition in the hall lives
+   in scouting.html's inline <style>, so the same data read bold on Scouting and
+   plain on People. One definition, both pages. */
+.who { font-weight: 600; }
+
+/* people.js:105 — notice() sets body.className='note'. .note lives in
+   my-projects.css, which People never loads, so every gated / empty / error
+   message on that page rendered unstyled. */
+.note { margin: 0; font-size: 13.5px; color: var(--ink-soft); }
+
 .empty-state { padding: 8px 0; }
 .empty-state p { margin: 0; font-size: 13.5px; color: var(--ink-soft); }
 .empty-state__action { display: inline-block; margin-top: 6px; font-size: 13px; }
@@ -2980,10 +3034,76 @@ html.palette-open { overflow: hidden; }
   /* The slot keeps its icon as an affordance; the label + hint would crowd the
      title on a phone. Ctrl/Cmd-K still opens it on a keyboard. */
   .topbar-jump__label, .topbar-jump__kbd { display: none; }
-  .topbar-jump { padding: 0 8px; }
+  /* ...but the button carries min-width: 0 so it can shrink in the topbar's flex
+     row, and with the label and the hint gone there is nothing left holding it
+     open: it measured 18x32 at 320px, under the 24px floor on the horizontal
+     axis. The icon-only button gets its width back explicitly (task 1003307). */
+  .topbar-jump { padding: 0 8px; min-width: 32px; justify-content: center; }
   .palette__panel { margin-top: 6vh; }
   .palette__group { min-width: 56px; }
 }
+/* ── the fact vocabulary (task 1003307) ──────────────────────────────────────
+   A row states facts about a record: its state, and the things it is scoped to.
+   The archetype (docs/design/mocks/hall/goals/v3.html, .status + .chip) says a
+   state is a HAIRLINE in its own colour with no fill, and a scope is a quiet
+   mono chip. Neither is pressable, so neither takes the accent.
+
+   These live here because five pages had already invented them separately —
+   .gov-pill, .harbor-pill, .gate-pill, .badge, .mod-chip — each re-spelling the
+   label voice as literals. The shell had no neutral pill to take: .chip below is
+   the accent-tinted FILTER chip, and it means "press me".
+
+   The state colours are the hall's fixed verification vocabulary, never the
+   world's warm accent states (ADR 0196 §2): a status must not re-tint when an
+   instance re-brands. */
+/* The five names below are the SAME pill, found by the task 1003307 audit:
+   government, harbor, gate, watch and modules each invented one because the
+   shell had none to take. They are aliased rather than renamed because renaming
+   would mean editing five JS row builders, and a class name is not what was
+   wrong — the duplicated, hand-rolled label voice was. Their own state
+   modifiers stay in their own sheets. */
+.fact-pill,
+.gov-pill,
+.harbor-pill,
+.gate-pill,
+.badge,
+.mod-chip {
+  display: inline-flex;
+  align-items: center;
+  min-height: 24px;
+  padding: 0 11px;
+  border: var(--line) solid currentColor;
+  border-radius: var(--radius-pill);
+  background: none;
+  font-family: var(--label-face);
+  font-size: 10.5px;
+  letter-spacing: var(--label-space);
+  text-transform: var(--label-case);
+  color: var(--ink-faint);
+  white-space: nowrap;
+}
+.fact-pill--open, .fact-pill--info { color: var(--info); }
+.fact-pill--blocked, .fact-pill--warn { color: var(--warn); }
+.fact-pill--achieved, .fact-pill--ok { color: var(--ok); }
+.fact-pill--danger { color: var(--danger); }
+.fact-pill--archived, .fact-pill--quiet { color: var(--ink-faint); }
+
+/* A scope, not a state: what the record is filed under. Mono because it is a
+   machine's word (a module key, a version id), --radius-sm because a pill would
+   read as a second status. */
+.fact-chip {
+  display: inline-flex;
+  align-items: center;
+  min-height: 24px;
+  padding: 0 10px;
+  border: var(--line) solid var(--rule-soft);
+  border-radius: var(--radius-sm);
+  font-family: var(--font-mono);
+  font-size: 11px;
+  color: var(--ink-faint);
+  white-space: nowrap;
+}
+
 /* task 1003276 — the human CATEGORY chip. Quiet by design: it sits beside the
    meta line, not above the title, because it is a label a person scans, never a
    status or a permission (ADR 0192). No hover/pointer affordance and no href —

package/modules/hall-ui/public/watch.css

@@ -20,8 +20,8 @@
   margin-top: 6px;
 }
 .watch-stat {
-  background: var(--surface);
-  border: 1px solid var(--border);
+  background: var(--bg-card);
+  border: var(--line) solid var(--rule-soft);
   border-radius: var(--radius-sm);
   padding: 12px 14px;
   text-align: center;
@@ -53,7 +53,7 @@ table.watch {
 table.watch th, table.watch td {
   padding: 6px 10px;
   text-align: left;
-  border-bottom: 1px solid var(--border);
+  border-bottom: var(--line) solid var(--rule-soft);
   vertical-align: top;
 }
 table.watch th {
@@ -62,31 +62,37 @@ table.watch th {
   text-transform: uppercase;
   letter-spacing: 0.06em;
   color: var(--ink-faint);
-  border-bottom: 1px solid var(--border-strong);
+  border-bottom: var(--line) solid var(--rule);
   white-space: nowrap;
 }
 table.watch td.num { text-align: right; font-variant-numeric: tabular-nums; }
 table.watch td.warn { color: var(--danger); font-weight: 600; }
-table.watch tr:hover td { background: var(--surface-2); }
+table.watch tr:hover td { background: var(--glass); }
 /* the kit ledger (audit log) shares the same numeric-cell affordances */
 .ledger td.num { text-align: right; font-variant-numeric: tabular-nums; }
 .ledger td.num.warn { color: var(--danger); font-weight: 600; }
-.watch-mono { font-family: ui-monospace, "SF Mono", Menlo, monospace; font-size: 0.84rem; color: var(--info); }
+.watch-mono { font-family: var(--font-mono); font-size: 0.84rem; color: var(--info); }
 .watch-empty { color: var(--ink-faint); font-size: 13px; padding: 10px 0; }
+/* .watch-subh is .watch-sub-h without the second hyphen, and watch.js emits
+   both (:400/832/857 vs :638/640). The base rule below already groups them, so
+   the two spellings LOOK the same — the only real divergence was :first-of-type
+   below, which matched one spelling and not the other, so a section whose first
+   sub-heading happened to be spelled the other way carried 16px of top margin
+   instead of 8. Grouped there too (task 1003307). */
 .watch-note { color: var(--ink-faint); font-size: 12.5px; margin-top: 10px; }
 .watch-rank { font-size: 12px; color: var(--ink-faint); }
 .watch-doc {
   white-space: pre-wrap;
   word-break: break-word;
-  font-family: ui-monospace, "SF Mono", Menlo, monospace;
+  font-family: var(--font-mono);
   font-size: 0.8rem;
   line-height: 1.5;
   max-height: 60vh;
   overflow-y: auto;
   padding: 12px;
   margin-top: 8px;
-  background: var(--surface-2);
-  border: 1px solid var(--border);
+  background: var(--glass);
+  border: var(--line) solid var(--rule-soft);
   border-radius: var(--radius-sm);
 }
 .watch-sub-h, .watch-subh {
@@ -96,26 +102,17 @@ table.watch tr:hover td { background: var(--surface-2); }
   color: var(--ink);
   margin: 16px 0 4px;
 }
-.watch-sub-h:first-of-type { margin-top: 8px; }
+.watch-sub-h:first-of-type, .watch-subh:first-of-type { margin-top: 8px; }
 
 /* Severity / status badges. */
-.badge {
-  display: inline-block;
-  font-size: 11px;
-  font-weight: 600;
-  letter-spacing: 0.04em;
-  text-transform: uppercase;
-  padding: 2px 8px;
-  border-radius: 999px;
-  border: 1px solid var(--border-strong);
-  color: var(--ink-soft);
-  white-space: nowrap;
-}
-.badge--sev-critical { background: var(--danger-weak); color: var(--danger); border-color: color-mix(in srgb, var(--danger) 35%, transparent); }
-.badge--sev-high { background: var(--warn-weak); color: var(--warn); border-color: color-mix(in srgb, var(--warn) 35%, transparent); }
-.badge--sev-medium { background: var(--info-weak); color: var(--info); border-color: color-mix(in srgb, var(--info) 30%, transparent); }
-.badge--sev-low { background: var(--surface-2); color: var(--ink-soft); }
-.badge--status { background: var(--surface-2); color: var(--ink-soft); }
+/* Base: the shell's .fact-pill (task 1003307). A severity is a state, so it is a
+   hairline in its own colour rather than a filled chip — four filled severities
+   stacked in one table row was the loudest thing on the Watch board. */
+.badge--sev-critical { color: var(--danger); }
+.badge--sev-high { color: var(--warn); }
+.badge--sev-medium { color: var(--info); }
+.badge--sev-low { color: var(--ink-faint); }
+.badge--status { color: var(--ink-faint); }
 
 /* Row action buttons + selects. */
 .watch-actions { display: flex; gap: 6px; flex-wrap: wrap; align-items: center; }
@@ -124,27 +121,27 @@ table.watch tr:hover td { background: var(--surface-2); }
   font-size: 12.5px;
   font-weight: 500;
   padding: 4px 12px;
-  border: 1px solid var(--border-strong);
-  background: var(--surface);
+  border: var(--line) solid var(--rule);
+  background: var(--bg-card);
   color: var(--ink-soft);
   border-radius: var(--radius-sm);
   cursor: pointer;
 }
-.watch-btn:hover:not(:disabled) { background: var(--surface-2); color: var(--ink); }
+.watch-btn:hover:not(:disabled) { background: var(--glass); color: var(--ink); }
 .watch-btn:disabled { opacity: 0.45; cursor: default; }
-.watch-btn--danger { color: var(--danger); border-color: color-mix(in srgb, var(--danger) 40%, var(--border-strong)); }
+.watch-btn--danger { color: var(--danger); border-color: color-mix(in srgb, var(--danger) 40%, var(--rule)); }
 .watch-select {
   font-family: var(--font-body);
   font-size: 13px;
   padding: 4px 6px;
-  border: 1px solid var(--border-strong);
-  background: var(--surface);
+  border: var(--line) solid var(--rule);
+  background: var(--bg-card);
   color: var(--ink);
   border-radius: var(--radius-sm);
 }
 
 .watch-req {
-  border-bottom: 1px solid var(--border);
+  border-bottom: var(--line) solid var(--rule-soft);
   padding: 10px 0;
   display: flex; gap: 14px; flex-wrap: wrap;
   align-items: flex-start; justify-content: space-between;
@@ -162,8 +159,8 @@ table.watch tr:hover td { background: var(--surface-2); }
   font-size: 13px; color: var(--ink);
 }
 .watch-trend__bar-track {
-  background: var(--surface-2); border-radius: 3px; height: 12px; overflow: hidden;
-  border: 1px solid var(--border);
+  background: var(--glass); border-radius: 3px; height: 12px; overflow: hidden;
+  border: var(--line) solid var(--rule-soft);
 }
 .watch-trend__bar-fill { background: var(--accent); height: 100%; display: block; }
 .watch-trend__val { text-align: right; font-variant-numeric: tabular-nums; color: var(--ink-soft); }
@@ -171,7 +168,7 @@ table.watch tr:hover td { background: var(--surface-2); }
 .watch-list { list-style: none; padding: 0; margin: 8px 0 0; }
 .watch-list li {
   display: flex; justify-content: space-between; gap: 12px;
-  padding: 5px 0; border-bottom: 1px solid var(--border);
+  padding: 5px 0; border-bottom: var(--line) solid var(--rule-soft);
   font-size: 13px;
 }
 .watch-list li .k { color: var(--ink); }
@@ -180,12 +177,12 @@ table.watch tr:hover td { background: var(--surface-2); }
 .watch-foot-link {
   display: inline-block; margin-top: 12px;
   font-size: 13px; font-weight: 500;
-  color: var(--accent-text);
+  color: var(--accent-ink);
 }
 
 /* Invite by GitHub username (task 1003044) — the Access requests section's own
    form: the way in on an invite-only project, a shortcut on an apply one. */
-.watch-invite { margin: 0 0 14px; padding: 12px 14px; border: 1px solid var(--rule-soft); border-radius: 10px; }
+.watch-invite { margin: 0 0 14px; padding: 12px 14px; border: var(--line) solid var(--rule-soft); border-radius: 10px; }
 .watch-invite__label { display: block; font-weight: 600; margin-bottom: 6px; }
 .watch-invite__row { display: flex; gap: 8px; align-items: center; flex-wrap: wrap; }
 .watch-invite__row input[type="text"] { flex: 1 1 200px; min-width: 0; }

package/modules/hall-ui/public/watch.html

@@ -22,7 +22,15 @@
   <div class="app-body">
     <header class="app-topbar" id="app-topbar"></header>
     <div class="app-content">
-      <div class="page-head"><p class="page-head__sub" id="watch-sub">Loading…</p></div>
+      <!-- task 1003307: the board page head. Watch shipped with NO <h1> at all
+           (one-header-main-h1 measured 0 on every render), so the re-skin
+           carries the structural fix rather than porting the hole. The heading
+           is the LABEL voice, not a display line: on a working surface the
+           page's own name is the least interesting thing on it. -->
+      <div class="page-head page-head--board">
+        <h1 class="page-head__h">Watch</h1>
+        <p class="page-head__sub" id="watch-sub">Loading…</p>
+      </div>
 
       <main id="hall-main">
         <!-- Restricted notice — shown by watch.js when the viewer is not an Archon. -->

package/modules/hall-ui/public/watch.states.json

@@ -0,0 +1,60 @@
+{
+  "_": "Watch — the attention board behind requireGovernmentPage. The SEALED state (a viewer below the gate) is a real render, not an error page, so it is a state here. Written with the re-skin (task 1003307) so this page is MEASURED at three widths in both modes rather than looked at once.",
+  "page": "/builders/watch",
+  "surface": "hall-ui",
+  "stub": {
+    "prefix": "/builders/"
+  },
+  "modeQuery": false,
+  "_feeds": "The kit's own stub carries no feed for this page, so against it the page renders its EMPTY and SKELETON states — which is the point: those are states the floors must hold in. Against the hall-preview harness the same states render populated. Every declared 404 below is a fixture the harness does not carry, named rather than hidden.",
+  "ignoreRequests": [
+    "/api/(gds|bongos)/me$",
+    "/api/(gds|bongos)/me/",
+    "/api/(gds|bongos)/goals",
+    "/api/(gds|bongos)/inbox/",
+    "/api/(gds|bongos)/versions",
+    "/api/(gds|bongos)/watch",
+    "/api/(gds|bongos)/analytics",
+    "/api/(gds|bongos)/tasks"
+  ],
+  "ignoreConsole": [
+    "/api/(gds|bongos)/",
+    "\\b401\\b",
+    "\\b403\\b",
+    "\\b404\\b",
+    "\\b501\\b",
+    "Unauthorized",
+    "Forbidden",
+    "Not Found",
+    "Not Implemented",
+    "load failed"
+  ],
+  "states": {
+    "board": {
+      "auth": true,
+      "actions": [
+        [
+          "wait",
+          700
+        ]
+      ],
+      "expect": {
+        "visible": [
+          "#watch-attention"
+        ]
+      }
+    },
+    "out": {
+      "auth": false
+    }
+  },
+  "reducedMotion": {
+    "auth": true,
+    "actions": [
+      [
+        "wait",
+        600
+      ]
+    ]
+  }
+}

package/package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "@bongos/core",
-  "version": "1.19.455",
+  "version": "1.19.457",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@bongos/core",
-      "version": "1.19.455",
+      "version": "1.19.457",
       "license": "AGPL-3.0-or-later",
       "dependencies": {
         "express": "^4.21.2",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bongos/core",
-  "version": "1.19.455",
+  "version": "1.19.457",
   "description": "Cloud Bongos — the AI-first build platform core (GDS + platform surfaces + module system), installed as a versioned dependency (ADR 0108).",
   "license": "AGPL-3.0-or-later",
   "main": "src/platform-server.js",

package/src/module-api.js

@@ -33,7 +33,7 @@ const { validateOrRespond, LIMITS, parseId, asyncHandler, corsPublicGet, parsePa
 // there. scripts/gds/bump-version.js still rewrites the literal below; it appends
 // the entry to that file. Look for a version's history there, not here.
 // ---------------------------------------------------------------------------
-const CORE_VERSION = '1.19.455'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
+const CORE_VERSION = '1.19.457'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
 
 // A namespaced logger so a module's log lines are attributable + consistent.
 // Usage: const log = api.logger('dev-box'); log.info('mounted');

package/tests/goal_subtitle.mjs

@@ -184,20 +184,26 @@ test('an unset subtitle is hidden; a set one is shown trimmed', () => {
   assert.equal(els.sub.textContent, 'the grep path');
 });
 
-test('on a LIST the subtitle is a tooltip, never a second visible line', () => {
+test('on a LIST the subtitle is a visible line; on the CARD it stays a tooltip', () => {
   // task 1003206: the row/card painters moved to goals-render.js — the second half
   // of the carve goals-lib.js began — so read the family, not one filename.
   const js = goalsPageSource();
-  // The helper lives in goals-lib.js — the file that exists because
+  // Both helpers live in goals-lib.js — the file that exists because
   // goals-page.js was over the excess-lines ratchet (task 1003135).
   const lib = read('modules', 'hall-ui', 'public', 'goals-lib.js');
   assert.match(lib, /function subtitleAttr/);
-  assert.match(lib, /escapeHtml\(t\)/, 'escaped, since it lands in an attribute');
-  for (const anchor of ['class="gr-title"', 'class="goal-card__title gr-title"']) {
-    const i = js.indexOf(anchor);
-    assert.ok(i > 0, `${anchor} still rendered`);
-    assert.match(js.slice(i, i + 160), /subtitleAttr\(g\)/, `${anchor} carries the tooltip`);
-  }
+  assert.match(lib, /function subtitleLine/);
+  assert.match(lib, /escapeHtml\(t\)/, 'escaped, since it lands in an attribute or a text node');
+  // task 1003307 gave the ROW its line back on the owner's call: 35 of 72 goals
+  // carry a subtitle, and a tooltip does not exist on a touch screen. The CARD
+  // has no room for a line, so it keeps the tooltip (task 1003224's shape).
+  const row = js.indexOf('class="gr-title"');
+  assert.ok(row > 0, 'the row title is still rendered');
+  assert.match(js.slice(row, row + 200), /subtitleLine\(g\)/, 'the row renders the visible line');
+  assert.doesNotMatch(js.slice(row, row + 160), /subtitleAttr\(g\)/, 'the row does not also carry the tooltip');
+  const card = js.indexOf('class="goal-card__title gr-title"');
+  assert.ok(card > 0, 'the card title is still rendered');
+  assert.match(js.slice(card, card + 160), /subtitleAttr\(g\)/, 'the card carries the tooltip');
 });
 
 test('every CSS token the new styles use actually exists', () => {