fix(schema): coerce ExecCommandEndEvent.exit_code to integer

Author: gersmann

.github/workflows/codex-autoreview.yml

@@ -1,21 +1,30 @@
-name: Codex Review
+name: Codex Review & Edits
 
 on:
   pull_request:
     types: [opened, synchronize, reopened, ready_for_review]
+  issue_comment:
+    types: [created]
+  pull_request_review:
+    types: [submitted]
+  pull_request_review_comment:
+    types: [created]
 
 permissions:
-  contents: read
+  contents: write
   pull-requests: write
 
 jobs:
   review:
-    name: Run Codex review
+    name: Run Codex review & edits
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
+        with:
+          # Ensure we can push back to the PR head branch
+          fetch-depth: 0
 
-      - name: Codex autonomous review
+      - name: Codex autonomous review & edits
         uses: gersmann/codex-review-action@latest
         with:
           openai_api_key: ${{ secrets.OPENAI_API_KEY }}

codex/protocol/types.py

@@ -1,7 +1,7 @@
 # generated by datamodel-codegen:
 # ruff: noqa: F821
 #   filename:  protocol.schema.json
-#   timestamp: 2025-09-14T16:20:04+00:00
+#   timestamp: 2025-09-14T16:47:21+00:00
 
 from __future__ import annotations
 
@@ -145,7 +145,7 @@ class ExecCommandEndEvent(BaseModelWithExtras):
     aggregated_output: str = Field(..., description="Captured aggregated output")
     call_id: str = Field(..., description="Identifier for the ExecCommandBegin that finished.")
     duration: str = Field(..., description="The duration of the command execution.")
-    exit_code: float = Field(..., description="The command's exit code.")
+    exit_code: int = Field(..., description="The command's exit code.")
     formatted_output: str = Field(
         ..., description="Formatted output from the command, as seen by the model."
     )
@@ -566,7 +566,7 @@ class TaskCompleteEvent(BaseModelWithExtras):
 
 
 class TaskStartedEvent(BaseModelWithExtras):
-    model_context_window: float | None = None
+    model_context_window: int | None = None
 
 
 class TextResourceContents(BaseModelWithExtras):
@@ -576,16 +576,16 @@ class TextResourceContents(BaseModelWithExtras):
 
 
 class TokenUsage(BaseModelWithExtras):
-    cached_input_tokens: float
-    input_tokens: float
-    output_tokens: float
-    reasoning_output_tokens: float
-    total_tokens: float
+    cached_input_tokens: int
+    input_tokens: int
+    output_tokens: int
+    reasoning_output_tokens: int
+    total_tokens: int
 
 
 class TokenUsageInfo(BaseModelWithExtras):
     last_token_usage: TokenUsage
-    model_context_window: float | None = None
+    model_context_window: int | None = None
     total_token_usage: TokenUsage
 
 
@@ -666,7 +666,7 @@ class EventMsgError(BaseModelWithExtras):
 
 
 class EventMsgTaskStarted(BaseModelWithExtras):
-    model_context_window: float | None = None
+    model_context_window: int | None = None
     type: Literal["task_started"]
 
 
@@ -767,7 +767,7 @@ class EventMsgExecCommandEnd(BaseModelWithExtras):
     aggregated_output: str = Field(..., description="Captured aggregated output")
     call_id: str = Field(..., description="Identifier for the ExecCommandBegin that finished.")
     duration: str = Field(..., description="The duration of the command execution.")
-    exit_code: float = Field(..., description="The command's exit code.")
+    exit_code: int = Field(..., description="The command's exit code.")
     formatted_output: str = Field(
         ..., description="Formatted output from the command, as seen by the model."
     )
@@ -845,8 +845,8 @@ class EventMsgGetHistoryEntryResponse(BaseModelWithExtras):
         None,
         description="The entry at the requested offset, if available and parseable.",
     )
-    log_id: float
-    offset: float
+    log_id: int
+    offset: int
     type: Literal["get_history_entry_response"]
 
 
@@ -1088,8 +1088,8 @@ class GetHistoryEntryResponseEvent(BaseModelWithExtras):
         None,
         description="The entry at the requested offset, if available and parseable.",
     )
-    log_id: float
-    offset: float
+    log_id: int
+    offset: int
 
 
 class GitDiffToRemoteResponse(BaseModelWithExtras):
@@ -1505,10 +1505,10 @@ class ResumeConversationResponse(BaseModelWithExtras):
 
 
 class SessionConfiguredEvent(BaseModelWithExtras):
-    history_entry_count: float = Field(
+    history_entry_count: int = Field(
         ..., description="Current number of entries in the history log."
     )
-    history_log_id: float = Field(
+    history_log_id: int = Field(
         ...,
         description="Identifier of the history log file (inode on Unix, 0 otherwise).",
     )
@@ -1525,10 +1525,10 @@ class SessionConfiguredEvent(BaseModelWithExtras):
 
 
 class EventMsgSessionConfigured(BaseModelWithExtras):
-    history_entry_count: float = Field(
+    history_entry_count: int = Field(
         ..., description="Current number of entries in the history log."
     )
-    history_log_id: float = Field(
+    history_log_id: int = Field(
         ...,
         description="Identifier of the history log file (inode on Unix, 0 otherwise).",
     )