koishi-plugin-kkk 3.3.0-beta.2 → 3.3.1

package/CHANGELOG.md

@@ -1,8 +1,17 @@
 # 更新日志
 
-## 3.3.0-beta.2
+## 3.3.1
 
-> **预览版**：给愿意先试的人用。npm 上默认安装的（latest）还是 3.2.2，想装这个用 npm i koishi-plugin-kkk@beta（或者面板里的「检查更新」换成 beta 通道）。
+> 3.3.0-beta.1 ~ beta.3 是给先行测试的人用的预览版，内容全部并进这一版；从 3.2.2 直接升上来就行。
+
+### 修了两个「设置根本没生效」的问题
+
+- **「强制在线播放的适配器」这类设置以前是静默失效的**：运行时配置是一份**手写白名单**，新加的开关（强制在线播放的适配器、面板「在线看」列、按需切片、解析去重…）忘了往里补一行，运行时读到的就是 undefined —— 表现就是 B站私聊明明配了强制在线播放，**还是去下载并发视频文件**。现在改成「字段表里的默认值打底 + 归一化覆盖」，以后再加开关不会漏。
+- **不支持的适配器不再尝试合并转发**：判断「这个适配器支不支持聊天记录」用的是**黑名单**（只排除 QQ 官方），于是任何自定义适配器都被当成支持 —— B站私聊适配器（platform 就是 bilibili）也被算进去了，整条转发发出去「没有拿到消息 ID」，连退回来的逐条发送也失败，用户什么都没收到。现在改成**白名单**（OneBot 系平台名，或适配器自带 sendGroupForwardMsg/sendPrivateForwardMsg），并会在日志里写明「适配器 xxx 不支持合并转发，本次逐条发送」。
+
+### 播放链接显示「已过期」的原因说清楚了
+
+- **端口被别的实例占掉，链接就会指向「另一台机器人」**：同机跑两台 Koishi（正式 + 测试）时只有一台能监听播放器端口。占不到端口的那台退回 Koishi 自己的端口，而域名反代的是独立端口 —— 用户点开链接访问的其实是**抢到端口的那台实例**，它的会话表里没有这个 token，页面就显示「链接已过期」。现在遇到这种情况会打一条**明确的警告**（写清「链接会显示已过期，请换端口或停掉占端口的那台实例」），不再是含糊的「启动失败」；生成链接时也会提醒一次。
 
 ### 控制台里的设置全部隐藏，面板补上「Koishi 设置」
 

package/lib/index.js

@@ -892,7 +892,16 @@ async function apply(ctx, rawConfig) {
         if (raw[key] !== undefined)
             legacy[key] = raw[key];
     }
-    const groupQq = { ...(qq ?? {}) };
+    /**
+     * **每个 qq 字段都要有值**：不能指望 Koishi 的 schema 默认值一定被填上 ——
+     * 线上实测踩过（用户报「B站私聊还是发视频、没给在线播放链接」）：
+     * 配置里没写过的字段，运行时读到的是 `undefined`，
+     * 于是「强制在线播放的适配器」（默认 bilibili）整个失效、面板列按钮的开关也失效。
+     *
+     * 所以这里用 `readQqOptions` 先铺一层**字段表里的默认值**（qqFields.json 是唯一出处），
+     * 再用用户显式配的值覆盖 —— 显式配置永远优先，缺的字段一律有默认。
+     */
+    const groupQq = { ...(0, qqOptions_1.readQqOptions)(raw), ...(qq ?? {}) };
     const groupNative = { ...(advanced ?? {}) };
     for (const key of qqOptions_1.QQ_KEYS)
         if (legacy[key] !== undefined)
