A second-digit bump for a playtest read back against the save file. Nine questions were asked of one three-Day game; three were bugs, three were the rules working and undocumented, three were decisions. Every save on the test server was replayed against this build BEFORE release, which is how the cost of each rule was known before it was chosen rather than discovered after. EVERY DISTRICT OPENS ON A DEPOT. A Whistle Post has one A/D track and is not a Passenger Facility, so the opening of every game was spent unable to work a passenger and one arrival away from a collision. Two A/D tracks and passengers from Stage 1 now; "Players start with Whistle Posts, not Depots" is the harder game, set when the game is created. The deck follows the choice — starting on Depots the four Depot upgrade cards are left out, because an upgrade must be to the next tier and a Depot card at a table of Depots is a dead draw. How much easier it is showed up as a test failure rather than an argument: the cue-coverage pool needed widening from 24 seeded games to 60 before it held one collision. NO SAVE WAS STRANDED BY IT, which took care. This is the one house rule that changes how a game is DEALT rather than how it plays, so replaying a save under the wrong opening is a different railroad from intent one — silently, with no error. `withSavedOpening` fills it on the replay paths ONLY. Putting it in the resolver instead made a fresh Cutthroat game deal Whistle Posts and read as Custom, which is how the distinction was found. THREE BUGS, ALL REPORTED FROM ONE GAME AND ALL CONFIRMED ON ITS SAVE. An Office held TWO TRAINS ON ONE A/D TRACK. The capacity test passed with nothing standing, the train the Interlocking had been holding at the Limits was moved into the free slot, and the arriving train was pushed in after it without anyone asking again whether there was room — so the collision §8.3 calls for never happened. The held train keeps priority; the newcomer now takes the consequence it would have met had the held train arrived first. THE HISTORY FROZE, permanently, and the log cap was not really the cause. Each seat's "what have I sent you" bookmark was an INDEX into an array the game trims, so once a seat's bookmark reached the limit the slice returned nothing for the rest of the game — at a different moment per seat, because each holds its own. That game's log ended at exactly the cap. Lines carry a sequence number now, which survives trimming; proven by pushing twice the cap through a simulated seat. §8.1 ASKED THE WRONG QUESTION TWICE. "Trains may pass" returned `clear` before the Subdivision was looked at, so a train entering a Double Track was released however busy the rest of it was — that, not anything about Control Points, is what let Train 8 out with no ruling. And a train standing at an Office was invisible to the scan, so one about to re-enter the very Subdivision being entered counted for nothing. Capacity is the test, not presence: a Depot with a track free is not in the way; a Whistle Post with its one track taken is. THINGS THAT HAPPENED SILENTLY NOW SAY SO — a train held against a facing one, a train released from the Limits (a side effect of somebody else's arrival, so it simply appeared at the Office), and the train an Interlocking is holding, whose explanatory tooltip has existed since #99 with NO renderer ever reading the flag. WHERE A MOVE IS REFUSED, AND WHY. `exploreMoves` decides where the rails go and the pick-up restrictions are enforced afterwards in `check`, so a square the rails reached and the card forbade was reachable, un-offered, and absent from the block list with nothing said. Those squares are blocked with the rule that blocks them now, and the reasons are got by ASKING `check` rather than re-deriving: a second implementation of the rules is exactly the failure the block list exists to avoid. A train may also always recover its own caboose — X13 prints "may drop but not pick up anything", and a train needs its caboose to be made up, so one that parted with it could never legally leave again. RULES DECIDED IN SEPTEMBER AND APPLIED HERE. A Modifier must sit square against its host, no diagonals. A passenger Modifier may not be played at a Whistle Post. Both were built, measured, held back for a fortnight so a playtest could finish, and applied now. A Second Section costs its card: `SECOND_SECTION` was declared in content.ts and never dealt, so the action was free and the bot ordered 26 accidental ones in a measured round. The card is dealt and spent — gating on a card the deck never holds would have deleted the mechanic rather than fixed it. THE DOCUMENTATION IS A SET OF PAGES, not five text files served as text/plain — a card reference is mostly tables, and as plain text a table is rows of pipes. Markdown is still the one copy; the build renders it, and publishes the .md beside each page. No Markdown library: this project has no runtime dependencies and one would be a poor first. The pages add what Markdown cannot carry without drifting — a nav across the set, a contents list built from the headings actually rendered, an anchor on every heading, a 70-character measure, and tables that are tables. They print as ink on paper. The references caught up with the rules, checked rather than assumed: two statements had gone from stale to misleading (the Quickstart told a new player to "get a Depot down as soon as one appears"), and four rules nobody could look up are written down — the Office tier table, §8.1 in practice, what the Circus Train pays for, and that a Realignment can be a card with no legal target. Adding one card to the deck reshuffles every seeded deal, which broke five fixtures. Each was a seed meaning "a game like this" — TODO #84, exactly — so seeds moved and pools widened rather than assertions weakening, and the clearance fixture pins its terrain the way `enhancements.test.ts` already does. The three published replays were re-recorded. Closes TODO #40, #42a, #108, #109 and #110. 1046 fast tests and 35 sim tests pass. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MUizFYCMHRWhbWwXhp7WPR
114 lines
6.4 KiB
TypeScript
114 lines
6.4 KiB
TypeScript
/**
|
|
* The documentation renderer.
|
|
*
|
|
* The five player-facing documents are written in Markdown — that is the one copy, and the whole
|
|
* reason TODO #15a exists — and rendered to pages at build time. This covers the subset those
|
|
* documents actually use, and the two properties that matter most: a table comes out as a TABLE
|
|
* (the entire point of rendering rather than serving text), and nothing in the prose can become
|
|
* markup by accident.
|
|
*/
|
|
|
|
import { describe, it } from 'node:test';
|
|
import assert from 'node:assert/strict';
|
|
|
|
import { readFileSync } from 'node:fs';
|
|
import { join } from 'node:path';
|
|
|
|
import { renderMarkdown } from '../scripts/markdown.ts';
|
|
|
|
const html = (src: string): string => renderMarkdown(src).html;
|
|
|
|
describe('the documentation renderer', () => {
|
|
it('turns a pipe table into a real table, with its alignment', () => {
|
|
// This is what rendering is FOR. A card reference is mostly tables, and as plain text a table
|
|
// is rows of pipes — which is exactly how the guide read when it was served as text/plain.
|
|
const out = html(
|
|
['| Card | Regions | Passes |', '| --- | ---: | :---: |', '| Plains | 1 | no |', '| Tunnel | 2 | yes |'].join('\n'),
|
|
);
|
|
assert.match(out, /<table>/, 'the table is not a table');
|
|
assert.match(out, /<thead><tr><th>Card<\/th>/, 'the header row is not a header');
|
|
assert.match(out, /<th class="ta-right">Regions<\/th>/, 'a right-aligned column lost its alignment');
|
|
assert.match(out, /<th class="ta-center">Passes<\/th>/, 'a centred column lost its alignment');
|
|
assert.match(out, /<td>Plains<\/td><td class="ta-right">1<\/td>/, 'a body row lost its cells');
|
|
assert.equal((out.match(/<tr>/g) ?? []).length, 3, 'wrong number of rows');
|
|
// Wrapped, so a wide table scrolls inside the page instead of widening it on a phone.
|
|
assert.match(out, /<div class="tablewrap">/, 'the table can widen the page on a narrow screen');
|
|
});
|
|
|
|
it('gives every heading an id and an anchor, numbered prefixes stripped', () => {
|
|
const { html: out, headings } = renderMarkdown('## 4.2 Local Operations\n\ntext\n');
|
|
assert.deepEqual(headings, [{ level: 2, text: '4.2 Local Operations', id: 'local-operations' }]);
|
|
assert.match(out, /<h2 id="local-operations">/, 'the heading has no id to link to');
|
|
assert.match(out, /<a class="anchor" href="#local-operations"/, 'the heading has no anchor');
|
|
});
|
|
|
|
it('numbers a repeated heading rather than pointing two links at one place', () => {
|
|
const { headings } = renderMarkdown('## Trains\n\na\n\n## Trains\n\nb\n');
|
|
assert.deepEqual(headings.map((h) => h.id), ['trains', 'trains-2']);
|
|
});
|
|
|
|
it('renders lists, quotes, rules and fenced code', () => {
|
|
assert.match(html('- one\n- two\n'), /<ul><li>one<\/li><li>two<\/li><\/ul>/);
|
|
assert.match(html('1. first\n2. second\n'), /<ol><li>first<\/li><li>second<\/li><\/ol>/);
|
|
assert.match(html('> a note\n> continued\n'), /<blockquote><p>a note continued<\/p><\/blockquote>/);
|
|
assert.match(html('---\n'), /<hr>/);
|
|
assert.match(html('```\nconst x = 1;\n```\n'), /<pre><code>const x = 1;<\/code><\/pre>/);
|
|
});
|
|
|
|
it('renders a table inside a block quote', () => {
|
|
// The rules reference puts one there, so this is not hypothetical.
|
|
const out = html('> | A | B |\n> | --- | --- |\n> | 1 | 2 |\n');
|
|
assert.match(out, /<blockquote><div class="tablewrap"><table>/, 'a quoted table did not render');
|
|
});
|
|
|
|
it('handles bold, italic and code spans, and leaves markup inside code alone', () => {
|
|
assert.match(html('**loud** and *quiet*\n'), /<strong>loud<\/strong> and <em>quiet<\/em>/);
|
|
// `**` inside backticks is a literal, which matters: the docs quote field names that way.
|
|
assert.match(html('`**not bold**`\n'), /<code>\*\*not bold\*\*<\/code>/);
|
|
assert.ok(!/<strong>/.test(html('`**not bold**`\n')), 'markup inside a code span was rendered');
|
|
});
|
|
|
|
it('escapes everything, so prose can never become markup', () => {
|
|
const out = html('A < B & C > D, and "quoted".\n');
|
|
assert.match(out, /A < B & C > D/, 'angle brackets or ampersands reached the page raw');
|
|
assert.ok(!/<script/i.test(html('<script>alert(1)</script>\n')), 'raw HTML passed through');
|
|
assert.match(html('<script>alert(1)</script>\n'), /<script>/, 'raw HTML was not escaped');
|
|
});
|
|
|
|
it('rewrites links between documents, and opens external ones in a new tab', () => {
|
|
const out = renderMarkdown(
|
|
'[Rules](rules.md) and [anchor](rules.md#draw) and [site](https://example.com)\n',
|
|
(href) => (/^https?:/.test(href) ? href : href.replace(/\.md(#|$)/, '.html$1')),
|
|
).html;
|
|
assert.match(out, /<a href="rules\.html">Rules<\/a>/, 'a link between documents still points at the Markdown');
|
|
assert.match(out, /<a href="rules\.html#draw">/, 'an anchored link lost its fragment');
|
|
assert.match(out, /<a href="https:\/\/example\.com" target="_blank" rel="noopener">/, 'an external link is not safe');
|
|
});
|
|
|
|
it('drops HTML comments, so the generated-card markers never show', () => {
|
|
// `build-card-reference.ts` writes its tables between `<!-- BEGIN CARDS: … -->` markers.
|
|
const out = html('before\n\n<!-- BEGIN CARDS: track -->\n| A |\n| --- |\n| 1 |\n<!-- END CARDS: track -->\n\nafter\n');
|
|
assert.ok(!/BEGIN CARDS/.test(out), 'a build marker is visible on the page');
|
|
assert.match(out, /<table>/, 'the generated table inside the markers was lost with them');
|
|
assert.match(out, /before/, 'content before the markers was lost');
|
|
assert.match(out, /after/, 'content after the markers was lost');
|
|
});
|
|
|
|
it('renders each real document without losing its tables or headings', () => {
|
|
// The documents themselves, not a fixture: what has to render is what is actually written.
|
|
for (const name of ['quickstart', 'rules', 'home-deck', 'mainline-deck', 'components']) {
|
|
const src = readFileSync(join(import.meta.dirname, '..', 'docs', `${name}.md`), 'utf8');
|
|
const { html: out, headings } = renderMarkdown(src);
|
|
assert.ok(headings.length > 2, `${name}.md rendered only ${headings.length} headings`);
|
|
assert.ok(out.length > 1000, `${name}.md rendered almost nothing`);
|
|
// No pipe table survives as text — that would mean a table failed to parse.
|
|
const stripped = out.replace(/<[^>]+>/g, '');
|
|
assert.ok(
|
|
!/^\s*\|\s*---/m.test(stripped),
|
|
`${name}.md has a table the renderer did not recognise`,
|
|
);
|
|
assert.ok(!/BEGIN CARDS/.test(out), `${name}.md leaked a build marker onto the page`);
|
|
}
|
|
});
|
|
});
|