@serverkgg/bridge 0.1.1 → 0.1.3

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@serverkgg/bridge",
-	"version": "0.1.1",
+	"version": "0.1.3",
 	"description": "Serverk game bridge and driver runtime, wire protocol, manifest schema, and maintainer CLI",
 	"homepage": "https://serverk.gg",
 	"license": "SEE LICENSE IN LICENSE.md",
@@ -14,7 +14,8 @@
 		"serverk-bridge": "src/cli/main.ts"
 	},
 	"files": [
-		"src"
+		"src",
+		"rules"
 	],
 	"exports": {
 		".": "./src/main.ts",

package/rules/package.md

@@ -0,0 +1,60 @@
+# Serverk package
+
+Generated by `serverk-bridge rules` from the pinned `@serverkgg/bridge`. Do not edit it here: every install rewrites it. The source is `modules/bridge/rules/package.md` in `serverk`.
+
+## This repository
+
+One Serverk package (a game or a bot runtime) on `@serverkgg/bridge`, with no pipeline: validated with `serverk-bridge validate`, released only from `../serverk`, the monorepo beside it.
+
+- When `../serverk` exists, read `../serverk/CLAUDE.md` first; its instructions apply here.
+- Before matching work, read its skill in `../serverk/.claude/skills/`: `serverk-copy` (every string a customer reads), `serverk-game-package` (a new game, its manifest, image or control channel), `serverk-game-art` (logo and banner), `serverk-deploy` (publishing to dev in `dev.md`, releasing in `bridge.md`).
+
+## The driver
+
+- `src/main.ts` is only `await serve(driver)`. `src/driver.ts` exports `driver: BridgeDriver` with the slots in this order (`install`, `lifecycle`, `events`, `query`, `backup`, `announce`, `pending`, `setup`, `terminal`, `panel`), then the panel modules under `modules`.
+- Key a bridge-owned module by its constant (`[RCON_ACCESS_MODULE]: rconAccess`); any other key is its export name, which a panel section names (`module: "settings"`).
+- Type each module you write with its `Bridge` alias and `kind` from `BridgeKind` (`export const settings: Bridge.Settings = { kind: BridgeKind.Settings, ... }`), never a `Bridge*Module` name. Export a helper-built module as returned (`export const deploy = createBotDeployModule()`).
+
+## `src/` folders
+
+- A driver slot lives in the folder of its name, exported under that name.
+- Group panel modules by kind in `actions/`, `collections/`, `details/`, `options/` and `settings/` (`Bridge.Actions`, `Bridge.Collection`, `Bridge.Detail`, `Bridge.Options`, `Bridge.Settings`). A feature spanning kinds gets its own folder (`worlds/`, `modpacks/`).
+- `shared/` holds parsers, file formats and control-channel calls several folders use; `testing/` holds test fakes. Every folder has an `index.ts` re-exporting each sibling, and other folders import only that barrel.
+- Use a bridge helper (`@serverkgg/bridge/steam`, `rcon`, `install`, `catalogs`, `presence`, `probes`, `utils`, `wine`, `net`, `bot`) before writing one. A helper a second package needs moves into the bridge, not a copy.
+
+## Code
+
+- Functions are `const` arrows. No classes, `function` declarations, default exports or comments.
+- Touch the server only through `Bridge.Context` (`files`, `exec`, `net`, `codec`, `rcon`, `variable`, `secret`), never `node:fs`, `fetch`, `Bun.spawn` or `process.env`.
+- Log through `context.log`: a lowercase message, details second. Never `console` in `src/`.
+- Refuse a customer's request with `new BridgeUserError({ ar, en })`; a plain lowercase English `Error` is for faults the customer cannot fix.
+- A timeout, limit, pattern or file name is a `SCREAMING_SNAKE_CASE` constant (`STOP_TIMEOUT_SECONDS`), exported when a test or another folder needs it.
+
+## Panel, setup and events
+
+- A game declares each tab as `const <name>Tab: Bridge.Tab`, listed in `export const panel: Bridge.Panel`; a bot's panel is the tab `createBotPanel()` from `@serverkgg/bridge/bot` builds. Reusable copy is a `Bridge.Text` constant.
+- Every label, title, help text and message is `{ ar, en }`, Arabic first.
+- A settings form targets `BridgeFormTarget.Settings` and module `settings`, with a one-sentence `help` and `restartHint: true`.
+- In a game, the roster and moderation tables go on `place: BridgePlace.Players`, metrics on `BridgePlace.Overview`.
+- A setup `Form` step points at an existing panel form by `tab`, `section` and field keys, never redeclaring a field; `Form` and `Open` steps state `required`. A game's setup ends with the `Open` step `invite` targeting `GuideOpenTab.Access`.
+- Emit event names from `BridgeEventName`, never strings, and list every event raised from code in `emits` (`patterns: []` when none match the log). A bot matches deployments with `DEPLOYMENT_EVENT_PATTERN`.
+- `install.run` runs on every boot: keep it idempotent with `readStamp` and `writeStamp` (`@serverkgg/bridge/install`), the stamp in `files.protected`. A game protects the default `.serverk-install.json`; a bot writes `.serverk/install.json` (`INSTALL_STAMP`, `shared/botRuntime.ts`) and protects `.serverk/**`.
+
+## `serverk.yml`
+
+- Line one is `# yaml-language-server: $schema=./serverk.schema.json`; rerun `bun run schema` when the bridge pin moves.
+- Write `ar:` before `en:` in every localized field.
+- `secrets` are platform API keys the game can use, `required: false`; customer settings are a panel form, and a bot customer's own values go through `environment.vault`.
+- Change an outcome by changing the declared value (`reset.keep: []`, `backup.only`), never by asking the platform to ignore a field.
+
+## Guides and image
+
+- A guide is `guides/<id>/ar.md` and `en.md`, starting at `##` with no `#` title; its title and summary live in `serverk.yml`. Every package has a `getting-started` guide tagged `start`.
+- `image/Dockerfile` keeps the shared contract: `ARG BRIDGE_VERSION` with `LABEL gg.serverk.bridge.version`, a `container` user with uid 1000 and home `/home/container`, `COPY dist/bridge /serverk/bridge`, `ENTRYPOINT ["/usr/bin/tini", "--", "/serverk/bridge"]`, and no `USER` line. Game files download at install, a bot's code arrives with a deploy; neither is baked in.
+- The outermost start wrapper ends in `exec`, with no `setsid`, `nohup` or double fork. Why: the bridge checks the uid of the pid it spawned, and stop escalation signals the process group, so an orphan outside it keeps the port.
+
+## Tests and the pin
+
+- Tests sit beside the file as `<name>.test.ts`, use `it` from `bun:test` (never `test`), and are named as a plain statement of behaviour, never `should`. `driver.test.ts` asserts the assembled driver's tab and section wiring, and in a game its setup targets and event names.
+- `package.json` pins `@serverkgg/bridge` to an exact published version, bumped only after a bridge publish from `serverk`, never within other work. `serverk-bridge link` points `node_modules` at `../serverk/modules/bridge` without touching the pin.
+- The two bot packages stay identical except where the runtime differs (`shared/botRuntime.ts`, `shared/botProcess.ts`, `terminal/terminal.ts`, the image's launcher and `Dockerfile`, and the tests asserting those). A change to any other file lands in both; behaviour both need belongs in `@serverkgg/bridge/bot`.

package/src/blog/articleFrontmatter.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, test } from "bun:test";
+import { describe, expect, it } from "bun:test";
 import { parseArticleFrontmatter, parseArticleTranslationFrontmatter, splitFrontmatter } from "./articleFrontmatter";
 
 const article = (front: string, body = "## عنوان\n\nنص.\n") => `---\n${front}\n---\n\n${body}`;
@@ -10,7 +10,7 @@ tags:
   - minecraft`;
 
 describe("splitFrontmatter", () => {
-	test("blanks the frontmatter lines so body line numbers stay true", () => {
+	it("blanks the frontmatter lines so body line numbers stay true", () => {
 		const split = splitFrontmatter("blog/x/ar.md", article(VALID));
 
 		expect(split.body.split("\n").slice(0, 7)).toEqual([
@@ -25,17 +25,17 @@ describe("splitFrontmatter", () => {
 		expect(split.body).toContain("## عنوان");
 	});
 
-	test("rejects a file that does not open with a fence", () => {
+	it("rejects a file that does not open with a fence", () => {
 		expect(() => splitFrontmatter("blog/x/ar.md", "## عنوان\n")).toThrow('must open with a "---" frontmatter block');
 	});
 
-	test("rejects a frontmatter block that is never closed", () => {
+	it("rejects a frontmatter block that is never closed", () => {
 		expect(() => splitFrontmatter("blog/x/ar.md", `---\n${VALID}\n`)).toThrow("never closed");
 	});
 });
 
 describe("parseArticleFrontmatter", () => {
-	test("reads every declared field", () => {
+	it("reads every declared field", () => {
 		const parsed = parseArticleFrontmatter(
 			"blog/x/ar.md",
 			article(`${VALID}\nupdatedAt: 2026-09-06\nvideo: dQw4w9WgXcQ`),
@@ -50,7 +50,7 @@ describe("parseArticleFrontmatter", () => {
 		]);
 	});
 
-	test("defaults tags to an empty list", () => {
+	it("defaults tags to an empty list", () => {
 		const parsed = parseArticleFrontmatter(
 			"blog/x/ar.md",
 			article("title: عنوان\nsummary: ملخص.\npublishedAt: 2026-09-05"),
@@ -60,13 +60,13 @@ describe("parseArticleFrontmatter", () => {
 		expect(parsed.front.updatedAt).toBeUndefined();
 	});
 
-	test("rejects a missing title", () => {
+	it("rejects a missing title", () => {
 		expect(() => parseArticleFrontmatter("blog/x/ar.md", article("summary: ملخص.\npublishedAt: 2026-09-05"))).toThrow(
 			"frontmatter is invalid",
 		);
 	});
 
-	test("rejects a publishedAt that is not a calendar date", () => {
+	it("rejects a publishedAt that is not a calendar date", () => {
 		expect(() =>
 			parseArticleFrontmatter(
 				"blog/x/ar.md",
@@ -75,7 +75,7 @@ describe("parseArticleFrontmatter", () => {
 		).toThrow("must be a calendar date");
 	});
 
-	test("rejects an unknown key", () => {
+	it("rejects an unknown key", () => {
 		expect(() => parseArticleFrontmatter("blog/x/ar.md", article(`${VALID}\nauthor: mohammed`))).toThrow(
 			"frontmatter is invalid",
 		);
@@ -83,7 +83,7 @@ describe("parseArticleFrontmatter", () => {
 });
 
 describe("parseArticleTranslationFrontmatter", () => {
-	test("lets a translation omit the language-independent fields", () => {
+	it("lets a translation omit the language-independent fields", () => {
 		const parsed = parseArticleTranslationFrontmatter(
 			"blog/x/en.md",
 			article("title: How to make a server\nsummary: The three ways."),

package/src/blog/compile.test.ts

@@ -1,4 +1,4 @@
-import { afterEach, beforeEach, describe, expect, test } from "bun:test";
+import { afterEach, beforeEach, describe, expect, it } from "bun:test";
 import { mkdirSync, mkdtempSync, rmSync, writeFileSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
@@ -33,11 +33,11 @@ describe("compileArticles", () => {
 		});
 	});
 
-	test("returns nothing when the package has no blog directory", () => {
+	it("returns nothing when the package has no blog directory", () => {
 		expect(compile().drafts).toEqual([]);
 	});
 
-	test("compiles both languages into one article", () => {
+	it("compiles both languages into one article", () => {
 		write(
 			"make-a-server",
 			article("title: كيف تسوي سيرفر\nsummary: ثلاث طرق.\npublishedAt: 2026-09-05\ntags:\n  - minecraft", "عنوان"),
@@ -63,7 +63,7 @@ describe("compileArticles", () => {
 		});
 	});
 
-	test("records both markdown files so the bundle digest covers them", () => {
+	it("records both markdown files so the bundle digest covers them", () => {
 		write(
 			"one",
 			article("title: عنوان\nsummary: ملخص.\npublishedAt: 2026-09-05", "عنوان"),
@@ -76,7 +76,7 @@ describe("compileArticles", () => {
 		]);
 	});
 
-	test("orders newest first and breaks ties by id", () => {
+	it("orders newest first and breaks ties by id", () => {
 		write(
 			"beta",
 			article("title: ب\nsummary: ملخص.\npublishedAt: 2026-09-05", "عنوان"),
@@ -100,7 +100,7 @@ describe("compileArticles", () => {
 		]);
 	});
 
-	test("rejects a translation that contradicts a language-independent field", () => {
+	it("rejects a translation that contradicts a language-independent field", () => {
 		write(
 			"one",
 			article("title: عنوان\nsummary: ملخص.\npublishedAt: 2026-09-05", "عنوان"),
@@ -110,7 +110,7 @@ describe("compileArticles", () => {
 		expect(compile).toThrow("a language-independent field must match");
 	});
 
-	test("accepts a translation that repeats a language-independent field with the same value", () => {
+	it("accepts a translation that repeats a language-independent field with the same value", () => {
 		write(
 			"one",
 			article("title: عنوان\nsummary: ملخص.\npublishedAt: 2026-09-05\ntags:\n  - minecraft", "عنوان"),
@@ -120,7 +120,7 @@ describe("compileArticles", () => {
 		expect(compile().drafts).toHaveLength(1);
 	});
 
-	test("reports a missing english translation", () => {
+	it("reports a missing english translation", () => {
 		mkdirSync(join(root, "blog", "one"), {
 			recursive: true,
 		});
@@ -132,7 +132,7 @@ describe("compileArticles", () => {
 		expect(compile).toThrow("missing article body blog/one/en.md");
 	});
 
-	test("resolves into a compiled article the contract accepts", () => {
+	it("resolves into a compiled article the contract accepts", () => {
 		write(
 			"one",
 			article("title: عنوان\nsummary: ملخص.\npublishedAt: 2026-09-05\nupdatedAt: 2026-09-06", "عنوان"),

package/src/bot/botDeploy.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, test } from "bun:test";
+import { describe, expect, it } from "bun:test";
 import type { BridgeValues } from "../core";
 import { BridgeEventName } from "../protocol";
 import {
@@ -121,7 +121,7 @@ const liveState = (deployment: string, source: BotDeploymentSource = BotDeployme
 };
 
 describe("deploymentLine", () => {
-	test("announces a ready deployment with the short commit", () => {
+	it("announces a ready deployment with the short commit", () => {
 		expect(
 			deploymentLine({
 				deployment: "dep_a1b2c3d4",
@@ -131,7 +131,7 @@ describe("deploymentLine", () => {
 		).toBe("serverk: deployment dep_a1b2c3d4 ready at 0123456789ab");
 	});
 
-	test("announces a ready deployment with no commit for files and snapshots", () => {
+	it("announces a ready deployment with no commit for files and snapshots", () => {
 		expect(
 			deploymentLine({
 				deployment: "dep_a1b2c3d4",
@@ -141,7 +141,7 @@ describe("deploymentLine", () => {
 		).toBe("serverk: deployment dep_a1b2c3d4 ready");
 	});
 
-	test("names the stage and one line of detail for a failure", () => {
+	it("names the stage and one line of detail for a failure", () => {
 		expect(
 			deploymentLine({
 				deployment: "dep_a1b2c3d4",
@@ -154,7 +154,7 @@ describe("deploymentLine", () => {
 		);
 	});
 
-	test("round-trips through the line pattern", () => {
+	it("round-trips through the line pattern", () => {
 		const ready = DEPLOYMENT_LINE.exec(
 			deploymentLine({
 				deployment: "dep_a1b2c3d4",
@@ -186,7 +186,7 @@ describe("deploymentLine", () => {
 		});
 	});
 
-	test("cuts the detail at 200 characters, counting an emoji as one the way the Python launcher does", () => {
+	it("cuts the detail at 200 characters, counting an emoji as one the way the Python launcher does", () => {
 		const line = deploymentLine({
 			deployment: "dep_a1b2c3d4",
 			outcome: BotDeploymentOutcome.Failed,
@@ -198,12 +198,12 @@ describe("deploymentLine", () => {
 		expect(DEPLOYMENT_LINE.exec(line)?.groups?.detail).toBe("🤖".repeat(200));
 	});
 
-	test("reads only deployment ids of at most 64 characters", () => {
+	it("reads only deployment ids of at most 64 characters", () => {
 		expect(DEPLOYMENT_LINE.test(`serverk: deployment ${"d".repeat(64)} ready`)).toBe(true);
 		expect(DEPLOYMENT_LINE.test(`serverk: deployment ${"d".repeat(65)} ready`)).toBe(false);
 	});
 
-	test("is reported as a server update tagged as a deployment", () => {
+	it("is reported as a server update tagged as a deployment", () => {
 		expect(DEPLOYMENT_EVENT_PATTERN.emit).toBe(BridgeEventName.ServerUpdated);
 		expect(DEPLOYMENT_EVENT_PATTERN.payload).toEqual({
 			kind: "deployment",
@@ -214,7 +214,7 @@ describe("deploymentLine", () => {
 });
 
 describe("requestDeploy", () => {
-	test("writes the request as JSON and restarts a running bot", async () => {
+	it("writes the request as JSON and restarts a running bot", async () => {
 		const { context, stored, events } = deployContext(true);
 
 		await requestDeploy(context, {
@@ -239,7 +239,7 @@ describe("requestDeploy", () => {
 		expect(Number.isNaN(Date.parse(markerOf(stored).requestedAt))).toBe(false);
 	});
 
-	test("records the request without a restart while the bot is stopped", async () => {
+	it("records the request without a restart while the bot is stopped", async () => {
 		const { context, stored, events } = deployContext(false);
 
 		await requestDeploy(context, {
@@ -259,7 +259,7 @@ describe("requestDeploy", () => {
 		});
 	});
 
-	test("does nothing for a deployment that is already live and reports it again", async () => {
+	it("does nothing for a deployment that is already live and reports it again", async () => {
 		const { context, stored, events, emitted } = deployContext(true, liveState("dep_a1b2c3d4"));
 
 		await requestDeploy(context, {
@@ -286,7 +286,7 @@ describe("requestDeploy", () => {
 		]);
 	});
 
-	test("reports no commit again for a live deployment of uploaded code", async () => {
+	it("reports no commit again for a live deployment of uploaded code", async () => {
 		const { context, emitted } = deployContext(true, liveState("dep_a1b2c3d4", BotDeploymentSource.Snapshot));
 
 		await requestDeploy(context, {
@@ -302,7 +302,7 @@ describe("requestDeploy", () => {
 		});
 	});
 
-	test("replaces a request the bot has not taken yet", async () => {
+	it("replaces a request the bot has not taken yet", async () => {
 		const { context, stored } = deployContext(false, {
 			[DEPLOY_MARKER]: "{}",
 		});
@@ -317,7 +317,7 @@ describe("requestDeploy", () => {
 		expect(markerOf(stored).deployment).toBe("dep_newer000");
 	});
 
-	test("still reinstalls a deployment that is already live", async () => {
+	it("still reinstalls a deployment that is already live", async () => {
 		const { context, events } = deployContext(true, liveState("dep_a1b2c3d4"));
 
 		await requestDeploy(context, {
@@ -330,7 +330,7 @@ describe("requestDeploy", () => {
 		expect(events.at(-1)).toBe("restart");
 	});
 
-	test("runs a new deployment when another one is live", async () => {
+	it("runs a new deployment when another one is live", async () => {
 		const { context, events } = deployContext(true, liveState("dep_old00000"));
 
 		await requestDeploy(context, {
@@ -345,7 +345,7 @@ describe("requestDeploy", () => {
 });
 
 describe("deployPending", () => {
-	test("is true only while a running bot has not taken the marker yet", async () => {
+	it("is true only while a running bot has not taken the marker yet", async () => {
 		const running = deployContext(true);
 
 		expect(await deployPending(running.context)).toBe(false);

package/src/bot/botEnvironment.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, test } from "bun:test";
+import { describe, expect, it } from "bun:test";
 import {
 	botProcessEnvironment,
 	customerEnvironment,
@@ -36,7 +36,7 @@ const config = (overrides: Partial<BotConfig> = {}): BotConfig => ({
 });
 
 describe("customerEnvironment", () => {
-	test("hands the process every vault key and nothing the platform or the runtime owns", () => {
+	it("hands the process every vault key and nothing the platform or the runtime owns", () => {
 		expect(
 			customerEnvironment(
 				{
@@ -71,7 +71,7 @@ describe("customerEnvironment", () => {
 		});
 	});
 
-	test("lets a secret win over a plain value under the same key", () => {
+	it("lets a secret win over a plain value under the same key", () => {
 		expect(
 			customerEnvironment(
 				{
@@ -88,7 +88,7 @@ describe("customerEnvironment", () => {
 });
 
 describe("launcherEnvironment", () => {
-	test("maps the platform-owned keys onto the launcher internals", () => {
+	it("maps the platform-owned keys onto the launcher internals", () => {
 		expect(
 			launcherEnvironment(
 				config({
@@ -116,7 +116,7 @@ describe("launcherEnvironment", () => {
 		});
 	});
 
-	test("keeps the git keys out when no repository is linked", () => {
+	it("keeps the git keys out when no repository is linked", () => {
 		expect(
 			launcherEnvironment(
 				config({
@@ -132,7 +132,7 @@ describe("launcherEnvironment", () => {
 		});
 	});
 
-	test("owns only SERVERK_ names the launcher strips before the bot starts", () => {
+	it("owns only SERVERK_ names the launcher strips before the bot starts", () => {
 		expect(LAUNCHER_KEYS.every((key) => key.startsWith("SERVERK_"))).toBe(true);
 		expect(LAUNCHER_KEYS).toContain(LAUNCHER_REPO_REF);
 		expect(LAUNCHER_KEYS).toContain(LAUNCHER_INSTALL_COMMAND);
@@ -141,7 +141,7 @@ describe("launcherEnvironment", () => {
 });
 
 describe("platformEnvironment", () => {
-	test("marks the process as a Serverk bot and names the platform when known", () => {
+	it("marks the process as a Serverk bot and names the platform when known", () => {
 		expect(platformEnvironment("abc", BotPlatform.Discord)).toEqual({
 			SERVERK_BOT: "1",
 			SERVERK_SERVER_CODE: "abc",
@@ -155,7 +155,7 @@ describe("platformEnvironment", () => {
 });
 
 describe("botProcessEnvironment", () => {
-	test("layers tool defaults under the customer, then the launcher and the platform", () => {
+	it("layers tool defaults under the customer, then the launcher and the platform", () => {
 		expect(
 			botProcessEnvironment({
 				tool: {

package/src/bot/botIdentity.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, test } from "bun:test";
+import { describe, expect, it } from "bun:test";
 import {
 	botNameOf,
 	IDENTITY_CACHE_FILE,
@@ -40,7 +40,7 @@ const memoryFiles = (initial: Record<string, string> = {}) => {
 };
 
 describe("LOGIN_LINE", () => {
-	test("reads the name from the login lines the popular libraries and our templates print", () => {
+	it("reads the name from the login lines the popular libraries and our templates print", () => {
 		const names = [
 			"Logged in as MyBot#1234, /ping is registered",
 			"Logged in as MyBot (ID: 1136470478027245683)",
@@ -56,19 +56,19 @@ describe("LOGIN_LINE", () => {
 		]);
 	});
 
-	test("ignores lines that only mention logging", () => {
+	it("ignores lines that only mention logging", () => {
 		expect(LOGIN_LINE.test("serverk: bot process starting")).toBe(false);
 		expect(LOGIN_LINE.test("user logged out")).toBe(false);
 	});
 });
 
 describe("identityFromToken", () => {
-	test("reads a Telegram bot id and a Discord application id without the network", () => {
+	it("reads a Telegram bot id and a Discord application id without the network", () => {
 		expect(identityFromToken(BotPlatform.Telegram, TELEGRAM_TOKEN)).toBe("7123456789");
 		expect(identityFromToken(BotPlatform.Discord, DISCORD_TOKEN)).toBe(DISCORD_ID);
 	});
 
-	test("refuses anything that is not shaped like the platform's token", () => {
+	it("refuses anything that is not shaped like the platform's token", () => {
 		expect(identityFromToken(BotPlatform.Discord, JWT)).toBeNull();
 		expect(identityFromToken(BotPlatform.Discord, TELEGRAM_TOKEN)).toBeNull();
 		expect(identityFromToken(BotPlatform.Telegram, DISCORD_TOKEN)).toBeNull();
@@ -77,7 +77,7 @@ describe("identityFromToken", () => {
 });
 
 describe("the name cache", () => {
-	test("keeps only the name the bot printed, per platform", async () => {
+	it("keeps only the name the bot printed, per platform", async () => {
 		const { files, stored } = memoryFiles();
 
 		await writeIdentityCache(files, {
@@ -95,7 +95,7 @@ describe("the name cache", () => {
 		expect(await readIdentityCache(files, BotPlatform.Discord)).toBeNull();
 	});
 
-	test("reads nothing from a broken or older cache", async () => {
+	it("reads nothing from a broken or older cache", async () => {
 		expect(
 			await readIdentityCache(
 				memoryFiles({
@@ -119,7 +119,7 @@ describe("the name cache", () => {
 });
 
 describe("inviteLink", () => {
-	test("invites a Discord bot by id and opens a Telegram bot by name", () => {
+	it("invites a Discord bot by id and opens a Telegram bot by name", () => {
 		expect(
 			inviteLink({
 				platform: BotPlatform.Discord,
@@ -138,7 +138,7 @@ describe("inviteLink", () => {
 		).toBe("https://t.me/serverk_test_bot");
 	});
 
-	test("offers nothing it cannot build", () => {
+	it("offers nothing it cannot build", () => {
 		expect(
 			inviteLink({
 				platform: BotPlatform.Discord,
@@ -159,7 +159,7 @@ describe("inviteLink", () => {
 });
 
 describe("watchLoginLine", () => {
-	test("remembers the first login name and stops listening", async () => {
+	it("remembers the first login name and stops listening", async () => {
 		const { files, stored } = memoryFiles();
 		const watchers: ((match: RegExpMatchArray) => void)[] = [];
 		let unsubscribed = 0;
@@ -210,7 +210,7 @@ describe("watchLoginLine", () => {
 });
 
 describe("resolveBotIdentity", () => {
-	test("combines the inferred platform, the id from the token and the remembered name", async () => {
+	it("combines the inferred platform, the id from the token and the remembered name", async () => {
 		const { files } = memoryFiles({
 			[IDENTITY_CACHE_FILE]: JSON.stringify({
 				platform: BotPlatform.Discord,
@@ -237,7 +237,7 @@ describe("resolveBotIdentity", () => {
 		});
 	});
 
-	test("knows nothing about a bot with no platform hint", async () => {
+	it("knows nothing about a bot with no platform hint", async () => {
 		expect(
 			await resolveBotIdentity({
 				files: memoryFiles().files,

package/src/bot/botPanel.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, test } from "bun:test";
+import { describe, expect, it } from "bun:test";
 import { BridgeKind, BridgeUserError, type BridgeValues } from "../core";
 import { BridgeLayout, BridgePlace } from "../panel";
 import { bridgePanelSchema } from "../wire";
@@ -19,7 +19,7 @@ import {
 const SHA = "0123456789abcdef0123456789abcdef01234567";
 
 describe("createBotPanel", () => {
-	test("is one bot tab holding only the identity card on the overview", () => {
+	it("is one bot tab holding only the identity card on the overview", () => {
 		const tab = createBotPanel();
 
 		expect(tab.id).toBe(BOT_TAB);
@@ -33,7 +33,7 @@ describe("createBotPanel", () => {
 		expect(identity?.layout === BridgeLayout.Detail && identity.module).toBe(BOT_IDENTITY_MODULE);
 	});
 
-	test("parses through the panel mirror the agent and the api read", () => {
+	it("parses through the panel mirror the agent and the api read", () => {
 		expect(
 			bridgePanelSchema.safeParse({
 				tabs: [
@@ -43,7 +43,7 @@ describe("createBotPanel", () => {
 		).toBe(true);
 	});
 
-	test("names the deploy module the platform invokes", () => {
+	it("names the deploy module the platform invokes", () => {
 		expect(BOT_DEPLOY_MODULE).toBe("botDeploy");
 		expect(Object.values(BotDeployAction).map(String)).toEqual([
 			"redeploy",
@@ -53,7 +53,7 @@ describe("createBotPanel", () => {
 });
 
 describe("deployRequestOf", () => {
-	test("takes the deployment, the pinned ref and the reinstall flag from the invoke values", () => {
+	it("takes the deployment, the pinned ref and the reinstall flag from the invoke values", () => {
 		expect(
 			deployRequestOf(
 				{
@@ -71,7 +71,7 @@ describe("deployRequestOf", () => {
 		});
 	});
 
-	test("treats a bare press as a redeploy of whatever is configured", () => {
+	it("treats a bare press as a redeploy of whatever is configured", () => {
 		expect(deployRequestOf({}, false)).toEqual({
 			deployment: null,
 			ref: null,
@@ -81,7 +81,7 @@ describe("deployRequestOf", () => {
 		expect(deployRequestOf({}, true).reinstall).toBe(true);
 	});
 
-	test("refuses a short ref, a bad id and a ref beside a snapshot", () => {
+	it("refuses a short ref, a bad id and a ref beside a snapshot", () => {
 		const refusals: BridgeValues[] = [
 			{
 				ref: "0123456",
@@ -111,7 +111,7 @@ describe("deployRequestOf", () => {
 });
 
 describe("the bot modules", () => {
-	test("read the identity while the bot is stopped and refresh it every minute", () => {
+	it("read the identity while the bot is stopped and refresh it every minute", () => {
 		const read = async () => null;
 
 		expect(createBotIdentityModule(read)).toEqual({
@@ -174,7 +174,7 @@ describe("the bot modules", () => {
 		};
 	};
 
-	test("deploy while the bot is stopped too, so the platform can start the new code", async () => {
+	it("deploy while the bot is stopped too, so the platform can start the new code", async () => {
 		const module = createBotDeployModule();
 		const { context, stored, restarts } = deployContext(false);
 
@@ -193,7 +193,7 @@ describe("the bot modules", () => {
 		expect(restarts()).toBe(0);
 	});
 
-	test("reinstall forces the flag and restarts a running bot", async () => {
+	it("reinstall forces the flag and restarts a running bot", async () => {
 		const module = createBotDeployModule();
 		const { context, stored, restarts } = deployContext(true);
 
@@ -205,7 +205,7 @@ describe("the bot modules", () => {
 		expect(restarts()).toBe(1);
 	});
 
-	test("refuses a second press while a running bot has not taken the first", async () => {
+	it("refuses a second press while a running bot has not taken the first", async () => {
 		const module = createBotDeployModule();
 		const { context } = deployContext(true, [
 			DEPLOY_MARKER,
@@ -213,4 +213,34 @@ describe("the bot modules", () => {
 
 		await expect(module.actions[BotDeployAction.Redeploy]?.(context, {})).rejects.toThrow(BridgeUserError);
 	});
+
+	it("greys out both deploy buttons with the refusal's words while a running bot has not taken a deploy", async () => {
+		const module = createBotDeployModule();
+		const pending = deployContext(true, [
+			DEPLOY_MARKER,
+		]);
+
+		const unavailable = await module.unavailable?.(pending.context);
+
+		expect(Object.keys(unavailable ?? {}).sort()).toEqual([
+			BotDeployAction.Redeploy,
+			BotDeployAction.Reinstall,
+		]);
+		await expect(module.actions[BotDeployAction.Redeploy]?.(pending.context, {})).rejects.toThrow(
+			unavailable?.[BotDeployAction.Redeploy]?.en,
+		);
+	});
+
+	it("offers both deploy buttons when nothing is pending or the bot is stopped", async () => {
+		const module = createBotDeployModule();
+
+		expect(await module.unavailable?.(deployContext(true).context)).toEqual({});
+		expect(
+			await module.unavailable?.(
+				deployContext(false, [
+					DEPLOY_MARKER,
+				]).context,
+			),
+		).toEqual({});
+	});
 });