@ai-devkit/agent-manager 0.28.1 → 0.30.0

package/dist/__tests__/adapters/ClaudeCodeAdapter.test.js

@@ -2,7 +2,7 @@
  * Tests for ClaudeCodeAdapter
  */ import * as fs from 'fs';
 import * as path from 'path';
-import { ClaudeCodeAdapter } from '../../adapters/ClaudeCodeAdapter.js';
+import { ClaudeCodeAdapter } from '../../providers/claude/ClaudeCodeAdapter.js';
 import { AgentStatus } from '../../adapters/AgentAdapter.js';
 import { listAgentProcesses, enrichProcesses, captureProcessSnapshot } from '../../utils/process.js';
 import { batchGetSessionFileBirthtimes } from '../../utils/session.js';
@@ -910,148 +910,6 @@ describe('ClaudeCodeAdapter', ()=>{
             });
         });
     });
-    describe('discoverSessions', ()=>{
-        let tmpDir;
-        beforeEach(()=>{
-            tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'claude-test-'));
-            adapter.projectsDir = path.join(tmpDir, 'projects');
-        });
-        afterEach(()=>{
-            fs.rmSync(tmpDir, {
-                recursive: true,
-                force: true
-            });
-        });
-        it('should return empty when projects dir does not exist', ()=>{
-            adapter.projectsDir = path.join(tmpDir, 'nonexistent');
-            const discoverSessions = adapter.discoverSessions.bind(adapter);
-            const result = discoverSessions([
-                {
-                    pid: 1,
-                    command: 'claude',
-                    cwd: '/test',
-                    tty: ''
-                }
-            ]);
-            expect(result).toEqual([]);
-        });
-        it('should scan only directories matching process CWDs', ()=>{
-            const projectsDir = path.join(tmpDir, 'projects');
-            adapter.projectsDir = projectsDir;
-            const discoverSessions = adapter.discoverSessions.bind(adapter);
-            // /my/project → -my-project (encoded dir)
-            const encodedDir = path.join(projectsDir, '-my-project');
-            fs.mkdirSync(encodedDir, {
-                recursive: true
-            });
-            // Also create another dir that should NOT be scanned
-            const otherDir = path.join(projectsDir, '-other-project');
-            fs.mkdirSync(otherDir, {
-                recursive: true
-            });
-            const mockFiles = [
-                {
-                    sessionId: 's1',
-                    filePath: path.join(encodedDir, 's1.jsonl'),
-                    projectDir: encodedDir,
-                    birthtimeMs: 1710800324000,
-                    resolvedCwd: ''
-                }
-            ];
-            mockedBatchGetSessionFileBirthtimes.mockReturnValue(mockFiles);
-            const processes = [
-                {
-                    pid: 1,
-                    command: 'claude',
-                    cwd: '/my/project',
-                    tty: ''
-                }
-            ];
-            const result = discoverSessions(processes);
-            expect(result).toHaveLength(1);
-            expect(result[0].resolvedCwd).toBe('/my/project');
-            // batchGetSessionFileBirthtimes called once with all dirs
-            expect(mockedBatchGetSessionFileBirthtimes).toHaveBeenCalledTimes(1);
-            expect(mockedBatchGetSessionFileBirthtimes).toHaveBeenCalledWith([
-                encodedDir
-            ]);
-        });
-        it('should return empty when encoded dir does not exist', ()=>{
-            const projectsDir = path.join(tmpDir, 'projects');
-            fs.mkdirSync(projectsDir, {
-                recursive: true
-            });
-            adapter.projectsDir = projectsDir;
-            const discoverSessions = adapter.discoverSessions.bind(adapter);
-            // Process CWD /test encodes to -test, but that dir doesn't exist
-            const result = discoverSessions([
-                {
-                    pid: 1,
-                    command: 'claude',
-                    cwd: '/test',
-                    tty: ''
-                }
-            ]);
-            expect(result).toEqual([]);
-            expect(mockedBatchGetSessionFileBirthtimes).not.toHaveBeenCalled();
-        });
-        it('should deduplicate when multiple processes share same CWD', ()=>{
-            const projectsDir = path.join(tmpDir, 'projects');
-            adapter.projectsDir = projectsDir;
-            const discoverSessions = adapter.discoverSessions.bind(adapter);
-            const encodedDir = path.join(projectsDir, '-my-project');
-            fs.mkdirSync(encodedDir, {
-                recursive: true
-            });
-            mockedBatchGetSessionFileBirthtimes.mockReturnValue([
-                {
-                    sessionId: 's1',
-                    filePath: path.join(encodedDir, 's1.jsonl'),
-                    projectDir: encodedDir,
-                    birthtimeMs: 1710800324000,
-                    resolvedCwd: ''
-                }
-            ]);
-            const processes = [
-                {
-                    pid: 1,
-                    command: 'claude',
-                    cwd: '/my/project',
-                    tty: ''
-                },
-                {
-                    pid: 2,
-                    command: 'claude',
-                    cwd: '/my/project',
-                    tty: ''
-                }
-            ];
-            const result = discoverSessions(processes);
-            // Should only call batch once with deduplicated dir
-            expect(mockedBatchGetSessionFileBirthtimes).toHaveBeenCalledTimes(1);
-            expect(mockedBatchGetSessionFileBirthtimes).toHaveBeenCalledWith([
-                encodedDir
-            ]);
-            expect(result).toHaveLength(1);
-        });
-        it('should skip processes with empty cwd', ()=>{
-            const projectsDir = path.join(tmpDir, 'projects');
-            fs.mkdirSync(projectsDir, {
-                recursive: true
-            });
-            adapter.projectsDir = projectsDir;
-            const discoverSessions = adapter.discoverSessions.bind(adapter);
-            const result = discoverSessions([
-                {
-                    pid: 1,
-                    command: 'claude',
-                    cwd: '',
-                    tty: ''
-                }
-            ]);
-            expect(result).toEqual([]);
-        });
-    });
     describe('helper methods', ()=>{
         describe('determineStatus', ()=>{
             it('should return "unknown" for sessions with no last entry type', ()=>{
@@ -1234,205 +1092,6 @@ describe('ClaudeCodeAdapter', ()=>{
                 force: true
             });
         });
-        describe('tryPidFileMatching', ()=>{
-            let sessionsDir;
-            let projectsDir;
-            beforeEach(()=>{
-                sessionsDir = path.join(tmpDir, 'sessions');
-                projectsDir = path.join(tmpDir, 'projects');
-                fs.mkdirSync(sessionsDir, {
-                    recursive: true
-                });
-                adapter.sessionsDir = sessionsDir;
-                adapter.projectsDir = projectsDir;
-            });
-            const makeProc = (pid, cwd = '/project/test', startTime)=>({
-                    pid,
-                    command: 'claude',
-                    cwd,
-                    tty: 'ttys001',
-                    startTime
-                });
-            const writePidFile = (pid, sessionId, cwd, startedAt)=>{
-                fs.writeFileSync(path.join(sessionsDir, `${pid}.json`), JSON.stringify({
-                    pid,
-                    sessionId,
-                    cwd,
-                    startedAt,
-                    kind: 'interactive',
-                    entrypoint: 'cli'
-                }));
-            };
-            const writeJsonl = (cwd, sessionId)=>{
-                const encoded = cwd.replace(/[^a-zA-Z0-9]/g, '-');
-                const projDir = path.join(projectsDir, encoded);
-                fs.mkdirSync(projDir, {
-                    recursive: true
-                });
-                const filePath = path.join(projDir, `${sessionId}.jsonl`);
-                fs.writeFileSync(filePath, JSON.stringify({
-                    type: 'assistant',
-                    timestamp: new Date().toISOString()
-                }));
-                return filePath;
-            };
-            it('should return direct match when PID file and JSONL both exist within time tolerance', ()=>{
-                const startTime = new Date();
-                const proc = makeProc(1001, '/project/test', startTime);
-                writePidFile(1001, 'session-abc', '/project/test', startTime.getTime());
-                writeJsonl('/project/test', 'session-abc');
-                const tryMatch = adapter.tryPidFileMatching.bind(adapter);
-                const { direct, fallback } = tryMatch([
-                    proc
-                ]);
-                expect(direct).toHaveLength(1);
-                expect(fallback).toHaveLength(0);
-                expect(direct[0].sessionFile.sessionId).toBe('session-abc');
-                expect(direct[0].sessionFile.resolvedCwd).toBe('/project/test');
-                expect(direct[0].process.pid).toBe(1001);
-            });
-            it('should fall back when PID file exists but JSONL is missing', ()=>{
-                const startTime = new Date();
-                const proc = makeProc(1002, '/project/test', startTime);
-                writePidFile(1002, 'nonexistent-session', '/project/test', startTime.getTime());
-                // No JSONL file written
-                const tryMatch = adapter.tryPidFileMatching.bind(adapter);
-                const { direct, fallback } = tryMatch([
-                    proc
-                ]);
-                expect(direct).toHaveLength(0);
-                expect(fallback).toHaveLength(1);
-                expect(fallback[0].pid).toBe(1002);
-            });
-            it('should fall back when startedAt is stale (>60s from proc.startTime)', ()=>{
-                const startTime = new Date();
-                const staleTime = startTime.getTime() - 90_000; // 90 seconds earlier
-                const proc = makeProc(1003, '/project/test', startTime);
-                writePidFile(1003, 'stale-session', '/project/test', staleTime);
-                writeJsonl('/project/test', 'stale-session');
-                const tryMatch = adapter.tryPidFileMatching.bind(adapter);
-                const { direct, fallback } = tryMatch([
-                    proc
-                ]);
-                expect(direct).toHaveLength(0);
-                expect(fallback).toHaveLength(1);
-            });
-            it('should accept PID file when startedAt is within 60s tolerance', ()=>{
-                const startTime = new Date();
-                const closeTime = startTime.getTime() - 30_000; // 30 seconds earlier — within tolerance
-                const proc = makeProc(1004, '/project/test', startTime);
-                writePidFile(1004, 'close-session', '/project/test', closeTime);
-                writeJsonl('/project/test', 'close-session');
-                const tryMatch = adapter.tryPidFileMatching.bind(adapter);
-                const { direct, fallback } = tryMatch([
-                    proc
-                ]);
-                expect(direct).toHaveLength(1);
-                expect(fallback).toHaveLength(0);
-            });
-            it('should fall back when PID file is absent', ()=>{
-                const proc = makeProc(1005, '/project/test', new Date());
-                // No PID file written
-                const tryMatch = adapter.tryPidFileMatching.bind(adapter);
-                const { direct, fallback } = tryMatch([
-                    proc
-                ]);
-                expect(direct).toHaveLength(0);
-                expect(fallback).toHaveLength(1);
-            });
-            it('should fall back when PID file contains malformed JSON', ()=>{
-                const proc = makeProc(1006, '/project/test', new Date());
-                fs.writeFileSync(path.join(sessionsDir, '1006.json'), 'not valid json {{{');
-                const tryMatch = adapter.tryPidFileMatching.bind(adapter);
-                expect(()=>{
-                    const { direct, fallback } = tryMatch([
-                        proc
-                    ]);
-                    expect(direct).toHaveLength(0);
-                    expect(fallback).toHaveLength(1);
-                }).not.toThrow();
-            });
-            it('should fall back for all processes when sessions dir does not exist', ()=>{
-                adapter.sessionsDir = path.join(tmpDir, 'nonexistent-sessions');
-                const processes = [
-                    makeProc(2001, '/a', new Date()),
-                    makeProc(2002, '/b', new Date())
-                ];
-                const tryMatch = adapter.tryPidFileMatching.bind(adapter);
-                const { direct, fallback } = tryMatch(processes);
-                expect(direct).toHaveLength(0);
-                expect(fallback).toHaveLength(2);
-            });
-            it('should correctly split mixed processes (some with PID files, some without)', ()=>{
-                const startTime = new Date();
-                const proc1 = makeProc(3001, '/project/one', startTime);
-                const proc2 = makeProc(3002, '/project/two', startTime);
-                const proc3 = makeProc(3003, '/project/three', startTime);
-                writePidFile(3001, 'session-one', '/project/one', startTime.getTime());
-                writeJsonl('/project/one', 'session-one');
-                writePidFile(3003, 'session-three', '/project/three', startTime.getTime());
-                writeJsonl('/project/three', 'session-three');
-                // proc2 has no PID file
-                const tryMatch = adapter.tryPidFileMatching.bind(adapter);
-                const { direct, fallback } = tryMatch([
-                    proc1,
-                    proc2,
-                    proc3
-                ]);
-                expect(direct).toHaveLength(2);
-                expect(fallback).toHaveLength(1);
-                expect(direct.map((d)=>d.process.pid).sort()).toEqual([
-                    3001,
-                    3003
-                ]);
-                expect(fallback[0].pid).toBe(3002);
-            });
-            it('should skip stale-file check when proc.startTime is undefined', ()=>{
-                const proc = makeProc(4001, '/project/test', undefined); // no startTime
-                writePidFile(4001, 'no-time-session', '/project/test', Date.now() - 999_999);
-                writeJsonl('/project/test', 'no-time-session');
-                const tryMatch = adapter.tryPidFileMatching.bind(adapter);
-                const { direct, fallback } = tryMatch([
-                    proc
-                ]);
-                // startTime undefined → stale check skipped → direct match
-                expect(direct).toHaveLength(1);
-                expect(fallback).toHaveLength(0);
-            });
-        });
-        describe('getProjectDir', ()=>{
-            const encode = (cwd)=>adapter.getProjectDir(cwd);
-            it('should replace path separators with hyphens', ()=>{
-                const expected = path.join(adapter.projectsDir, '-Users-foo-bar');
-                expect(encode('/Users/foo/bar')).toBe(expected);
-            });
-            it('should encode underscores as hyphens (matches Claude Code CLI)', ()=>{
-                const expected = path.join(adapter.projectsDir, '-Users-foo-my-project');
-                expect(encode('/Users/foo/my_project')).toBe(expected);
-            });
-            it('should encode dots as hyphens', ()=>{
-                const expected = path.join(adapter.projectsDir, '-Users-foo--worktrees-x');
-                expect(encode('/Users/foo/.worktrees/x')).toBe(expected);
-            });
-            it('should collide paths that differ only in non-alphanumeric chars', ()=>{
-                // The encoding is intentionally lossy — callers must
-                // disambiguate via session JSONL contents, not dir name.
-                expect(encode('/a/b_c')).toBe(encode('/a/b-c'));
-                expect(encode('/a/b_c')).toBe(encode('/a/b.c'));
-            });
-            it('should resolve to a real session dir when cwd contains underscores', ()=>{
-                const cwd = '/Users/foo/my_project';
-                const projectsDir = adapter.projectsDir;
-                const expectedDir = path.join(projectsDir, '-Users-foo-my-project');
-                fs.mkdirSync(expectedDir, {
-                    recursive: true
-                });
-                const sessionFile = path.join(expectedDir, 'session-underscore.jsonl');
-                fs.writeFileSync(sessionFile, '');
-                expect(encode(cwd)).toBe(expectedDir);
-                expect(fs.existsSync(path.join(encode(cwd), 'session-underscore.jsonl'))).toBe(true);
-            });
-        });
         describe('readSession', ()=>{
             it('should parse session file with timestamps, cwd, and entry type', ()=>{
                 const readSession = adapter.parser.readSession.bind(adapter.parser);