github-router 0.3.33 → 0.3.35

package/dist/browser-bridge/index.js

@@ -3613,6 +3613,32 @@ var import_subprotocol = /* @__PURE__ */ __toESM(require_subprotocol(), 1);
 var import_websocket = /* @__PURE__ */ __toESM(require_websocket(), 1);
 var import_websocket_server = /* @__PURE__ */ __toESM(require_websocket_server(), 1);
 
+//#endregion
+//#region src/lib/browser-mcp/bridge-paths.ts
+/**
+* Filesystem path where the browser bridge writes its discovery file
+* (`{pid, port, token, startedAt}`) and where the install-check reads
+* it from. Used by `src/browser-bridge/index.ts` (writer) AND by
+* `src/lib/browser-mcp/install-check.ts` (reader).
+*
+* MUST be a single computation. Historically the bridge special-cased
+* win32 to `%LOCALAPPDATA%\github-router` while the install-check used
+* `~/.local/share/github-router` (the canonical `PATHS.APP_DIR` from
+* `src/lib/paths.ts`, which has no win32 branch). On Windows the
+* writer and reader never met, so the install-check returned
+* `bridge_not_running` even with a healthy bridge. Centralized here so
+* the regression test in `tests/browser-bridge-discovery-path.test.ts`
+* can pin the round-trip.
+*
+* Mirrors `PATHS.APP_DIR` from `src/lib/paths.ts` (XDG-style on every
+* platform). The bridge bundle pulls this file in via tsdown's
+* relative-import bundling; no runtime dependency on `src/lib/paths.ts`
+* is introduced.
+*/
+function discoveryPath() {
+	return path.join(homedir(), ".local", "share", "github-router", "browser-mcp", "bridge.json");
+}
+
 //#endregion
 //#region src/browser-bridge/index.ts
 try {
@@ -3620,17 +3646,6 @@ try {
 } catch {}
 const HEARTBEAT_MS = 5e3;
 const HEARTBEAT_MISS_LIMIT = 3;
-function appDir() {
-	if (platform() === "win32") {
-		const local = process$1.env.LOCALAPPDATA;
-		if (local) return path.join(local, "github-router");
-		return path.join(homedir(), "AppData", "Local", "github-router");
-	}
-	return path.join(homedir(), ".local", "share", "github-router");
-}
-function discoveryPath() {
-	return path.join(appDir(), "browser-mcp", "bridge.json");
-}
 function writeDiscoveryFile(payload) {
 	const file = discoveryPath();
 	mkdirSync(path.dirname(file), { recursive: true });

package/dist/main.js

@@ -929,31 +929,15 @@ async function autoUpdateClaude(latestVersion) {
 //#endregion
 //#region src/lib/port.ts
 const DEFAULT_PORT = 8787;
-/**
-* Default model for `github-router claude`. The Anthropic-published dashed
-* slug (`claude-opus-4-7`) — NOT the Copilot-internal slug
-* (`claude-opus-4.7-1m-internal`) — because Claude Code 2.1.126's `/model`
-* UI is backed by a hardcoded registry of Anthropic slugs, and an
-* unrecognized slug causes the menu to highlight "Opus 4" with a
-* "Newer version available" hint instead of "Opus 4.7 (1M context)".
-*
-* The proxy's `resolveModel` (`src/lib/utils.ts`) translates this to
-* Copilot's `claude-opus-4.7-1m-internal` (enterprise) or
-* `claude-opus-4.7` (Pro+/Business/Max) at request time via the
-* family-preference + version-match branch — round-trip covered by
-* `tests/lib-utils.test.ts:154`.
-*
-* `DEFAULT_CLAUDE_MODEL_FALLBACKS` covers major.minor regressions only;
-* 1M↔200K downgrade is handled inside the resolver, so we don't need
-* separate `-1m` entries here.
-*/
-const DEFAULT_CLAUDE_MODEL = "claude-opus-4-7";
 const DEFAULT_CLAUDE_MODEL_FALLBACKS = ["claude-opus-4-6", "claude-opus-4-5"];
 /**
 * Cap-aware default picker for `ANTHROPIC_MODEL` on the implicit-default
-* path. Returns `claude-opus-4-7[1m]` when the live Copilot catalog
-* contains a `*-opus-4.7-1m*` variant (enterprise tier), else
-* `DEFAULT_CLAUDE_MODEL` (the bare slug).
+* path. Returns `claude-opus-${family}[1m]` when the live Copilot catalog
+* contains an `opus-${family}-1m*` variant (enterprise tier), else the
+* bare `claude-opus-${family}` slug. `family` defaults to `"4.7"` so the
+* no-arg call preserves the original behavior; explicit values like
+* `"4.6"` or `"4.8"` are used to honor the `github-router claude
+* -m <version>` family shorthand.
 *
 * The `[1m]` literal-bracket suffix is Claude Code's local 1M-context
 * unlock — cc-backup `src/utils/context.ts:35-40` matches `/\[1m\]/i`
@@ -980,12 +964,21 @@ const DEFAULT_CLAUDE_MODEL_FALLBACKS = ["claude-opus-4-6", "claude-opus-4-5"];
 * can't tell the difference between "no catalog yet" and "no 1M
 * variant" — defaulting safe-side preserves the pre-change behavior).
 */
-function pickClaudeDefault() {
-	if (state.models?.data.some((m) => /opus-4[.-]7-1m(?:$|-)/i.test(m.id)) ?? false) {
-		consola.info(`Catalog contains opus-4.7-1m variant; defaulting ANTHROPIC_MODEL to "${DEFAULT_CLAUDE_MODEL}[1m]" so Claude Code accounts for 1M context locally. Set CLAUDE_CODE_DISABLE_1M_CONTEXT=1 to opt out (HIPAA), or pass --model ${DEFAULT_CLAUDE_MODEL} to pin 200K.`);
-		return `${DEFAULT_CLAUDE_MODEL}[1m]`;
+const DEFAULT_OPUS_FAMILY = "4.7";
+function pickClaudeDefault(opusFamily = DEFAULT_OPUS_FAMILY) {
+	const dotted = opusFamily.replace(/-/g, ".");
+	const bareSlug = `claude-opus-${dotted.replace(/\./g, "-")}`;
+	const versionPattern = dotted.replace(/\./g, "[.-]");
+	const oneMRegex = new RegExp(`opus-${versionPattern}-1m(?:$|-)`, "i");
+	const familyRegex = new RegExp(`opus-${versionPattern}(?:$|[-.])`, "i");
+	const models = state.models?.data ?? [];
+	const has1m = models.some((m) => oneMRegex.test(m.id));
+	if (opusFamily !== DEFAULT_OPUS_FAMILY && state.models && models.length > 0 && !models.some((m) => familyRegex.test(m.id))) consola.warn(`Requested Opus family "${dotted}" not found in Copilot catalog; using "${bareSlug}" anyway (resolveModel may not find a backend for it).`);
+	if (has1m) {
+		consola.info(`Catalog contains opus-${dotted}-1m variant; defaulting ANTHROPIC_MODEL to "${bareSlug}[1m]" so Claude Code accounts for 1M context locally. Set CLAUDE_CODE_DISABLE_1M_CONTEXT=1 to opt out (HIPAA), or pass --model ${bareSlug} to pin 200K.`);
+		return `${bareSlug}[1m]`;
 	}
-	return DEFAULT_CLAUDE_MODEL;
+	return bareSlug;
 }
 /**
 * Default model for `github-router codex`. `gpt-5.5` is the new flagship
@@ -2587,6 +2580,32 @@ function hasSupportedBrowserInstalled() {
 	return detectSupportedBrowsers().length > 0;
 }
 
+//#endregion
+//#region src/lib/browser-mcp/bridge-paths.ts
+/**
+* Filesystem path where the browser bridge writes its discovery file
+* (`{pid, port, token, startedAt}`) and where the install-check reads
+* it from. Used by `src/browser-bridge/index.ts` (writer) AND by
+* `src/lib/browser-mcp/install-check.ts` (reader).
+*
+* MUST be a single computation. Historically the bridge special-cased
+* win32 to `%LOCALAPPDATA%\github-router` while the install-check used
+* `~/.local/share/github-router` (the canonical `PATHS.APP_DIR` from
+* `src/lib/paths.ts`, which has no win32 branch). On Windows the
+* writer and reader never met, so the install-check returned
+* `bridge_not_running` even with a healthy bridge. Centralized here so
+* the regression test in `tests/browser-bridge-discovery-path.test.ts`
+* can pin the round-trip.
+*
+* Mirrors `PATHS.APP_DIR` from `src/lib/paths.ts` (XDG-style on every
+* platform). The bridge bundle pulls this file in via tsdown's
+* relative-import bundling; no runtime dependency on `src/lib/paths.ts`
+* is introduced.
+*/
+function discoveryPath() {
+	return path.join(homedir(), ".local", "share", "github-router", "browser-mcp", "bridge.json");
+}
+
 //#endregion
 //#region src/lib/browser-mcp/native-host-installer.ts
 const NMH_HOST_ID = "com.githubrouter.browser";
@@ -2815,9 +2834,6 @@ function installNativeHostForAll(browsers) {
 
 //#endregion
 //#region src/lib/browser-mcp/install-check.ts
-function discoveryPath() {
-	return path.join(PATHS.APP_DIR, "browser-mcp", "bridge.json");
-}
 function readBridgeDiscovery() {
 	try {
 		const raw = readFileSync(discoveryPath(), "utf8");
@@ -10879,7 +10895,7 @@ function initProxyFromEnv() {
 //#endregion
 //#region package.json
 var name = "github-router";
-var version = "0.3.33";
+var version = "0.3.35";
 
 //#endregion
 //#region src/lib/approval.ts
@@ -13000,7 +13016,7 @@ const claude = defineCommand({
 		model: {
 			alias: "m",
 			type: "string",
-			description: "Override the default model for Claude Code"
+			description: "Override the default model for Claude Code. Accepts a full slug (e.g. claude-opus-4-7) or an Opus family shorthand (e.g. 4.7, 4.8, 4.6) which expands to the best variant for that family — adding the [1m] suffix when a 1M-context backend is in the catalog."
 		},
 		"codex-mcp": {
 			type: "boolean",
@@ -13079,7 +13095,8 @@ const claude = defineCommand({
 		}
 		enableFileLogging();
 		const usingDefault = !args.model;
-		let chosenSlug = args.model ?? pickClaudeDefault();
+		const opusFamilyShorthand = args.model?.match(/^(\d+\.\d+)$/)?.[1];
+		let chosenSlug = opusFamilyShorthand ? pickClaudeDefault(opusFamilyShorthand) : args.model ?? pickClaudeDefault();
 		let resolvedSlug = resolveModel(chosenSlug);
 		if (usingDefault && state.models) {
 			const inCache = (slug) => state.models?.data.some((m) => m.id === resolveModel(slug)) ?? false;