thinkpool-pair 0.7.280 → 0.7.281

package/bridge.mjs

@@ -46,7 +46,7 @@ import { createPermNotifier, shouldNotifyTurnDone, permissionSummary, clipSummar
 // resolveProviderEnv(id) → {ANTHROPIC_BASE_URL,ANTHROPIC_AUTH_TOKEN,ANTHROPIC_MODEL} for a
 // registered custom provider, or null for the built-in/unknown (leave the default env intact).
 // Multi-provider BYOK slice 1: a lane spawned with a `provider` id runs on that endpoint.
-import { resolveProviderEnv, providerNameMap, publicKeyB64, announceProviders, listProviders, addProvider, removeProvider, unseal, effectiveLaneModel, sameProviderEnv, providerModel } from './providers.mjs'
+import { resolveProviderEnv, resolveProviderRef, providerNameMap, publicKeyB64, announceProviders, listProviders, addProvider, removeProvider, unseal, effectiveLaneModel, sameProviderEnv, providerModel } from './providers.mjs'
 import { validateProviderSwitch, providerSwitchPlan, BUILTIN_PROVIDER } from './switch-provider.mjs'
 import { createSdkMcpServer, tool } from '@anthropic-ai/claude-agent-sdk'
 import { z } from 'zod'
@@ -2071,6 +2071,10 @@ function openStructured({ id, runtime = 'claude', model, models, effort, resume,
   const isIndependentMain = !entry.spawnedBy && !entry.sideParent && !entry.flowRole && (entry.spawnDepth || 0) === 0
   const canOpenMainConductor = isIndependentMain && entry.cascadeRole !== 'conductor'
   const canSpawnWorkers = isIndependentMain
+  const registeredProviderChoices = () => announceProviders()
+    .filter((provider) => provider.id !== 'anthropic')
+    .map((provider) => `${provider.name}${provider.model ? ` (${provider.model})` : ''}`)
+    .join(', ')
   const resolveAgentOpen = (args, { worker = false } = {}) => {
     const runtime = args?.runtime || entry.runtime || 'claude'
     if (runtime === 'codex' && args?.provider) return { error: 'A Codex terminal uses its Codex/OpenAI login in v1; custom bridge providers are not wired to Codex yet. Omit provider or open a Claude terminal for that provider.' }
@@ -2084,6 +2088,11 @@ function openStructured({ id, runtime = 'claude', model, models, effort, resume,
     if (runtime === 'hermes' && !modelCatalogValues(catalog).has(args.model)) {
       return { error: `Could not open a Hermes worker on ${JSON.stringify(args.model)} — that exact model is not in this parent session's ACP catalog.` }
     }
+    const provider = runtime === 'claude' && args?.provider ? resolveProviderRef(args.provider) : args?.provider
+    if (runtime === 'claude' && args?.provider && !provider) {
+      const choices = registeredProviderChoices()
+      return { error: `Unknown registered provider ${JSON.stringify(args.provider)}. No lane was opened and the built-in Claude login was not used.${choices ? ` Available provider names: ${choices}.` : ' No custom providers are registered on this host.'}` }
+    }
     if (runtime === 'claude' && !args?.provider && args?.model && /^gpt-/i.test(args.model)) {
       return { error: `Could not open a Claude terminal on Codex model ${JSON.stringify(args.model)}. Choose runtime="codex" or a Claude model.` }
     }
@@ -2096,7 +2105,7 @@ function openStructured({ id, runtime = 'claude', model, models, effort, resume,
       // permission-card mode from the conductor and never honor a lower explicit
       // override: spawn_terminal always opens them in bypassPermissions.
       mode: worker ? 'bypassPermissions' : (args?.mode || (entry.mode === 'plan' ? 'default' : entry.mode)),
-      provider: args?.provider,
+      provider,
     }
   }
   const dispatchContext = (now = Date.now()) => ({
@@ -2414,7 +2423,7 @@ function openStructured({ id, runtime = 'claude', model, models, effort, resume,
           model: z.string().optional().describe('model for the chosen runtime; Hermes workers require an exact ACP catalog ID such as nous:anthropic/claude-sonnet-4.6'),
           sliceType: z.enum(['scaffold', 'feature', 'fix', 'review']).optional().describe('optional cascade slice tier; omitted preserves normal inheritance/default, explicit model wins'),
           runtime: z.enum(['claude', 'codex', 'hermes']).optional().describe('agent runtime for the new lane; Hermes is allowed only from a top-level Hermes parent with an explicit catalog model'),
-          provider: z.string().optional().describe('optional registered LLM provider id to run this lane on (from the account\'s provider registry); omit for the default Claude/Anthropic path'),
+          provider: z.string().optional().describe(`optional registered Claude-compatible provider name or id; omit for built-in Anthropic${registeredProviderChoices() ? `. Available now: ${registeredProviderChoices()}` : ''}`),
           mode: z.enum(['default', 'acceptEdits', 'bypassPermissions', 'plan']).optional().describe('accepted for compatibility; spawned workers always run autonomously in bypassPermissions'),
         },
         async (args) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "thinkpool-pair",
-  "version": "0.7.280",
+  "version": "0.7.281",
   "description": "Share a local coding-agent CLI (Claude Code, Codex, Gemini, Aider, …) into a ThinkPool Code room, live.",
   "type": "module",
   "bin": {

package/providers.mjs

@@ -333,6 +333,27 @@ export function announceProviders() {
   return [{ id: BUILTIN_ID, name: 'Anthropic (Claude)', model: null, group: BUILTIN_ID }, ...custom]
 }
 
+/**
+ * Resolve an agent-facing provider reference without ever falling through to the
+ * built-in Claude login. Accept an exact id, a unique case-insensitive display
+ * name, or a unique configured model. Unknown/ambiguous references return null
+ * so callers can fail closed and show the current safe choices.
+ */
+export function resolveProviderRef(ref) {
+  if (!ref || ref === BUILTIN_ID) return BUILTIN_ID
+  const raw = String(ref).trim()
+  if (!raw) return BUILTIN_ID
+  const providers = loadProviders()
+  const exact = providers.find((provider) => provider.id === raw)
+  if (exact) return exact.id
+  const needle = raw.toLocaleLowerCase()
+  const matches = providers.filter((provider) => (
+    String(provider.name || '').trim().toLocaleLowerCase() === needle
+    || String(provider.model || '').trim().toLocaleLowerCase() === needle
+  ))
+  return matches.length === 1 ? matches[0].id : null
+}
+
 /**
  * id → name map for the ROOM announce's per-lane provider projection (name-only,
  * NEVER key or baseUrl). Built once per announce so a cross-device viewer resolves

package/thinkpool-capabilities.json

@@ -1,6 +1,6 @@
 {
   "schemaVersion": 1,
-  "bundleVersion": 6,
+  "bundleVersion": 7,
   "contracts": [
     {
       "id": "room-coordination",
@@ -41,7 +41,8 @@
     },
     {
       "id": "work-routing",
-      "version": 2,
+      "version": 3,
+      "providerRoutingContract": "A Claude worker may select a connected Anthropic-compatible provider by durable id, unique display name, or unique configured model. Unknown or ambiguous references fail closed and must never fall through to built-in Claude.",
       "routes": [
         {
           "id": "work-routing",