dsh-llm-fallbacks 0.4.2 → 0.4.3

package/dist/state.d.ts

@@ -31,6 +31,19 @@ import type { FallbackSwitchReason } from './events.ts';
 import { CooldownStore, StepFailureSet } from './cooldown.ts';
 import { RecoveryStore } from './recovery.ts';
 import type { RecoveryPolicy } from './config.ts';
+/**
+ * What a switch's failure was actually about.
+ *
+ * `'route'` (the default) is every failure the plugin has ever walked: the
+ * provider/model itself is unhealthy (auth, quota, rate limit, ...), so the
+ * from-route is cooled down and counted as a route failure.
+ * `'request'` is a failure of the REQUEST on a healthy route — a
+ * `CONTEXT_WINDOW_EXCEEDED` rejection is the request being too big, not the
+ * model being down. `commit()` keeps the step-scoped bookkeeping for those
+ * but writes no suppression and no recovery failure (see `src/index.ts`
+ * `REQUEST_SCOPED_CODES`).
+ */
+export type SwitchScope = 'route' | 'request';
 /** Decision awaiting application at the next `agent/request` boundary (spec §5.1). */
 export interface PendingSwitch {
     from: {
@@ -43,6 +56,13 @@ export interface PendingSwitch {
     };
     role: string;
     reason: FallbackSwitchReason;
+    /**
+     * What the failure was about — absent means `'route'`, the semantics every
+     * switch had before request-scoped codes existed (same optional-with-a-
+     * documented-default idiom as `FallbacksRole.fallback`). Only the
+     * `agent/request-error` listener ever sets `'request'`.
+     */
+    scope?: SwitchScope;
 }
 /**
  * One plugin-originated subagent switch the host allowlist blocked (plan

package/dist/tui-settings.d.ts

@@ -125,12 +125,14 @@ export declare const JSON_FIELD_MAX_DRAFT_BYTES: number;
 export declare function buildFallbacksTuiSection(): TuiSettingsSection;
 /**
  * Register the `fallbacks` section on the optional `tuiSettingsSections`
- * service. First-fiber-only (`serviceOwned === true` — mirrors
- * `installTuiClient` and the gateway/typert multi-fiber dedupe; the host
- * registry throws on a duplicate namespace, so a deduped later fiber must
- * never register). The service is optional: a composition without
- * `dsh-tui-settings-sections` keeps the plugin working and simply omits the
- * TUI settings surface.
+ * service. Gated by `serviceOwned` — the optimistic apply-time claim (see
+ * `apply()` in src/index.ts) — with a sibling dedupe guard inside the child,
+ * mirroring `installTuiClient`: the claim is not a fact, so a fiber applying
+ * inside the claim window can race another fiber's registration, and the
+ * host registry's duplicate throw must degrade (no-op disposer + debug log)
+ * instead of failing the child. Any other error stays loud. The service is
+ * optional: a composition without `dsh-tui-settings-sections` keeps the
+ * plugin working and simply omits the TUI settings surface.
  *
  * The inject child returns the registry disposer so cordis withdraws the
  * registration when this fiber (or the service) goes away.

package/dist/tui.d.ts

@@ -42,11 +42,18 @@ export interface TuiCommandTreeProvider {
 export declare const FALLBACKS_TUI_ROOT = "fallbacks";
 /**
  * Register the `/fallbacks` provider on the optional `tuiCommandTrees`
- * service. First-fiber-only (`serviceOwned === true` — mirrors the
- * gateway/typert multi-fiber dedupe; the host registry throws on duplicate
- * roots, so a deduped later fiber must never register). The service is
- * optional: a composition without `dsh-tui-command-trees` keeps the plugin
- * working and simply omits the TUI surface.
+ * service. Gated by the OPTIMISTIC `serviceOwned` claim taken at apply time
+ * (the service itself is provided inside the settings inject child, after
+ * apply returns — see `apply()` in src/index.ts), with the sibling dedupe
+ * guard inside the child: the claim is not a fact, so a fiber applying
+ * inside the claim window registers here and, when it loses the race for
+ * the host registry's root, degrades via the `already registered` catch
+ * below — only a post-settlement duplicate fiber skips at this gate (its
+ * claim INITIALIZED false: the service was already visible when the claim
+ * was taken; the child-side catch correction settles strictly after apply()
+ * and can never influence this synchronous read). The service is optional:
+ * a composition without `dsh-tui-command-trees` keeps the plugin working
+ * and simply omits the TUI surface.
  *
  * The inject child returns the registry disposer so cordis withdraws the
  * registration when this fiber (or the service) goes away.

package/package.json

@@ -5,7 +5,7 @@
     "dsh",
     "dsh-plugin"
   ],
-  "version": "0.4.2",
+  "version": "0.4.3",
   "type": "module",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",

package/scripts/build-client.ts

@@ -362,7 +362,12 @@ for (const match of injectedCssMatches) {
     }
   }
 }
-const classMapMatches = [...bundleText.matchAll(/\b\w+_module_css_default\s*=\s*(\{[\s\S]*?\n\s*\});/g)]
+// The class-map object literal is FLAT (string → string, emitted by
+// cssModuleContents as one JSON.stringify), so the lazy match to the first
+// `};` is exact for both emit shapes the printer produces — multi-entry maps
+// across lines and a single-entry map inline (`{ "badge": "_<8hex>_badge" };`
+// has no newline before its closing brace).
+const classMapMatches = [...bundleText.matchAll(/\b\w+_module_css_default\s*=\s*(\{[\s\S]*?\});/g)]
 if (classMapMatches.length === 0) {
   throw new Error('client bundle contract: no inlined css-module class map found — check the dsh-css-modules-inline plugin')
 }

package/scripts/prepare-release.ts

@@ -79,7 +79,7 @@ export function parseArgs(argv: string[]): { version?: string; autoBump: boolean
  * version stays in its prerelease line, bumping only the numeric tail.
  */
 export function autoBumpPatch(current: string): string {
-  const m = /^(\d+)\.(\d+)\.(\d+)(?:-(.+))?$/.exec(current)
+  const m = current.match(/^(\d+)\.(\d+)\.(\d+)(?:-(.+))?$/)
   if (!m) throw new Error(`Cannot auto-bump "${current}": not a parseable X.Y.Z[-pre] version`)
   const [, major, minor, patch, pre] = m
   if (pre === undefined) return `${major}.${minor}.${Number(patch) + 1}`