Skip to content

Conversation

@pwang347
Copy link
Member

No description provided.

@pwang347 pwang347 marked this pull request as ready for review October 28, 2025 17:30
@pwang347 pwang347 requested review from bhavyaus and Copilot October 28, 2025 17:30
@vs-code-engineering vs-code-engineering bot added this to the October 2025 milestone Oct 28, 2025
@pwang347 pwang347 enabled auto-merge October 28, 2025 17:34
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes all references to "Hidden Model B" from the codebase, including the hash-based model detection function and its associated constants. This cleanup suggests that the hidden model experiment or feature is being discontinued.

Key changes:

  • Removed HIDDEN_MODEL_B_HASHES constant and isHiddenModelB() function from chat model capabilities
  • Updated model capability functions to remove Hidden Model B checks
  • Removed matchesModel() from XAIPromptResolver (falls back to family prefix matching)

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
src/platform/endpoint/common/chatModelCapabilities.ts Removed HIDDEN_MODEL_B_HASHES constant, isHiddenModelB() function, and updated modelSupportsReplaceString() and modelCanUseReplaceStringExclusively() to remove Hidden Model B checks
src/extension/prompts/node/agent/xAIPrompts.tsx Removed isHiddenModelB import and matchesModel() static method from XAIPromptResolver
src/extension/prompts/node/agent/agentPrompt.tsx Removed isHiddenModelB import and usage in AgentUserMessage.render()
src/extension/intents/node/agentIntent.ts Removed isHiddenModelB import and checks in getAgentTools() function

bhavyaus
bhavyaus previously approved these changes Oct 28, 2025
@pwang347 pwang347 added this pull request to the merge queue Oct 28, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Oct 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants