@lawaty/lacode 0.2.0 → 0.2.1

package/README.md

@@ -352,7 +352,7 @@ as an opencode provider for you. It is **off by default**: if you do not set
 | `enabled` | `false` | Turns the integration on. |
 | `baseURL` | `http://localhost:20128/v1` | Gateway base URL, normalised to exactly one `/v1`. |
 | `apiKey` | — | Bearer key. **`OMNIROUTE_API_KEY` overrides this value.** The key is never logged. |
-| `combo` | — | An OmniRoute combo (or `auto`) used as the routing model. Always exposed first. |
+| `combo` | — | An OmniRoute combo id (or `auto`) to expose as a selectable model even though the gateway does not list it. Exposed first. |
 | `models` | — | A **hard allowlist**: only these ids are ever exposed, even if the gateway lists more. |
 | `tiers` | — | `{ "primary": [...], "backup": [...] }` preference order over models the allowlist already permits. |
 | `stickyPerSession` | `true` | Keep a session on one model while that model is healthy. |
@@ -379,18 +379,25 @@ model while it is cooling after a transient failure — 429, 408 and 5xx, honour
 `Retry-After` and otherwise using capped exponential backoff. Non-transient
 errors (400, 401, 403, 404) are surfaced to you, never failed over.
 
-**OmniRoute becomes the pool's primary slot.** This is the point of the
-integration: when the gateway is healthy, LaCode replaces the pool's **first
-slot** — the one the pool takes first — with
-`omniroute/<combo | first allowlisted model | auto>`, inheriting that slot's
-weight, so pooled mechanical work (exploration, implementation, cartography)
-drains through OmniRoute before it reaches the bundled opencode/opencode-go
-("zen") models. On the default `pool.txt` shown below, gemini's slot becomes
-OmniRoute — same slot count, same weights — and bunny / step-5 / big-pickle keep
-their places behind it. The replacement is skipped, leaving the pool exactly as
-it was, if the gateway is disabled, unreachable, returns no usable models, or if
-your own pool already routes on the `omniroute` provider (LaCode never stacks a
-second slot on a provider you already chose). See
+**Using OmniRoute in the pool.** The integration only *registers* the provider —
+it never touches your pool, and it does not place a slot for you. To route pooled
+work (exploration, implementation, cartography) through the gateway, list it in
+your pool like any other model:
+
+```
+# ~/.config/lacode/pool.txt
+omniroute/auto             3
+opencode/space-bunny-free  2
+opencode-go/step-5-preview-free 1
+opencode/big-pickle        1
+```
+
+`omniroute/auto` then takes slot 1 (weight 3) and drains pooled mechanical work
+before it reaches the bundled opencode/opencode-go ("zen") models. Any
+`omniroute/<id>` works, provided the id is exposed — by `combo`/`models`, or the
+whole catalog when neither is set. Because the plugin writes the pool nothing,
+the only thing that changes when the gateway is down is that `omniroute/<id>`
+fails; every other slot keeps working. See
 [docs/USER-STORIES.md](docs/USER-STORIES.md) US-28.
 
 **A note on mid-session switching.** opencode 1.18.34 exposes no hook to change
@@ -413,7 +420,7 @@ local instance with `REQUIRE_API_KEY=false`, the literal placeholder
 | `401` / `Invalid API key` | `REQUIRE_API_KEY=true` and the key is missing or wrong | Create a key in the dashboard, export `OMNIROUTE_API_KEY`, or set `REQUIRE_API_KEY=false` locally and use `sk_omniroute`. |
 | `404` on a URL containing `/v1/v1/` | A `baseURL` that already carried `/v1` was given twice | LaCode normalises this, so it should not come from LaCode; check a hand-written `provider.omniroute` block, which LaCode leaves untouched. |
 | Provider registered but models are missing | You already had `provider.omniroute` | Remove it and let LaCode write it, or add the models yourself — LaCode never modifies an existing provider. |
-| Pooled work still runs on the bundled models | The gateway was unhealthy/disabled at startup, or your own pool already routes on `omniroute` | Check the startup log for `omniroute health check failed` / `no usable models`; if you added your own `omniroute/…` pool slot, remove it to let LaCode place one. |
+| Pooled work still runs on the bundled models | The gateway was unhealthy/disabled at startup, or you have no `omniroute/…` slot in your pool | Check the startup log for `omniroute health check failed` / `no usable models`, then add `omniroute/<model>` to `pool.txt` (or `pool.json`) to route pooled work through the gateway. |
 
 ### Which models run which agent
 

package/dist/index.js

@@ -250,51 +250,19 @@ function state(opts = {}) {
       dir,
       modelsFile,
       inlineModels: models,
-      primarySlot: null,
       origin: "unresolved",
       slots: []
     };
   }
   return g[key];
 }
