bingocode 1.1.200-beta.14 → 1.1.200-beta.15

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bingocode",
-  "version": "1.1.200-beta.14",
+  "version": "1.1.200-beta.15",
   "type": "module",
   "bin": {
     "claude": "bin/claude-win.cjs",

package/src/server/services/providerService.ts

@@ -602,12 +602,6 @@ export class ProviderService {
     const writtenPaths: string[] = []
     const failedPaths: string[] = []
 
-    // Save pre-link bingo env so we can restore it on disconnect.
-    const bingoSettings = await this.readSettings()
-    const preLinkEnv = bingoSettings.env ?? null
-    bingoSettings.vscodePreLinkEnv = preLinkEnv
-    await this.writeSettings(bingoSettings)
-
     // Fix #8: Snapshot original VS Code claudeCode per path for unlink restore.
     const vscodePreLinkClaudeCode: Record<string, unknown> = {}
     // Fix #9: Backup original contents for rollback on partial failure.
@@ -724,6 +718,7 @@ export class ProviderService {
     }
 
     // Persist so re-sync on restart and unlink work correctly
+    const bingoSettings = await this.readSettings()
     bingoSettings.vscodeLinked = true
     bingoSettings.vscodeLinkedPaths = writtenPaths
     bingoSettings.vscodePreLinkClaudeCode = vscodePreLinkClaudeCode
@@ -741,32 +736,18 @@ export class ProviderService {
   }
 
   /**
-   * Restore pre-link bingo env so the CLI no longer routes through the proxy,
-   * and reset VS Code settings to their pre-link state (the original claudeCode
-   * that was saved in vscodePreLinkClaudeCode). Falls back to a minimal stub
-   * if no snapshot exists.
+   * Reset VS Code settings to pre-link state.  Does NOT touching bingo
+   * settings.env — the CLI keeps routing through the proxy independently.
    *
-   * Without this, the only way to stop VS Code from using bingo would be to
-   * delete ~/.claude/bingo/settings.json entirely — which would also kill CLI
-   * routing.  Saving/restoring the pre-link env keeps CLI and VS Code independent.
+   * Restores the original claudeCode snapshot (vscodePreLinkClaudeCode)
+   * when available, otherwise writes an override that points VS Code to
+   * api.anthropic.com so the extension shows its native login screen.
    */
   async unlinkVscode(): Promise<{ paths: string[]; linked: boolean }> {
     const bingoSettings = await this.readSettings()
     const linkedPaths = (bingoSettings.vscodeLinkedPaths as string[]) || []
 
-    // Restore bingo env to pre-link state
-    const preLinkEnv = bingoSettings.vscodePreLinkEnv
-    if (preLinkEnv !== undefined) {
-      if (preLinkEnv === null || Object.keys(preLinkEnv as Record<string, unknown>).length === 0) {
-        delete bingoSettings.env
-      } else {
-        bingoSettings.env = preLinkEnv
-      }
-      delete bingoSettings.vscodePreLinkEnv
-    }
-
-    // Fix #8: Restore original claudeCode snapshot per-path instead of a hardcoded stub.
-    // Fall back to { preferredLocation: "panel" } when no snapshot exists for a path.
+    // Restore original claudeCode snapshot per-path. Fall back to a clean
     const preLinkClaudeCode =
       (bingoSettings.vscodePreLinkClaudeCode as Record<string, unknown> | undefined) || {}
 
@@ -774,7 +755,12 @@ export class ProviderService {
       try {
         const raw = await fs.readFile(settingsPath, "utf-8")
         const settings = JSON.parse(raw)
-        settings.claudeCode = preLinkClaudeCode[settingsPath] ?? { preferredLocation: "panel" }
+        settings.claudeCode = preLinkClaudeCode[settingsPath] ?? {
+          preferredLocation: "panel",
+          environmentVariables: [
+            { name: "ANTHROPIC_BASE_URL", value: "https://api.anthropic.com" },
+          ],
+        }
         await fs.writeFile(settingsPath, JSON.stringify(settings, null, 4) + '\n')
       } catch {
         // File may have been moved or deleted since linking — harmless
@@ -784,6 +770,7 @@ export class ProviderService {
     bingoSettings.vscodeLinked = false
     delete bingoSettings.vscodeLinkedPaths
     delete bingoSettings.vscodePreLinkClaudeCode
+    delete bingoSettings.vscodePreLinkEnv
     await this.writeSettings(bingoSettings)
 
     return { paths: linkedPaths, linked: false }

package/src/tools.ts

@@ -87,8 +87,6 @@ import { TaskListTool } from './tools/TaskListTool/TaskListTool.js'
 import uniqBy from 'lodash-es/uniqBy.js'
 import { isToolSearchEnabledOptimistic } from './utils/toolSearch.js'
 import { isTodoV2Enabled } from './utils/tasks.js'
-import { getInitialSettings } from './utils/settings/settings.js'
-import { getCurrentSessionBrainMode } from './utils/sessionStorage.js'
 // Dead code elimination: conditional import for CLAUDE_CODE_VERIFY_PLAN
 /* eslint-disable custom-rules/no-process-env-top-level, @typescript-eslint/no-require-imports */
 const VerifyPlanExecutionTool =
@@ -271,13 +269,9 @@ export function filterToolsByDenyRules<
   return tools.filter(tool => !getDenyRuleForTool(permissionContext, tool))
 }
 
-function isBrainModeActive(): boolean {
-  return getInitialSettings().brainMode === true || getCurrentSessionBrainMode() === true
-}
-
 export const getTools = (permissionContext: ToolPermissionContext, brainMode?: boolean): Tools => {
   // Brain mode: coordinator only needs Agent + AskUserQuestion
-  if (brainMode ?? isBrainModeActive()) {
+  if (brainMode) {
     return filterToolsByDenyRules([AgentTool, AskUserQuestionTool], permissionContext)
   }
 
@@ -359,7 +353,7 @@ export function assembleToolPool(
   mcpTools: Tools,
   brainMode?: boolean,
 ): Tools {
-  const effectiveBrainMode = brainMode ?? isBrainModeActive()
+  const effectiveBrainMode = brainMode ?? false
   const builtInTools = getTools(permissionContext, effectiveBrainMode)
 
   // When brainMode is active, skip MCP tools entirely