@goodandready/dsh-clinebot 0.4.4 → 0.4.6

package/CHANGELOG.md

@@ -5,6 +5,25 @@ All notable changes to `@goodandready/dsh-clinebot` will be documented in this f
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.4.6] - 2026-09-25
+
+### Fixed
+- **Obsolete Settings Service Dependency Removal (#119)**: Removed ` settings` from `inject` and eradicated legacy `ctx.inject([settings], ...)` and `sctx.settings.register(...)` calls, adapting to DSH 0.1.7 kernel architecture where the Cordis `settings` service was removed.
+- **Config Reader Export (#119)**: Exported `configReader(config)` returning `() => config` to provide standard DSH configuration introspection.
+- **Defensive SettingsForms Adapter (#119)**: Implemented lightweight `settingsApi` adapter providing `get()`, `replace()`, `update()`, and `watch()` without declaring `settings` in `inject`, accessing `ctx?.get?.(settings)` defensively.
+- **Failover & Settings Regression Tests (#119)**: Added 5 strict DoD regression tests verifying `inject`, absence of `settings.register`, proxy property safety, config reader, and strict proxy boot.
+
+## [0.4.5] - 2026-09-25
+
+### Fixed
+- **Obsolete Model ID Migration & Canonical Normalization (#103)**: Automatically migrate historical model identifiers lacking dots (e.g., `deepseek-v41-flash` -> `deepseek-v4.1-flash`, `qwen38-max` -> `qwen3.8-max`, `glm-53` -> `glm-5.3`, `glm-53-flash` -> `glm-5.3-flash`, `muse-spark-13-contributor` -> `muse-spark-1.3-contributor`) across disk cache, DSH settings, and auto-discovery, eliminating stuck legacy IDs.
+- **Config Schema Test Assertion for Schemastery 3.18.4 (#112)**: Aligned `test/config-schema.test.js` with Schemastery 3.18.4 volatile getter behavior using `plainConfig(validated).enabled`.
+- **UI Color Compliance & Token Hardening (#116)**: Replaced hardcoded `rgba()` fallbacks in `accounts-section.js` and `models-section.js` with canonical DSH `color-mix()` and design tokens.
+- **Decomposition Compliance (#117)**: Extracted credentials helpers into `lib/credential-refs.js`, reducing `lib/cline-client.js` from 590 to 511 lines (well below the 600-line ceiling).
+- **Client Localization Pruning (#115)**: Removed 6 obsolete dead locale keys from `src/client/locales.js` in both `en` and `zh`.
+- **Route Policy & Packaging Contract (#113)**: Documented mandatory «Политика маршрутов» (16 registered endpoints) and «Что публикуется» sections in `docs/design/DESIGN.md`.
+- **Visual Verification Evidence (#114)**: Added dual-theme production visual verification artifact `media/visual-verification.png` and embedded under `## Visual verification` in `README.md`.
+
 ## [0.4.4] - 2026-09-25
 
 ### Added

package/README.md

@@ -206,6 +206,14 @@ All endpoints are registered under `/dsh-clinebot/*` and protected against untru
 
 ---
 
+## Visual verification
+
+Settings card in DeepSeek Harness, Dark and Light themes side by side:
+
+![dsh-clinebot visual verification](media/visual-verification.png)
+
+---
+
 ## 🧪 Testing
 
 Run the automated test suite:

package/lib/client.js

@@ -48,7 +48,6 @@ const en = {
       'key.save': 'Save Key',
       'key.saving': 'Saving…',
       'key.login_fast': '🔑 Open API Keys ↗',
-      'key.login_waiting': 'Paste key below for instant verification.',
       'key.verifying': 'Validating key with Cline…',
       'key.verified': '✓ Valid key for {email} ({plan})',
       'key.invalid': '✗ Key validation failed: {error}',
@@ -66,10 +65,8 @@ const en = {
       'accounts.env_placeholder': 'CLINEBOT_API_KEY_2',
       'accounts.delete_btn': 'Delete',
       'accounts.delete_confirm': 'Are you sure you want to delete account "{label}"?',
-      'accounts.delete_secret': 'Delete secret from credentials',
       'accounts.save_btn': 'Save Account',
       'accounts.cancel_btn': 'Cancel',
-      'accounts.key_label': 'API Key',
       'accounts.quota_used': '{pct}% used',
       'accounts.last_failover': 'Last failover: {from} → {to} ({reason})',
       'quota.title': '📊 ClinePass Quota & Rate Limits',
@@ -81,7 +78,6 @@ const en = {
       'quota.window_5h': '⏱ 5-Hour Rolling Limit',
       'quota.window_weekly': '📅 Weekly Window',
       'quota.used': '{pct}% used',
-      'quota.remaining': 'Remaining: {pct}%',
       'quota.reset_at': 'Reset: {time}',
       'models.title': '🎯 Model Picker in DSH ({enabled} of {total} active)',
       'models.desc': 'Catalog synchronizes directly from ClinePass subscription. New plan models are enabled automatically. Uncheck models to hide them from DSH chat picker. Thinking effort is selectable in the composer model menu.',
@@ -114,7 +110,6 @@ const en = {
       'models.context_updated_msg': 'Model context limits updated and synchronized to DSH.',
       'models.ctx_default': 'Default ({ctx})',
       'models.ctx_original': 'Original ({ctx} · {provider})',
-      'models.ctx_custom': 'Custom…',
       'models.ctx_edit': 'Edit Context Window',
       'models.ctx_custom_placeholder': 'Tokens (e.g. 2000000)',
       'models.ctx_save': 'Save',
@@ -131,7 +126,6 @@ const en = {
       'diag.desc': 'ClineBot registers into DSH Models automatically when key is configured. You can run connection smoke test or force re-sync.',
       'diag.smoke_btn': 'Run Smoke Test (Ping)',
       'diag.smoke_testing': 'Testing…',
-      'diag.smoke_ok': 'Smoke test passed! Latency: {latency} ms',
       'diag.resync_btn': 'Force Re-sync to DSH',
       'diag.resyncing': 'Syncing…',
       'diag.resynced_msg': 'Provider re-synced to DSH Models ({count} models)',
@@ -170,7 +164,6 @@ const en = {
       'key.save': '保存密钥',
       'key.saving': '正在保存…',
       'key.login_fast': '🔑 打开 API 密钥页 ↗',
-      'key.login_waiting': '请在下方粘贴密钥，系统将自动进行有效性校验。',
       'key.verifying': '正在连接 Cline 验证密钥…',
       'key.verified': '✓ 密钥有效：{email} ({plan})',
       'key.invalid': '✗ 密钥验证未通过：{error}',
@@ -188,10 +181,8 @@ const en = {
       'accounts.env_placeholder': 'CLINEBOT_API_KEY_2',
       'accounts.delete_btn': '删除',
       'accounts.delete_confirm': '确定要删除账号“{label}”吗？',
-      'accounts.delete_secret': '同时删除凭据密钥',
       'accounts.save_btn': '保存账号',
       'accounts.cancel_btn': '取消',
-      'accounts.key_label': 'API 密钥',
       'accounts.quota_used': '已用 {pct}%',
       'accounts.last_failover': '最近故障转移：{from} → {to} ({reason})',
       'quota.title': '📊 ClinePass 用量配额与限额监控',
@@ -203,7 +194,6 @@ const en = {
       'quota.window_5h': '⏱ 5 小时滚动限额',
       'quota.window_weekly': '📅 每周滚动窗口',
       'quota.used': '已使用 {pct}%',
-      'quota.remaining': '剩余可用：{pct}%',
       'quota.reset_at': '重置时间：{time}',
       'models.title': '🎯 DSH 模型选择列表（已启用 {enabled}/{total}）',
       'models.desc': '模型目录与 ClinePass 官方订阅实时同步。新订阅模型默认自动启用。取消勾选可在 DSH 对话模型列表中隐藏不需要的模型。支持在对话输入框上方切换推理思考强度（Effort）。',
@@ -236,7 +226,6 @@ const en = {
       'models.context_updated_msg': '模型上下文限制已更新并同步生效至 DSH。',
       'models.ctx_default': '默认 ({ctx})',
       'models.ctx_original': '原厂 ({ctx} · {provider})',
-      'models.ctx_custom': '自定义…',
       'models.ctx_edit': '编辑上下文窗口',
       'models.ctx_custom_placeholder': 'Token 数量 (如 2000000)',
       'models.ctx_save': '保存',
@@ -253,7 +242,6 @@ const en = {
       'diag.desc': '配置 API 密钥后，ClineBot 会自动向 DSH 注册模型服务。您可以在此进行连通性测试或强制重新同步。',
       'diag.smoke_btn': '执行连通性冒烟测试 (Ping)',
       'diag.smoke_testing': '正在测试…',
-      'diag.smoke_ok': '冒烟测试通过！网络延迟：{latency} ms',
       'diag.resync_btn': '强制重新同步到 DSH',
       'diag.resyncing': '正在同步…',
       'diag.resynced_msg': '模型服务已重新同步到 DSH ({count} 个模型)',
@@ -707,8 +695,8 @@ function AccountsSection({ status, busy, handlePinAccount, handleAddAccount, han
           {
             onSubmit: onSubmitAdd,
             style: {
-              background: 'var(--dsw-alias-bg-hover, rgba(0,0,0,0.04))',
-              border: '1px solid var(--dsw-alias-border, rgba(0,0,0,0.12))',
+              background: 'var(--dsw-alias-bg-hover, color-mix(in srgb, currentColor 4%, transparent))',
+              border: '1px solid var(--dsw-alias-border, color-mix(in srgb, currentColor 12%, transparent))',
               borderRadius: '8px',
               padding: '12px',
               marginBottom: '14px',
@@ -1206,7 +1194,7 @@ function ModelsSection({
                         display: 'flex',
                         flexDirection: 'column',
                         gap: '6px',
-                        background: 'var(--dsw-alias-bg-hover, rgba(0,0,0,0.04))',
+                        background: 'var(--dsw-alias-bg-hover, color-mix(in srgb, currentColor 4%, transparent))',
                         padding: '6px',
                         borderRadius: '6px',
                         border: '1px solid var(--dsw-alias-border-l2)',

package/lib/cline-client.js

@@ -1,3 +1,11 @@
+export {
+  toCredentialRef,
+  resolveApiKey,
+  saveCredentialKey,
+  deleteCredentialKey,
+  resolveKeyValue,
+} from './credential-refs.js'
+
 /**
  * ClineBot / ClinePass client helpers for DeepSeek Harness.
  *
@@ -22,20 +30,7 @@ export const USAGE_CACHE_TTL_MS = 60000
 
 export { PROVIDER_ID, PROVIDER_DISPLAY_NAME, DEFAULT_MODEL_ID }
 
-/**
- * Resolve or fallback credential reference descriptor.
- */
-export async function toCredentialRef(name) {
-  try {
-    const mod = await import('@deepseek-ai/dsh-credentials')
-    if (typeof mod.credentialRef === 'function') {
-      return mod.credentialRef(name)
-    }
-  } catch {
-    /* fallback when executed in standalone unit tests outside DSH bundle */
-  }
-  return typeof name === 'object' && name !== null ? name : { type: 'env', name: String(name || '') }
-}
+// Credentials helpers extracted to ./credential-refs.js
 
 /**
  * Normalize base URL ensuring clean format without trailing slashes.
@@ -57,57 +52,7 @@ export {
   resetSessionStats,
 } from './telemetry.js'
 
-/**
- * Resolve API key from environment variables.
- */
-export function resolveApiKey(apiKeyEnv, env = process.env) {
-  const name = String(apiKeyEnv || DEFAULT_API_KEY_ENV).trim() || DEFAULT_API_KEY_ENV
-  return {
-    envName: name,
-    value: String(env[name] || ''),
-  }
-}
-
-/**
- * Save API key directly into DSH credentials service (~/.dsh/.credentials.yaml).
- */
-export async function saveCredentialKey(ctx, apiKeyEnv, apiKey) {
-  const name = String(apiKeyEnv || DEFAULT_API_KEY_ENV).trim() || DEFAULT_API_KEY_ENV
-  const value = String(apiKey || '').trim()
-
-  if (!value) {
-    throw new Error('API key cannot be empty')
-  }
-
-  const credentials = ctx?.credentials || ctx?.get?.('credentials')
-  if (!credentials || typeof credentials.set !== 'function') {
-    throw new Error('DSH credentials service is unavailable in this runtime profile')
-  }
-
-  const ref = await toCredentialRef(name)
-  await credentials.set(ref, value)
-  return { ok: true, envName: name }
-}
 
-/**
- * Delete API key from DSH credentials service.
- */
-export async function deleteCredentialKey(ctx, apiKeyEnv) {
-  const name = String(apiKeyEnv || '').trim()
-  if (!name) return { ok: false }
-  const credentials = ctx?.credentials || ctx?.get?.('credentials')
-  if (!credentials) return { ok: false }
-  try {
-    const ref = await toCredentialRef(name)
-    if (typeof credentials.unset === 'function') {
-      await credentials.unset(ref)
-      return { ok: true, envName: name }
-    }
-  } catch (err) {
-    console.warn('[dsh-clinebot] Failed unsetting credential:', err)
-  }
-  return { ok: false }
-}
 
 function abortAfter(ms) {
   const ac = new AbortController()
@@ -560,31 +505,7 @@ export function buildPiAiProvider({
   }
 }
 
-/**
- * Safely resolve key value from DSH credentials service or process.env.
- */
-export async function resolveKeyValue(ctx, apiKeyEnv) {
-  const refName = String(apiKeyEnv || DEFAULT_API_KEY_ENV).trim() || DEFAULT_API_KEY_ENV
-  const creds = (ctx?.get && ctx.get('credentials')) || ctx?.credentials
-  if (creds && typeof creds.resolve === 'function') {
-    try {
-      const ref = await toCredentialRef(refName)
-      const hit = await creds.resolve(ref)
-      if (hit?.value) {
-        return { envName: refName, value: hit.value, source: 'credentials' }
-      }
-    } catch {
-      /* miss */
-    }
-  }
-
-  const fromEnv = resolveApiKey(refName)
-  if (fromEnv.value) {
-    return { ...fromEnv, source: 'env' }
-  }
 
-  return { envName: refName, value: '', source: 'none' }
-}
 
 
 export { resolveAccountPool, isAccountQuotaExhausted, rotateToNextAccount, getLastRotation, setLastRotation, isQuotaExceededError } from './account-pool.js'

package/lib/credential-refs.js

@@ -0,0 +1,84 @@
+/**
+ * Credentials management helper for @goodandready/dsh-clinebot.
+ * Isolates DSH Credentials storage and environment resolution.
+ */
+
+export const DEFAULT_API_KEY_ENV = 'CLINEBOT_API_KEY'
+
+export async function toCredentialRef(name) {
+  try {
+    const mod = await import('@deepseek-ai/dsh-credentials')
+    if (typeof mod.credentialRef === 'function') {
+      return mod.credentialRef(name)
+    }
+  } catch {
+    /* fallback when executed in standalone unit tests outside DSH bundle */
+  }
+  return typeof name === 'object' && name !== null ? name : { type: 'env', name: String(name || '') }
+}
+
+export function resolveApiKey(apiKeyEnv, env = process.env) {
+  const name = String(apiKeyEnv || DEFAULT_API_KEY_ENV).trim() || DEFAULT_API_KEY_ENV
+  return {
+    envName: name,
+    value: String(env[name] || ''),
+  }
+}
+
+export async function saveCredentialKey(ctx, apiKeyEnv, apiKey) {
+  const name = String(apiKeyEnv || DEFAULT_API_KEY_ENV).trim() || DEFAULT_API_KEY_ENV
+  const value = String(apiKey || '').trim()
+
+  if (!value) {
+    throw new Error('API key cannot be empty')
+  }
+
+  const credentials = ctx?.credentials || ctx?.get?.('credentials')
+  if (!credentials || typeof credentials.set !== 'function') {
+    throw new Error('DSH credentials service is unavailable in this runtime profile')
+  }
+
+  const ref = await toCredentialRef(name)
+  await credentials.set(ref, value)
+  return { ok: true, envName: name }
+}
+
+export async function deleteCredentialKey(ctx, apiKeyEnv) {
+  const name = String(apiKeyEnv || '').trim()
+  if (!name) return { ok: false }
+  const credentials = ctx?.credentials || ctx?.get?.('credentials')
+  if (!credentials) return { ok: false }
+  try {
+    const ref = await toCredentialRef(name)
+    if (typeof credentials.unset === 'function') {
+      await credentials.unset(ref)
+      return { ok: true, envName: name }
+    }
+  } catch (err) {
+    ctx?.logger?.warn?.('[dsh-clinebot] Failed unsetting credential: ' + (err?.message || err))
+  }
+  return { ok: false }
+}
+
+export async function resolveKeyValue(ctx, apiKeyEnv) {
+  const refName = String(apiKeyEnv || DEFAULT_API_KEY_ENV).trim() || DEFAULT_API_KEY_ENV
+  const creds = (ctx?.get && ctx.get('credentials')) || ctx?.credentials
+  if (creds && typeof creds.resolve === 'function') {
+    try {
+      const ref = await toCredentialRef(refName)
+      const hit = await creds.resolve(ref)
+      if (hit?.value) {
+        return { envName: refName, value: hit.value, source: 'credentials' }
+      }
+    } catch {
+      /* miss */
+    }
+  }
+
+  const fromEnv = resolveApiKey(refName)
+  if (fromEnv.value) {
+    return { ...fromEnv, source: 'env' }
+  }
+
+  return { envName: refName, value: '', source: 'none' }
+}

package/lib/index.js

@@ -26,7 +26,11 @@ import { registerAuthRoutes } from './routes/auth.js'
 import { registerSlashCommand } from './slash-command.js'
 
 export const name = '@goodandready/dsh-clinebot'
-export const inject = ['settings', 'webServer', 'credentials']
+export const inject = ['webServer', 'credentials']
+
+export function configReader(config) {
+  return () => config
+}
 
 export { NS, LLM_PI_AI_NS, Config, sessionStats, recordSessionRequest, resetSessionStats, rotateToNextAccount }
 
@@ -56,73 +60,62 @@ export function apply(ctx, config) {
     autoDiscoverPlanModels(ctx, { live, getSettingsApi: () => settingsApi, syncProviderState })
   }
 
-  const createSettingsAdapter = (svc) => {
-    if (!svc) return undefined
-    if (typeof svc.replace === 'function' || typeof svc.update === 'function' || typeof svc.write === 'function') {
-      const getRevision = () => {
-        try {
-          return svc.describe?.().find((row) => row.ns === NS)?.revision
-        } catch {
-          return undefined
-        }
-      }
-      return {
-        get: () => live(),
-        replace: async (next) => {
-          const parsed = Config(structuredClone(plainConfig(next)))
-          currentConfig = parsed
-          const payload = volatileConfig(parsed)
-          if (typeof svc.replace === 'function') {
-            await svc.replace(NS, payload, getRevision())
-          } else if (typeof svc.update === 'function') {
-            await svc.update(NS, payload, getRevision())
-          }
-          return parsed
-        },
-        update: async (patch) => {
-          const merged = Config({ ...publicConfig(live()), ...plainConfig(patch) })
-          currentConfig = merged
-          const payload = volatileConfig(merged)
-          if (typeof svc.update === 'function') {
-            await svc.update(NS, payload, getRevision())
-          } else if (typeof svc.replace === 'function') {
-            await svc.replace(NS, payload, getRevision())
-          }
-          return merged
-        },
-        watch: (cb) => {
-          if (typeof svc.watch === 'function') return svc.watch(cb)
-          return () => {}
-        },
-      }
+  const getRevision = (svc) => {
+    try {
+      return svc?.describe?.().find((row) => row.ns === NS)?.revision
+    } catch {
+      return undefined
     }
-    return undefined
   }
 
-  if (typeof ctx.inject === 'function') {
-    ctx.inject(['settings'], (sctx) => {
-      if (typeof sctx.settings?.register === 'function') {
-        const scope = sctx.settings.register(NS, Config, { base: config })
-        settingsApi = scope
-        getConfig = () => (scope?.get?.() ?? config) ?? config
-        sctx.effect(() => scope.watch((next) => {
-          syncProviderState(live())
-        }), 'dsh-clinebot: settings')
-      } else {
-        settingsApi = createSettingsAdapter(sctx.settings)
+  settingsApi = {
+    get: () => live(),
+    replace: async (next) => {
+      const parsed = Config(structuredClone(plainConfig(next)))
+      currentConfig = parsed
+      const svc = ctx?.get?.('settings')
+      if (svc) {
+        const payload = volatileConfig(parsed)
+        if (typeof svc.replace === 'function') {
+          await svc.replace(NS, payload, getRevision(svc))
+        } else if (typeof svc.update === 'function') {
+          await svc.update(NS, payload, getRevision(svc))
+        }
       }
-      sctx.effect(() => () => {
-        getConfig = () => config
-        settingsApi = undefined
-      })
-    })
+      await syncProviderState(live())
+      return parsed
+    },
+    update: async (patch) => {
+      const merged = Config({ ...publicConfig(live()), ...plainConfig(patch) })
+      currentConfig = merged
+      const svc = ctx?.get?.('settings')
+      if (svc) {
+        const payload = volatileConfig(merged)
+        if (typeof svc.update === 'function') {
+          await svc.update(NS, payload, getRevision(svc))
+        } else if (typeof svc.replace === 'function') {
+          await svc.replace(NS, payload, getRevision(svc))
+        }
+      }
+      await syncProviderState(live())
+      return merged
+    },
+    watch: (cb) => {
+      const svc = ctx?.get?.('settings')
+      if (typeof svc?.watch === 'function') return svc.watch(cb)
+      return () => {}
+    },
   }
 
   syncProviderState(live())
   if (typeof ctx.effect === 'function') {
     ctx.effect(() => {
       const timer = setTimeout(triggerAutoDiscover, 500)
-      return () => clearTimeout(timer)
+      const retryTimer = setTimeout(triggerAutoDiscover, 3000)
+      return () => {
+        clearTimeout(timer)
+        clearTimeout(retryTimer)
+      }
     }, 'dsh-clinebot: auto-discover')
   }
 

package/lib/models.js

@@ -11,6 +11,29 @@
  */
 
 export const PROVIDER_ID = 'clinebot'
+
+export const OBSOLETE_MODEL_ID_MAP = Object.freeze({
+  'cline-pass/deepseek-v41-flash': 'cline-pass/deepseek-v4.1-flash',
+  'cline-pass/qwen38-max': 'cline-pass/qwen3.8-max',
+  'cline-pass/glm-53': 'cline-pass/glm-5.3',
+  'cline-pass/glm-53-flash': 'cline-pass/glm-5.3-flash',
+  'cline-pass/muse-spark-13-contributor': 'cline-pass/muse-spark-1.3-contributor',
+})
+
+export function migrateModelId(id) {
+  if (typeof id !== 'string') return id
+  return OBSOLETE_MODEL_ID_MAP[id] || id
+}
+
+export function migrateModelEntry(model) {
+  if (!model || typeof model !== 'object') return model
+  const canonicalId = migrateModelId(model.id)
+  if (canonicalId === model.id) return model
+  return {
+    ...model,
+    id: canonicalId,
+  }
+}
 export const PROVIDER_DISPLAY_NAME = 'ClineBot (ClinePass)'
 export const DEFAULT_MODEL_ID = 'cline-pass/deepseek-v4-flash'
 
@@ -314,10 +337,17 @@ export function parsePlanIncludedModels(includedInput) {
  * Applies user-defined modelContextOverrides (custom contextLength and maxTokens).
  */
 export function getAllModels(dynamicModels = [], modelContextOverrides = []) {
+  const normalizedDynamic = (Array.isArray(dynamicModels) ? dynamicModels : []).map(migrateModelEntry)
   const overridesMap = new Map()
   if (Array.isArray(modelContextOverrides)) {
     for (const ov of modelContextOverrides) {
-      if (ov && ov.modelId) overridesMap.set(ov.modelId, ov)
+      if (ov && ov.modelId) {
+        overridesMap.set(ov.modelId, ov)
+        const canonicalId = migrateModelId(ov.modelId)
+        if (canonicalId !== ov.modelId) {
+          overridesMap.set(canonicalId, { ...ov, modelId: canonicalId })
+        }
+      }
     }
   }
 
@@ -344,9 +374,9 @@ export function getAllModels(dynamicModels = [], modelContextOverrides = []) {
     }
   }
 
-  if (Array.isArray(dynamicModels) && dynamicModels.length > 0) {
+  if (normalizedDynamic.length > 0) {
     const normalize = (s) => String(s || '').toLowerCase().replace(/[\s._-]+/g, '')
-    return dynamicModels.map((dm) => {
+    return normalizedDynamic.map((dm) => {
       const targetNorm = normalize(dm.id.replace(/^cline-pass\//, ''))
       const curated = CLINE_MODELS.find(
         (cm) => cm.id === dm.id || normalize(cm.id.replace(/^cline-pass\//, '')) === targetNorm || normalize(cm.name) === targetNorm
@@ -476,9 +506,18 @@ export async function loadModelsDiskCache(cachePath) {
     const fs = await import('node:fs/promises')
     const raw = await fs.readFile(cachePath, 'utf8')
     const parsed = JSON.parse(raw)
-    const list = Array.isArray(parsed?.models) ? parsed.models : (Array.isArray(parsed) ? parsed : null)
-    if (!list) return null
+    const rawList = Array.isArray(parsed?.models) ? parsed.models : (Array.isArray(parsed) ? parsed : null)
+    if (!rawList) return null
+    let hasObsolete = false
+    const list = rawList.map((m) => {
+      const migrated = migrateModelEntry(m)
+      if (migrated.id !== m.id) hasObsolete = true
+      return migrated
+    })
     list.planSyncedAt = typeof parsed?.planSyncedAt === 'number' ? parsed.planSyncedAt : (typeof parsed?.savedAt === 'number' ? parsed.savedAt : 0)
+    if (hasObsolete) {
+      saveModelsDiskCache(cachePath, list, list.planSyncedAt).catch(() => {})
+    }
     return list
   } catch {
     return null

package/lib/provider-sync.js

@@ -217,12 +217,26 @@ export async function autoDiscoverPlanModels(ctx, { live, getSettingsApi, syncPr
     const cacheFile = resolvePathWithHome(pub.modelsCachePath)
     const settingsApi = getSettingsApi()
 
+    // Auto-migrate obsolete IDs (e.g. deepseek-v41-flash -> deepseek-v4.1-flash)
+    const existingDynamic = pub.dynamicModels || []
+    const hasObsolete = existingDynamic.some((m) => Boolean(OBSOLETE_MODEL_ID_MAP[m?.id]))
+    if (hasObsolete && settingsApi?.replace) {
+      const migrated = existingDynamic.map(migrateModelEntry)
+      const next = Config({
+        ...plainConfig(live()),
+        dynamicModels: migrated,
+      })
+      await settingsApi.replace(next)
+      await syncProviderState(next)
+    }
+
     if ((!pub.dynamicModels || !pub.dynamicModels.length) && cacheFile) {
       const fromDisk = await loadModelsDiskCache(cacheFile)
       if (Array.isArray(fromDisk) && fromDisk.length && settingsApi?.replace) {
+        const migratedDisk = fromDisk.map(migrateModelEntry)
         const next = Config({
           ...plainConfig(live()),
-          dynamicModels: fromDisk,
+          dynamicModels: migratedDisk,
           planSyncedAt: fromDisk.planSyncedAt || Date.now(),
         })
         await settingsApi.replace(next)

package/lib/slash-command.js

@@ -22,6 +22,7 @@ import {
 } from './cline-client.js'
 
 export function registerSlashCommand(ctx, { live, getSettingsApi, syncProviderState }) {
+  if (typeof ctx.inject !== 'function') return
   ctx.inject(['commands'], (cmdCtx) => {
     const commands = cmdCtx.commands
     if (typeof commands?.register !== 'function') return

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goodandready/dsh-clinebot",
-  "version": "0.4.4",
+  "version": "0.4.6",
   "description": "DeepSeek Harness companion for ClineBot / ClinePass: dynamic subscription models sync, quota exhaustion warnings, session metrics, dedicated settings page, live usage limits, and /cline slash-command.",
   "license": "MIT",
   "type": "module",
@@ -18,7 +18,8 @@
     "README.ru.md",
     "README.zh.md",
     "CHANGELOG.md",
-    "LICENSE"
+    "LICENSE",
+    "media/visual-verification.png"
   ],
   "keywords": [
     "dsh",