@bongos/core 1.19.1045 → 1.19.1047

package/tests/people_directory_clip.mjs

@@ -1,269 +0,0 @@
-// tests/people_directory_clip.mjs — the People directory says so when it is
-// clipped (task 1003110, goal 1000095).
-//
-// WHY THIS FILE EXISTS. `people.js` asks `GET /scouting` for 500 rows, which is
-// that route's HARD CEILING (parsePagination maxLimit: 500). The rollup returns
-// no count of the whole set and the route takes no offset, so a response that
-// comes back FULL is the only truncation signal the page will ever get — and
-// until this task the page simply printed "500 builders listed" and said nothing
-// else. A directory that lies by omission is worse than one that admits it is
-// incomplete.
-//
-// It runs the REAL people.js in a vm against a stub DOM and a fake fetch, so
-// what is asserted is the sentence a reader actually sees, not a regex agreeing
-// with a regex. A source-text assertion would have passed just as happily with
-// the comparison written the wrong way round.
-//
-// The four behaviours:
-//   1. the page asks for exactly the number its copy then names (no drift)
-//   2. a SHORT response is not hedged — the exact count is the honest one
-//   3. a FULL response hedges every sentence that states a total: the pill, the
-//      subtitle, and a footnote about the lenses
-//   4. one row short of the ceiling is NOT clipped (the boundary, in the
-//      direction that would otherwise cry wolf on every load)
-//
-// Run:  node tests/people_directory_clip.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 read = (rel) => fs.readFileSync(path.join(ROOT, rel), 'utf8');
-const KIT_JS = read('modules/hall-ui/public/hall-kit.js');
-const PEOPLE_JS = read('modules/hall-ui/public/people.js');
-const PEOPLE_HTML = read('modules/hall-ui/public/people.html');
-
-// The route's ceiling, spelled here as the CONTRACT rather than read out of the
-// page — a test that imports the page's own constant cannot notice it changing.
-const CEILING = 500;
-
-const { test, summary } = makeRunner();
-const tick = () => new Promise((r) => setTimeout(r, 0));
-
-function makeEl(id) {
-  return {
-    id,
-    innerHTML: '', textContent: '', value: '',
-    hidden: false, disabled: false, className: '',
-    options: [], style: {}, dataset: {},
-    classList: { add() {}, remove() {}, toggle() {}, contains: () => false },
-    appendChild() {}, setAttribute() {}, removeAttribute() {},
-    addEventListener() {},
-    querySelector() { return makeEl(); },
-    querySelectorAll() { return []; },
-  };
-}
-
-// `count` builders, each distinct enough that a filter lens has something to bite.
-// The REAL role vocabulary, for the same reason this file runs the real kit rather than a
-// stub: a stub of roleLabel() would agree with a roleLabel() that was wrong. dom-utils.js
-// is pure enough to run against two small objects (tests/collab_page.mjs does the same).
-function realRoleLabel() {
-  const src = read('modules/hall-ui/public/dom-utils.js');
-  const win = { location: { search: '', hash: '', pathname: '/' }, addEventListener() {}, localStorage: null };
-  const doc = { addEventListener() {}, querySelector: () => null, querySelectorAll: () => [], documentElement: { dataset: {} } };
-  new Function('window', 'document', src)(win, doc);
-  return win.OTB.roleLabel;
-}
-
-function buildersFor(count) {
-  return Array.from({ length: count }, (_, i) => ({
-    github_id: 1000 + i,
-    github_login: `builder${i}`,
-    display_name: `Builder ${i}`,
-    avatar_url: null,
-    project_count: 1,
-    total_credits: count - i,
-    total_tasks_shipped: 1,
-    total_karma: 1,
-    last_active: '2026-09-01T00:00:00Z',
-    disciplines: ['engineer'],
-  }));
-}
-
-function harness({ count }) {
-  const els = new Map();
-  const getEl = (id) => {
-    if (!els.has(id)) els.set(id, makeEl(id));
-    return els.get(id);
-  };
-  const documentObj = {
-    documentElement: { dataset: {}, style: {}, classList: { add() {}, remove() {} } },
-    getElementById: getEl,
-    querySelector(sel) { return getEl(String(sel).replace(/^#/, '')); },
-    querySelectorAll() { return []; },
-    createElement() { return makeEl(); },
-    addEventListener() {},
-  };
-
-  const requests = [];
-  const store = new Map();
-  const windowObj = {
-    location: { assign() {}, href: '/people' },
-    localStorage: {
-      getItem: (k) => (store.has(k) ? store.get(k) : null),
-      setItem: (k, v) => store.set(k, String(v)),
-      removeItem: (k) => store.delete(k),
-    },
-    scrollTo() {},
-    addEventListener() {},
-    matchMedia: () => ({ matches: false, addEventListener() {}, addListener() {} }),
-    OTBBranding: { currencyLabel: () => 'Credits' },
-    OTB: {
-      // task 1003936 — the role axis vocabulary (dom-utils OTB.role*). PRESENT for
-      // the same reason categoryChip is: an undefined helper throws mid-render.
-      roleAxisLabel: () => 'Role',
-      roleOptions: () => [{ value: '', label: '— any —' }, { value: 'engineer', label: 'Engineer' }, { value: 'artist', label: 'Artist' }],
-      roleIsUnset: (d) => !d || d === 'unclassified',
-      ROLE_UNSET: 'unclassified',
-      escapeHtml: (s) => String(s == null ? '' : s),
-      $: (sel) => documentObj.querySelector(sel),
-      num: (n) => Number(n) || 0,
-      fmtInt: (n) => String(Number(n) || 0),
-      fmtCompact: (n) => String(Number(n) || 0),
-      fmtDate: (iso) => String(iso ?? '').slice(0, 10),
-      rankLabel: (r) => String(r ?? ''),
-      // The REAL one, replacing a hand-written reimplementation that used to sit here.
-      // Two reasons it had to go, and the second is the point of this file: it was a
-      // duplicate object key once the real helper arrived (no-dupe-keys, the later
-      // definition silently winning), and a stub that restates roleLabel's own logic
-      // can drift from it and the test would never notice — which is the argument this
-      // file already makes for running the real kit.
-      roleLabel: realRoleLabel(),
-      toast() {},
-    },
-  };
-
-  const sandbox = {
-    window: windowObj, document: documentObj, location: windowObj.location,
-    console: { error() {}, warn() {}, log() {} },
-    setTimeout, clearTimeout, URLSearchParams, encodeURIComponent, Date, Math, JSON,
-    fetch: (url) => {
-      requests.push(String(url));
-      return Promise.resolve({
-        ok: true,
-        status: 200,
-        json: () => Promise.resolve({ builders: buildersFor(count) }),
-      });
-    },
-  };
-  sandbox.globalThis = sandbox;
-  vm.createContext(sandbox);
-  // The real kit, for the same reason sessions_kit_pager.mjs runs it: the page is
-  // held to the shipped contract, not to a stub that agrees with it.
-  vm.runInContext(KIT_JS, sandbox, { filename: 'hall-kit.js' });
-  vm.runInContext(PEOPLE_JS, sandbox, { filename: 'people.js' });
-
-  return { getEl, requests };
-}
-
-// ---------------------------------------------------------------------------
-
-await test('the page markup carries the footnote slot, hidden until the page writes it', () => {
-  assert.match(PEOPLE_HTML, /id="people-foot"/, 'the clipped-directory footnote has a mount');
-  const tag = PEOPLE_HTML.match(/<p[^>]*id="people-foot"[^>]*>/);
-  assert.ok(tag, 'the footnote is a <p>');
-  assert.match(tag[0], /\bhidden\b/, 'it starts hidden — an empty note is still a visible gap');
-  assert.match(tag[0], /class="ledger__note"/, "the shell's ledger footnote class, not a bespoke one");
-});
-
-await test('the page asks for exactly the ceiling, and asks for it ONCE', async () => {
-  const h = harness({ count: 3 });
-  await tick(); await tick();
-  assert.equal(h.requests.length, 1, 'one directory read');
-  assert.match(h.requests[0], /\/api\/bongos\/scouting\?limit=/);
-  const limit = Number(new URLSearchParams(h.requests[0].split('?')[1]).get('limit'));
-  assert.equal(limit, CEILING, `the page must ask for the route ceiling (${CEILING})`);
-});
-
-// --- task 1004040 (artist review of 1003936): the Directory speaks the role --------
-
-await test('the role vocabulary a Directory chip renders through says the word, not the column', () => {
-  // The 1003936 grade named this as a nit — craftsHtml() rendering the raw discipline
-  // instead of routing it through roleLabel() — and a nit is advisory, so it survived two
-  // more rounds. What a reader saw: lowercase 'engineer', 'ui' where the house writes
-  // 'UI', and, for a builder who has set no role, the database's own 'unclassified' —
-  // the example V2 names, in the exact place it names it ("a filter option and a row tag").
-  //
-  // The BEHAVIOUR is executed: this is the shipped roleLabel, not a stub of it.
-  const roleLabel = realRoleLabel();
-  assert.equal(roleLabel('engineer'), 'Engineer');
-  assert.equal(roleLabel('ui'), 'UI', 'the acronym keeps its case');
-  assert.equal(roleLabel('unclassified'), 'No role yet', 'a sentence a person would say');
-
-  // The CALL SITE is read, because it cannot be executed here: the harness's document
-  // fake returns a fresh detached node from querySelector(), so the kit fills a <tbody>
-  // that never reaches the markup this file can see. Observing a rendered row would mean
-  // deepening that fake, which is a change to a shared harness and not an artist review's
-  // business. Named in the session log rather than left as a silent gap.
-  const src = read('modules/hall-ui/public/people.js');
-  assert.match(src, /esc\(roleLabel\(d\)\)/, 'the chip routes through the role vocabulary');
-  assert.doesNotMatch(src, /'<span class="fact-chip">' \+ esc\(String\(d\)\)/,
-    'the raw column must not be rendered as a chip');
-});
-
-await test('a SHORT directory is not hedged — the exact count is the honest one', async () => {
-  const h = harness({ count: 3 });
-  await tick(); await tick();
-  assert.equal(h.getEl('people-count').textContent, '3', 'the pill states the real count');
-  assert.equal(h.getEl('people-count').hidden, false);
-  assert.match(h.getEl('people-sub').textContent, /^3 builders listed/, 'the subtitle states the real count');
-  assert.doesNotMatch(h.getEl('people-sub').textContent, /first|more than/i,
-    'nothing is clipped, so nothing may suggest it is');
-  assert.equal(h.getEl('people-foot').hidden, true, 'no footnote on a complete directory');
-  assert.equal(h.getEl('people-foot').textContent, '', 'and no stale text left behind it');
-});
-
-await test('a FULL response hedges the pill, the subtitle AND the lenses', async () => {
-  const h = harness({ count: CEILING });
-  await tick(); await tick();
-
-  const pill = h.getEl('people-count').textContent;
-  assert.equal(pill, `${CEILING}+`, 'the pill may not state a total the page cannot know');
-
-  const sub = h.getEl('people-sub').textContent;
-  assert.match(sub, new RegExp(`first ${CEILING}\\b`), 'the subtitle says which slice this is');
-  assert.match(sub, /more than this page can load/, 'and says the directory is larger');
-  assert.doesNotMatch(sub, new RegExp(`^${CEILING} builders listed`),
-    'the old sentence claimed the slice was the whole set');
-
-  const foot = h.getEl('people-foot');
-  assert.equal(foot.hidden, false, 'the footnote is shown');
-  // THE LENSES ARE THE SHARPER LIE. Search/sort/craft re-order only the loaded
-  // set, so on a clipped directory a builder outside the top 500 cannot be found
-  // by any of them — a reader told only "the list is long" would still trust the
-  // search box.
-  assert.match(foot.textContent, /search/i, 'the footnote names the search');
-  assert.match(foot.textContent, /sort/i, 'and the sort lenses');
-  assert.match(foot.textContent, new RegExp(`will not appear`),
-    'and says plainly that a builder outside the slice is unreachable through them');
-});
-
-await test('one row short of the ceiling is NOT clipped', async () => {
-  const h = harness({ count: CEILING - 1 });
-  await tick(); await tick();
-  assert.equal(h.getEl('people-count').textContent, String(CEILING - 1),
-    'a nearly-full page is still a complete one');
-  assert.equal(h.getEl('people-foot').hidden, true, 'crying wolf here would train the notice away');
-});
-
-await test('every number in the clipped copy is the number that was requested', async () => {
-  const h = harness({ count: CEILING });
-  await tick(); await tick();
-  const limit = new URLSearchParams(h.requests[0].split('?')[1]).get('limit');
-  // The drift this guards: someone raises the request to 1000 and leaves "500"
-  // written in three sentences. Every integer the clipped copy prints must be
-  // the one the request carried.
-  const copy = [h.getEl('people-count').textContent, h.getEl('people-sub').textContent,
-    h.getEl('people-foot').textContent].join(' ');
-  const printed = copy.match(/\d+/g) || [];
-  assert.ok(printed.length >= 3, 'the clipped copy states the number more than once');
-  for (const n of printed) {
-    assert.equal(n, limit, `copy prints ${n} but the page requested ${limit}`);
-  }
-});
-
-summary();

package/tests/people_name_only.mjs

@@ -1,263 +0,0 @@
-// tests/people_name_only.mjs — the People directory lists a hidden builder by
-// name alone (task 1002291 / R15, ADR 0334; privacy spec D2's recruiter column:
-// "listed name-only (if not opted out)").
-//
-// GET /scouting now answers two tiers: `builders`, ranked on the cross-project
-// record, and `name_only`, the builders who hid that record but are still open
-// to recruiting. The server decides who is in each (tests/scouting_owner_access_-
-// and_reach.mjs); this file holds the PAGE to the rules that keep the second
-// tier honest on screen:
-//   - it is its own panel, revealed only when the server sends a row
-//   - a row is the identity cell and nothing else — no figure, no role, no date
-//   - it is never ranked: alphabetical whatever the sort lens says
-//   - search reads a name; a role filter never matches (the role is part of the
-//     record the builder hid)
-//   - the page still makes ONE read
-//
-// It runs the REAL people.js and the REAL kit in a vm, over a DOM fake one step
-// deeper than tests/people_directory_clip.mjs's: an element keeps the node it
-// hands out for a selector, so the <tbody> the kit fills is the one read back.
-//
-// Run:  node tests/people_name_only.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 read = (rel) => fs.readFileSync(path.join(ROOT, rel), 'utf8');
-const KIT_JS = read('modules/hall-ui/public/hall-kit.js');
-const PEOPLE_JS = read('modules/hall-ui/public/people.js');
-const PEOPLE_HTML = read('modules/hall-ui/public/people.html');
-
-const { test, summary } = makeRunner();
-const tick = () => new Promise((r) => setTimeout(r, 0));
-
-function makeEl(id) {
-  const kids = new Map();
-  const listeners = {};
-  return {
-    id,
-    innerHTML: '', textContent: '', value: '',
-    hidden: false, disabled: false, className: '',
-    options: [], style: {}, dataset: {},
-    classList: { add() {}, remove() {}, toggle() {}, contains: () => false },
-    appendChild() {}, setAttribute() {}, removeAttribute() {},
-    addEventListener(type, fn) { (listeners[type] ||= []).push(fn); },
-    fire(type) { (listeners[type] || []).forEach((fn) => fn({ target: { matches: () => true } })); },
-    querySelector(sel) {
-      if (!kids.has(sel)) kids.set(sel, makeEl(sel));
-      return kids.get(sel);
-    },
-    querySelectorAll() { return []; },
-  };
-}
-
-function realRoleLabel() {
-  const src = read('modules/hall-ui/public/dom-utils.js');
-  const win = { location: { search: '', hash: '', pathname: '/' }, addEventListener() {}, localStorage: null };
-  const doc = { addEventListener() {}, querySelector: () => null, querySelectorAll: () => [], documentElement: { dataset: {} } };
-  new Function('window', 'document', src)(win, doc);
-  return win.OTB.roleLabel;
-}
-
-// Ranked rows carry a record; the sentinels are what a name-only row must never show.
-const RANKED = [
-  { github_id: 1, github_login: 'zed-ranked', display_name: 'Zed', avatar_url: null, project_count: 2,
-    total_credits: 900, total_tasks_shipped: 9, total_karma: 4, last_active: '2026-09-01T00:00:00Z', disciplines: ['engineer'] },
-  { github_id: 2, github_login: 'amy-ranked', display_name: 'Amy', avatar_url: null, project_count: 1,
-    total_credits: 100, total_tasks_shipped: 1, total_karma: 1, last_active: '2026-09-02T00:00:00Z', disciplines: ['artist'] },
-];
-// Alphabetical by login, as the server sends them.
-const NAMED = [
-  { github_login: 'bea-hidden', display_name: 'Bea', avatar_url: null },
-  { github_login: 'cal-hidden', display_name: null, avatar_url: 'https://avatars.githubusercontent.com/u/7?v=4' },
-];
-
-function harness(payload) {
-  const els = new Map();
-  const getEl = (id) => {
-    if (!els.has(id)) els.set(id, makeEl(id));
-    return els.get(id);
-  };
-  const documentObj = {
-    documentElement: { dataset: {}, style: {}, classList: { add() {}, remove() {} } },
-    getElementById: getEl,
-    querySelector(sel) { return getEl(String(sel).replace(/^#/, '')); },
-    querySelectorAll() { return []; },
-    createElement() { return makeEl(); },
-    addEventListener() {},
-  };
-  const requests = [];
-  const store = new Map();
-  const windowObj = {
-    location: { assign() {}, href: '/people', search: '' },
-    localStorage: {
-      getItem: (k) => (store.has(k) ? store.get(k) : null),
-      setItem: (k, v) => store.set(k, String(v)),
-      removeItem: (k) => store.delete(k),
-    },
-    addEventListener() {},
-    OTBBranding: { currencyLabel: () => 'Credits' },
-    OTB: {
-      roleAxisLabel: () => 'Role',
-      roleLabel: realRoleLabel(),
-      fmtInt: (n) => String(Number(n) || 0),
-      rankLabel: (r) => String(r ?? ''),
-    },
-  };
-  const sandbox = {
-    window: windowObj, document: documentObj, location: windowObj.location,
-    console: { error() {}, warn() {}, log() {} },
-    CSS: { escape: (s) => String(s) },
-    setTimeout, clearTimeout, URLSearchParams, encodeURIComponent, Date, Math, JSON,
-    fetch: (url) => {
-      requests.push(String(url));
-      return Promise.resolve({ ok: true, status: 200, json: () => Promise.resolve(payload) });
-    },
-  };
-  sandbox.globalThis = sandbox;
-  vm.createContext(sandbox);
-  vm.runInContext(KIT_JS, sandbox, { filename: 'hall-kit.js' });
-  vm.runInContext(PEOPLE_JS, sandbox, { filename: 'people.js' });
-
-  const filters = () => getEl('people-filters');
-  return {
-    getEl,
-    requests,
-    rankedRows: () => getEl('people-body').querySelector('tbody').innerHTML,
-    namedRows: () => getEl('people-named-body').querySelector('tbody').innerHTML,
-    // Drive the kit's filter bar the way a reader does: set the control, fire.
-    setFilter(id, value) {
-      filters().querySelector(`#${id}`).value = value;
-      filters().fire(id === 'people-q' ? 'input' : 'change');
-    },
-  };
-}
-
-const loginsIn = (html) => [...html.matchAll(/class="dir-login">@([^<]+)</g)].map((m) => m[1]);
-
-// ---------------------------------------------------------------------------
-
-await test('the page carries the name-only panel, hidden until the server sends a row', () => {
-  const tag = PEOPLE_HTML.match(/<section[^>]*id="people-named-scroll"[^>]*>/);
-  assert.ok(tag, 'a panel of its own — a hidden builder is not a row in the ranked ledger');
-  assert.match(tag[0], /class="scroll"/, "the shell's panel, not a fourth card level");
-  assert.match(tag[0], /\bhidden\b/, 'it ships hidden: an instance with no such builder shows nothing');
-  assert.match(PEOPLE_HTML, /id="people-named-h"[^>]*>[^<]+<span class="count-pill" id="people-named-count" hidden>/,
-    'a heading with its own count');
-  const lede = (PEOPLE_HTML.match(/<p class="scroll__lede" id="people-named-lede">([^<]+)<\/p>/) || [])[1] || '';
-  assert.match(lede, /hidden/i, 'the lede says why these builders show no record, in the same word Settings uses (hide_stats)');
-  assert.doesNotMatch(lede, /private/i, '"private" is account_visibility\'s word, not hide_stats\'');
-  assert.match(lede, /ranked/i, 'and that they are never ranked');
-  assert.match(lede, /filter/i, 'and that a role filter never finds them');
-  assert.match(PEOPLE_HTML, /id="people-named-body"/, 'and a mount for the ledger');
-});
-
-await test('name-only rows render the identity cell and nothing else', async () => {
-  const h = harness({ builders: RANKED, name_only: NAMED });
-  await tick(); await tick();
-  assert.equal(h.getEl('people-named-scroll').hidden, false, 'revealed once a row arrives');
-  assert.equal(h.getEl('people-named-count').textContent, '2');
-  assert.equal(h.getEl('people-named-count').hidden, false);
-  const rows = h.namedRows();
-  assert.deepEqual(loginsIn(rows), ['bea-hidden', 'cal-hidden'], 'each builder, by login');
-  assert.match(rows, /<a class="lb-name dir-name" href="https:\/\/github\.com\/bea-hidden"/,
-    'the same door the ranked row has — the one page every listed builder has');
-  assert.match(rows, /<img class="dir-avatar" src="https:\/\/avatars\.githubusercontent\.com\/u\/7\?v=4"/, 'and the picture');
-  assert.match(rows, />cal-hidden</, 'a builder with no display name reads as their login');
-  assert.doesNotMatch(rows, /ledger__num|ledger__date|fact-chip/,
-    'no figure, no date and no role cell: every one of those is the record the builder hid');
-});
-
-await test('the ranked ledger is untouched by the second tier', async () => {
-  const h = harness({ builders: RANKED, name_only: NAMED });
-  await tick(); await tick();
-  assert.deepEqual(loginsIn(h.rankedRows()), ['zed-ranked', 'amy-ranked'],
-    'ranked in the server order, as before — and no hidden builder among them');
-  assert.equal(h.getEl('people-count').textContent, '2', 'its count is the ranked count');
-  assert.match(h.getEl('people-sub').textContent, /2 more .*by name only/,
-    'the subtitle counts both tiers, so it does not claim the ranked list is everyone');
-});
-
-await test('a sort lens never re-orders the name-only list — it is not ranked', async () => {
-  const h = harness({ builders: RANKED, name_only: [...NAMED].reverse() });
-  await tick(); await tick();
-  const before = loginsIn(h.namedRows());
-  for (const lens of ['ships', 'credits', 'karma', 'projects', 'recent']) {
-    h.setFilter('people-sort', lens);
-    assert.deepEqual(loginsIn(h.namedRows()), before, `the ${lens} lens has nothing to sort a hidden record by`);
-  }
-  assert.deepEqual(loginsIn(h.rankedRows()), ['amy-ranked', 'zed-ranked'],
-    'CONTROL: the ranked list DID re-order — the last lens, most recently active, puts Amy first');
-});
-
-await test('search reads a name; a role filter never matches a hidden builder', async () => {
-  const h = harness({ builders: RANKED, name_only: NAMED });
-  await tick(); await tick();
-  h.setFilter('people-q', 'bea');
-  assert.deepEqual(loginsIn(h.namedRows()), ['bea-hidden'], 'a hidden builder is findable by name');
-  assert.deepEqual(loginsIn(h.rankedRows()), [], 'and the ranked list filters the same search');
-  h.setFilter('people-q', '');
-  h.setFilter('people-craft', 'engineer');
-  assert.deepEqual(loginsIn(h.namedRows()), [], 'the role is part of the hidden record, so a role filter finds no one here');
-  assert.match(h.namedRows(), /ledger__empty/, 'and says so in the panel rather than leaving it blank');
-  assert.deepEqual(loginsIn(h.rankedRows()), ['zed-ranked'], 'CONTROL: the role filter does apply to the ranked list');
-});
-
-await test('no name-only rows — or an older server that sends no such key — leaves the panel hidden', async () => {
-  for (const payload of [{ builders: RANKED, name_only: [] }, { builders: RANKED }]) {
-    const h = harness(payload);
-    await tick(); await tick();
-    assert.equal(h.getEl('people-named-scroll').hidden, true, 'never revealed');
-    assert.deepEqual(loginsIn(h.rankedRows()), ['zed-ranked', 'amy-ranked'], 'and the directory renders as before');
-  }
-});
-
-await test('only hidden builders listed: the page renders them rather than the empty-directory notice', async () => {
-  const h = harness({ builders: [], name_only: NAMED });
-  await tick(); await tick();
-  assert.doesNotMatch(h.getEl('people-body').textContent, /Nobody is publicly listed/,
-    'the directory is not empty — two builders are listed by name');
-  assert.deepEqual(loginsIn(h.namedRows()), ['bea-hidden', 'cal-hidden']);
-  assert.match(h.rankedRows(), /ledger__empty/, 'the ranked ledger states its own empty case');
-  const sub = h.getEl('people-sub').textContent;
-  assert.match(sub, /^2 builders listed across hosted projects by name only\./,
-    'the subtitle counts who IS listed');
-  assert.doesNotMatch(sub, /\b0 builders|more by name only/, 'not "0 builders … 2 more" — a count of nothing');
-});
-
-await test('a FULL name-only response is hedged, and says a search cannot reach past it', async () => {
-  const CEILING = 500;
-  const full = Array.from({ length: CEILING }, (_, i) => ({ github_login: `h${String(i).padStart(3, '0')}`, display_name: null, avatar_url: null }));
-  let h = harness({ builders: RANKED, name_only: full });
-  await tick(); await tick();
-  assert.equal(h.getEl('people-named-count').textContent, `${CEILING}+`, 'the pill may not state a total the page cannot know');
-  const foot = h.getEl('people-named-foot');
-  assert.equal(foot.hidden, false, 'the footnote shows on a clipped list');
-  assert.match(foot.textContent, /clipped at the first 500 by login/);
-  assert.match(foot.textContent, /Search only reads what this page loaded/, 'naming the one lens that reads this list');
-  assert.match(PEOPLE_HTML, /<p class="ledger__note" id="people-named-foot" hidden><\/p>/, 'the slot ships hidden');
-  h = harness({ builders: RANKED, name_only: full.slice(0, CEILING - 1) });
-  await tick(); await tick();
-  assert.equal(h.getEl('people-named-foot').hidden, true, 'one short of the ceiling is a complete list');
-  assert.equal(h.getEl('people-named-count').textContent, String(CEILING - 1));
-});
-
-await test('both tiers empty: the consent-boundary notice still explains the empty page', async () => {
-  const h = harness({ builders: [], name_only: [] });
-  await tick(); await tick();
-  assert.match(h.getEl('people-body').textContent, /Nobody is publicly listed/);
-  assert.equal(h.getEl('people-named-scroll').hidden, true, 'and the second panel stays hidden');
-});
-
-await test('the page still makes ONE read for both tiers', async () => {
-  const h = harness({ builders: RANKED, name_only: NAMED });
-  await tick(); await tick();
-  assert.equal(h.requests.length, 1);
-  assert.match(h.requests[0], /^\/api\/bongos\/scouting\?limit=/);
-});
-
-summary();