dsh-dafeiyu-emotion 0.2.6 → 0.2.8

package/CHANGELOG.md

@@ -1,5 +1,16 @@
 # Changelog
 
+## 0.2.8
+
+- Fixed `backendRoot` showing as `[object Object]` on the settings page. DSH projects volatile `Config` fields as `{ get() }` refs; `readSettings()` and `resolveOptions()` now unwrap them before path/string/number handling.
+- Added `plainConfig` regression coverage so a volatile ref can never be stringified into a fake path again.
+
+## 0.2.7
+
+- Fixed a host activation failure: `apply` read `ctx.settings` while the plugin only declared `webServer`. Cordis throws `cannot get property "settings" without inject`, so the whole host plugin failed to mount and `/plugins/dsh-dafeiyu-emotion/config` returned 404.
+- Added `settings` to the declared inject list so the host can mount and register the health, analyze and config routes.
+- Added a regression assertion that both `webServer` and `settings` are declared dependencies.
+
 ## 0.2.6
 
 - Fixed a blocking `Config` validation error: the 12 MiB `maxAudioBytes` default violated the old 1 MiB step relative to the 256 KiB minimum. Removed the schema step so the shipped default and the settings inputs validate.
@@ -17,7 +28,7 @@
 
 - Exported the Schemastery `Config` schema so DSH can persist setting changes from the panel.
 - Marked every config field volatile, matching the DSH settings service contract.
-- 0.2.4 shipped the settings UI; on hosts that require a registered volatile schema, saving failed. Install 0.2.5 for the full read/write path.
+- 0.2.4 shipped the settings UI; on hosts that require a registered volatile schema, saving failed. 0.2.5 added the schema but its 1 MiB step rejected the 12 MiB default; install 0.2.6 instead.
 
 ## 0.2.4
 

package/README.md

@@ -111,9 +111,9 @@ dsh plugin --profile desktop add github:LiDu-china/dsh-dafeiyu-emotion
 ### 通过 Release tarball
 
 ```bash
-curl -L -o dsh-dafeiyu-emotion-0.2.6.tgz \
-  https://github.com/LiDu-china/dsh-dafeiyu-emotion/releases/download/v0.2.6/dsh-dafeiyu-emotion-0.2.6.tgz
-dsh plugin --profile desktop add "D:/path/to/dsh-dafeiyu-emotion-0.2.6.tgz"
+curl -L -o dsh-dafeiyu-emotion-0.2.8.tgz \
+  https://github.com/LiDu-china/dsh-dafeiyu-emotion/releases/download/v0.2.8/dsh-dafeiyu-emotion-0.2.8.tgz
+dsh plugin --profile desktop add "D:/path/to/dsh-dafeiyu-emotion-0.2.8.tgz"
 ```
 
 > DSH Desktop 的 profile 由 Electron 应用管理；命令行安装请使用桌面应用自带的 `dsh` 命令。

package/docs/MARKETPLACE.md

