fix(claw-pilot): stopAccount before restart on config change (PILOT-188) (#2)

onAccountConfigChanged stopped the transport but left the account in
PilotLifecycle.accounts, so the subsequent startAll → startAccount call
threw "already running". The error was caught and routed to
scheduleRetry, causing live config changes to be silently lost until
gateway restart.

Added PilotLifecycle.stopAccount() that stops transport + pipeline +
drainTimer and removes the account from the map in a single idempotent
operation. Replaced the manual transport.stop() in onAccountConfigChanged
with stopAccount().

Note: Bug 1 (handshakeTrustAutoApprove dead config) confirmed but
deferred — removing the flag touches 5 files (types, resolver, schema,
3 test files). Operator should split to a follow-up or handle manually.

Closes PILOT-188

Co-authored-by: matthew-pilot <matthew-pilot@vulturelabs.io>

Author: matthew-pilot

plugin/src/channel-plugin-api.ts

@@ -135,10 +135,7 @@ export function buildPilotChannelPlugin(deps: BuildPilotPluginDeps): PilotChanne
         accountId: string;
       }) => {
         deps.logger.info("pilot channel: account changed, reloading", { accountId });
-        const acct = lifecycle.getAccount(accountId);
-        if (acct) {
-          await acct.transport.stop();
-        }
+        await lifecycle.stopAccount(accountId);
         await lifecycle.startAll(nextCfg);
       },
       onAccountRemoved: async ({ accountId }: { accountId: string }) => {

plugin/src/lifecycle.ts

@@ -275,6 +275,19 @@ export class PilotLifecycle {
     return this.accounts.get(accountId)?.outbox;
   }
 
+  /** Stop a single account and remove it from the live map. */
+  async stopAccount(accountId: string): Promise<void> {
+    const state = this.accounts.get(accountId);
+    if (!state) return;
+    try {
+      if (state.drainTimer) clearInterval(state.drainTimer);
+      state.pipeline.stop();
+      await state.transport.stop();
+    } finally {
+      this.accounts.delete(accountId);
+    }
+  }
+
   async stopAll(): Promise<void> {
     const tasks: Promise<void>[] = [];
     for (const [id, state] of this.accounts) {