feat(im): 新增 Lark 机器人接入

[DavidShenXD]

这次改了什么

摘要

在 Cindy 现有 IM transport 和桌面编排中加入 Lark 支持，让用户可以在飞书与 Lark 之间选择对应开放平台，并复用一致的私聊、附件、交互卡片、Agent 交互和跨对话通知能力。实现保持 Cindy 自有结构、命名与文案，同时保留现有飞书内部 channel ID 以兼容历史配置和调用方。

变更类型

• feat 新功能
• fix 缺陷修复
• refactor / perf 重构或性能优化
• docs / test / chore 文档、测试或工程维护
• 其他：

范围

• 关联 Issue / 需求：Lark IM 工具接入
• 本 PR 包含：飞书/Lark 服务选择、SDK 与开放平台域名路由、注册流程、凭证服务类型持久化、owner 边界、Lark 对话标题、冲突弹窗、MCP 工具文案、四语 i18n、术语登记及回归测试
• 明确不包含：新增技术设计 Markdown、服务端改动、Mobile 原生配置或依赖、真实企业凭证
• 用户可见变化：Settings 的飞书机器人入口升级为“飞书 / Lark 机器人”，可选择对应服务并打开其开放平台；Lark 对话和冲突提示显示正确品牌
• 是否存在 breaking change：无。旧凭证未保存服务类型时继续按飞书处理，现有 IPC/channel/tool ID 保持兼容

UI 变化

• Settings 的 IM Bot 配置新增飞书/Lark 服务选择；注册入口、已保存服务与冲突弹窗会随服务切换
• 引用的设计规范：docs/design-rules/DESIGN.md §2 Color Palette & Roles、§4 Component Stylings、§10 Theme System & Token Reference（含 Light / Dark 双模式门槛）、§11 Voice & Content。新增控件沿用语义 token，没有新增硬编码颜色

怎么验证的

自动验证

pnpm test:unit
结果：通过；Desktop、Mobile、IM、MCP 及其余要求执行的 workspace 均为 PASS

pnpm --filter desktop run --if-present typecheck
pnpm --filter @cindy/im build
pnpm --filter @cindy/mcps build
pnpm --filter @cindy/wechat-ilink run --if-present typecheck
结果：全部通过

pnpm --filter @cindy/im test
结果：22 个测试文件、243 项测试通过

pnpm check:i18n
pnpm check:i18n-glossary
结果：四语 key 一致，术语门禁无新增违规

pnpm check:dco
git diff --check origin/main..HEAD
结果：通过；PR 范围为 1 个 DCO 签名提交

手工验证

• 不涉及：本次未使用真实 Lark 企业凭证进行端到端联调

未执行的验证

• 未执行真实 Lark 企业 App 的 WebSocket、收发消息、附件和卡片联调：当前环境没有企业凭证
• 未执行 Settings 的 Light / Dark 实机目检；实现沿用现有语义 token，但不将代码复用视为已完成视觉验证

风险

风险分类

• 无已知风险
• SQLite / migration
• system prompt
• 协议兼容
• 权限 / 安全 / 用户数据
• 原生层 / fingerprint / OTA
• 跨平台差异
• 其他：依赖 Lark 开放平台与 SDK 的线上行为

影响与回滚

• 影响范围：Desktop Settings、@cindy/im 的飞书兼容 transport、相关 Desktop IM 编排和内置 IM 通知工具。凭证继续使用现有本地 secrets 存储，只新增服务类型标记
• 回滚 / 降级方式：回滚本提交即可；用户侧也可清除机器人凭证或继续选择飞书服务
• Mobile runtime fingerprint：未变化。本 PR 没有修改 apps/mobile、Mobile 原生配置、原生依赖、config plugin 或原生模块

提交前检查

• 已 review 完整 diff
• 每个 commit 都带 DCO 签名（git commit -s，见 DCO）
• UI 改动已在「UI 变化」注明引用的设计规范章节
• 未提交凭证、令牌或授权文件
• 已补充必要的既有工具说明；未提交新增设计 Markdown
• 已确认测试结果或说明未执行原因

[Copilot]

Pull request overview

在现有 Feishu IM bot 通道的基础上引入 Lark 支持：增加“Feishu / Lark”服务选择与域名路由、凭证服务类型持久化与兼容旧凭证默认 Feishu，并同步更新 Desktop Settings UI、MCP 工具提示文案、i18n/术语登记与相关测试覆盖。

