ReviewOS

also looking at this

stacks/uplink

feat(engine): answer texts with Codex as well as Claude Code

#7
Merged glennmichael123 wants to merge feat/codex-engine into main
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.

app/Commands/Uplink.tsmodifiedformatting only+1-1
Changes to app/Commands/Uplink.ts
@@ -1,7 +1,7 @@
1import type { EngineId } from '../Uplink/engine'
12import { copyFileSync, existsSync, watch } from 'node:fs'
23import process from 'node:process'
34import { defineCommand } from '@stacksjs/cli'
4import type { EngineId } from '../Uplink/engine'
55import { loadConfig } from '../Uplink/config'
66import { checksPass, runChecks, writeHeartbeat } from '../Uplink/doctor'
77import { ENGINE_IDS, isEngineId } from '../Uplink/engine'
app/Uplink/engines.tsmodified+1-2
Changes to app/Uplink/engines.ts
@@ -2,8 +2,7 @@ import type { UplinkConfig } from './config'
22import type { Engine, EngineId } from './engine'
33import { CodexEngine } from './codex-engine'
44import { systemPrompt } from './config'
5import { ClaudeEngine } from './engine'
6import { ENGINE_IDS } from './engine'
5import { ClaudeEngine, ENGINE_IDS } from './engine'
76
87/**
98 * Builds engines from config, so the watcher, the app and the doctor all select