Skip to content

fix(reader): roll back rewound conversation turns - #64

Closed
El1mir wants to merge 1 commit into
aa2246740:mainfrom
El1mir:main
Closed

El1mir wants to merge 1 commit into
aa2246740:mainfrom
El1mir:main

Conversation

@El1mir

@El1mir El1mir commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

作为 #63 的开发部分已经完成了,而且实际在本机上测试过了

效果为回退时完全回退到上次用户消息输入前的reader,并为这个收起过程添加了动画

aa2246740 added a commit that referenced this pull request Oct 26, 2026
Conflicts were again the shared lint cleanup, the AssistantNode that #65
moved to AssistantNode.tsx (#64 did not change it), and the sticky-lane
rule #66 rewrote next to #64's retiring-lane margin; keep both CSS
intents. Restore inboxAttachmentCount, which #64 deleted as unused while
the pending-inbox row still calls it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
aa2246740 added a commit that referenced this pull request Oct 26, 2026
…ula (#74)

Conflicts were the shared lint cleanup (McpAppFrame setReady, live-turn
foldableRun, GroupStatus props), the AssistantNode move from #65, and the
#66/#64 sticky-lane and turn-spacing rules next to the new .userCluster
rules. Kept the integration side for all of those and took #74's bubble
rules. Restored inboxAttachmentCount, which #74 also dropped as unused
while #65 still calls it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
aa2246740 added a commit that referenced this pull request Oct 26, 2026
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
aa2246740 added a commit that referenced this pull request Oct 26, 2026
, #74 and CI lint

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@aa2246740

Copy link
Copy Markdown
Owner

谢谢!这个 PR 的提交 242461e 已经随 #76 合进 main(合并提交 5f77e6f),作者署名保留。为了接到当前 main 上,它是重新整理过的,提交 ID 变了,所以 GitHub 不会把这里标成已合并,我手动关掉。

在你的基础上,验收时又补了几处:

  • 按 dsh-rewind-plugin 实际的裁剪范围回退,不会把被裁掉的工具调用恢复回来,也不会打乱历史顺序;
  • 被回退的内容里有焦点时切走标签页,内容照样收起,焦点移到还在的轮次;
  • 阅读列被聚焦时不再画整列的焦点框。

在真实宿主里用 dsh-rewind-plugin 0.15.3 做了真实回退:220ms 收起、减少动态效果时直接移除、取消或失败时内容保留,都通过了。会随下一个版本发布。

@aa2246740 aa2246740 closed this Oct 26, 2026
@aa2246740 aa2246740 mentioned this pull request Oct 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants