@serverkgg/bridge 0.1.2 → 0.1.4

package/src/manifest/manifest.test.ts

@@ -1,7 +1,8 @@
-import { describe, expect, test } from "bun:test";
+import { describe, expect, it } from "bun:test";
 import { z } from "zod";
 import { PORT_BLOCK_MAX_COUNT, PortJoin, PortProtocol } from "../core";
 import { GameCategory } from "./category";
+import { ManifestExtensionKind, ManifestExtensionSource } from "./extension";
 import { buildJsonSchema } from "./jsonSchema";
 import { PackageKind } from "./kind";
 import {
@@ -174,7 +175,7 @@ const withPortActiveWhen = (activeWhen: unknown) => {
 };
 
 describe("defaults", () => {
-	test("fills in the optional sections so consumers never guard", () => {
+	it("fills in the optional sections so consumers never guard", () => {
 		const manifest = manifestSchema.parse(base());
 
 		expect(manifest.backup).toEqual({
@@ -195,11 +196,127 @@ describe("defaults", () => {
 		expect(manifest.container.ports[0]?.hint).toBeUndefined();
 		expect(manifest.container.ports[0]?.join).toBe(PortJoin.Combined);
 		expect(manifest.container.ports[0]?.activeWhen).toEqual([]);
+		expect(manifest.metadata.extensions).toEqual([]);
+	});
+});
+
+describe("extensions", () => {
+	const withExtensions = (extensions: unknown) => {
+		const manifest = base();
+
+		return {
+			...manifest,
+			metadata: {
+				...manifest.metadata,
+				extensions,
+			},
+		};
+	};
+
+	it("carry each kind with its catalogs, and no catalog when it is added by hand", () => {
+		const manifest = manifestSchema.parse(
+			withExtensions([
+				{
+					kind: ManifestExtensionKind.Mods,
+					sources: [
+						ManifestExtensionSource.Modrinth,
+						ManifestExtensionSource.CurseForge,
+					],
+				},
+				{
+					kind: ManifestExtensionKind.Plugins,
+				},
+			]),
+		);
+
+		expect(manifest.metadata.extensions).toEqual([
+			{
+				kind: ManifestExtensionKind.Mods,
+				sources: [
+					ManifestExtensionSource.Modrinth,
+					ManifestExtensionSource.CurseForge,
+				],
+			},
+			{
+				kind: ManifestExtensionKind.Plugins,
+				sources: [],
+			},
+		]);
+	});
+
+	it("refuse a kind declared twice", () => {
+		expect(
+			failure(
+				withExtensions([
+					{
+						kind: ManifestExtensionKind.Mods,
+					},
+					{
+						kind: ManifestExtensionKind.Mods,
+						sources: [
+							ManifestExtensionSource.CurseForge,
+						],
+					},
+				]),
+			),
+		).toEqual([
+			{
+				path: "metadata.extensions.1.kind",
+				message: 'extension "mods" is declared twice',
+			},
+		]);
+	});
+
+	it("refuse a catalog listed twice for one kind", () => {
+		expect(
+			failure(
+				withExtensions([
+					{
+						kind: ManifestExtensionKind.Mods,
+						sources: [
+							ManifestExtensionSource.Modrinth,
+							ManifestExtensionSource.Modrinth,
+						],
+					},
+				]),
+			),
+		).toEqual([
+			{
+				path: "metadata.extensions.0.sources.1",
+				message: 'source "modrinth" is listed twice for mods',
+			},
+		]);
+	});
+
+	it("refuse a catalog the platform does not know", () => {
+		expect(
+			manifestSchema.safeParse(
+				withExtensions([
+					{
+						kind: ManifestExtensionKind.Mods,
+						sources: [
+							"steam-workshop",
+						],
+					},
+				]),
+			).success,
+		).toBe(false);
+	});
+
+	it("render into the JSON schema", () => {
+		const rendered = JSON.stringify(buildJsonSchema());
+
+		for (const value of [
+			...Object.values(ManifestExtensionKind),
+			...Object.values(ManifestExtensionSource),
+		]) {
+			expect(rendered).toContain(`"${value}"`);
+		}
 	});
 });
 
 describe("port presentation", () => {
-	test("keeps the localized name, hint and join style a port declares", () => {
+	it("keeps the localized name, hint and join style a port declares", () => {
 		const manifest = manifestSchema.parse(
 			withPort({
 				name: {
@@ -222,7 +339,7 @@ describe("port presentation", () => {
 		expect(manifest.container.ports[0]?.join).toBe(PortJoin.Separate);
 	});
 
-	test("rejects a port name missing one language", () => {
+	it("rejects a port name missing one language", () => {
 		expect(
 			failure(
 				withPort({
@@ -241,7 +358,7 @@ describe("port presentation", () => {
 });
 
 describe("port srv", () => {
-	test("keeps the service label a port declares", () => {
+	it("keeps the service label a port declares", () => {
 		const manifest = manifestSchema.parse(
 			withPortSrv({
 				service: "_minecraft",
@@ -253,7 +370,7 @@ describe("port srv", () => {
 		});
 	});
 
-	test("rejects a service label without its underscore", () => {
+	it("rejects a service label without its underscore", () => {
 		expect(
 			failure(
 				withPortSrv({
@@ -270,7 +387,7 @@ describe("port srv", () => {
 });
 
 describe("port activeWhen", () => {
-	test("keeps every condition a port declares", () => {
+	it("keeps every condition a port declares", () => {
 		const manifest = manifestSchema.parse(
 			withPortActiveWhen([
 				{
@@ -306,7 +423,7 @@ describe("port activeWhen", () => {
 		]);
 	});
 
-	test("rejects a condition that declares neither values nor empty", () => {
+	it("rejects a condition that declares neither values nor empty", () => {
 		expect(
 			failure(
 				withPortActiveWhen([
@@ -323,7 +440,7 @@ describe("port activeWhen", () => {
 		]);
 	});
 
-	test("rejects a condition whose values list is empty", () => {
+	it("rejects a condition whose values list is empty", () => {
 		expect(
 			failure(
 				withPortActiveWhen([
@@ -341,7 +458,7 @@ describe("port activeWhen", () => {
 		]);
 	});
 
-	test("rejects a fifth condition", () => {
+	it("rejects a fifth condition", () => {
 		expect(
 			failure(
 				withPortActiveWhen(
@@ -361,7 +478,7 @@ describe("port activeWhen", () => {
 		]);
 	});
 
-	test("rejects a variable that is not an environment key", () => {
+	it("rejects a variable that is not an environment key", () => {
 		expect(
 			failure(
 				withPortActiveWhen([
@@ -381,7 +498,7 @@ describe("port activeWhen", () => {
 });
 
 describe("cross-entry checks", () => {
-	test("rejects an image that does not match metadata.id", () => {
+	it("rejects an image that does not match metadata.id", () => {
 		const manifest = base();
 
 		manifest.container.image = "serverkgg/minecraft-java";
@@ -394,7 +511,7 @@ describe("cross-entry checks", () => {
 		]);
 	});
 
-	test("rejects duplicate secret keys", () => {
+	it("rejects duplicate secret keys", () => {
 		const manifest = {
 			...base(),
 			secrets: [
@@ -417,7 +534,7 @@ describe("cross-entry checks", () => {
 		]);
 	});
 
-	test("rejects duplicate protected paths and notable matches", () => {
+	it("rejects duplicate protected paths and notable matches", () => {
 		const manifest = {
 			...base(),
 			files: {
@@ -452,7 +569,7 @@ describe("cross-entry checks", () => {
 		]);
 	});
 
-	test("rejects duplicate port keys and overlapping host ranges", () => {
+	it("rejects duplicate port keys and overlapping host ranges", () => {
 		const manifest = base();
 		const [game] = manifest.container.ports;
 
@@ -481,7 +598,7 @@ describe("cross-entry checks", () => {
 		]);
 	});
 
-	test("rejects a guide tag that is not declared", () => {
+	it("rejects a guide tag that is not declared", () => {
 		const manifest = {
 			...base(),
 			guides: [
@@ -508,7 +625,7 @@ describe("cross-entry checks", () => {
 		]);
 	});
 
-	test("rejects one body file serving both locales", () => {
+	it("rejects one body file serving both locales", () => {
 		const manifest = {
 			...base(),
 			guides: [
@@ -534,7 +651,7 @@ describe("cross-entry checks", () => {
 });
 
 describe("field validation", () => {
-	test("explains a bad host range instead of saying nothing", () => {
+	it("explains a bad host range instead of saying nothing", () => {
 		const manifest = base();
 		const [game] = manifest.container.ports;
 
@@ -556,7 +673,7 @@ describe("field validation", () => {
 		]);
 	});
 
-	test("explains a default that sits outside its range", () => {
+	it("explains a default that sits outside its range", () => {
 		const manifest = base();
 		const [game] = manifest.container.ports;
 
@@ -574,7 +691,7 @@ describe("field validation", () => {
 		]);
 	});
 
-	test("explains a bad colour", () => {
+	it("explains a bad colour", () => {
 		const manifest = {
 			...base(),
 			files: {
@@ -597,7 +714,7 @@ describe("field validation", () => {
 		]);
 	});
 
-	test("field failures short-circuit the cross-entry checks", () => {
+	it("field failures short-circuit the cross-entry checks", () => {
 		const manifest = base();
 
 		manifest.metadata.id = "Minecraft";
@@ -615,7 +732,7 @@ describe("field validation", () => {
 });
 
 describe("packageSchema", () => {
-	test("routes on kind without a separate pre-parse", () => {
+	it("routes on kind without a separate pre-parse", () => {
 		const game = packageSchema.parse(base());
 
 		expect(game.kind).toBe(PackageKind.Game);
@@ -642,7 +759,7 @@ describe("packageSchema", () => {
 		expect(guides.kind).toBe(PackageKind.Guide);
 	});
 
-	test("guide packages get the same guide checks", () => {
+	it("guide packages get the same guide checks", () => {
 		const result = guidePackageSchema.safeParse({
 			schemaVersion: MANIFEST_SCHEMA_VERSION,
 			kind: "GuidePackage",
@@ -669,7 +786,7 @@ describe("packageSchema", () => {
 		expect(result.error && z.prettifyError(result.error)).toContain('tag "start" is not declared in guideTags');
 	});
 
-	test("carries up to 64 guides", () => {
+	it("carries up to 64 guides", () => {
 		const many = (count: number) =>
 			Array.from(
 				{
@@ -684,14 +801,14 @@ describe("packageSchema", () => {
 });
 
 describe("presence", () => {
-	test("is optional, so a game that declares none parses", () => {
+	it("is optional, so a game that declares none parses", () => {
 		const result = manifestSchema.safeParse(base());
 
 		expect(result.success).toBe(true);
 		expect(result.success && result.data.presence).toBeUndefined();
 	});
 
-	test("defaults the name key to player and the field list to empty", () => {
+	it("defaults the name key to player and the field list to empty", () => {
 		const result = manifestSchema.safeParse({
 			...base(),
 			presence: {},
@@ -702,7 +819,7 @@ describe("presence", () => {
 		expect(result.success && result.data.presence?.fields).toEqual([]);
 	});
 
-	test("accepts an avatar template and localized fields", () => {
+	it("accepts an avatar template and localized fields", () => {
 		const result = manifestSchema.safeParse({
 			...base(),
 			presence: {
@@ -722,7 +839,7 @@ describe("presence", () => {
 		expect(result.success && result.data.presence?.fields[0]?.inline).toBe(true);
 	});
 
-	test("accepts a serverk avatar template", () => {
+	it("accepts a serverk avatar template", () => {
 		const result = manifestSchema.safeParse({
 			...base(),
 			presence: {
@@ -736,7 +853,7 @@ describe("presence", () => {
 		expect(result.success && result.data.presence?.avatar).toBe("serverk://avatars/hytale/{playerId}");
 	});
 
-	test("refuses an avatar template that is neither https nor a serverk avatar", () => {
+	it("refuses an avatar template that is neither https nor a serverk avatar", () => {
 		const result = manifestSchema.safeParse({
 			...base(),
 			presence: {
@@ -750,7 +867,7 @@ describe("presence", () => {
 		);
 	});
 
-	test("refuses a serverk scheme that is not an avatar path", () => {
+	it("refuses a serverk scheme that is not an avatar path", () => {
 		const result = manifestSchema.safeParse({
 			...base(),
 			presence: {
@@ -764,7 +881,7 @@ describe("presence", () => {
 		);
 	});
 
-	test("refuses a serverk avatar template longer than the limit", () => {
+	it("refuses a serverk avatar template longer than the limit", () => {
 		const result = manifestSchema.safeParse({
 			...base(),
 			presence: {
@@ -775,12 +892,12 @@ describe("presence", () => {
 		expect(result.success).toBe(false);
 	});
 
-	test("tells a serverk avatar template apart from an https one", () => {
+	it("tells a serverk avatar template apart from an https one", () => {
 		expect(isServerkAvatarTemplate("serverk://avatars/hytale/{playerId}")).toBe(true);
 		expect(isServerkAvatarTemplate("https://mc-heads.net/avatar/{skin}/128")).toBe(false);
 	});
 
-	test("refuses a payload key that could never be a capture group", () => {
+	it("refuses a payload key that could never be a capture group", () => {
 		const result = manifestSchema.safeParse({
 			...base(),
 			presence: {
@@ -792,7 +909,7 @@ describe("presence", () => {
 		expect(result.error && z.prettifyError(result.error)).toContain("must be a valid capture group name");
 	});
 
-	test("refuses the same field declared twice", () => {
+	it("refuses the same field declared twice", () => {
 		const result = manifestSchema.safeParse({
 			...base(),
 			presence: {
@@ -815,7 +932,7 @@ describe("presence", () => {
 });
 
 describe("container runtime", () => {
-	test("materializes the defaults when the manifest declares no runtime", () => {
+	it("materializes the defaults when the manifest declares no runtime", () => {
 		const manifest = manifestSchema.parse(base());
 
 		expect(manifest.container.runtime).toEqual({
@@ -829,7 +946,7 @@ describe("container runtime", () => {
 		expect(MANIFEST_RUNTIME_DEFAULTS).toEqual(manifest.container.runtime);
 	});
 
-	test("keeps every field a manifest declares", () => {
+	it("keeps every field a manifest declares", () => {
 		const manifest = manifestSchema.parse(
 			withRuntime({
 				platform: ManifestPlatform.Wine,
@@ -857,7 +974,7 @@ describe("container runtime", () => {
 		});
 	});
 
-	test("clamps an install budget above the ceiling", () => {
+	it("clamps an install budget above the ceiling", () => {
 		expect(
 			failure(
 				withRuntime({
@@ -872,7 +989,7 @@ describe("container runtime", () => {
 		]);
 	});
 
-	test("clamps a boot budget below the floor", () => {
+	it("clamps a boot budget below the floor", () => {
 		expect(
 			failure(
 				withRuntime({
@@ -887,7 +1004,7 @@ describe("container runtime", () => {
 		]);
 	});
 
-	test("clamps shared memory and the pids limit to their range", () => {
+	it("clamps shared memory and the pids limit to their range", () => {
 		expect(
 			failure(
 				withRuntime({
@@ -907,7 +1024,7 @@ describe("container runtime", () => {
 		]);
 	});
 
-	test("refuses a fourth capability", () => {
+	it("refuses a fourth capability", () => {
 		expect(
 			failure(
 				withRuntime({
@@ -927,7 +1044,7 @@ describe("container runtime", () => {
 		]);
 	});
 
-	test("refuses a capability outside the allow-list", () => {
+	it("refuses a capability outside the allow-list", () => {
 		expect(
 			failure(
 				withRuntime({
@@ -944,7 +1061,7 @@ describe("container runtime", () => {
 		]);
 	});
 
-	test("refuses a wine platform running on the default shared memory", () => {
+	it("refuses a wine platform running on the default shared memory", () => {
 		expect(
 			failure(
 				withRuntime({
@@ -959,7 +1076,7 @@ describe("container runtime", () => {
 		]);
 	});
 
-	test("accepts a wine platform once shared memory clears the floor", () => {
+	it("accepts a wine platform once shared memory clears the floor", () => {
 		const manifest = manifestSchema.parse(
 			withRuntime({
 				platform: ManifestPlatform.Wine,
@@ -972,7 +1089,7 @@ describe("container runtime", () => {
 });
 
 describe("manifestRuntimeOf", () => {
-	test("reads the runtime a manifest declares", () => {
+	it("reads the runtime a manifest declares", () => {
 		const manifest = manifestSchema.parse(
 			withRuntime({
 				platform: ManifestPlatform.Wine,
@@ -987,7 +1104,7 @@ describe("manifestRuntimeOf", () => {
 		});
 	});
 
-	test("falls back to the defaults for a bundle published before the field existed", () => {
+	it("falls back to the defaults for a bundle published before the field existed", () => {
 		const manifest = manifestSchema.parse(base());
 		const legacy = {
 			...manifest,
@@ -1002,14 +1119,14 @@ describe("manifestRuntimeOf", () => {
 });
 
 describe("port mirroring", () => {
-	test("defaults to no mirror and no follower", () => {
+	it("defaults to no mirror and no follower", () => {
 		const manifest = manifestSchema.parse(base());
 
 		expect(manifest.container.ports[0]?.mirror).toBe(false);
 		expect(manifest.container.ports[0]?.follows).toBeUndefined();
 	});
 
-	test("keeps the mirror flag and the follow rule a manifest declares", () => {
+	it("keeps the mirror flag and the follow rule a manifest declares", () => {
 		const manifest = manifestSchema.parse(
 			withPorts([
 				udpPort("game", 27_015, 27_015),
@@ -1030,7 +1147,7 @@ describe("port mirroring", () => {
 		});
 	});
 
-	test("refuses a follow rule naming a port that is not declared", () => {
+	it("refuses a follow rule naming a port that is not declared", () => {
 		expect(
 			failure(
 				withPorts([
@@ -1051,7 +1168,7 @@ describe("port mirroring", () => {
 		]);
 	});
 
-	test("refuses a port that follows itself", () => {
+	it("refuses a port that follows itself", () => {
 		expect(
 			failure(
 				withPorts([
@@ -1072,7 +1189,7 @@ describe("port mirroring", () => {
 		]);
 	});
 
-	test("refuses a follow rule crossing protocols", () => {
+	it("refuses a follow rule crossing protocols", () => {
 		const [tcp] = base().container.ports;
 
 		if (!tcp) {
@@ -1100,7 +1217,7 @@ describe("port mirroring", () => {
 		]);
 	});
 
-	test("refuses a port that both follows and is followed", () => {
+	it("refuses a port that both follows and is followed", () => {
 		expect(
 			failure(
 				withPorts([
@@ -1129,7 +1246,7 @@ describe("port mirroring", () => {
 		]);
 	});
 
-	test("lets a follower's host range overlap the port it follows", () => {
+	it("lets a follower's host range overlap the port it follows", () => {
 		const manifest = manifestSchema.parse(
 			withPorts([
 				udpPort("game", 27_015, 27_015),
@@ -1146,7 +1263,7 @@ describe("port mirroring", () => {
 		expect(manifest.container.ports[1]?.hostRange.min).toBe(27_016);
 	});
 
-	test("still refuses two independent ports whose host ranges overlap", () => {
+	it("still refuses two independent ports whose host ranges overlap", () => {
 		expect(
 			failure(
 				withPorts([
@@ -1162,7 +1279,7 @@ describe("port mirroring", () => {
 		]);
 	});
 
-	test("refuses an offset outside its range", () => {
+	it("refuses an offset outside its range", () => {
 		expect(
 			failure(
 				withPorts([
@@ -1204,13 +1321,13 @@ describe("port pairs and blocks", () => {
 		},
 	});
 
-	test("defaults a port to a single binding", () => {
+	it("defaults a port to a single binding", () => {
 		const manifest = manifestSchema.parse(base());
 
 		expect(manifest.container.ports[0]?.count).toBe(1);
 	});
 
-	test("accepts a tcp+udp join port beside a udp media block", () => {
+	it("accepts a tcp+udp join port beside a udp media block", () => {
 		const manifest = manifestSchema.parse(
 			withPorts([
 				pairPort("game", 19_132),
@@ -1235,7 +1352,7 @@ describe("port pairs and blocks", () => {
 		]);
 	});
 
-	test("refuses a count above the block limit", () => {
+	it("refuses a count above the block limit", () => {
 		expect(
 			failure(
 				withPorts([
@@ -1250,7 +1367,7 @@ describe("port pairs and blocks", () => {
 		]);
 	});
 
-	test("refuses a block whose host range cannot hold it", () => {
+	it("refuses a block whose host range cannot hold it", () => {
 		expect(
 			failure(
 				withPorts([
@@ -1281,7 +1398,7 @@ describe("port pairs and blocks", () => {
 		},
 	});
 
-	test("lets a single port span the whole cap", () => {
+	it("lets a single port span the whole cap", () => {
 		const manifest = manifestSchema.parse(
 			withPorts([
 				spanning("game", 20_000, 20_999, 1),
@@ -1291,7 +1408,7 @@ describe("port pairs and blocks", () => {
 		expect(manifest.container.ports[0]?.hostRange.max).toBe(20_999);
 	});
 
-	test("refuses a single port spanning one more than the cap", () => {
+	it("refuses a single port spanning one more than the cap", () => {
 		expect(
 			failure(
 				withPorts([
@@ -1306,7 +1423,7 @@ describe("port pairs and blocks", () => {
 		]);
 	});
 
-	test("counts the span of a block in slots, so 100 slots of 64 pass", () => {
+	it("counts the span of a block in slots, so 100 slots of 64 pass", () => {
 		const manifest = manifestSchema.parse(
 			withPorts([
 				spanning("media", 20_000, 26_399, 64),
@@ -1316,7 +1433,7 @@ describe("port pairs and blocks", () => {
 		expect(manifest.container.ports[0]?.count).toBe(64);
 	});
 
-	test("refuses a block spanning more slots than the cap", () => {
+	it("refuses a block spanning more slots than the cap", () => {
 		expect(
 			failure(
 				withPorts([
@@ -1331,7 +1448,7 @@ describe("port pairs and blocks", () => {
 		]);
 	});
 
-	test("refuses a block whose default leaves no room for the whole block", () => {
+	it("refuses a block whose default leaves no room for the whole block", () => {
 		expect(
 			failure(
 				withPorts([
@@ -1353,7 +1470,7 @@ describe("port pairs and blocks", () => {
 		]);
 	});
 
-	test("refuses a non-mirrored block that runs past the last port", () => {
+	it("refuses a non-mirrored block that runs past the last port", () => {
 		expect(
 			failure(
 				withPorts([
@@ -1372,7 +1489,7 @@ describe("port pairs and blocks", () => {
 		]);
 	});
 
-	test("refuses the player-address fields on a block", () => {
+	it("refuses the player-address fields on a block", () => {
 		expect(
 			failure(
 				withPorts([
@@ -1407,7 +1524,7 @@ describe("port pairs and blocks", () => {
 		]);
 	});
 
-	test("refuses a block that follows a port", () => {
+	it("refuses a block that follows a port", () => {
 		expect(
 			failure(
 				withPorts([
@@ -1429,7 +1546,7 @@ describe("port pairs and blocks", () => {
 		]);
 	});
 
-	test("refuses a port that follows a block", () => {
+	it("refuses a port that follows a block", () => {
 		expect(
 			failure(
 				withPorts([
@@ -1451,7 +1568,7 @@ describe("port pairs and blocks", () => {
 		]);
 	});
 
-	test("refuses srv on a tcp+udp port", () => {
+	it("refuses srv on a tcp+udp port", () => {
 		expect(
 			failure(
 				withPorts([
@@ -1471,7 +1588,7 @@ describe("port pairs and blocks", () => {
 		]);
 	});
 
-	test("refuses a tcp+udp port that follows another", () => {
+	it("refuses a tcp+udp port that follows another", () => {
 		expect(
 			failure(
 				withPorts([
@@ -1493,7 +1610,7 @@ describe("port pairs and blocks", () => {
 		]);
 	});
 
-	test("refuses a port that follows a tcp+udp port", () => {
+	it("refuses a port that follows a tcp+udp port", () => {
 		expect(
 			failure(
 				withPorts([
@@ -1515,7 +1632,7 @@ describe("port pairs and blocks", () => {
 		]);
 	});
 
-	test("refuses a tcp+udp port overlapping a udp port on the shared protocol", () => {
+	it("refuses a tcp+udp port overlapping a udp port on the shared protocol", () => {
 		expect(
 			failure(
 				withPorts([
@@ -1531,7 +1648,7 @@ describe("port pairs and blocks", () => {
 		]);
 	});
 
-	test("lets a tcp port and a udp port share a host range", () => {
+	it("lets a tcp port and a udp port share a host range", () => {
 		const [tcp] = base().container.ports;
 
 		if (!tcp) {
@@ -1548,7 +1665,7 @@ describe("port pairs and blocks", () => {
 		expect(manifest.container.ports).toHaveLength(2);
 	});
 
-	test("refuses a manifest whose pairs expand past the socket port limit", () => {
+	it("refuses a manifest whose pairs expand past the socket port limit", () => {
 		expect(
 			failure(
 				withPorts(
@@ -1625,17 +1742,18 @@ const yamlSource = (document: unknown): PackageSource => {
 };
 
 describe("bot packages", () => {
-	test("parse with no ports and no category", () => {
+	it("parse with no ports and no category", () => {
 		const manifest = botManifestSchema.parse(botBase());
 
 		expect(manifest.kind).toBe(PackageKind.Bot);
 		expect(manifest.container.ports).toEqual([]);
 		expect("category" in manifest.metadata).toBe(false);
+		expect("extensions" in manifest.metadata).toBe(false);
 		expect(manifest.secrets).toBeUndefined();
 		expect(manifest.presence).toBeUndefined();
 	});
 
-	test("default the ports to none when a bot declares no ports key", () => {
+	it("default the ports to none when a bot declares no ports key", () => {
 		const { container, ...rest } = botBase();
 		const manifest = botManifestSchema.parse({
 			...rest,
@@ -1647,7 +1765,7 @@ describe("bot packages", () => {
 		expect(manifest.container.ports).toEqual([]);
 	});
 
-	test("refuse a secrets list with the environment vault named as the place for per-bot values", () => {
+	it("refuse a secrets list with the environment vault named as the place for per-bot values", () => {
 		expect(
 			botFailure({
 				...botBase(),
@@ -1661,7 +1779,7 @@ describe("bot packages", () => {
 		]);
 	});
 
-	test("keep the environment vault off unless the package turns it on", () => {
+	it("keep the environment vault off unless the package turns it on", () => {
 		expect(botManifestSchema.parse(botBase()).environment.vault).toBe(false);
 		expect(
 			botManifestSchema.parse({
@@ -1673,7 +1791,7 @@ describe("bot packages", () => {
 		).toBe(true);
 	});
 
-	test("refuse a vault flag that is not a boolean", () => {
+	it("refuse a vault flag that is not a boolean", () => {
 		expect(
 			botFailure({
 				...botBase(),
@@ -1686,7 +1804,7 @@ describe("bot packages", () => {
 		]);
 	});
 
-	test("refuse a presence block", () => {
+	it("refuse a presence block", () => {
 		expect(
 			botFailure({
 				...botBase(),
@@ -1702,7 +1820,7 @@ describe("bot packages", () => {
 		]);
 	});
 
-	test("still run the shared package checks", () => {
+	it("still run the shared package checks", () => {
 		expect(
 			botFailure({
 				...botBase(),
@@ -1719,7 +1837,7 @@ describe("bot packages", () => {
 		]);
 	});
 
-	test("keep a port a bot declares for a webhook", () => {
+	it("keep a port a bot declares for a webhook", () => {
 		const manifest = botManifestSchema.parse({
 			...botBase(),
 			container: {
@@ -1731,7 +1849,7 @@ describe("bot packages", () => {
 		expect(manifest.container.ports).toHaveLength(1);
 	});
 
-	test("reach the runtime reader like a game does", () => {
+	it("reach the runtime reader like a game does", () => {
 		expect(manifestRuntimeOf(botManifestSchema.parse(botBase()))).toEqual({
 			...MANIFEST_RUNTIME_DEFAULTS,
 			pidsLimit: 256,
@@ -1740,7 +1858,7 @@ describe("bot packages", () => {
 });
 
 describe("package kinds", () => {
-	test("route all three kinds through packageSchema", () => {
+	it("route all three kinds through packageSchema", () => {
 		expect(packageSchema.parse(base()).kind).toBe(PackageKind.Game);
 		expect(packageSchema.parse(botBase()).kind).toBe(PackageKind.Bot);
 		expect(
@@ -1752,13 +1870,13 @@ describe("package kinds", () => {
 		).toBe(PackageKind.Guide);
 	});
 
-	test("keep a game package on the game rules, so it still needs a port", () => {
+	it("keep a game package on the game rules, so it still needs a port", () => {
 		const result = packageSchema.safeParse(withPorts([]));
 
 		expect(result.success).toBe(false);
 	});
 
-	test("refuse a guide package where a runnable package is expected", () => {
+	it("refuse a guide package where a runnable package is expected", () => {
 		expect(
 			runnablePackageSchema.safeParse(
 				guidePackage([
@@ -1768,7 +1886,7 @@ describe("package kinds", () => {
 		).toBe(false);
 	});
 
-	test("tell a game from a bot", () => {
+	it("tell a game from a bot", () => {
 		const game = runnablePackageSchema.parse(base());
 		const bot = runnablePackageSchema.parse(botBase());
 
@@ -1778,24 +1896,24 @@ describe("package kinds", () => {
 		expect(isGameManifest(bot)).toBe(false);
 	});
 
-	test("keep the old game schema name pointing at the game schema for this release", () => {
+	it("keep the old game schema name pointing at the game schema for this release", () => {
 		expect(manifestSchema).toBe(gameManifestSchema);
 	});
 
-	test("read a bot package from its source", () => {
+	it("read a bot package from its source", () => {
 		const { manifest } = readPackage(yamlSource(botBase()));
 
 		expect(manifest.kind).toBe(PackageKind.Bot);
 		expect(manifest.metadata.id).toBe("bot-node");
 	});
 
-	test("read a game package from its source unchanged", () => {
+	it("read a game package from its source unchanged", () => {
 		const { manifest } = readPackage(yamlSource(base()));
 
 		expect(isGameManifest(manifest) && manifest.secrets).toEqual([]);
 	});
 
-	test("render every kind literal into the JSON schema", () => {
+	it("render every kind literal into the JSON schema", () => {
 		const rendered = JSON.stringify(buildJsonSchema());
 
 		for (const kind of Object.values(PackageKind)) {