@@ -952,7 +961,18 @@ async function apply(ctx, rawConfig) {
     const dataRoot = node_path_1.default.isAbsolute(config.dataPath) ? config.dataPath : node_path_1.default.resolve(ctx.baseDir ?? process.cwd(), config.dataPath);
     (0, runtime_1.bindRuntime)({
         ctx,
+        /**
+         * ⚠️ 这份 config **不是**原封不动转发，而是运行时真正读的那一份（`tryGetRuntime().config`）。
+         *
+         * 以前它是一个**手写白名单** —— 于是每加一个新开关（例如「强制在线播放的适配器」
+         * `forceOnlinePlayer`），忘了往这里补一行，运行时就读到 undefined，功能整个静默失效：
+         * 线上真实故障就是「B站私聊该给在线播放链接，结果还是去发视频文件」。
+         *
+         * 现在改成 `...readQqOptions(config)` 打底（qqFields.json 里的字段一个不落，缺的用默认值），
+         * 下面那些需要**归一化**的字段（端口取整数、分钟数夹范围、布尔用 !== false 这种）再覆盖上去。
+         */
         config: {
+            ...(0, qqOptions_1.readQqOptions)(config),
             masters: config.masters ?? [],
             debug: config.debug,
             dataPath: config.dataPath,

package/lib/karin/module/utils/ParseForward.js

@@ -41,9 +41,31 @@ function parseForwardPeer(e) {
 function platformOf(e) {
     return String(e?.bot?.bot?.platform ?? e?.bot?.platform ?? '');
 }
-/** 这台部署的适配器支不支持合并转发 */
+/**
+ * OneBot 系适配器（NapCat / Lagrange / go-cqhttp / Chronocat…）：它们真的有「聊天记录」这个概念。
+ *
+ * 注意这份名单是**白名单**：compat 那边的 isForwardSupported 是黑名单（只排除 QQ 官方），
+ * 于是**任何自定义适配器都会被当成支持** —— 线上真实故障就是这样：
+ * B站私聊适配器（koishi-plugin-adapter-bilibili-dm，platform 就是 bilibili）被认成支持合并转发，
+ * 结果整条转发发出去「没有拿到消息 ID」（适配器根本没有聊天记录这种东西），
+ * 连退回来的逐条发送也失败了 —— 用户什么都没收到。
+ */
+const ONEBOT_LIKE = /onebot|napcat|lagrange|go-?cqhttp|chronocat|mirai/i;
+/** 这台部署的适配器支不支持合并转发（**只有明确支持才算支持**） */
 function canForwardParseResult(e) {
-    return (0, node_karin_1.isForwardSupported)({ platform: platformOf(e) });
+    const raw = e?.bot?.bot ?? e?.bot;
+    /**
+     * ① 适配器自己带合并转发 API（OneBot 系的 sendGroupForwardMsg / sendPrivateForwardMsg）
+     *    —— 这条最可靠，不依赖平台名怎么写。
+     */
+    const internal = raw?.internal ?? e?.bot?.internal;
+    if (internal && (typeof internal.sendGroupForwardMsg === 'function' || typeof internal.sendPrivateForwardMsg === 'function'))
+        return true;
+    /** ② 其余只认 OneBot 系的平台名 */
+    const platform = platformOf(e);
+    if (!platform)
+        return false;
+    return ONEBOT_LIKE.test(platform);
 }
 /** 平台名 → 配置段名（合并转发的平台开关就写在各自的平台段里） */
 const PLATFORM_SECTIONS = {
@@ -311,8 +333,13 @@ function withParseForward(handler, platform) {
                 + '.forward 都没打开），按逐条发送处理');
             return await handler(e, next);
         }
-        if (!parseForwardPeer(e) || !canForwardParseResult(e))
+        if (!parseForwardPeer(e))
             return await handler(e, next);
+        if (!canForwardParseResult(e)) {
+            // 说明白为什么没合并：以前这里静默退化，用户只看到「转发了但没收到」
+            node_karin_1.logger.mark('[合并转发] 适配器 ' + (platformOf(e) || '未知') + ' 不支持合并转发（不是 OneBot 系、也没有 forward API），本次逐条发送');
+            return await handler(e, next);
+        }
         return await (0, node_karin_1.runWithForwardBag)(parseForwardPeer(e), async () => {
             try {
                 return await handler(e, next);

package/lib/player/index.js

@@ -199,6 +199,8 @@ function forceOnlinePlayerAdapters() {
     }
 }
 /** 这个**适配器**是不是在「强制在线播放」名单里 */
+/** 「本实例独立端口没监听上」这条只提醒一次，不然每条链接都刷一行 */
+let warnedStandaloneMismatch = false;
 function isForceOnlinePlayerAdapter(platform) {
     if (!platform)
         return false;
@@ -366,8 +368,20 @@ let warnedLocalBase = false;
 function buildPlayerLink(token) {
     const config = (0, runtime_1.tryGetRuntime)()?.config ?? {};
     const base = String(config.playerBaseUrl ?? '').replace(/\/+$/, '');
-    if (base)
+    if (base) {
+        /**
+         * 本实例**没监听上独立端口**，但链接指向公网域名 —— 那个域名反代的是独立端口，
+         * 于是用户点开访问的其实是**抢到端口的另一台实例**（它的会话表里没有这个 token），
+         * 页面就会显示「链接已过期」。线上真实故障，这里至少要说清楚一次。
+         */
+        if (!(0, server_1.isStandalonePlayerReady)() && !warnedStandaloneMismatch) {
+            warnedStandaloneMismatch = true;
+            node_karin_1.logger.warn('[在线播放] 本实例没有监听上独立端口（被别的进程占了），但链接指向 ' + base
+                + ' —— 如果那个域名反代到同一个端口，用户点开会看到「链接已过期」（其实连的是抢到端口的那台实例）。'
+                + '请换一个「播放器端口」，或者停掉同机另一台占着该端口的 Koishi 实例');
+        }
         return base + '/kkk/player/' + token;
+    }
     const port = Number(config.playerPort) > 0 ? Number(config.playerPort) : koishiPort();
     const link = 'http://' + localAddress() + ':' + port + '/kkk/player/' + token;
     if (!warnedLocalBase) {

package/lib/player/server.js

@@ -36,7 +36,7 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.PLAYER_ROUTE_PREFIX = void 0;
+exports.isStandalonePlayerReady = exports.PLAYER_ROUTE_PREFIX = void 0;
 exports.sanitizeDownloadName = sanitizeDownloadName;
 exports.downloadDisposition = downloadDisposition;
 exports.handlePlayerRequest = handlePlayerRequest;
@@ -64,10 +64,25 @@ 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");
 const page_1 = require("./page");
 const store_1 = require("./store");
 /** 路由前缀 */
 exports.PLAYER_ROUTE_PREFIX = '/kkk/player/';
+/** 独立端口是否真的监听上了：没监听上时链接指向的公网域名很可能打不到本实例 */
+let standaloneReady = false;
+/** 查询独立端口是否监听上了（给「生成链接」那边做提示用） */
+const isStandalonePlayerReady = () => standaloneReady;
+exports.isStandalonePlayerReady = isStandalonePlayerReady;
+/** 面板里配的「播放器公网地址」（没配就是空串） */
+function publicBaseUrl() {
+    try {
+        return String((0, runtime_1.tryGetRuntime)()?.config?.playerBaseUrl ?? '').trim();
+    }
+    catch {
+        return '';
+    }
+}
 const TEXT_TYPE = 'text/plain; charset=utf-8';
 const HTML_TYPE = 'text/html; charset=utf-8';
 /** 过期 / 无效令牌的统一回应：页面直接渲染「链接已过期」，接口退回纯文本 */
@@ -405,13 +420,23 @@ function registerPlayerRoutes({ ctx, port }) {
             });
         });
         server.on('error', (error) => {
-            node_karin_1.logger.warn('[在线播放] 播放器端口 ' + port + ' 启动失败（' + String(error?.code ?? error?.message ?? error)
-                + '），已退回 Koishi 自己的端口；请在配置里换一个端口或留空');
+            const code = String(error?.code ?? error?.message ?? error);
+            /**
+             * 端口被别的进程占了：**必须说清楚后果** —— 线上真实故障就是「用户点开播放链接看到『链接已过期』」。
+             *
+             * 成因：这台实例退回 Koishi 自己的端口，而另一台实例（比如同机的测试实例）抢到了这个端口；
+             * 域名反代到该端口 → 用户访问的其实是**另一台实例**，它当然没有这个会话。
+             */
+            node_karin_1.logger.error('[在线播放] 播放器端口 ' + port + ' 被占用（' + code + '），本次退回 Koishi 自己的端口'
+                + (publicBaseUrl() ? '。注意：域名 ' + publicBaseUrl() + ' 反代的就是 ' + port + ' 的话，用户点开链接会看到「链接已过期」'
+                    + ' —— 说明这台实例不是 8888 的占用者（同机另一个 Koishi 实例抢到了），换一个端口或先停掉那台实例' : ''));
+            standaloneReady = false;
             // 端口被占用不能把插件带崩：退回 ctx.server 再挂一遍
             registerOnKoishi(ctx, disposers);
         });
         try {
             server.listen(port, () => {
+                standaloneReady = true;
                 node_karin_1.logger.info('[在线播放] 播放器已监听独立端口 ' + port);
             });
             disposers.push(() => { try {

package/package.json

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

package/src/index.ts

@@ -885,7 +885,16 @@ export async function apply (ctx: Context, rawConfig: Config) {
   for (const key of [...NATIVE_KEYS, ...QQ_KEYS]) {
     if (raw[key] !== undefined) legacy[key] = raw[key]
   }
-  const groupQq: any = { ...(qq ?? {}) }
+  /**
+   * **每个 qq 字段都要有值**：不能指望 Koishi 的 schema 默认值一定被填上 ——
+   * 线上实测踩过（用户报「B站私聊还是发视频、没给在线播放链接」）：
+   * 配置里没写过的字段，运行时读到的是 `undefined`，
+   * 于是「强制在线播放的适配器」（默认 bilibili）整个失效、面板列按钮的开关也失效。
+   *
+   * 所以这里用 `readQqOptions` 先铺一层**字段表里的默认值**（qqFields.json 是唯一出处），
+   * 再用用户显式配的值覆盖 —— 显式配置永远优先，缺的字段一律有默认。
+   */
+  const groupQq: any = { ...readQqOptions(raw as any), ...(qq ?? {}) }
   const groupNative: any = { ...(advanced ?? {}) }
   for (const key of QQ_KEYS) if (legacy[key] !== undefined) groupQq[key] = legacy[key]
   for (const key of NATIVE_KEYS) if (legacy[key] !== undefined) groupNative[key] = legacy[key]
@@ -940,7 +949,18 @@ export async function apply (ctx: Context, rawConfig: Config) {
 
   bindRuntime({
     ctx,
+    /**
+     * ⚠️ 这份 config **不是**原封不动转发，而是运行时真正读的那一份（`tryGetRuntime().config`）。
+     *
+     * 以前它是一个**手写白名单** —— 于是每加一个新开关（例如「强制在线播放的适配器」
+     * `forceOnlinePlayer`），忘了往这里补一行，运行时就读到 undefined，功能整个静默失效：
+     * 线上真实故障就是「B站私聊该给在线播放链接，结果还是去发视频文件」。
+     *
+     * 现在改成 `...readQqOptions(config)` 打底（qqFields.json 里的字段一个不落，缺的用默认值），
+     * 下面那些需要**归一化**的字段（端口取整数、分钟数夹范围、布尔用 !== false 这种）再覆盖上去。
+     */
     config: {
+      ...readQqOptions(config),
       masters: config.masters ?? [],
       debug: config.debug,
       dataPath: config.dataPath,

package/src/karin/module/utils/ParseForward.ts

@@ -35,9 +35,30 @@ function platformOf (e: any): string {
   return String(e?.bot?.bot?.platform ?? e?.bot?.platform ?? '')
 }
 
-/** 这台部署的适配器支不支持合并转发 */
+/**
+ * OneBot 系适配器（NapCat / Lagrange / go-cqhttp / Chronocat…）：它们真的有「聊天记录」这个概念。
+ *
+ * 注意这份名单是**白名单**：compat 那边的 isForwardSupported 是黑名单（只排除 QQ 官方），
+ * 于是**任何自定义适配器都会被当成支持** —— 线上真实故障就是这样：
+ * B站私聊适配器（koishi-plugin-adapter-bilibili-dm，platform 就是 bilibili）被认成支持合并转发，
+ * 结果整条转发发出去「没有拿到消息 ID」（适配器根本没有聊天记录这种东西），
+ * 连退回来的逐条发送也失败了 —— 用户什么都没收到。
+ */
+const ONEBOT_LIKE = /onebot|napcat|lagrange|go-?cqhttp|chronocat|mirai/i
+
+/** 这台部署的适配器支不支持合并转发（**只有明确支持才算支持**） */
 export function canForwardParseResult (e: any): boolean {
-  return isForwardSupported({ platform: platformOf(e) })
+  const raw: any = e?.bot?.bot ?? e?.bot
+  /**
+   * ① 适配器自己带合并转发 API（OneBot 系的 sendGroupForwardMsg / sendPrivateForwardMsg）
+   *    —— 这条最可靠，不依赖平台名怎么写。
+   */
+  const internal: any = raw?.internal ?? e?.bot?.internal
+  if (internal && (typeof internal.sendGroupForwardMsg === 'function' || typeof internal.sendPrivateForwardMsg === 'function')) return true
+  /** ② 其余只认 OneBot 系的平台名 */
+  const platform = platformOf(e)
+  if (!platform) return false
+  return ONEBOT_LIKE.test(platform)
 }
 
 /** 平台名 → 配置段名（合并转发的平台开关就写在各自的平台段里） */
@@ -302,7 +323,12 @@ export function withParseForward<F extends (e: any, next?: any) => any> (handler
         + '.forward 都没打开），按逐条发送处理')
       return await handler(e, next)
     }
-    if (!parseForwardPeer(e) || !canForwardParseResult(e)) return await handler(e, next)
+    if (!parseForwardPeer(e)) return await handler(e, next)
+    if (!canForwardParseResult(e)) {
+      // 说明白为什么没合并：以前这里静默退化，用户只看到「转发了但没收到」
+      logger.mark('[合并转发] 适配器 ' + (platformOf(e) || '未知') + ' 不支持合并转发（不是 OneBot 系、也没有 forward API），本次逐条发送')
+      return await handler(e, next)
+    }
     return await runWithForwardBag(parseForwardPeer(e), async () => {
       try {
         return await handler(e, next)

package/src/player/index.ts

@@ -22,7 +22,7 @@ import { PLUGIN_DIR_NAME, tryGetRuntime } from '../compat/runtime'
 // 解析阶段（「下载进度」指令读的就是这里登记的状态）
 import { DOWNLOAD_STAGES, clearParseStage, updateDownloadStage } from '../karin/module/utils/Network/Downloader'
 import { getParseOverride } from '../karin/module/utils/ParseOverride'
-import { registerPlayerRoutes } from './server'
+import { isStandalonePlayerReady, registerPlayerRoutes } from './server'
 import {
   normalizeExpireMinutes,
   registerPlayerSession,
@@ -151,6 +151,9 @@ export function forceOnlinePlayerAdapters (): string[] {
 }
 
 /** 这个**适配器**是不是在「强制在线播放」名单里 */
+/** 「本实例独立端口没监听上」这条只提醒一次，不然每条链接都刷一行 */
+let warnedStandaloneMismatch = false
+
 export function isForceOnlinePlayerAdapter (platform?: string): boolean {
   if (!platform) return false
   return forceOnlinePlayerAdapters().includes(String(platform).toLowerCase())
@@ -318,7 +321,20 @@ let warnedLocalBase = false
 export function buildPlayerLink (token: string): string {
   const config: any = tryGetRuntime()?.config ?? {}
   const base = String(config.playerBaseUrl ?? '').replace(/\/+$/, '')
-  if (base) return base + '/kkk/player/' + token
+  if (base) {
+    /**
+     * 本实例**没监听上独立端口**，但链接指向公网域名 —— 那个域名反代的是独立端口，
+     * 于是用户点开访问的其实是**抢到端口的另一台实例**（它的会话表里没有这个 token），
+     * 页面就会显示「链接已过期」。线上真实故障，这里至少要说清楚一次。
+     */
+    if (!isStandalonePlayerReady() && !warnedStandaloneMismatch) {
+      warnedStandaloneMismatch = true
+      logger.warn('[在线播放] 本实例没有监听上独立端口（被别的进程占了），但链接指向 ' + base
+        + ' —— 如果那个域名反代到同一个端口，用户点开会看到「链接已过期」（其实连的是抢到端口的那台实例）。'
+        + '请换一个「播放器端口」，或者停掉同机另一台占着该端口的 Koishi 实例')
+    }
+    return base + '/kkk/player/' + token
+  }
   const port = Number(config.playerPort) > 0 ? Number(config.playerPort) : koishiPort()
   const link = 'http://' + localAddress() + ':' + port + '/kkk/player/' + token
   if (!warnedLocalBase) {

package/src/player/server.ts

@@ -23,6 +23,8 @@ import path from 'node:path'
 
 import { logger } from 'node-karin'
 
+import { tryGetRuntime } from '../../compat/runtime'
+
 import { renderExpiredPage, renderPlayerPage } from './page'
 import {
   getPlayerSession,
@@ -58,6 +60,21 @@ export interface PlayerHttpResponse {
 /** 路由前缀 */
 export const PLAYER_ROUTE_PREFIX = '/kkk/player/'
 
+/** 独立端口是否真的监听上了：没监听上时链接指向的公网域名很可能打不到本实例 */
+let standaloneReady = false
+
+/** 查询独立端口是否监听上了（给「生成链接」那边做提示用） */
+export const isStandalonePlayerReady = (): boolean => standaloneReady
+
+/** 面板里配的「播放器公网地址」（没配就是空串） */
+function publicBaseUrl (): string {
+  try {
+    return String((tryGetRuntime()?.config as any)?.playerBaseUrl ?? '').trim()
+  } catch {
+    return ''
+  }
+}
+
 const TEXT_TYPE = 'text/plain; charset=utf-8'
 const HTML_TYPE = 'text/html; charset=utf-8'
 
@@ -387,13 +404,23 @@ export function registerPlayerRoutes ({ ctx, port }: { ctx: any, port: number })
       })
     })
     server.on('error', (error: any) => {
-      logger.warn('[在线播放] 播放器端口 ' + port + ' 启动失败（' + String(error?.code ?? error?.message ?? error)
-        + '），已退回 Koishi 自己的端口；请在配置里换一个端口或留空')
+      const code = String(error?.code ?? error?.message ?? error)
+      /**
+       * 端口被别的进程占了：**必须说清楚后果** —— 线上真实故障就是「用户点开播放链接看到『链接已过期』」。
+       *
+       * 成因：这台实例退回 Koishi 自己的端口，而另一台实例（比如同机的测试实例）抢到了这个端口；
+       * 域名反代到该端口 → 用户访问的其实是**另一台实例**，它当然没有这个会话。
+       */
+      logger.error('[在线播放] 播放器端口 ' + port + ' 被占用（' + code + '），本次退回 Koishi 自己的端口'
+        + (publicBaseUrl() ? '。注意：域名 ' + publicBaseUrl() + ' 反代的就是 ' + port + ' 的话，用户点开链接会看到「链接已过期」'
+          + ' —— 说明这台实例不是 8888 的占用者（同机另一个 Koishi 实例抢到了），换一个端口或先停掉那台实例' : ''))
+      standaloneReady = false
       // 端口被占用不能把插件带崩：退回 ctx.server 再挂一遍
       registerOnKoishi(ctx, disposers)
     })
     try {
       server.listen(port, () => {
+        standaloneReady = true
         logger.info('[在线播放] 播放器已监听独立端口 ' + port)
       })
       disposers.push(() => { try { server.close() } catch { /* 忽略 */ } })