@goodandready/dsh-key-rotation 0.8.0 → 0.8.2

package/README.md

@@ -45,7 +45,7 @@
 - **🕒 Accurate Midnight PST Resets**: Corrected UTC-8 timezone calculation offset sign for calendar quota reset windows.
 - **🧹 Lifecycle Timer Cleanup**: Wrapped `canaryTimer` and `selfHealTimer` in Cordis effect scopes, eliminating background orphaned intervals on hot reload.
 - **⚡ Stale Lock Recovery in Load Balancer**: Added expired lock detection to `pickLeastLoaded` for uninterrupted least-connections routing.
-- **🌐 Full Chinese Localization**: Added complete `zh` locale dictionary to the React settings dashboard for comprehensive 3-language parity.
+- **🌐 Localization (Changed in v0.8.2)**: Source strings are English-only. Russian/Chinese UI comes from the DSH core locale service and translation plugins (`props.t`). Active locale: host snapshot → first `navigator.languages` entry → `en` (#277).
 
 
 ### 🚀 What's New in v0.7.31
@@ -301,5 +301,5 @@ MIT © [GooDAnDReaDY](https://github.com/GooDAnDReaDY)
 - **Lifecycle Cleanups**: Wrapped `credentials.resolve` patch and `ctx.on` event handlers (`llm/stream`, `agent/request-error`) in `ctx.effect` scopes with guaranteed unmount cleanup (#238, #239).
 - **Settings & Secret Roles**: Added `.role('secret')` to `incidentGitHubToken` and `webhookActionToken` in `Config` schema for automatic UI masking (#237).
 - **Settings Architecture & UI**: Added native `settingsScope` snapshot reading/saving in settings card with graceful bridge fallback (#235).
-- **Localization**: Localized `settings.section` sidebar fallback label via `t('title')` and registered `zh` dictionary in `ctx.locale` alongside `en` and `ru` (#236).
+- **Localization (Changed in v0.8.2)**: Card uses `props.t` with `locale: NS`; plugin registers only `en`; no `settings.section` fallback and no bundled `ru`/`zh` tables (#236, #275, #277).
 - **Dead Code Purge**: Removed obsolete `mountDashboard` routine after header-chip migration (#240).

package/README.ru.md

@@ -295,7 +295,7 @@ MIT © [GooDAnDReaDY](https://github.com/GooDAnDReaDY)
 - **Очистка жизненного цикла**: Патч `credentials.resolve` и слушатели событий `ctx.on` (`llm/stream`, `agent/request-error`) переведены в скоупы `ctx.effect` с автоматическим восстановлением функций и отпиской при выгрузке плагина (#238, #239).
 - **Роли секретов в схеме**: Полям `incidentGitHubToken` и `webhookActionToken` в схеме `Config` присвоена роль `.role('secret')` для маскирования в UI (#237).
 - **Архитектура настроек**: Добавлена нативная интеграция со снимками `settingsScope` в карточке настроек с безопасным фоллбеком на HTTP-мост (#235).
-- **Локализация**: Фоллбек секции настроек `settings.section` переведён на локализованную метку `t('title')` со слотом `locale: NS`, а китайский словарь `zh` зарегистрирован в `ctx.locale` наряду с `en` и `ru` (#236).
+- **Локализация (Changed in v0.8.2)**: Исходные строки только на английском (`en`). Русский и китайский приходят через core `props.t` / translation-плагины. Фолбек активной локали: snapshot → первый `navigator.languages` → `en`. Свой `settings.section` и встроенные `ru`/`zh`-таблицы удалены (#236, #275, #277).
 - **Удаление мёртвого кода**: Удалена неиспользуемая функция `mountDashboard` после перехода на header chip (#240).
 
 ### Параметры circuit breaker (Changed in v0.8.0)

package/README.zh.md

@@ -259,7 +259,7 @@ MIT © [GooDAnDReaDY](https://github.com/GooDAnDReaDY)
 - **生命周期清理**: 将 `credentials.resolve` 猴子补丁和 `ctx.on` 事件监听器 (`llm/stream`, `agent/request-error`) 封装在 `ctx.effect` 作用域内，确保卸载时自动注销并恢复原始方法 (#238, #239)。
 - **配置密钥角色**: 在 `Config` Schema 中为 `incidentGitHubToken` 和 `webhookActionToken` 增加 `.role('secret')`，避免明文泄露并在 UI 中掩码显示 (#237)。
 - **设置架构与状态**: 在设置卡片中增加原生 `settingsScope` 绑定支持，保留 HTTP 桥接安全回退机制 (#235)。
-- **本地化与文案**: 侧边栏备用项 `settings.section` 标签支持本地化 `t('title')` 并配置 `locale: NS`，并在 `ctx.locale` 中注册 `zh` 中文字典 (#236)。
+- **本地化与文案 (Changed in v0.8.2)**: 插件仅注册英文源字符串 `en`；俄文/中文由 DSH 核心 locale 与 translation 插件通过 `props.t` 提供。活动语言回退：snapshot → 首个 `navigator.languages` → `en`。已移除 `settings.section` 回退与内置 `ru`/`zh` 表 (#236, #275, #277)。
 - **死代码清理**: 移除 header-chip 迁移后残留的废弃 `mountDashboard` 函数 (#240)。
 
 

package/lib/client.js

@@ -1,4 +1,4 @@
-// dsh-key-rotation — Settings section ("Key Rotation" / "Ротация ключей").
+// dsh-key-rotation — Settings card ("Key Rotation" / "Ротация ключей").
 // Renders in Settings → Plugins → Plugin settings via the settings.plugin.item slot and
 // edits the plugin's `dsh-key-rotation` settings namespace through the
 // loopback-fenced config bridge at /dsh-key-rotation/config.
@@ -9,11 +9,10 @@
 // manual route typing is ever needed. The plugin derives the fallback chain and
 // auto-creates clone routes from the key count.
 //
-// Localization: the plugin registers its own en/ru dictionaries with the DSH
-// locale service (ctx.locale.register) and resolves the "active" locale
-// through ctx.locale.getSnapshot().active + ctx.locale.subscribe() via
-// React.useSyncExternalStore, so the UI switches language live whenever the
-// DSH UI locale changes (Settings → Language).
+// Localization: source strings are English only (ctx.locale.register(NS, { en })).
+// Other languages come from the DSH core locale service / translation plugins
+// via props.t (slot locale: NS). Active locale prefers ctx.locale.getSnapshot().active,
+// else first navigator.languages entry, else 'en' — same fallback chain as DSH core.
 window.__ModuleLoader__.load({
   id: '@goodandready/dsh-key-rotation',
   factory: (require) => {
@@ -94,152 +93,43 @@ window.__ModuleLoader__.load({
       poolExhausted: 'pool exhausted — all keys cooling',
       resetting: 'Resetting…',
       keyLabel: 'Key {n}',
+      undoProvider: 'Provider removed',
+      undoKey: 'Key removed',
+      undo: 'Undo',
     };
-    const ru = {
-      title: 'Ротация ключей',
-      filterAll: 'Все',
-      filterReady: 'Готовы',
-      filterCooldown: 'В кулдауне',
-      filterErrors: 'С ошибками',
-      subtitle: 'Ротация API-ключей по провайдерам: пулы ключей, автоматическое переключение при исчерпании квоты или лимита, кулдаун и восстановление.',
-      cardDesc: 'Ротация API-ключей по провайдерам: пулы ключей, автоматическое переключение при исчерпании квоты или лимита, кулдаун и восстановление.',
-      loading: 'Загрузка…',
-      notRegistered: 'не зарегистрирован',
-      removeKey: 'Удалить ключ',
-      removeProvider: 'Удалить провайдера',
-      addKey: '+ Добавить ключ',
-      addKeyTitle: 'Добавить API-ключ',
-      noProviders: 'Провайдеры ещё не зарегистрированы в DSH — выбирать не из чего.',
-      addProvider: '+ Добавить провайдера',
-      desc: 'Ротация API-ключей по провайдерам. Для каждого провайдера укажите его API-ключи (имена env, хранятся в учётных данных DSH). Плагин ведёт модель по ключам провайдера по порядку и переключается на следующий при исчерпании квоты/превышении лимита.',
-      cooldown: 'Задержка после сбоя (мс)',
-      scheduleDays: 'Расписание ротации (дней, 0=выкл)',
-      switchCodes: 'Коды переключения (через запятую)',
-      providersTitle: 'Провайдеры и их ключи',
-      save: 'Сохранить',
-      discard: 'Отменить',
-      saving: 'Сохранение…',
-      moveUp: 'Выше',
-      moveDown: 'Ниже',
-      keyActive: 'используется',
-      keyReady: 'готов',
-      keyCooling: 'остывает, {s}с',
-      keyMissing: 'ключ не найден',
-      switchesNone: 'переключений не было',
-      switchesSome: 'переключений: {n} · последнее: {reason}, {ago}',
-      justNow: 'только что',
-      minutesAgo: '{n} мин назад',
-      hoursAgo: '{n} ч назад',
-      codesTitle: 'Переключаться при этих сбоях',
-      keyValuePlaceholder: 'вставьте ключ и нажмите «Сохранить»',
-      keySave: 'Сохранить ключ',
-      keySaved: 'сохранён',
-      keyFromEnv: 'задан в окружении, отсюда не меняется',
-      keyWriteFailed: 'не удалось сохранить ключ: {msg}',
-      notSecretShape: 'сохранено, но значение не похоже на API-ключ — проверьте опечатки',
-      rpmTitle: 'запросов/мин: {u} использовано, {r} осталось',
-      budgetLabel: 'бюджет:',
-      exportCsv: 'CSV',
-      weightHint: 'вес в круге: сколько раз ключ участвует в ротации (1 = поровну)',
-      retestBroken: 'Re-test',
-      retestFail: 'перепроверка не прошла — ключ всё ещё недоступен',
-      snapshotExport: 'Снапшот ⬇',
-      snapshotImport: 'Восстановить',
-      keyHint: 'Значение хранится в учётных данных DSH и обратно в браузер не отдаётся — показываются только последние 5 символов. Имена переменных создаются автоматически; наведите на ключ, чтобы увидеть используемое имя.',
-      brokenKey: 'сломан (3× AUTH)',
-      keyExpired: 'истёк',
-      keyExpiringSoon: 'истекает через {n} д',
-      exportPools: 'Экспорт',
-      exportOne: '⬇',
-      usedAgo: '{ago} назад',
-      importPools: 'Импорт',
-      importEnv: 'Импорт .env',
-      resetCooldown: 'Сбросить кулдаун',
-      testAll: 'Тест всех ключей',
-      testing: 'Тестирование…',
-      testKey: 'Тест',
-      testOk: 'OK',
-      testFail: 'FAIL',
-      poolExhausted: 'пул исчерпан — все ключи остывают',
-      resetting: 'Сброс…',
-      keyLabel: 'Ключ {n}',
-    };
-    const zh = {
-      title: '密钥轮换',
-      filterAll: '全部',
-      filterReady: '就绪',
-      filterCooldown: '冷却中',
-      filterErrors: '故障',
-      subtitle: '按提供商轮换 API 密钥：密钥池管理、配额/限流自动故障转移、平滑冷却与自愈。',
-      cardDesc: '按提供商轮换 API 密钥：密钥池管理、配额/限流自动故障转移、平滑冷却与自愈。',
-      loading: '加载中…',
-      notRegistered: '未注册',
-      removeKey: '删除密钥',
-      removeProvider: '删除提供商',
-      addKey: '+ 添加密钥',
-      addKeyTitle: '添加 API 密钥',
-      noProviders: 'DSH 尚未注册任何提供商 — 暂无可选项目。',
-      addProvider: '+ 添加提供商',
-      desc: '按提供商轮换 API 密钥。为每个提供商配置其 API 密钥（环境变量名，存储于 DSH 凭据中）。插件按序调度请求，遇配额或速率超限自动切换至下一密钥。',
-      cooldown: '故障后冷却时间 (毫秒)',
-      scheduleDays: '轮换排期 (天数，0=关闭)',
-      switchCodes: '切换触发错误码 (逗号分隔)',
-      providersTitle: '提供商与密钥池',
-      save: '保存',
-      discard: '放弃',
-      saving: '保存中…',
-      moveUp: '上移',
-      moveDown: '下移',
-      keyActive: '正在使用',
-      keyReady: '就绪',
-      keyCooling: '冷却中，{s}秒',
-      keyMissing: '凭据未找到',
-      switchesNone: '暂无切换',
-      switchesSome: '已切换：{n} 次 · 最近：{reason}，{ago}',
-      justNow: '刚刚',
-      minutesAgo: '{n} 分钟前',
-      hoursAgo: '{n} 小时前',
-      codesTitle: '遇以下故障自动切换',
-      keyValuePlaceholder: '粘贴密钥，然后点击保存',
-      keySave: '保存密钥',
-      keySaved: '已保存',
-      keyFromEnv: '来自环境变量，此处只读',
-      keyWriteFailed: '无法保存密钥：{msg}',
-      notSecretShape: '已保存，但格式不像常见 API 密钥 — 请检查是否有拼写错误',
-      rpmTitle: '请求数/分钟：已用 {u}，剩余 {r}',
-      budgetLabel: '预算：',
-      exportCsv: '导出 CSV',
-      weightHint: '轮询权重：该密钥参与循环的轮次（1 = 均等分担）',
-      retestBroken: '重新测试',
-      retestFail: '重试失败 — 密钥仍不可用',
-      snapshotExport: '快照 ⬇',
-      snapshotImport: '恢复快照',
-      keyHint: '密钥安全保存在 DSH 凭据中，绝不返回浏览器 — 界面仅展示末尾 5 位字符。变量名自动生成，鼠标悬停可查看。',
-      brokenKey: '已损坏 (连续 3 次鉴权失败)',
-      keyExpired: '已过期',
-      keyExpiringSoon: '{n} 天后过期',
-      exportPools: '导出',
-      exportOne: '⬇',
-      usedAgo: '{ago}',
-      importPools: '导入',
-      importEnv: '导入 .env',
-      resetCooldown: '重置冷却',
-      testAll: '测试全部密钥',
-      testing: '测试中…',
-      testKey: '测试',
-      testOk: '正常',
-      testFail: '失败',
-      poolExhausted: '密钥池耗尽 — 所有密钥均处于冷却状态',
-      resetting: '重置中…',
-      keyLabel: '密钥 {n}',
-    };
-
 
     // Коды, на которых имеет смысл переключать ключ. Список из хоста
     // (DEFAULT_SWITCH_CODES); конфиг может содержать и свои — они показываются
     // отдельными отмеченными галочками, чтобы правило нельзя было потерять.
     const KNOWN_CODES = ['QUOTA', 'RATE_LIMIT', 'SERVER', 'TIMEOUT', 'TRANSPORT', 'EMPTY_RESPONSE', 'UNKNOWN_MODEL', 'AUTH'];
 
+    // #273: keep the settings list item mounted if the section throws.
+    class KeyRotationErrorBoundary extends React.Component {
+      constructor(props) {
+        super(props);
+        this.state = { error: null };
+      }
+      static getDerivedStateFromError(error) {
+        return { error };
+      }
+      componentDidCatch(error, info) {
+        try { console.error('[dsh-key-rotation] settings card crashed', error, info); } catch (_) {}
+      }
+      render() {
+        if (this.state.error) {
+          return React.createElement('div', { className: 'krot krot-err', style: { padding: 12 } },
+            React.createElement('p', null, 'Key Rotation settings failed to render.'),
+            React.createElement('p', { style: { fontSize: 11, opacity: 0.8 } }, String(this.state.error?.message || this.state.error)),
+            React.createElement('button', {
+              type: 'button',
+              className: 'krot-btn',
+              onClick: () => this.setState({ error: null }),
+            }, 'Retry'));
+        }
+        return this.props.children;
+      }
+    }
+
     /** Опрос статуса ротации, пока раздел настроек открыт (smart polling). */
     function useRotationStatus() {
       const [byProvider, setByProvider] = React.useState({});
@@ -392,7 +282,17 @@ window.__ModuleLoader__.load({
             const active = ctx.locale.getSnapshot().active;
             if (typeof active === 'string' && active) return active;
           }
-          return typeof navigator !== 'undefined' ? String(navigator.language || '').slice(0, 2) : '';
+          // Match DSH core: first entry of navigator.languages, else en.
+          if (typeof navigator !== 'undefined') {
+            const langs = (navigator.languages && navigator.languages.length)
+              ? Array.from(navigator.languages)
+              : (navigator.language ? [navigator.language] : []);
+            for (const lang of langs) {
+              const code = String(lang || '').slice(0, 2).toLowerCase();
+              if (code) return code;
+            }
+          }
+          return 'en';
         }, [ctx])
       );
     }
@@ -401,9 +301,18 @@ window.__ModuleLoader__.load({
       return (key) => (DICT && DICT[key]) || (fallbackKeys && fallbackKeys[key]) || key;
     }
 
+    // Prefer the host slot translator (props.t + translation plugins); fall back to en source.
+    function resolveT(props) {
+      const coreT = props && typeof props.t === 'function' ? props.t : null;
+      if (!coreT) return makeT(en, en);
+      return (key) => {
+        const v = coreT(key);
+        return (v && v !== key) ? v : (en[key] || key);
+      };
+    }
+
     function KeyRotationSection(props) {
-      const DICT = props.locale === 'ru' ? ru : (props.locale === 'zh' ? zh : en);
-      const t = makeT(DICT, en);
+      const t = resolveT(props);
       const settingsScope = React.useMemo(() => {
         if (props.ctx && props.ctx.settingsScope) {
           try {
@@ -414,9 +323,31 @@ window.__ModuleLoader__.load({
         }
         return null;
       }, [props.ctx]);
+      // #273: getSnapshot must be referentially stable or React 18 unmounts the tree
+      const scopeCacheRef = React.useRef({ has: false, value: null });
+      const getScopeSnapshot = React.useCallback(() => {
+        if (!settingsScope || typeof settingsScope.getSnapshot !== 'function') return null;
+        let next = null;
+        try {
+          next = settingsScope.getSnapshot();
+        } catch (_) {
+          return null;
+        }
+        const prev = scopeCacheRef.current;
+        if (prev.has && Object.is(prev.value, next)) return prev.value;
+        // shallow-compare common snapshot shape to avoid identity thrash
+        if (prev.has && prev.value && next
+          && prev.value.status === next.status
+          && prev.value.revision === next.revision
+          && prev.value.value === next.value) {
+          return prev.value;
+        }
+        scopeCacheRef.current = { has: true, value: next };
+        return next;
+      }, [settingsScope]);
       const scopeSnapshot = React.useSyncExternalStore(
-        React.useMemo(() => (cb) => (settingsScope ? settingsScope.subscribe(cb) : () => {}), [settingsScope]),
-        React.useCallback(() => (settingsScope ? settingsScope.getSnapshot() : null), [settingsScope])
+        React.useMemo(() => (cb) => (settingsScope && typeof settingsScope.subscribe === 'function' ? settingsScope.subscribe(cb) : () => {}), [settingsScope]),
+        getScopeSnapshot
       );
       const [state, setState] = React.useState({ status: 'loading', value: null, revision: 0, error: '', providers: [] });
       const [draft, setDraft] = React.useState(null);
@@ -799,10 +730,10 @@ window.__ModuleLoader__.load({
         });
 
         const filterBar = keys.length > 1 ? h('div', { className: 'krot-filter-bar' },
-          h('button', { type: 'button', className: 'krot-pill' + (statusFilter === 'all' ? ' krot-pill-active' : ''), onClick: () => setStatusFilter('all') }, (t('filterAll') || 'Все') + ' (' + keys.length + ')'),
-          h('button', { type: 'button', className: 'krot-pill' + (statusFilter === 'ready' ? ' krot-pill-active' : ''), onClick: () => setStatusFilter('ready') }, (t('filterReady') || 'Готовы') + ' (' + readyCount + ')'),
-          cooldownCount > 0 ? h('button', { type: 'button', className: 'krot-pill krot-pill-warn' + (statusFilter === 'cooldown' ? ' krot-pill-active' : ''), onClick: () => setStatusFilter('cooldown') }, (t('filterCooldown') || 'В кулдауне') + ' (' + cooldownCount + ')') : null,
-          errorCount > 0 ? h('button', { type: 'button', className: 'krot-pill krot-pill-err' + (statusFilter === 'error' ? ' krot-pill-active' : ''), onClick: () => setStatusFilter('error') }, (t('filterErrors') || 'С ошибками') + ' (' + errorCount + ')') : null,
+          h('button', { type: 'button', className: 'krot-pill' + (statusFilter === 'all' ? ' krot-pill-active' : ''), onClick: () => setStatusFilter('all') }, t('filterAll') + ' (' + keys.length + ')'),
+          h('button', { type: 'button', className: 'krot-pill' + (statusFilter === 'ready' ? ' krot-pill-active' : ''), onClick: () => setStatusFilter('ready') }, t('filterReady') + ' (' + readyCount + ')'),
+          cooldownCount > 0 ? h('button', { type: 'button', className: 'krot-pill krot-pill-warn' + (statusFilter === 'cooldown' ? ' krot-pill-active' : ''), onClick: () => setStatusFilter('cooldown') }, t('filterCooldown') + ' (' + cooldownCount + ')') : null,
+          errorCount > 0 ? h('button', { type: 'button', className: 'krot-pill krot-pill-err' + (statusFilter === 'error' ? ' krot-pill-active' : ''), onClick: () => setStatusFilter('error') }, t('filterErrors') + ' (' + errorCount + ')') : null,
         ) : null;
 
         const keyRows = filteredIndices.map(({ key, kIndex }) => {
@@ -1068,7 +999,7 @@ window.__ModuleLoader__.load({
         h('p', { className: 'krot-hint' }, t('keyHint')),
         secretError ? h('p', { className: 'krot-err' }, secretError) : null,
         state.error ? h('p', { className: 'krot-err' }, state.error) : null,
-        undo ? h('div', { className: 'krot-foot' }, h('span', { className: 'krot-hint' }, undo.type === 'provider' ? 'Удалён провайдер' : 'Удалён ключ'), btn('Undo', doUndo, {})) : null,
+        undo ? h('div', { className: 'krot-foot' }, h('span', { className: 'krot-hint' }, undo.type === 'provider' ? t('undoProvider') : t('undoKey')), btn(t('undo'), doUndo, {})) : null,
         h('div', { className: 'krot-foot' },
           btn(t('save'), save, { primary: true }),
           btn(t('discard'), load, {}),
@@ -1128,7 +1059,7 @@ window.__ModuleLoader__.load({
         h('button', {
           type: 'button',
           className: 'krot-header-chip',
-          title: 'Ротация ключей / Key Rotation Pools',
+          title: 'Key Rotation',
           onClick: () => setOpen((v) => !v),
         },
           h('span', { style: { width: '8px', height: '8px', borderRadius: '50%', background: color, flex: 'none', boxShadow: `0 0 6px ${color}` } }),
@@ -1136,8 +1067,8 @@ window.__ModuleLoader__.load({
           h('span', { style: { fontSize: '9px', opacity: 0.6 } }, open ? '▲' : '▼')
         ),
         open ? h('div', { className: 'krot-popover' },
-          h('div', { className: 'krot-pop-title' }, 'Пулы ротации ключей'),
-          !poolEntries.length ? h('div', { style: { fontSize: '12px', color: 'var(--dsw-alias-label-tertiary)' } }, 'Нет активных пулов') : null,
+          h('div', { className: 'krot-pop-title' }, 'Key rotation pools'),
+          !poolEntries.length ? h('div', { style: { fontSize: '12px', color: 'var(--dsw-alias-label-tertiary)' } }, 'No active pools') : null,
           poolEntries.map(([name, p]) => {
             const c = p.exhausted ? '#e5484d' : (p.healthy < p.total ? '#f5a623' : '#30a46c');
             return h('div', { key: name, className: 'krot-pop-row' },
@@ -1153,7 +1084,7 @@ window.__ModuleLoader__.load({
     }
 
     function apply(ctx) {
-      ctx.effect(() => ctx.locale.register(NS, { en, ru, zh }), 'dsh-key-rotation: dictionaries');
+      ctx.effect(() => ctx.locale.register(NS, { en }), 'dsh-key-rotation: dictionaries');
       // Header chip (#201): status dot in the session header utilities slot,
       // same slot dsh-gitea / dsh-subscriptions use for their header widgets.
       ctx.effect(() => {
@@ -1175,7 +1106,7 @@ window.__ModuleLoader__.load({
       // key MUST equal the settings namespace (NS), else the tab silently skips it.
       function KeyRotationCard(props) {
         const locale = useLocale();
-        const t = makeT(locale === 'ru' ? ru : (locale === 'zh' ? zh : en), en);
+        const t = resolveT(props);
         const [open, setOpen] = React.useState(false);
         return h('div', { className: 'krot-card' + (open ? ' krot-card-open' : '') },
           h('button', { type: 'button', className: 'krot-card-header', 'aria-expanded': open, onClick: () => setOpen((v) => !v) },
@@ -1185,32 +1116,18 @@ window.__ModuleLoader__.load({
             h('span', { className: 'krot-card-chevron' + (open ? ' krot-card-chevron-open' : ''), 'aria-hidden': 'true' },
               h('svg', { width: 14, height: 14, viewBox: '0 0 14 14', fill: 'none', stroke: 'currentColor', strokeWidth: 1.5, style: { display: 'block' } },
                 h('path', { d: 'M3.5 5.25L7 8.75L10.5 5.25' })))),
-          open ? h('div', { className: 'krot-card-body' }, h(KeyRotationSection, { ...props, locale })) : null);
+          open ? h('div', { className: 'krot-card-body' }, h(KeyRotationErrorBoundary, null, h(KeyRotationSection, { ...props, locale }))) : null);
       }
-      const tryPluginItem = () => {
-        try {
-          ctx.slots.inject('settings.plugin.item', () =>
-            ctx.slots.register(
-              { name: 'settings.plugin.item', key: NS, locale: NS, inject: () => ({ ctx }) },
-              KeyRotationCard,
-            ));
-          return true;
-        } catch { return false; }
-      };
-      if (!tryPluginItem()) {
-        // Fallback for builds without the Plugins tab slot: keep the sidebar section.
-        const t = ctx.locale ? ctx.locale.bind(NS) : null;
-        ctx.slots.inject('settings.section', () => ctx.slots.register(
-          {
-            name: 'settings.section',
-            id: 'dsh-key-rotation',
-            order: 20,
-            locale: NS,
-            label: () => (t ? t('title') : 'Key Rotation') || 'Key Rotation',
-            inject: () => ({ ctx }),
-          },
-          (props) => h(KeyRotationSection, { ...props, locale: useLocale() }),
-        ));
+      // #275: card only — no top-level sidebar row. The host sidebar is a
+      // shared flat list; a duplicate row wastes a slot and confuses Settings.
+      try {
+        ctx.slots.inject('settings.plugin.item', () =>
+          ctx.slots.register(
+            { name: 'settings.plugin.item', key: NS, locale: NS, inject: () => ({ ctx }) },
+            KeyRotationCard,
+          ));
+      } catch (e) {
+        try { console.error('[dsh-key-rotation] settings.plugin.item register failed', e); } catch (_) {}
       }
     }
 

package/lib/http-bridge.js

@@ -69,7 +69,8 @@ export async function writeSection(ctx, ns, section, expectedRevision, res) {
 export function providerCatalog(ctx, cloneIds) {
   const seen = new Set();
   const out = [];
-  for (const info of ctx.llm.listProviders()) {
+  const llm = ctx.get('llm');
+  for (const info of (llm?.listProviders?.() ?? [])) {
     if (seen.has(info.id) || cloneIds.has(info.id)) continue;
     seen.add(info.id);
     out.push({ id: info.id, name: info.name ?? info.id });

package/lib/index.js

@@ -794,7 +794,12 @@ export function apply(ctx, config = {}) {
   }), 'dsh-key-rotation: agent/request-error');
 
   ctx.inject(['settings'], (sctx) => {
-    const scope = sctx.settings.register(NS, Config, { base: config });
+    const settingsSvc = sctx.get('settings');
+    if (!settingsSvc || typeof settingsSvc.register !== 'function') {
+      console.warn('[dsh-key-rotation] settings service unavailable — config scope not registered');
+      return;
+    }
+    const scope = settingsSvc.register(NS, Config, { base: config });
     getConfig = () => scope.get() ?? config;
     sctx.effect(() => () => {
       getConfig = () => config;

package/lib/rotate.js

@@ -98,7 +98,13 @@ export function createRotate(deps) {
         let inner;
         try {
           // mark the internal dispatch so the interceptor does not re-rotate
-          inner = dispatchStorage.run(reqStore, () => ctx.llm.stream({ ...options, [MARKER]: true }));
+          inner = dispatchStorage.run(reqStore, () => {
+            const llm = ctx.get('llm');
+            if (!llm || typeof llm.stream !== 'function') {
+              throw new Error('dsh-key-rotation: llm service unavailable');
+            }
+            return llm.stream({ ...options, [MARKER]: true });
+          });
         } catch (e) {
           const curRef = reqStore.pickedRef ?? pool.state.lastUsed;
           penalizeRef(curRef, e?.code ?? 'TRANSPORT', String(e?.message ?? ''));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goodandready/dsh-key-rotation",
-  "version": "0.8.0",
+  "version": "0.8.2",
   "packageManager": "pnpm@10.33.2",
   "description": "Per-provider API key rotation for DeepSeek Harness: a key pool per provider, auto-created clone routes, and switching to the next key on quota/rate-limit errors. Includes a Settings section (Key Rotation) to edit the key pools, cooldown and switch codes.",
   "keywords": [