refactor(gateway): drop unused helper methods

vincentkoc · vincentkoc · commit 9594300f8c92 · 2026-06-20T04:14:45.000+08:00
diff --git a/src/gateway/call.ts b/src/gateway/call.ts
@@ -449,10 +449,6 @@ export const testing = {
     gatewayCallDeps.loadDeviceAuthToken =
       deps?.loadDeviceAuthToken ?? defaultGatewayCallDeps.loadDeviceAuthToken;
   },
-  setCreateGatewayClientForTests(createGatewayClient?: typeof defaultCreateGatewayClient): void {
-    gatewayCallDeps.createGatewayClient =
-      createGatewayClient ?? defaultGatewayCallDeps.createGatewayClient;
-  },
   resetDepsForTests(): void {
     gatewayCallDeps.createGatewayClient = defaultGatewayCallDeps.createGatewayClient;
     gatewayCallDeps.getRuntimeConfig = defaultGatewayCallDeps.getRuntimeConfig;
diff --git a/src/gateway/exec-approval-manager.ts b/src/gateway/exec-approval-manager.ts
@@ -121,16 +121,6 @@ export class ExecApprovalManager<TPayload = ExecApprovalRequestPayload> {
     return promise;
   }
 
-  /**
-   * @deprecated Use register() instead for explicit separation of registration and waiting.
-   */
-  async waitForDecision(
-    record: ExecApprovalRecord<TPayload>,
-    timeoutMs: number,
-  ): Promise<ExecApprovalDecision | null> {
-    return this.register(record, timeoutMs);
-  }
-
   resolve(recordId: string, decision: ExecApprovalDecision, resolvedBy?: string | null): boolean {
     const pending = this.pending.get(recordId);
     if (!pending) {
diff --git a/src/gateway/node-registry.ts b/src/gateway/node-registry.ts
@@ -361,11 +361,6 @@ export class NodeRegistry {
     });
   }
 
-  /** Update command list while keeping it within the node's declared command surface. */
-  updateCommands(nodeId: string, commands: readonly string[]): NodeSession | null {
-    return this.updateSurface(nodeId, { commands });
-  }
-
   updateSurface(
     nodeId: string,
     surface: {
