- CLI/message: include a stable top-level `messageId` in `openclaw message --json` output when channel sends return one. (#84191) Thanks @100menotu001.

- Plugins/hooks: apply a default 30-second timeout to `before_compaction` and `after_compaction` hooks so a hung plugin handler no longer blocks compaction completion. (#84153)

- Discord: preserve disabled presentation buttons when adapting and rendering Discord message controls. (#84188) Thanks @100menotu001.

- Twitch: add a test-only client-manager registry reset helper so non-isolated Twitch tests can clear cached managers between cases. Fixes #83887. (#84244) Thanks @hclsys.

- Plugins/perf: thread explicit plugin discovery results through `loadBundledCapabilityRuntimeRegistry`, `resolveBundledPluginSources`, and `listChannelCatalogEntries` so callers that already hold a discovery result skip redundant filesystem walks. Thanks @SebTardif.

- harden update restart script creation [AI]. (#84088) Thanks @pgondhi987.

- Docker: keep the bundled Codex plugin in official release image keep lists so the default OpenAI agent harness remains available after Docker pruning. Fixes #83613. (#83626) Thanks @YuanHanzhong.

import { afterEach, describe, expect, it, vi } from "vitest";

import {

clearRegistryForTest,

getClientManager,

getOrCreateClientManager,

} from "./client-manager-registry.js";

import type { ChannelLogSink } from "./types.js";

function makeLogger(): ChannelLogSink {

return {

info: vi.fn(),

warn: vi.fn(),

error: vi.fn(),

debug: vi.fn(),

};

}

describe("client manager registry", () => {

afterEach(async () => {

await clearRegistryForTest();

});

it("clears cached managers for hot module test isolation", async () => {

const firstManager = getOrCreateClientManager("default", makeLogger());

const disconnectAll = vi.spyOn(firstManager, "disconnectAll");

expect(getClientManager("default")).toBe(firstManager);

expect(getOrCreateClientManager("default", makeLogger())).toBe(firstManager);

await clearRegistryForTest();

expect(disconnectAll).toHaveBeenCalledOnce();

expect(getClientManager("default")).toBeUndefined();

expect(getOrCreateClientManager("default", makeLogger())).not.toBe(firstManager);

});

});

registry.delete(accountId);

entry.logger.info(`Unregistered client manager for account: ${accountId}`);

}

/**

* Test-only: clear the module-level registry of all client manager entries.

*

* Mirrors the `clearForTest` escape hatch on `TwitchClientManager`. Without

* this, the module-level `registry` Map survives across tests when vitest

* is run with `--isolate=false` (or any harness that does not tear the

* module graph down between cases), and a stale entry from one test will

* shadow `getOrCreateClientManager` calls in subsequent tests, silently

* handing back another test's mocked logger/manager. See #83887.

*

* Production code MUST NOT call this. It disconnects cached managers before

* clearing the registry so tests do not leave handlers or clients behind.

*/

export async function clearRegistryForTest(): Promise<void> {

const entries = [...registry.values()];

try {

await Promise.all(entries.map((entry) => entry.manager.disconnectAll()));

} finally {

registry.clear();

}

}