refactor(extensions): remove unused helper exports

vincentkoc · vincentkoc · commit 1eaace70e3b4 · 2026-06-19T00:33:04.000+08:00
diff --git a/extensions/browser/src/browser/config.test.ts b/extensions/browser/src/browser/config.test.ts
@@ -11,7 +11,6 @@ import {
   resolveBrowserConfig,
   resolveManagedBrowserHeadlessMode,
   resolveProfile,
-  shouldStartLocalBrowserServer,
 } from "./config.js";
 import { getBrowserProfileCapabilities } from "./profile-capabilities.js";
 
@@ -47,7 +46,6 @@ describe("browser config", () => {
     expect(resolved.enabled).toBe(true);
     expect(resolved.controlPort).toBe(18791);
     expect(resolved.color).toBe("#FF4500");
-    expect(shouldStartLocalBrowserServer(resolved)).toBe(true);
     expect(resolved.cdpHost).toBe("127.0.0.1");
     expect(resolved.cdpProtocol).toBe("http");
     const profile = resolveProfile(resolved, resolved.defaultProfile);
diff --git a/extensions/browser/src/browser/config.ts b/extensions/browser/src/browser/config.ts
@@ -655,8 +655,3 @@ export function getManagedBrowserMissingDisplayError(
     `Set ${OPENCLAW_BROWSER_HEADLESS_ENV}=1, remove the headed override, or launch under Xvfb.`
   );
 }
-
-/** Return whether local browser control should start for a resolved config. */
-export function shouldStartLocalBrowserServer(_resolved: unknown) {
-  return true;
-}
diff --git a/extensions/device-pair/index.test.ts b/extensions/device-pair/index.test.ts
@@ -54,7 +54,6 @@ vi.mock("./notify.js", () => ({
   armPairNotifyOnce: vi.fn(async () => false),
   formatPendingRequests: vi.fn(() => "No pending device pairing requests."),
   handleNotifyCommand: vi.fn(async () => ({ text: "notify" })),
-  registerPairingNotifierService: vi.fn(),
 }));
 
 import {
diff --git a/extensions/device-pair/notify.ts b/extensions/device-pair/notify.ts
@@ -482,7 +482,3 @@ export function createPairingNotifierService(api: OpenClawPluginApi): OpenClawPl
     },
   };
 }
-
-export function registerPairingNotifierService(api: OpenClawPluginApi): void {
-  api.registerService(createPairingNotifierService(api));
-}
diff --git a/extensions/feishu/src/client.ts b/extensions/feishu/src/client.ts
@@ -236,13 +236,6 @@ export function createEventDispatcher(account: ResolvedFeishuAccount): Lark.Even
   });
 }
 
-/**
- * Get a cached client for an account (if exists).
- */
-export function getFeishuClient(accountId: string): Lark.Client | null {
-  return clientCache.get(accountId)?.client ?? null;
-}
-
 /**
  * Clear client cache for a specific account or all accounts.
  */
diff --git a/extensions/feishu/src/lifecycle.test-support.ts b/extensions/feishu/src/lifecycle.test-support.ts
@@ -144,7 +144,6 @@ vi.mock("./client.js", () => {
       start: vi.fn(),
     })),
     createEventDispatcher: createEventDispatcherMock,
-    getFeishuClient: vi.fn(() => null),
     getFeishuUserAgent: vi.fn(() => "openclaw-feishu-test"),
     pluginVersion: "test",
     setFeishuClientRuntimeForTest: vi.fn(),

Original file line number	Diff line number	Diff line change
`@@ -655,8 +655,3 @@ export function getManagedBrowserMissingDisplayError(`
`655`	`655`	`Set ${OPENCLAW_BROWSER_HEADLESS_ENV}=1, remove the headed override, or launch under Xvfb.`
`656`	`656`	`);`
`657`	`657`	`}`
`658`		`-`
`659`		`-/** Return whether local browser control should start for a resolved config. */`
`660`		`-export function shouldStartLocalBrowserServer(_resolved: unknown) {`
`661`		`- return true;`
`662`		`-}`
Original file line number	Diff line number	Diff line change
`@@ -482,7 +482,3 @@ export function createPairingNotifierService(api: OpenClawPluginApi): OpenClawPl`
`482`	`482`	`},`
`483`	`483`	`};`
`484`	`484`	`}`
`485`		`-`
`486`		`-export function registerPairingNotifierService(api: OpenClawPluginApi): void {`
`487`		`- api.registerService(createPairingNotifierService(api));`
`488`		`-}`