Add testing skill for the OpenCodex management API - #616
devin-ai-integration[bot] wants to merge 4 commits into
Conversation
Co-authored-by: Epinephrine <luvs01@hanmail.net>
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configurationConfiguration used: Repository: luvs01/opencodex/.coderabbit.yaml Review profile: ASSERTIVE Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
✅ Deterministic PR hygiene checks passed. |
⏳ DRAFT
What to do
Review readiness checklist
0/4 boxes ticked. Current head: |
OPENCODEX_HOME relocates only opencodex state; startup still syncs client homes unless clientIntegrations.* are off AND CODEX_HOME/GROK_HOME/ CLAUDE_CONFIG_DIR/OPENCODEX_CLAUDE_DESKTOP_CONFIG_DIR point at scratch. codexAutoStart:false never gated those writes. Co-Authored-By: Epinephrine <luvs01@hanmail.net>
Co-Authored-By: Epinephrine <luvs01@hanmail.net>
|
Devin is currently unreachable - the session may have died. |
| - Residual writes the recipe does NOT cover (macOS only, opencodex-owned artifacts | ||
| only): startup always runs `refreshOwnedRaycastCatalog` (rewrites an existing | ||
| opencodex-owned Raycast provider entry under the OS home — no env override) and | ||
| `reconcileShellHook` (removes the opencodex-marked block from `~/.zshrc` when the |
There was a problem hiding this comment.
🔍 Scratch-home warning may also apply outside macOS
The skill labels Raycast writes macOS-only, but refreshOwnedRaycastCatalog has no platform gate. Check whether Windows testers with an owned Raycast catalog also need isolation.
Was this helpful? React with 👍 or 👎 to provide feedback.
| ## Calling /api/* | ||
| - Header: `x-opencodex-api-key: <token>` (or `Authorization: Bearer <token>`). No token → | ||
| `401 {"error":"opencodex admin token required"}`. Origin header NOT required for curl. | ||
| - Useful routes: `GET/PUT /api/lab/automation` (status has `schedulerRunning` — live | ||
| interval presence, not just policy), `POST /api/lab/automation/run` (SYNCHRONOUS — the | ||
| 200 response IS the terminal run record), `GET /api/lab/automation/runs`. | ||
| - PUT policy body: `{"policy":{"enabled":true,"layers":{"protocolConformance":true}}}`; | ||
| merges with disk policy atomically. | ||
| - Manual run body: `{"evidenceLayer":"protocol_conformance","scenarioId":"responses-core.protocol.request-shape"}` | ||
| — protocol_conformance runs need NO provider (in-process fixture harness; upstream is | ||
| deliberately dead). live_route_compatibility needs providerName+modelId in config. | ||
| - Scheduler tick is `LAB_AUTOMATION_HARD_MAX.schedulerTickMs` = 60s — scheduled work only | ||
| appears in `/runs` after the first tick; runs persist to `<OPENCODEX_HOME>/lab/automation-state.json`. |
There was a problem hiding this comment.
🔍 The described curl walkthrough is missing
The PR description promises curl-based baseline, enable, and disable checks. The skill supplies routes and request bodies, but no curl commands or transition checks. Confirm whether the walkthrough belongs here.
Was this helpful? React with 👍 or 👎 to provide feedback.
|
Closing in favor of #632 (identical diff on current dev). This PR is permanently stuck in draft: after the post-ready push, the review gate seeded |
Adds a testing skill documenting how to boot OpenCodex as a foreground server on a scratch OPENCODEX_HOME and drive the Lab automation management API with curl: admin auth header, baseline/enable/disable state transitions, manual run dispatch, and clean shutdown. Verified against the real server on Windows.
Devin Session: https://app.devin.ai/sessions/cfc5342a3c4d4b7eb9c1b57b2adca7c6
Review readiness checklist
This PR stays in draft until every box below is ticked. Tick all four boxes once the requirements are met:
Required local validation passed; commands, results, and any full-suite exception are documented.
I pushed my PR to a recent dev commit (at most 10 behind; a maintainer may still ask for the exact tip before merge).
I resolved all correct Codex and CodeRabbit findings.
My PR is ready for review.
Open in Devin Desktop: https://app.devin.ai/desktop/session/cfc5342a3c4d4b7eb9c1b57b2adca7c6?variant=devin
Requested by: @luvs01