@1presence/bridge 0.72.0 → 0.74.0

package/dist/claudeAuth.js

@@ -1,12 +1,5 @@
 import { spawn } from 'node:child_process';
 import { createInterface } from 'node:readline';
-/**
- * Parse the stdout of `claude auth status --json`. Extracts the first JSON object
- * (defensive against any banner/log noise around it). Returns null when there is
- * no parseable object or no boolean `loggedIn` field — i.e. an output shape we
- * don't recognise, which the caller treats as "unknown, don't block".
- * Exported for unit testing.
- */
 export function parseAuthStatus(stdout) {
     const start = stdout.indexOf('{');
     const end = stdout.lastIndexOf('}');
@@ -28,14 +21,6 @@ export function parseAuthStatus(stdout) {
         return null;
     }
 }
-/**
- * Ask the local `claude` CLI whether it's signed in via `claude auth status
- * --json`. Zero-cost — no model turn runs. Resolves the parsed status, or null
- * when we can't determine it (CLI not installed, too old for `auth status`,
- * errored, or timed out). null means "unknown": a probe we can't trust must
- * never gate startup — the first turn's error path still guides the user if auth
- * is genuinely broken.
- */
 export function probeClaudeAuth(timeoutMs = 8000) {
     return new Promise((resolve) => {
         let settled = false;
@@ -54,21 +39,13 @@ export function probeClaudeAuth(timeoutMs = 8000) {
         const timer = setTimeout(() => { try {
             proc.kill('SIGKILL');
         }
-        catch { /* already gone */ } finish(null); }, timeoutMs);
+        catch { } finish(null); }, timeoutMs);
         let out = '';
         proc.stdout?.on('data', (c) => { out += c.toString('utf-8'); });
         proc.on('error', () => { clearTimeout(timer); finish(null); });
         proc.on('close', () => { clearTimeout(timer); finish(parseAuthStatus(out)); });
     });
 }
-/**
- * Launch the local Claude Code subscription sign-in (`claude auth login
- * --claudeai`) with inherited stdio, so the user completes the browser / paste
- * flow right here. Resolves true when the child exits (any exit code — the caller
- * re-probes to learn the outcome), or false if it can't be spawned at all (CLI
- * missing / too old for `auth login`), in which case the poll below still carries
- * a sign-in the user does in another terminal.
- */
 export function launchClaudeLogin() {
     return new Promise((resolve) => {
         let proc;
@@ -84,16 +61,6 @@ export function launchClaudeLogin() {
     });
 }
 const sleep = (ms) => new Promise((r) => setTimeout(r, ms));
-/**
- * Poll `claude auth status` until it reports signed-in, or the timeout elapses.
- * This is also what catches a sign-in the user completes in ANOTHER terminal
- * window: on macOS the credential lives in the Keychain, which no process can
- * watch, so polling is the only cross-platform detector. The Agent SDK re-reads
- * credentials on every query(), so once this returns true the very next turn
- * works with no bridge restart. `onWaiting` fires once, the first time we find
- * the user still isn't signed in (so the caller can print a waiting line without
- * spamming it). Returns true if signed in, false on timeout.
- */
 export async function waitForClaudeLogin(opts = {}) {
     const intervalMs = opts.intervalMs ?? 3000;
     const timeoutMs = opts.timeoutMs ?? 5 * 60 * 1000;
@@ -112,10 +79,6 @@ export async function waitForClaudeLogin(opts = {}) {
         await sleep(intervalMs);
     }
 }
-/**
- * Minimal Y/n prompt. Returns the boolean answer; empty input returns `def`.
- * Caller is responsible for only invoking this when stdin is a TTY.
- */
 export function promptYesNo(question, def) {
     return new Promise((resolve) => {
         const rl = createInterface({ input: process.stdin, output: process.stdout });

package/dist/config.js

@@ -1,19 +1,6 @@
 import { emitKeypressEvents } from 'readline';
 import { spawn } from 'child_process';
-// ─── In-memory model choice ───────────────────────────────────────────────────
-//
-// The bridge prompts for a model on every interactive startup. The choice is
-// kept in memory for the life of the process — nothing is written to disk.
-// In a non-TTY environment the prompt is skipped and Claude Code's own default
-// is used.
 let selectedModel = null;
-// ─── Default-model probe ──────────────────────────────────────────────────────
-/**
- * Asks the local `claude` CLI which model it would pick by default, by reading
- * the `model` field of the `system/init` stream-json event and killing the
- * process before any model call completes. Returns null on timeout or if the
- * CLI isn't installed — we never want this auxiliary lookup to block startup.
- */
 function detectClaudeDefaultModel() {
     return new Promise((resolve) => {
         let settled = false;
@@ -64,34 +51,12 @@ function detectClaudeDefaultModel() {
                         return;
                     }
                 }
-                catch { /* ignore non-JSON lines */ }
+                catch { }
             }
         });
-        // Send one empty user message so claude proceeds past startup and emits
-        // the init event. We kill before any assistant turn runs, so no tokens
-        // are billed.
         proc.stdin?.end('{"type":"user","message":{"role":"user","content":""}}\n');
     });
 }
