@jerryliang122/openclaw-qqbot 1.0.7 → 1.0.9

package/src/tools/secret-input.ts

@@ -10,23 +10,31 @@
  * 记录里，且 AI 随后要读取使用；secret kind 是「受保护的机密」
  * （write-only，代理不可读），存进去 AI 反而用不了（2026-08-30 用户反馈）。
  *
+ * 会话目标解析（2026-09-23 修复）：双层回退——请求级 ALS 优先，框架
+ * run 级 deliveryContext 兜底。openclaw 把 turn 延迟到入站消息异步子树
+ * 之外执行（followup/collect 队列）时 ALS 为空，此前工具直接报
+ * 「无法获取当前会话目标」失败；工厂注册形式拿到的 deliveryContext
+ * 由框架按当前 run 的 sessionCtx 派生，延迟 turn 下依然正确（见
+ * src/tools/tool-session.ts）。
+ *
  * 非阻塞：工具发卡后立即返回，AI 应结束本轮；执行与回执完全由插件自治。
  */
 import type { OpenClawPluginApi } from 'openclaw/plugin-sdk';
-import { tryGetBotForAccount } from '../bot-instance.js';
+import { resolveGatewayForSend } from '../outbound/outbound-service.js';
 import { isValidSecretName } from '../features/secret-store-cli.js';
 import {
   DEFAULT_SECRET_INPUT_TTL_MS,
   cancelPendingSecretInput,
   registerPendingSecretInput,
 } from '../features/secret-input-store.js';
-import { getRequestAccountId, getRequestTarget } from '../request-context.js';
-import { parseTarget } from '../outbound/target.js';
+import { parseTarget, QQBOT_PREFIX_RE } from '../outbound/target.js';
 import { createPluginLogger } from '../utils/plugin-logger.js';
+import { resolveToolSessionRoute, type ToolDeliveryContext } from './tool-session.js';
 import type { InlineKeyboard, KeyboardButton } from '../types.js';
 
-// 工具 execute 无 logger 参数可传；request-context 在工具执行期间处于活跃，
-// 模块级 logger 的 enrichMeta 会自动带上 accountId/openId/messageId
+// 工具 execute 无 logger 参数可传；入站消息处理链路内 request-context
+// 处于活跃，模块级 logger 的 enrichMeta 自动带上 accountId/openId/messageId
+// （延迟/排队 turn 中 ALS 为空，路由解析回退 deliveryContext）
 const toolLog = createPluginLogger({ prefix: '[tool:secret-input]' });
 
 // ========== JSON Schema ==========
