Skip to content

WIP: Add VS Code provider controls#962

Closed
techbrewboss wants to merge 1 commit into
Gitlawb:mainfrom
techbrewboss:issue-961
Closed

WIP: Add VS Code provider controls#962
techbrewboss wants to merge 1 commit into
Gitlawb:mainfrom
techbrewboss:issue-961

Conversation

@techbrewboss
Copy link
Copy Markdown
Collaborator

Summary

  • adds slash-command suggestions in the VS Code chat composer
  • adds provider/model picker controls plus a Provider Manager UI for configured profiles
  • preserves startup-profile credentials when saving provider changes
  • tightens OpenAI custom auth validation so header names require values

Addresses #961

Validation

  • npm run lint (vscode-extension/openclaude-vscode)
  • bun test src/*.test.js (vscode-extension/openclaude-vscode)
  • bun test src/utils/providerValidation.test.ts

Copy link
Copy Markdown
Collaborator

@Vasanthdev2004 Vasanthdev2004 left a comment

Choose a reason for hiding this comment

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

Targeted maintainer triage of current head 98c4902d12e328ec45576220c91f94f51ec2c8b4; not a full code review because this is marked WIP/draft and is not mergeable.

Verdict: Needs changes

Blocking issues:

  1. GitHub reports the branch as conflicting/dirty with main; please rebase and resolve conflicts first.
  2. The current check result is from April 30 and predates a lot of extension/provider-manager work on main, so it is not enough signal for merge readiness anymore.
  3. The PR overlaps with sensitive extension provider controls and profile persistence behavior. After rebase, please rerun the VS Code extension tests plus the provider validation tests listed in the PR body and update the PR with the new results.

Once the branch is current and no longer WIP, this can get a real code review against the latest extension architecture.

@techbrewboss
Copy link
Copy Markdown
Collaborator Author

Doing a fresh pr with updates

@techbrewboss techbrewboss deleted the issue-961 branch May 9, 2026 23:53
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.

2 participants