dsh-my-observability 0.3.0 → 0.3.2

package/lib/parts/native-tabs.js

@@ -0,0 +1,45 @@
+/**
+ * 原生侧边栏页签注册助手（issue #187 批 1；scripts/build.mjs 拼接片段）。
+ *
+ * 原生结构：每个面板一个**页签类型**（sidebarRightTabs.register）+ 两个
+ * **keyed 席位**（slots.register，body 与 title）。id 全局唯一且是席位的
+ * key，一个 key 只能注册一个席位 —— 本插件有两个独立面板，因此每个 kind 用
+ * 自己的 id（沿用迁移前 better-sidebar 的 tab id）。
+ *
+ * 片段无 import/export，共享 client.src.js 的 factory 作用域：本函数的参数
+ * 全部由调用方（apply）注入，便于单测与阅读。
+ */
+function registerNativeTab(ctx, tabs, slots, spec) {
+  ctx.effect(
+    () =>
+      tabs.register({
+        id: spec.id,
+        kind: spec.id,
+        title: spec.title,
+        guide: [{ order: spec.order, title: spec.title }],
+      }),
+    'dsh-my-observability: ' + spec.label + ' tab',
+  )
+  ctx.effect(
+    () =>
+      slots.inject('sidebar.right.pane.tab', () =>
+        slots.register({ name: 'sidebar.right.pane.tab', key: spec.id }, spec.Body),
+      ),
+    'dsh-my-observability: ' + spec.label + ' tab body',
+  )
+  ctx.effect(
+    () =>
+      slots.inject('sidebar.right.pane.tab.title', () =>
+        slots.register({ name: 'sidebar.right.pane.tab.title', key: spec.id }, spec.Title),
+      ),
+    'dsh-my-observability: ' + spec.label + ' tab title',
+  )
+}
+
+/** 原生 tab title 席位：宿主给定标题优先，否则回退面板自带文案。 */
+function nativeTabTitle(fallback) {
+  return function TabTitle(props) {
+    const info = typeof props.useTabInfo === 'function' ? props.useTabInfo() : undefined
+    return createElement('span', null, info?.tab?.title ?? fallback())
+  }
+}

package/lib/resource-monitor.js

@@ -1,149 +1,97 @@
 /**
- * dsh-my-observability — 资源采样监控 + 降级看门狗。
+ * dsh-my-observability — 资源采样监控 + 降级看门狗（**dsh-shared 原语的消费方**）。
  *
- * 每 intervalMs（默认 15s）采样本进程 CPU/内存与审计文件大小/写入速率，
- * 保留最近 MAX_HISTORY 个样本（ring buffer），并做阈值评估（resource-rules）。
- * 用途：让「写放大/资源超限」在运行期当小时可见（9/2 事故复盘结论：
- * 15 小时 300GB 写入零监控是事故未被及时发现的主因）。
+ * 第三批（issue #198）：本文件原本是插件私有的看门狗实现（采样 → 阈值判定 →
+ * 连续确认降级/恢复），现已改为 `dsh-shared` 的 `createResourceGuard` 消费方——
+ * 抽出来的原语必须有人真的用，否则等于没抽。
  *
- * 降级看门狗（issue #127 资源占用防护）：关键阈值（写放大/文件字节）连续
- * DEGRADE_CONFIRM_COUNT 次超限 → 触发 onDegrade 回调（宿主降级：停落盘等）；
- * 连续 RECOVER_CONFIRM_COUNT 次正常 → 触发 onRecover 回调（宿主恢复 + 全量快照）。
- * 判定为纯函数（resource-rules.shouldEnterDegrade/shouldExitDegrade），可单测。
+ * 保留的插件职责（宿主特有，不进 shared）：
+ *  - **采样源**：`createProcessSampler`（CPU/RSS/审计文件字节/写入速率）
+ *    + `$DSH_HOME` 目录总字节这个插件特有维度；
+ *  - **降级动作**：由 index.ts 的 onDegrade/onRecover 决定（停落盘 / 恢复 + 全量快照）。
  *
- * 采样自身开销：15s 一次 process.cpuUsage/memoryUsage + fs.stat（<0.01% CPU、
- * 零分配大对象），远低于「监控不能放大被监控对象」的护栏（resource-budget-review）。
+ * 行为等价（改造前后逐条对齐，见 test/host-resource-guard.mjs）：
+ *  - 采样间隔默认 15s；历史 ring buffer 60 样本；
+ *  - 首个样本无窗口 → 不告警、不进历史，仅作后续窗口的 prev；
+ *  - 关键阈值只有 write-rate / file-size；CPU/内存超限只告警不降级；
+ *  - 连续 3 次超限 → 降级；连续 3 次正常 → 恢复；降级中不重复触发回调。
+ *
+ * 资源开销：采样 15s 一次、<0.01% CPU；history 固定 60 条（不随运行时长增长）。
+ * 已知存量代价：`homeBytes` 维度每次采样递归扫 $DSH_HOME（O(文件数)），
+ * 新接入方不要复制这个维度（见 shared README 的采样边界）。
  */
