makecindy / makecindy/cindy

维护者确认:PR #4457 企业飞书 OAuth Broker 与插件保留 ID

Open
#4,531 1 comment 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.