@serverkgg/bridge 0.1.11 → 0.1.13

package/src/runtime/bridgeSetupHandlers.test.ts

@@ -45,6 +45,9 @@ const contextOf = (): BridgeContext => {
 
 	return createContext(settings, supervisor, {
 		onLog: log,
+		onPrint() {
+			return;
+		},
 		onEmit() {
 			return;
 		},

package/src/runtime/bridgeSupervisor.ts

@@ -4,8 +4,9 @@ import { BridgeLifecycleSubject } from "../modules/bridgeLifecycleSubject";
 import { BRIDGE_STOP_SIGNALS, type BridgeStopSignal } from "../modules/bridgeStopSignal";
 import { BRIDGE_CRASH_LINE_LIMIT, BridgeCrashPhase, type BridgeCrashRecord } from "../protocol";
 import { stripAnsi } from "../utils";
+import { createLineSpacing } from "./bridgeLineSpacing";
 import type { BridgeLeveledLogger } from "./bridgeLog";
-import { errorText } from "./bridgeLog";
+import { errorText, print } from "./bridgeLog";
 import { settleUid, unprivilegedArgv } from "./bridgePrivilege";
 import { readProcessGroup, signalGroup } from "./bridgeSignal";
 
@@ -286,11 +287,7 @@ export const createSupervisor = (options: SupervisorOptions) => {
 				}
 			: raw;
 
-		if (entry.stream === BridgeStream.Stderr) {
-			process.stderr.write(`${entry.line}\n`);
-		} else {
-			process.stdout.write(`${entry.line}\n`);
-		}
+		print(entry.line, entry.stream);
 
 		tail.push(entry);
 
@@ -306,6 +303,7 @@ export const createSupervisor = (options: SupervisorOptions) => {
 	const pump = async (stream: ReadableStream<Uint8Array>, kind: BridgeStream) => {
 		const decoder = new TextDecoder();
 		const reader = stream.getReader();
+		const spacing = createLineSpacing();
 
 		let buffer = "";
 
@@ -325,7 +323,7 @@ export const createSupervisor = (options: SupervisorOptions) => {
 
 			buffer = split.rest;
 
-			for (const line of split.lines) {
+			for (const line of split.lines.flatMap(spacing)) {
 				emit({
 					stream: kind,
 					line,
@@ -333,10 +331,10 @@ export const createSupervisor = (options: SupervisorOptions) => {
 			}
 		}
 
-		if (buffer.length > 0) {
+		for (const line of spacing(buffer)) {
 			emit({
 				stream: kind,
-				line: buffer,
+				line,
 			});
 		}
 	};

package/src/runtime/serve.ts

@@ -19,9 +19,15 @@ import { validateDriver } from "./bridgeDriver";
 import { createEventRing } from "./bridgeEvents";
 import { decideInstall, recountHold } from "./bridgeInstall";
 import { createBridgeLiveConfig } from "./bridgeLiveConfig";
-import { errorDetail, errorText, log as stdoutLog } from "./bridgeLog";
+import { errorDetail, errorText, print, log as stdoutLog } from "./bridgeLog";
 import { pendingResult, rejectedPending, unsupportedPending } from "./bridgePending";
-import { createBridgeRedaction, redactLocalized, redactLogger, secretFieldKeys } from "./bridgeRedaction";
+import {
+	createBridgeRedaction,
+	redactLocalized,
+	redactLogger,
+	redactPrinter,
+	secretFieldKeys,
+} from "./bridgeRedaction";
 import { createRegistry } from "./bridgeRegistry";
 import { createServer } from "./bridgeServer";
 import { setupSummaryOf } from "./bridgeSetup";
@@ -182,6 +188,7 @@ export const serve = async (driver: BridgeDriver) => {
 		supervisor,
 		{
 			onLog: log,
+			onPrint: redactPrinter(print, redact),
 			onEmit(event, payload) {
 				ring.emit(event, payload);
 			},

package/src/steam/steamcmdNarrator.test.ts

@@ -20,7 +20,7 @@ const recorder = () => {
 };
 
 describe("steamcmdNarrator", () => {
-	it("reports download progress in ten percent steps", () => {
+	it("reports download progress in ten percent steps, each line with the percent steamcmd printed", () => {
 		const { log, lines, fields } = recorder();
 		const narrate = steamcmdNarrator(log, "palworld");
 
@@ -36,10 +36,10 @@ describe("steamcmdNarrator", () => {
 		}
 
 		expect(lines).toEqual([
-			"downloading palworld",
-			"downloading palworld",
-			"downloading palworld",
-			"downloading palworld",
+			"downloading palworld 2%",
+			"downloading palworld 11%",
+			"downloading palworld 25%",
+			"downloading palworld 99%",
 		]);
 		expect(fields).toEqual([
 			{
@@ -74,6 +74,19 @@ describe("steamcmdNarrator", () => {
 		]);
 	});
 
+	it("says how much of the download is done when steamcmd prints the byte counts", () => {
+		const { log, lines } = recorder();
+		const narrate = steamcmdNarrator(log, "palworld");
+
+		narrate("Update state (0x61) downloading, progress: 1.11 (127644881 / 11530459441)");
+		narrate("Update state (0x61) downloading, progress: 40.02 (4614180864 / 11530459441)");
+
+		expect(lines).toEqual([
+			"downloading palworld 1% (122 MB of 10.7 GB)",
+			"downloading palworld 40% (4.3 GB of 10.7 GB)",
+		]);
+	});
+
 	it("passes notable lines through with the ansi stripped", () => {
 		const { log, lines } = recorder();
 		const narrate = steamcmdNarrator(log, "rust");

package/src/steam/steamcmdNarrator.ts

@@ -1,12 +1,33 @@
 import type { BridgeLog } from "../context";
 import { stripAnsi } from "../utils";
 
-const PROGRESS = /Update state \(0x[0-9a-f]+\)\s*(?<phase>[^,]+),\s*progress:\s*(?<percent>\d+)\.\d+/i;
+const PROGRESS =
+	/Update state \(0x[0-9a-f]+\)\s*(?<phase>[^,]+),\s*progress:\s*(?<percent>\d+)\.\d+(?:\s*\((?<done>\d+)\s*\/\s*(?<total>\d+)\))?/i;
+
+const GIGABYTE = 1024 ** 3;
+
+const MEGABYTE = 1024 ** 2;
 
 const PROGRESS_STEP = 10;
 
 const NOTABLE = /^(Error|Warning|Failed|Success|Logging in|Connecting|Update state \(0x\d+\) (?!downloading))/i;
 
+const byteSizeOf = (bytes: number) => {
+	if (bytes >= GIGABYTE) {
+		return `${(bytes / GIGABYTE).toFixed(1)} GB`;
+	}
+
+	return `${Math.round(bytes / MEGABYTE)} MB`;
+};
+
+const progressLineOf = (phase: string, label: string, percent: number, done: number, total: number) => {
+	if (total < MEGABYTE) {
+		return `${phase} ${label} ${percent}%`;
+	}
+
+	return `${phase} ${label} ${percent}% (${byteSizeOf(done)} of ${byteSizeOf(total)})`;
+};
+
 export const steamcmdNarrator = (log: BridgeLog, label: string) => {
 	let phase = "";
 	let reported = -1;
@@ -28,7 +49,10 @@ export const steamcmdNarrator = (log: BridgeLog, label: string) => {
 			if (step > reported) {
 				reported = step;
 
-				log(`${phase} ${label}`, {
+				const done = Number(progress.groups.done ?? 0);
+				const total = Number(progress.groups.total ?? 0);
+
+				log(progressLineOf(phase, label, percent, done, total), {
 					percent: step,
 				});
 			}

package/src/wine/protonCommand.test.ts

@@ -1,6 +1,6 @@
 import { describe, expect, it } from "bun:test";
 import type { BridgeContext } from "../context";
-import { protonCommand, protonCompatPath } from "./protonCommand";
+import { ProtonVerb, protonCommand, protonCompatPath } from "./protonCommand";
 
 const ensured: string[] = [];
 
@@ -41,6 +41,22 @@ describe("protonCommand", () => {
 		]);
 	});
 
+	it("runs straight in the prefix when asked, so the game keeps the container's stdout", async () => {
+		expect(
+			(
+				await protonCommand(context, {
+					appId: "2394010",
+					exe: "Pal/Binaries/Win64/PalServer-Win64-Shipping-Cmd.exe",
+					verb: ProtonVerb.RunInPrefix,
+				})
+			).slice(-3),
+		).toEqual([
+			"/opt/proton/proton",
+			"runinprefix",
+			"Pal/Binaries/Win64/PalServer-Win64-Shipping-Cmd.exe",
+		]);
+	});
+
 	it("ensures the compat and steam directories", async () => {
 		ensured.length = 0;
 

package/src/wine/protonCommand.ts

@@ -7,6 +7,11 @@ export const PROTON_COMPAT_DIRECTORY = ".proton";
 
 export const STEAM_COMPAT_CLIENT_DIRECTORY = ".steam";
 
+export enum ProtonVerb {
+	Run = "run",
+	RunInPrefix = "runinprefix",
+}
+
 export interface ProtonCommandOptions {
 	appId: string;
 	exe: string;
@@ -16,6 +21,7 @@ export interface ProtonCommandOptions {
 	steamDir?: string;
 	home?: string;
 	environment?: Record<string, string>;
+	verb?: ProtonVerb;
 }
 
 export const protonCompatPath = (appId: string, compatDir: string = PROTON_COMPAT_DIRECTORY) => {
@@ -42,7 +48,7 @@ export const protonCommand = async (context: BridgeContext, options: ProtonComma
 		"env",
 		...Object.entries(environment).map(([key, value]) => `${key}=${value}`),
 		`${protonDir}/proton`,
-		"run",
+		options.verb ?? ProtonVerb.Run,
 		options.exe,
 		...(options.args ?? []),
 	];

package/src/wire/panel.test.ts

@@ -13,6 +13,7 @@ import {
 	BridgePlace,
 	type BridgeSetupManifest,
 	BridgeSetupStepKind,
+	type BridgeTerminalManifest,
 	BridgeUploadMode,
 } from "@serverkgg/bridge/protocol";
 import { bridgeAvailabilitySchema, bridgeCellsSchema, bridgeManifestSchema } from "./panel";
@@ -783,3 +784,34 @@ describe("the availability schema", () => {
 		expect(parsed.modules[0]?.actions).toContain(BridgeAction.Availability);
 	});
 });
+
+const terminalManifest = (shell: boolean): BridgeManifest => ({
+	version: "1.0.0",
+	panel: null,
+	terminal: {
+		commands: [],
+		rules: [],
+		shell,
+	},
+	setup: null,
+	modules: [],
+	events: [],
+});
+
+describe("the terminal mirror", () => {
+	it("carries the shell flag through unchanged", () => {
+		expect(bridgeManifestSchema.parse(terminalManifest(true)).terminal?.shell).toBe(true);
+	});
+
+	it("defaults a terminal from an older bridge that carries no shell flag to false", () => {
+		const older = terminalManifest(true);
+
+		delete (older.terminal as Partial<BridgeTerminalManifest>).shell;
+
+		expect(bridgeManifestSchema.parse(older).terminal).toEqual({
+			commands: [],
+			rules: [],
+			shell: false,
+		});
+	});
+});

package/src/wire/panel.ts

@@ -318,6 +318,7 @@ export const bridgeTerminalRuleSchema: z.ZodType<BridgeTerminalRuleManifest> = z
 export const bridgeTerminalManifestSchema: z.ZodType<BridgeTerminalManifest> = z.object({
 	commands: z.array(bridgeTerminalCommandSchema).max(128),
 	rules: z.array(bridgeTerminalRuleSchema).max(64),
+	shell: z.boolean().default(false),
 });
 
 const setupStepBase = {