dsh-my-observability 0.3.2 → 0.3.4

package/CHANGELOG.md

@@ -2,6 +2,24 @@
 
 本文件记录 dsh-my-observability 的所有版本变更。格式遵循 [Keep a Changelog](https://keepachangelog.com/zh-CN/1.1.0/)，版本号遵循 [语义化版本](https://semver.org/lang/zh-CN/)。
 
+## [0.3.4] - 2026-09-18
+
+### 变更
+
+- fix(my-observability): 去掉仅内部使用的 4 个导出（knip 死代码门禁）
+- docs(guard,observability,mermaid-render): 设置页面板真实截图 + README 效果图
+- fix(my-observability): 拆分设置视图至尺寸门禁内
+- feat(my-observability): 设置页配置面板（aiReview / aiTimeoutMs 可视化编辑）
+
+## [0.3.3] - 2026-09-18
+
+### 变更
+
+- fix(my-observability): 去掉仅内部使用的 4 个导出（knip 死代码门禁）
+- docs(guard,observability,mermaid-render): 设置页面板真实截图 + README 效果图
+- fix(my-observability): 拆分设置视图至尺寸门禁内
+- feat(my-observability): 设置页配置面板（aiReview / aiTimeoutMs 可视化编辑）
+
 ## [0.3.2] - 2026-09-17
 
 ### 变更

package/README.md

@@ -5,6 +5,8 @@
 <div align="center">
   <img alt="轨迹回放面板：侧边栏时间轴展示 agent 行为（状态/模型流/工具调用）" src="https://unpkg.com/dsh-my-observability/assets/replay-panel.png" width="640" />
   <img alt="Git 工具面板：仓库状态 / 差异 / 提交前审查 / 类型化提交" src="https://unpkg.com/dsh-my-observability/assets/git-panel.png" width="640" />
+  <br/>
+  <img alt="设置页配置面板（设置 → 插件 → 可观测性）：AI 审查增强开关 + AI 审查超时" src="https://unpkg.com/dsh-my-observability/assets/settings-panel.png" width="640" />
 </div>
 
 **dsh-my-observability**：DSH 可观测性 + Git 工程工具插件——**事件审计**记录 agent 的每一次行为（状态变化 / 模型流 / 工具调用与结果），**轨迹回放面板**按时间轴回放会话轨迹；**结构化 Git 工具**提供 Conventional Commits 类型化提交；**增量 diff 审查**在提交前用规则引擎（可选 AI 增强）检查调试残留、密钥泄露等问题。
@@ -38,6 +40,8 @@ server 端改动需重启 `dsh web`；client 端改动硬刷新浏览器（Cmd/C
 | `aiReview`    | 增量 diff 审查启用 AI 增强（默认 `true`；agents 服务不可用时自动降级） |
 | `aiTimeoutMs` | AI 审查超时（毫秒，默认 60000）                                        |
 
+`aiReview` 与 `aiTimeoutMs` 可在 **设置 → 插件 → 可观测性** 里可视化编辑：保存即写回 profile 层 `cordis.patch.yml` 的插件行（行 id `observability`，其余手写键原样保留），DSH 热重载后立即生效，无需重启。其余字段（`aiProvider` / `aiModel` / `aiCwd` / `resourceIntervalMs` 等）仍按上表在 `cordis.patch.yml` 手写；非法值不会写坏配置（`aiReview` 非布尔忽略，`aiTimeoutMs` 非正/非有限回退 60000）。
+
 ## 限制与说明
 
 - **Git 工具作用于本机仓库**：仓库路径由你在面板输入（localStorage 记住），仅本机可访问（loopback 围栏）。

package/lib/client.js

@@ -37,8 +37,7 @@ window.__ModuleLoader__.load({
     Object.defineProperty(exports, Symbol.toStringTag, { value: 'Module' })
     const { createElement, useEffect, useState } = require('react')
 
-    // 原生页签身份（issue #187 批 1）：id/kind = 迁移前 better-sidebar 的 tab id
-    // （每个面板一个类型，id 全局唯一且是席位的 key），order = 迁移前的指南页顺序。
+    // 原生页签身份（issue #187 批 1）：id/kind 沿用迁移前 better-sidebar 的 tab id（每面板一个类型，id 全局唯一且是席位 key），order 为指南页顺序。
     const REPLAY_TAB_ID = 'dsh-my-observability:replay'
     const GIT_TAB_ID = 'dsh-my-observability:git'
     const REPLAY_TAB_ORDER = 40
@@ -194,6 +193,23 @@ const strings = {
     loading: () => (isZh() ? '加载中…' : 'Loading…'),
     emptyDiff: () => (isZh() ? '（空）' : '(empty)'),
     noRepo: () => (isZh() ? '请输入仓库路径' : 'Enter a repo path'),
+    // 设置页（设置 → 插件 → 可观测性，issue #383）
+    settingsTitle: () => (isZh() ? '可观测性' : 'Observability'),
+    settingsSectionTitle: () => (isZh() ? 'AI 审查' : 'AI review'),
+    settingsAiReviewLabel: () => (isZh() ? 'AI 审查增强' : 'AI review enhancement'),
+    settingsAiReviewHint: () => isZh()
+        ? '增量 diff 审查调用 AI agent 补充规则引擎结论（agents 服务不可用或超时自动降级为纯规则）'
+        : 'Let an AI agent augment the rule-engine review of the incremental diff (degrades to rules only when the agents service is unavailable)',
+    settingsAiTimeoutLabel: () => (isZh() ? 'AI 审查超时（ms）' : 'AI review timeout (ms)'),
+    settingsAiTimeoutHint: () => isZh()
+        ? '单次 AI 审查的最长等待；非正数或非有限值在保存时回退为 60000'
+        : 'Max wait per AI review; non-positive or non-finite values fall back to 60000 on save',
+    settingsSave: () => (isZh() ? '保存' : 'Save'),
+    settingsSaved: () => (isZh() ? '已保存并生效' : 'Saved and applied'),
+    settingsSaveFailed: () => (isZh() ? '保存失败' : 'Save failed'),
+    settingsLoadFailedHint: () => isZh()
+        ? '无法读取 /observability/api/config：请确认服务端插件已加载（改过 server 端后需重启 dsh web）'
+        : 'Cannot read /observability/api/config: make sure the host half is loaded (restart dsh web after host-side changes)',
 };
 
     // ── shared icons (inline, stroke=currentColor, matching better-sidebar) ──
@@ -1632,6 +1648,167 @@ function GitPanel() {
     }), createElement(DiffView, { diff }), createElement(ReviewReport, { report }), createElement(CommitForm, { repo, onCommitted }));
 }
 
