@bongos/core 1.19.280 → 1.19.281

package/modules/hall-ui/public/goals-lib.js

@@ -0,0 +1,99 @@
+// goals-lib.js — the Goals page's PURE presentational helpers, exposed as
+// window.OTBGoalsLib. Carved out of the oversized goals-page.js (task 1003135;
+// the fitness excess-lines ratchet). Everything here is a pure function over
+// its arguments (or a constant) — no `state`, no DOM reads — so it can move
+// without dragging the page's IIFE state along. Loaded by goals.html BEFORE
+// goals-page.js (the task-lib.js / hall-kit.js pattern, tasks 1002463 / 2171).
+(function () {
+  'use strict';
+
+  // Settled task statuses (paid or awaiting land) vs terminal "deliberately
+  // cut" states — neither work-to-do nor done. CUT is excluded from the todo
+  // list and the conflict scan the same way migration 017 keeps them out of
+  // version_progress ("abandoned is a terminal state (deliberately cut), not
+  // pending work"). Without this, an abandoned task shows up under a goal's
+  // "Work to do" as if it were still queued.
+  const DONE_STATUSES = new Set(['completed', 'confirmed', 'shipped']);
+  const CUT_STATUSES = new Set(['abandoned', 'dropped']);
+  // Genuinely-open work under a goal: not done, not cut → {backlog, ready,
+  // active, blocked}.
+  const isTodoStatus = (s) => !DONE_STATUSES.has(s) && !CUT_STATUSES.has(s);
+
+  // The criteria progress ring (satisfied/total), green once achieved.
+  function ring(satisfied, total, achieved, size = 42) {
+    const pct = total > 0 ? satisfied / total : (achieved ? 1 : 0);
+    const r = 16;
+    const circ = 2 * Math.PI * r;
+    const dash = (pct * circ).toFixed(1);
+    const gap = (circ - pct * circ).toFixed(1);
+    const stroke = achieved ? 'var(--ok)' : 'var(--accent)';
+    return `<svg class="goal-ring" width="${size}" height="${size}" viewBox="0 0 42 42" role="img" aria-label="${satisfied} of ${total} criteria satisfied">
+        <circle cx="21" cy="21" r="${r}" fill="none" stroke="var(--rule-soft)" stroke-width="4"></circle>
+        <circle cx="21" cy="21" r="${r}" fill="none" stroke="${stroke}" stroke-width="4" stroke-linecap="round" stroke-dasharray="${dash} ${gap}" transform="rotate(-90 21 21)"></circle>
+        <text x="21" y="25" text-anchor="middle" class="goal-ring__label">${satisfied}/${total}</text>
+      </svg>`;
+  }
+
+  function memberName(m) {
+    return m.display_name || (m.github_login ? `@${m.github_login}` : `builder #${m.builder_id}`);
+  }
+
+  // ---- task-readiness rollup (hall redesign task 1002439) -------------------
+  // Dependencies are a TASK property; a goal is only "blocked" in the derived
+  // sense that all its actionable tasks wait on upstream tasks (the Tasks page,
+  // not this one, owns the actual task→task graph). So the Goals page rolls each
+  // goal's tasks into these buckets:
+  //   done     — settled (shipped/confirmed/completed)
+  //   active   — a live claim ("in progress")
+  //   ready    — claimable now (deps met + promoted)
+  //   blocked  — backlog with ≥1 unshipped dependency
+  //   backlog  — backlog, no unmet deps (just not started/promoted)
+  function readinessOf(t) {
+    const s = t.status;
+    if (DONE_STATUSES.has(s)) return 'done';
+    if (CUT_STATUSES.has(s)) return 'cut';
+    if (s === 'active') return 'active';
+    if (s === 'ready') return 'ready';
+    if (s === 'backlog' && (t.blocked_by || []).some((d) => !d.satisfied)) return 'blocked';
+    return 'backlog';
+  }
+  const EMPTY_ROLLUP = { done: 0, active: 0, ready: 0, blocked: 0, backlog: 0, cut: 0, total: 0, nextReady: null };
+
+  // task 1002439 — a goal row's readiness rollup line (only the non-zero buckets).
+  const ROLL_DOTS = [
+    ['ready', 'ready', 'var(--ok)'],
+    ['active', 'in progress', 'var(--info)'],
+    ['blocked', 'blocked', 'var(--terracotta)'],
+    ['backlog', 'backlog', 'var(--ink-faint)'],
+  ];
+  function goalRollLine(r) {
+    const parts = ROLL_DOTS.filter(([k]) => r[k] > 0)
+      .map(([k, label, c]) => `<span class="gr-d"><i style="background:${c}"></i><b>${r[k]}</b> ${label}</span>`);
+    return parts.length ? `<div class="gr-roll">${parts.join('')}</div>` : '';
+  }
+
+  // Free-text search over a goal's todo list (work.js:261-274, minus the
+  // goal-title match — a single goal makes that redundant). '#N', 'N' and a
+  // bare number all match the id.
+  function matchesTodoSearch(t, q) {
+    if (!q) return true;
+    const needle = q.replace(/^#/, '');
+    if (!needle) return true;
+    if (String(t.id).includes(needle)) return true;
+    if ((t.title || '').toLowerCase().includes(needle)) return true;
+    return false;
+  }
+
+  // ms helpers for the goal ordering (task 1002965): updated_at moves on every
+  // task write, shipped_at folded in because a ship path can stamp it without
+  // touching updated_at.
+  const msOf = (iso) => { const t = iso ? Date.parse(iso) : NaN; return Number.isNaN(t) ? 0 : t; };
+  const taskActivityMs = (t) => Math.max(msOf(t.updated_at), msOf(t.shipped_at));
+
+  window.OTBGoalsLib = {
+    DONE_STATUSES, CUT_STATUSES, isTodoStatus,
+    ring, memberName,
+    readinessOf, EMPTY_ROLLUP, goalRollLine,
+    matchesTodoSearch, msOf, taskActivityMs,
+  };
+})();

package/modules/hall-ui/public/goals-page.js

@@ -35,13 +35,18 @@
   // empty/skeleton states — adopted here by task 2171.
   const { makePager, paginate, wirePager, pagerBarHtml, makeTabs, makeFilterBar,
     emptyStateHtml, skeletonRows } = window.OTBKit;
+  // This page's pure presentational helpers (goals-lib.js, task 1003135 — carved
+  // out of this oversized file; the fitness excess-lines ratchet). Destructured
+  // so every call site below reads exactly as before.
+  const { DONE_STATUSES, CUT_STATUSES, isTodoStatus, ring, memberName,
+    readinessOf, EMPTY_ROLLUP, goalRollLine, matchesTodoSearch,
+    msOf, taskActivityMs } = window.OTBGoalsLib;
   const API = '/api/bongos';
   // Generated typed client (task 1996 / R09), served as window.BongosClient by
   // /lib/bongos-client.js. baseUrl:'' passes the full same-origin paths this file
   // builds through unchanged; result-mode ({ status, ok, data }) preserves control flow 1:1.
   const api = window.BongosClient.createClient({ baseUrl: '', credentials: 'same-origin', throwOnError: false });
 
-  const DONE_STATUSES = new Set(['completed', 'confirmed', 'shipped']);
   // task 1888 — the Work tables and the conflicts list render capped; the pager
   // under each expands to the full set.
   const CONFLICT_CAP = 3;
@@ -59,15 +64,6 @@
     todo: makePager('goal-todo', 'otb.goals.todoPerPage'),
     done: makePager('goal-done', 'otb.goals.donePerPage'),
   };
-  // Terminal "deliberately cut" states — neither work-to-do nor done. Excluded
-  // from the todo list and the conflict scan the same way migration 017 keeps
-  // them out of version_progress ("abandoned is a terminal state (deliberately
-  // cut), not pending work"). Without this, an abandoned task shows up under a
-  // goal's "Work to do" as if it were still queued.
-  const CUT_STATUSES = new Set(['abandoned', 'dropped']);
-  // Genuinely-open work under a goal: not done, not cut → {backlog, ready,
-  // active, blocked}.
-  const isTodoStatus = (s) => !DONE_STATUSES.has(s) && !CUT_STATUSES.has(s);
   // Record deep-links (#/task|idea|blocker/N) are owned by the shared overlay
   // (task-detail.js). This page must NOT treat them as a route change (Bug 2).
   const DEEPLINK_RE = /^#\/(?:task|idea|blocker)\/\d+$/;
@@ -148,44 +144,7 @@
     return `<span class="goal-badge ${cls}">${escapeHtml(status)}</span>`;
   }
 
-  function ring(satisfied, total, achieved, size = 42) {
-    const pct = total > 0 ? satisfied / total : (achieved ? 1 : 0);
-    const r = 16;
-    const circ = 2 * Math.PI * r;
-    const dash = (pct * circ).toFixed(1);
-    const gap = (circ - pct * circ).toFixed(1);
-    const stroke = achieved ? 'var(--ok)' : 'var(--accent)';
-    return `<svg class="goal-ring" width="${size}" height="${size}" viewBox="0 0 42 42" role="img" aria-label="${satisfied} of ${total} criteria satisfied">
-        <circle cx="21" cy="21" r="${r}" fill="none" stroke="var(--rule-soft)" stroke-width="4"></circle>
-        <circle cx="21" cy="21" r="${r}" fill="none" stroke="${stroke}" stroke-width="4" stroke-linecap="round" stroke-dasharray="${dash} ${gap}" transform="rotate(-90 21 21)"></circle>
-        <text x="21" y="25" text-anchor="middle" class="goal-ring__label">${satisfied}/${total}</text>
-      </svg>`;
-  }
-
-  function memberName(m) {
-    return m.display_name || (m.github_login ? `@${m.github_login}` : `builder #${m.builder_id}`);
-  }
-
-  // ---- task-readiness rollup (hall redesign task 1002439) --------------------
-  // Dependencies are a TASK property; a goal is only "blocked" in the derived
-  // sense that all its actionable tasks wait on upstream tasks (the Tasks page,
-  // not this one, owns the actual task→task graph). So the Goals page rolls each
-  // goal's tasks into these buckets:
-  //   done     — settled (shipped/confirmed/completed)
-  //   active   — a live claim ("in progress")
-  //   ready    — claimable now (deps met + promoted)
-  //   blocked  — backlog with ≥1 unshipped dependency
-  //   backlog  — backlog, no unmet deps (just not started/promoted)
-  function readinessOf(t) {
-    const s = t.status;
-    if (DONE_STATUSES.has(s)) return 'done';
-    if (CUT_STATUSES.has(s)) return 'cut';
-    if (s === 'active') return 'active';
-    if (s === 'ready') return 'ready';
-    if (s === 'backlog' && (t.blocked_by || []).some((d) => !d.satisfied)) return 'blocked';
-    return 'backlog';
-  }
-  const EMPTY_ROLLUP = { done: 0, active: 0, ready: 0, blocked: 0, backlog: 0, cut: 0, total: 0, nextReady: null };
+  // ring / memberName / readinessOf / EMPTY_ROLLUP → goals-lib.js (task 1003135).
   const rollupOf = (goalId) => state.rollups.get(String(goalId)) || EMPTY_ROLLUP;
 
   // ---- goal ordering (task 1002965) -----------------------------------------
