koishi-plugin-kkk 3.3.1 → 3.3.3

package/CHANGELOG.md

@@ -1,5 +1,16 @@
 # 更新日志
 
+## 3.3.3
+
+### 同一条卡片消息不再解析两遍
+
+- **卡片消息被重复投递时只处理第一遍**：QQ 会把同一次发送投递多遍（客户端重发、指令按钮 = 消息 + 交互事件），实测同一条 B站卡片会在几秒内进来两次 —— 于是 OCR + 搜索各跑两遍、面板也可能发两条，看起来就是「解析完了又解析一遍」。现在卡片这条链路在 OCR **之前**就去重（按「平台 + 频道 + 用户 + 卡片正文」），并且**先把图片链接的签名参数去掉再比**：同一条卡片重投时链接会重新签名，直接拿原文比会当成两条不同的消息。
+- 顺带加了排查用的 debug 日志：收到卡片时会打 `收到卡片消息（messageId=…，平台=…）`，同一条卡片的两次投递 messageId 是否相同，一眼就能看出是平台重复投递还是别的插件又发了一遍。
+
+## 3.3.2
+
+> 修 3.3.1 的一个**致命笔误**：播放器模块里一行 import 多写了一层 `..`，导致插件在**加载时**就失败（`Cannot find module '../../compat/runtime'`），整个插件都起不来。构建链路只做转译不解析路径，所以当时没拦住 —— 现在补了一个「每个编译产物都 require 得动」的自测（scripts/smoke-require.cjs）。
+
 ## 3.3.1
 
 > 3.3.0-beta.1 ~ beta.3 是给先行测试的人用的预览版，内容全部并进这一版；从 3.2.2 直接升上来就行。

package/lib/index.js

@@ -706,6 +706,33 @@ function registerCommands(ctx, logger, autoParse) {
                     logger.debug('卡片来源不是 B站（%s），跳过卡片解析', cardPlatform || '未知');
                     return next();
                 }