+    "use strict";
+// ── 设置页签（issue #383）：设置 → 插件 → 可观测性 ────────────────────
+// 只暴露 README 已记录的 aiReview / aiTimeoutMs 两项；其余键（aiProvider /
+// aiModel / aiCwd / resourceIntervalMs / resourceLimits）继续由用户在
+// cordis.patch.yml 手写（服务端写回时会合并保留原条目已存在的键）。
+//
+// 保存 → PUT /observability/api/config → 写回 profile patch 文件；DSH 的
+// watchUserPatches 热重载 patch 后重新 apply（保存即生效）。保存成功后面板
+// 只更新本地状态：这两项在服务端生效，前端无缓存需要同步。
+//
+// 片段文件：无 import/export，共享 factory 作用域（apiJson / strings /
+// createElement 由 replay.js / i18n.js / 模板提供）。
+const SETTINGS_API = '/observability/api/config';
+/** 设置页样式：只用自己的类名前缀 + 宿主语义 token（随 effect 注入/卸载）。 */
+const SETTINGS_STYLES = `
+.dsh-my-observability-settings{display:flex;flex-direction:column;gap:10px;padding:12px}
+.dsh-my-observability-settings-section{display:flex;flex-direction:column;gap:8px}
+.dsh-my-observability-settings-section-title{font:var(--dsw-font-xs-strong-13);color:var(--dsw-alias-label-secondary)}
+.dsh-my-observability-settings-row{display:flex;align-items:center;justify-content:space-between;gap:8px;padding:8px 10px;border:1px solid var(--dsw-alias-border-l2);border-radius:8px;background:var(--dsw-alias-bg-layer-2)}
+.dsh-my-observability-settings-info{display:flex;flex-direction:column;gap:2px;min-width:0}
+.dsh-my-observability-settings-label{font:var(--dsw-font-xs-strong-13);color:var(--dsw-alias-label-primary)}
+.dsh-my-observability-settings-hint{font:var(--dsw-font-xxs-12);color:var(--dsw-alias-label-tertiary);line-height:1.5}
+.dsh-my-observability-settings-toggle{flex:none;width:34px;height:20px;border-radius:10px;border:1px solid var(--dsw-alias-border-l2);background:color-mix(in srgb, var(--dsw-alias-label-tertiary) 30%, transparent);position:relative;cursor:pointer;transition:background var(--ds-transition-duration-slow) var(--ds-ease-in-out),border-color var(--ds-transition-duration-slow) var(--ds-ease-in-out)}
+.dsh-my-observability-settings-toggle[data-on="true"]{background:var(--dsw-alias-state-success-primary);border-color:transparent}
+.dsh-my-observability-settings-toggle::after{content:"";position:absolute;top:2px;left:2px;width:14px;height:14px;border-radius:50%;background:var(--dsw-alias-label-primary);transition:transform var(--ds-transition-duration-slow) var(--ds-ease-in-out),background var(--ds-transition-duration-slow) var(--ds-ease-in-out)}
+.dsh-my-observability-settings-toggle[data-on="true"]::after{transform:translateX(12px);background:var(--dsw-alias-label-primary-foreground)}
+.dsh-my-observability-settings-input{flex:none;width:120px;height:28px;padding:0 8px;border-radius:6px;border:1px solid var(--dsw-alias-border-l2);background:var(--dsw-alias-bg-layer-1);color:var(--dsw-alias-label-primary);font:var(--dsw-font-xxs-12)}
+.dsh-my-observability-settings-input:focus{outline:none;border-color:var(--dsw-alias-interactive-primary)}
+.dsh-my-observability-settings-actions{display:flex;align-items:center;gap:8px}
+.dsh-my-observability-settings-btn{height:28px;padding:0 14px;border-radius:6px;cursor:pointer;border:1px solid var(--dsw-alias-border-l2);background:var(--dsw-alias-interactive-bg);color:var(--dsw-alias-label-primary);font:var(--dsw-font-xxs-12)}
+.dsh-my-observability-settings-btn:hover{background:var(--dsw-alias-interactive-bg-hover)}
+.dsh-my-observability-settings-status{font:var(--dsw-font-xxs-12);color:var(--dsw-alias-label-tertiary)}
+.dsh-my-observability-settings-saved{font:var(--dsw-font-xxs-12);color:var(--dsw-alias-state-success-primary)}
+.dsh-my-observability-settings-error{font:var(--dsw-font-xxs-12);color:var(--dsw-alias-state-error-primary)}
+`;
+/** 开关行（布尔配置项）。 */
+function SettingsToggleRow({ label, hint, on, onChange, }) {
+    return createElement('div', { className: 'dsh-my-observability-settings-row' }, createElement('div', { className: 'dsh-my-observability-settings-info' }, createElement('div', { className: 'dsh-my-observability-settings-label' }, label), createElement('div', { className: 'dsh-my-observability-settings-hint' }, hint)), createElement('div', {
+        className: 'dsh-my-observability-settings-toggle',
+        'data-on': String(on),
+        role: 'switch',
+        'aria-checked': String(on),
+        onClick: () => onChange(!on),
+    }));
+}
+/** 数字输入行（AI 审查超时 ms）。 */
+function SettingsNumberRow({ label, hint, value, onChange, }) {
+    return createElement('div', { className: 'dsh-my-observability-settings-row' }, createElement('div', { className: 'dsh-my-observability-settings-info' }, createElement('div', { className: 'dsh-my-observability-settings-label' }, label), createElement('div', { className: 'dsh-my-observability-settings-hint' }, hint)), createElement('input', {
+        className: 'dsh-my-observability-settings-input',
+        type: 'number',
+        min: '1',
+        step: '1000',
+        value: String(value),
+        onChange: (e) => onChange(Number(e.target.value)),
+    }));
+}
+/** 保存配置：PUT 配置端点；失败只显示失败提示（绝不误报已保存）。 */
+function saveSettings(draft, setSaved, setSaveError) {
+    setSaved(false);
+    setSaveError(false);
+    apiJson(SETTINGS_API, {
+        method: 'PUT',
+        headers: { 'content-type': 'application/json' },
+        body: JSON.stringify(draft),
+    })
+        .then(() => setSaved(true))
+        .catch(() => setSaveError(true));
+}
+/** 配置加载失败视图（读不到配置时不显示表单，避免用默认值覆盖真实配置）。 */
+function SettingsLoadError({ onRetry }) {
+    return createElement('div', { className: 'dsh-my-observability-settings' }, createElement('div', { className: 'dsh-my-observability-settings-error' }, strings.loadError()), createElement('div', { className: 'dsh-my-observability-settings-status' }, strings.settingsLoadFailedHint()), createElement('div', { className: 'dsh-my-observability-settings-actions' }, createElement('button', { className: 'dsh-my-observability-settings-btn', onClick: onRetry }, strings.retry())));
+}
+/** 加载态（配置未返回前的占位，避免先渲染出默认值表单再被回填覆盖）。 */
+function SettingsLoading() {
+    return createElement('div', { className: 'dsh-my-observability-settings' }, createElement('div', { className: 'dsh-my-observability-settings-status' }, strings.loading()));
+}
+/** 配置区块：区块标题 + 两个配置行（AI 审查开关 / 超时）。 */
+function SettingsConfigSection({ draft, patch, }) {
+    return createElement('div', { className: 'dsh-my-observability-settings-section' }, createElement('div', { className: 'dsh-my-observability-settings-section-title' }, strings.settingsSectionTitle()), createElement(SettingsToggleRow, {
+        label: strings.settingsAiReviewLabel(),
+        hint: strings.settingsAiReviewHint(),
+        on: draft.aiReview !== false,
+        onChange: (value) => patch('aiReview', value),
+    }), createElement(SettingsNumberRow, {
+        label: strings.settingsAiTimeoutLabel(),
+        hint: strings.settingsAiTimeoutHint(),
+        value: draft.aiTimeoutMs,
+        onChange: (value) => patch('aiTimeoutMs', value),
+    }));
+}
+/** 保存区块：保存按钮 + 成功/失败提示（互斥，保存失败绝不显示"已保存"）。 */
+function SettingsActions({ save, saved, saveError, }) {
+    return createElement('div', { className: 'dsh-my-observability-settings-actions' }, createElement('button', { className: 'dsh-my-observability-settings-btn', onClick: save }, strings.settingsSave()), saved ? createElement('span', { className: 'dsh-my-observability-settings-saved' }, strings.settingsSaved()) : null, saveError
+        ? createElement('span', { className: 'dsh-my-observability-settings-error' }, strings.settingsSaveFailed())
+        : null);
+}
+/** 设置页视图：GET 回填 → 编辑 → 保存（各区块拆成子组件，控制单函数长度）。 */
+function ObservabilitySettingsView() {
+    const [draft, setDraft] = useState(null);
+    const [loading, setLoading] = useState(true);
+    const [failed, setFailed] = useState(false);
+    const [saved, setSaved] = useState(false);
+    const [saveError, setSaveError] = useState(false);
+    const load = () => {
+        setLoading(true);
+        setSaved(false);
+        setSaveError(false);
+        apiJson(SETTINGS_API)
+            .then((value) => {
+            setDraft(value);
+            setLoading(false);
+        })
+            .catch(() => {
+            setLoading(false);
+            setFailed(true);
+        });
+    };
+    useEffect(() => {
+        load();
+    }, []);
+    if (loading)
+        return createElement(SettingsLoading, null);
+    if (draft === null)
+        return createElement(SettingsLoadError, { onRetry: load });
+    const patch = (key, value) => setDraft({ ...draft, [key]: value });
+    return createElement('div', { className: 'dsh-my-observability-settings' }, createElement(SettingsConfigSection, { draft, patch }), createElement(SettingsActions, {
+        save: () => saveSettings(draft, setSaved, setSaveError),
+        saved,
+        saveError,
+    }));
+}
+/** 附加设置页签：slots 服务缺失（精简上下文 / 宿主未提供）时静默跳过。 */
+function attachObservabilitySettingsTab(ctx) {
+    // strict=false：首屏时 slots 提供者 fiber 可能尚未 active，strict 模式的
+    // ctx.get 返回 undefined → 页签静默消失（要等 HMR 才出现）。取到实例即可，
+    // 注册本身由 slots.inject 等待槽位声明。
+    const slots = typeof ctx.get === 'function' ? ctx.get('slots', false) : undefined;
+    if (slots === undefined || slots === null)
+        return;
+    ctx.effect(() => {
+        if (typeof document === 'undefined' || typeof document.head === 'undefined')
+            return () => { };
+        const style = document.createElement('style');
+        style.setAttribute('data-dsh-my-observability-settings', 'styles');
+        style.textContent = SETTINGS_STYLES;
+        document.head.appendChild(style);
+        return () => {
+            if (style.parentNode !== null)
+                style.parentNode.removeChild(style);
+        };
+    }, 'dsh-my-observability: settings styles');
+    ctx.effect(() => slots.inject('settings.plugins.tab', () => slots.register({
+        name: 'settings.plugins.tab',
+        // id 必须全局唯一（list 型槽位的 tab key）：复用别人的 id 会顶掉
+        // 对方那一格（宿主契约：fresh id 追加在已有条目旁）。
+        id: 'dsh-my-observability-settings',
+        order: 92,
+        label: () => strings.settingsTitle(),
+    }, ObservabilitySettingsView)), 'dsh-my-observability: settings tab registration');
+}
+
     "use strict";
 // ── 样式（DSH 语义 token，随 activation 注入 / teardown 卸载）──────
 // 前缀 dsh-my-observability-（issue #54：与 dsh-my-guard 前缀分离，消除跨插件类名冲突）。
