mirror of
https://github.com/paperclipai/paperclip.git
synced 2026-11-10 12:37:55 +01:00
## Thinking Path > - Paperclip is the open source app people use to manage AI agents for work > - The Codex adapter and the native runner run Codex in local hosts and in managed sandboxes, and the model catalog lists the models an operator can select > - The Codex backend accepts a new model with ChatGPT sign-in only from a recent enough Codex CLI. An older CLI fails every turn with `The '<model>' model is not supported when using Codex with a ChatGPT account.` > - After #14942 added `gpt-6.1-sol`, an operator selected it for an agent in a managed Daytona sandbox. The sandbox image still shipped Codex 0.156.0. The connection test and runs failed with that sentence, which reads like an account problem > - Paperclip only checked the Codex compatibility window (`>=0.149.0 <0.161.0`), so 0.156.0 passed and the failure surfaced from the backend without a cause > - This pull request records the verified Codex CLI floor for each gated model, compares the installed `codex --version` with that floor in the environment Test and in the remote runner, and names the backend rejection when it still happens > - The benefit is a precise, actionable message ("gpt-6.1-sol requires Codex CLI 0.159.0 or newer; detected 0.156.0; promote a sandbox image with Codex 0.160.0") instead of an opaque 400, and no doomed hello probe ## Linked Issues or Issue Description Refs #14942 **Bug: selecting `gpt-6.1-sol` in a managed sandbox fails with a ChatGPT account error.** **Steps to reproduce** 1. Promote a sandbox image that ships Codex CLI 0.156.0. 2. Create a Codex agent with a ChatGPT sign-in connection, select `gpt-6.1-sol`, and run the environment Test or a task in that sandbox. **Expected behavior** The Test or the run tells the operator that the Codex CLI in the sandbox is older than the model needs. **Actual behavior** The Test reports `codex_hello_probe_failed` with `{"type":"error","status":400,"error":{"type":"invalid_request_error","message":"The 'gpt-6.1-sol' model is not supported when using Codex with a ChatGPT account."}}`. A run fails with "The selected model is not supported by the current ChatGPT connection." **Evidence** - Codex 0.156.1 and older are rejected for `gpt-6.1-sol` with ChatGPT sign-in: https://github.com/openai/codex/issues/49396 - Codex 0.159.0 and 0.159.2 are accepted: https://github.com/openai/codex/issues/49464 and https://github.com/decolua/9router/issues/4471 (same error, fixed by raising the client version identity from 0.155.0 to 0.159.0) - Codex 0.157.0 release notes: "Add GPT-6 Sol and Luna to the model catalog": https://github.com/openai/codex/releases/tag/rust-v0.157.0 - GPT-6.1 Sol is included for Plus, Pro, Business, Enterprise, and Edu with ChatGPT sign-in: https://learn.chatgpt.com/docs/models ## What Changed - `packages/adapters/codex-local/src/index.ts`: add `minimumCodexCliVersionForModel` (`gpt-6.1-sol` → 0.159.0, `gpt-6-sol` and `gpt-6-luna` → 0.157.0, with the sources above), `parseCodexCliVersionOutput`, `codexCliVersionAtLeast`, and `CODEX_CHATGPT_MODEL_REJECTION_RE`. Models without a verified floor return `null`, so nothing probes them. The agent configuration doc describes the floors. - `packages/adapters/codex-local/src/server/cli-version.ts` (new): run `codex --version` where the run would execute it (local, SSH, or sandbox), compare it with the model floor, and build the `codex_cli_version_compatible` / `codex_cli_version_incompatible` checks. Map the backend rejection sentence to `codex_hello_probe_model_rejected` with the detected CLI version and a hint that separates a stale CLI from a plan that does not include the model. - `packages/adapters/codex-local/src/server/test.ts` (CLI lane Test): run the version check before the hello probe when the model has a floor. Skip the hello probe when the CLI is too old (`codex_hello_probe_skipped_cli_version`). When the probe still fails with the backend sentence, report `codex_hello_probe_model_rejected` instead of the generic `codex_hello_probe_failed`. The new codes do not match the auth-failure patterns, so a managed connection is not invalidated. - `packages/adapters/codex-local/src/server/acp.ts` (ACP lane Test): for remote targets, run the same version check against the shared `codex` the ACP server spawns. - `server/src/services/native-runtime/native-session-executor.ts`: after the compatibility-window check, compare the remote Codex with the configured model's floor and fail with `runner_remote_provider_artifact_incompatible: <model> requires Codex <floor> or newer with ChatGPT sign-in, received <version> from the sandbox image; promote a sandbox image with Codex 0.160.0 or configure PAPERCLIP_RUNNER_REMOTE_CODEX_NPM_SPEC=...`. When a preinstalled Codex fails this check and an npm spec is configured, the existing fallback installs the pinned release. - Tests: adapter metadata, CLI-lane Test (too old, compatible, no floor, backend rejection), ACP-lane Test (too old, compatible, no floor), and the remote runner floor (nine version/model cases). - `doc/adapter-model-audit-2026-10-02.md`: record the floors and the reason. No catalog entry, default model, saved agent configuration, pin, lockfile, or image definition changes. ## Verification Local (Node 25.9, pnpm 9.15.4 via corepack): ```sh pnpm --filter @paperclipai/adapter-codex-local typecheck pnpm --filter @paperclipai/adapter-codex-local exec vitest run # 81 passed pnpm --filter @paperclipai/server exec vitest run \ src/services/native-runtime/native-session-executor.test.ts \ src/services/native-runtime/codex-runtime-compatibility.test.ts -t Codex # 84 passed ``` Also run locally: the full `native-session-executor.test.ts` file (533 passed) and the full Codex adapter suite (81 passed). Follow-up commit `9fd4d3b25` (Greptile P2: the ACP-lane version probe ignored the agent's configured env): the probe now receives the adapter's string-valued `env` entries, the same ones `buildCodexAcpConfig` hands to remote ACP runs, so a `PATH` override selects the same `codex` for the Test as for the run. New test covers a `PATH` + `CODEX_HOME` override and a dropped non-string entry. Re-run on that head: `pnpm --filter @paperclipai/adapter-codex-local typecheck` clean; full Codex adapter suite 503 passed (31 files). Not run here: `pnpm -r typecheck` for `server` (the direct `tsc --noEmit` was killed by the sandbox memory cap; the adapter package typecheck passes and CI covers the server), `pnpm build`, browser suites, and a live sandbox probe. No UI files changed, so token gates do not apply. Manual check for a reviewer: set an agent's Codex model to `gpt-6.1-sol`, point its environment at a sandbox whose `codex --version` prints `codex-cli 0.156.0`, and run the environment Test. The result contains `codex_cli_version_incompatible` with `Detected Codex CLI 0.156.0.` and no hello probe check. With Codex 0.160.0 the Test contains `codex_cli_version_compatible` and runs the hello probe. ## Risks - A floor is enforced for all authentication modes, because the runner does not know the credential method at verification time. An OpenAI API key on Codex 0.157.0 or 0.158.0 with `gpt-6.1-sol` is now rejected before launch. Paperclip pins Codex 0.160.0 everywhere, so this only affects installs that lag the pin, and the message names the fix. - The 0.159.0 floor for `gpt-6.1-sol` is the oldest stable release verified to work; 0.157.0 and 0.158.0 were not verified either way. If OpenAI accepts an older client, the floor can be lowered in one table. - The `codex --version` probe adds one short process run to the Test only for models with a floor. - Rollback: revert this pull request. No data or configuration migrates. ## Model Used Claude Fable 5.1 (`claude-fable-5-1`), Anthropic, with extended thinking, tool use, and web research, operating as a Paperclip agent through Claude Code. ## Checklist - [x] I have included a thinking path that traces from project context to this change - [x] I have specified the model used (with version and capability details) - [x] I have checked ROADMAP.md and confirmed this PR does not duplicate planned core work - [x] I have searched GitHub for duplicate or related PRs and linked them above - [x] I have either (a) linked existing issues with `Fixes: #` / `Closes #` / `Refs #` OR (b) described the issue in-PR following the relevant issue template - [x] I have not referenced internal/instance-local Paperclip issues or links (only public GitHub `#NNN` / `github.com/paperclipai/paperclip` URLs) - [x] My branch name describes the change (e.g. `docs/...`, `fix/...`) and contains no internal Paperclip ticket id or instance-derived details - [x] I have run tests locally and they pass - [x] I have added or updated tests where applicable - [x] I have updated relevant documentation to reflect my changes - [x] I have considered and documented any risks above - [ ] All Paperclip CI gates are green - [ ] Greptile is 5/5 with no open P2s, recommendations, or follow-ups - [x] I will address all Greptile and reviewer comments before requesting merge 🤖 Generated with [Claude Code](https://claude.com/claude-code) --------- Co-authored-by: Bender (Fable) <noreply@paperclip.ing> Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
582 lines
23 KiB
TypeScript
582 lines
23 KiB
TypeScript
import type {
|
||
AdapterEnvironmentCheck,
|
||
AdapterEnvironmentTestContext,
|
||
AdapterEnvironmentTestResult,
|
||
} from "@paperclipai/adapter-utils";
|
||
import {
|
||
asString,
|
||
parseObject,
|
||
ensurePathInEnv,
|
||
} from "@paperclipai/adapter-utils/server-utils";
|
||
import {
|
||
ensureAdapterExecutionTargetCommandResolvable,
|
||
ensureAdapterExecutionTargetDirectory,
|
||
maybeRunSandboxInstallCommand,
|
||
runAdapterExecutionTargetProcess,
|
||
describeAdapterExecutionTarget,
|
||
resolveAdapterExecutionTargetCwd,
|
||
prepareAdapterExecutionTargetRuntime,
|
||
} from "@paperclipai/adapter-utils/execution-target";
|
||
import fs from "node:fs/promises";
|
||
import path from "node:path";
|
||
import os from "node:os";
|
||
import { parseCodexJsonl } from "./parse.js";
|
||
import { SANDBOX_INSTALL_COMMAND } from "../index.js";
|
||
import { codexHomeDir, readCodexAuthInfo } from "./quota.js";
|
||
import { buildCodexExecArgs } from "./codex-args.js";
|
||
import {
|
||
isManagedCodexHomePath,
|
||
prepareManagedCodexHome,
|
||
resolveSharedCodexHomeDir,
|
||
seedManagedCodexHome,
|
||
} from "./codex-home.js";
|
||
import {
|
||
isCodexAuthCacheEnabled,
|
||
resolveCodexAuthCacheEntryPath,
|
||
selectVendCredential,
|
||
} from "./codex-auth-cache.js";
|
||
import { resolveCodexExecutionEngineForRun, testCodexAcpEnvironment } from "./acp.js";
|
||
import { ADAPTER_AUTH_MISSING_CHECK_CODE } from "./auth-check.js";
|
||
import { checkCodexCliVersionForModel, codexHelloProbeModelRejectionCheck } from "./cli-version.js";
|
||
|
||
function summarizeStatus(checks: AdapterEnvironmentCheck[]): AdapterEnvironmentTestResult["status"] {
|
||
if (checks.some((check) => check.level === "error")) return "fail";
|
||
if (checks.some((check) => check.level === "warn")) return "warn";
|
||
return "pass";
|
||
}
|
||
|
||
function isNonEmpty(value: unknown): value is string {
|
||
return typeof value === "string" && value.trim().length > 0;
|
||
}
|
||
|
||
function firstNonEmptyLine(text: string): string {
|
||
return (
|
||
text
|
||
.split(/\r?\n/)
|
||
.map((line) => line.trim())
|
||
.find(Boolean) ?? ""
|
||
);
|
||
}
|
||
|
||
function commandLooksLike(command: string, expected: string): boolean {
|
||
const base = path.basename(command).toLowerCase();
|
||
return base === expected || base === `${expected}.cmd` || base === `${expected}.exe`;
|
||
}
|
||
|
||
function summarizeProbeDetail(stdout: string, stderr: string, parsedError: string | null): string | null {
|
||
const raw = parsedError?.trim() || firstNonEmptyLine(stderr) || firstNonEmptyLine(stdout);
|
||
if (!raw) return null;
|
||
const clean = raw.replace(/\s+/g, " ").trim();
|
||
const max = 240;
|
||
return clean.length > max ? `${clean.slice(0, max - 1)}…` : clean;
|
||
}
|
||
|
||
const CODEX_AUTH_REQUIRED_RE =
|
||
/(?:not\s+logged\s+in|login\s+required|authentication\s+required|unauthorized|invalid(?:\s+or\s+missing)?\s+api(?:[_\s-]?key)?|openai[_\s-]?api[_\s-]?key|api[_\s-]?key.*required|please\s+run\s+`?codex\s+login`?)/i;
|
||
|
||
const PROBE_CLEANUP_WARNING = "[paperclip] Codex probe cleanup incomplete";
|
||
|
||
async function prepareCodexHelloProbe(input: {
|
||
runId: string;
|
||
companyId: string;
|
||
target: AdapterEnvironmentTestContext["executionTarget"] | null;
|
||
targetIsRemote: boolean;
|
||
cwd: string;
|
||
command: string;
|
||
args: string[];
|
||
env: Record<string, string>;
|
||
probeApiKey: string | null;
|
||
managedAiConnection?: boolean;
|
||
}): Promise<{
|
||
command: string;
|
||
args: string[];
|
||
env: Record<string, string>;
|
||
cleanup: () => Promise<void>;
|
||
}> {
|
||
let preparedRuntime: Awaited<ReturnType<typeof prepareAdapterExecutionTargetRuntime>> | null = null;
|
||
let preparedRuntimeWorkspaceLocalDir: string | null = null;
|
||
let probeHomeLocalDir: string | null = null;
|
||
|
||
const cleanup = async () => {
|
||
await preparedRuntime?.restoreWorkspace().catch(() => {});
|
||
if (preparedRuntimeWorkspaceLocalDir) {
|
||
await fs.rm(preparedRuntimeWorkspaceLocalDir, { recursive: true, force: true }).catch(() => {});
|
||
}
|
||
if (probeHomeLocalDir) {
|
||
await fs.rm(probeHomeLocalDir, { recursive: true, force: true }).catch(() => {});
|
||
}
|
||
};
|
||
|
||
if (input.targetIsRemote && !input.probeApiKey) {
|
||
// Prepare the exact home a real run would use, mirroring execute.ts: vend
|
||
// the shared credential's freshest same-identity cached copy, then seed the
|
||
// effective home — the company default when no CODEX_HOME is configured, a
|
||
// Paperclip-managed override (the per-agent home) seeded in place — and
|
||
// stage that home's credentials. A genuine external override manages its
|
||
// own auth: its bytes are staged as-is and it is never seeded or mutated.
|
||
// Without this mirror the probe exercises a different credential than the
|
||
// run, and the Test and real runs can disagree in both directions.
|
||
const configuredCodexHome = isNonEmpty(input.env.CODEX_HOME)
|
||
? path.resolve(input.env.CODEX_HOME.trim())
|
||
: null;
|
||
const configuredHomeIsManaged =
|
||
configuredCodexHome != null &&
|
||
isManagedCodexHomePath(process.env, input.companyId, configuredCodexHome);
|
||
if (!input.managedAiConnection && isCodexAuthCacheEnabled(process.env)) {
|
||
// Identity-anchored cache vend, exactly as execute runs it before the
|
||
// seeding below. Best-effort: a vend failure never blocks the probe, and
|
||
// the probe then stages the shared credential as-is.
|
||
const sharedHomeAuthPath = path.join(resolveSharedCodexHomeDir(process.env), "auth.json");
|
||
await selectVendCredential(
|
||
sharedHomeAuthPath,
|
||
(accountId) => resolveCodexAuthCacheEntryPath(process.env, accountId, input.companyId),
|
||
async () => {},
|
||
).catch(() => undefined);
|
||
}
|
||
let effectiveHome: string;
|
||
if (configuredCodexHome == null) {
|
||
effectiveHome = await prepareManagedCodexHome(process.env, async () => {}, input.companyId, {
|
||
apiKey: null,
|
||
});
|
||
} else {
|
||
if (configuredHomeIsManaged) {
|
||
await seedManagedCodexHome(configuredCodexHome, process.env, async () => {}, {
|
||
apiKey: null,
|
||
});
|
||
}
|
||
effectiveHome = configuredCodexHome;
|
||
}
|
||
|
||
// Upload only the credential/config files the login probe needs, not the
|
||
// entire effective CODEX_HOME. A real managed home accumulates hundreds of
|
||
// MB of session/state history (`sessions/`, `state_*.sqlite`, …); tarring
|
||
// and streaming all of it into the sandbox made the environment Test probe
|
||
// take many minutes and look like it hung. The hello probe only needs auth.
|
||
probeHomeLocalDir = await fs.mkdtemp(
|
||
path.join(os.tmpdir(), `paperclip-codex-probe-home-${input.runId}-`),
|
||
);
|
||
let seededAuth = false;
|
||
let seededConfig = false;
|
||
for (const file of ["auth.json", "config.toml"]) {
|
||
// `fs.readFile` follows the home's `auth.json` symlink into the host's
|
||
// `~/.codex`, so we copy the resolved bytes as a plain file.
|
||
const contents = await fs.readFile(path.join(effectiveHome, file)).catch(() => null);
|
||
if (contents) {
|
||
await fs.writeFile(path.join(probeHomeLocalDir, file), contents);
|
||
if (file === "auth.json") seededAuth = true;
|
||
if (file === "config.toml") seededConfig = true;
|
||
}
|
||
}
|
||
|
||
// When the host has no Codex credentials to seed, don't override CODEX_HOME.
|
||
// Pointing Codex at an empty uploaded home would mask any login already
|
||
// baked into the sandbox (e.g. a captured custom-image snapshot); leaving
|
||
// CODEX_HOME unset lets the probe exercise that in-sandbox login instead.
|
||
if (!seededAuth && !(input.managedAiConnection && seededConfig)) {
|
||
return {
|
||
command: input.command,
|
||
args: input.args,
|
||
env: { ...input.env },
|
||
cleanup,
|
||
};
|
||
}
|
||
|
||
preparedRuntimeWorkspaceLocalDir = await fs.mkdtemp(
|
||
path.join(os.tmpdir(), `paperclip-codex-envtest-${input.runId}-`),
|
||
);
|
||
preparedRuntime = await prepareAdapterExecutionTargetRuntime({
|
||
runId: input.runId,
|
||
target: input.target,
|
||
adapterKey: "codex",
|
||
workspaceLocalDir: preparedRuntimeWorkspaceLocalDir,
|
||
// Pass `input.cwd` as the base (not a pre-built per-run subdir).
|
||
// `prepareRemoteManagedRuntime` itself appends
|
||
// `.paperclip-runtime/runs/<runId>/workspace` to whatever it gets, so
|
||
// pre-building a per-run path here would double-nest the run ID.
|
||
workspaceRemoteDir: input.cwd,
|
||
installCommand: SANDBOX_INSTALL_COMMAND,
|
||
detectCommand: input.command,
|
||
assets: [
|
||
{
|
||
key: "home",
|
||
localDir: probeHomeLocalDir,
|
||
followSymlinks: true,
|
||
},
|
||
],
|
||
});
|
||
|
||
return {
|
||
command: input.command,
|
||
args: input.args,
|
||
env: preparedRuntime.assetDirs.home
|
||
? { ...input.env, CODEX_HOME: preparedRuntime.assetDirs.home }
|
||
: { ...input.env },
|
||
cleanup,
|
||
};
|
||
}
|
||
|
||
if (input.probeApiKey) {
|
||
const probeHome = input.targetIsRemote
|
||
? path.posix.join(input.cwd, ".paperclip-runtime", "codex", `probe-home-${input.runId}`)
|
||
: path.join(os.tmpdir(), `paperclip-codex-probe-${input.runId}`);
|
||
// The local finally path retries cleanup independently of the model result.
|
||
if (!input.targetIsRemote) probeHomeLocalDir = probeHome;
|
||
return {
|
||
command: "sh",
|
||
args: [
|
||
"-c",
|
||
`set -e; umask 077; mkdir -p "$CODEX_HOME"; printf "%s" "$_PAPERCLIP_CODEX_AUTH_JSON" > "$CODEX_HOME/auth.json"; unset _PAPERCLIP_CODEX_AUTH_JSON; cleanup() { result=$?; trap - EXIT; rm -f "$CODEX_HOME/auth.json" || true; rm -rf "$CODEX_HOME" || printf '%s\\n' '${PROBE_CLEANUP_WARNING}' >&2; exit "$result"; }; trap cleanup EXIT; trap 'exit 130' INT; trap 'exit 143' TERM; "$0" "$@"`,
|
||
input.command,
|
||
...input.args,
|
||
],
|
||
env: {
|
||
...input.env,
|
||
CODEX_HOME: probeHome,
|
||
_PAPERCLIP_CODEX_AUTH_JSON: JSON.stringify({ OPENAI_API_KEY: input.probeApiKey }),
|
||
},
|
||
cleanup,
|
||
};
|
||
}
|
||
|
||
return {
|
||
command: input.command,
|
||
args: input.args,
|
||
env: { ...input.env },
|
||
cleanup,
|
||
};
|
||
}
|
||
|
||
export async function testEnvironment(
|
||
ctx: AdapterEnvironmentTestContext,
|
||
): Promise<AdapterEnvironmentTestResult> {
|
||
const engineSelection = await resolveCodexExecutionEngineForRun({
|
||
config: parseObject(ctx.config),
|
||
executionTarget: ctx.executionTarget,
|
||
});
|
||
if (engineSelection.unavailableReason) {
|
||
return {
|
||
adapterType: "codex_local",
|
||
status: "fail",
|
||
checks: [{
|
||
code: "adapter_engine_unavailable",
|
||
level: "error",
|
||
message: engineSelection.unavailableReason,
|
||
hint: ctx.executionTarget?.kind === "remote"
|
||
? "In the agent’s runtime settings, select the CLI engine, or use a sandbox image with the Codex ACP server installed."
|
||
: undefined,
|
||
}],
|
||
testedAt: new Date().toISOString(),
|
||
};
|
||
}
|
||
if (engineSelection.engine === "acp") {
|
||
return testCodexAcpEnvironment(ctx);
|
||
}
|
||
|
||
const checks: AdapterEnvironmentCheck[] = [];
|
||
const config = parseObject(ctx.config);
|
||
const command = asString(config.command, "codex");
|
||
const target = ctx.executionTarget ?? null;
|
||
const targetIsRemote = target?.kind === "remote";
|
||
const targetIsSandbox = target?.kind === "remote" && target.transport === "sandbox";
|
||
const cwd = resolveAdapterExecutionTargetCwd(target, asString(config.cwd, ""), process.cwd());
|
||
const targetLabel = targetIsRemote
|
||
? ctx.environmentName ?? describeAdapterExecutionTarget(target)
|
||
: null;
|
||
const runId = `codex-envtest-${Date.now()}-${Math.random().toString(16).slice(2)}`;
|
||
|
||
if (targetLabel) {
|
||
checks.push({
|
||
code: "codex_environment_target",
|
||
level: "info",
|
||
message: `Probing inside environment: ${targetLabel}`,
|
||
});
|
||
}
|
||
|
||
try {
|
||
await ensureAdapterExecutionTargetDirectory(runId, target, cwd, {
|
||
cwd,
|
||
env: {},
|
||
createIfMissing: true,
|
||
});
|
||
checks.push({
|
||
code: "codex_cwd_valid",
|
||
level: "info",
|
||
message: `Working directory is valid: ${cwd}`,
|
||
});
|
||
} catch (err) {
|
||
checks.push({
|
||
code: "codex_cwd_invalid",
|
||
level: "error",
|
||
message: err instanceof Error ? err.message : "Invalid working directory",
|
||
detail: cwd,
|
||
});
|
||
}
|
||
|
||
const envConfig = parseObject(config.env);
|
||
const env: Record<string, string> = {};
|
||
for (const [key, value] of Object.entries(envConfig)) {
|
||
if (typeof value === "string") env[key] = value;
|
||
}
|
||
const runtimeEnv = ensurePathInEnv({ ...process.env, ...env });
|
||
const installCheck = await maybeRunSandboxInstallCommand({
|
||
runId,
|
||
target,
|
||
adapterKey: "codex",
|
||
installCommand: SANDBOX_INSTALL_COMMAND,
|
||
detectCommand: command,
|
||
env,
|
||
});
|
||
if (installCheck) checks.push(installCheck);
|
||
try {
|
||
await ensureAdapterExecutionTargetCommandResolvable(command, target, cwd, runtimeEnv);
|
||
checks.push({
|
||
code: "codex_command_resolvable",
|
||
level: "info",
|
||
message: `Command is executable: ${command}`,
|
||
});
|
||
} catch (err) {
|
||
checks.push({
|
||
code: "codex_command_unresolvable",
|
||
level: "error",
|
||
message: err instanceof Error ? err.message : "Command is not executable",
|
||
detail: command,
|
||
});
|
||
}
|
||
|
||
const configOpenAiKey = env.OPENAI_API_KEY;
|
||
const hostOpenAiKey = targetIsRemote || Object.hasOwn(env, "OPENAI_API_KEY")
|
||
? undefined : process.env.OPENAI_API_KEY;
|
||
if (config.managedAiRouting) {
|
||
checks.push({
|
||
code: "codex_managed_provider_configured",
|
||
level: "info",
|
||
message: "Testing the selected connection’s provider and model.",
|
||
});
|
||
} else if (isNonEmpty(configOpenAiKey) || isNonEmpty(hostOpenAiKey)) {
|
||
const source = isNonEmpty(configOpenAiKey) ? "adapter config env" : "server environment";
|
||
checks.push({
|
||
code: "codex_openai_api_key_present",
|
||
level: "info",
|
||
message: "OPENAI_API_KEY is set for Codex authentication.",
|
||
detail: `Detected in ${source}.`,
|
||
});
|
||
} else if (!targetIsRemote) {
|
||
// Local-only auth file check. On remote targets, the probe will surface
|
||
// any missing-auth errors directly from the remote `codex` invocation.
|
||
const codexHome = isNonEmpty(env.CODEX_HOME) ? env.CODEX_HOME : undefined;
|
||
const codexAuth = await readCodexAuthInfo(codexHome).catch(() => null);
|
||
if (codexAuth) {
|
||
checks.push({
|
||
code: "codex_native_auth_present",
|
||
level: "info",
|
||
message: "Codex is authenticated via its own auth configuration.",
|
||
detail: codexAuth.email ? `Logged in as ${codexAuth.email}.` : `Credentials found in ${path.join(codexHome ?? codexHomeDir(), "auth.json")}.`,
|
||
});
|
||
} else {
|
||
checks.push({
|
||
code: "codex_openai_api_key_missing",
|
||
level: "warn",
|
||
message: "OPENAI_API_KEY is not set. Codex runs may fail until authentication is configured.",
|
||
hint: "Set OPENAI_API_KEY in adapter env, shell environment, or run `codex auth` to log in.",
|
||
});
|
||
}
|
||
}
|
||
|
||
const canRunProbe =
|
||
checks.every((check) => check.code !== "codex_cwd_invalid" && check.code !== "codex_command_unresolvable");
|
||
// Models with a verified CLI floor are compared against the installed Codex
|
||
// before the hello probe. A probe on an older CLI only produces the backend's
|
||
// "not supported when using Codex with a ChatGPT account" rejection, which
|
||
// hides the real gap: the CLI (often a stale sandbox image), not the account.
|
||
let detectedCliVersion: string | null = null;
|
||
let configuredModelIsCompatible = true;
|
||
if (canRunProbe && commandLooksLike(command, "codex")) {
|
||
const versionCheck = await checkCodexCliVersionForModel({
|
||
runId,
|
||
model: asString(config.model, ""),
|
||
command,
|
||
target,
|
||
cwd,
|
||
env,
|
||
});
|
||
if (versionCheck) {
|
||
checks.push(versionCheck.check);
|
||
configuredModelIsCompatible = versionCheck.compatible;
|
||
detectedCliVersion = versionCheck.detectedVersion;
|
||
}
|
||
}
|
||
if (canRunProbe && !configuredModelIsCompatible) {
|
||
checks.push({
|
||
code: "codex_hello_probe_skipped_cli_version",
|
||
level: "info",
|
||
message: "Skipped hello probe because the installed Codex CLI cannot use the configured model.",
|
||
});
|
||
} else if (canRunProbe) {
|
||
if (!commandLooksLike(command, "codex")) {
|
||
checks.push({
|
||
code: "codex_hello_probe_skipped_custom_command",
|
||
level: "info",
|
||
message: "Skipped hello probe because command is not `codex`.",
|
||
detail: command,
|
||
hint: "Use the `codex` CLI command to run the automatic login and installation probe.",
|
||
});
|
||
} else {
|
||
const execArgs = buildCodexExecArgs(
|
||
{ ...config, fastMode: false },
|
||
{ skipGitRepoCheck: targetIsSandbox },
|
||
);
|
||
// A connection test needs one small response, not plugin catalog sync,
|
||
// repository instructions, or a durable session. Keep provider/model
|
||
// configuration intact while removing unrelated startup work.
|
||
const args = [...execArgs.args];
|
||
args.splice(args.length - 1, 0,
|
||
"-c", "features.plugins=false",
|
||
"-c", "features.remote_plugin=false",
|
||
"-c", "project_doc_max_bytes=0",
|
||
...(args.includes("--ephemeral") ? [] : ["--ephemeral"]),
|
||
);
|
||
if (execArgs.fastModeIgnoredReason) {
|
||
checks.push({
|
||
code: "codex_fast_mode_unsupported_model",
|
||
level: "warn",
|
||
message: execArgs.fastModeIgnoredReason,
|
||
hint: "Switch the agent model to GPT-5.4 or enter a manual model ID to enable Codex Fast mode.",
|
||
});
|
||
}
|
||
if (targetIsSandbox) {
|
||
checks.push({
|
||
code: "codex_git_repo_check_skipped",
|
||
level: "info",
|
||
message: "Added --skip-git-repo-check for sandbox hello probes.",
|
||
hint: "Codex requires an explicit trust bypass in headless remote sandbox workspaces.",
|
||
});
|
||
}
|
||
|
||
// Codex CLI (>= 0.122) ignores the OPENAI_API_KEY env var and only reads
|
||
// credentials from $CODEX_HOME/auth.json. When we have a key available,
|
||
// wrap the probe with a shell that materializes a per-run auth.json so
|
||
// the CLI can authenticate. The key content is passed via env (not on
|
||
// the command line) to avoid leaking it into process listings.
|
||
// Managed connections already contain their exact auth and provider config.
|
||
// Replacing that home with a native-key probe would test another provider.
|
||
const probeApiKey = config.managedAiConnection ? null : isNonEmpty(configOpenAiKey)
|
||
? configOpenAiKey
|
||
: isNonEmpty(hostOpenAiKey)
|
||
? hostOpenAiKey
|
||
: null;
|
||
const preparedProbe = await prepareCodexHelloProbe({
|
||
managedAiConnection: Boolean(config.managedAiConnection),
|
||
runId,
|
||
companyId: ctx.companyId,
|
||
target,
|
||
targetIsRemote,
|
||
cwd,
|
||
command,
|
||
args,
|
||
env,
|
||
probeApiKey,
|
||
});
|
||
try {
|
||
const probe = await runAdapterExecutionTargetProcess(
|
||
runId,
|
||
target,
|
||
preparedProbe.command,
|
||
preparedProbe.args,
|
||
{
|
||
cwd,
|
||
env: preparedProbe.env,
|
||
timeoutSec: 45,
|
||
graceSec: 5,
|
||
stdin: "Respond with hello.",
|
||
onLog: async () => {},
|
||
},
|
||
);
|
||
const parsed = parseCodexJsonl(probe.stdout);
|
||
// Plugin-catalog login is separate from model authentication. Its
|
||
// warnings must not explain an unrelated provider/process failure.
|
||
const providerStderr = probe.stderr.split(/\r?\n/)
|
||
.filter((line) => !/\bcodex_core_plugins(?:::|:)/.test(line) && !line.includes(PROBE_CLEANUP_WARNING))
|
||
.join("\n");
|
||
const detail = summarizeProbeDetail(probe.stdout, providerStderr, parsed.errorMessage);
|
||
const authEvidence = parsed.errorMessage?.trim() || providerStderr;
|
||
if (probe.stderr.includes(PROBE_CLEANUP_WARNING)) {
|
||
checks.push({
|
||
code: "codex_probe_cleanup_incomplete",
|
||
level: "info",
|
||
message: "Temporary probe files could not be fully removed; this does not change the connection result.",
|
||
});
|
||
}
|
||
|
||
if (probe.timedOut) {
|
||
checks.push({
|
||
code: "codex_hello_probe_timed_out",
|
||
level: "warn",
|
||
message: "Codex hello probe timed out.",
|
||
hint: "Retry the probe. If this persists, verify Codex can run `Respond with hello` from this directory manually.",
|
||
});
|
||
} else if ((probe.exitCode ?? 1) === 0) {
|
||
const summary = parsed.summary.trim();
|
||
const hasHello = /\bhello\b/i.test(summary);
|
||
checks.push({
|
||
code: hasHello ? "codex_hello_probe_passed" : "codex_hello_probe_unexpected_output",
|
||
level: hasHello ? "info" : "warn",
|
||
message: hasHello
|
||
? "Codex hello probe succeeded."
|
||
: "Codex probe ran but did not return `hello` as expected.",
|
||
...(summary ? { detail: summary.replace(/\s+/g, " ").trim().slice(0, 240) } : {}),
|
||
...(hasHello
|
||
? {}
|
||
: {
|
||
hint: "Try the probe manually (`codex exec --json -` then prompt: Respond with hello) to inspect full output.",
|
||
}),
|
||
});
|
||
} else if (CODEX_AUTH_REQUIRED_RE.test(authEvidence)) {
|
||
checks.push({
|
||
code: "codex_hello_probe_auth_required",
|
||
level: "warn",
|
||
message: "Codex CLI is installed, but authentication is not ready.",
|
||
...(detail ? { detail } : {}),
|
||
hint: probeApiKey
|
||
? "OPENAI_API_KEY was provided but Codex still rejected the request. Verify the key is valid for the OpenAI Responses API (e.g. `curl -H \"Authorization: Bearer $OPENAI_API_KEY\" https://api.openai.com/v1/models`), or run `codex login` and seed `~/.codex/auth.json`."
|
||
: "Codex CLI does not read OPENAI_API_KEY from the environment; set OPENAI_API_KEY in this adapter's config (so Paperclip writes it to `$CODEX_HOME/auth.json`) or run `codex login` on the host first.",
|
||
});
|
||
if (targetIsSandbox) {
|
||
// Emit the neutral canonical check so the user interface can decide
|
||
// login eligibility from a stable code. The user interface does not
|
||
// read the message text or the top-level status.
|
||
checks.push({
|
||
code: ADAPTER_AUTH_MISSING_CHECK_CODE,
|
||
level: "warn",
|
||
message: "This environment has no ready authentication for this adapter.",
|
||
...(detail ? { detail } : {}),
|
||
hint: "Provide credentials for this adapter, or start login in the environment.",
|
||
});
|
||
}
|
||
} else {
|
||
const modelRejection = codexHelloProbeModelRejectionCheck({
|
||
evidence: `${parsed.errorMessage ?? ""}\n${providerStderr}\n${probe.stdout}`,
|
||
detectedVersion: detectedCliVersion,
|
||
targetIsSandbox,
|
||
});
|
||
checks.push(modelRejection ?? {
|
||
code: "codex_hello_probe_failed",
|
||
level: "error",
|
||
message: "Codex hello probe failed.",
|
||
...(detail ? { detail } : {}),
|
||
hint: "Run `codex exec --json -` manually in this working directory and prompt `Respond with hello` to debug.",
|
||
});
|
||
}
|
||
} finally {
|
||
await preparedProbe.cleanup();
|
||
}
|
||
}
|
||
}
|
||
|
||
return {
|
||
adapterType: ctx.adapterType,
|
||
status: summarizeStatus(checks),
|
||
checks,
|
||
testedAt: new Date().toISOString(),
|
||
};
|
||
}
|