refactor: remove deprecated gateway alias and legacy compat

- Remove the hidden `agentio gateway` alias of `agentio daemon`
- Delete the config.gateway → config.daemon migration and the legacy
  `gateway` config field
- Drop the AGENTIO_GATEWAY_URL / AGENTIO_GATEWAY_API_KEY env fallbacks
  (AGENTIO_DAEMON_* only)

Typecheck clean; 215 tests pass.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: plosson

src/commands/config-import.test.ts

@@ -147,11 +147,6 @@ const SAMPLE_SERVER: ServerStateLike = {
   ],
 };
 
-const SAMPLE_GATEWAY = {
-  apiKey: 'gw_preserved_gateway_key',
-  server: { port: 7890, host: '0.0.0.0' },
-};
-
 /* ------------------------------------------------------------------ */
 /* replace mode preserves config.server                               */
 /* ------------------------------------------------------------------ */
@@ -194,59 +189,6 @@ describe('config import (replace mode) — preserves config.server', () => {
     expect(server.clients).toEqual(SAMPLE_SERVER.clients);
   });
 
-  test('preserves config.gateway across import (migrated to daemon)', async () => {
-    await writeConfig({
-      profiles: { rss: [{ name: 'feeds' }] },
-      gateway: SAMPLE_GATEWAY,
-    });
-    const { key, blob } = await exportCurrentConfig();
-
-    await writeConfig({
-      profiles: {},
-      gateway: SAMPLE_GATEWAY,
-    });
-    const importRes = await runCli(['config', 'import'], {
-      AGENTIO_KEY: key,
-      AGENTIO_CONFIG: blob,
-    });
-    expect(importRes.exitCode).toBe(0);
-
-    // loadConfig migrates gateway → daemon on first load; the vault is
-    // re-persisted with daemon, so readConfig() (which reads the raw vault)
-    // sees daemon, not gateway.
-    const final = await readConfig();
-    expect(final.daemon).toEqual(SAMPLE_GATEWAY);
-    expect(final.gateway).toBeUndefined();
-    expect(profileNames(final, 'rss')).toEqual(['feeds']);
-  });
-
-  test('preserves both server and gateway in the same config (gateway migrated to daemon)', async () => {
-    await writeConfig({
-      profiles: { gmail: [{ name: 'p1' }] },
-      server: SAMPLE_SERVER,
-      gateway: SAMPLE_GATEWAY,
-    });
-    const { key, blob } = await exportCurrentConfig();
-
-    await writeConfig({
-      profiles: {},
-      server: SAMPLE_SERVER,
-      gateway: SAMPLE_GATEWAY,
-    });
-    const importRes = await runCli(['config', 'import'], {
-      AGENTIO_KEY: key,
-      AGENTIO_CONFIG: blob,
-    });
-    expect(importRes.exitCode).toBe(0);
-
-    // loadConfig migrates gateway → daemon on first load; the vault is
-    // re-persisted with daemon, not gateway.
-    const final = await readConfig();
-    expect(final.server).toEqual(SAMPLE_SERVER);
-    expect(final.daemon).toEqual(SAMPLE_GATEWAY);
-    expect(final.gateway).toBeUndefined();
-  });
-
   test('replace still REPLACES profiles (not merge)', async () => {
     await writeConfig({
       profiles: { gmail: [{ name: 'p1' }] },

src/config/config-manager.test.ts

@@ -2,7 +2,7 @@ import { afterEach, beforeEach, describe, expect, test } from 'bun:test';
 import { mkdtemp, rm } from 'fs/promises';
 import { tmpdir } from 'os';
 import { join } from 'path';
-import { migrateGatewayToDaemon, resolveProfile, setProfile } from './config-manager';
+import { resolveProfile, setProfile } from './config-manager';
 import { clearVaultCache, saveVault, CURRENT_VAULT_VERSION } from '../vault/vault';
 import {
   clearPassphraseCache,
@@ -60,30 +60,3 @@ describe('resolveProfile multi-profile case', () => {
     }
   });
 });
-
-describe('migrateGatewayToDaemon', () => {
-  test('copies gateway into daemon when daemon absent', () => {
-    const input = { profiles: {}, gateway: { apiKey: 'k' } };
-    const out = migrateGatewayToDaemon(input);
-    expect(out.daemon).toEqual({ apiKey: 'k' });
-    expect(out.gateway).toBeUndefined();
-  });
-
-  test('leaves daemon alone when already present', () => {
-    const input = {
-      profiles: {},
-      gateway: { apiKey: 'old' },
-      daemon: { apiKey: 'new' },
-    };
-    const out = migrateGatewayToDaemon(input);
-    expect(out.daemon?.apiKey).toBe('new');
-    expect(out.gateway).toBeUndefined();
-  });
-
-  test('is a no-op when neither is present', () => {
-    const input = { profiles: {} };
-    const out = migrateGatewayToDaemon(input);
-    expect(out.daemon).toBeUndefined();
-    expect(out.gateway).toBeUndefined();
-  });
-});

src/config/config-manager.ts

@@ -30,34 +30,9 @@ export async function ensureConfigDir(): Promise<void> {
   }
 }
 
-/**
- * Migrate legacy `config.gateway` into `config.daemon` in place.
- * Pure: returns a new Config value; does not mutate input.
- * - If `daemon` already exists, drops `gateway` untouched.
- * - Otherwise moves `gateway` verbatim under `daemon`.
- */
-export function migrateGatewayToDaemon(config: Config): Config {
-  if (!config.gateway) return config;
-  if (config.daemon) {
-    const { gateway: _drop, ...rest } = config;
-    return rest;
-  }
-  const { gateway, ...rest } = config;
-  return { ...rest, daemon: gateway };
-}
-
 export async function loadConfig(): Promise<Config> {
   const vault = await loadVault();
-  const migrated = migrateGatewayToDaemon(vault.config);
-  // If migration changed anything, persist it.
-  if (migrated !== vault.config) {
-    await saveVault({
-      version: CURRENT_VAULT_VERSION,
-      config: migrated,
-      credentials: vault.credentials,
-    });
-  }
-  return migrated;
+  return vault.config;
 }
 
 export async function saveConfig(config: Config): Promise<void> {

src/daemon/client.ts

@@ -1,7 +1,7 @@
 import { CliError } from '../utils/errors';
 import { loadConfig } from '../config/config-manager';
 import { getEnv } from '../config/config-manager';
-import type { GatewayConfig, HealthResponse } from './types';
+import type { HealthResponse } from './types';
 
 let cachedConfig: { url: string; apiKey: string } | null = null;
 
@@ -11,25 +11,19 @@ let cachedConfig: { url: string; apiKey: string } | null = null;
 async function getDaemonConnection(): Promise<{ url: string; apiKey: string }> {
   if (cachedConfig) return cachedConfig;
 
-  // Check environment variables first; legacy AGENTIO_GATEWAY_* still works.
+  // Check environment variables first.
   const envUrl =
-    process.env.AGENTIO_DAEMON_URL || process.env.AGENTIO_GATEWAY_URL ||
-    await getEnv('AGENTIO_DAEMON_URL') || await getEnv('AGENTIO_GATEWAY_URL');
+    process.env.AGENTIO_DAEMON_URL || await getEnv('AGENTIO_DAEMON_URL');
   const envApiKey =
-    process.env.AGENTIO_DAEMON_API_KEY || process.env.AGENTIO_GATEWAY_API_KEY ||
-    await getEnv('AGENTIO_DAEMON_API_KEY') || await getEnv('AGENTIO_GATEWAY_API_KEY');
+    process.env.AGENTIO_DAEMON_API_KEY || await getEnv('AGENTIO_DAEMON_API_KEY');
 
   if (envUrl) {
     cachedConfig = { url: envUrl, apiKey: envApiKey || '' };
     return cachedConfig;
   }
 
-  // Load from config (post-migration: config.daemon; pre-migration: config.gateway)
-  const config = await loadConfig() as unknown as {
-    daemon?: GatewayConfig;
-    gateway?: GatewayConfig;
-  };
-  const daemonConfig = config.daemon ?? config.gateway;
+  const config = await loadConfig();
+  const daemonConfig = config.daemon;
 
   // Construct URL from server host:port (local daemon)
   const host = daemonConfig?.server?.host ?? '127.0.0.1';
@@ -64,7 +58,7 @@ async function request<T>(method: string, endpoint: string, body?: unknown): Pro
 
     if (!response.ok) {
       if (response.status === 401) {
-        throw new CliError('AUTH_FAILED', 'Daemon authentication failed', 'Check AGENTIO_DAEMON_API_KEY (or legacy AGENTIO_GATEWAY_API_KEY)');
+        throw new CliError('AUTH_FAILED', 'Daemon authentication failed', 'Check AGENTIO_DAEMON_API_KEY');
       }
 
       const errorData = await response.json().catch(() => ({})) as { error?: string };

src/index-program.ts

@@ -84,7 +84,6 @@ export function createProgram(): Command {
   registerMcpCommands(program);
   registerDocsCommand(program);
   registerDaemonCommands(program);
-  registerDaemonCommands(program, { base: 'gateway', deprecated: true });
   registerDoctorCommand(program);
   registerProfileCommands(program);
   registerReauthCommand(program);

src/types/config.test.ts

@@ -13,14 +13,4 @@ describe('DaemonConfig', () => {
     };
     expect(cfg.scheduler?.watchedFolders[0].path).toBe('/tmp/x');
   });
-
-  test('Config accepts both daemon and gateway (back-compat)', () => {
-    const cfg: Config = {
-      profiles: {},
-      daemon: { apiKey: 'a' },
-      gateway: { apiKey: 'b' },
-    };
-    expect(cfg.daemon?.apiKey).toBe('a');
-    expect(cfg.gateway?.apiKey).toBe('b');
-  });
 });

src/types/config.ts

@@ -56,7 +56,6 @@ export interface Config {
   };
   env?: Record<string, string>;
   daemon?: DaemonConfig;
-  gateway?: GatewayConfig;  // legacy; read-only, migrated to daemon on load
 }
 
 export type ServiceName = 'gdocs' | 'gdrive' | 'gmail' | 'gcal' | 'gtasks' | 'gchat' | 'gsheets' | 'gslides' | 'gscript' | 'github' | 'jira' | 'confluence' | 'slack' | 'telegram' | 'discourse' | 'sql';