makecindy / makecindy/cindy

PR #4389 需要维护者确认:新增 move_sessions 会话移动能力

Open
#4,499 1 comment 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 https://github.com/makecindy/cindy/pull/4389 (`feat(helper): 新增 move_sessions 会话移动工具`, 作者 @cofyc) 命中维护者确认门,需要维护者在 PR 上直接 Approve 后才能继续自动审查合并。

## 这次改了什么
Part of #4385 (T3)。在 `cindy_helper` control 类目新增 `move_sessions`(把历史会话移动到项目 / 移回对话),并建立后续会话操作工具共用的骨架:
- `packages/lizi-mcps`:共享类型与批次护栏、`move_sessions` 工具、MCP server 的 `sessionOps` 注入点
- Desktop main:依赖全注入的 host 业务体 + 真实装配;写路径复用 `updateSessionInDb`
- 守卫与 GUI 同口径(远程 / 运行中含 Orca worker / IM 接管 / 已归档删除 / 空草稿 / review / Bot 会话整批拒绝)
- 另改审批策略:改写工作目录的 `move_sessions` 需逐次用户确认

## 为何需要确认
**product(用户看得见的交互)**
- 标题是 feat,且给 agent 增加了可代办「移动到项目 / 对话」的新能力。
- GUI 像素不变,但用户可见交互流向变了:模型可批量改写会话工作区;改目录时还会弹确认。
- 这不是内部逻辑修补,也不是已有功能的机械补充。

**arch(结构变了)**
- 核心路径约 986 行(≥150),且任意类型 ≥800。
- 跨包 API / 模块边界有实质变化:`XdtHelperMcpDeps.sessionOps`、`DesktopMcpProvidersDeps.isSessionTurnRunning`、`updateSessionInDb` 新增 `beforeWrite` 钩子、`SqliteBindingStore.runExclusive`、新的 sessionOperations 业务层。
- 不是同结构内的实现替换或机械搬迁。

## 请求
请维护者直接在 PR 上 **Approve**。若认为需要作者修改,请 **Request Changes**。不要只留言或只摘标签。

---
关联 PR:#4389(作者 @cofyc);本 issue 由 review-pr 流程自动创建,用于先讨论该 PR 涉及的产品 / UI 变更,维护者确认后 PR 会恢复推进。

Contributor guide

Open the contributing guide

Research direction

Start with PR #4389 and review its changes in packages/lizi-mcps and the Desktop main session-operations path, including the listed dependency and approval-policy changes. Done means a maintainer directly approves the PR or requests changes; this issue is a review gate rather than an independent implementation task.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
ai, desktop
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Active
Clarity
Clearly specified
Newbie friendliness
10/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.