@@ -72,7 +72,7 @@ https://github.com/LiDu-china/dsh-dafeiyu-emotion/releases/download/v0.2.3/dsh-d
 dsh plugin --profile desktop add dsh-dafeiyu-emotion
 ```
 
-Published as `dsh-dafeiyu-emotion@0.2.6` on npm. The 1024Store entry refreshes its one-click install automatically. GitHub-source and Release-tarball installs remain available as fallbacks.
+Published as `dsh-dafeiyu-emotion@0.2.8` on npm. The 1024Store entry refreshes its one-click install automatically. GitHub-source and Release-tarball installs remain available as fallbacks.
 
 ## Privacy
 

package/docs/RELEASE-0.2.7.md

@@ -0,0 +1,15 @@
+## 0.2.7
+
+修复设置页显示“设置暂时不可用 / 读取设置失败（404）”的 Host 激活问题。
+
+- 0.2.5/0.2.6 在 `apply` 中读取了 `ctx.settings`，但插件只声明了 `webServer` 依赖。Cordis 会抛 `cannot get property "settings" without inject`，导致整个 Host 插件没有挂载，`/plugins/dsh-dafeiyu-emotion/config` 返回 404。
+- 0.2.7 把 `settings` 加入声明依赖，Host 现在可以正常注册 health、analyze 和 config 路由。
+- 新增回归断言：`webServer` 和 `settings` 必须同时出现在 `inject` 中。
+
+## 安装
+
+```bash
+dsh plugin --profile desktop add dsh-dafeiyu-emotion
+```
+
+升级后请完全退出并重新启动 the agent harness，让 Host 重新加载插件。

package/docs/RELEASE-0.2.8.md

@@ -0,0 +1,16 @@
+## 0.2.8
+
+修复设置页把增强后端目录显示成 `[object Object]` 的问题。
+
+- DSH 的 settings 服务会把 `Config` 的 volatile 字段投影成 `{ get() }` 引用。
+- 0.2.7 直接对引用做 `String()`，得到 `[object Object]`，再被 `resolve()` 拼成 `C:\...\profiles\desktop\[object Object]`，于是状态卡显示“后端目录不存在”。
+- 0.2.8 在 `readSettings()` 和 `resolveOptions()` 中统一用 `plainConfig()` 递归解包，再处理路径、语言和数字。
+- 新增回归测试，确保 volatile ref 不会被字符串化成假路径。
+
+## 安装
+
+```bash
+dsh plugin --profile desktop add dsh-dafeiyu-emotion
+```
+
+升级后请完全退出并重新启动 the agent harness，让 Host 重新读取 0.2.8。

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-dafeiyu-emotion",
-  "version": "0.2.6",
+  "version": "0.2.8",
   "description": "大肥鱼语音情绪耳朵：听语气、识别情绪，并让 DSH 按对方当下的状态自然回复。 · Voice + emotion-aware companion for DeepSeek Harness.",
   "type": "module",
   "main": "src/index.js",

package/scripts/test-analyze-handler.mjs

@@ -2,7 +2,10 @@ import assert from 'node:assert/strict'
 import { EventEmitter } from 'node:events'
 import { Readable } from 'node:stream'
 
-import { apply } from '../src/index.js'
+import { apply, inject } from '../src/index.js'
+
+assert.ok(inject.includes('webServer'), 'webServer must be a declared dependency')
+assert.ok(inject.includes('settings'), 'settings must be a declared dependency')
 
 const ANALYZE = '/plugins/dsh-dafeiyu-emotion/analyze'
 const HEALTH = '/plugins/dsh-dafeiyu-emotion/health'

package/scripts/test-config-schema.mjs

@@ -1,6 +1,6 @@
 import assert from 'node:assert/strict'
 
-import { Config } from '../src/index.js'
+import { Config, plainConfig, resolveOptions } from '../src/index.js'
 
 // The DSH settings service parses the schema with its defaults before a plugin
 // row can become configurable. A default that violates its own step/min/max
@@ -50,4 +50,30 @@ assert.throws(
   'unknown languages must be rejected by the schema',
 )
 
+// DSH projects volatile fields as { get() } refs. They must be unwrapped
+// before path resolution; otherwise backendRoot becomes "[object Object]".
+{
+  const ref = (value) => ({ get: () => value })
+  assert.deepEqual(
+    plainConfig({
+      backendRoot: ref('D:/voice-sense'),
+      nested: { language: ref('auto') },
+    }),
+    {
+      backendRoot: 'D:/voice-sense',
+      nested: { language: 'auto' },
+    },
+  )
+  const options = resolveOptions({
+    backendRoot: ref('D:/voice-sense'),
+    pythonPath: ref(''),
+    language: ref('auto'),
+    maxAudioBytes: ref(8 * 1024 * 1024),
+  })
+  assert.equal(options.backendRoot.includes('[object Object]'), false)
+  assert.ok(options.backendRoot.endsWith('voice-sense'))
+  assert.equal(options.language, 'auto')
+  assert.equal(options.maxAudioBytes, 8 * 1024 * 1024)
+}
+
 console.log('config schema tests passed')

package/src/index.js

@@ -8,7 +8,7 @@ import { fileURLToPath } from 'node:url'
 import Schema from '@deepseek-ai/schemastery'
 
 export const name = 'dsh-dafeiyu-emotion'
-export const inject = ['webServer']
+export const inject = ['webServer', 'settings']
 
 const ANALYZE_ENDPOINT = '/plugins/dsh-dafeiyu-emotion/analyze'
 const HEALTH_ENDPOINT = '/plugins/dsh-dafeiyu-emotion/health'
@@ -137,17 +137,32 @@ function asNumber(value, fallback) {
   return Number.isFinite(numeric) ? numeric : fallback
 }
 
+/**
+ * DSH settings projects volatile Config fields as live refs (`{ get() }`).
+ * Unwrap them before path/string/number handling, otherwise String(ref)
+ * becomes "[object Object]" and a path is resolved into the profile cwd.
+ */
+function plainConfig(value) {
+  if (value && typeof value.get === 'function') return plainConfig(value.get())
+  if (Array.isArray(value)) return value.map(plainConfig)
+  if (value && typeof value === 'object') {
+    return Object.fromEntries(Object.entries(value).map(([key, child]) => [key, plainConfig(child)]))
+  }
+  return value
+}
+
 function resolveOptions(raw = {}) {
+  const value = plainConfig(raw)
   return {
-    enabled: raw.enabled !== false,
-    backendRoot: String(raw.backendRoot || process.env.DSH_DAFEIYU_EMOTION_BACKEND || DEFAULTS.backendRoot).trim()
-      ? resolve(String(raw.backendRoot || process.env.DSH_DAFEIYU_EMOTION_BACKEND || DEFAULTS.backendRoot))
+    enabled: value.enabled !== false,
+    backendRoot: String(value.backendRoot || process.env.DSH_DAFEIYU_EMOTION_BACKEND || DEFAULTS.backendRoot).trim()
+      ? resolve(String(value.backendRoot || process.env.DSH_DAFEIYU_EMOTION_BACKEND || DEFAULTS.backendRoot))
       : '',
-    pythonPath: String(raw.pythonPath || '').trim(),
-    language: ['zh', 'en', 'auto'].includes(raw.language) ? raw.language : DEFAULTS.language,
-    maxAudioBytes: Math.min(32 * 1024 * 1024, Math.max(256 * 1024, asNumber(raw.maxAudioBytes, DEFAULTS.maxAudioBytes))),
-    startupTimeoutMs: Math.min(600000, Math.max(10000, asNumber(raw.startupTimeoutMs, DEFAULTS.startupTimeoutMs))),
-    requestTimeoutMs: Math.min(600000, Math.max(10000, asNumber(raw.requestTimeoutMs, DEFAULTS.requestTimeoutMs))),
+    pythonPath: String(value.pythonPath || '').trim(),
+    language: ['zh', 'en', 'auto'].includes(value.language) ? value.language : DEFAULTS.language,
+    maxAudioBytes: Math.min(32 * 1024 * 1024, Math.max(256 * 1024, asNumber(value.maxAudioBytes, DEFAULTS.maxAudioBytes))),
+    startupTimeoutMs: Math.min(600000, Math.max(10000, asNumber(value.startupTimeoutMs, DEFAULTS.startupTimeoutMs))),
+    requestTimeoutMs: Math.min(600000, Math.max(10000, asNumber(value.requestTimeoutMs, DEFAULTS.requestTimeoutMs))),
   }
 }
 
@@ -209,7 +224,7 @@ function readSettings(service) {
   try {
     const entry = service.describe().find((row) => row?.ns === SETTINGS_NAMESPACE)
     if (!entry) return null
-    return { ...(entry.value || {}), ...(entry.user || {}) }
+    return plainConfig({ ...(entry.value || {}), ...(entry.user || {}) })
   } catch {
     return null
   }
@@ -1032,6 +1047,8 @@ export {
   createConfigHandler,
   decorateAnalysis,
   normalizeConfigPatch,
+  plainConfig,
   publicConfig,
+  resolveOptions,
   transcribeWithSpeech,
 }