Skip to content

Fix event names in exec docs.#4833

Merged
pakrym-oai merged 2 commits intomainfrom
pakrym/fix-event-names-in-exec-docs
Oct 6, 2025
Merged

Fix event names in exec docs.#4833
pakrym-oai merged 2 commits intomainfrom
pakrym/fix-event-names-in-exec-docs

Conversation

@pakrym-oai
Copy link
Copy Markdown
Collaborator

@pakrym-oai pakrym-oai commented Oct 6, 2025

Fixes: #4776

@pakrym-oai
Copy link
Copy Markdown
Collaborator Author

@codex look for other discrepancies between docs and implementation

@pakrym-oai pakrym-oai marked this pull request as ready for review October 6, 2025 17:01
@chatgpt-codex-connector
Copy link
Copy Markdown
Contributor

Summary

  • Documented the error top-level JSONL event emitted by codex exec --json so the docs match the implementation’s event enum. docs/exec.mdL23-L31
  • Expanded the list of supported JSONL item types to include the todo_list and error variants produced by the exec event stream. docs/exec.mdL32-L41

Testing

  • ⚠️ Not Run (not requested)

View task →

@pakrym-oai pakrym-oai enabled auto-merge (squash) October 6, 2025 17:07
@pakrym-oai pakrym-oai disabled auto-merge October 6, 2025 17:07
@pakrym-oai pakrym-oai merged commit 661663c into main Oct 6, 2025
9 checks passed
@pakrym-oai pakrym-oai deleted the pakrym/fix-event-names-in-exec-docs branch October 6, 2025 17:07
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 6, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

JSON output mode docs are out of date

2 participants