@@ -197,8 +156,6 @@
   //              falling back to the goal row's own updated_at when the goal
   //              has no tasks at all, so a task-less goal still orders sanely
   //              instead of sinking to the bottom forever.
-  const msOf = (iso) => { const t = iso ? Date.parse(iso) : NaN; return Number.isNaN(t) ? 0 : t; };
-  const taskActivityMs = (t) => Math.max(msOf(t.updated_at), msOf(t.shipped_at));
   const goalCreatedMs = (g) => msOf(g.created_at);
   const goalActivityMs = (g) => state.activity.get(String(g.id)) || msOf(g.updated_at);
 
@@ -271,6 +228,40 @@
     }
   }
 
+  // task 1003135 — the Owner dropdown. 'Goal owner' is goals.created_by (the
+  // fixed authority, migration 160), returned by GET /goals as a raw builder id;
+  // labels come from the directory the page already lazily caches (loadDirectory,
+  // task 1877), same label idiom as the invite dropdown. created_by is NULLABLE
+  // (system-attributed — e.g. the migration-160 '<version> — general' catch-alls),
+  // hence the explicit 'system / unattributed' option. Mirrors
+  // populateVersionFilter above; fired after boot's fetch resolves (async because
+  // of the directory fetch — fire-and-forget, the select fills when ready).
+  async function populateOwnerFilter() {
+    const sel = $('#f-goal-owner');
+    if (!sel) return;
+    const ownerIds = Array.from(new Set(
+      state.goals.filter((g) => g.created_by != null).map((g) => String(g.created_by))));
+    const hasSystem = state.goals.some((g) => g.created_by == null);
+    const dir = ownerIds.length ? await loadDirectory() : [];
+    const byId = new Map((dir || []).map((b) => [String(b.id), b]));
+    const label = (id) => {
+      const b = byId.get(id);
+      return b ? (b.display_name || '@' + b.github_login) : `builder #${id}`;
+    };
+    for (const id of ownerIds.sort((a, b) => label(a).localeCompare(label(b)))) {
+      const opt = document.createElement('option');
+      opt.value = id;
+      opt.textContent = label(id);
+      sel.appendChild(opt);
+    }
+    if (hasSystem) {
+      const opt = document.createElement('option');
+      opt.value = 'none';
+      opt.textContent = 'system / unattributed';
+      sel.appendChild(opt);
+    }
+  }
+
   // task 1760 — priority is a WITHIN-version ordering, so the drag affordance
   // only makes sense once the grid is narrowed to exactly one version (the
   // "any" filter mixes versions and dragging across them would be meaningless).
@@ -292,6 +283,18 @@
     const ver = $('#f-goal-version')?.value || '';
     const status = $('#f-goal-status')?.value || '';
     const mineOnly = $('#f-goal-mine')?.value === 'mine';
+    // task 1003135 — free-text search (goal title OR id; '#N', 'N' and a bare
+    // number all match — the id is already rendered on the row, task 1002163)
+    // and the Owner dropdown ('none' = created_by NULL, system-attributed).
+    // Reads the DOM ids directly, matching this function's existing style.
+    const q = ($('#f-goals-search')?.value || '').trim().toLowerCase();
+    const needle = q.replace(/^#/, '');
+    const owner = $('#f-goal-owner')?.value || '';
+    const matchesSearch = (g) => !needle
+      || String(g.id).includes(needle)
+      || (g.title || '').toLowerCase().includes(needle);
+    const matchesOwner = (g) => !owner
+      || (owner === 'none' ? g.created_by == null : String(g.created_by) === owner);
     const myId = selfId();
     const iAmMember = (g) => {
       const d = state.details.get(String(g.id));
@@ -300,22 +303,11 @@
     return state.goals.filter((g) =>
       (!ver || g.version_id === ver) &&
       (!status || g.status === status) &&
+      matchesSearch(g) &&
+      matchesOwner(g) &&
       (!mineOnly || iAmMember(g)));
   }
 
-  // task 1002439 — a goal row's readiness rollup line (only the non-zero buckets).
-  const ROLL_DOTS = [
-    ['ready', 'ready', 'var(--ok)'],
-    ['active', 'in progress', 'var(--info)'],
-    ['blocked', 'blocked', 'var(--terracotta)'],
-    ['backlog', 'backlog', 'var(--ink-faint)'],
-  ];
-  function goalRollLine(r) {
-    const parts = ROLL_DOTS.filter(([k]) => r[k] > 0)
-      .map(([k, label, c]) => `<span class="gr-d"><i style="background:${c}"></i><b>${r[k]}</b> ${label}</span>`);
-    return parts.length ? `<div class="gr-roll">${parts.join('')}</div>` : '';
-  }
-
   // task 1002439 — one goal, rendered as a rollup row: criteria ring · title +
   // meta + readiness rollup · one next action (claim the next ready task, drill
   // into what's blocking, or a review/achieved note).
@@ -515,15 +507,6 @@
     const el = $('#f-goal-search');
     return el ? el.value.trim().toLowerCase() : '';
   }
