obsidian-integration-testing 12.1.2 → 12.3.0

package/dist/lib/cjs/transport-desktop-cdp.d.cts

@@ -37,7 +37,13 @@ export interface DesktopCdpTransportConfig {
     cdpPort?: number;
     /**
      * Timeout in milliseconds for individual CDP commands.
-     * Defaults to `30000`.
+     *
+     * A test's closure travels as one `Runtime.evaluate` command, so this is also
+     * the desktop per-eval cap. The default is `DEFAULT_EVAL_CAP_IN_MILLISECONDS`,
+     * exported from the package root — import it rather than restating the number,
+     * so a closure sized against the cap follows it if it ever moves.
+     *
+     * @default `30000`
      */
     commandTimeoutInMilliseconds?: number;
     /**
@@ -191,6 +197,17 @@ export declare class DesktopCdpTransport implements ObsidianTransport {
     private messageId;
     private readonly ownedConfig;
     private ownedInstance;
+    /**
+     * The in-flight commands of every socket this transport has opened, keyed by
+     * the CDP message id each is waiting for. One entry per command, added by
+     * {@link sendCommand} and removed by whichever of the reply and the timeout
+     * comes first — see {@link getPendingCommands} for why this exists at all.
+     *
+     * Weak by socket so a closed temporary connection is collectable with its
+     * pending map; the dispatcher listener is the socket's own, so nothing here
+     * keeps it alive.
+     */
+    private readonly pendingCommandsBySocket;
     /**
      * Vault paths THIS transport opened via {@link registerVault}, normalized by
      * {@link normalizeVaultPathForComparison}. A worker attached to a
@@ -474,6 +491,32 @@ export declare class DesktopCdpTransport implements ObsidianTransport {
      * @throws {Error} When no CDP endpoint is configured yet.
      */
     private getPageTargets;
+    /**
+     * Returns a socket's in-flight command map, installing its reply dispatcher on
+     * first use.
+     *
+     * ONE `message` listener per socket, for the socket's whole life — not one per
+     * command. A listener per in-flight command makes the count scale with how many
+     * evaluations a suite has in flight at once, so a consumer that fires eleven
+     * concurrent `evalInObsidian` calls crosses Node's default `maxListeners` of 10
+     * and is told its transport is leaking memory, which it is not: measured
+     * 2026-09-15, fifteen concurrent evaluations registered fifteen listeners on the
+     * one long-lived socket and the count fell back to zero the moment they settled.
+     * A warning that fires on correct use is worse than no warning, because the real
+     * signal — an entry that is added and never removed — is then indistinguishable
+     * from it. With one dispatcher the listener count is a constant, and a genuine
+     * leak shows instead as a pending map that never empties.
+     *
+     * The dispatcher is deliberately forgiving about what arrives: a CDP **event**
+     * carries no `id` and belongs to no command, and a frame that is not JSON at all
+     * belongs to nothing this transport sent. Both are ignored. The old per-command
+     * handler parsed unguarded, so one frame that is not JSON threw once per
+     * in-flight command, out of the event dispatch, where nothing could catch it.
+     *
+     * @param ws - The socket whose replies are being awaited.
+     * @returns The socket's map of in-flight commands, keyed by CDP message id.
+     */
+    private getPendingCommands;
     /**
      * Kills the currently-running owned instance (if any) and waits for it to exit,
      * so the next launch gets a pristine single-window instance.