-import { statSync, readdirSync } from 'node:fs'
-import { join } from 'node:path'
-import { homedir } from 'node:os'
-import { evaluateResourceAlerts, shouldEnterDegrade, shouldExitDegrade, DEFAULT_LIMITS } from './resource-rules.js'
-import { jsonlFile } from './store-persist.js'
-
-const DEFAULT_INTERVAL_MS = 15000
-const MAX_HISTORY = 60
-
-/** 创建资源监控器：{ sample, start, stop }。options: intervalMs / limits / onDegrade / onRecover。 */
+import { readdirSync, statSync } from 'node:fs';
+import { join } from 'node:path';
+import { homedir } from 'node:os';
+import { createProcessSampler, createResourceGuard } from 'dsh-shared';
+import { jsonlFile } from './store-persist.js';
+const DEFAULT_INTERVAL_MS = 15000;
+/** 创建资源监控器：{ sample, start, stop, isDegraded }（薄适配 dsh-shared 看门狗）。 */
 export function createResourceMonitor(ctx, options = {}) {
-  const intervalMs =
-    Number.isFinite(options.intervalMs) && options.intervalMs > 0 ? options.intervalMs : DEFAULT_INTERVAL_MS
-  const limits = { ...DEFAULT_LIMITS, ...(options.limits ?? {}) }
-  const onDegrade = typeof options.onDegrade === 'function' ? options.onDegrade : null
-  const onRecover = typeof options.onRecover === 'function' ? options.onRecover : null
-  const state = {
-    timer: null,
-    file: jsonlFile(),
-    dshHome: process.env.DSH_HOME || join(homedir(), '.dsh'),
-    lastSample: null,
-    lastCpu: process.cpuUsage(),
-    history: [],
-    degraded: false,
-  }
-  const monitor = {
-    sample: () => sample(state, limits, onDegrade, onRecover),
-    start: () => startMonitor(state, intervalMs, monitor),
-    stop: () => stopMonitor(state),
-    isDegraded: () => state.degraded,
-  }
-  return monitor
-}
-
-/** 采样一次：CPU 使用率（窗口内 user+sys）/RSS/审计文件字节/写入速率 + 告警 + 降级判定。 */
-function sample(state, limits, onDegrade, onRecover) {
-  const now = Date.now()
-  const cpu = process.cpuUsage()
-  const cpuDelta = cpu.user - state.lastCpu.user + (cpu.system - state.lastCpu.system) // µs
-  state.lastCpu = cpu
-  const mem = process.memoryUsage()
-  const memoryBytes = mem.rss
-  let fileBytes = 0
-  try {
-    fileBytes = statSync(state.file).size
-  } catch {
-    // 审计文件尚未创建：字节为 0
-  }
-  let homeBytes = 0
-  try {
-    homeBytes = dshHomeSize(state.dshHome)
-  } catch {
-    // $DSH_HOME 不可达
-  }
-  const prev = state.lastSample
-  if (prev !== null) {
-    const deltaMs = Math.max(now - prev.time, 1)
-    // CPU 单核折算：cpuDelta(µs) / deltaMs(ms) / 1000 → 百分比（×100）
-    const cpuPercent = (cpuDelta / 1000 / deltaMs) * 100
-    const byteDelta = fileBytes - prev.fileBytes
-    const writeRateBytesPerHour = byteDelta > 0 ? (byteDelta / deltaMs) * 3600 * 1000 : 0
-    const sample = { time: now, cpuPercent, memoryBytes, fileBytes, writeRateBytesPerHour, homeBytes }
-    state.history.push(sample)
-    if (state.history.length > MAX_HISTORY) state.history.splice(0, state.history.length - MAX_HISTORY)
-    state.lastSample = sample
-    updateDegradeState(state, limits, onDegrade, onRecover)
+    const home = process.env.DSH_HOME || join(homedir(), '.dsh');
+    const collect = options.collect ?? defaultSampler(home);
+    // 配置容错保持与改造前一致：非法 intervalMs 回退默认（而不是 fail-fast 让插件起不来）
+    const intervalMs = Number.isFinite(options.intervalMs) && options.intervalMs > 0
+        ? options.intervalMs
+        : DEFAULT_INTERVAL_MS;
+    const guard = createResourceGuard({
+        collect,
+        limits: options.limits,
+        intervalMs,
+        historySize: options.historySize,
+        enterConfirmCount: options.enterConfirmCount,
+        exitConfirmCount: options.exitConfirmCount,
+        now: options.now,
+        onDegrade: (snapshot) => options.onDegrade?.(snapshot),
+        onRecover: (snapshot) => options.onRecover?.(snapshot),
+        logger: ctxLogger(ctx),
+        prefix: '[dsh-my-observability]',
+    });
     return {
-      ...sample,
-      history: [...state.history],
-      alerts: evaluateResourceAlerts(sample, limits),
-      degraded: state.degraded,
-    }
-  }
-  state.lastSample = { time: now, fileBytes, memoryBytes, cpuPercent: 0, writeRateBytesPerHour: 0, homeBytes }
-  return { ...state.lastSample, history: [...state.history], alerts: [], degraded: state.degraded }
-}
-
-/**
- * 降级状态机：未降级且连续超限 → 进入降级（回调）；已降级且连续正常 → 退出降级（回调）。
- * 判定纯函数见 resource-rules.js；本函数只持有状态并触发宿主回调。
- */
-function updateDegradeState(state, limits, onDegrade, onRecover) {
-  if (!state.degraded) {
-    if (shouldEnterDegrade(state.history, limits)) {
-      state.degraded = true
-      onDegrade?.()
-    }
-  } else if (shouldExitDegrade(state.history, limits)) {
-    state.degraded = false
-    onRecover?.()
-  }
+        sample: () => guard.sample(),
+        start: () => guard.start(),
+        stop: () => guard.stop(),
+        isDegraded: () => guard.isDegraded(),
+        history: () => guard.history(),
+        stats: () => guard.stats(),
+        guard: () => guard,
+    };
 }