Changes:

• 在 @cindy/im 的 Feishu 通道中新增 service: 'feishu' | 'lark'，用于 OpenAPI / WebSocket SDK 域名路由与凭证持久化（旧凭证缺省回落 Feishu）。
• Desktop Settings 的“飞书机器人”升级为“飞书 / Lark 机器人”，支持服务切换、打开对应开放平台、以及冲突弹窗与通知文案的品牌适配。
• MCP 工具提示与术语表/i18n 更新，并新增/更新多处单测覆盖服务路由与存储语义。

Reviewed changes

Copilot reviewed 37 out of 38 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
packages/wechat-ilink/src/transport.ts	调整 iLink aes_key 注释表述
packages/lizi-mcps/src/prompts/cindy_feishu_bot/tools/send_message_to_user.md	工具说明扩展到 Feishu/Lark
packages/lizi-mcps/src/prompts/cindy_feishu_bot/tools/send_file_to_user.md	工具说明扩展到 Feishu/Lark
packages/lizi-mcps/src/prompts/cindy_feishu_bot/tools/list_tools.md	工具列表文案更新为 Feishu/Lark
packages/lizi-mcps/src/prompts/cindy_feishu_bot/tools/call_tool.md	工具调用说明文案更新
packages/lizi-mcps/src/cindy_feishuBotMcpServer.ts	MCP server 文案/格式调整与错误提示更新
packages/lizi-im/src/feishu/wsClient.ts	WSClient 增加按 service 的 Domain 路由与相关文案更新
packages/lizi-im/src/feishu/storage.ts	凭证新增 service 存储键，兼容旧凭证默认 Feishu
packages/lizi-im/src/feishu/outbound.ts	OpenAPI Client 增加按 service 的 Domain 路由
packages/lizi-im/src/feishu/messages.ts	生命周期离线文案泛化为“当前 IM”
packages/lizi-im/src/feishu/ipc.ts	IPC 支持保存/注册时传入 service，并将 service 暴露到 public state
packages/lizi-im/src/feishu/internal-types.ts	新增 FeishuService 与 BotCredentials.service 字段
packages/lizi-im/src/feishu/index.ts	FeishuIM 增加 getService() 供上层路由/展示
packages/lizi-im/src/feishu/appRegistration.ts	注册流程支持按 brand 路由 endpoints
packages/lizi-im/src/feishu/tests/wsClientConflict.test.ts	覆盖 WSClient domain 路由与格式调整
packages/lizi-im/src/feishu/tests/storageService.test.ts	新增 service 存储/回滚语义单测
packages/lizi-im/src/feishu/tests/serviceRouting.test.ts	新增 outbound domain 路由单测
packages/lizi-im/src/feishu/tests/ipcAccountScope.test.ts	IPC payload 增加 service 并覆盖注册/切换语义
packages/lizi-im/src/feishu/tests/appRegistration.test.ts	新增 app registration 的 Lark endpoint 单测
i18n/GLOSSARY.md	术语表登记 Lark（proposed）
i18n/glossary.json	glossary.json 新增 lark 条目（proposed）
apps/desktop/src/renderer/vite-env.d.ts	Renderer 侧 ElectronAPI 类型增加 service 与 registrationBegin 入参
apps/desktop/src/renderer/i18n/locales/zh-CN/common.json	中文文案：Feishu/Lark 选择、冲突弹窗与通知文案更新
apps/desktop/src/renderer/i18n/locales/ko/common.json	韩文文案：Feishu/Lark 选择与相关文案更新
apps/desktop/src/renderer/i18n/locales/ja/common.json	日文文案：Feishu/Lark 选择与相关文案更新
apps/desktop/src/renderer/i18n/locales/en/common.json	英文文案：Feishu/Lark 选择与相关文案更新
apps/desktop/src/renderer/hooks/useFeishuBotRegistration.ts	注册 hook 增加 service 入参并透传到 IPC
apps/desktop/src/renderer/hooks/useFeishuBot.ts	状态/缓存/保存逻辑增加 service 字段
apps/desktop/src/renderer/hooks/tests/useFeishuBot.test.ts	覆盖 service hydrate + save 的单测
apps/desktop/src/renderer/components/settings/FeishuBotSection.tsx	Settings UI 增加 Feishu/Lark 服务选择与开放平台跳转路由
apps/desktop/src/renderer/components/feishuBot/FeishuConflictDialogHost.tsx	冲突弹窗 host 增加按 service 路由打开平台链接
apps/desktop/src/renderer/components/feishuBot/FeishuConflictDialog.tsx	冲突弹窗增加 service 占位符渲染
apps/desktop/src/preload/preload.ts	preload API：getState/save/registrationBegin 增加 service 透传
apps/desktop/src/main/im/shared/types.ts	session 标题前缀支持 string 或 thunk
apps/desktop/src/main/im/shared/turnRunner.ts	生成标题逻辑支持动态前缀函数
apps/desktop/src/main/im/feishu/uiText.ts	Feishu UI 文案泛化为“IM”表述
apps/desktop/src/main/im/feishu/adapter.ts	会话标题前缀随 service 切换（Feishu/Lark）
apps/desktop/src/main/im/feishu/tests/feishuAdapter.test.ts	覆盖 adapter 在 Lark 服务下的标题前缀行为

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[DavidShenXD]

