Skip to content

SEL-316: AMW main: Trust release gate fails because gateway/docs/openapi.json is stale - #152

Merged
PetrefiedThunder merged 1 commit into
mainfrom
agent/codex/sel-316
Oct 20, 2026
Merged

PetrefiedThunder merged 1 commit into
mainfrom
agent/codex/sel-316

Conversation

@PetrefiedThunder

@PetrefiedThunder PetrefiedThunder commented Oct 19, 2026 •

Copy link
Copy Markdown
Collaborator

Linear: SEL-316

Coding engine: codex, dispatched by Grok Bot for C.Lee.

What changed

Regenerated gateway/docs/openapi.json with the repository export script. The only schema change removes the advertised default object for POST /v1/launch/preflight, matching the merged route change to Body(default_factory=PreflightRequest). The request body remains optional; no unintended route or schema drift was found.

How verified

  • Before regeneration, from gateway/: UV_CACHE_DIR=/private/tmp/sel-316-uv-cache UV_PYTHON=3.12 ~/metacode/fleet/heavy uv run --with-requirements requirements.txt python scripts/export_openapi.py --check , failed with the reported stale-schema message.
  • Regenerated from gateway/: UV_CACHE_DIR=/private/tmp/sel-316-uv-cache UV_PYTHON=3.12 uv run --with-requirements requirements.txt python scripts/export_openapi.py , wrote docs/openapi.json.
  • After regeneration, from gateway/: UV_CACHE_DIR=/private/tmp/sel-316-uv-cache UV_PYTHON=3.12 uv run --with-requirements requirements.txt python scripts/export_openapi.py --check , passed: docs/openapi.json matches app.openapi().
  • From gateway/: UV_CACHE_DIR=/private/tmp/sel-316-uv-cache UV_PYTHON=3.12 ~/metacode/fleet/heavy uv run --with-requirements requirements.txt python -m pytest -q tests/test_preflight.py , 18 passed.
  • git diff --check , passed; the tracked diff contains only gateway/docs/openapi.json (1 insertion, 6 deletions).

Not done / follow-ups

The full Trust release gate and hosted CI were not run; the wrapper can confirm the PR gate. No other tracked files were changed.


Note

Low Risk
Documentation-only OpenAPI regeneration with no runtime or API behavior change beyond removing stale schema metadata.

Overview
Regenerates gateway/docs/openapi.json so the Trust release gate’s OpenAPI check passes again.

For POST /v1/launch/preflight, the request body schema no longer advertises an inline default object on top of PreflightRequest—only the $ref remains. That matches the live FastAPI route (Body(default_factory=PreflightRequest)): the body stays optional, with defaults coming from the Pydantic model rather than a duplicated default block in the exported spec.

Reviewed by Cursor Bugbot for commit 7d227fe. Bugbot is set up for automated code reviews on this repo. Configure here.

…json is stale (SEL-316)

Linear: SEL-316
Engine: codex (fleet coding engine, Grok Bot dispatch)
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-19T14:07:22.570528Z 7d227fe PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@PetrefiedThunder
PetrefiedThunder merged commit 2ba21ad into main Oct 20, 2026
43 checks passed
@PetrefiedThunder
PetrefiedThunder deleted the agent/codex/sel-316 branch October 20, 2026 03:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant