refactor: standardize agent inbox config loading (#33)

* refactor: standardize agent inbox config loading

* refactor: reuse loaded relay config

Author: Doctor-wu

.env.example

@@ -5,4 +5,4 @@ dist/
 data/
 .agent-inbox/
 .worktrees/
-.DS_Store
+.DS_Store

.gitignore

@@ -28,7 +28,7 @@ A Discord bot that lets users interact with Codex directly in Discord threads.
 3. Tool usage (file reads, edits, bash commands) shown as formatted blocks
 4. Support `/ask` for quick questions without file system tools
 5. Error handling: timeouts, rate limits, graceful shutdown
-6. Use environment variables for all secrets (dotenv)
+6. Store secrets and runtime config in `~/.agent-inbox/config.jsonl`
 
 ## Code Style
 - ESM modules
@@ -41,5 +41,5 @@ A Discord bot that lets users interact with Codex directly in Discord threads.
 ## Setup
 - pnpm init, install deps
 - tsconfig.json with strict mode, ESM
-- .env.example with required vars
-- .gitignore for node_modules, .env, dist
+- `~/.agent-inbox/config.jsonl` with required records
+- .gitignore for node_modules, dist

AGENTS.md

@@ -28,7 +28,7 @@ A Discord bot that lets users interact with Claude Code directly in Discord thre
 3. Tool usage (file reads, edits, bash commands) shown as formatted blocks
 4. Support `/ask` for quick questions without file system tools
 5. Error handling: timeouts, rate limits, graceful shutdown
-6. Use environment variables for all secrets (dotenv)
+6. Store secrets and runtime config in `~/.agent-inbox/config.jsonl`
 
 ## Code Style
 - ESM modules
@@ -41,5 +41,5 @@ A Discord bot that lets users interact with Claude Code directly in Discord thre
 ## Setup
 - pnpm init, install deps
 - tsconfig.json with strict mode, ESM
-- .env.example with required vars
-- .gitignore for node_modules, .env, dist
+- `~/.agent-inbox/config.jsonl` with required records
+- .gitignore for node_modules, dist

CLAUDE.md

@@ -41,9 +41,12 @@ Configuration is stored in `~/.agent-inbox/config.jsonl`. Example:
 {"type":"meta","version":1}
 {"type":"im","id":"discord","enabled":true,"config":{"token":"your-bot-token","clientId":"your-client-id"}}
 {"type":"im","id":"feishu","enabled":false,"config":{"appId":"","appSecret":""}}
-{"type":"runtime","config":{"agentTimeoutMs":600000}}
+{"type":"im","id":"slack","enabled":false,"config":{"botToken":"","appToken":"","signingSecret":"","socketMode":true}}
+{"type":"runtime","config":{"agentTimeoutMs":600000,"claudeBin":"claude","codexBin":"codex","opencodeBin":"opencode"}}
 ```
 
+Standalone adapter runs use the same `~/.agent-inbox/config.jsonl` file. There is no repo-local `.env` bootstrap path.
+
 ### Step 3: Start
 
 ```bash
@@ -139,9 +142,10 @@ pnpm start
 # Development mode (run adapters independently)
 pnpm dev:discord
 pnpm dev:feishu
+pnpm dev:slack
 ```
 
-Use the repo root `.env` file for local environment variables during development. See `.env.example` for reference.
+All adapter development commands load configuration from `~/.agent-inbox/config.jsonl`.
 
 ### Build Outputs
 

README.md

@@ -7,12 +7,14 @@ describe('app config', () => {
     const parsed = parseConfigJsonl([
       '{"type":"meta","version":1}',
       '{"type":"im","id":"discord","enabled":true,"note":"discord","config":{"token":"abc","clientId":"123"}}',
+      '{"type":"im","id":"slack","enabled":true,"config":{"botToken":"xoxb","appToken":"xapp","signingSecret":"secret"}}',
       '{"type":"im","id":"feishu","enabled":true,"config":{"appId":"app-1"}}',
       '{"type":"runtime","config":{"agentTimeoutMs":1200}}',
     ].join('\n'));
 
-    expect(parsed.availableIms).toHaveLength(1);
+    expect(parsed.availableIms).toHaveLength(2);
     expect(parsed.availableIms[0]?.id).toBe('discord');
+    expect(parsed.availableIms[1]?.id).toBe('slack');
     expect(parsed.runtime.agentTimeoutMs).toBe(1200);
   });
 
@@ -30,14 +32,24 @@ describe('app config', () => {
   it('ignores invalid last used platform preference values', () => {
     const parsed = parseConfigJsonl([
       '{"type":"meta","version":1}',
-      '{"type":"local-preferences","lastUsedPlatform":"slack"}',
+      '{"type":"local-preferences","lastUsedPlatform":"teams"}',
       '{"type":"im","id":"discord","enabled":true,"config":{"token":"abc","clientId":"123"}}',
     ].join('\n'));
 
     expect((parsed as { lastUsedPlatform?: string }).lastUsedPlatform).toBeUndefined();
     expect(parsed.errors).toHaveLength(0);
   });
 
+  it('accepts Slack as a valid last used platform', () => {
+    const parsed = parseConfigJsonl([
+      '{"type":"meta","version":1}',
+      '{"type":"local-preferences","lastUsedPlatform":"slack"}',
+      '{"type":"im","id":"slack","enabled":true,"config":{"botToken":"xoxb","appToken":"xapp","signingSecret":"secret"}}',
+    ].join('\n'));
+
+    expect((parsed as { lastUsedPlatform?: string }).lastUsedPlatform).toBe('slack');
+  });
+
   it('reports malformed lines without crashing the whole file', () => {
     const parsed = parseConfigJsonl('{"type":"meta","version":1}\nnope');
 

apps/agent-inbox/src/__tests__/config.test.ts

@@ -27,4 +27,31 @@ describe('runtime dispatch', () => {
     expect(process.env['DISCORD_TOKEN']).toBe('discord-token');
     expect(process.env['AGENT_TIMEOUT_MS']).toBe('1234');
   });
+
+  it('dispatches to the Slack runtime and applies Slack env vars', async () => {
+    const startSlackRuntime = vi.fn(async () => {});
+    const selectedIm: AvailableIm = {
+      id: 'slack',
+      config: {
+        botToken: 'xoxb-test-token',
+        appToken: 'xapp-test-token',
+        signingSecret: 'test-signing-secret',
+        socketMode: false,
+      },
+    };
+
+    await startSelectedIm(
+      selectedIm,
+      { agentTimeoutMs: 4321 },
+      resolveRelayPaths('/tmp/runtime-dispatch-slack'),
+      {
+        slack: async () => ({ startSlackRuntime }),
+      },
+    );
+
+    expect(startSlackRuntime).toHaveBeenCalledOnce();
+    expect(process.env['SLACK_BOT_TOKEN']).toBe('xoxb-test-token');
+    expect(process.env['SLACK_SOCKET_MODE']).toBe('false');
+    expect(process.env['AGENT_TIMEOUT_MS']).toBe('4321');
+  });
 });

apps/agent-inbox/src/__tests__/runtime.test.ts

@@ -123,4 +123,28 @@ describe('setup flow', () => {
       expect.objectContaining({ message: 'Encrypt key (optional)' }),
     );
   });
+
+  it('writes a slack IM record through the interactive setup flow', async () => {
+    const tempHome = await mkdtemp(join('/tmp', 'agent-inbox-setup-'));
+    const paths = resolveRelayPaths(tempHome);
+
+    (prompts as any).__setResponses([
+      'slack',
+      'xoxb-test-token',
+      'xapp-test-token',
+      'test-signing-secret',
+      true,
+    ]);
+
+    const loaded = await runSetup(paths, ['discord', 'feishu', 'slack']);
+
+    expect(loaded.availableIms).toEqual(
+      expect.arrayContaining([
+        expect.objectContaining({ id: 'slack' }),
+      ]),
+    );
+
+    const saved = await readFile(paths.configFile, 'utf-8');
+    expect(saved).toContain('"id":"slack"');
+  });
 });