@billjr99/pi-openai-compat 1.1.16 → 1.1.17

package/README.md

@@ -59,23 +59,25 @@ If pi is already running when you install, type `/reload` first.
 | **llmproxy** | `http://localhost:8080/v1` (editable) | Keyless by default; bearer token if your instance requires one |
 | **Custom** | Any URL you supply | Optional bearer token |
 
-> **llmproxy and the `__` model-id rewrite.**
-> [llmproxy](https://github.com/BillJr99/llmproxy) advertises model ids in the
-> `provider__model` form (e.g. `openrouter__gpt-4`) and exposes virtual models
-> such as `llmproxy__free` and `llmproxy__loadbalanced`. pi rejects model ids that
-> contain `__`, so without special handling **every llmproxy model is silently
-> dropped from `/model`**. The llmproxy template therefore sets
-> `rewriteDoubleUnderscore: true`: the extension rewrites the first `__` of each id
-> to `/` (e.g. `openrouter__gpt-4` → `openrouter/gpt-4`, `llmproxy__free` →
-> `llmproxy/free`) before registering with pi. llmproxy canonicalizes that slash
-> form back to `__` on each request, so routing still works.
+> **The `__` → `/` model-id rewrite (display compatibility).**
+> Some providers advertise model ids in a `provider__model` form. For example,
+> [llmproxy](https://github.com/BillJr99/llmproxy) exposes ids like
+> `openrouter__gpt-4` and virtual models such as `llmproxy__free` and
+> `llmproxy__loadbalanced`. The extension can optionally rewrite the first `__`
+> of each id to `/` (e.g. `openrouter__gpt-4` → `openrouter/gpt-4`,
+> `llmproxy__free` → `llmproxy/free`) before registering with pi, so the ids
+> display in the more familiar `provider/model` slash form. Upstreams that use
+> the `__` convention canonicalize the slash form back to `__` on each request,
+> so routing still works.
 >
-> The flag is a per-provider config field (`rewriteDoubleUnderscore`, default
-> **false**) and is enabled automatically only for the llmproxy template — other
-> providers are unaffected. If you added llmproxy as a **Custom** endpoint instead
-> of via the llmproxy template, set `"rewriteDoubleUnderscore": true` on that
-> provider in `~/.config/pi-openai-compat/config.json` (or re-run `/compat-login`
-> and pick **llmproxy (local)**).
+> This is a presentation-only convenience controlled by a per-provider config
+> field (`rewriteDoubleUnderscore`, default **false**). It is enabled
+> automatically for the llmproxy template purely for display consistency — other
+> providers are unaffected, and you can turn it on or off freely. If you added a
+> provider as a **Custom** endpoint and want the slash-form display, set
+> `"rewriteDoubleUnderscore": true` on that provider in
+> `~/.config/pi-openai-compat/config.json` (or re-run `/compat-login` and pick a
+> template that enables it). Leaving it off keeps the original `__` ids.
 
 > **Providers whose model catalog lives at a non-standard `/models` path (as of June 2026)**
 > Some providers don't return models at `<base_url>/models`. The extension
@@ -347,13 +349,13 @@ For OpenRouter: some keys are restricted to free-tier models only.
 For NIM: confirm your account has inference access enabled.
 
 **`/compat-login` reports N models but far fewer appear in `/model`**
-This is the classic llmproxy symptom: pi drops every model id containing `__`, so
-llmproxy's `provider__model` ids (and `llmproxy__free` / `llmproxy__loadbalanced`)
-never show. Make sure you logged in via the **llmproxy (local)** template (it sets
-`rewriteDoubleUnderscore: true` automatically). If you used **Custom**, add
-`"rewriteDoubleUnderscore": true` to that provider in
-`~/.config/pi-openai-compat/config.json` and re-run `/compat-refresh`. After the
-fix the ids appear in slash form (`openrouter/gpt-4`, `llmproxy/free`).
+The usual cause is a **scoped or restricted API key/account** — the catalog
+endpoint advertises the full model list, but your credentials are only entitled
+to a subset, so pi only surfaces the ones you can actually use. Check whether your
+key is limited to specific models or a free tier on the provider's dashboard.
+(This is unrelated to the `__` → `/` rewrite, which is a display-only convenience —
+see the note in **Supported providers** above. Toggling `rewriteDoubleUnderscore`
+only changes how ids are displayed, not how many appear.)
 
 **Models appear in `/model` but requests fail**
 Check `/compat-login` ran successfully (no error message).

package/index.ts

@@ -49,11 +49,11 @@ interface ProviderConfig {
   // only once rather than on every session_start while the config stays stale.
   staleNotified?: boolean;
   // When true, rewrite the FIRST "__" of every model id to "/" before
-  // registering with pi. pi rejects model ids containing "__", so providers like
-  // llmproxy (which uses "provider__model" ids) would otherwise have their entire
-  // catalog dropped from /model. The rewrite is the inverse of llmproxy's own
-  // request-side canonicalization, so requests still round-trip. Defaults false;
-  // the wizard turns it on only for the llmproxy template.
+  // registering with pi, so ids from providers that use a "provider__model"
+  // convention (e.g. llmproxy) display in the more familiar "provider/model"
+  // slash form. This is a display-only convenience: such upstreams canonicalize
+  // the slash form back to "__" on each request, so requests still round-trip.
+  // Defaults false; the wizard turns it on only for the llmproxy template.
   rewriteDoubleUnderscore?: boolean;
 }
 
@@ -105,9 +105,10 @@ const TEMPLATES: Record<string, {
   /** Keep only models whose task.name matches this string (case-insensitive). */
   modelsKeepTask?: string;
   /**
-   * Rewrite the first "__" of each model id to "/" before registering with pi.
-   * pi drops model ids containing "__", so this is required for llmproxy (whose
-   * ids are "provider__model"). Default false; set true only where pi needs it.
+   * Rewrite the first "__" of each model id to "/" before registering with pi,
+   * for display compatibility — ids from "provider__model" upstreams (e.g.
+   * llmproxy) then show in the familiar "provider/model" slash form. Display-only;
+   * default false; set true where you want the slash-form presentation.
    */
   rewriteDoubleUnderscore?: boolean;
 }> = {
@@ -302,8 +303,8 @@ const TEMPLATES: Record<string, {
     keyless: true,
     promptUrl: true,
     // llmproxy advertises "provider__model" ids (and virtuals like
-    // "llmproxy__free"). pi rejects "__" in model ids, so rewrite the first
-    // "__" to "/" here; llmproxy canonicalizes the slash form back on requests.
+    // "llmproxy__free"). Rewrite the first "__" to "/" here so they display in
+    // the familiar slash form; llmproxy canonicalizes it back on requests.
     rewriteDoubleUnderscore: true,
   },
   vercel: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@billjr99/pi-openai-compat",
-  "version": "1.1.16",
+  "version": "1.1.17",
   "description": "pi-coding-agent extension: OpenAI-compatible endpoint support (OpenRouter, NVIDIA NIM, Nous Portal, Ollama, custom)",
   "author": "Bill Mongan <https://github.com/BillJr99>",
   "license": "MIT",