-
-/** 启动周期采样（幂等）。 */
-function startMonitor(state, intervalMs, monitor) {
-  if (state.timer === null) {
-    state.timer = setInterval(() => {
-      void monitor.sample()
-    }, intervalMs)
-    if (state.timer.unref) state.timer.unref()
-  }
-  return state.timer
+/** 取宿主的 logger（回调异常 warn 用）；不可用则返回 undefined（看门狗不因此失效）。 */
+function ctxLogger(ctx) {
+    const logger = ctx?.logger;
+    if (typeof logger?.warn !== 'function')
+        return undefined;
+    return { warn: (message) => logger.warn?.(message) };
 }
-
-/** 停止采样（幂等）。 */
-function stopMonitor(state) {
-  if (state.timer !== null) {
-    clearInterval(state.timer)
-    state.timer = null
-  }
+/** 默认采样源：进程维度 + 审计文件字节/写入速率 + $DSH_HOME 总字节。 */
+function defaultSampler(home) {
+    return createProcessSampler({
+        file: jsonlFile(),
+        extra: () => ({ homeBytes: dshHomeSize(home) }),
+    });
 }
-
 /** 递归计算目录总字节（best-effort，跳过不可达文件）。 */
 function dshHomeSize(dir) {
-  let total = 0
-  try {
-    for (const entry of readdirSync(dir, { withFileTypes: true })) {
-      const full = join(dir, entry.name)
-      if (entry.isDirectory()) {
-        total += dshHomeSize(full)
-      } else if (entry.isFile()) {
-        try {
-          total += statSync(full).size
-        } catch {
-          // 文件不可达
+    let total = 0;
+    try {
+        for (const entry of readdirSync(dir, { withFileTypes: true })) {
+            const full = join(dir, entry.name);
+            if (entry.isDirectory()) {
+                total += dshHomeSize(full);
+            }
+            else if (entry.isFile()) {
+                try {
+                    total += statSync(full).size;
+                }
+                catch {
+                    // 文件不可达
+                }
+            }
         }
-      }
     }
-  } catch {
-    // 目录不可达
-  }
-  return total
+    catch {
+        // 目录不可达
+    }
+    return total;
 }

package/lib/resource-rules.js

@@ -1,102 +1,12 @@
 /**
- * dsh-my-observability — 资源阈值判定（纯函数）。
+ * dsh-my-observability — 资源阈值判定（**dsh-shared 原语的消费方**，issue #198 第三批）。
  *
- * 阈值口径与 skill `resource-budget-review` 一致（设计/开发/运行三遍校验
- * 的运行期部分）：写放大/CPU/内存超限在采样数据上提前暴露，告警可查询。
- * 纯函数便于单测与配置覆盖（config.resourceLimits 浅合并）。
- */
-
-/** 默认阈值（DSH 插件场景，来源 resource-budget-review 五维表）。 */
-export const DEFAULT_LIMITS = {
-  /** 审计文件写入速率上限：50 MB/小时。 */
-  writeRateBytesPerHour: 50 * 1024 * 1024,
-  /** 审计文件大小上限：50 MB（循环 compact 后应远小于此）。 */
-  fileBytes: 50 * 1024 * 1024,
-  /** 本进程 CPU 均值上限：10%（单核折算，采样窗口内 user+sys / 时长）。 */
-  cpuPercent: 10,
-  /** 本进程 RSS 上限：500 MB。 */
-  memoryBytes: 500 * 1024 * 1024,
-}
-
-/** 采样数据 → 告警列表（[{ rule, level, message, value, limit }]）。 */
-export function evaluateResourceAlerts(sample, limits = DEFAULT_LIMITS) {
-  const alerts = []
-  if (typeof sample.writeRateBytesPerHour === 'number' && sample.writeRateBytesPerHour > limits.writeRateBytesPerHour) {
-    alerts.push({
-      rule: 'write-rate',
-      level: 'error',
-      message: `审计写入速率 ${fmtMB(sample.writeRateBytesPerHour)}/h 超过上限 ${fmtMB(limits.writeRateBytesPerHour)}/h（写放大风险）`,
-      value: sample.writeRateBytesPerHour,
-      limit: limits.writeRateBytesPerHour,
-    })
-  }
-  if (typeof sample.fileBytes === 'number' && sample.fileBytes > limits.fileBytes) {
-    alerts.push({
-      rule: 'file-size',
-      level: 'error',
-      message: `审计文件 ${fmtMB(sample.fileBytes)} 超过上限 ${fmtMB(limits.fileBytes)}（请检查 compact/轮转）`,
-      value: sample.fileBytes,
-      limit: limits.fileBytes,
-    })
-  }
-  if (typeof sample.cpuPercent === 'number' && sample.cpuPercent > limits.cpuPercent) {
-    alerts.push({
-      rule: 'cpu',
-      level: 'warn',
-      message: `本进程 CPU ${Math.round(sample.cpuPercent)}% 超过上限 ${limits.cpuPercent}%`,
-      value: sample.cpuPercent,
-      limit: limits.cpuPercent,
-    })
-  }
-  if (typeof sample.memoryBytes === 'number' && sample.memoryBytes > limits.memoryBytes) {
-    alerts.push({
-      rule: 'memory',
-      level: 'warn',
-      message: `本进程内存 ${fmtMB(sample.memoryBytes)} 超过上限 ${fmtMB(limits.memoryBytes)}`,
-      value: sample.memoryBytes,
-      limit: limits.memoryBytes,
-    })
-  }
-  return alerts
-}
-
-/** 连续多少次采样超限（关键阈值）判定进入降级（模块内默认值）。 */
-const DEGRADE_CONFIRM_COUNT = 3
-
-/** 连续多少次采样正常判定退出降级（模块内默认值）。 */
-const RECOVER_CONFIRM_COUNT = 3
-
-/**
- * 关键阈值判定：磁盘写放大风险主导（write-rate / file-size 任一超限即记超限）。
- * CPU/内存超限告警但不触发落盘降级（避免误伤正常大请求峰值）。
- */
-function isCriticalOverLimit(sample, limits = DEFAULT_LIMITS) {
-  return (
-    (sample.writeRateBytesPerHour ?? 0) > limits.writeRateBytesPerHour || (sample.fileBytes ?? 0) > limits.fileBytes
-  )
-}
-
-/**
- * 降级判定（纯函数）：最近 confirmCount 个样本**全部**关键阈值超限 → 进入降级。
- * 历史样本不足 confirmCount 时不降级（冷启动保护）。
- * 返回布尔（shouldEnterDegrade）。
- */
-export function shouldEnterDegrade(history, limits = DEFAULT_LIMITS, confirmCount = DEGRADE_CONFIRM_COUNT) {
-  const recent = history.slice(-confirmCount)
-  if (recent.length < confirmCount) return false
-  return recent.every((sample) => isCriticalOverLimit(sample, limits))
-}
-
-/**
- * 恢复判定（纯函数）：最近 confirmCount 个样本**全部**关键阈值正常 → 退出降级。
- * 历史样本不足 confirmCount 时不恢复（避免刚降级立即恢复抖动）。
+ * 第三批把阈值判定与看门狗状态机抽到 `dsh-shared` 的 `resource-rules` /
+ * `resource-guard`，本文件只做**别名 re-export**：
+ *  - 保留插件内既有的导入路径与名称（`DEFAULT_LIMITS` 等），路由/测试/配置契约不变；
+ *  - 判定口径（50MB/h 写入速率、50MB 文件、10% CPU、500MB RSS；相等不算超限；
+ *    连续 3 次确认）由 shared 单点维护，插件不再各存一份。
+ *
+ * 原语边界与反例见 plugins/dsh-shared/README.md 与 docs/共享工具包/概述.md。
  */
-export function shouldExitDegrade(history, limits = DEFAULT_LIMITS, confirmCount = RECOVER_CONFIRM_COUNT) {
-  const recent = history.slice(-confirmCount)
-  if (recent.length < confirmCount) return false
-  return recent.every((sample) => !isCriticalOverLimit(sample, limits))
-}
-
-function fmtMB(bytes) {
-  return `${(bytes / 1024 / 1024).toFixed(1)} MB`
-}
+export { DEFAULT_RESOURCE_LIMITS as DEFAULT_LIMITS, evaluateResourceAlerts, shouldEnterDegrade, shouldExitDegrade, } from 'dsh-shared';

package/lib/review.js

@@ -13,118 +13,109 @@
  *
  * AI 增强审查见 lib/ai.js（可选，agents 服务可用时追加 LLM 结论）。
  */
-import { isTestFile, isSourceFile } from './diff.js'
-
+import { isTestFile, isSourceFile } from './diff.js';
 /** 单文件变更行数阈值（large-diff 规则）。 */
-export const LARGE_DIFF_LINES = 300
-
+export const LARGE_DIFF_LINES = 300;
 /** 对已解析 diff 运行规则引擎，返回 { summary, issues }。 */
 export function reviewRules(parsed) {
-  const issues = []
-  for (const file of parsed.files) {
-    for (const added of file.addedLines) {
-      pushIf(issues, debugIssue(file, added))
-      pushIf(issues, secretIssue(file, added))
-      pushIf(issues, conflictIssue(file, added))
-      pushIf(issues, todoIssue(file, added))
-      pushIf(issues, trailingIssue(file, added))
+    const issues = [];
+    for (const file of parsed.files) {
+        for (const added of file.addedLines) {
+            pushIf(issues, debugIssue(file, added));
+            pushIf(issues, secretIssue(file, added));
+            pushIf(issues, conflictIssue(file, added));
+            pushIf(issues, todoIssue(file, added));
+            pushIf(issues, trailingIssue(file, added));
+        }
+        pushIf(issues, largeDiffIssue(file));
+        pushIf(issues, binaryIssue(file));
     }
-    pushIf(issues, largeDiffIssue(file))
-    pushIf(issues, binaryIssue(file))
-  }
-  pushIf(issues, noTestIssue(parsed))
-  return summarize(parsed, issues)
+    pushIf(issues, noTestIssue(parsed));
+    return summarize(parsed, issues);
 }
-
 /** 汇总：变更统计 + 问题分级计数。 */
 function summarize(parsed, issues) {
-  const files = parsed.files
-  const insertions = files.reduce((sum, file) => sum + file.insertions, 0)
-  const deletions = files.reduce((sum, file) => sum + file.deletions, 0)
-  return {
-    issues,
-    summary: {
-      files: files.length,
-      insertions,
-      deletions,
-      issues: issues.length,
-      errors: countBy(issues, 'error'),
-      warnings: countBy(issues, 'warning'),
-      infos: countBy(issues, 'info'),
-      binary: parsed.binary,
-    },
-  }
+    const files = parsed.files;
+    const insertions = files.reduce((sum, file) => sum + file.insertions, 0);
+    const deletions = files.reduce((sum, file) => sum + file.deletions, 0);
+    return {
+        issues,
+        summary: {
+            files: files.length,
+            insertions,
+            deletions,
+            issues: issues.length,
+            errors: countBy(issues, 'error'),
+            warnings: countBy(issues, 'warning'),
+            infos: countBy(issues, 'info'),
+            binary: parsed.binary,
+        },
+    };
 }
-
 function countBy(issues, severity) {
-  return issues.filter((issue) => issue.severity === severity).length
+    return issues.filter((issue) => issue.severity === severity).length;
 }
-
 function pushIf(issues, issue) {
-  if (issue !== undefined) issues.push(issue)
+    if (issue !== undefined)
+        issues.push(issue);
 }
-
 /** 调试残留（console.* / debugger / print 族）。 */
 function debugIssue(file, added) {
-  if (!/console\.(log|debug|warn|info|error)|debugger\b|\bprint\(|println\(|System\.out\.print/.test(added.text))
-    return undefined
-  return issue('warning', 'debug-statement', file, added, '调试残留语句（console/print/debugger）')
+    if (!/console\.(log|debug|warn|info|error)|debugger\b|\bprint\(|println\(|System\.out\.print/.test(added.text))
+        return undefined;
+    return issue('warning', 'debug-statement', file, added, '调试残留语句（console/print/debugger）');
 }
-
 /** 密钥/凭据硬编码。 */
 function secretIssue(file, added) {
-  if (
-    !/(password|passwd|api[_-]?key|secret|access[_-]?token|auth[_-]?token)\s*[:=]\s*['"][^'"]{4,}['"]/i.test(added.text)
-  )
-    return undefined
-  return issue('error', 'secret-leak', file, added, '疑似硬编码密钥/凭据')
+    if (!/(password|passwd|api[_-]?key|secret|access[_-]?token|auth[_-]?token)\s*[:=]\s*['"][^'"]{4,}['"]/i.test(added.text))
+        return undefined;
+    return issue('error', 'secret-leak', file, added, '疑似硬编码密钥/凭据');
 }
-
 /** 合并冲突标记残留。 */
 function conflictIssue(file, added) {
-  if (!/^(<<<<<<<|=======|>>>>>>>)/.test(added.text)) return undefined
-  return issue('error', 'conflict-marker', file, added, '合并冲突标记残留')
+    if (!/^(<<<<<<<|=======|>>>>>>>)/.test(added.text))
+        return undefined;
+    return issue('error', 'conflict-marker', file, added, '合并冲突标记残留');
 }
-
 /** TODO/FIXME 标记。 */
 function todoIssue(file, added) {
-  if (!/\b(TODO|FIXME|HACK|XXX)\b/.test(added.text)) return undefined
-  return issue('info', 'todo-marker', file, added, 'TODO/FIXME 标记')
+    if (!/\b(TODO|FIXME|HACK|XXX)\b/.test(added.text))
+        return undefined;
+    return issue('info', 'todo-marker', file, added, 'TODO/FIXME 标记');
 }
-
 /** 尾随空格。 */
 function trailingIssue(file, added) {
-  if (!/\s+$/.test(added.text)) return undefined
-  return issue('info', 'trailing-space', file, added, '行尾多余空格')
+    if (!/\s+$/.test(added.text))
+        return undefined;
+    return issue('info', 'trailing-space', file, added, '行尾多余空格');
 }
-
 /** 单文件变更超阈值。 */
 function largeDiffIssue(file) {
-  const changed = file.insertions + file.deletions
-  if (changed <= LARGE_DIFF_LINES || file.binary) return undefined
-  return issue('warning', 'large-diff', file, null, `单文件变更 ${changed} 行（>${LARGE_DIFF_LINES}），建议拆分提交`)
+    const changed = file.insertions + file.deletions;
+    if (changed <= LARGE_DIFF_LINES || file.binary)
+        return undefined;
+    return issue('warning', 'large-diff', file, null, `单文件变更 ${changed} 行（>${LARGE_DIFF_LINES}），建议拆分提交`);
 }
-
 /** 二进制文件变更。 */
 function binaryIssue(file) {
-  if (!file.binary) return undefined
-  return issue('warning', 'binary-file', file, null, '二进制文件变更')
+    if (!file.binary)
+        return undefined;
+    return issue('warning', 'binary-file', file, null, '二进制文件变更');
 }
-
 /** 有源码变更但无测试变更。 */
 function noTestIssue(parsed) {
-  const sourceFiles = parsed.files.filter((file) => isSourceFile(file.path))
-  const testFiles = parsed.files.filter((file) => isTestFile(file.path))
-  if (sourceFiles.length === 0 || testFiles.length > 0) return undefined
-  return issue('info', 'no-test-change', null, null, '有源码变更但没有测试文件变更')
+    const sourceFiles = parsed.files.filter((file) => isSourceFile(file.path));
+    const testFiles = parsed.files.filter((file) => isTestFile(file.path));
+    if (sourceFiles.length === 0 || testFiles.length > 0)
+        return undefined;
+    return issue('info', 'no-test-change', null, null, '有源码变更但没有测试文件变更');
 }
-
 function issue(severity, rule, file, added, message) {
-  return {
-    severity,
-    rule,
-    file: file?.path ?? '',
-    line: added?.line ?? 0,
-    message,
-  }
+    return {
+        severity,
+        rule,
+        file: file?.path ?? '',
+        line: added?.line ?? 0,
+        message,
+    };
 }