-function replacePrimarySlot(base, primary) {
-  const problems = [];
-  const model = typeof primary?.model === "string" ? primary.model.trim() : "";
-  if (!model.includes("/")) {
-    problems.push(`primary slot: model must be written provider/model-id, got "${model}"`);
-    return { slots: base, problems };
-  }
-  const provider = providerOf(model);
-  if (base.some((slot) => providerOf(slot.model) === provider)) {
-    problems.push(`primary slot ${model}: provider ${provider} is already in the pool; the primary slot is skipped`);
-    return { slots: base, problems };
-  }
-  if (base.length === 0) return { slots: [{ index: 1, model, weight: Math.max(1, primary.weight ?? 1) }], problems };
-  let weight = primary.weight;
-  if (weight !== void 0 && (typeof weight !== "number" || !Number.isFinite(weight) || weight <= 0)) {
-    problems.push(`primary slot ${model}: weight must be a positive number; inheriting the replaced slot's weight`);
-    weight = void 0;
-  }
-  const slots = base.map((slot, i) => i === 0 ? { ...slot, model, weight: weight ?? slot.weight } : slot);
-  return { slots, problems };
-}
-function setPrimarySlot(s, primary) {
-  s.primarySlot = primary;
-  if (s.slots.length > 0) s.slots = [];
-}
 function ensureSlots(s, onWarn) {
   if (s.slots.length > 0) return s.slots;
   const { slots, problems, origin } = resolveModels({ models: s.inlineModels, modelsFile: s.modelsFile });
-  let resolved = slots;
-  if (s.primarySlot !== null) {
-    const replaced = replacePrimarySlot(slots, s.primarySlot);
-    resolved = replaced.slots;
-    problems.push(...replaced.problems);
-  }
-  s.slots = resolved;
+  s.slots = slots;
   s.origin = origin;
   for (const problem of problems) onWarn(problem);
-  return resolved;
+  return slots;
 }
 var claimKey = (sessionID, callID) => `${sessionID}:${callID}`;
 var selfFile = (id) => `claims.${id}.json`;
@@ -2639,7 +2607,6 @@ var omniRouteHooks = async (input, options, deps = {}) => {
   });
   const selector = createSelector(config, { now: deps.now });
   let disabled = false;
