@bongos/core 1.19.781 → 1.19.783

package/modules/hall-ui/public/collab.js

@@ -1,19 +1,23 @@
 // modules/hall-ui/public/collab.js — the Collab page (/builders/collab).
 // Task 1003172 / C02 of goal 1000075, criterion collab-page-matches-my-craft;
-// rebuilt on the hall's row archetype by task 1003471 (goal 1000081).
+// rebuilt on the hall's row archetype by task 1003471 (goal 1000081); made a
+// THREE-BLOCK room of people by task 1003942 (owner direction: "way too text
+// heavy, doesn't feel collaborative at all").
 //
-// The hall surface for C01's help requests. Two panels, in this order on
-// purpose: what is ASKED OF YOU first, because someone is waiting on a human
-// answer, then YOUR OWN ASKS, which are a status check rather than a call to act.
-// "Asked of you" reads in runs — by name first, then each craft — because a
-// personal ask is the one you owe an answer.
+// The hall surface for C01's help requests, in three blocks. Top left, ASKED OF
+// YOU: someone is waiting on a human answer, so it reads first. Top right, YOUR
+// ASKS: a status check, with the two settle actions. Below, wide, ALL OPEN ASKS:
+// everything anyone in the project has asked, which is what makes the page a
+// room rather than an inbox. Each block shows its NEWEST THREE and a "show all"
+// door to the rest; each head carries its count and the faces of who is asking.
 //
-// A row is the family's .ov-row: the ask's first sentence as the title and the
-// rest as the sub (collab-lib titleOf), WHO IS ASKING as a fact with a door to
-// their profile, the task or goal as a chip, and the AGE as a state pill in the
-// right column, so a glance down the list reads the ages in words and in the
-// verification colours. The band rules live in collab-lib.js where a test can
-// execute them.
+// A row is the family's .ov-row--lead: the person's FACE first (collab-lib
+// faceOf: the asker on what was asked of you and on the board, the addressee on
+// your own asks), the ask's first sentence as the title and the rest as the sub
+// (titleOf), the direction as a fact with a door to their profile, the task or
+// goal as a chip, and the AGE as a state pill in the right column, so a glance
+// down a list reads the ages in words and in the verification colours. Every
+// rule that can be pure lives in collab-lib.js where a test can execute it.
 //
 // MATCHING IS THE SERVER'S JOB, not this file's. GET /help-requests/for-me derives
 // the viewer's crafts from their own profile; sending a craft from here would be
@@ -33,6 +37,10 @@
   const { escapeHtml, $, toast } = window.OTB;
   const L = window.OTBCollabLib;
 
+  // The condensed view: the newest three, the rest behind "show all".
+  const SHOWN = 3;
+  const MAX_FACES = 6;
+
   let settling = false;
 
   // The age pill: the band's words, in the band's colour (collab.css .cb-age--*).
@@ -59,6 +67,27 @@
     return `<a class="fact-chip" href="${escapeHtml(ctx.href)}">${escapeHtml(ctx.kind)} #${escapeHtml(String(ctx.id))}</a>`;
   }
 
+  // The face: the picture when the lib vouched for a src, else the initial on
+  // the glass; a craft wears its letter in the label face. Decorative on
+  // purpose (alt="", aria-hidden) — the name in the facts line is the door, and
+  // two doors to one profile on one row would be noise for a screen reader.
+  function faceHtml(face) {
+    if (!face) return '';
+    if (face.avatar) {
+      return `<img class="cb-face" src="${escapeHtml(face.avatar)}" alt="" loading="lazy" title="${escapeHtml(face.name)}">`;
+    }
+    const kind = face.kind === 'craft' ? ' cb-face--craft' : '';
+    return `<span class="cb-face cb-face--initial${kind}" aria-hidden="true" title="${escapeHtml(face.name)}">${escapeHtml(face.initial)}</span>`;
+  }
+
+  // The stack in a block's head: each distinct asker once, capped, then +N.
+  function facesHtml(list) {
+    if (!list.length) return '';
+    const shown = list.slice(0, MAX_FACES).map(faceHtml).join('');
+    const more = list.length > MAX_FACES ? `<span class="cb-face cb-face--initial" aria-hidden="true">+${list.length - MAX_FACES}</span>` : '';
+    return shown + more;
+  }
+
   // A name as a door to that builder's profile, or plain text when the server
   // sent no id to link to.
   function personLink(id, name) {
@@ -68,6 +97,18 @@
       : escapeHtml(name);
   }
 
