维护者确认:PR #4457 企业飞书 OAuth Broker 与插件保留 ID
- Dominant language
- TypeScript
- Stars
- 2.7k
- Forks
- 395
- Avg merge
- 21h 48m
- Merged PRs (30d)
- 776
Description
需要维护者在 PR 上直接 Approve(需要作者修改则 Request Changes)。
## 这次改动
- Desktop 为企业飞书插件增加 OAuth Broker 动态 bootstrap / 一次性事务绑定,插件可不内置 AppID。
- 保留旧版 `clientId` 兼容路径;legacy `haoplay-feishu` 在 owner-namespace 恢复时有例外。
- 安装规则与 `idReserved` 文案从「保留前缀」改为「保留命名空间或 ID」,并把精确官方 ID 注入错误提示变量。
## 为何需要确认
1. **产品 / UI**:用户可见的插件安装规则与保留 ID 错误文案发生变化(多语言 common.json + i18n 默认变量)。
2. **插件基座**:改到 `cindy-brain` OAuth 账号/Broker/Flow 与 `shared/ghost.ts` 官方 ID 判定,影响全部已装插件的授权与保留身份边界;此类不因 bugfix/纯技术改动豁免。
3. **架构**:核心路径约 425 行,OAuth 配置来源从清单内置变为服务端事务,并调整官方 ID / 恢复白名单形状。
请重点核:事务与企业归属绑定、旧 App 配置兼容、以及 `haoplay-feishu` 恢复例外是否满足存量插件兼容(升级后用户什么都不做仍可用,且不把未验证来源抬成官方身份)。
依赖 cindy-server PR #656 已合并。真实飞书 OAuth 尚未手工验证。
---
关联 PR:#4457(作者 @kelv1nq1an);本 issue 由 review-pr 流程自动创建,用于先讨论该 PR 涉及的产品 / UI 变更,维护者确认后 PR 会恢复推进。
Contributor guide
Research direction
Start by reading PR #4457 and its changes to cindy-brain OAuth account/Broker/Flow, shared/ghost.ts, common.json, and the i18n default variables. Check the transaction-to-enterprise binding, legacy App configuration compatibility, and the haoplay-feishu restoration exception. Done means the maintainer can approve the PR with confidence that existing plugins remain usable and unverified sources are not treated as official identities.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- authentication, desktop, localization
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Active
- Clarity
- Needs clarification
- Newbie friendliness
- 15/100