@@ -1810,8 +1987,10 @@ function injectStyles() {
       // 样式先注入：不依赖任何服务（服务缺失/时序未就绪也不影响面板配色）。
       ctx.effect(() => injectStyles(), 'dsh-my-observability: styles')
 
-      const tabs = ctx.sidebarRightTabs
-      const slots = ctx.slots
+      // 设置页签（issue #383）：只依赖 slots（strict=false 读取），与侧边栏面板所需的 sidebarRightTabs 无关，故先于判空注册。
+      attachObservabilitySettingsTab(ctx)
+
+      const { sidebarRightTabs: tabs, slots } = ctx
       // 判空必须同时覆盖 null 与 undefined（typeof null 是 object，会骗过 === undefined）。
       if (tabs == null || slots == null) return
 

package/lib/client.src.js

@@ -37,8 +37,7 @@ window.__ModuleLoader__.load({
     Object.defineProperty(exports, Symbol.toStringTag, { value: 'Module' })
     const { createElement, useEffect, useState } = require('react')
 
-    // 原生页签身份（issue #187 批 1）：id/kind = 迁移前 better-sidebar 的 tab id
-    // （每个面板一个类型，id 全局唯一且是席位的 key），order = 迁移前的指南页顺序。
+    // 原生页签身份（issue #187 批 1）：id/kind 沿用迁移前 better-sidebar 的 tab id（每面板一个类型，id 全局唯一且是席位 key），order 为指南页顺序。
     const REPLAY_TAB_ID = 'dsh-my-observability:replay'
     const GIT_TAB_ID = 'dsh-my-observability:git'
     const REPLAY_TAB_ORDER = 40
@@ -53,6 +52,7 @@ window.__ModuleLoader__.load({
     /*__PART_RESOURCE__*/
     /*__PART_REPLAY_EXT__*/
     /*__PART_GIT__*/
+    /*__PART_SETTINGS__*/
     /*__PART_STYLES__*/
 
     // ── 插件体：样式注入 + 两个原生页签注册 ─────────────────────────────
@@ -73,8 +73,10 @@ window.__ModuleLoader__.load({
       // 样式先注入：不依赖任何服务（服务缺失/时序未就绪也不影响面板配色）。
       ctx.effect(() => injectStyles(), 'dsh-my-observability: styles')
 
-      const tabs = ctx.sidebarRightTabs
-      const slots = ctx.slots
+      // 设置页签（issue #383）：只依赖 slots（strict=false 读取），与侧边栏面板所需的 sidebarRightTabs 无关，故先于判空注册。
+      attachObservabilitySettingsTab(ctx)
+
+      const { sidebarRightTabs: tabs, slots } = ctx
       // 判空必须同时覆盖 null 与 undefined（typeof null 是 object，会骗过 === undefined）。
       if (tabs == null || slots == null) return
 

package/lib/config-routes.js

@@ -0,0 +1,88 @@
+/**
+ * dsh-my-observability — 设置页配置端点（GET/PUT /observability/api/config，issue #383）。
+ *
+ * 设置 → 插件 → 可观测性 面板只暴露 README 已记录的 `aiReview` /
+ * `aiTimeoutMs` 两项；其余字段（aiProvider / aiModel / aiCwd /
+ * resourceIntervalMs / resourceLimits）继续由用户在 cordis.patch.yml 手写，
+ * 设置页保存不得连带覆盖它们。
+ *
+ * 写回复用 dsh-shared 的配置持久化原语（currentProfile / patchFileOf /
+ * writePatchConfig），并**先合并该条目已有键**：writePatchConfig 的语义是
+ * 「删除同 id 旧条目 → 追加新条目」，只写两项会把用户手写的其余配置一起删掉
+ * （数据破坏，issue #383 验收项「不写坏配置」）。
+ */
+import { readFile } from 'node:fs/promises';
+import { currentProfile, extractConfig, patchFileOf, readJsonBody, writeJson, writePatchConfig } from 'dsh-shared';
+/** aiTimeoutMs 默认值（与 src/index.ts buildOptions 的兜底共用同一口径）。 */
+const DEFAULT_AI_TIMEOUT_MS = 60000;
+/**
+ * 写回行 id：必须与 plugins/dsh-my-observability/cordis.patch.yml 里的插件行
+ * id 一致——loader 按行 id 匹配配置，id 不符会新增孤儿行、原行配置不变。
+ */
+const CONFIG_ROW_ID = 'observability';
+/** aiTimeoutMs 规整：非数字 / 非有限 / 非正 → 回退默认（绝不把 0/NaN 写进 patch）。
+ *  导出供 src/index.ts 的 buildOptions 复用，保证「默认值口径」只有一处定义。
+ *  （其余仅本文件内部使用的符号不导出：knip 会把无外部引用的导出判为 dead code。） */
+export function normalizeTimeout(value) {
+    return typeof value === 'number' && Number.isFinite(value) && value > 0 ? value : DEFAULT_AI_TIMEOUT_MS;
+}
+/** 当前生效配置（GET 的 value，也是设置页表单的回填源）。 */
+function configValueOf(options) {
+    return { aiReview: options.aiReview !== false, aiTimeoutMs: normalizeTimeout(options.aiTimeoutMs) };
+}
+/**
+ * 校验 + 规整 PUT payload：非对象 → undefined（调用方回 400）。
+ *
+ * 非法**字段**只忽略该字段、不整单拒绝：设置页一次提交两项，因一个字段非法
+ * 而丢掉另一项合法修改，用户会误以为保存无效（issue #383 验收项「非法值回退
+ * 默认」）；aiTimeoutMs 非法时回退默认而非保留旧值，与 README 记录的默认口径
+ * 一致。
+ */
+function normalizeConfigPatch(payload, current) {
+    if (payload === null || typeof payload !== 'object' || Array.isArray(payload))
+        return undefined;
+    const raw = payload;
+    const next = { aiReview: current.aiReview, aiTimeoutMs: current.aiTimeoutMs };
+    if (typeof raw.aiReview === 'boolean')
+        next.aiReview = raw.aiReview;
+    if (raw.aiTimeoutMs !== undefined)
+        next.aiTimeoutMs = normalizeTimeout(raw.aiTimeoutMs);
+    return next;
+}
+/** 读取 patch 文件中该行已有的 config 块（文件不存在 / 解析失败 → 空对象）。 */
+async function readExistingConfig(file) {
+    try {
+        return extractConfig(await readFile(file, 'utf8'), CONFIG_ROW_ID) ?? {};
+    }
+    catch {
+        return {};
+    }
+}
+/** 写回 profile 层 patch 文件：合并已有键后整体重写（保留用户手写的其他配置）。 */
+export async function persistSettingsConfig(config) {
+    const file = patchFileOf(currentProfile());
+    const existing = await readExistingConfig(file);
+    await writePatchConfig(file, CONFIG_ROW_ID, { ...existing, ...config });
+}
+/**
+ * 分派配置端点：GET 返回当前生效值；PUT 校验 → 写回 + 即时生效。
+ * 未匹配（含其他动词）返回 false，交由调用方回 404（保持既有路由契约）。
+ */
+export async function dispatchConfigRoutes(method, request, response, options, onChange) {
+    if (method !== 'config')
+        return false;
+    if (request.method === 'GET') {
+        writeJson(response, 200, { ok: true, value: configValueOf(options) });
+        return true;
+    }
+    if (request.method !== 'PUT')
+        return false;
+    const next = normalizeConfigPatch(await readJsonBody(request), configValueOf(options));
+    if (next === undefined) {
+        writeJson(response, 400, { ok: false, error: { message: 'invalid config' } });
+        return true;
+    }
+    await onChange(next);
+    writeJson(response, 200, { ok: true, value: next });
+    return true;
+}

package/lib/index.js

@@ -22,6 +22,7 @@
 import { createStore } from './store.js';
 import { attachAuditListeners } from './audit.js';
 import { registerObservabilityRoutes } from './routes.js';
+import { persistSettingsConfig, normalizeTimeout } from './config-routes.js';
 import { createResourceMonitor } from './resource-monitor.js';
 export const name = 'dsh-my-observability';
 export const inject = ['webServer'];
@@ -32,9 +33,26 @@ function buildOptions(config) {
         aiModel: config?.aiModel,
         aiCwd: config?.aiCwd,
         aiReview: config?.aiReview !== false,
-        aiTimeoutMs: Number.isFinite(config?.aiTimeoutMs) && config?.aiTimeoutMs > 0
-            ? config?.aiTimeoutMs
-            : 60000,
+        // 与设置页保存共用同一默认值口径（config-routes.normalizeTimeout）。
+        aiTimeoutMs: normalizeTimeout(config?.aiTimeoutMs),
+    };
+}
+/**
+ * 设置页保存回调（issue #383）：**先写回 profile patch 文件、再更新内存**——
+ * 写盘失败时保持内存旧值（内存与磁盘不劈叉），并把失败如实抛给路由层；
+ * DSH 的 watchUserPatches 会热重载 patch 文件，保存即生效。
+ */
+function settingsChangeHandler(ctx, options) {
+    return async (next) => {
+        try {
+            await persistSettingsConfig(next);
+        }
+        catch (error) {
+            ctx.logger.warn(`[dsh-my-observability] 配置保存失败（操作=config/save，原因=${error instanceof Error ? error.message : String(error)}）`);
+            throw error;
+        }
+        Object.assign(options, next);
+        ctx.logger.info?.(`[dsh-my-observability] 设置已保存（aiReview=${next.aiReview}, aiTimeoutMs=${next.aiTimeoutMs}）`);
     };
 }
 export function apply(ctx, config) {
@@ -59,8 +77,8 @@ export function apply(ctx, config) {
         },
     });
     monitor.start();
-    // ── 路由（查询 / git 工具 / diff 审查 / 资源）───────────────────────
-    registerObservabilityRoutes(ctx, store, monitor, options);
+    // ── 路由（查询 / git 工具 / diff 审查 / 资源 / 设置页配置）──────────
+    registerObservabilityRoutes(ctx, store, monitor, options, settingsChangeHandler(ctx, options));
     // ── 卸载冲刷：清防抖定时器 + 立即落盘 + 停采样 ──────────────────────
     ctx.effect(() => {
         monitor.stop();

package/lib/routes.js

@@ -9,14 +9,16 @@
  *  - GET  /git/diff?repo=&staged=  — 差异文本
  *  - POST /git/commit              — 类型化提交（Conventional Commits）
  *  - POST /review                  — 增量 diff 审查（规则引擎 + 可选 AI）
+ *  - GET/PUT /config               — 设置页配置读写（aiReview / aiTimeoutMs）
  */
 import { isTrustedApiRequest, readJsonBody, writeJson, writeError } from 'dsh-shared';
+import { dispatchConfigRoutes } from './config-routes.js';
 import { gitStatus, gitDiff, gitCommit } from './git.js';
 import { parseDiff } from './diff.js';
 import { reviewRules } from './review.js';
 import { runAiReview } from './ai.js';
 /** 注册 /observability/api 路由（effect 持有 disposer）。 */
-export function registerObservabilityRoutes(ctx, store, monitor, options) {
+export function registerObservabilityRoutes(ctx, store, monitor, options, onConfigChange) {
     const webRuntime = ctx.get ? ctx.get('webRuntime') : undefined;
     const trustedHosts = webRuntime !== undefined && webRuntime !== null && Array.isArray(webRuntime.trustedHosts)
         ? webRuntime.trustedHosts
@@ -25,11 +27,11 @@ export function registerObservabilityRoutes(ctx, store, monitor, options) {
     ctx.effect(() => ctx.webServer.register({
         kind: 'prefix',
         path: '/observability/api',
-        handler: apiHandler(ctx, fence, store, monitor, options),
+        handler: apiHandler(ctx, fence, store, monitor, options, onConfigChange),
     }), 'dsh-my-observability: /observability/api routes');
 }
 /** 统一 handler：fence → 方法分派 → 404/错误兜底。 */
-function apiHandler(ctx, fence, store, monitor, options) {
+function apiHandler(ctx, fence, store, monitor, options, onConfigChange) {
     return async (request, response) => {
         if (!fence(request)) {
             writeJson(response, 403, { ok: false, error: { code: 'forbidden', message: 'forbidden' } });
@@ -43,7 +45,7 @@ function apiHandler(ctx, fence, store, monitor, options) {
             // 状态（曾经的 CI flaky：固定 40ms sleep 赌 readFile 跑完，慢机器上
             // 查询读到 0 条事件 → 0 !== 1）。whenReady 就绪后立即 resolve，无延迟。
             await store.whenReady();
-            const handled = await dispatchMethod(method, request, response, url, ctx, store, monitor, options);
+            const handled = await dispatchMethod(method, request, response, url, ctx, store, monitor, options, onConfigChange);
             if (!handled) {
                 writeJson(response, 404, {
                     ok: false,
@@ -61,11 +63,12 @@ function isMethod(method, request, name, verb) {
     return method === name && request.method === verb;
 }
 /** 按 method 分派到具体 handler；未识别返回 false（调用方回 404）。 */
-async function dispatchMethod(method, request, response, url, ctx, store, monitor, options) {
-    const handled = await dispatchCore(method, request, response, url, ctx, store, monitor, options);
-    if (handled)
+async function dispatchMethod(method, request, response, url, ctx, store, monitor, options, onConfigChange) {
+    if (await dispatchCore(method, request, response, url, ctx, store, monitor, options))
         return true;
-    return dispatchExtended(method, request, response, url, ctx, store);
+    if (dispatchExtended(method, request, response, url, ctx, store))
+        return true;
+    return dispatchConfigRoutes(method, request, response, options, onConfigChange);
 }
 /** 核心路由（复杂度 ≤10）。 */
 async function dispatchCore(method, request, response, url, ctx, store, monitor, options) {
@@ -275,13 +278,8 @@ async function aiOutcome(ctx, payload, options, diffText, report) {
     });
 }
 // ── HTTP helpers ───────────────────────────────────────────────────────────
-function queryOf(url, name) {
-    return url.searchParams.get(name) ?? '';
-}
-function repoOf(url) {
-    const repo = url.searchParams.get('repo');
-    return typeof repo === 'string' ? repo : '';
-}
+const queryOf = (url, name) => url.searchParams.get(name) ?? '';
+const repoOf = (url) => url.searchParams.get('repo') ?? '';
 function limitOf(url) {
     const raw = url.searchParams.get('limit');
     const parsed = raw === null ? 0 : Number(raw);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-my-observability",
-  "version": "0.3.2",
+  "version": "0.3.4",
   "description": "DSH 可观测性 + Git 工程工具插件：会话轨迹回放（时间轴）、事件审计（agent 行为记录）、结构化 Git 提交（Conventional Commits）、增量 diff 审查（提交前审查）。DSH web plugin: trajectory replay timeline, event audit log, structured git commits, pre-commit incremental diff review.",
   "type": "module",
   "main": "lib/index.js",