makecindy / makecindy/cindy

维护者确认:PR #3255 架构改动(并行权限请求串行队列)

Open
#3,302 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
TypeScript
Stars
2.7k
Forks
401
Avg merge
21h 48m
Merged PRs (30d)
776

Description

PR #3255(https://github.com/makecindy/cindy/pull/3255)触发了架构确认门。

## 为什么要确认
为修并行权限卡互相覆盖导致的 600s 超时,这次在 `maker-ipc/register.ts` 引入了新的 `PermissionQueue` / `PermissionRunContext` / `cancelPermissionQueueForSession`,把并行 permission interaction 改成按会话串行调度,并改变接管(`/ctr`)时排队请求的处理方式。

这是 Desktop main 权限交互调度的模块内新契约,不是同结构补丁。

## 怎么确认
请维护者直接在 PR 上 Approve。需要修改就 Request Changes,作者改完后再 Approve 即放行。普通评论或摘标签不构成通过。

当前仍有未 resolve 的 review thread(含接管时排队权限如何迁移),确认时请一并看。

---
关联 PR:#3255(作者 @yuaiccc);本 issue 由 review-pr 流程自动创建,用于先讨论该 PR 涉及的技术架构调整,维护者确认后 PR 会恢复推进。

Contributor guide

Open the contributing guide

Research direction

Start with PR #3255 and maker-ipc/register.ts, focusing on PermissionQueue, PermissionRunContext, and cancelPermissionQueueForSession. Read the unresolved review thread about transferring queued permissions during /ctr takeover. Done means the architectural concerns are resolved and a maintainer approves the PR.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
authorization, desktop
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
15/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.