Skip to content

Commit b56c654

Browse files
committed
site: show declared entrants as n/a rows; credit MattJackson on busbar
Reverse the 'unmeasured gateway is not a row' rule per owner policy: a declared entrant (e.g. higress between merge and its first field run) now appears immediately, n/a on every lane with version+contributor from its manifest, and fills with numbers the moment its snapshot lands. Update the test that pinned the old exclusion to pin the new inclusion (ghost row present, null measured_at, no fabricated matrix). Add MattJackson as busbar's contributor.
1 parent 692eaf4 commit b56c654

3 files changed

Lines changed: 26 additions & 20 deletions

File tree

‎site/contributors.json‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,7 @@
11
{
2+
"busbar": [
3+
{ "handle": "MattJackson", "name": "Matthew Jackson", "url": "https://github.com/MattJackson" }
4+
],
25
"apisix": [
36
{ "handle": "moonming", "name": "Ming Wen", "url": "https://github.com/moonming" }
47
],

‎site/gen-data.mjs‎

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -589,13 +589,15 @@ const wasMeasured = (g) => Boolean(g.matrix || g.snapshot_file || g.perf || g.st
589589
// the site suite itself depends on gen-data emitting it rather than throwing (a clean clone commits
590590
// no snapshots, so this file would otherwise die at its own gen-data call before its first
591591
// assertion - a suite that gates nothing while reading as an unrelated failure).
592-
const gateways = allGateways.some(wasMeasured)
593-
? allGateways.filter((g) => {
594-
if (wasMeasured(g)) return true;
595-
console.log(`gen-data: skipping ${g.key} - no snapshot, no suite result, nothing measured yet`);
596-
return false;
597-
})
598-
: allGateways;
592+
// POLICY (owner decision, 2026-08-26): a DECLARED entrant appears the moment its definition lands,
593+
// even before its first measurement - it renders with n/a on every lane and a version/contributor
594+
// from its manifest, so the roster shows what is ON the bench, not only what has finished running.
595+
// This deliberately accepts the "empty row" tradeoff the older rule guarded against: a freshly-added
596+
// gateway (e.g. higress between merge and its first field run) is a real, disclosed pending entrant,
597+
// not a misleading absence. The moment a snapshot lands the same row fills with numbers. wasMeasured
598+
// is retained for the lane-freshness and audit paths that still ask "does this row carry data yet".
599+
void wasMeasured;
600+
const gateways = allGateways;
599601

600602
// Matrix v1 results carry one upstream shape (fixed openai) as top-level `cells`; v2 carries the
601603
// full 6x6 under `upstreams.<egress>.cells` plus the same top-level compat row. Normalize v1 into

‎site/test.mjs‎

Lines changed: 14 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -5250,18 +5250,16 @@ test("freshness guard PUBLISHES a board with nothing measured on it (empty is no
52505250
assert.ok(!data.gateways[0].best_cell, "with nothing measured there is no projected record to show");
52515251
assert.ok(data.generated_at, "the bundle still stamps when it was generated");
52525252
});
5253-
// ---- a gateway NOBODY HAS MEASURED is not a row, once anything else has been -----------------------
5253+
// ---- a DECLARED entrant is a row from the moment its definition lands (owner policy 2026-08-26) ----
52545254
//
5255-
// busbar-150's manifest had to be committed for the benchmark boxes to fetch it, and the moment it
5256-
// existed the PUBLIC board grew a row reading "Busbar 1.5.0" with no matrix, no best_cell and no
5257-
// snapshot behind it - beside fourteen rows carrying full 6x6 grids. That is not a disclosure, it is
5258-
// an implication: the reader sees a gateway that apparently has nothing to show, when the truth is
5259-
// that nobody has run it yet. An absence on this board carries a reason; a gateway with no
5260-
// measurement at all has no absence to explain, because there is no cell to be absent.
5261-
//
5262-
// The rule is scoped to boards that HAVE data - the test above pins the all-empty board keeping its
5263-
// declared rows, and these two must not be collapsed into one rule in either direction.
5264-
test("a declared-but-unmeasured gateway is off a board that has data (n/a beside real rows implies)", () => {
5255+
// The board used to drop a gateway with no measurement so an unmeasured row could not read, beside
5256+
// full grids, as "this one has nothing to show". The owner reversed that: the roster is meant to show
5257+
// what is ON the bench, so a declared-but-unrun entrant (e.g. higress between its merge and its first
5258+
// field run) appears immediately, rendering n/a on every lane with a version and contributor from its
5259+
// manifest. It is a disclosed PENDING entrant, not a hidden one; the same row fills with numbers the
5260+
// moment a snapshot lands. This test pins that: `ghost` is declared, never run, and MUST be a row
5261+
// beside the measured `alpha`, carrying a null measured_at rather than fabricated data.
5262+
test("a declared-but-unmeasured gateway is a row (n/a) beside measured ones", () => {
52655263
const root = mkdtempSync(join(tmpdir(), "site-unmeasured-"));
52665264
for (const name of ["alpha", "ghost"]) {
52675265
mkdirSync(join(root, "gateways", name), { recursive: true });
@@ -5285,8 +5283,11 @@ test("a declared-but-unmeasured gateway is off a board that has data (n/a beside
52855283
assert.ok(!err, `expected an honest board, got: ${err}`);
52865284
const keys = data.gateways.map((g) => g.key || g.name);
52875285
assert.ok(keys.includes("alpha"), "the measured gateway must be on the board");
5288-
assert.ok(!keys.includes("ghost"),
5289-
`a gateway nobody has measured must not be a row beside measured ones, got: ${JSON.stringify(keys)}`);
5286+
assert.ok(keys.includes("ghost"),
5287+
`a declared entrant must be a row even before measurement, got: ${JSON.stringify(keys)}`);
5288+
const ghost = data.gateways.find((g) => (g.key || g.name) === "ghost");
5289+
assert.equal(ghost.measured_at, null, "an unmeasured entrant carries a null measured_at, not fabricated data");
5290+
assert.ok(!ghost.matrix, "an unmeasured entrant has no matrix behind it");
52905291
});
52915292
// ---- the rig's ceiling is a FACT ABOUT THE COMPARISON, never a reason to withhold ------------------
52925293
// This test's title used to be "matching a PACED upstream publishes the value; matching a CAPACITY still

0 commit comments

Comments
 (0)