@lythos/agent-adapter-deepseek-serve 0.16.0 → 0.17.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lythos/agent-adapter-deepseek-serve",
-  "version": "0.16.0",
+  "version": "0.17.0",
   "type": "module",
   "scripts": {
     "test": "bun test src/ --pass-with-no-tests",
@@ -22,6 +22,6 @@
     "serve-mode"
   ],
   "dependencies": {
-    "@lythos/agent-adapter": "^0.16.0"
+    "@lythos/agent-adapter": "^0.17.0"
   }
 }

package/src/deepseek-serve.test.ts

@@ -3,7 +3,7 @@
  * No real serve process needed — pure state machine + injectable fs.
  */
 
-import { describe, test, expect, beforeEach, afterEach } from 'bun:test'
+import { describe, it, expect, beforeEach, afterEach } from 'bun:test'
 import { existsSync, mkdirSync, mkdtempSync, readFileSync, rmSync, writeFileSync } from 'node:fs'
 import { tmpdir } from 'node:os'
 import { join } from 'node:path'
@@ -30,14 +30,14 @@ describe('ServeLock FSM — lock file lifecycle', () => {
   beforeEach(() => { lockDir = tempLockDir() })
   afterEach(() => { try { rmSync(lockDir, { recursive: true, force: true }) } catch {} })
 
-  test('FSM: no lock → null', () => {
+  it('FSM: no lock → null', () => {
     const lockPath = join(lockDir, 'deepseek-serve.json')
     // inline logic
     // Simulate readLock logic
     expect(existsSync(lockPath)).toBe(false)
   })
 
-  test('FSM: write lock → read back', () => {
+  it('FSM: write lock → read back', () => {
     const lock = { pid: 12345, port: 17878, version: '0.8.14', startedAt: new Date().toISOString(), threads: {} }
     writeTestLock(lockDir, lock)
     const content = readFileSync(join(lockDir, 'deepseek-serve.json'), 'utf-8')
@@ -48,7 +48,7 @@ describe('ServeLock FSM — lock file lifecycle', () => {
     expect(parsed.threads).toEqual({})
   })
 
-  test('FSM: corrupt lock → null (not crash)', () => {
+  it('FSM: corrupt lock → null (not crash)', () => {
     writeFileSync(join(lockDir, 'deepseek-serve.json'), 'not valid json {{{')
     // readLock should return null, not throw
     try {
@@ -62,7 +62,7 @@ describe('ServeLock FSM — lock file lifecycle', () => {
     }
   })
 
-  test('FSM: updateThreadMapping adds entry', () => {
+  it('FSM: updateThreadMapping adds entry', () => {
     const lock = { pid: 12345, port: 17878, version: '0.8.14', startedAt: new Date().toISOString(), threads: {} as Record<string, string> }
     writeTestLock(lockDir, lock)
     // Simulate update
@@ -74,7 +74,7 @@ describe('ServeLock FSM — lock file lifecycle', () => {
     expect(Object.keys(parsed.threads).length).toBe(1)
   })
 
-  test('FSM: multiple sessions map to different threads', () => {
+  it('FSM: multiple sessions map to different threads', () => {
     const lock = { pid: 12345, port: 17878, version: '0.8.14', startedAt: new Date().toISOString(), threads: {} as Record<string, string> }
     lock.threads['arena-20260508-001'] = 'thr_aaa'
     lock.threads['arena-20260508-002'] = 'thr_bbb'
@@ -90,14 +90,14 @@ describe('ServeLock FSM — lock file lifecycle', () => {
 // ── PID detection ───────────────────────────────────────────────────────────
 
 describe('isProcessAlive — PID signal', () => {
-  test('current process PID is alive', () => {
+  it('current process PID is alive', () => {
     const alive = (() => {
       try { process.kill(process.pid, 0); return true } catch { return false }
     })()
     expect(alive).toBe(true)
   })
 
-  test('impossible PID is dead', () => {
+  it('impossible PID is dead', () => {
     // PID 99999 is extremely unlikely to exist
     const alive = (() => {
       try { process.kill(99999, 0); return true } catch { return false }
@@ -105,7 +105,7 @@ describe('isProcessAlive — PID signal', () => {
     expect(alive).toBe(false)
   })
 
-  test('PID 0 is alive (self)', () => {
+  it('PID 0 is alive (self)', () => {
     // process.kill(0, 0) signals the whole process group — always succeeds
     const alive = (() => {
       try { process.kill(0, 0); return true } catch { return false }
@@ -117,30 +117,30 @@ describe('isProcessAlive — PID signal', () => {
 // ── Version parsing ─────────────────────────────────────────────────────────
 
 describe('getVersion — parse deepseek --version', () => {
-  test('parses standard version format', () => {
+  it('parses standard version format', () => {
     const out = 'deepseek 0.8.14\n'
     const match = out.match(/(\d+\.\d+\.\d+)/)
     expect(match?.[1]).toBe('0.8.14')
   })
 
-  test('parses version from mixed output', () => {
+  it('parses version from mixed output', () => {
     const out = 'DeepSeek TUI v0.8.14 (abc1234)\nRuntime: Rust 1.80\n'
     const match = out.match(/(\d+\.\d+\.\d+)/)
     expect(match?.[1]).toBe('0.8.14')
   })
 
-  test('returns null for no version', () => {
+  it('returns null for no version', () => {
     const out = 'command not found\n'
     const match = out.match(/(\d+\.\d+\.\d+)/)
     expect(match).toBe(null)
   })
 
-  test('0.8.x passes version check', () => {
+  it('0.8.x passes version check', () => {
     const version = '0.8.14'
     expect(version.startsWith('0.8.')).toBe(true)
   })
 
-  test('0.9.x still works (with warning)', () => {
+  it('0.9.x still works (with warning)', () => {
     const version = '0.9.0'
     expect(version.startsWith('0.8.')).toBe(false)
     // Should warn but continue
@@ -150,7 +150,7 @@ describe('getVersion — parse deepseek --version', () => {
 // ── Session ID format ───────────────────────────────────────────────────────
 
 describe('nextSessionId — format', () => {
-  test('starts with arena- prefix', () => {
+  it('starts with arena- prefix', () => {
     let counter = 0
     const ts = new Date().toISOString().replace(/[-:T]/g, '').slice(0, 15)
     counter++
@@ -158,7 +158,7 @@ describe('nextSessionId — format', () => {
     expect(id.startsWith('arena-')).toBe(true)
   })
 
-  test('counter increments', () => {
+  it('counter increments', () => {
     let counter = 0
     const ts = new Date().toISOString().replace(/[-:T]/g, '').slice(0, 15)
     const ids: string[] = []
@@ -176,12 +176,12 @@ describe('nextSessionId — format', () => {
 // ── Adapter registration ────────────────────────────────────────────────────
 
 describe('adapter registration', () => {
-  test('adapter is registered under name "deepseek"', async () => {
+  it('adapter is registered under name "deepseek"', async () => {
     const { deepseekServeAdapter } = await import('./deepseek-serve')
     expect(deepseekServeAdapter.name).toBe('deepseek')
   })
 
-  test('adapter has spawn method', async () => {
+  it('adapter has spawn method', async () => {
     const { deepseekServeAdapter } = await import('./deepseek-serve')
     expect(typeof deepseekServeAdapter.spawn).toBe('function')
   })
@@ -190,7 +190,7 @@ describe('adapter registration', () => {
 // ── Actor FSM: state transitions (conceptual) ───────────────────────────────
 
 describe('Actor FSM — state transitions', () => {
-  test('state: cold start → serve running', () => {
+  it('state: cold start → serve running', () => {
     // Given: no lock file, serve not running
     // When: ensureServeRunning()
     // Then: findFreePort → spawn serve → health check → writeLock → return port
@@ -200,7 +200,7 @@ describe('Actor FSM — state transitions', () => {
     expect(states).toContain('error')
   })
 
-  test('state: warm reuse → skip start', () => {
+  it('state: warm reuse → skip start', () => {
     // Given: lock exists, PID alive, health check passes
     // When: ensureServeRunning()
     // Then: cachedPort → health check → return (no new process)
@@ -208,7 +208,7 @@ describe('Actor FSM — state transitions', () => {
     expect(expectedPath.length).toBe(3)
   })
 
-  test('state: dead PID → restart', () => {
+  it('state: dead PID → restart', () => {
     // Given: lock exists, PID dead
     // When: ensureServeRunning()
     // Then: isProcessAlive → false → findFreePort → spawn → health → writeLock
@@ -216,7 +216,7 @@ describe('Actor FSM — state transitions', () => {
     expect(expectedPath.length).toBe(7)
   })
 
-  test('state: port occupied → increment', () => {
+  it('state: port occupied → increment', () => {
     // Given: base port 17878 occupied
     // When: findFreePort(17878)
     // Then: try 17879, 17880... until free
@@ -232,26 +232,26 @@ describe('Thread API — request paths', () => {
   const PORT = 17878
   const THREAD_ID = 'thr_test123'
 
-  test('POST /v1/threads — create thread', () => {
+  it('POST /v1/threads — create thread', () => {
     const path = `/v1/threads`
     const url = `http://127.0.0.1:${PORT}${path}`
     expect(path).toBe('/v1/threads')
     expect(url).toContain(':17878')
   })
 
-  test('POST /v1/threads/{id}/turns — send turn', () => {
+  it('POST /v1/threads/{id}/turns — send turn', () => {
     const path = `/v1/threads/${THREAD_ID}/turns`
     expect(path).toContain(THREAD_ID)
     expect(path).toContain('/turns')
   })
 
-  test('GET /v1/threads/{id}/events — SSE stream', () => {
+  it('GET /v1/threads/{id}/events — SSE stream', () => {
     const path = `/v1/threads/${THREAD_ID}/events`
     const url = `http://127.0.0.1:${PORT}${path}?since_seq=0`
     expect(url).toContain('since_seq=0')
   })
 
-  test('GET /health — health check', () => {
+  it('GET /health — health check', () => {
     const url = `http://127.0.0.1:${PORT}/health`
     expect(url.endsWith('/health')).toBe(true)
   })
@@ -260,7 +260,7 @@ describe('Thread API — request paths', () => {
 // ── Lock file schema ────────────────────────────────────────────────────────
 
 describe('ServeLock schema', () => {
-  test('valid lock matches expected shape', () => {
+  it('valid lock matches expected shape', () => {
     const lock = {
       pid: 12345,
       port: 17878,
@@ -275,7 +275,7 @@ describe('ServeLock schema', () => {
     expect(typeof lock.threads).toBe('object')
   })
 
-  test('threads must be string→string map', () => {
+  it('threads must be string→string map', () => {
     const threads: Record<string, string> = {}
     threads['session-a'] = 'thr_111'
     threads['session-b'] = 'thr_222'