@haoyiyin/9router 0.1.5 → 0.1.6

package/README.md

@@ -18,15 +18,15 @@ Environment variables are supported as a fallback: `ROUTER_API_BASE` / `ROUTER_A
 
 ## Combo models & thinking
 
-Combo models (`sub` / `free` / `main`, `owned_by: "combo"`) advertise no capabilities, so reasoning is off by default. If every upstream in a combo route supports thinking, opt in via `comboReasoning` in `~/.pi/agent/haoyiyin/9router.json`:
+Combo models (`sub` / `free` / `main`, `owned_by: "combo"`) advertise no capabilities from the router, so their reasoning is decided by `comboReasoning` in `~/.pi/agent/haoyiyin/9router.json`. It defaults to **on** (assumes combo routes hit thinking-capable upstreams) with the standard `off..high` range. Opt out per machine with:
 
 ```json
 {
-  "comboReasoning": true
+  "comboReasoning": false
 }
 ```
 
-`true` enables the default `off..high` range. To unlock `max`/`xhigh`, pass a custom `thinkingLevelMap` (keys: `off`, `minimal`, `low`, `medium`, `high`, `xhigh`, `max`; values: what gets sent to the router, `null` hides the level):
+To unlock `max`/`xhigh`, pass a custom `thinkingLevelMap` (keys: `off`, `minimal`, `low`, `medium`, `high`, `xhigh`, `max`; values: what gets sent to the router, `null` hides the level):
 
 ```json
 {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@haoyiyin/9router",
-  "version": "0.1.5",
+  "version": "0.1.6",
   "type": "module",
   "main": "src/index.ts",
   "description": "Pi extension — 9Router provider (OpenCode Free oc/-free models + router models) + fetch/search/transcribe tools",

package/src/config.ts

@@ -8,9 +8,10 @@ export interface RouterConfig {
   baseUrl: string;
   apiKey: string;
   /**
-   * Opt-in to force reasoning:true onto combo models (`owned_by === "combo"`).
-   * Combo models carry no capabilities from the router, so reasoning is normally
-   * off. Set `true` for the default off..high range, or pass a custom
+   * Force reasoning:true onto combo models (`owned_by === "combo"`).
+   * Combo models carry no capabilities from the router, so this switch decides
+   * their reasoning status. Defaults to `true` (combo routes are assumed to hit
+   * thinking-capable upstreams); set `false` to opt out, or pass a custom
    * `thinkingLevelMap` to unlock max/xhigh on the combo routes.
    */
   comboReasoning?: boolean | { thinkingLevelMap?: Record<string, string | null> };
@@ -49,7 +50,7 @@ export function loadConfig(): RouterConfig {
 
   const apiKey = fileKey ?? process.env.ROUTER_API_KEY ?? "";
 
-  return { baseUrl, apiKey, comboReasoning };
+  return { baseUrl, apiKey, comboReasoning: comboReasoning ?? true };
 }
 
 export function saveConfig(cfg: RouterConfig): void {

package/src/provider.ts

@@ -85,8 +85,8 @@ function getThinkingFormat(model: RouterModel): "openai" | "deepseek" {
 /**
  * Derive pi's thinkingLevelMap from the router's advertised capabilities.
  * DeepSeek exposes low/high/max (mirrors pi's own deepseek catalog); openai-style
- * models keep the default off..high range, only hiding `off` when thinking cannot
- * be disabled. No map is returned when the backend does not declare reasoning.
+ * models expose off..max, hiding `off` when thinking cannot be disabled. No map
+ * is returned when the backend does not declare reasoning.
  */
 export function getThinkingLevelMap(
 	model: RouterModel,
@@ -96,15 +96,19 @@ export function getThinkingLevelMap(
 	if (cap.thinkingFormat === "deepseek") {
 		return { minimal: null, low: "low", medium: null, high: "high", max: "max" };
 	}
-	if (cap.thinkingCanDisable === false) return { off: null };
-	return undefined;
+	// openai-style: reasoning_effort takes the level name directly. Unlock
+	// xhigh/max, and hide off when thinking cannot be disabled.
+	return {
+		...(cap.thinkingCanDisable === false ? { off: null } : {}),
+		xhigh: "xhigh",
+		max: "max",
+	};
 }
 
 /**
  * Combo models (`owned_by === "combo"`) carry no capabilities from the router, so
- * reasoning is normally off. Operators who guarantee their combo routes only hit
- * thinking-capable upstreams can opt in via `comboReasoning` in 9router.json:
- * `true` for the default off..high range, or an object with a custom
+ * their reasoning is decided by `comboReasoning` in 9router.json: default `true`
+ * (off..high range), `false` disables, or an object with a custom
  * `thinkingLevelMap` to unlock max/xhigh.
  */
 export function resolveComboOptIn(