feat(proxy): unify proxy connect into one command with target flags - #479
Open
vadimvlasenko wants to merge 7 commits into
Open
feat(proxy): unify proxy connect into one command with target flags#479vadimvlasenko wants to merge 7 commits into
vadimvlasenko wants to merge 7 commits into
Conversation
New connectors/vscode-claude-code.ts connector writes claudeCode.disableLoginPrompt and claudeCode.environmentVariables (ANTHROPIC_BASE_URL/ANTHROPIC_AUTH_TOKEN) into the VS Code Claude Code extension's settings.json, upserting only the CodeMie-managed env var entries and preserving everything else. Exports getVsCodeProductDir and writeAtomically from vscode.ts for reuse instead of duplicating per-OS path/atomic-write logic. Part of vscode-claude-noauth-desktop-connect (Tasks 1-2). Generated with AI Co-Authored-By: codemie-ai <codemie.ai@gmail.com>
Adds an opt-in --vscode-claude-code flag (plus --insiders) to 'codemie proxy connect desktop'. When passed, after Claude Desktop's own config is written, it calls writeVsCodeClaudeCodeConfig with the same local gateway URL/key, non-fatally (its own try/catch, no rollback of the already-succeeded Desktop config on failure). Part of vscode-claude-noauth-desktop-connect (Task 3). Generated with AI Co-Authored-By: codemie-ai <codemie.ai@gmail.com>
….CR-005) - CR-001: getVsCodeClaudeCodeSettingsPath now throws ConfigurationError when the target VS Code/Insiders product directory doesn't exist, mirroring vscode.ts's getVsCodeLanguageModelsPath, instead of silently writing into a directory tree nothing will ever read. - CR-002: a malformed (non-array) existing claudeCode.environmentVariables value is now logged and replaced instead of silently discarded. - CR-003: upsertManagedEnvVars no longer crashes on null/non-object array entries — they are dropped with a warning instead of throwing a raw TypeError. - CR-004: pre-existing duplicate-named managed entries are now de-duplicated (keeping the freshly-written value) instead of leaving stale duplicates. - CR-005: removed index.ts's duplicate, untested secret-bearing log call — the connector already logs the same event internally with sanitized values, so the redundant/untested call site is gone rather than needing its own test. Adds Vitest coverage for all four connector-level fixes and the new install-presence check (getVsCodeClaudeCodeSettingsPath). Part of vscode-claude-noauth-desktop-connect (code-review.final fix-up). Generated with AI Co-Authored-By: codemie-ai <codemie.ai@gmail.com>
…connect Generated with AI Co-Authored-By: codemie-ai <codemie.ai@gmail.com>
Replace the connect desktop / connect vscode subcommands with a single `codemie proxy connect` taking orthogonal target flags (--claude-desktop, --vscode, --vscode-claude-code) over one daemon lifecycle. Adds a per-target summary and a non-zero exit on partial failure; deprecated aliases are retained with highlighted notices; bare connect prints a target list. Config writers unchanged. Adds connect-orchestrator.ts and 24 unit/wiring tests. Refs PR #479. Generated with AI Co-Authored-By: codemie-ai <codemie.ai@gmail.com>
vadimvlasenko
changed the title
Reconcile #481 (managed-MCP oauth-shape logging + Claude Desktop managed-settings override warning) into the unified connect orchestrator's runClaudeDesktop; index.ts resolved to the unified command structure. Generated with AI Co-Authored-By: codemie-ai <codemie.ai@gmail.com>
…ommands Add a "Connect clients via CodeMie Proxy" section covering the unified command with composable target flags (--claude-desktop, --vscode, --vscode-claude-code), and mark the connect desktop / connect vscode subcommands as deprecated aliases. Swap the primary command examples in the VS Code BYOK and Claude Desktop sections to the new flags. Generated with AI Co-Authored-By: codemie-ai <codemie.ai@gmail.com>
8nevil8
approved these changes
Aug 12, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Unifies
codemie proxy connectinto a single command that takes orthogonal target flags, replacing the separateconnect desktopandconnect vscodesubcommands. One invocation can configure any combination of the Claude Desktop app, VS Code Copilot BYOK models, and the VS Code Claude Code extension over a single daemon lifecycle. The earlier opt-in--vscode-claude-codeflag (previously aconnect desktopsub-flag) is reworked into a standalone target on the unified command.Changes
codemie proxy connectwith composable flags:--claude-desktop,--vscode,--vscode-claude-code— the union is written over one daemon lifecycle.process.exitCode = 1on any partial failure; the daemon is kept if ≥1 target succeeds and rolled back only if all fail and it was started this run.connectprints a friendly target list and exits 0 (writes nothing).connect desktop/connect vscodekept as deprecated aliases with a highlightedchalk.bold.yellownotice pointing to the new flag form.--insiderswith no VS Code target warns and continues.claude-desktop/vscode-byok) — no new telemetry value, single-target runs spawn a byte-identical daemon.daemonMatchesRequestpath for every caller (the loose desktop gate removed).connect-orchestrator.ts; config writers unchanged.enablePositionalOptions()added onproxy/connectso aliases can reuse--profile.Testing
Checklist
npm run ci)connect vscode+vscode-byok; the reworked--vscode-claude-codeflag was unreleased