@hydranium/core 1.0.0-next.267 → 1.0.0-next.269

package/lib/testing/playwright/server-log-fixture.d.ts

@@ -16,6 +16,21 @@ export interface ServerLogFixtures {
      * `attachServerLog` in `beforeEach` / `afterEach` instead.
      */
     serverLogWorkspace: string;
+    /**
+     * Fail a test whose part of the server log holds a `Warn` or `Error` line
+     * that {@link serverLogAllowlist} does not match. Off by default; reads
+     * nothing when capture is off.
+     */
+    serverLogClean: boolean;
+    /**
+     * The `Warn` and `Error` lines {@link serverLogClean} lets through: each key
+     * is why the line is expected, and its pattern is matched against the whole
+     * line, label and timestamp included. A pattern takes neither the `g` nor
+     * the `y` flag, which make `test` resume from its last match. An object
+     * rather than a list, because Playwright reads an array whose second entry
+     * is an object as a fixture's `[value, options]`.
+     */
+    serverLogAllowlist: Readonly<Record<string, RegExp>>;
 }
 /** Full fixture set including the internal side-effect-only auto fixture. */
 interface ServerLogTestFixtures extends ServerLogFixtures {
@@ -28,6 +43,7 @@ interface ServerLogTestFixtures extends ServerLogFixtures {
  * (teardown) — i.e. the `beforeEach` + `afterEach` pair fused into one auto
  * fixture. Compose onto any test base, or use the pre-extended {@link test}.
  * Override `serverLogWorkspace` to wire the workspace; otherwise it's inert.
+ * Set `serverLogClean` to also fail a test on an unexpected `Warn` or `Error`.
  */
 export declare const serverLogFixtures: Fixtures<ServerLogTestFixtures, object, PlaywrightTestArgs & PlaywrightTestOptions>;
 /** Pre-extended test with {@link serverLogFixtures} applied, for adopters on the plain base. */

package/lib/testing/playwright/server-log-fixture.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"server-log-fixture.d.ts","sourceRoot":"","sources":["../../../src/testing/playwright/server-log-fixture.ts"],"names":[],"mappings":"AAAA;;;;;;;kFAOkF;AAKlF,OAAO,EAAgB,KAAK,QAAQ,EAAE,KAAK,kBAAkB,EAAE,KAAK,qBAAqB,EAAE,MAAM,kBAAkB,CAAC;AAGpH,MAAM,WAAW,iBAAiB;IAC/B;;;;;;OAMG;IACH,kBAAkB,EAAE,MAAM,CAAC;CAC7B;AAED,6EAA6E;AAC7E,UAAU,qBAAsB,SAAQ,iBAAiB;IACtD,sFAAsF;IACtF,gBAAgB,EAAE,IAAI,CAAC;CACzB;AAED;;;;;;GAMG;AACH,eAAO,MAAM,iBAAiB,EAAE,QAAQ,CAAC,qBAAqB,EAAE,MAAM,EAAE,kBAAkB,GAAG,qBAAqB,CAcjH,CAAC;AAEF,gGAAgG;AAChG,eAAO,MAAM,IAAI,+MAAwD,CAAC"}
+{"version":3,"file":"server-log-fixture.d.ts","sourceRoot":"","sources":["../../../src/testing/playwright/server-log-fixture.ts"],"names":[],"mappings":"AAAA;;;;;;;kFAOkF;AAKlF,OAAO,EAAgB,KAAK,QAAQ,EAAE,KAAK,kBAAkB,EAAE,KAAK,qBAAqB,EAAE,MAAM,kBAAkB,CAAC;AAUpH,MAAM,WAAW,iBAAiB;IAC/B;;;;;;OAMG;IACH,kBAAkB,EAAE,MAAM,CAAC;IAC3B;;;;OAIG;IACH,cAAc,EAAE,OAAO,CAAC;IACxB;;;;;;;OAOG;IACH,kBAAkB,EAAE,QAAQ,CAAC,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC,CAAC;CACvD;AAED,6EAA6E;AAC7E,UAAU,qBAAsB,SAAQ,iBAAiB;IACtD,sFAAsF;IACtF,gBAAgB,EAAE,IAAI,CAAC;CACzB;AAED;;;;;;;GAOG;AACH,eAAO,MAAM,iBAAiB,EAAE,QAAQ,CAAC,qBAAqB,EAAE,MAAM,EAAE,kBAAkB,GAAG,qBAAqB,CA+BjH,CAAC;AAEF,gGAAgG;AAChG,eAAO,MAAM,IAAI,+MAAwD,CAAC"}

package/lib/testing/playwright/server-log-fixture.js

@@ -10,24 +10,43 @@
 // The plain functions in `server-log-capture.ts` stay playwright-free; only
 // adopters that use this fixture need `@playwright/test` installed.
 import { test as base } from '@playwright/test';
-import { attachServerLog, markServerLog } from './server-log-capture.js';
+import { attachServerLog, DEFAULT_SERVER_LOG_ATTACH_ENV, findUnexpectedServerLogLines, markServerLog, resolveServerLogPath } from './server-log-capture.js';
+import { readServerLogRegion } from './server-log-region.js';
 /**
  * Composable fixtures for per-test server-log capture: writes the start marker
  * before each test (setup) and attaches the log on failure after each test
  * (teardown) — i.e. the `beforeEach` + `afterEach` pair fused into one auto
  * fixture. Compose onto any test base, or use the pre-extended {@link test}.
  * Override `serverLogWorkspace` to wire the workspace; otherwise it's inert.
+ * Set `serverLogClean` to also fail a test on an unexpected `Warn` or `Error`.
  */
 export const serverLogFixtures = {
     serverLogWorkspace: ['', { option: true }],
+    serverLogClean: [false, { option: true }],
+    serverLogAllowlist: [{}, { option: true }],
     serverLogCapture: [
-        async ({ serverLogWorkspace }, use) => {
-            if (serverLogWorkspace) {
-                markServerLog(base.info(), serverLogWorkspace);
+        async ({ serverLogWorkspace, serverLogClean, serverLogAllowlist }, use) => {
+            if (!serverLogWorkspace) {
+                await use(undefined);
+                return;
             }
+            markServerLog(base.info(), serverLogWorkspace);
             await use(undefined);
-            if (serverLogWorkspace) {
-                await attachServerLog(base.info(), serverLogWorkspace);
+            const file = resolveServerLogPath(serverLogWorkspace);
+            const region = file ? readServerLogRegion(file) : '';
+            let problem;
+            if (serverLogClean) {
+                const unexpected = findUnexpectedServerLogLines(region, serverLogAllowlist);
+                if (unexpected.length > 0) {
+                    problem = `The server logged ${unexpected.length} unexpected Warn or Error line(s):\n${unexpected.join('\n')}`;
+                }
+            }
+            // The failure-only policy reads the test's status, which failing here
+            // does not change; a run set to attach nothing still attaches nothing.
+            const attachOn = process.env[DEFAULT_SERVER_LOG_ATTACH_ENV] === 'never' ? 'never' : 'always';
+            await attachServerLog(base.info(), serverLogWorkspace, problem ? { attachOn } : {});
+            if (problem) {
+                throw new Error(problem);
             }
         },
         { auto: true }

package/lib/testing/playwright/server-log-fixture.js.map

@@ -1 +1 @@
-{"version":3,"file":"server-log-fixture.js","sourceRoot":"","sources":["../../../src/testing/playwright/server-log-fixture.ts"],"names":[],"mappings":"AAAA;;;;;;;kFAOkF;AAElF,kFAAkF;AAClF,4EAA4E;AAC5E,oEAAoE;AACpE,OAAO,EAAE,IAAI,IAAI,IAAI,EAAsE,MAAM,kBAAkB,CAAC;AACpH,OAAO,EAAE,eAAe,EAAE,aAAa,EAAE,MAAM,yBAAyB,CAAC;AAmBzE;;;;;;GAMG;AACH,MAAM,CAAC,MAAM,iBAAiB,GAAwF;IACnH,kBAAkB,EAAE,CAAC,EAAE,EAAE,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;IAC1C,gBAAgB,EAAE;QACf,KAAK,EAAE,EAAE,kBAAkB,EAAE,EAAE,GAAG,EAAiB,EAAE;YAClD,IAAI,kBAAkB,EAAE,CAAC;gBACtB,aAAa,CAAC,IAAI,CAAC,IAAI,EAAE,EAAE,kBAAkB,CAAC,CAAC;YAClD,CAAC;YACD,MAAM,GAAG,CAAC,SAAS,CAAC,CAAC;YACrB,IAAI,kBAAkB,EAAE,CAAC;gBACtB,MAAM,eAAe,CAAC,IAAI,CAAC,IAAI,EAAE,EAAE,kBAAkB,CAAC,CAAC;YAC1D,CAAC;QACJ,CAAC;QACD,EAAE,IAAI,EAAE,IAAI,EAAE;KAChB;CACH,CAAC;AAEF,gGAAgG;AAChG,MAAM,CAAC,MAAM,IAAI,GAAG,IAAI,CAAC,MAAM,CAAwB,iBAAiB,CAAC,CAAC"}
+{"version":3,"file":"server-log-fixture.js","sourceRoot":"","sources":["../../../src/testing/playwright/server-log-fixture.ts"],"names":[],"mappings":"AAAA;;;;;;;kFAOkF;AAElF,kFAAkF;AAClF,4EAA4E;AAC5E,oEAAoE;AACpE,OAAO,EAAE,IAAI,IAAI,IAAI,EAAsE,MAAM,kBAAkB,CAAC;AACpH,OAAO,EACJ,eAAe,EACf,6BAA6B,EAC7B,4BAA4B,EAC5B,aAAa,EACb,oBAAoB,EACtB,MAAM,yBAAyB,CAAC;AACjC,OAAO,EAAE,mBAAmB,EAAE,MAAM,wBAAwB,CAAC;AAkC7D;;;;;;;GAOG;AACH,MAAM,CAAC,MAAM,iBAAiB,GAAwF;IACnH,kBAAkB,EAAE,CAAC,EAAE,EAAE,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;IAC1C,cAAc,EAAE,CAAC,KAAK,EAAE,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;IACzC,kBAAkB,EAAE,CAAC,EAAE,EAAE,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;IAC1C,gBAAgB,EAAE;QACf,KAAK,EAAE,EAAE,kBAAkB,EAAE,cAAc,EAAE,kBAAkB,EAAE,EAAE,GAAG,EAAiB,EAAE;YACtF,IAAI,CAAC,kBAAkB,EAAE,CAAC;gBACvB,MAAM,GAAG,CAAC,SAAS,CAAC,CAAC;gBACrB,OAAO;YACV,CAAC;YACD,aAAa,CAAC,IAAI,CAAC,IAAI,EAAE,EAAE,kBAAkB,CAAC,CAAC;YAC/C,MAAM,GAAG,CAAC,SAAS,CAAC,CAAC;YACrB,MAAM,IAAI,GAAG,oBAAoB,CAAC,kBAAkB,CAAC,CAAC;YACtD,MAAM,MAAM,GAAG,IAAI,CAAC,CAAC,CAAC,mBAAmB,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;YACrD,IAAI,OAA2B,CAAC;YAChC,IAAI,cAAc,EAAE,CAAC;gBAClB,MAAM,UAAU,GAAG,4BAA4B,CAAC,MAAM,EAAE,kBAAkB,CAAC,CAAC;gBAC5E,IAAI,UAAU,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;oBACzB,OAAO,GAAG,qBAAqB,UAAU,CAAC,MAAM,uCAAuC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;gBAClH,CAAC;YACJ,CAAC;YACD,sEAAsE;YACtE,uEAAuE;YACvE,MAAM,QAAQ,GAAG,OAAO,CAAC,GAAG,CAAC,6BAA6B,CAAC,KAAK,OAAO,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,QAAQ,CAAC;YAC7F,MAAM,eAAe,CAAC,IAAI,CAAC,IAAI,EAAE,EAAE,kBAAkB,EAAE,OAAO,CAAC,CAAC,CAAC,EAAE,QAAQ,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;YACpF,IAAI,OAAO,EAAE,CAAC;gBACX,MAAM,IAAI,KAAK,CAAC,OAAO,CAAC,CAAC;YAC5B,CAAC;QACJ,CAAC;QACD,EAAE,IAAI,EAAE,IAAI,EAAE;KAChB;CACH,CAAC;AAEF,gGAAgG;AAChG,MAAM,CAAC,MAAM,IAAI,GAAG,IAAI,CAAC,MAAM,CAAwB,iBAAiB,CAAC,CAAC"}

package/lib/testing/playwright/server-log-region.d.ts

@@ -0,0 +1,15 @@
+/********************************************************************************
+ * Copyright (c) 2026 EclipseSource and others.
+ *
+ * This program and the accompanying materials are made available under the
+ * terms of the MIT License which is available in the project root.
+ *
+ * SPDX-License-Identifier: MIT
+ ********************************************************************************/
+/**
+ * The part of `file` written since the last read of it, up to its last complete
+ * line, since the server may be mid-write. Every test reads, checked or not, so
+ * a test that is not checked leaves no lines to the next one.
+ */
+export declare function readServerLogRegion(file: string): string;
+//# sourceMappingURL=server-log-region.d.ts.map

package/lib/testing/playwright/server-log-region.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"server-log-region.d.ts","sourceRoot":"","sources":["../../../src/testing/playwright/server-log-region.ts"],"names":[],"mappings":"AAAA;;;;;;;kFAOkF;AAUlF;;;;GAIG;AACH,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CASxD"}

package/lib/testing/playwright/server-log-region.js

@@ -0,0 +1,30 @@
+/********************************************************************************
+ * Copyright (c) 2026 EclipseSource and others.
+ *
+ * This program and the accompanying materials are made available under the
+ * terms of the MIT License which is available in the project root.
+ *
+ * SPDX-License-Identifier: MIT
+ ********************************************************************************/
+import { existsSync, readFileSync } from 'node:fs';
+/**
+ * How far each log has been read. A suite starts its app before its first
+ * test's marker, so reading from the marker would leave startup unread.
+ */
+const readUpTo = new Map();
+/**
+ * The part of `file` written since the last read of it, up to its last complete
+ * line, since the server may be mid-write. Every test reads, checked or not, so
+ * a test that is not checked leaves no lines to the next one.
+ */
+export function readServerLogRegion(file) {
+    if (!existsSync(file)) {
+        return '';
+    }
+    const log = readFileSync(file);
+    const end = log.lastIndexOf('\n') + 1;
+    const text = log.subarray(readUpTo.get(file) ?? 0, end).toString('utf-8');
+    readUpTo.set(file, end);
+    return text;
+}
+//# sourceMappingURL=server-log-region.js.map

package/lib/testing/playwright/server-log-region.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"server-log-region.js","sourceRoot":"","sources":["../../../src/testing/playwright/server-log-region.ts"],"names":[],"mappings":"AAAA;;;;;;;kFAOkF;AAElF,OAAO,EAAE,UAAU,EAAE,YAAY,EAAE,MAAM,SAAS,CAAC;AAEnD;;;GAGG;AACH,MAAM,QAAQ,GAAG,IAAI,GAAG,EAAkB,CAAC;AAE3C;;;;GAIG;AACH,MAAM,UAAU,mBAAmB,CAAC,IAAY;IAC7C,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,EAAE,CAAC;QACrB,OAAO,EAAE,CAAC;IACb,CAAC;IACD,MAAM,GAAG,GAAG,YAAY,CAAC,IAAI,CAAC,CAAC;IAC/B,MAAM,GAAG,GAAG,GAAG,CAAC,WAAW,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;IACtC,MAAM,IAAI,GAAG,GAAG,CAAC,QAAQ,CAAC,QAAQ,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,GAAG,CAAC,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC;IAC1E,QAAQ,CAAC,GAAG,CAAC,IAAI,EAAE,GAAG,CAAC,CAAC;IACxB,OAAO,IAAI,CAAC;AACf,CAAC"}

package/lib/util/connection-liveness.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"connection-liveness.d.ts","sourceRoot":"","sources":["../../src/util/connection-liveness.ts"],"names":[],"mappings":"AAAA;;;;;;;kFAOkF;AAuClF;;;;;;;;;;;;;;;GAeG;AACH,wBAAgB,qBAAqB,CAAC,GAAG,EAAE,OAAO,GAAG,OAAO,CAO3D;AAOD;;;;;;;;;;GAUG;AACH,wBAAgB,iBAAiB,CAAC,GAAG,EAAE,OAAO,GAAG,OAAO,CASvD"}
+{"version":3,"file":"connection-liveness.d.ts","sourceRoot":"","sources":["../../src/util/connection-liveness.ts"],"names":[],"mappings":"AAAA;;;;;;;kFAOkF;AAyClF;;;;;;;;;;;;;;;GAeG;AACH,wBAAgB,qBAAqB,CAAC,GAAG,EAAE,OAAO,GAAG,OAAO,CAO3D;AAOD;;;;;;;;;;GAUG;AACH,wBAAgB,iBAAiB,CAAC,GAAG,EAAE,OAAO,GAAG,OAAO,CASvD"}

package/lib/util/connection-liveness.js

@@ -6,6 +6,7 @@
  *
  * SPDX-License-Identifier: MIT
  ********************************************************************************/
+import { describeError } from '@hydranium/protocol';
 /**
  * Phrases that mean a call lost its connection. Matched on the message because
  * the condition surfaces in shapes that share no discriminator:
@@ -60,7 +61,7 @@ export function isConnectionGoneError(err) {
     if (isDestroyedStreamError(err)) {
         return true;
     }
-    const message = err instanceof Error ? err.message : String(err);
+    const message = describeError(err);
     const normalized = message.toLowerCase();
     return CONNECTION_GONE_PATTERNS.some(pattern => pattern.test(normalized));
 }
@@ -85,7 +86,7 @@ export function isClientGoneError(err) {
     if (err instanceof Error && 'code' in err && typeof err.code === 'string' && SOCKET_GONE_CODES.has(err.code)) {
         return true;
     }
-    const message = err instanceof Error ? err.message : String(err);
+    const message = describeError(err);
     return SOCKET_GONE_PATTERN.test(message.toLowerCase());
 }
 //# sourceMappingURL=connection-liveness.js.map

package/lib/util/connection-liveness.js.map

@@ -1 +1 @@
-{"version":3,"file":"connection-liveness.js","sourceRoot":"","sources":["../../src/util/connection-liveness.ts"],"names":[],"mappings":"AAAA;;;;;;;kFAOkF;AAElF;;;;;;;;;;;;;;;;;;;;;;GAsBG;AACH,MAAM,wBAAwB,GAAG;IAC9B,wBAAwB;IACxB,sBAAsB;IACtB,yBAAyB;IACzB,uBAAuB;IACvB,8BAA8B;CAChC,CAAC;AAEF,MAAM,qBAAqB,GAAG,sBAAsB,CAAC;AAErD,SAAS,sBAAsB,CAAC,GAAY;IACzC,OAAO,GAAG,YAAY,KAAK,IAAI,MAAM,IAAI,GAAG,IAAI,GAAG,CAAC,IAAI,KAAK,qBAAqB,CAAC;AACtF,CAAC;AAED;;;;;;;;;;;;;;;GAeG;AACH,MAAM,UAAU,qBAAqB,CAAC,GAAY;IAC/C,IAAI,sBAAsB,CAAC,GAAG,CAAC,EAAE,CAAC;QAC/B,OAAO,IAAI,CAAC;IACf,CAAC;IACD,MAAM,OAAO,GAAG,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC;IACjE,MAAM,UAAU,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC;IACzC,OAAO,wBAAwB,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC,CAAC;AAC7E,CAAC;AAED,MAAM,iBAAiB,GAAG,IAAI,GAAG,CAAC,CAAC,OAAO,EAAE,YAAY,CAAC,CAAC,CAAC;AAE3D,4FAA4F;AAC5F,MAAM,mBAAmB,GAAG,wBAAwB,CAAC;AAErD;;;;;;;;;;GAUG;AACH,MAAM,UAAU,iBAAiB,CAAC,GAAY;IAC3C,IAAI,qBAAqB,CAAC,GAAG,CAAC,EAAE,CAAC;QAC9B,OAAO,IAAI,CAAC;IACf,CAAC;IACD,IAAI,GAAG,YAAY,KAAK,IAAI,MAAM,IAAI,GAAG,IAAI,OAAO,GAAG,CAAC,IAAI,KAAK,QAAQ,IAAI,iBAAiB,CAAC,GAAG,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC;QAC5G,OAAO,IAAI,CAAC;IACf,CAAC;IACD,MAAM,OAAO,GAAG,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC;IACjE,OAAO,mBAAmB,CAAC,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,CAAC,CAAC;AAC1D,CAAC"}
+{"version":3,"file":"connection-liveness.js","sourceRoot":"","sources":["../../src/util/connection-liveness.ts"],"names":[],"mappings":"AAAA;;;;;;;kFAOkF;AAElF,OAAO,EAAE,aAAa,EAAE,MAAM,qBAAqB,CAAC;AAEpD;;;;;;;;;;;;;;;;;;;;;;GAsBG;AACH,MAAM,wBAAwB,GAAG;IAC9B,wBAAwB;IACxB,sBAAsB;IACtB,yBAAyB;IACzB,uBAAuB;IACvB,8BAA8B;CAChC,CAAC;AAEF,MAAM,qBAAqB,GAAG,sBAAsB,CAAC;AAErD,SAAS,sBAAsB,CAAC,GAAY;IACzC,OAAO,GAAG,YAAY,KAAK,IAAI,MAAM,IAAI,GAAG,IAAI,GAAG,CAAC,IAAI,KAAK,qBAAqB,CAAC;AACtF,CAAC;AAED;;;;;;;;;;;;;;;GAeG;AACH,MAAM,UAAU,qBAAqB,CAAC,GAAY;IAC/C,IAAI,sBAAsB,CAAC,GAAG,CAAC,EAAE,CAAC;QAC/B,OAAO,IAAI,CAAC;IACf,CAAC;IACD,MAAM,OAAO,GAAG,aAAa,CAAC,GAAG,CAAC,CAAC;IACnC,MAAM,UAAU,GAAG,OAAO,CAAC,WAAW,EAAE,CAAC;IACzC,OAAO,wBAAwB,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC,CAAC;AAC7E,CAAC;AAED,MAAM,iBAAiB,GAAG,IAAI,GAAG,CAAC,CAAC,OAAO,EAAE,YAAY,CAAC,CAAC,CAAC;AAE3D,4FAA4F;AAC5F,MAAM,mBAAmB,GAAG,wBAAwB,CAAC;AAErD;;;;;;;;;;GAUG;AACH,MAAM,UAAU,iBAAiB,CAAC,GAAY;IAC3C,IAAI,qBAAqB,CAAC,GAAG,CAAC,EAAE,CAAC;QAC9B,OAAO,IAAI,CAAC;IACf,CAAC;IACD,IAAI,GAAG,YAAY,KAAK,IAAI,MAAM,IAAI,GAAG,IAAI,OAAO,GAAG,CAAC,IAAI,KAAK,QAAQ,IAAI,iBAAiB,CAAC,GAAG,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC;QAC5G,OAAO,IAAI,CAAC;IACf,CAAC;IACD,MAAM,OAAO,GAAG,aAAa,CAAC,GAAG,CAAC,CAAC;IACnC,OAAO,mBAAmB,CAAC,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,CAAC,CAAC;AAC1D,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hydranium/core",
-  "version": "1.0.0-next.267",
+  "version": "1.0.0-next.269",
   "description": "Foundational runtime of the hydranium framework: AST coordination layer (services, integrity, AST extension, serialization, multi-client documents) plus the LSP textual head at the /lsp subpath. Consumed by protocol-head packages (@hydranium/data-server, @hydranium/glsp-server).",
   "keywords": [
     "ast",
@@ -83,8 +83,8 @@
     "diff": "^5.2.0"
   },
   "devDependencies": {
-    "@hydranium/langium": "1.0.0-next.267",
-    "@hydranium/protocol": "1.0.0-next.267",
+    "@hydranium/langium": "1.0.0-next.269",
+    "@hydranium/protocol": "1.0.0-next.269",
     "@playwright/test": "^1.40.0",
     "@types/diff": "^5.2.0",
     "rimraf": "^5.0.0",
@@ -96,8 +96,8 @@
     "vscode-languageserver-types": "^3.17.5"
   },
   "peerDependencies": {
-    "@hydranium/langium": "1.0.0-next.267",
-    "@hydranium/protocol": "1.0.0-next.267",
+    "@hydranium/langium": "1.0.0-next.269",
+    "@hydranium/protocol": "1.0.0-next.269",
     "@playwright/test": "^1.40.0",
     "vscode-jsonrpc": "^9.0.0",
     "vscode-languageserver": "~10.0.1",

package/src/documents/ast-document-manager.ts

@@ -17,7 +17,8 @@ import {
    asModelVersion,
    type ModelVersion,
    type TextVersion,
-   UNRECORDED_VERSION
+   UNRECORDED_VERSION,
+   describeErrorWithStack
 } from '@hydranium/protocol';
 import { type AstNode, DocumentState, type LangiumDocument, type LangiumDocuments, URI, UriUtils } from '@hydranium/langium';
 import { type LogNameOptions } from '../langium/diagnostics/logger.js';
@@ -443,8 +444,11 @@ export class DefaultAstDocumentManager<
          }
          const built = this.getDocument(args.uri);
          if (creates && built) {
-            // Without an LSP head nothing else builds the opened text.
-            void this.services.workspace.VersionSyncService.syncTo(built.uri, this.textDocuments.version(args.uri));
+            // Without an LSP head nothing else builds the opened text. Nothing
+            // waits on the build either, so its failure ends here.
+            this.services.workspace.VersionSyncService.syncTo(built.uri, this.textDocuments.version(args.uri))?.catch((error: unknown) =>
+               this.tracer.with(args.uri).error(`The build of the opened text failed. ${describeErrorWithStack(error)}`)
+            );
          }
       }
       return Disposable.create(() => this.close(args));

package/src/documents/document-release-handler.ts

@@ -8,7 +8,7 @@
  ********************************************************************************/
 
 import { type URI, UriUtils } from '@hydranium/langium';
-import { type CanonicalUri, type Tracer } from '@hydranium/protocol';
+import { type CanonicalUri, type Tracer, describeError, describeErrorWithStack } from '@hydranium/protocol';
 import { type CancellationToken, type Disposable } from 'vscode-languageserver';
 import { type LogNameOptions } from '../langium/diagnostics/logger.js';
 import { HYDRANIUM_BUILD_REASONS } from '../langium/document-builder/document-builder.js';
@@ -38,7 +38,7 @@ export interface ReleasedDocument {
  */
 export class DocumentReleaseSkippedError extends Error {
    constructor(cause: unknown) {
-      super(`Release skipped: the connection or the workspace went away. ${cause instanceof Error ? cause.message : String(cause)}`, {
+      super(`Release skipped: the connection or the workspace went away. ${describeError(cause)}`, {
          cause
       });
       this.name = 'DocumentReleaseSkippedError';
@@ -149,16 +149,13 @@ export class DefaultDocumentReleaseHandler implements DocumentReleaseHandler {
          }
          // The build holds the file's text: only what followed its parse, such as the diagnostics publish, was lost.
          const { error } = outcome;
-         this.tracer
-            .with(released.uri)
-            .debug(`Revert on release skipped after its parse: ${error instanceof Error ? error.message : String(error)}`);
+         this.tracer.with(released.uri).debug(`Revert on release skipped after its parse: ${describeError(error)}`);
          return;
       }
       if (outcome.kind === 'failed') {
+         // Handled below, by a recovery build or the client holding the document again; one that fails rejects on its own.
          const { error } = outcome;
-         this.tracer
-            .with(released.uri)
-            .error(`Revert on release dropped. ${error instanceof Error ? (error.stack ?? error.message) : String(error)}`);
+         this.tracer.with(released.uri).warn(`Revert on release failed; building in its place. ${describeErrorWithStack(error)}`);
       }
       const parsed = 'parsed' in outcome && outcome.parsed;
       // Without it, a revert stopped short, or an override that built nothing,
@@ -171,15 +168,12 @@ export class DefaultDocumentReleaseHandler implements DocumentReleaseHandler {
             this.services.workspace.LangiumDocuments.getDocument(target) !== undefined) ||
          outcome.kind === 'failed';
       const recover = async (): Promise<void> => {
-         const failure = 'Build after a revert that stopped short failed; the build keeps the released text';
-         let built: boolean;
          try {
-            built = await this.requestRecoveryBuild(released, target, 'decision' in outcome && outcome.decision === 'remove');
+            await this.requestRecoveryBuild(released, target, 'decision' in outcome && outcome.decision === 'remove');
          } catch (err: unknown) {
-            throw new Error(`${failure}. ${err instanceof Error ? err.message : String(err)}`, { cause: err });
-         }
-         if (!built) {
-            throw new Error(failure);
+            throw new Error(`Build after a revert that stopped short failed; the build keeps the released text. ${describeError(err)}`, {
+               cause: err
+            });
          }
       };
       await this.waitForReleaseBuild(released, target, parsed, stoppedShort && !released.isReclaimed() ? recover : undefined);
@@ -290,8 +284,8 @@ export class DefaultDocumentReleaseHandler implements DocumentReleaseHandler {
       }
    }
 
-   /** The build in place of a release that stopped short; answers whether it ran. */
-   protected requestRecoveryBuild(released: ReleasedDocument, target: URI, deleted: boolean): Promise<boolean> {
+   /** The build in place of a release that stopped short; rejects with its error when it was given up. */
+   protected requestRecoveryBuild(released: ReleasedDocument, target: URI, deleted: boolean): Promise<void> {
       return this.services.workspace.VersionSyncService.requestRecoveryBuild(target, {
          deleted,
          reason: HYDRANIUM_BUILD_REASONS.didRelease,
@@ -319,8 +313,8 @@ export class DefaultDocumentReleaseHandler implements DocumentReleaseHandler {
       const workspace = this.services.workspace;
       // A client holding the document again keeps what the build holds for it.
       if (parsed || workspace.LangiumDocuments.getDocument(target) === undefined || released.isReclaimed()) {
-         // Settled now, so a failed recovery build is only logged.
-         startRecovery?.().catch((err: unknown) => this.tracer.with(released.uri).error(err instanceof Error ? err.message : String(err)));
+         // The release got what it waits for, so a failed recovery build is diagnosis; its retry warned.
+         startRecovery?.().catch((err: unknown) => this.tracer.with(released.uri).debug(describeErrorWithStack(err)));
          return Promise.resolve();
       }
       return new Promise<void>((resolve, reject) => {
@@ -352,9 +346,9 @@ export class DefaultDocumentReleaseHandler implements DocumentReleaseHandler {
                }
             },
             (err: unknown) => {
-               // Settled by a parse that came first, so only logged.
+               // Settled by a parse or removal that came first, so diagnosis; its retry warned.
                if (ended) {
-                  this.tracer.with(released.uri).error(err instanceof Error ? err.message : String(err));
+                  this.tracer.with(released.uri).debug(describeErrorWithStack(err));
                } else {
                   finish(() => reject(err));
                }

package/src/documents/hydranium-text-documents.ts

@@ -54,7 +54,9 @@ import {
    type TextVersion,
    STALE_VERSION,
    type Tracer,
-   UNRECORDED_VERSION
+   UNRECORDED_VERSION,
+   describeError,
+   describeErrorWithStack
 } from '@hydranium/protocol';
 import { type LogNameOptions } from '../langium/diagnostics/logger.js';
 import { LANGUAGE_CLIENT_ID } from './client-ids.js';
@@ -467,9 +469,11 @@ export class HydraniumTextDocuments<T extends TextDocument = TextDocument> exten
                return;
             }
          }
-         this.log(
+         // Diagnosis: one comes with every keystroke.
+         this.logUri(
             document.uri,
-            `Update to version ${document.version} by ${this.formatClientId(clientId)}${committed.changed ? '' : ' (content unchanged)'}`
+            `Update to version ${document.version} by ${this.formatClientId(clientId)}${committed.changed ? '' : ' (content unchanged)'}`,
+            'debug'
          );
          this.__onDidChangeContent.fire(Object.freeze({ document, clientId }));
       }
@@ -501,9 +505,10 @@ export class HydraniumTextDocuments<T extends TextDocument = TextDocument> exten
       // Unchanged text calls no update: a configuration may return a new document for one.
       const { document, changed } =
          synced.getText() === text ? { document: synced, changed: false } : this.commitChange(key, synced, [{ text }], clientId);
-      this.log(
+      this.logUri(
          document.uri,
-         `Update to version ${document.version} by ${this.formatClientId(clientId)}${changed ? '' : ' (content unchanged)'}`
+         `Update to version ${document.version} by ${this.formatClientId(clientId)}${changed ? '' : ' (content unchanged)'}`,
+         'debug'
       );
       this.__onDidChangeContent.fire(Object.freeze({ document, clientId }));
       return document.version;
@@ -643,9 +648,7 @@ export class HydraniumTextDocuments<T extends TextDocument = TextDocument> exten
             if (isDocumentReleaseSkippedError(err)) {
                this.tracer.with(released.uri).debug(err.message);
             } else {
-               this.tracer
-                  .with(released.uri)
-                  .error(`Release handler failed. ${err instanceof Error ? (err.stack ?? err.message) : String(err)}`);
+               this.tracer.with(released.uri).error(`Release handler failed. ${describeErrorWithStack(err)}`);
             }
             // Left unsettled, a watcher keeps the document dirty though the store answers clean.
             announceClean(false);
@@ -709,9 +712,7 @@ export class HydraniumTextDocuments<T extends TextDocument = TextDocument> exten
             this.services.workspace.FileSystemProvider.readFile(UriUtils.toUri(uri))
          );
       } catch (err: unknown) {
-         this.tracer
-            .with(uri)
-            .debug(`Editor save not announced: reading the file back failed. ${err instanceof Error ? err.message : String(err)}`);
+         this.tracer.with(uri).debug(`Editor save not announced: reading the file back failed. ${describeError(err)}`);
          return;
       }
       // What the file holds, not what the editor meant to write: an editor
@@ -1204,7 +1205,7 @@ export class HydraniumTextDocuments<T extends TextDocument = TextDocument> exten
             this.services.workspace.FileSystemProvider.readFile(UriUtils.toUri(key))
          );
       } catch (err: unknown) {
-         this.tracer.with(key).debug(`Disk baseline: the file cannot be read. ${err instanceof Error ? err.message : String(err)}`);
+         this.tracer.with(key).debug(`Disk baseline: the file cannot be read. ${describeError(err)}`);
       }
       this.setDiskBaseline(key, onDisk);
    }

package/src/documents/language-client-shadow.ts

@@ -333,10 +333,11 @@ export class DefaultLanguageClientShadow<T extends TextDocument = TextDocument>
          notifyOutcome: outcome => {
             if (!settled) {
                settled = true;
+               // Routine: an edit in the editor moved the version on before the push arrived.
                if (outcome === 'refused' && version !== null) {
                   this.tracer
                      .with(key)
-                     .warn(
+                     .debug(
                         `Language client refused applyEdit addressed at version ${version} (it last declared version ${state.declaredVersion})`
                      );
                }

package/src/documents/version-sync-service.ts

@@ -8,7 +8,7 @@
  ********************************************************************************/
 
 import { DocumentState, type LangiumDocument, type URI } from '@hydranium/langium';
-import { type Tracer, UNRECORDED_VERSION } from '@hydranium/protocol';
+import { type Tracer, UNRECORDED_VERSION, describeError, describeErrorWithStack } from '@hydranium/protocol';
 import { Disposable, Emitter, type Event } from 'vscode-languageserver-protocol';
 import { type LogNameOptions } from '../langium/diagnostics/logger.js';
 import { type ServerSharedServices } from '../langium/module.js';
@@ -46,20 +46,21 @@ export interface VersionSyncService {
    /**
     * Build `uri` from a `WorkspaceLock` read, so the build queues behind a
     * running one rather than cancelling it. Requests made while one for `uri`
-    * is pending share its next build. Resolves `true` once a build that carried
-    * this request completed, or once a check before a build found it no longer
+    * is pending share its next build. Resolves once a build that carried this
+    * request completed, or once a check before a build found it no longer
     * needed or deferred. A build a later write cancelled does not count: the
-    * request rides the next one. Resolves `false` once builds carrying this
-    * request have failed more often than it may retry, so a caller waiting on
-    * the build can stop. Never rejects.
+    * request rides the next one. Rejects with the last build's error once builds
+    * carrying this request have failed more often than it may retry, so a
+    * caller waiting on the build can stop; the caller ends that failure, and
+    * logs it if nothing else reports it.
     */
-   requestRecoveryBuild(uri: URI, request: RecoveryBuildRequest): Promise<boolean>;
+   requestRecoveryBuild(uri: URI, request: RecoveryBuildRequest): Promise<void>;
    /**
     * Build `uri` unless it is {@link isSyncedTo synced to} `textVersion`, asked
     * again inside the lock. `undefined` when no build is needed, else the
-    * {@link requestRecoveryBuild} answer.
+    * {@link requestRecoveryBuild} promise.
     */
-   syncTo(uri: URI, textVersion: number): Promise<boolean> | undefined;
+   syncTo(uri: URI, textVersion: number): Promise<void> | undefined;
    /**
     * Whether `uri`'s root was parsed from text at `textVersion` or later; requests
     * nothing. A root with no recorded version or a placeholder counts as synced,
@@ -85,7 +86,10 @@ export interface VersionSyncService {
 export interface QueuedRecoveryBuild {
    readonly request: RecoveryBuildRequest;
    failures: number;
-   readonly answer: (built: boolean) => void;
+   /** Settles the request once a build carried it or it was no longer needed. */
+   readonly resolve: () => void;
+   /** Settles the request once it has used its retries, with the last build's error. */
+   readonly reject: (error: unknown) => void;
 }
 
 /** The requests waiting on one URI's builds in {@link DefaultVersionSyncService}. */
@@ -102,7 +106,7 @@ export class DefaultVersionSyncService implements VersionSyncService {
    protected readonly recordedEmitter = new Emitter<LangiumDocument>();
    protected readonly deferredBuildChecks = new Set<(uri: URI) => boolean>();
    protected readonly pendingBuilds = new Map<string, PendingRecoveryBuilds>();
-   /** Retries of each request after a failed build before it is answered `false`. */
+   /** Retries of each request after a failed build before it is rejected. */
    protected readonly maxBuildRetries: number = 1;
 
    constructor(
@@ -142,19 +146,22 @@ export class DefaultVersionSyncService implements VersionSyncService {
          }
          ledger.record(root, version, origin.text);
       }
-      void this.syncTo(document.uri, store.version(uri));
+      // Nothing waits on this build, so its failure ends here.
+      this.syncTo(document.uri, store.version(uri))?.catch((error: unknown) =>
+         this.tracer.withUri(uri).error(`The build that would sync the root to its text failed. ${describeErrorWithStack(error)}`)
+      );
       this.recordedEmitter.fire(document);
    }
 
-   syncTo(uri: URI, textVersion: number): Promise<boolean> | undefined {
+   syncTo(uri: URI, textVersion: number): Promise<void> | undefined {
       const behind = (): boolean => !this.isSyncedTo(uri, textVersion);
       return behind() ? this.requestRecoveryBuild(uri, { stillNeeded: behind }) : undefined;
    }
 
-   requestRecoveryBuild(uri: URI, request: RecoveryBuildRequest): Promise<boolean> {
-      return new Promise<boolean>(answer => {
+   requestRecoveryBuild(uri: URI, request: RecoveryBuildRequest): Promise<void> {
+      return new Promise<void>((resolve, reject) => {
          const key = uri.toString();
-         const queued: QueuedRecoveryBuild = { request, failures: 0, answer };
+         const queued: QueuedRecoveryBuild = { request, failures: 0, resolve, reject };
          const pending = this.pendingBuilds.get(key);
          if (pending) {
             // Not dropped: the pending build may have read the text before this request's change.
@@ -189,8 +196,8 @@ export class DefaultVersionSyncService implements VersionSyncService {
 
    /**
     * Run `entry`'s requests, one build per batch, until none is queued, and
-    * answer each. A request a failed build carried is queued again
-    * {@link maxBuildRetries} times, then answered `false`; one a cancelled
+    * settle each. A request a failed build carried is queued again
+    * {@link maxBuildRetries} times, then rejected; one a cancelled
     * build carried is queued again without counting. The entry is dropped
     * in the same tick as the last check of the queue, so no request lands in
     * an entry nothing runs any more.
@@ -210,7 +217,7 @@ export class DefaultVersionSyncService implements VersionSyncService {
             await lock.read(() => {
                carried = entry.queued.splice(0);
                const needed = carried.filter(queued => this.isStillNeeded(uri, queued.request));
-               carried.filter(queued => !needed.includes(queued)).forEach(queued => queued.answer(true));
+               carried.filter(queued => !needed.includes(queued)).forEach(queued => queued.resolve());
                carried = needed;
                if (needed.length > 0) {
                   build = this.scheduleBatch(
@@ -227,18 +234,19 @@ export class DefaultVersionSyncService implements VersionSyncService {
                entry.queued.unshift(...carried);
                continue;
             }
-            carried.forEach(queued => queued.answer(true));
+            carried.forEach(queued => queued.resolve());
          } catch (err: unknown) {
-            const detail = err instanceof Error ? (err.stack ?? err.message) : String(err);
             const retried = carried.filter(queued => queued.failures++ < this.maxBuildRetries);
             const givenUp = carried.filter(queued => !retried.includes(queued));
             if (retried.length > 0) {
-               tracer.warn(`Requested build failed; retrying. ${detail}`);
+               // The only line about this failure when the retry succeeds.
+               tracer.warn(`Requested build failed; retrying. ${describeErrorWithStack(err)}`);
                entry.queued.unshift(...retried);
             }
             if (givenUp.length > 0) {
-               tracer.error(`Requested build failed again; giving up. ${detail}`);
-               givenUp.forEach(queued => queued.answer(false));
+               // Each requester ends the failure, so the log line is theirs.
+               tracer.debug(`Requested build failed again; giving up. ${describeError(err)}`);
+               givenUp.forEach(queued => queued.reject(err));
             }
          }
       }

package/src/langium/diagnostics/lsp-logger.ts

@@ -19,6 +19,7 @@ import {
    type LogThreshold,
    ObservableValue,
    type MaybeObservableValue,
+   describeErrorWithStack,
    parseLogLevel
 } from '@hydranium/protocol';
 import { type URI } from '@hydranium/langium';
@@ -308,7 +309,7 @@ function formatLogArg(value: unknown): string {
       return value;
    }
    if (value instanceof Error) {
-      return value.stack ?? value.message;
+      return describeErrorWithStack(value);
    }
    if (typeof value === 'object') {
       try {