+  // Splice a profile link over a person's name inside an escaped sentence. The
+  // replacement is a FUNCTION, not a string: String.prototype.replace treats
+  // `$&`, `$$`, "$`" and `$'` specially in a string replacement, and the name is
+  // builder-controlled (a display_name or login) — so a display name containing
+  // `$&` would duplicate or mangle the sentence around it. Not an injection
+  // (both halves are escaped either way), but a real way to corrupt someone
+  // else's row by choosing your own name.
+  function linkName(escapedLine, id, name) {
+    const link = personLink(id, name);
+    return (name && link) ? escapedLine.replace(escapeHtml(name), () => link) : escapedLine;
+  }
+
   // DIRECTION, on the row (owner report, task 1003274): "<name> asked you by
   // name" incoming, "You asked <name> by name" outgoing — so an outgoing ask can
   // never be mistaken for an incoming one, which is exactly the mistake the
@@ -79,26 +120,32 @@
     if (!line) return '';
     const name = dir === 'in' ? L.askerOf(req) : (req.needs_builder_name || req.needs_builder_login);
     const id = dir === 'in' ? req.requested_by : req.needs_builder_id;
-    const link = personLink(id, name);
-    // The replacement is a FUNCTION, not a string. String.prototype.replace
-    // treats `$&`, `$$`, "$`" and `$'` specially in a string replacement, and the
-    // name here is builder-controlled (a display_name or login) — so a display
-    // name containing `$&` would duplicate or mangle the sentence around it. Not
-    // an injection (both halves are escaped either way), but a real way to
-    // corrupt someone else's row by choosing your own name.
-    const html = (name && link)
-      ? escapeHtml(line).replace(escapeHtml(name), () => link)
-      : escapeHtml(line);
+    return `<span class="ov-fact">${linkName(escapeHtml(line), id, name)}</span>`;
+  }
+
+  // The board's third-person line ("Alpha asked Beta by name"), both people as
+  // doors. The asker is linked first, then the addressee, each with the
+  // function replacer above.
+  function boardFact(req) {
+    const line = L.boardLine(req);
+    if (!line) return '';
+    let html = linkName(escapeHtml(line), req.requested_by, L.askerOf(req));
+    if (req.needs_builder_id != null) {
+      html = linkName(html, req.needs_builder_id, req.needs_builder_name || req.needs_builder_login);
+    }
     return `<span class="ov-fact">${html}</span>`;
   }
 
-  // A row for the "asked of you" list. There is deliberately NO "claim this"
+  // A row for the "asked of you" block. There is deliberately NO "claim this"
   // button: answering is a conversation, and inventing a reply thread here would
   // be reimplementing the Agora, which this goal explicitly forbids (C06 owns the
   // chat mirror). What the page owes a helper is the request, its age, its
