also looking at this
feat(engine): answer texts with Codex as well as Claude Code
#7
18 files
+1080
-142
Review threads live on the whole diff, not on one commit, so none are shown here - a thread's line means something in the branch's final form, and painting it into an intermediate step would put it on code it is not about.
| @@ -1,7 +1,7 @@ | ||
| 1 | import type { EngineId } from '../Uplink/engine' | |
| 1 | 2 | import { copyFileSync, existsSync, watch } from 'node:fs' |
| 2 | 3 | import process from 'node:process' |
| 3 | 4 | import { defineCommand } from '@stacksjs/cli' |
| 4 | import type { EngineId } from '../Uplink/engine' | |
| 5 | 5 | import { loadConfig } from '../Uplink/config' |
| 6 | 6 | import { checksPass, runChecks, writeHeartbeat } from '../Uplink/doctor' |
| 7 | 7 | import { ENGINE_IDS, isEngineId } from '../Uplink/engine' |
| @@ -2,8 +2,7 @@ import type { UplinkConfig } from './config' | ||
| 2 | 2 | import type { Engine, EngineId } from './engine' |
| 3 | 3 | import { CodexEngine } from './codex-engine' |
| 4 | 4 | import { systemPrompt } from './config' |
| 5 | import { ClaudeEngine } from './engine' | |
| 6 | import { ENGINE_IDS } from './engine' | |
| 5 | import { ClaudeEngine, ENGINE_IDS } from './engine' | |
| 7 | 6 | |
| 8 | 7 | /** |
| 9 | 8 | * Builds engines from config, so the watcher, the app and the doctor all select |