fix(postprocess): stop injecting noqa on root line to preserve Field(...) metadata

Author: gersmann

codex/protocol/types.py

@@ -1,7 +1,7 @@
 # generated by datamodel-codegen:
 # ruff: noqa: F821
 #   filename:  protocol.schema.json
-#   timestamp: 2025-09-14T16:17:29+00:00
+#   timestamp: 2025-09-14T16:18:22+00:00
 
 from __future__ import annotations
 
@@ -1214,7 +1214,7 @@ class ResponseItem(
         | ResponseItem7
         | ResponseItem8
         | ResponseItem9
-    )  # noqa: F821
+    )
 
 
 class ResumeConversationParams(BaseModelWithExtras):
@@ -1383,7 +1383,7 @@ class ClientRequest(RootModel):
         | ClientRequestGetUserSavedConfig
         | ClientRequestGetUserAgent
         | ClientRequestExecOneOffCommand
-    )  # noqa: F821 = Field(..., description='Request from the client to the server.')
+    ) = Field(..., description="Request from the client to the server.")
 
 
 class ConversationHistoryResponseEvent(BaseModelWithExtras):
@@ -1495,7 +1495,7 @@ class EventMsg(RootModel):
         | EventMsgTurnAborted
         | EventMsgShutdownComplete
         | EventMsgConversationHistory
-    )  # noqa: F821 = Field(..., description='Response event from the agent')
+    ) = Field(..., description="Response event from the agent")
 
 
 class ResumeConversationResponse(BaseModelWithExtras):

scripts/postprocess_protocol_types.py

@@ -27,7 +27,8 @@ def main() -> int:
         lambda m: f"class {m.group(1)}(RootModel):",
         s,
     )
-    s = _re.sub(r"(\s+root:\s*\([^\)]+\))", r"\1  # noqa: F821", s)
+    # Avoid inserting inline comments before Field(...) assignments on the root line,
+    # rely on the file-level ruff directive instead.
     # If previous runs introduced quotes around variant names, strip them inside union lines
     s = _re.sub(
         r"'((?:EventMsg|ClientRequest|ServerRequest|ServerNotification|InputItem)[A-Za-z0-9_]+)'",