-  // context, and who to answer.
-  function forMeRow(req, nowMs) {
-    return `<li class="ov-row" data-id="${escapeHtml(String(req.id))}">
+  // context, and who to answer — and their face. A by-name ask is the personal
+  // call on this reader, so its row is the LIT one (.cb-row--live).
+  function forMeRow(req, nowMs, extra) {
+    const live = req.needs_builder_id != null ? ' cb-row--live' : '';
+    return `<li class="ov-row ov-row--lead cb-row${live}" data-id="${escapeHtml(String(req.id))}"${extra ? ' data-extra hidden' : ''}>
+      ${faceHtml(L.faceOf(req, 'in'))}
       <div class="ov-row__main">${titleHtml(req)}
         <div class="ov-row__facts">${directionFact(req, 'in')}${contextChip(req)}</div>
       </div>
@@ -106,11 +153,13 @@
     </li>`;
   }
 
-  // Your own ask, with the two settle actions. Only the asker can settle — the
-  // server enforces it in the UPDATE's WHERE, this is just the affordance.
-  function mineRow(req, nowMs) {
+  // Your own ask, led by who you asked, with the two settle actions. Only the
+  // asker can settle — the server enforces it in the UPDATE's WHERE, this is
+  // just the affordance.
+  function mineRow(req, nowMs, extra) {
     const id = escapeHtml(String(req.id));
-    return `<li class="ov-row" data-id="${id}">
+    return `<li class="ov-row ov-row--lead cb-row" data-id="${id}"${extra ? ' data-extra hidden' : ''}>
+      ${faceHtml(L.faceOf(req, 'out'))}
       <div class="ov-row__main">${titleHtml(req)}
         <div class="ov-row__facts">${directionFact(req, 'out')}${contextChip(req)}</div>
       </div>
@@ -123,48 +172,79 @@
     </li>`;
   }
 