@@ -104,89 +112,113 @@ function json(data: unknown) {
 
 // ========== 注册入口 ==========
 
+// 工厂注册形式（2026-09-23 修复）：框架调用工厂时携带当次 run 的
+// deliveryContext——turn 被框架延迟/排队执行（ALS 已失效）时，工具
+// 仍能从 deliveryContext 解析当前会话目标（见 tool-session.ts）。
 export function registerSecretInputTool(api: OpenClawPluginApi): void {
   api.registerTool(
-    {
-      name: 'qqbot_secret_input',
-      label: 'QQBot Secret Input Card',
-      description:
-        '当用户要交付一个新密钥/凭据（API Key、Token 等）给系统保存时调用：' +
-        '向用户私聊发送一张密钥输入卡片，用户把密钥粘贴在输入框发送后，' +
-        '插件会自动拦截该消息并执行 openclaw secrets store set 完成保存，' +
-        '结果由插件直接回复用户。密钥内容不会进入对话。' +
-        '必须由用户提供 name 参数（环境变量名）。',
-      parameters: SecretInputSchema,
-      async execute(_toolCallId, params) {
-        const p = params as { name?: string; description?: string };
-        const name = (p.name ?? '').trim();
-        if (!name) return json({ error: 'name 为必填参数（环境变量名）' });
-        if (!isValidSecretName(name)) {
-          return json({
-            error:
-              '环境变量名不合法：必须以大写字母开头，仅含大写字母/数字/下划线，最长 128 字符' +
-              `（收到 "${name}"）。示例：OPENWEATHER_API_KEY。请修正后重试。`,
-          });
-        }
-        // 恒为 env：见文件头注释（卡片值 = 代理可读环境，secret 是 write-only 代理不可读）
-        const kind = 'env' as const;
+    (factoryCtx) => {
+      const delivery = (factoryCtx as { deliveryContext?: ToolDeliveryContext }).deliveryContext;
+      return {
+        name: 'qqbot_secret_input',
+        label: 'QQBot Secret Input Card',
+        description:
+          '当用户要交付一个新密钥/凭据（API Key、Token 等）给系统保存时调用：' +
+          '向用户私聊发送一张密钥输入卡片，用户把密钥粘贴在输入框发送后，' +
+          '插件会自动拦截该消息并执行 openclaw secrets store set 完成保存，' +
+          '结果由插件直接回复用户。密钥内容不会进入对话。' +
+          '必须由用户提供 name 参数（环境变量名）。',
+        parameters: SecretInputSchema,
+        async execute(_toolCallId: string, params: unknown) {
+          const p = params as { name?: string; description?: string };
+          const name = (p.name ?? '').trim();
+          if (!name) return json({ error: 'name 为必填参数（环境变量名）' });
+          if (!isValidSecretName(name)) {
+            return json({
+              error:
+                '环境变量名不合法：必须以大写字母开头，仅含大写字母/数字/下划线，最长 128 字符' +
+                `（收到 "${name}"）。示例：OPENWEATHER_API_KEY。请修正后重试。`,
+            });
+          }
+          // 恒为 env：见文件头注释（卡片值 = 代理可读环境，secret 是 write-only 代理不可读）
+          const kind = 'env' as const;
 
-        const target = getRequestTarget();
-        const accountId = getRequestAccountId() ?? 'default';
-        if (!target) {
-          return json({ error: '无法获取当前会话目标，此工具仅支持在 QQ Bot 消息会话中使用' });
-        }
-        const parsed = parseTarget(target);
-        if (parsed.scope !== 'c2c' || !parsed.targetId) {
-          return json({
-            error:
-              `密钥输入卡片仅支持私聊（c2c）会话，当前目标 ${target} 不适用` +
-              '——密钥不应出现在群聊中。请引导用户在私聊中发起。',
-          });
-        }
+          const route = resolveToolSessionRoute(delivery);
+          if (!route) {
+            return json({
+              error:
+                '当前运行没有 QQ 会话来源（可能由定时任务或框架内部触发的 run），' +
+                '无法定位要发卡的用户。请引导用户在 QQ 私聊中直接发起密钥输入，' +
+                '不要在无会话上下文的任务里调用本工具。',
+            });
+          }
+          // scope 必须在 parseTarget 之前显式判定：parseTarget 会把 channel:
+          // 归一化为 c2c（target.ts「channel 不被 SDK ChatScope 支持」），
+          // 频道会话若不在此拦截就会把频道 ID 当 c2c openid 发卡
+          // （Sourcery 复审，2026-09-23）
+          const scope = QQBOT_PREFIX_RE.exec(route.target)?.[1]?.toLowerCase();
+          const parsed = parseTarget(route.target);
+          if (scope !== 'c2c' || !parsed.targetId) {
+            return json({
+              error:
+                `密钥输入卡片仅支持私聊（c2c）会话，当前目标 ${route.target} 不适用` +
+                '——密钥不应出现在群聊或频道中。请引导用户在私聊中发起。',
+            });
+          }
 
-        const bot = tryGetBotForAccount(accountId);
-        if (!bot) {
-          return json({ error: `账户 ${accountId} 的 gateway 尚未启动，无法发送密钥输入卡片` });
-        }
+          // 发送账户解析：路由 accountId（缺省 default）→ 精确命中，
+          // 未命中且仅单账号运行时回退（与出站发送路径一致，issue #15 约定）。
+          // pending 必须登记在**实际发送账号**名下——捕获中间件按其网关的
+          // accountId 查询 pending。
+          const requestedAccountId = route.accountId ?? 'default';
+          const resolvedGateway = resolveGatewayForSend(requestedAccountId);
+          if (!resolvedGateway) {
+            return json({ error: `账户 ${requestedAccountId} 的 gateway 尚未启动，无法发送密钥输入卡片` });
+          }
+          const accountId = resolvedGateway.accountId;
+          const bot = resolvedGateway.gw.bot;
 
-        const description = p.description?.trim() || undefined;
-        // 先登记后发卡（避免「先发后登」竞态，同 multi-question 约定）
-        registerPendingSecretInput({
-          accountId,
-          senderOpenid: parsed.targetId,
-          name,
-          kind,
-          description,
-          createdAt: Date.now(),
-        });
-        try {
-          await bot.sendTextWithKeyboard(
-            { scope: 'c2c', targetId: parsed.targetId },
-            buildSecretInputCard(name, description, DEFAULT_SECRET_INPUT_TTL_MS / 60_000),
-            buildSecretInputKeyboard() as never,
-          );
-        } catch (err) {
-          cancelPendingSecretInput(accountId, parsed.targetId);
-          toolLog.error(`card send failed: ${err instanceof Error ? err.message : String(err)}`, { name, accountId });
-          return json({
-            error: `密钥输入卡片发送失败：${err instanceof Error ? err.message : String(err)}`,
+          const description = p.description?.trim() || undefined;
+          // 先登记后发卡（避免「先发后登」竞态，同 multi-question 约定）
+          registerPendingSecretInput({
+            accountId,
+            senderOpenid: parsed.targetId,
+            name,
+            kind,
+            description,
+            createdAt: Date.now(),
           });
-        }
+          try {
+            await bot.sendTextWithKeyboard(
+              { scope: 'c2c', targetId: parsed.targetId },
+              buildSecretInputCard(name, description, DEFAULT_SECRET_INPUT_TTL_MS / 60_000),
+              buildSecretInputKeyboard() as never,
+            );
+          } catch (err) {
+            cancelPendingSecretInput(accountId, parsed.targetId);
+            toolLog.error(`card send failed: ${err instanceof Error ? err.message : String(err)}`, { name, accountId });
+            return json({
+              error: `密钥输入卡片发送失败：${err instanceof Error ? err.message : String(err)}`,
+            });
+          }
 
-        toolLog.info(`card sent, pending registered name=${name} ttlMinutes=${DEFAULT_SECRET_INPUT_TTL_MS / 60_000}`);
+          toolLog.info(
+            `card sent, pending registered name=${name} accountId=${accountId} route=${route.source} ttlMinutes=${DEFAULT_SECRET_INPUT_TTL_MS / 60_000}`,
+          );
 
-        return json({
-          ok: true,
-          name,
-          kind,
-          command: `openclaw secrets store set ${name} --kind env --value <用户输入>`,
-          instruction:
-            '卡片已发送给用户。用户下一条私聊消息将被插件自动拦截（不会进入对话），' +
-            `执行上述命令保存密钥，成功/失败由插件直接回复用户。` +
-            '请直接结束本轮（可告知用户按卡片提示发送密钥），不要等待用户输入，' +
-            '也不要让用户把密钥内容发到对话里。',
-        });
-      },
+          return json({
+            ok: true,
+            name,
+            kind,
+            command: `openclaw secrets store set ${name} --kind env --value <用户输入>`,
+            instruction:
+              '卡片已发送给用户。用户下一条私聊消息将被插件自动拦截（不会进入对话），' +
+              `执行上述命令保存密钥，成功/失败由插件直接回复用户。` +
+              '请直接结束本轮（可告知用户按卡片提示发送密钥），不要等待用户输入，' +
+              '也不要让用户把密钥内容发到对话里。',
+          });
+        },
+      };
     },
     { name: 'qqbot_secret_input' },
   );

package/src/tools/tool-session.ts

@@ -0,0 +1,74 @@
+/**
+ * 工具执行时的会话路由解析（双层回退）
+ *
+ * 背景（2026-09-23 事故）：qqbot_secret_input 等工具此前仅靠
+ * request-context（AsyncLocalStorage）获取当前会话目标。ALS 只在
+ * handleMessage → dispatchToOpenClaw 的异步子树内有效；openclaw 框架
+ * 经常把 agent turn 延迟到该子树之外执行——活动 turn 之后排队的
+ * followup/collect 批处理、框架自有定时器驱动的队列 drain 等。此时
+ * 工具执行处 ALS 为空，工具报「无法获取当前会话目标」直接失败。
+ *
+ * 修复：优先 ALS（逐消息精确，含 messageId/openId 供日志富化）；
+ * ALS 缺失时回退到框架在工具工厂上下文里提供的 deliveryContext
+ * （openclaw 按当前 run 的 sessionCtx 派生的可信投递路由：
+ * { channel, to, accountId }，qqbot 通道的 to 即规范目标地址
+ * qqbot:c2c:{openid} / qqbot:group:{group_openid}）。
+ *
+ * 工厂注册形式：api.registerTool((ctx) => ({ ...tool }), { name })。
+ * 框架在装配工具面（或缓存描述符路径下的每次执行）时调用工厂，
+ * ctx 携带当次 run 的 deliveryContext——描述符缓存键包含
+ * deliveryContext，不会跨会话复用旧路由。
+ */
+
+import { getRequestAccountId, getRequestTarget } from '../request-context.js';
+
+/** 框架 deliveryContext 中本项目用到的字段（见 openclaw OpenClawPluginToolContext） */
+export interface ToolDeliveryContext {
+  channel?: string;
+  to?: string;
+  accountId?: string;
+  threadId?: string;
+}
+
+/** 解析出的会话路由 */
+export interface ToolSessionRoute {
+  /** 规范目标地址：qqbot:c2c:{openid} / qqbot:group:{group_openid} */
+  target: string;
+  /** 账户 ID（多账户场景；可能为空，由调用方决定回退策略） */
+  accountId?: string;
+  /** 路由来源：request-context（ALS）优先，delivery-context（框架 run 级）回退 */
+  source: 'request-context' | 'delivery-context';
+}
+
+/**
+ * 解析当前工具调用所属的会话路由。
+ *
+ * 返回 undefined 表示当前 run 没有可识别的 QQ 会话来源
+ * （如 cron/定时任务、内部 UI 发起的 run）——调用方应返回明确错误。
+ */
+export function resolveToolSessionRoute(
+  delivery?: ToolDeliveryContext | null,
+): ToolSessionRoute | undefined {
+  // 第一层：请求级 ALS（入站消息处理链路内，逐消息精确）
+  const alsTarget = getRequestTarget();
+  if (alsTarget) {
+    return {
+      target: alsTarget,
+      accountId: getRequestAccountId(),
+      source: 'request-context',
+    };
+  }
+
+  // 第二层：框架 run 级 deliveryContext（延迟/排队 turn 的回退）
+  const to = typeof delivery?.to === 'string' ? delivery.to.trim() : '';
+  if (!to) return undefined;
+  const channel = typeof delivery?.channel === 'string' ? delivery.channel.trim() : '';
+  // 防御：deliveryContext 属于其他通道（理论上不会发生）时不误认
+  if (channel && channel !== 'qqbot') return undefined;
+  if (!/^qqbot:(c2c|group|channel):/i.test(to)) return undefined;
+  return {
+    target: to,
+    accountId: delivery?.accountId,
+    source: 'delivery-context',
+  };
+}

package/src/utils/pkg-version.ts

@@ -16,7 +16,7 @@ export function getPackageVersion(): string {
 }
 
 /**
- * OpenClaw 框架版本（PluginRuntime.version，peer >=2026.9.2 恒存在）。
+ * OpenClaw 框架版本（PluginRuntime.version，peer >=2026.9.1 恒存在）。
  */
 export function getOpenclawVersion(runtimeVersion?: string): string {
   return runtimeVersion && runtimeVersion !== 'unknown' ? runtimeVersion : 'unknown';