+                /**
+                 * **同一条卡片消息被投递多遍时，只处理第一遍**。
+                 *
+                 * 线上实测：同一条 B站卡片会在几秒内进来两次（平台重投 / 另一个中间件再发一遍），
+                 * 于是 OCR + 搜索各跑两遍、面板也可能发两条 —— 用户看到的就是「解析完了又解析一遍」。
+                 *
+                 * 去重键里的卡片正文要**先去掉 URL 的签名参数**：同一条卡片重投时图片链接会被重新签名，
+                 * 直接拿原文哈希会认为是两条不同的消息（踩过这个坑，第二次照样 OCR）。
+                 */
+                const cardSignature = raw
+                    .replace(/https?:\/\/[^\s"'<>]+/g, (url) => url.split("?")[0])
+                    .replace(/\s+/g, " ")
+                    .trim()
+                    .slice(0, 2000);
+                /**
+                 * 打一条「收到卡片」的日志（debug 级，排查重复投递用）。
+                 *
+                 * 同一条卡片连着来两次、而 **messageId 也相同** → 这条事件被重复投递/重复处理；
+                 * messageId 不同 → 平台那边确实又发了一条。排查「到底谁重复了」就看它。
+                 */
+                logger.debug('收到卡片消息（messageId=%s，平台=%s）', String(session.messageId ?? '无'), String(session.platform ?? '未知'));
+                const { acquireParseLock } = await Promise.resolve().then(() => __importStar(require("./karin/module/utils/ParseLock")));
+                const cardKey = ["card", String(session.platform ?? ""), String(session.channelId ?? ""), String(session.userId ?? ""), cardSignature].join(":");
+                if (!acquireParseLock(cardKey)) {
+                    logger.debug("短时间内重复的同一条卡片消息，已忽略（不再重复 OCR/搜索）: %s", cardKey.slice(0, 120));
+                    return;
+                }
                 const send = async (content) => {
                     try {
                         await session.send(content);

package/lib/player/server.js

@@ -64,7 +64,8 @@ const node_fs_1 = __importDefault(require("node:fs"));
 const node_http_1 = __importDefault(require("node:http"));
 const node_path_1 = __importDefault(require("node:path"));
 const node_karin_1 = require("../compat/node-karin.js");
-const runtime_1 = require("../../compat/runtime");
+// 注意只有一层 .. ：src/player/server.ts → src/compat/runtime（写两层会变成根目录下的 compat，直接 MODULE_NOT_FOUND）
+const runtime_1 = require("../compat/runtime");
 const page_1 = require("./page");
 const store_1 = require("./store");
 /** 路由前缀 */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "koishi-plugin-kkk",
-  "version": "3.3.1",
+  "version": "3.3.3",
   "description": "抖音 / B站 / 快手 / 小红书 视频解析与动态推送，带 QQ 交互面板与弹幕烧录（karin-plugin-kkk 的 Koishi 移植版）",
   "main": "lib/index.js",
   "typings": "lib/index.d.ts",

package/scripts/smoke-require.cjs

@@ -0,0 +1,91 @@
+/**
+ * 「每个编译产物都 require 得动」冒烟测试。
+ *
+ * 起因（线上真实故障）：给 player/server.ts 加了一行
+ * `import { tryGetRuntime } from '../../compat/runtime'` —— 多写了一层 `..`，
+ * 编译出来变成 `require("../../compat/runtime")`（指向插件根目录下的 compat，不存在）。
+ * TypeScript 那条链路只做转译、不解析路径，构建时毫无提示；**插件在用户机器上直接加载失败**：
+ *
+ *     [W] config failed to load kkk
+ *     Error: Cannot find module '../../compat/runtime'
+ *
+ * 这个测试就是把 lib 下每个 .js 都 require 一遍 —— 路径写错、模块顶层就抛异常，全都跑不掉。
+ *
+ * 用法：node scripts/smoke-require.cjs
+ */
+const fs = require('node:fs')
+const path = require('node:path')
+const { createRequire } = require('node:module')
+
+const pluginRoot = path.resolve(__dirname, '..')
+const libRoot = path.join(pluginRoot, 'lib')
+
+/** 绑定最小运行时：很多模块在顶层就读配置/日志 */
+const runtime = require(path.join(libRoot, 'compat', 'runtime.js'))
+const noop = () => {}
+runtime.bindRuntime({
+  ctx: {
+    get: () => undefined,
+    logger: () => ({ info: noop, warn: noop, error: noop, debug: noop, mark: noop }),
+    bots: [],
+    registry: new Map(),
+    on: noop,
+    middleware: noop
+  },
+  config: { app: {} },
+  dataRoot: path.join(pluginRoot, 'data-smoke-require'),
+  pluginRoot,
+  master: () => []
+})
+
+const walk = (dir) => {
+  const out = []
+  for (const item of fs.readdirSync(dir, { withFileTypes: true })) {
+    const full = path.join(dir, item.name)
+    if (item.isDirectory()) out.push(...walk(full))
+    else if (item.name.endsWith('.js')) out.push(full)
+  }
+  return out
+}
+
+const req = createRequire(path.join(libRoot, 'index.js'))
+const files = walk(libRoot)
+const failures = []
+let ok = 0
+let esm = 0
+/** 缺第三方依赖（宿主没装）的模块：单独列出，不算失败 */
+const optional = []
+for (const file of files) {
+  try {
+    req(file)
+    ok++
+  } catch (error) {
+    const message = String(error?.message ?? error)
+    /** 模板类模块是 ESM（用 import 加载的），require 报这个是**预期**的，不算失败 */
+    const isEsm = error?.code === 'ERR_REQUIRE_ESM' || error?.code === 'ERR_REQUIRE_ASYNC_MODULE'
+      || /Cannot use import statement outside a module|require\(\) of ES Module/.test(message)
+    if (isEsm) { esm++; continue }
+    /**
+     * 缺**第三方包**（cors / node:sqlite 之类）不算这次要抓的问题：有些模块是动态 import 的可选路径，
+     * 依赖装没装取决于宿主环境。**相对路径**找不到才是真问题（就是这次线上的那种），照旧算失败。
+     */
+    const missing = /Cannot find module '([^']+)'/.exec(message)
+    if (missing && !missing[1].startsWith('.')) {
+      optional.push(path.relative(libRoot, file) + ' → 缺依赖 ' + missing[1])
+      continue
+    }
+    failures.push({ file: path.relative(libRoot, file), message })
+  }
+}
+
+console.log('检查了 ' + files.length + ' 个编译产物：require 成功 ' + ok + ' 个，ESM（预期跳过）' + esm + ' 个，缺第三方依赖（预期跳过）' + optional.length + ' 个')
+for (const item of optional) console.log('  · ' + item)
+if (failures.length === 0) {
+  console.log('\n=== 通过：每个模块都加载得动（没有写错的 import 路径 / 顶层异常） ===')
+  // 用 process.exit：require 这些模块时可能有别的东西把 exitCode 置成了非 0
+  process.exit(0)
+} else {
+  for (const item of failures) console.log('  ❌ ' + item.file + ' → ' + item.message.split('\n')[0])
+  console.log('\n=== 失败：' + failures.length + ' 个模块加载不了 ===')
+  process.exit(1)
+}

package/src/index.ts

@@ -708,6 +708,35 @@ function registerCommands (
           logger.debug('卡片来源不是 B站（%s），跳过卡片解析', cardPlatform || '未知')
           return next()
         }
+        /**
+         * **同一条卡片消息被投递多遍时，只处理第一遍**。
+         *
+         * 线上实测：同一条 B站卡片会在几秒内进来两次（平台重投 / 另一个中间件再发一遍），
+         * 于是 OCR + 搜索各跑两遍、面板也可能发两条 —— 用户看到的就是「解析完了又解析一遍」。
+         *
+         * 去重键里的卡片正文要**先去掉 URL 的签名参数**：同一条卡片重投时图片链接会被重新签名，
+         * 直接拿原文哈希会认为是两条不同的消息（踩过这个坑，第二次照样 OCR）。
+         */
+        const cardSignature = raw
+          .replace(/https?:\/\/[^\s"'<>]+/g, (url: string) => url.split("?")[0])
+          .replace(/\s+/g, " ")
+          .trim()
+          .slice(0, 2000)
+        /**
+         * 打一条「收到卡片」的日志（debug 级，排查重复投递用）。
+         *
+         * 同一条卡片连着来两次、而 **messageId 也相同** → 这条事件被重复投递/重复处理；
+         * messageId 不同 → 平台那边确实又发了一条。排查「到底谁重复了」就看它。
+         */
+        logger.debug('收到卡片消息（messageId=%s，平台=%s）',
+          String((session as any).messageId ?? '无'),
+          String((session as any).platform ?? '未知'))
+        const { acquireParseLock } = await import("./karin/module/utils/ParseLock")
+        const cardKey = ["card", String((session as any).platform ?? ""), String((session as any).channelId ?? ""), String((session as any).userId ?? ""), cardSignature].join(":")
+        if (!acquireParseLock(cardKey)) {
+          logger.debug("短时间内重复的同一条卡片消息，已忽略（不再重复 OCR/搜索）: %s", cardKey.slice(0, 120))
+          return
+        }
         const send = async (content: any) => {
           try {
             await (session as any).send(content)

package/src/player/server.ts

@@ -23,7 +23,8 @@ import path from 'node:path'
 
 import { logger } from 'node-karin'
 
-import { tryGetRuntime } from '../../compat/runtime'
+// 注意只有一层 .. ：src/player/server.ts → src/compat/runtime（写两层会变成根目录下的 compat，直接 MODULE_NOT_FOUND）
+import { tryGetRuntime } from '../compat/runtime'
 
 import { renderExpiredPage, renderPlayerPage } from './page'
 import {