-  function matchesTodoSearch(t, q) {
-    if (!q) return true;
-    const needle = q.replace(/^#/, '');
-    if (!needle) return true;
-    if (String(t.id).includes(needle)) return true;
-    if ((t.title || '').toLowerCase().includes(needle)) return true;
-    return false;
-  }
-
   // Copy-prompt (work.js:454-503). deHashRefs stops a bare #NNN autolinking to
   // GitHub in a pasted session; the prompt points the builder at the hall task URL.
   async function copyPromptForTask(taskId, btn) {
@@ -1810,6 +1793,9 @@
       const gres = await getJSON(`${API}/goals`);
       state.goals = gres.goals || [];
       populateVersionFilter();
+      // task 1003135 — fire-and-forget: the Owner options fill when the (cached)
+      // directory resolves; boot never waits on a label lookup.
+      populateOwnerFilter().catch(() => {});
       // Prefetch details (ring + members on the cards). Sequentially-parallel,
       // failure-tolerant: a card without detail still renders.
       // task 1002439 — also fold in per-goal task-readiness rollups (one request
@@ -1848,12 +1834,21 @@
   // taken by the goal-detail To-do bar further down this file).
   let savedGoalSort = null;
   try { savedGoalSort = window.localStorage.getItem(GOAL_SORT_PREF); } catch (_) { /* private mode */ }
+  // task 1003135 — Search declared on the kit spec (live 'input' listener comes
+  // free, hall-kit.js:273; no debounce needed, the match is pure client-side over
+  // the already-loaded state.goals). Id is f-goals-search — f-goal-search is
+  // already taken by the goal-detail To-do bar further down this file. Owner
+  // options are filled after boot's fetch (populateOwnerFilter), the
+  // populateVersionFilter pattern. No URL riding (url defaults off here and this
+  // page is deliberately url:false — it owns the #/goal/N hash route).
   makeFilterBar({
     mount: $('#goals-filters'),
+    search: { id: 'f-goals-search', label: 'Search', placeholder: 'goal # or name…' },
     primary: [
       { id: 'f-goal-version', label: 'Version', options: [{ value: '', label: '— any —' }] },
       { id: 'f-goal-status', label: 'Status', options: [{ value: '', label: '— any —' }, 'open', 'achieved', 'archived'] },
       { id: 'f-goal-mine', label: 'Membership', options: [{ value: '', label: 'All goals' }, { value: 'mine', label: 'My goals' }] },
+      { id: 'f-goal-owner', label: 'Owner', options: [{ value: '', label: '— any —' }] },
       { id: 'f-goal-order',
         label: 'Sort',
         value: GOAL_SORTS[savedGoalSort] ? savedGoalSort : undefined,

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

@@ -198,6 +198,7 @@
 <script src="/builders/task-lib.js"></script>
 <script src="/builders/task-detail.js?v=2026-07-03-taskdetail"></script>
 <script src="/builders/brand-apply.js?v=2026-06-27-r75"></script>
-<script src="/builders/goals-page.js?v=2026-07-10-goals5"></script>
+<script src="/builders/goals-lib.js?v=2026-08-21-goalslib1"></script>
+<script src="/builders/goals-page.js?v=2026-08-21-goals6"></script>
 </body>
 </html>

package/package-lock.json

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bongos/core",
-  "version": "1.19.280",
+  "version": "1.19.281",
   "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

@@ -987,8 +987,10 @@ const { validateOrRespond, LIMITS, parseId, asyncHandler, corsPublicGet, parsePa
 //         landed since 1.19.278 with no explicit bump. run 32521664476. (task 1002620)
 //   1.19.280 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
 //         landed since 1.19.279 with no explicit bump. run 32523185119. (task 1002620)
+//   1.19.281 — CI auto-patch (publish-on-merge, ADR 0161): carrier for merges
+//         landed since 1.19.280 with no explicit bump. run 32525071763. (task 1002620)
 // ---------------------------------------------------------------------------
-const CORE_VERSION = '1.19.280'; // CI auto-patch carrier (ADR 0161); see changelog above
+const CORE_VERSION = '1.19.281'; // CI auto-patch carrier (ADR 0161); see changelog above
 
 // 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_criterion_confirm_ui.mjs

@@ -37,6 +37,9 @@ const SRC = fs.readFileSync(path.join(ROOT, 'modules/hall-ui/public/goals-page.j
 // the REAL file in the same context rather than stubbing it — stubbing would let
 // the extraction rot without this, the only test that executes either board.
 const TASK_LIB_SRC = fs.readFileSync(path.join(ROOT, 'modules/hall-ui/public/task-lib.js'), 'utf8');
+// task 1003135: the page's pure presentational helpers moved to goals-lib.js
+// (window.OTBGoalsLib) — run the REAL file too, same reasoning as task-lib above.
+const GOALS_LIB_SRC = fs.readFileSync(path.join(ROOT, 'modules/hall-ui/public/goals-lib.js'), 'utf8');
 
 const { test, summary } = makeRunner();
 
@@ -132,6 +135,7 @@ async function render({ rank = 'metic', goalStatus = 'open', criteria = Object.v
   sandbox.globalThis = sandbox;
   vm.createContext(sandbox);
   vm.runInContext(TASK_LIB_SRC, sandbox, { filename: 'task-lib.js' });
+  vm.runInContext(GOALS_LIB_SRC, sandbox, { filename: 'goals-lib.js' });
   vm.runInContext(SRC, sandbox, { filename: 'goals-page.js' });
 
   // Let boot()'s promise chain settle (me → goals → details → route → render).

package/tests/goals_lib.mjs

@@ -0,0 +1,70 @@
+// tests/goals_lib.mjs — the Goals page's extracted pure helpers (goals-lib.js,
+// task 1003135). Pins (a) that the lib loads standalone in a bare window — it
+// must not silently grow a dependency on the page's IIFE state, which is the
+// whole reason it could move out of the oversized goals-page.js — and (b) the
+// behaviour of the helpers the page destructures.
+//
+// Run:  node tests/goals_lib.mjs
+
+import { strict as assert } from 'node:assert';
+import fs from 'node:fs';
+import path from 'node:path';
+import vm from 'node:vm';
+import { fileURLToPath } from 'node:url';
+import { makeRunner } from './helpers.mjs';
+
+const ROOT = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '..');
+const SRC = fs.readFileSync(path.join(ROOT, 'modules/hall-ui/public/goals-lib.js'), 'utf8');
+const { test, summary } = makeRunner();
+
+const windowObj = {};
+const sandbox = { window: windowObj, Set, Map, Math, Number, String, Array, Object, Date, JSON };
+sandbox.globalThis = sandbox;
+vm.createContext(sandbox);
+vm.runInContext(SRC, sandbox, { filename: 'goals-lib.js' });
+const lib = windowObj.OTBGoalsLib;
+
+await test('loads standalone in a bare window (no page-state dependency) and exports the full surface', () => {
+  assert.ok(lib, 'window.OTBGoalsLib must be set');
+  for (const k of ['DONE_STATUSES', 'CUT_STATUSES', 'isTodoStatus', 'ring', 'memberName',
+    'readinessOf', 'EMPTY_ROLLUP', 'goalRollLine', 'matchesTodoSearch', 'msOf', 'taskActivityMs']) {
+    assert.ok(k in lib, `missing export: ${k}`);
+  }
+});
+
+await test('readinessOf buckets match the task-1002439 rules', () => {
+  assert.equal(lib.readinessOf({ status: 'shipped' }), 'done');
+  assert.equal(lib.readinessOf({ status: 'abandoned' }), 'cut');
+  assert.equal(lib.readinessOf({ status: 'active' }), 'active');
+  assert.equal(lib.readinessOf({ status: 'ready' }), 'ready');
+  assert.equal(lib.readinessOf({ status: 'backlog', blocked_by: [{ satisfied: false }] }), 'blocked');
+  assert.equal(lib.readinessOf({ status: 'backlog' }), 'backlog');
+});
+
+await test('matchesTodoSearch: bare number, #N and title fragments all match (task 1003135 search semantics)', () => {
+  const t = { id: 1002163, title: 'Show the goal id on the row' };
+  assert.ok(lib.matchesTodoSearch(t, ''));
+  assert.ok(lib.matchesTodoSearch(t, '1002163'));
+  assert.ok(lib.matchesTodoSearch(t, '#1002163'));
+  assert.ok(lib.matchesTodoSearch(t, '2163'), 'substring id match');
+  assert.ok(lib.matchesTodoSearch(t, 'goal id'));
+  assert.ok(!lib.matchesTodoSearch(t, 'zzz'));
+  assert.ok(lib.matchesTodoSearch(t, '#'), 'a lone # narrows nothing');
+});
+
+await test('goalRollLine renders only non-zero buckets; ring is well-formed SVG', () => {
+  const line = lib.goalRollLine({ ready: 2, active: 0, blocked: 1, backlog: 0 });
+  assert.match(line, /<b>2<\/b> ready/);
+  assert.match(line, /<b>1<\/b> blocked/);
+  assert.ok(!line.includes('in progress'), 'zero buckets are omitted');
+  assert.equal(lib.goalRollLine({ ready: 0, active: 0, blocked: 0, backlog: 0 }), '');
+  assert.match(lib.ring(2, 5, false), /aria-label="2 of 5 criteria satisfied"/);
+});
+
+await test('memberName label idiom: display name, then @login, then builder #id', () => {
+  assert.equal(lib.memberName({ display_name: 'A', github_login: 'a', builder_id: 1 }), 'A');
+  assert.equal(lib.memberName({ github_login: 'a', builder_id: 1 }), '@a');
+  assert.equal(lib.memberName({ builder_id: 1 }), 'builder #1');
+});
+
+summary();