-  // "Asked of you" as runs under a group head: by name first, then each craft.
-  function groupedHtml(rows, nowMs) {
-    const groups = [];
-    for (const r of rows) {
-      const g = L.groupOf(r);
-      let hit = groups.find((x) => x.id === g.id);
-      if (!hit) { hit = { id: g.id, label: g.label, rows: [] }; groups.push(hit); }
-      hit.rows.push(r);
+  // A row on the board of everyone's asks: the asker's face, the third-person
+  // line, no actions — settling is the asker's, in their own block.
+  function allRow(req, nowMs, extra) {
+    return `<li class="ov-row ov-row--lead cb-row" data-id="${escapeHtml(String(req.id))}"${extra ? ' data-extra hidden' : ''}>
+      ${faceHtml(L.faceOf(req, 'in'))}
+      <div class="ov-row__main">${titleHtml(req)}
+        <div class="ov-row__facts">${boardFact(req)}${contextChip(req)}</div>
+      </div>
+      <div class="ov-row__right">${ageChip(req, nowMs)}${whenLine(req.created_at)}</div>
+    </li>`;
+  }
+
+  // ---- a block: count, faces, the newest three, the door to the rest -----------
+  //
+  // `key` names the block's ids (cb-<key>, cb-<key>-n, -faces, -empty, -more).
+  // Rows past SHOWN render with data-extra + hidden, and the button flips them —
+  // no re-render, no second fetch, no motion. The button says the whole number
+  // ("Show all 7"), so a reader knows what is behind the door before opening it.
+  function paintBlock(key, rows, rowFn, nowMs, faces) {
+    const list = rows.slice().sort(L.byNewestFirst);
+    const { shown, hidden } = L.splitShown(list, SHOWN);
+    const host = $(`#cb-${key}`);
+    if (host) {
+      host.innerHTML = shown.map((r) => rowFn(r, nowMs, false)).join('')
+        + hidden.map((r) => rowFn(r, nowMs, true)).join('');
+    }
+    const n = $(`#cb-${key}-n`);
+    if (n) n.textContent = list.length ? String(list.length) : '';
+    const empty = $(`#cb-${key}-empty`);
+    if (empty) empty.hidden = list.length > 0;
+    const facesEl = $(`#cb-${key}-faces`);
+    if (facesEl) facesEl.innerHTML = faces ? facesHtml(L.askers(list)) : '';
+    const more = $(`#cb-${key}-more`);
+    if (more) {
+      more.hidden = hidden.length === 0;
+      more.setAttribute('aria-expanded', 'false');
+      more.textContent = `Show all ${list.length}`;
+      more.dataset.total = String(list.length);
     }
-    groups.sort((a, b) => Number(b.id === 'you') - Number(a.id === 'you'));
-    return groups.map((g) => `<section class="ov-group">
-      <div class="ov-group__head"><span class="ov-group__name">${escapeHtml(g.label)}</span><span class="ov-group__n">${g.rows.length}</span></div>
-      <ul class="ov-rows">${g.rows.map((r) => forMeRow(r, nowMs)).join('')}</ul>
-    </section>`).join('');
   }
 
-  function paint(hostSel, emptySel, html, n) {
-    const host = $(hostSel);
-    const empty = $(emptySel);
-    if (host) host.innerHTML = html;
-    if (empty) empty.hidden = n > 0;
+  function toggleMore(btn) {
+    const key = btn.id.replace(/^cb-|-more$/g, '');
+    const host = $(`#cb-${key}`);
+    if (!host) return;
+    const open = btn.getAttribute('aria-expanded') === 'true';
+    host.querySelectorAll('[data-extra]').forEach((el) => { el.hidden = open; });
+    btn.setAttribute('aria-expanded', open ? 'false' : 'true');
+    btn.textContent = open ? `Show all ${btn.dataset.total}` : 'Show fewer';
   }
 
   async function load() {
     const nowMs = Date.now();
-    const [forMe, mine] = await Promise.all([
+    const [forMe, mine, all] = await Promise.all([
       api.helpRequests.getHelpRequestsForMe({}),
       api.helpRequests.getHelpRequestsMine({ query: { status: 'open' } }),
+      // The board read (task 1003942) — GET /help-requests, every open ask in the
+      // project. Through api.request because the generated client predates it.
+      api.request('GET', '/help-requests'),
     ]);
 
-    const forMeRows = ((forMe.ok && forMe.data && forMe.data.help_requests) || []).slice().sort(L.byAgeOldestFirst);
-    const mineRows = ((mine.ok && mine.data && mine.data.help_requests) || []).slice().sort(L.byAgeOldestFirst);
+    const rowsOf = (r) => ((r && r.ok && r.data && r.data.help_requests) || []);
+    const crafts = (forMe.ok && forMe.data && forMe.data.crafts) || [];
 
-    paint('#cb-for-me', '#cb-for-me-empty', groupedHtml(forMeRows, nowMs), forMeRows.length);
-    paint('#cb-mine', '#cb-mine-empty', mineRows.map((r) => mineRow(r, nowMs)).join(''), mineRows.length);
+    paintBlock('for-me', rowsOf(forMe), forMeRow, nowMs, true);
+    paintBlock('mine', rowsOf(mine), mineRow, nowMs, false);
+    paintBlock('all', rowsOf(all), allRow, nowMs, true);
 
     const sub = $('#cb-sub');
     if (sub) {
-      const crafts = (forMe.ok && forMe.data && forMe.data.crafts) || [];
       sub.textContent = crafts.length
-        ? `Matching your craft: ${crafts.join(', ')}.`
-        : 'You have not set a craft, so only requests addressed to you by name reach you.';
+        ? `Builders asking builders. Asks to ${crafts.join(', ')}s, and to you by name, land in your block.`
+        : 'Builders asking builders. With no craft set, only asks addressed to you by name land in your block.';
     }
 
     // The social half. Best-effort and LAST: an introductions read that fails
@@ -192,8 +272,11 @@
 
   function mingleRow(pair, meId) {
     const o = otherOf(pair, meId);
-    const who = escapeHtml(o.name || o.login || `builder ${o.id}`);
-    return `<li class="ov-row">
+    const whoName = o.name || o.login || `builder ${o.id}`;
+    const who = escapeHtml(whoName);
+    const face = { kind: 'person', id: o.id, name: whoName, avatar: null, initial: L.initialOf(whoName) };
+    return `<li class="ov-row ov-row--lead cb-row">
+      ${faceHtml(face)}
       <div class="ov-row__main">
         <a class="ov-row__title" href="/builders/profile?id=${escapeHtml(String(o.id))}">${who}</a>
         <p class="ov-row__sub">A ${escapeHtml(String(o.craft))} — you are a ${escapeHtml(String(o.myCraft))}. Mingle pairs across crafts.</p>
@@ -207,6 +290,13 @@
     </li>`;
   }
 
+  function paint(hostSel, emptySel, html, n) {
+    const host = $(hostSel);
+    const empty = $(emptySel);
+    if (host) host.innerHTML = html;
+    if (empty) empty.hidden = n > 0;
+  }
+
   async function loadMingle() {
     const section = $('#cb-mingle-section');
     if (!section) return;
@@ -257,13 +347,16 @@
   }
 
   document.addEventListener('click', (ev) => {
-    const mingleBtn = ev.target && ev.target.closest ? ev.target.closest('[data-mingle]') : null;
+    const closest = (sel) => (ev.target && ev.target.closest ? ev.target.closest(sel) : null);
+    const more = closest('.cb-more');
+    if (more) { ev.preventDefault(); toggleMore(more); return; }
+    const mingleBtn = closest('[data-mingle]');
     if (mingleBtn) {
       ev.preventDefault();
       decideMingle(Number(mingleBtn.getAttribute('data-id')), mingleBtn.getAttribute('data-mingle'));
       return;
     }
-    const btn = ev.target && ev.target.closest ? ev.target.closest('[data-settle]') : null;
+    const btn = closest('[data-settle]');
     if (!btn) return;
     ev.preventDefault();
     settle(Number(btn.getAttribute('data-id')), btn.getAttribute('data-settle'));

package/modules/hall-ui/public/shell.js

@@ -254,7 +254,9 @@
       // task 1003172 (C02, goal 1000075): help requests addressed to you. In
       // Community because the group is picked by WHOSE data it is, and this is
       // other builders reaching you — not your own numbers (that is You).
-      { id: 'collab', label: 'Collab', icon: 'leaderboard', href: '/collab', gate: 'signed' },
+      // task 1003942: the second consumer of the count badge. `help_requests` is
+      // the need id builder-needs.js emits with a `count`; applyCounts reads it.
+      { id: 'collab', label: 'Collab', icon: 'leaderboard', href: '/collab', gate: 'signed', count: 'help_requests' },
       // task 1003441: was the #leaderboard hash view on the home page, which is
       // why this group read as one real page plus a link into index.html. Now a
       // page, so no `view` key. Labelled Builders — 'roster' is the word

package/modules/lifecycle/help-requests.js

@@ -55,7 +55,9 @@ const COLS = 'id, requested_by, needs_craft, needs_builder_id, what_is_stuck, ta
 // bare for the writes, whose RETURNING cannot reach a joined alias.
 const LIST_COLS = `${COLS.split(', ').map((c) => `h.${c}`).join(', ')}, `
   + 'a.github_login AS requested_by_login, a.display_name AS requested_by_name, '
-  + 'n.github_login AS needs_builder_login, n.display_name AS needs_builder_name';
+  + 'a.avatar_url AS requested_by_avatar_url, '
+  + 'n.github_login AS needs_builder_login, n.display_name AS needs_builder_name, '
+  + 'n.avatar_url AS needs_builder_avatar_url';
 const LIST_FROM = 'FROM lifecycle_help_requests h '
   + 'LEFT JOIN builders a ON a.id = h.requested_by '
   + 'LEFT JOIN builders n ON n.id = h.needs_builder_id';
@@ -200,6 +202,26 @@ async function listMine({ builderId, status = null, limit = MAX_ROWS }, deps = {
   return rows;
 }
 
+// The BOARD read (task 1003942, owner direction): every open ask in the project,
+// newest first, for the Collab page's third block. The two reads above answer
+// "what is addressed to me" and "what did I ask"; this one answers "what is
+// anyone asking", which is what makes the page a room rather than an inbox.
+// Any signed-in builder may read it: an ask is addressed to a craft or a person
+// INSIDE this project, and every reader here is a member of it. It never takes
+// a craft or a builder from the caller — there is nothing to narrow by, so the
+// ?craft= hole C01's grader found cannot reopen here.
+async function listOpenAll({ limit = MAX_ROWS } = {}, deps = {}) {
+  const activePool = deps.pool || pool;
+  const { rows } = await activePool.query(
+    `SELECT ${LIST_COLS} ${LIST_FROM}
+      WHERE h.status = 'open'
+      ORDER BY h.created_at DESC, h.id DESC
+      LIMIT $1`,
+    [capRows(limit)]
+  );
+  return rows;
+}
+
 async function getHelpRequest(id, deps = {}) {
   const activePool = deps.pool || pool;
   const { rows } = await activePool.query(`SELECT ${COLS} FROM lifecycle_help_requests WHERE id = $1`, [id]);
@@ -224,6 +246,6 @@ async function setHelpRequestStatus({ id, builderId, status }, deps = {}) {
 module.exports = {
   HELP_CRAFTS, helpCrafts, disciplineForCraft, craftForDiscipline, craftsForBuilder,
   validateHelpRequest, MAX_STUCK, MAX_ROWS, capRows,
-  builderExists, createHelpRequest, listOpenForBuilder, listMine, getHelpRequest, setHelpRequestStatus,
+  builderExists, createHelpRequest, listOpenForBuilder, listMine, listOpenAll, getHelpRequest, setHelpRequestStatus,
   countOpenForBuilder, craftMapSql,
 };

package/modules/lifecycle/routes/help-requests.js

@@ -2,7 +2,8 @@
 // read for "I am stuck" (task 1003171 / C01, goal 1000075; criterion
 // collab-ask-for-help).
 //
-// Four routes, no more: ask, see what is addressed to me, see my own, settle mine.
+// Five routes: ask, see what is addressed to me, see my own, see the whole open
+// board (task 1003942), settle mine.
 // The Collab PAGE is C02 and the matching rules beyond craft are its job — this
 // surface deliberately stops at "the request is addressed and answerable".
 //
@@ -88,6 +89,15 @@ module.exports = function buildHelpRequestsRouter() {
     return res.json({ help_requests: rows, count: rows.length });
   }, { errorCode: 'help_request_list_failed' }));
 
+  // The BOARD read (task 1003942): every open ask in the project, newest first,
+  // for the Collab page's "all open asks" block. rank: any-builder — an ask is
+  // addressed inside this project and every reader here is a member; the route
+  // takes no craft and no builder from the caller, so there is nothing to widen.
+  router.get('/help-requests', api.requireBuilder, asyncHandler('GET /help-requests', async (req, res) => {
+    const rows = await help.listOpenAll();
+    return res.json({ help_requests: rows, count: rows.length });
+  }, { errorCode: 'help_request_list_failed' }));
+
   // Only the ASKER settles their own request: 'answered' when someone helped,
   // 'withdrawn' when they no longer need it. Nobody is assigned by being asked —
   // the same consent posture ADR 0187 §3 gives task recommendations.

package/package-lock.json

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bongos/core",
-  "version": "1.19.781",
+  "version": "1.19.783",
   "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/scripts/gds/start.js

@@ -112,6 +112,30 @@ function formatCostRow(cost) {
   return `${fmtUsd(spent)} logged spend  ·  ${credits}c reward shipped`;
 }
 
+// inboxOpenCount — how many ideas are OPEN in the inbox, from a GET /inbox response.
+//
+// WHY THIS IS NOT ideas.length (task 1003454). GET /inbox answers
+// { ideas, open_count }, where `ideas` is a PAGE (default 50 rows) and
+// `open_count` is the real total, counted server-side under the same rank gate
+// as the rows. Reading the array length therefore reported the page size: the
+// card said "50 ideas in inbox" while the payload said 99, and once the queue
+// passed one page the figure pinned at exactly 50 forever — a number that looks
+// plausible, never moves, and understates the backlog the card exists to make
+// visible. capture.js already destructured open_count correctly, which is what
+// made this an oversight rather than a convention.
+//
+// The page length stays as a FALLBACK so an older core that predates open_count
+// still renders a figure rather than "N/A". Returns null when neither is
+// readable, which the callers render as "ideas: N/A".
+function inboxOpenCount(inboxRes) {
+  if (!inboxRes || !inboxRes.ok) return null;
+  const data = inboxRes.data;
+  if (data && typeof data.open_count === 'number') return data.open_count;
+  if (data && Array.isArray(data.ideas)) return data.ideas.length;
+  if (data && typeof data.count === 'number') return data.count;
+  return null;
+}
+
 function formatRecentRow({ recentShipped, inboxCount, blockerCount }) {
   const bits = [];
   // "1 ship in last 24h" — recent_shipped returns tasks shipped within
@@ -741,9 +765,7 @@ async function main() {
   const progress = progressRes.ok && Array.isArray(progressRes.data?.progress) ? progressRes.data.progress : null;
   const cost = costRes.ok ? costRes.data : null;
   const recentShipped = recentRes.ok && Array.isArray(recentRes.data?.tasks) ? recentRes.data.tasks : null;
-  const inboxCount = inboxRes.ok && Array.isArray(inboxRes.data?.ideas)
-    ? inboxRes.data.ideas.length
-    : (inboxRes.ok && typeof inboxRes.data?.count === 'number' ? inboxRes.data.count : null);
+  const inboxCount = inboxOpenCount(inboxRes);
   const blockerCount = blockersRes.ok && Array.isArray(blockersRes.data?.blockers)
     ? blockersRes.data.blockers.filter((b) => b.status === 'open' || !b.status).length
     : null;
@@ -1023,6 +1045,7 @@ if (require.main === module) {
 }
 
 module.exports = {
+  inboxOpenCount,
   rankCrossDisciplineRecommendations,
   rebaseWarningModel,
   rebaseWarningMarkdown,

package/src/module-api.js

@@ -71,7 +71,7 @@ const { responsibilityFor, ROLE_RESPONSIBILITIES } = require('./role-responsibil
 // 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.781'; // CI auto-patch carrier (ADR 0161); changelog: docs/module-api-changelog.md
+const CORE_VERSION = '1.19.783'; // 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/builder_needs.mjs

@@ -331,5 +331,29 @@ t('nothing waiting renders nothing at all', () => {
   assert.equal(html, '');
 });
 
+// task 1003942: the help-requests need feeds the Collab nav badge, which reads
+// the need's `count` (the field boardVotesNeed already carried) and follows its
+// action to the page. Both were missing: no count, and a '/#/collab' hash href
+// pointing at a view the index stopped routing when Collab became a page.
+console.log('help_requests need carries a count and points at the page (task 1003942):');
+
+t('open help requests state their number, machine-readable', () => {
+  const n = needs.helpRequestsNeed({ openHelpRequests: 4 });
+  assert.equal(n.count, 4);
+  assert.equal(n.state, 'action_needed');
+  assert.equal(needs.helpRequestsNeed({ openHelpRequests: 1 }).count, 1);
+});
+
+t('the action goes to /collab, the page — not the retired hash view', () => {
+  assert.equal(needs.helpRequestsNeed({ openHelpRequests: 2 }).action.href, '/collab');
+  assert.equal(needs.taskRecommendationsNeed({ openTaskRecommendations: 1 }).action.href, '/collab');
+  assert.equal(needs.mingleIntroductionsNeed({ openMingleIntroductions: 1 }).action.href, '/collab');
+});
+
+t('zero open requests is no need at all — nothing for a badge to count', () => {
+  assert.equal(needs.helpRequestsNeed({ openHelpRequests: 0 }), null);
+  assert.equal(needs.helpRequestsNeed({}), null);
+});
+
 console.log(`\n${passed} passed, ${failed} failed`);
 process.exit(failed ? 1 : 0);