反馈已处理

已处理：

• 冲突弹窗复用现有飞书 / Lark 服务名翻译，中文环境正确显示“飞书”。
• 修正 NO_CHAT_CONTEXT 描述，使其与“无当前 chatId 且 bot 未绑定 owner”的实际语义一致。

变更：

• FeishuConflictDialog 的 serviceName 改由 settings.feishuBot.services.* 提供。
• 更新 cindy_feishu_bot.call_tool 工具说明。

验证：

• pnpm --filter desktop run --if-present typecheck：通过。
• pnpm --filter @cindy/mcps test：通过（423 passed，2 skipped）。
• pnpm check:i18n：通过。
• pnpm check:i18n-glossary：通过。
• pnpm test:unit：通过。
• pnpm check:dco：通过（2 commits signed off）。

备注：

• 已 rebase 最新 main 并使用 --force-with-lease 更新 PR 分支；未执行 UI 实机目检，未等待 GitHub Actions 完成。

[Copilot]

Pull request overview

Copilot reviewed 37 out of 38 changed files in this pull request and generated no new comments.

Suppressed comments (6)

apps/desktop/src/renderer/components/settings/FeishuBotSection.tsx:339

• 这个服务选择控件用 role="radiogroup"/"radio" 模拟单选，但当前按钮缺少 roving tabIndex 与方向键切换处理，键盘用户很难在两个选项间移动，属于可访问性缺口。建议跟 Settings 里 LanguageSection 的实现对齐：仅选中项 tabIndex=0，并在 ArrowLeft/Right/Up/Down 时切换服务。

              <button
                key={service}
                type="button"
                role="radio"
                aria-checked={selected}
                onClick={() => props.setService(service)}
                className={cn(

apps/desktop/src/renderer/i18n/locales/zh-CN/common.json:470

• 这里的 aria 文案仍然只提到“飞书通知”，但该开关已升级为“飞书 / Lark 消息”。为避免读屏文案与实际功能不一致，建议同步更新 feishuAria。

      "feishuLabel": "对话完成或需要回复时发送飞书 / Lark 消息",
      "feishuHint": "通过已绑定的飞书或 Lark 机器人，向你私聊发送一条消息；仅在窗口失焦时发送。",
      "feishuDisabledHint": "需要先在「飞书 / Lark 机器人」中配置 bot 并向它发送一条消息以完成绑定。",
      "feishuAria": "切换对话完成的飞书通知"

apps/desktop/src/renderer/i18n/locales/en/common.json:470

• 这里的 aria 文案仍然写的是“FeiShu notifications”，但该开关现在覆盖 Feishu / Lark。建议把 feishuAria 更新为与 UI 一致的“Feishu / Lark notifications”，并统一大小写（上文已使用 Feishu）。

      "feishuLabel": "Send a Feishu / Lark DM when a session finishes or needs a reply",
      "feishuHint": "Sends a private message via the linked Feishu or Lark bot; only when the window is unfocused.",
      "feishuDisabledHint": "First set up the bot in \"Feishu / Lark Bot\" and DM it once to complete binding.",
      "feishuAria": "Toggle session-done FeiShu notifications"

apps/desktop/src/renderer/i18n/locales/ja/common.json:470

• ここは aria 文言がまだ Feishu のみに言及していますが、機能は Feishu / Lark に拡張されています。feishuAria も同じく Feishu / Lark に更新して、読み上げ内容と実機表示を一致させてください。

      "feishuLabel": "セッション完了または返信が必要な時に Feishu / Lark メッセージを送信",
      "feishuHint": "連携済みの Feishu または Lark ボットからプライベートメッセージを送信します。ウィンドウが非アクティブの場合のみ送信。",
      "feishuDisabledHint": "先に「Feishu / Lark ボット」で設定し、ボットに 1 度メッセージを送ってバインドを完了してください。",
      "feishuAria": "セッション完了時の Feishu 通知を切り替え"

apps/desktop/src/renderer/i18n/locales/ko/common.json:470

• aria 안내 문구가 아직 Feishu만 가리키고 있는데, 실제 기능은 Feishu / Lark 로 확장되었습니다. feishuAria 를 Feishu / Lark 로 맞춰 스크린리더 문구와 UI 의미가 어긋나지 않게 해주세요.

      "feishuLabel": "세션 완료 또는 응답 필요 시 Feishu / Lark 메시지 전송",
      "feishuHint": "연동된 Feishu 또는 Lark 봇을 통해 개인 메시지를 보냅니다. 창이 비활성 상태일 때만 전송됩니다.",
      "feishuDisabledHint": "먼저 \"Feishu / Lark 봇\"에서 봇을 설정하고 한 번 메시지를 보내 바인딩을 완료하세요.",
      "feishuAria": "세션 완료 Feishu 알림 전환"

packages/lizi-im/src/feishu/outbound.ts:62

• 错误信息仍然写死为 “feishu connection not established”，在支持 Lark 后会误导排查（尤其是 service=lark 时）。建议把文案改成更中性的 “IM bot connection not established”。

function ensureClient(): Lark.Client {
  if (!client)
    throw new Error('[feishu/outbound] Lark.Client not bound — feishu connection not established');
  return client;

[greptile-apps]

Greptile Summary

本次变更在现有飞书兼容通道中加入 Lark 服务支持，并完成注册、连接与界面品牌路由。

• 持久化凭证所属服务，并按飞书或 Lark 选择 SDK 域名、开放平台入口及用户文案
• 注册轮询可根据租户品牌从飞书端点切换至 Lark，并持续处理 pending 与 slow_down
• 为桌面设置、冲突提示、会话标题、MCP 文案和四语本地化补充对应支持

Confidence Score: 5/5

当前修复覆盖了此前报告的 Lark 跨域凭证补拉及暂态轮询问题，PR 看起来可以安全合并。

当前代码在发现 Lark 租户后切换轮询端点，并在 Lark 返回 pending、slow_down 或凭证尚不完整时继续轮询，因此没有阻塞性故障残留。

Important Files Changed

Filename	Overview
packages/lizi-im/src/feishu/ipc.ts	注册轮询现可识别 Lark 租户、跨域补拉凭证，并在暂态响应后继续轮询。
packages/lizi-im/src/feishu/appRegistration.ts	注册响应保留暂态阶段发现的租户品牌，为后续端点切换提供依据。
packages/lizi-im/src/feishu/wsClient.ts	WebSocket 客户端根据持久化服务类型选择飞书或 Lark SDK 域名。
packages/lizi-im/src/feishu/storage.ts	凭证存储新增服务类型，同时为历史凭证保留飞书默认值。
apps/desktop/src/renderer/components/settings/FeishuBotSection.tsx	设置界面新增飞书与 Lark 服务选择及对应品牌入口。

Sequence Diagram

sequenceDiagram
    participant UI as Desktop Settings
    participant IPC as IM IPC
    participant F as Feishu Registration
    participant L as Lark Registration
    participant WS as IM WebSocket
    UI->>IPC: 开始飞书/Lark 注册
    IPC->>F: 获取设备码并开始轮询
    alt 检测到 Lark 租户
        F-->>IPC: "tenantBrand=lark"
        IPC->>L: 继续轮询直至凭证完整
        L-->>IPC: clientId + clientSecret
    else 飞书租户
        F-->>IPC: clientId + clientSecret
    end
    IPC->>IPC: 持久化凭证及服务类型
    IPC->>WS: 使用对应服务域名连接

Loading

_{Reviews (7): Last reviewed commit: "fix(im): persist Lark registration polli..." | Re-trigger Greptile}

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 84b35bd2f1

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

[DavidShenXD]

反馈已处理

已处理：

• 恢复 Lark 一键注册的 Feishu bootstrap：设备码 begin 固定请求 accounts.feishu.cn，用户选择的服务仅决定验证页域名。
• 恢复 Lark 跨域凭证补拉：首次从 Feishu 轮询识别到 tenantBrand=lark 且缺少 clientSecret 时，立即切到 Lark 端点补拉完整凭证。

变更：

• 调整 appRegistration.ts 与 ipc.ts 的注册端点和跨域轮询逻辑，并保留异步补拉后的 account scope 校验。
• 更新注册端点测试，新增 Feishu→Lark 两阶段轮询及 Lark 凭证持久化测试。
• 修复提交：9a7c48a0，已推送至 xdt/auto-35j0hm。

验证：

• pnpm --filter @cindy/im exec vitest run src/feishu/__tests__/appRegistration.test.ts src/feishu/__tests__/ipcAccountScope.test.ts：通过，2 个文件、19 个测试。
• pnpm --filter @cindy/im run build：通过。
• pnpm test:unit -- --workspace-concurrency=1：通过，全仓 unit workspace 均通过。
• pnpm --filter @cindy/im run --if-present typecheck：通过（该包无 typecheck script，命令按规则跳过；已由 build 执行 tsc --noEmit）。
• pnpm check:dco：通过。
• pnpm --filter @cindy/im run lint：未通过，仅命中本次未修改的 Telegram 既有未使用函数（telegram/inbound.ts、telegram/index.ts）。

备注：

• 未执行真实 Lark 租户扫码注册：需要外部账号环境；协议行为已按官方 Lark CLI 实现核对并由单元测试覆盖。
• 未等待 GitHub Actions 或其他自动检查完成。

[Copilot]

Pull request overview

Copilot reviewed 37 out of 38 changed files in this pull request and generated no new comments.

Suppressed comments (2)

packages/lizi-im/src/feishu/outbound.ts:62

• ensureClient() 的异常信息仍写死为 “feishu connection not established”，但当前 outbound 已支持按 service 路由到 Feishu/Lark 两套域名；这里会在排障时误导为仅 Feishu 连接问题。建议改为更通用的 IM 表述（保留 tag 前缀即可）。

function ensureClient(): Lark.Client {
  if (!client)
    throw new Error('[feishu/outbound] Lark.Client not bound — feishu connection not established');
  return client;

packages/lizi-im/src/feishu/ipc.ts:86

• 这里对 appId/appSecret 的长度校验发生在 trim() 之前：当 payload 传入全空白字符串时会通过校验，但随后被 trim 成空串并继续保存/连接，可能导致凭证被写入为空值并引发后续连接异常。建议对 trim 后的值再做一次非空校验（或改为先 trim 再校验）。

      (p.service !== 'feishu' && p.service !== 'lark') ||
      p.appId.length === 0 ||
      p.appSecret.length === 0
    ) {
      throw new Error('[INVALID_PAYLOAD] appId, appSecret and service required');

[chatgpt-codex-connector]

💡 Codex Review

cindy/packages/lizi-im/src/feishu/ipc.ts

Line 304 in 9a7c48a

if (success.tenantBrand === 'lark' && !success.clientSecret) {

Handle cross-brand pending registration responses

Fresh evidence after the earlier bootstrap feedback is that this version now starts polling on Feishu, but it only switches to Lark after a Feishu success result with missing credentials. The official Lark CLI says the cross-brand tenant signal can arrive alongside non-final responses and that incomplete credentials should keep polling (source); in that case pollAppRegistration returns pending without exposing tenant_brand, so this loop keeps polling Feishu until expiry (or emits the missing-secret error) even though the user approved the Lark flow. Lark one-click registration can therefore time out despite successful authorization.

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

[MagicLizi]

⏸️ 本 PR 触发 product 签字门（新增飞书 Bot 集成功能），等待放行名单成员 Approve 后继续。

讨论 issue 见下方链接。放行方式：直接在本 PR 上 Approve；需要修改就 Request Changes，作者改完后重新 Approve 即放行。

讨论 issue:#1220

[DavidShenXD]

反馈已处理

已处理：

• 已确认 product 签字门有效：本 PR 新增飞书 / Lark Bot 接入，需要等待放行名单成员在 PR 上 Approve。

变更：

• 无代码或文档改动；该反馈属于人工产品审核流程，不能也不应通过代码修改消除。

验证：

• 未执行：本轮仅处理 product 签字门反馈，没有代码变更。

备注：

• PR 当前等待 product 放行名单成员 Approve；未自行 Approve，也未 merge。需要调整时可由审核人 Request Changes，修改完成后再重新 Approve。

[Copilot]

Pull request overview

Copilot reviewed 40 out of 41 changed files in this pull request and generated no new comments.

Suppressed comments (5)

packages/lizi-im/src/feishu/outbound.ts:61

• 这里的异常信息仍写成“feishu connection not established”，但该 outbound 客户端现在同时用于 Feishu / Lark（domain 由 creds.service 决定）。错误信息应改为更通用的 IM 语义，避免在 Lark 场景下误导排障。
  apps/desktop/src/renderer/i18n/locales/en/common.json:470
• 本次已将通知文案升级为“Feishu / Lark”，但 aria 文案仍是 “FeiShu notifications”，屏幕阅读器会读到与实际功能不一致的品牌信息。建议同步更新为 Feishu / Lark。

      "feishuLabel": "Send a Feishu / Lark DM when a session finishes or needs a reply",
      "feishuHint": "Sends a private message via the linked Feishu or Lark bot; only when the window is unfocused.",
      "feishuDisabledHint": "First set up the bot in \"Feishu / Lark Bot\" and DM it once to complete binding.",
      "feishuAria": "Toggle session-done FeiShu notifications"

apps/desktop/src/renderer/i18n/locales/zh-CN/common.json:470

• 本次已将通知文案升级为“飞书 / Lark”，但 aria 文案仍是“飞书通知”，对辅助技术用户会产生不一致的提示。建议同步更新 aria 文案为“飞书 / Lark”。

      "feishuLabel": "对话完成或需要回复时发送飞书 / Lark 消息",
      "feishuHint": "通过已绑定的飞书或 Lark 机器人，向你私聊发送一条消息；仅在窗口失焦时发送。",
      "feishuDisabledHint": "需要先在「飞书 / Lark 机器人」中配置 bot 并向它发送一条消息以完成绑定。",
      "feishuAria": "切换对话完成的飞书通知"

apps/desktop/src/renderer/i18n/locales/ja/common.json:470

• 本次已将通知文案升级为“Feishu / Lark”，但 aria 文案仍是“Feishu 通知”。为避免屏幕阅读器读到不一致的品牌信息，建议同步更新为 Feishu / Lark。

      "feishuLabel": "セッション完了または返信が必要な時に Feishu / Lark メッセージを送信",
      "feishuHint": "連携済みの Feishu または Lark ボットからプライベートメッセージを送信します。ウィンドウが非アクティブの場合のみ送信。",
      "feishuDisabledHint": "先に「Feishu / Lark ボット」で設定し、ボットに 1 度メッセージを送ってバインドを完了してください。",
      "feishuAria": "セッション完了時の Feishu 通知を切り替え"

apps/desktop/src/renderer/i18n/locales/ko/common.json:470

• 本次已将通知文案升级为“Feishu / Lark”，但 aria 文案仍是“Feishu 알림”。为保证辅助技术提示一致，建议同步更新 aria 文案为 Feishu / Lark。

      "feishuLabel": "세션 완료 또는 응답 필요 시 Feishu / Lark 메시지 전송",
      "feishuHint": "연동된 Feishu 또는 Lark 봇을 통해 개인 메시지를 보냅니다. 창이 비활성 상태일 때만 전송됩니다.",
      "feishuDisabledHint": "먼저 \"Feishu / Lark 봇\"에서 봇을 설정하고 한 번 메시지를 보내 바인딩을 완료하세요.",
      "feishuAria": "세션 완료 Feishu 알림 전환"

[chatgpt-codex-connector]

💡 Codex Review

cindy/packages/lizi-im/src/feishu/ipc.ts

Line 304 in 5f9fff6

if (success.tenantBrand === 'lark' && !success.clientSecret) {

Keep polling after Lark brand discovery

When a Lark registration reports tenant_brand=lark before complete credentials are ready, this branch only switches after a success result with a missing secret and then performs a single Lark poll; if Lark returns authorization_pending/slow_down or another incomplete 200 response, the flow falls through to the missing-secret error instead of continuing on the Lark domain. The official discovery loop switches as soon as tenant_brand is present, even on pending responses, and keeps polling incomplete credentials (source), so valid Lark one-click registrations can fail intermittently depending on poll timing.

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

[dashhuang]

产品签字确认，同意恢复推进。此批准仅代表产品 / UI 方向确认；现有技术审阅意见仍需按流程处理。

[Copilot]

Pull request overview

Copilot reviewed 40 out of 41 changed files in this pull request and generated no new comments.

Suppressed comments (3)

apps/desktop/src/renderer/components/settings/FeishuBotSection.tsx:366

• 这里用 role="radiogroup"/role="radio" 实现服务切换，但缺少 roving tabIndex 与方向键切换处理；键盘用户在组内导航体验会明显弱于同项目的 LanguageSection（支持 Arrow 键）。建议给选中项设置 tabIndex=0，其余为 -1，并处理 ArrowLeft/Right/Up/Down 切换选项。

              <button
                key={service}
                type="button"
                role="radio"
                aria-checked={selected}

packages/lizi-im/src/feishu/outbound.ts:62

• ensureClient() 抛错文案仍写成“feishu connection not established”，在接入 Lark 后该错误也可能发生于 Lark 场景，会误导排障方向。建议改成与服务无关的 IM 连接描述。
  packages/lizi-im/src/feishu/ipc.ts:308
• 当 tenantBrand=lark 且首次 poll 未返回 clientSecret 时，这里会再 poll 一次 Lark；但这次 poll 没有 try/catch。若网络/SDK 抛错，会导致 pollRegistrationInBackground 直接抛出（因为它是后台 fire-and-forget），从而漏发 registration-status:error，且可能产生未处理的 Promise rejection。建议对二次 poll 同样做错误捕获并广播 error。

[DavidShenXD]

反馈已处理

已处理：

• 已修复 Lark 品牌发现信息在 authorization_pending 或凭证未完整响应中被丢弃的问题；检测到 tenant_brand=lark 后，轮询状态机立即从飞书域切换到 Lark 域。
• 已修复 Lark 二次 poll 只尝试一次的问题；Lark 返回 pending、slow_down 或未完整凭证时，会在原注册截止时间内继续轮询，不再提前判定注册失败。

变更：

• appRegistration poll 结果携带可选 tenantBrand，并将无错误但凭证未完整的响应视为 pending。
• 注册状态机维护当前 pollBrand，一次性即时切换到 Lark，并以实际轮询域作为最终凭证品牌兜底。
• 新增品牌发现、跨域切换、Lark pending 重试和不完整凭证重试测试。

验证：

• pnpm --filter @cindy/im test：247/247 通过。
• pnpm --filter @cindy/im build：通过。
• 相关文件 ESLint：通过。
• pnpm test:unit：通过。
• pnpm check:dco：通过。

备注：

• 已推送提交 a45377b；未等待 GitHub Actions 或其他远端检查完成。

[Copilot]

Pull request overview

Copilot reviewed 40 out of 41 changed files in this pull request and generated no new comments.

Suppressed comments (1)

packages/lizi-im/src/feishu/outbound.ts:62

• 这里的报错文案仍写成 “feishu connection not established”，在已支持 Lark 的情况下会误导排查（同样的调用路径也可能是 Lark 域名/凭证）。建议改成与品牌无关的描述（例如 “IM connection not established” / “IM bot connection not established”），或把 service 打出来。

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a45377bfa8

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

[MagicLizi]

@DavidShenXD 👋 这个 PR 还有 1 条 review conversation 没 resolve(apps/desktop/src/renderer/i18n/locales/en/common.json),auto-review 因此暂时跳过、没法继续审查 / 合并。

如果你已经按评论改完或回应了,请到对应 thread 上点 Resolve conversation;全部 resolve 后,下一轮 auto-review 会自动重新审查这个 PR。

[DavidShenXD]

反馈已处理

已处理：

• 修正 CN/dev 个人云账号场景的个人 IM 提示，不再展示该场景不可用的 Lark。

变更：

• 将 en、zh-CN、ja、ko 的 personalFeishuOnly 文案恢复为仅描述飞书；未改动渠道可见性逻辑。

验证：

• pnpm test:unit：通过。
• pnpm --filter desktop run --if-present typecheck：通过。
• pnpm check:i18n-glossary：通过。

备注：

• 首轮测试曾遇到 Windows Vitest fork / 测试锁环境性失败，最终完整重跑通过；无剩余代码风险。

[Copilot]

Pull request overview

Copilot reviewed 40 out of 41 changed files in this pull request and generated no new comments.

Suppressed comments (1)

packages/lizi-im/src/feishu/outbound.ts:62

• 这里的异常信息仍写成“feishu connection not established”，但该模块已支持根据凭据选择 Feishu/Lark domain；错误文案建议改为更中性的 IM/机器人连接描述，避免误导排查。

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5139ab53c9

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

[Copilot]

Pull request overview

Copilot reviewed 40 out of 41 changed files in this pull request and generated no new comments.

Suppressed comments (5)

packages/lizi-im/src/feishu/outbound.ts:62

• ensureClient() 的错误信息仍写成“feishu connection not established”，在已支持 Lark 的情况下会误导排障（同一错误会发生在 Lark 连接未绑定时）。建议改成更中性的 IM 表述（或同时涵盖 Feishu/Lark）。
  apps/desktop/src/renderer/i18n/locales/en/common.json:470
• 这里的 aria 文案仍写成 “FeiShu notifications”，但同一段落已升级为 “Feishu / Lark”。建议同步更新以避免辅助功能读屏与实际功能不一致（顺便统一 Feishu 的大小写）。

      "feishuLabel": "Send a Feishu / Lark DM when a session finishes or needs a reply",
      "feishuHint": "Sends a private message via the linked Feishu or Lark bot; only when the window is unfocused.",
      "feishuDisabledHint": "First set up the bot in \"Feishu / Lark Bot\" and DM it once to complete binding.",
      "feishuAria": "Toggle session-done FeiShu notifications"

apps/desktop/src/renderer/i18n/locales/zh-CN/common.json:470

• 这里的 aria 文案仍只提“飞书”，但同一段落已改为“飞书 / Lark”。Screen reader 会读到不一致的服务名，建议同步更新。

      "feishuLabel": "对话完成或需要回复时发送飞书 / Lark 消息",
      "feishuHint": "通过已绑定的飞书或 Lark 机器人，向你私聊发送一条消息；仅在窗口失焦时发送。",
      "feishuDisabledHint": "需要先在「飞书 / Lark 机器人」中配置 bot 并向它发送一条消息以完成绑定。",
      "feishuAria": "切换对话完成的飞书通知"

apps/desktop/src/renderer/i18n/locales/ja/common.json:470

• aria 文案仍只提 Feishu，但同一段落已变为 Feishu / Lark。读屏会产生不一致，建议同步更新。

      "feishuLabel": "セッション完了または返信が必要な時に Feishu / Lark メッセージを送信",
      "feishuHint": "連携済みの Feishu または Lark ボットからプライベートメッセージを送信します。ウィンドウが非アクティブの場合のみ送信。",
      "feishuDisabledHint": "先に「Feishu / Lark ボット」で設定し、ボットに 1 度メッセージを送ってバインドを完了してください。",
      "feishuAria": "セッション完了時の Feishu 通知を切り替え"

apps/desktop/src/renderer/i18n/locales/ko/common.json:470

• aria 문구가 아직 Feishu만 언급하지만, 같은 섹션의 라벨/힌트는 Feishu / Lark로 확장되었습니다. 스크린리더 일관성을 위해 aria 문구도 함께 갱신하는 게 좋습니다.

      "feishuLabel": "세션 완료 또는 응답 필요 시 Feishu / Lark 메시지 전송",
      "feishuHint": "연동된 Feishu 또는 Lark 봇을 통해 개인 메시지를 보냅니다. 창이 비활성 상태일 때만 전송됩니다.",
      "feishuDisabledHint": "먼저 \"Feishu / Lark 봇\"에서 봇을 설정하고 한 번 메시지를 보내 바인딩을 완료하세요.",
      "feishuAria": "세션 완료 Feishu 알림 전환"

[MagicLizi]

Review passed: well-structured Lark integration with proper input validation at IPC boundaries, transactional credential storage with rollback, backward-compatible defaults, correct owner-boundary clearing, masked secrets in logs, and comprehensive test coverage. No P0/P1 findings.

[MagicLizi]

Lark 接入架构扎实——凭证事务性存储带回滚、服务切换时正确清理边界、secrets 全程 mask,测试也覆盖到了新路径的关键分支。