-// The fixed menu — keep the option count small so the timeout default is easy
-// to glance at. Option 1's `model: null` means "let Claude Code pick" (no
-// `--model` flag passed to the subprocess). This is the DYNAMIC default: it
-// tracks whatever model Claude Code actually serves for the user's plan (the
-// probe above surfaces it as a live suffix — on most accounts today that's the
-// 1M-context Opus) and self-heals if a pinned id is ever unservable. The
-// remaining rows pin explicit ids for users who want a specific model.
-// `num` must stay contiguous 1..N in array order — the jump-key handler maps a
-// typed digit `n` to `idx = n - 1`.
-//
-// NB there is no way to *enumerate* the models available on subscription auth:
-// the `/v1/models` API needs an API key (the bridge strips ANTHROPIC_API_KEY —
-// see claude.ts) and the CLI exposes only the single default via the init event.
-// So this list is hand-maintained — add/remove ids here as the roster changes.
-// (We dropped the standalone `claude-opus-4-8[1m]` row: it duplicated option 1,
-// whose Claude Code default already resolves to the 1M Opus on subscription, and
-// pinning `[1m]` risked a hard failure where the null default degrades gracefully.
-// An earlier every-turn `400 / "unknown"` was once blamed on `[1m]` but was
-// actually a tool-schema bug that hit every model — see vault/Bugs.md.)
 const MODEL_OPTIONS = [
     { num: 1, model: null, label: 'Use Claude Code default' },
     { num: 2, model: 'claude-opus-5', label: 'claude-opus-5' },
@@ -103,10 +68,6 @@ const MODEL_OPTIONS = [
     { num: 8, model: 'claude-haiku-4-5', label: 'claude-haiku-4-5' },
 ];
 const PROMPT_TIMEOUT_MS = 10_000;
-// Default to option 1 ("Use Claude Code default") — the dynamic choice that
-// follows whatever Claude Code serves for the user's plan (currently the 1M
-// Opus on most accounts) and degrades gracefully rather than pinning a possibly
-// unservable id. Users can arrow to a pinned model if they want one.
 const DEFAULT_OPTION_NUM = 1;
 function promptForModel(defaultModel) {
     return new Promise((resolve) => {
@@ -187,11 +148,6 @@ function promptForModel(defaultModel) {
         }, PROMPT_TIMEOUT_MS);
     });
 }
-/**
- * Asks the user which model to use for this bridge session. The choice lives
- * in memory only — every startup re-prompts. In a non-TTY environment the
- * prompt is skipped and Claude Code's own default is used.
- */
 export async function ensureModelChoice() {
     if (!process.stdin.isTTY) {
         selectedModel = null;
@@ -206,7 +162,6 @@ export async function ensureModelChoice() {
         console.log(`\nUsing your Claude Code default${defaultModel ? ` (${defaultModel})` : ''}.\n`);
     }
 }
-/** Returns the model id chosen for this session, or null to defer to Claude Code's own default. */
 export function getBridgeModel() {
     return selectedModel;
 }