The module's half of PLAN_FIXES §6 step 2 (decisions D181-D185, docs#288). - F13/F14 (D170, D183): `world.expired`, recognisable from protocol 13 by its `what`, is handed to core as the resource the zone step ledgered (`world`, `<serverId>:<id>`) through ctx.events.expired, which records it `expired`. coreApi moves to ^1.11.0 (website#209). - F8 (D184): `plugin.loaded` / `plugin.unloaded` mark the permission sync dirty when the plugin added or removed permissions, so an unresolved grant lands on the next tick instead of the fifteen-minute audit. - Catalogue: plugin.loaded/unloaded, world.expired and lease.expired are staff kinds. The last two were never classified (default deny kept them off public pages); the test now covers every event kind through protocol 13. - F7: permission and title pushes hold while the stored hello says `worldReady: false` (a human's "sync now" does not); a failed or refused permission sync now logs at warn. - F2 (D185): the killfeed names an NPC attacker — a family (Scientist, Bandit guard, Bradley APC…) or the prefab without its variant digits (wolf2 → Wolf). - F5/F6: a link code is asked of the servers that minted one in the last six minutes first, then of the rest, each group in parallel; "unsure" only when one of the minting servers is unreachable. - D182: the admin server list carries the ZoneManager helper's state from the hello, and the servers page says what a missing or failed helper costs. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01E14m6SuuY6i1vASFeGDBeY
178 lines
7.1 KiB
JavaScript
178 lines
7.1 KiB
JavaScript
// ── The boundary, asserted ────────────────────────────────────────────────
|
||
//
|
||
// `catalogue.js` is the only thing standing between a frame carrying an IP
|
||
// address and a public page, so it gets a suite of its own rather than being
|
||
// covered incidentally by a route test.
|
||
//
|
||
// The most valuable test here is the last one: it holds the classification
|
||
// against the specification in `docs/rust-link/PROTOCOL.md` §8.4. Without it the
|
||
// two drift the first time somebody adds a kind to the protocol, and the drift
|
||
// is silent in the direction that matters — a new kind is simply never served,
|
||
// until the day somebody "fixes" that by adding it to the wrong list.
|
||
|
||
const test = require('node:test')
|
||
const assert = require('node:assert')
|
||
|
||
const catalogue = require('../catalogue')
|
||
|
||
test('an unknown kind is not public — the default is deny', () => {
|
||
assert.equal(catalogue.isPublic('player.death'), true)
|
||
assert.equal(catalogue.isPublic('something.new'), false)
|
||
assert.equal(catalogue.isPublic(''), false)
|
||
assert.equal(catalogue.isPublic(undefined), false)
|
||
|
||
// The shape of the mistake this prevents: a kind a LATER protocol adds, which
|
||
// this build ingests happily and would publish on the day it first arrived if
|
||
// the filter were a deny list.
|
||
assert.equal(catalogue.isKnown('player.location'), false)
|
||
assert.equal(catalogue.isPublic('player.location'), false)
|
||
})
|
||
|
||
test('nothing carrying an IP address, a report or an identity is public', () => {
|
||
for (const kind of [
|
||
'player.login.attempt',
|
||
'player.approved',
|
||
'player.banned',
|
||
'player.unbanned',
|
||
'player.reported',
|
||
'entity.destroyed',
|
||
// Protocol 3. A link request on a public killfeed would tell everyone which
|
||
// Steam id is about to become a named website account, and an unlink would
|
||
// say when somebody stopped being one.
|
||
'account.link.requested',
|
||
'account.unlinked',
|
||
]) {
|
||
assert.equal(catalogue.isPublic(kind), false, `${kind} must not be public`)
|
||
assert.ok(catalogue.STAFF_KINDS.includes(kind), `${kind} must be classified, not merely absent`)
|
||
}
|
||
})
|
||
|
||
test('a viewer with no kinds asked for gets the allowlist, never everything', () => {
|
||
const asPublic = catalogue.kindsFor({})
|
||
const asAdmin = catalogue.kindsFor({ admin: true })
|
||
|
||
// The public view with nothing said about presence is the kinds that name
|
||
// nobody — a wipe, a start, a shutdown.
|
||
assert.deepEqual(
|
||
asPublic,
|
||
catalogue.PUBLIC_KINDS.filter((k) => !catalogue.PRESENCE_KINDS.includes(k)),
|
||
)
|
||
assert.equal(asAdmin.length, catalogue.ALL_KINDS.length)
|
||
|
||
// The property that makes the route safe by construction: there is no argument
|
||
// a caller can omit that turns the filter off.
|
||
assert.ok(asPublic.length > 0)
|
||
assert.ok(!asPublic.includes('player.banned'))
|
||
})
|
||
|
||
test('a kind a viewer may not see is dropped, not refused', () => {
|
||
const asked = catalogue.kindsFor({ presence: true, requested: ['player.death', 'player.banned'] })
|
||
|
||
assert.deepEqual(asked, ['player.death'])
|
||
|
||
// Without the presence audience a death is dropped too.
|
||
assert.deepEqual(catalogue.kindsFor({ requested: ['player.death', 'server.wipe'] }), ['server.wipe'])
|
||
|
||
// Asking for only forbidden kinds answers with nothing to select, which the
|
||
// model turns into an empty list — the events are, as far as this viewer is
|
||
// concerned, not there.
|
||
assert.deepEqual(catalogue.kindsFor({ requested: ['player.banned'] }), [])
|
||
|
||
// And an admin gets what they asked for.
|
||
assert.deepEqual(catalogue.kindsFor({ admin: true, requested: ['player.banned'] }), [
|
||
'player.banned',
|
||
])
|
||
})
|
||
|
||
test('every kind is classified exactly once', () => {
|
||
const seen = new Set()
|
||
|
||
for (const kind of catalogue.ALL_KINDS) {
|
||
assert.ok(!seen.has(kind), `${kind} appears in both lists`)
|
||
seen.add(kind)
|
||
}
|
||
|
||
assert.equal(seen.size, catalogue.PUBLIC_KINDS.length + catalogue.STAFF_KINDS.length)
|
||
})
|
||
|
||
test('the classification covers exactly the event kinds the protocol defines, through protocol 13', () => {
|
||
// The spec lives in another repository, so the list is restated here rather
|
||
// than parsed — and restating it is the point: adding a kind to the protocol
|
||
// without deciding who may see it has to fail somewhere, and this is where.
|
||
//
|
||
// Sourced from docs/rust-link/PROTOCOL.md §8.4.
|
||
const PROTOCOL_4 = [
|
||
'player.connected',
|
||
'player.disconnected',
|
||
'player.respawned',
|
||
'player.death',
|
||
'player.chat',
|
||
'player.tally',
|
||
'entity.destroyed',
|
||
'player.reported',
|
||
'player.banned',
|
||
'player.unbanned',
|
||
'player.login.attempt',
|
||
'player.approved',
|
||
'server.wipe',
|
||
'server.initialized',
|
||
'server.shutdown',
|
||
'account.link.requested',
|
||
'account.unlinked',
|
||
'perm.drift',
|
||
// Protocol 6 (§12). Clan membership is members-only (D49), so every one of
|
||
// these is staff-class here and reaches members through core's Team feed.
|
||
'clan.created',
|
||
'clan.disbanded',
|
||
'clan.member.added',
|
||
'clan.member.left',
|
||
'clan.member.kicked',
|
||
// Protocol 8 (§14) and protocol 9 (§15): an event's machinery ending on its
|
||
// own deadline. Never classified until protocol 13 made `world.expired`
|
||
// recognisable (F13) — default deny kept both off public pages meanwhile.
|
||
'lease.expired',
|
||
'world.expired',
|
||
// Protocol 13 (§19). A configuration save's outcome carries the server's
|
||
// log tail, which is an operator's console: staff only.
|
||
'config.outcome',
|
||
// Protocol 13 (§19, F8, D184). Which plugins a server runs and what each
|
||
// registers: an operator's inventory.
|
||
'plugin.loaded',
|
||
'plugin.unloaded',
|
||
]
|
||
|
||
assert.deepEqual([...catalogue.ALL_KINDS].sort(), [...PROTOCOL_4].sort())
|
||
|
||
for (const kind of PROTOCOL_4.filter((k) => k.startsWith('clan.'))) {
|
||
assert.equal(catalogue.isPublic(kind), false, `${kind} is members-only and must not be public`)
|
||
}
|
||
|
||
assert.equal(catalogue.isPublic('config.outcome'), false, 'a server’s log tail must not be public')
|
||
for (const kind of ['plugin.loaded', 'plugin.unloaded', 'lease.expired', 'world.expired']) {
|
||
assert.equal(catalogue.isPublic(kind), false, `${kind} is staff-only`)
|
||
}
|
||
})
|
||
|
||
test('every kind that names a player who was on is behind the presence setting', () => {
|
||
// The org lead's rule (2026-09-22): nothing tells who is online by default.
|
||
// Each of these says a named player was on the server at a given moment.
|
||
for (const kind of [
|
||
'player.connected',
|
||
'player.disconnected',
|
||
'player.respawned',
|
||
'player.death',
|
||
'player.chat',
|
||
'player.tally',
|
||
]) {
|
||
assert.ok(catalogue.isPresence(kind), `${kind} must be gated as presence`)
|
||
assert.ok(!catalogue.kindsFor({}).includes(kind), `${kind} must not reach a default public view`)
|
||
assert.ok(catalogue.kindsFor({ presence: true }).includes(kind))
|
||
}
|
||
|
||
// A presence kind is a subset of the public ones, never a staff kind widened.
|
||
for (const kind of catalogue.PRESENCE_KINDS) assert.ok(catalogue.PUBLIC_KINDS.includes(kind))
|
||
|
||
// And what is left names nobody.
|
||
assert.deepEqual(catalogue.kindsFor({}).sort(), ['server.initialized', 'server.shutdown', 'server.wipe'])
|
||
})
|