-  let poolModel = null;
   const probe = async () => {
     if (config.healthCheck) {
       const healthy = await checkHealth({
@@ -2674,8 +2641,6 @@ var omniRouteHooks = async (input, options, deps = {}) => {
       return;
     }
     selector.setCandidates(candidates);
-    const chosen = config.combo ?? (config.models !== void 0 && config.models.length > 0 ? candidates[0] : "auto");
-    poolModel = `omniroute/${chosen}`;
     await log("info", `omniroute: ${candidates.length} model(s) from ${config.baseURL}: ${candidates.join(", ")}`);
   };
   let started = null;
@@ -2687,10 +2652,8 @@ var omniRouteHooks = async (input, options, deps = {}) => {
       return;
     }
     const exposed = selector.candidates();
-    const target = poolModel !== null ? poolModel.slice("omniroute/".length) : null;
-    const ids = target !== null && !exposed.includes(target) ? [target, ...exposed] : exposed;
     const models = {};
-    for (const id of ids) models[id] = { name: id };
+    for (const id of exposed) models[id] = { name: id };
     providers.omniroute = {
       name: "OmniRoute",
       // The documented opencode registration mechanism for a gateway speaking
@@ -2709,10 +2672,6 @@ var omniRouteHooks = async (input, options, deps = {}) => {
       },
       models
     };
-    if (poolModel !== null) {
-      const ps = state(options ?? {});
-      setPrimarySlot(ps, { model: poolModel });
-    }
   };
   return {
     config: async (cfg) => {

package/dist/pool.js

@@ -250,51 +250,19 @@ function state(opts = {}) {
       dir,
       modelsFile,
       inlineModels: models,
-      primarySlot: null,
       origin: "unresolved",
       slots: []
     };
   }
   return g[key];
 }
-function replacePrimarySlot(base, primary) {
-  const problems = [];
-  const model = typeof primary?.model === "string" ? primary.model.trim() : "";
-  if (!model.includes("/")) {
-    problems.push(`primary slot: model must be written provider/model-id, got "${model}"`);
-    return { slots: base, problems };
-  }
-  const provider = providerOf(model);
-  if (base.some((slot) => providerOf(slot.model) === provider)) {
-    problems.push(`primary slot ${model}: provider ${provider} is already in the pool; the primary slot is skipped`);
-    return { slots: base, problems };
-  }
-  if (base.length === 0) return { slots: [{ index: 1, model, weight: Math.max(1, primary.weight ?? 1) }], problems };
-  let weight = primary.weight;
-  if (weight !== void 0 && (typeof weight !== "number" || !Number.isFinite(weight) || weight <= 0)) {
-    problems.push(`primary slot ${model}: weight must be a positive number; inheriting the replaced slot's weight`);
-    weight = void 0;
-  }
-  const slots = base.map((slot, i) => i === 0 ? { ...slot, model, weight: weight ?? slot.weight } : slot);
-  return { slots, problems };
-}
-function setPrimarySlot(s, primary) {
-  s.primarySlot = primary;
-  if (s.slots.length > 0) s.slots = [];
-}
 function ensureSlots(s, onWarn) {
   if (s.slots.length > 0) return s.slots;
   const { slots, problems, origin } = resolveModels({ models: s.inlineModels, modelsFile: s.modelsFile });
-  let resolved = slots;
-  if (s.primarySlot !== null) {
-    const replaced = replacePrimarySlot(slots, s.primarySlot);
-    resolved = replaced.slots;
-    problems.push(...replaced.problems);
-  }
-  s.slots = resolved;
+  s.slots = slots;
   s.origin = origin;
   for (const problem of problems) onWarn(problem);
-  return resolved;
+  return slots;
 }
 var claimKey = (sessionID, callID) => `${sessionID}:${callID}`;
 var selfFile = (id) => `claims.${id}.json`;
@@ -615,11 +583,9 @@ export {
   refreshCatalog,
   release,
   releaseStuck,
-  replacePrimarySlot,
   resolveModels,
   retryAfterMs,
   selfFile,
-  setPrimarySlot,
   slotFor,
   snapshot,
   splitModel,

package/docs/USER-STORIES.md

@@ -140,16 +140,16 @@ correct exactly when a whitelist exists (it is a real restriction, and the pool
 models would otherwise resolve to nothing) and is not merely unnecessary when one
 does not.
 
-### US-28: A local gateway (OmniRoute) becomes the pool's primary slot
-**As a** user running a local OpenAI-compatible gateway, **I want** LaCode to drain the pool's mechanical work through it first, **so that** the bundled opencode/opencode-go ("zen") models stop carrying that load.
+### US-28: A local gateway (OmniRoute) is registered as an ordinary provider
+**As a** user running a local OpenAI-compatible gateway, **I want** LaCode to register it so its models are selectable and I can name one in my pool, **so that** *I* decide whether pooled work drains through it instead of the bundled models.
 
 **Story ID(s) for tests:** US-28
 **Acceptance criteria:**
-- [ ] Given `omniroute.enabled` with a reachable gateway and a `combo` or `models` choice, When the config hook runs, Then slot 1 of the pool is `omniroute/<combo | first allowlisted model>` — replacing the first slot's model IN PLACE and inheriting that slot's weight — so it is the model the pool takes first.
-- [ ] Given OmniRoute is disabled, enabled with an invalid block, unhealthy, or returns no usable models, When the pool resolves, Then no slot is changed and the resolved slots are exactly what the chain produced — the behaviour before the integration existed.
-- [ ] Given the user's own pool already routes on the `omniroute` provider, When the integration is healthy, Then the first slot is left exactly as written and no second slot is stacked on that provider.
-- [ ] Given the primary slot is injected, When the pool resolves, Then the number of slots is unchanged and every slot after the first is byte-identical to what the chain produced — nothing is dropped or renumbered, because the replacement creates no extra slot.
-- [ ] Given an `omniroute` slot is injected, When the pool binds its variants, Then it does not warn that the provider is missing, because OmniRoute registers the provider before the pool's config hook runs.
+- [ ] Given `omniroute.enabled` with a reachable gateway, When the config hook runs, Then LaCode registers a `provider.omniroute` entry exposing the gateway's models (everything it lists, or filtered by `models`/`combo`), and writes NOTHING into the pool.
+- [ ] Given any pool source (bundled preset, `pool.json`, `pool.txt`, or `options.models`), When OmniRoute is enabled and healthy, Then every slot resolves exactly as it would without the integration — the slot list is unchanged, so opting in means the user lists `omniroute/<model>` themselves.
+- [ ] Given a pool slot naming an `omniroute/<model>`, When the pool resolves and binds, Then that slot routes as an ordinary slot, and the pool does not warn the provider is missing, because OmniRoute registers the provider before the pool's config hook runs.
+- [ ] Given OmniRoute is disabled, enabled with an invalid block, unhealthy, or returns no usable models, When the config hook runs, Then no provider is registered, no pool slot changes, and a failure is warned about once.
+- [ ] Given the gateway is unreachable, When pooled work runs, Then the only thing affected is a slot that names `omniroute/<model>`; every other slot keeps working, because the plugin never put an `omniroute` slot in the pool.
 
 ---
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lawaty/lacode",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Ultimate opencode setup for Go users — cost-controlled pooled subagents, hang recovery, and notifications for opencode.",
   "license": "MIT",
   "type": "module",