fix(app-server): preserve v2 response fields

Author: gersmann

codex/app_server/models.py

@@ -71,6 +71,7 @@ class ReasoningEffortOption(AppServerResultModel):
 
 
 class ModelInfo(AppServerResultModel):
+    additional_speed_tiers: list[str] | None = Field(default_factory=list)
     availability_nux: ModelAvailabilityNux | None = None
     default_reasoning_effort: protocol.ReasoningEffort
     description: str
@@ -110,6 +111,7 @@ class SkillInterface(AppServerResultModel):
 class SkillInfo(AppServerResultModel):
     name: str
     description: str
+    dependencies: protocol.SkillDependencies | None = None
     enabled: bool
     path: str
     scope: str
@@ -120,7 +122,7 @@ class SkillInfo(AppServerResultModel):
 class SkillsListEntry(AppServerResultModel):
     cwd: str
     skills: list[SkillInfo]
-    errors: list[str] = Field(default_factory=list)
+    errors: list[protocol.SkillErrorInfo] = Field(default_factory=list)
 
 
 class SkillsListResult(AppServerResultModel):

codex/app_server/options.py

@@ -519,6 +519,10 @@ class AppServerThreadListOptions(_AppServerOptionsModel):
         default=None,
         description="Sent as thread/list sortKey.",
     )
+    sort_direction: protocol.SortDirection | None = Field(
+        default=None,
+        description="Sent as thread/list sortDirection.",
+    )
     source_kinds: list[protocol.ThreadSourceKind] | None = Field(
         default=None,
         description="Sent as thread/list sourceKinds.",

tests/test_app_server_client.py

@@ -107,6 +107,7 @@ def _model_list_payload() -> JsonObject:
                 "id": "gpt-5.4",
                 "model": "gpt-5.4",
                 "displayName": "GPT-5.4",
+                "additionalSpeedTiers": ["flex"],
                 "description": "Primary model",
                 "hidden": False,
                 "defaultReasoningEffort": "medium",
@@ -578,6 +579,7 @@ def test_app_server_turn_and_list_options_use_protocol_owned_types() -> None:
         input=[{"type": "text", "text": "Summarize this repo."}],
     )
     list_params = AppServerThreadListOptions(
+        sort_direction=protocol.SortDirection("asc"),
         sort_key=protocol.ThreadSortKey("updated_at"),
         source_kinds=[protocol.ThreadSourceKind("appServer")],
     ).to_params()
@@ -594,6 +596,7 @@ def test_app_server_turn_and_list_options_use_protocol_owned_types() -> None:
         "threadId": "thr-1",
     }
     assert list_params.model_dump(mode="python", by_alias=True, exclude_none=True) == {
+        "sortDirection": "asc",
         "sortKey": "updated_at",
         "sourceKinds": ["appServer"],
     }
@@ -1334,6 +1337,9 @@ def list_skills(message: JsonObject) -> JsonObject:
                                     "name": "skill-creator",
                                     "description": "Create a skill",
                                     "enabled": True,
+                                    "dependencies": {
+                                        "tools": [{"type": "cli", "value": "git"}],
+                                    },
                                     "interface": {
                                         "displayName": "Skill Creator",
                                         "shortDescription": "Create or update skills",
@@ -1347,7 +1353,12 @@ def list_skills(message: JsonObject) -> JsonObject:
                                     "scope": "repo",
                                 }
                             ],
-                            "errors": [],
+                            "errors": [
+                                {
+                                    "message": "missing dependency",
+                                    "path": "/repo/.codex/skills/broken/SKILL.md",
+                                }
+                            ],
                         }
                     ]
                 },
@@ -1755,10 +1766,16 @@ def windows_sandbox_setup_start(message: JsonObject) -> JsonObject:
         windows_setup = await client.windows_sandbox.setup_start(mode="elevated", cwd="C:/repo")
 
         assert models[0].display_name == "GPT-5.4"
+        assert models[0].additional_speed_tiers == ["flex"]
         assert model_page.data[0].display_name == "GPT-5.4"
+        assert model_page.data[0].additional_speed_tiers == ["flex"]
         assert apps[0].id == "demo-app"
         assert app_page.data[0].id == "demo-app"
         assert skills[0].cwd == "/repo"
+        assert skills[0].errors[0].message == "missing dependency"
+        assert skills[0].errors[0].path == "/repo/.codex/skills/broken/SKILL.md"
+        assert skills[0].skills[0].dependencies is not None
+        assert skills[0].skills[0].dependencies.tools[0].value == "git"
         assert skills[0].skills[0].interface is not None
         assert skills[0].skills[0].interface.display_name == "Skill Creator"
         assert skills[0].skills[0].short_description == "Create or update skills"