dsh-chime-sound 0.3.2 → 0.3.3

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 0.3.3 (2026-09-16)
+
+- **`koffi` 改为运行时动态 `import()`**，不再在模块顶层静态引入。dsh 的插件加载器把「入口模块 import 失败」当作**整棵插件树启动失败**（`Cordis startup failed because these plugin(s) could not be resolved`），所以静态引入一个可选的原生 FFI 包会把「没声音」升级成「profile 起不来」。现在 koffi 解析失败只以降级处理（`warn` + 诊断日志留痕），插件本体照常加载，其它插件也不受影响。
+- **非 Windows 平台提前 `return`**：原先只发一条 warn 就继续往下走，于是仍会订阅 `session/event`、每个回合读一次 wav、再在 Win32 绑定处抛错被吞掉，并周期性重报服务缺失。现在平台门控在最前面直接返回，这些噪音消失。
+- 版本号 `0.3.3`；`dsh-plugin.json` 的 `version` 同步。
+
 ## 0.3.2 (2026-09-13)
 
 - 文档：README 顶部加 CI 状态徽章；发布章节写明走 npm 可信发布（OIDC），仓库内不存放令牌

package/README.md

@@ -4,6 +4,8 @@
 
 **English** · [中文](README.zh.md)
 
+Built for [dsh-TUI](https://github.com/ccch1mneyyy/dsh-TUI).
+
 Notification sounds for dsh-tui (originally named dsh-notice-sound, renamed on 2026-09-07).
 
 - **Triggers:** the agent calling `ask_user_question` (`session/event` `tool/call` — a nudge to come back and answer) and a reply finishing (`session/event` `turn/end`; agent-loop emits this once per turn, while `assistant/message` fires on every tool step and is deliberately not used). Both distinguish a focused from an unfocused terminal.

package/README.zh.md

@@ -4,6 +4,8 @@
 
 **中文** · [English](README.md)
 
+为 [dsh-TUI](https://github.com/ccch1mneyyy/dsh-TUI) 构建。
+
 dsh-tui 提示音效插件（原名 dsh-notice-sound，2026-09-07 更名）。
 
 - 触发：代理调用询问模块 `ask_user_question`（`session/event` `tool/call`，提醒用户回来回答问题）与回复完成（`session/event` `turn/end`，agent-loop 每回合仅发一次；`assistant/message` 会随每个工具步骤多次触发，不采用），并区分终端聚焦 / 未聚焦。

package/dsh-plugin.json

@@ -3,7 +3,7 @@
   "manifestVersion": "0.15",
   "id": "com.dsh-tui.chime",
   "name": "chime",
-  "version": "0.3.2",
+  "version": "0.3.3",
   "facets": {
     "host": {
       "entry": "lib/index.js",

package/lib/index.js

@@ -44,7 +44,6 @@ import { fileURLToPath } from 'node:url';
 import { appendFileSync, readFileSync } from 'node:fs';
 import { homedir } from 'node:os';
 import z from '@deepseek-ai/schemastery';
-import koffi from 'koffi';
 import { scaleWav } from './wav.js';
 
 const HERE = dirname(fileURLToPath(import.meta.url));
@@ -118,10 +117,20 @@ const RETRY_LIMIT = 60; // ~24s of service readiness attempts
 const POOL_SIZE = 8;
 const POOL_COOLDOWN_MS = 5000;
 
+/**
+ * koffi, resolved by a RUNTIME import in `apply()` — deliberately not a static
+ * one. The loader treats a failed entry import as a whole-tree boot failure
+ * (`assertEntriesLoaded` → "Cordis startup failed"), so a module-scope import
+ * of an optional native FFI package would turn "no sound" into "the profile
+ * does not start".
+ */
+let koffi = null;
+
 /** Lazy, shared Win32 bindings (koffi). */
 let win32 = null;
 function loadWin32() {
     if (win32 === null) {
+        if (koffi === null) throw new Error('koffi is not loaded (runtime import failed, or this is not a Win32 host)');
         const pvoid = koffi.pointer('void');
         const api = (lib, name, result, args) => lib.func('__stdcall', name, result, args);
         const user32 = koffi.load('user32.dll');
@@ -232,13 +241,26 @@ function registerSection(sections) {
  * this plugin, never the boot. Message events come from the session/event
  * bus (no Component identity required — see module header).
  */
-export function apply(ctx, config) {
+export async function apply(ctx, config) {
     diag('apply started');
+    // TUI-RUN-001: the plugin is Win32-only (winmm/user32 via koffi). The
+    // platform gate runs FIRST and RETURNS: on a platform that can never play,
+    // subscribing to session/event would only produce retry noise.
     if (process.platform !== 'win32') {
-        // TUI-RUN-001: the plugin is Win32-only (winmm/user32 via koffi); on
-        // other platforms it degrades to a no-op — never a boot failure.
         diag(`platform=${process.platform}: sounds disabled (Win32-only)`);
         ctx.logger.warn(`chime: Win32-only plugin (winmm/user32); notification sounds will not play on ${process.platform}`);
+        return;
+    }
+    // Runtime import, not a module-scope one: a missing koffi must degrade this
+    // plugin, never fail the whole plugin tree (see the note on `koffi` above).
+    try {
+        koffi = (await import('koffi')).default;
+    }
+    catch (error) {
+        const reason = error instanceof Error ? error.message : String(error);
+        diag(`koffi import failed: ${reason}`);
+        ctx.logger.warn(`chime: could not load koffi (${reason}); notification sounds are disabled`);
+        return;
     }
     ctx.effect(function* () {
         let scope = null;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-chime-sound",
-  "version": "0.3.2",
+  "version": "0.3.3",
   "description": "Notification sounds for dsh-tui: distinct tones for conversation done / user question, focused vs unfocused terminal, 9 selectable sound groups, master volume 0-150% in 5% steps, settings section, zh/en",
   "license": "MIT",
   "type": "module",