fix(rust): align model list request contract

Remove the unconfirmed skipCache option and keep cwd-only repository scoping.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Author: Eljo George

rust/src/generated/api_types.rs

@@ -8484,7 +8484,7 @@ pub struct ModelSetReasoningEffortResult {
     pub reasoning_effort: String,
 }
 
-/// Optional GitHub token, working directory, and cache controls used to resolve available models.
+/// Optional GitHub token and working directory used to resolve available models.
 ///
 /// <div class="warning">
 ///
@@ -8501,9 +8501,6 @@ pub struct ModelsListRequest {
     /// GitHub token for per-user model listing. When provided, resolves this token to determine the user's Copilot plan and available models instead of using the global auth.
     #[serde(skip_serializing_if = "Option::is_none")]
     pub git_hub_token: Option<String>,
-    /// When true, bypasses cached model data and refreshes the available model list.
-    #[serde(skip_serializing_if = "Option::is_none")]
-    pub skip_cache: Option<bool>,
 }
 
 /// Target model identifier and optional reasoning effort, summary, capability overrides, and context tier.

rust/tests/api_types_test.rs

@@ -106,19 +106,15 @@ fn permission_event_exposes_managed_approval_required() {
 }
 
 #[test]
-fn models_list_request_serializes_repository_options() {
+fn models_list_request_serializes_repository_cwd() {
     let request = ModelsListRequest {
         cwd: Some("/workspace/repository".to_string()),
         git_hub_token: None,
-        skip_cache: Some(true),
     };
 
     assert_eq!(
         serde_json::to_value(request).unwrap(),
-        serde_json::json!({
-            "cwd": "/workspace/repository",
-            "skipCache": true
-        })
+        serde_json::json!({ "cwd": "/workspace/repository" })
     );
     assert_eq!(
         serde_json::to_value(ModelsListRequest::default()).unwrap(),

rust/tests/session_test.rs

@@ -4045,7 +4045,7 @@ async fn rpc_namespace_client_models_list_dispatches_correctly() {
 }
 
 #[tokio::test]
-async fn rpc_namespace_client_models_list_sends_repository_options() {
+async fn rpc_namespace_client_models_list_sends_repository_cwd() {
     let (session, mut server) = create_session_pair().await;
     let session = Arc::new(session);
 
@@ -4057,7 +4057,6 @@ async fn rpc_namespace_client_models_list_sends_repository_options() {
             .list_with_params(ModelsListRequest {
                 cwd: Some("/workspace/repository".to_string()),
                 git_hub_token: None,
-                skip_cache: Some(true),
             })
             .await
     });
@@ -4067,8 +4066,7 @@ async fn rpc_namespace_client_models_list_sends_repository_options() {
     assert_eq!(
         request["params"],
         serde_json::json!({
-            "cwd": "/workspace/repository",
-            "skipCache": true
+            "cwd": "/workspace/repository"
         })
     );
     server

scripts/codegen/rust.ts

@@ -17,8 +17,8 @@ import { fileURLToPath } from "url";
 import { promisify } from "util";
 import type { JSONSchema7, JSONSchema7Definition } from "json-schema";
 import {
+	addCwdToModelsListRequest,
 	addManagedApprovalRequiredToPermissionRequests,
-	addModelsListRequestOptions,
 	type ApiSchema,
 	type DefinitionCollections,
 	EXCLUDED_EVENT_TYPES,
@@ -2220,7 +2220,7 @@ async function generate(): Promise<void> {
 	);
 	const apiSchema = propagateInternalVisibility(
 		postProcessSchema(
-			stripBooleanLiterals(addModelsListRequestOptions(apiRaw)) as JSONSchema7,
+			stripBooleanLiterals(addCwdToModelsListRequest(apiRaw)) as JSONSchema7,
 		),
 	) as unknown as ApiSchema;
 

scripts/codegen/utils.ts

@@ -497,25 +497,16 @@ export function addManagedApprovalRequiredToPermissionRequests<T extends JSONSch
 }
 
 /**
- * Add model listing options until the pinned CLI schema includes the fields.
+ * Add repository scoping to model listing until the pinned CLI schema includes the field.
  */
-export function addModelsListRequestOptions<T extends JSONSchema7>(schema: T): T {
+export function addCwdToModelsListRequest<T extends JSONSchema7>(schema: T): T {
     const cloned = cloneSchemaForCodegen(schema);
-    const properties: Record<string, JSONSchema7> = {
-        cwd: {
-            description:
-                "Working directory used to apply repository model policy. When omitted, model availability is account-global.",
-            type: ["string", "null"],
-        },
-        skipCache: {
-            description:
-                "When true, bypasses cached model data and refreshes the available model list.",
-            type: ["boolean", "null"],
-        },
+    const property: JSONSchema7 = {
+        description:
+            "Working directory used to apply repository model policy. When omitted, model availability is account-global.",
+        type: ["string", "null"],
     };
-    for (const property of Object.values(properties)) {
-        (property as Record<string, unknown>)["x-copilot-sdk-append-last"] = true;
-    }
+    (property as Record<string, unknown>)["x-copilot-sdk-append-last"] = true;
 
     for (const definitions of [cloned.definitions, cloned.$defs]) {
         if (!definitions) continue;
@@ -533,20 +524,13 @@ export function addModelsListRequestOptions<T extends JSONSchema7>(schema: T): T
                 (candidate.type === "object" || candidate.properties !== undefined),
         );
         if (!objectDefinition) continue;
-
-        let patched = false;
-        for (const [name, property] of Object.entries(properties)) {
-            if (objectDefinition.properties?.[name]) continue;
-            objectDefinition.properties = {
-                ...objectDefinition.properties,
-                [name]: cloneSchemaForCodegen(property),
-            };
-            patched = true;
-        }
-        if (patched) {
-            requestDefinition.description =
-                "Optional GitHub token, working directory, and cache controls used to resolve available models.";
-        }
+        if (objectDefinition.properties?.cwd) continue;
+        requestDefinition.description =
+            "Optional GitHub token and working directory used to resolve available models.";
+        objectDefinition.properties = {
+            ...objectDefinition.properties,
+            cwd: cloneSchemaForCodegen(property),
+        };
     }
 
     return cloned;