thinkpool-pair 0.7.346 → 0.7.348

package/bridge.mjs

@@ -55,7 +55,7 @@ import { readCodexDefaultModel, readCodexModels, codexConfigForMode, codexThread
 import { codexAccountUsageLine, codexCreditsReportLine, codexLimitReportLine } from './codex-commands.mjs'
 import { withMcpSessionFactory } from './codex-mcp-http.mjs'
 import { startStructuredSession } from './runtime-session.mjs'
-import { cleanTerminalName, fallbackTerminalName, modelTerminalNameInput } from './terminal-name.mjs'
+import { cleanTerminalName, modelTerminalNameInput } from './terminal-name.mjs'
 import { defaultStructuredMode, normalizeStructuredEffort, shouldDeferStructuredRuntime, structuredModeForSlice, structuredModeLocked, structuredModesForLane, structuredRuntimeForCommand, structuredRuntimeMetadata, structuredRuntimeSupportsMode } from './runtime-registry.mjs'
 import { commandCatalogForRuntime, commandHelpLine, reconcileCommandCatalog } from './command-catalog.mjs'
 import { gitDiffReport } from './git-diff-report.mjs'
@@ -1497,7 +1497,7 @@ const requestManagedTerminalName = async (id, task, retry = true) => {
     const data = await response.json().catch(() => ({}))
     // Bridge-created lanes can receive their task before the browser has
     // inserted the terminal row. Retry that non-billable race once; every
-    // provider failure simply keeps the already-visible local fallback.
+    // provider failure leaves the terminal on its ordinary default label.
     if (retry && response.status === 404 && data?.code === 'terminal_pending') {
       const timer = setTimeout(() => { void requestManagedTerminalName(id, task, false) }, 1500)
       timer.unref?.()
@@ -1511,18 +1511,16 @@ const requestManagedTerminalName = async (id, task, retry = true) => {
   }
 }
 
-// First real task only. The zero-token local name appears immediately; a
-// managed tiny model may improve it later without delaying the agent turn.
+// First real task only. Naming is managed-model-only: keep the terminal's
+// ordinary default label until Groq returns instead of showing a local guess.
 const autoNameTerminal = (id, text) => {
   const entry = sessions.get(id)
   if (!entry || termNames[id] || manualNameTouched.has(id) || autoNameAttempts.has(id)) return
   if (entry.log?.some((event) => event?.kind === 'you')) return
-  const fallback = fallbackTerminalName(text)
-  if (!fallback) return
-  autoNameAttempts.add(id)
-  applyAutoTerminalName(id, fallback)
   const task = modelTerminalNameInput(text)
-  if (task) void requestManagedTerminalName(id, task)
+  if (!task) return
+  autoNameAttempts.add(id)
+  void requestManagedTerminalName(id, task)
 }
 
 // (cross-person grantee yield removed 2026-07-06 — owner-only serving now; the owner's

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "thinkpool-pair",
-  "version": "0.7.346",
+  "version": "0.7.348",
   "description": "Connect Claude Code, Codex, or Hermes on your computer to a Thinkpool Code room.",
   "type": "module",
   "bin": {

package/terminal-name.mjs

@@ -23,6 +23,7 @@ const ISSUE = /\b(?:broken|buggy|crash(?:es|ed|ing)?|duplicate|error|fail(?:s|ed
 const REQUEST = /^(?:please\s+)?(?:can|could|would|will)\s+(?:we|you)\b|^(?:please\s+)?(?:how about|let's|let us|we need to|i want you to)\b/i
 const SECRET = /(?<![\p{L}\p{N}])(?:sk-(?:proj-)?[a-z0-9_-]{8,}|gsk_[a-z0-9_-]{8,}|xox[baprs]-[a-z0-9_-]{8,}|gh[pousr]_[a-z0-9_-]{8,}|glpat-[a-z0-9_-]{8,}|npm_[a-z0-9]{24,}|(?:sk|rk)_(?:live|test)_[a-z0-9]{8,}|whsec_[a-z0-9]{8,}|AIza[a-z0-9_-]{8,}|AKIA[A-Z0-9]{12,}|bearer\s+[a-z0-9._-]{8,}|eyJ[a-z0-9_-]{8,}\.[a-z0-9_-]{8,}\.[a-z0-9_-]{8,})(?![\p{L}\p{N}])/giu
 const HANDOFF_MARKER = /^---\s*(?:the person's next message|current person request(?:\s*\([^\n]*\))?|side task)\s*---\s*$/gim
+const CURRENT_REQUEST_MARKER = /^(?:#{1,6}\s*)?CURRENT PERSON REQUEST(?:\s*\([^\n]*\))?\s*:?\s*/gim
 const QUESTION_WORDS = new Set(['how', 'what', 'when', 'where', 'which', 'who', 'whom', 'whose', 'why'])
 const PREFERENCE_WORDS = new Set(['avoid', 'choose', 'include', 'need', 'use', 'want'])
 const PREFERENCE_OBJECT_WORDS = new Set(['avoid', 'choose', 'include', 'use'])
@@ -330,11 +331,20 @@ const truncateUtf8 = (value, maxBytes = MODEL_INPUT_MAX_BYTES) => {
   return result.trim()
 }
 
-// The managed namer never receives the raw room prompt. Reuse the local
-// extractor's authoritative-task selection, secret redaction, and host-reference
-// stripping, then add a byte ceiling so multilingual text stays cost-bounded.
+const managedTaskText = (value) => {
+  let text = stripPromptNoise(value)
+  if (!text) return null
+  const currentRequests = [...text.matchAll(CURRENT_REQUEST_MARKER)]
+  const currentRequest = currentRequests.at(-1)
+  if (currentRequest) text = text.slice(currentRequest.index + currentRequest[0].length)
+  return (explicitIntent(text) || text).trim() || null
+}
+
+// The managed namer never receives room/system envelopes, secrets, URLs, or
+// host paths. Preserve every clause of the current task so the model—not the
+// brittle local title heuristic—chooses the primary outcome and constraints.
 export function modelTerminalNameInput(text) {
-  const body = withoutHostReferences(withoutSecrets(extractTerminalTask(text) || ''))
+  const body = withoutHostReferences(withoutSecrets(managedTaskText(text) || ''))
     .replace(/[`*_>#()[\]{}]/g, ' ')
     .replace(/\s+/g, ' ')
     .trim()