mirror of
https://github.com/microsoft/vscode.git
synced 2026-05-17 13:50:46 +01:00
Rob Lourens
096e5abe9d
* Fix broken sessions list From #311751 Co-authored-by: Copilot <copilot@github.com> * fix tests * agentHost: settings followups - Unrelated: fix local agent host being broken on main following some overnight cleanups - Register a JSON schema for agent session settings (only when they're open since it has a perf cost) - When there are multiple custom approval buttons, show them in a dropdown similar to other tool calls. * Add tests for agent session settings schema registration * Bump @github/copilot from ^1.0.28 to ^1.0.34 Matches the version used in extensions/copilot. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * agentHost: tolerate synthetic Copilot models with empty capabilities After bumping @github/copilot to 1.0.34, listModels() returns a synthetic 'auto' router entry shaped like { id: 'auto', name: 'Auto', capabilities: {} }. The SDK's ModelInfo type still declares capabilities.limits.max_context_window_tokens as required, so reading it crashed _listModels and zeroed out the model list. Wrap the SDK type in a local ICopilotModelInfo with the actually-optional fields marked optional, and skip entries lacking max_context_window_tokens with a warn log instead of throwing. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> * agentHost: surface synthetic 'auto' model with optional maxContextWindow Instead of dropping the synthetic 'auto' router model, make IAgentModelInfo.maxContextWindow optional and pass it through as undefined when the SDK doesn't report a fixed context window. The agentHostLanguageModelProvider already handles undefined via '?? 0', and the protocol state type already had the field optional. Update the real-SDK listModels test to allow undefined maxContextWindow and to assert the 'auto' entry is present. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <copilot@github.com> Co-authored-by: Connor Peet <connor@peet.io> Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>