feat(web): add AionUi-inspired session activity feed (ZER-671) - #32
Closed
ZeroPointSix wants to merge 2 commits into
Closed
feat(web): add AionUi-inspired session activity feed (ZER-671)#32ZeroPointSix wants to merge 2 commits into
ZeroPointSix wants to merge 2 commits into
Conversation
Owner
Author
Context fact card(上下文事实卡)Target
Verified facts
Relevant code and tests
CI/CD and validation entrypoints
Assumptions not allowed
Risks / blockers
Proposed change scope
Execution summaryProblem
Root cause / verified facts
Solution
Files changed
Validation
CI/CD
Remaining risks / follow-ups
|
Owner
Author
四审 Context fact card(上下文事实卡)Target
Verified facts
Relevant code and tests
CI/CD and validation entrypoints
Assumptions not allowed
Risks / blockers
Proposed change scope
Execution summaryProblem
Root cause / verified facts
Solution
Files changed
Validation
CI/CD
Remaining risks / follow-ups
四审结论:Request Changes / 暂不建议合并。当前只剩一个明确代码 blocker:sparse multi-hunk diff 必须保证每个真实修改区域可见。 |
7 tasks
ZeroPointSix
pushed a commit
that referenced
this pull request
Aug 12, 2026
This was referenced Aug 12, 2026
Owner
Author
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR implements the W4 presentation layer for the session activity stream with a fixed mock transcript, without waiting for the W3 transport implementation. The session detail view now uses a continuous Coding Agent-style stream rather than a conventional left/right chat bubble timeline.
The implementation adapts AionUi's Apache-2.0 tool-call normalization pattern to Ant Design, adds expandable read-only tool cards, terminal output, and bounded file-diff hunks, and regenerates the tracked web build assets.
Review Contract
Goal
Provide a usable, source-adapted W4 activity-feed renderer in the session detail page. It must render a fixed transcript containing user and assistant text, collapsed thinking, plans, turn boundaries, tool calls, terminal output, errors, and file diffs. Tool-call cards must expose status, summary, input, output, duration, and every ACP diff item without hiding changes in large files.
Non-goals
This PR does not implement the ZER-670 W3 snapshot API, the SSE transport, a W3-to-
ActivityEntryadapter, any session mutation controls, permission approval, terminal termination, input composition, or AionUi's Electron and local-file-system integrations. It does not attempt a general-purpose full diff engine; snapshots are rendered as bounded readable hunks.Accepted Residual Risks
The page remains mock-backed until W3 exposes a stable transcript contract, so production transcript payload compatibility cannot yet be verified end to end. The typed
ActivityEntryboundary and ACP normalizer isolate this risk; if the W3 contract differs, the follow-up adapter can translate at that boundary without changing the rendered components. Extremely large single replacements are deliberately bounded to 240 changed lines to protect browser responsiveness; the preview keeps both leading and trailing changed lines and clearly reports omitted material, while the complete source remains available from the originating tool result.Acceptance Criteria
update.contentarrays containing one or more diff items normalize into visible diff models.pnpm lint,pnpm test,pnpm build, andgit diff --checkpass.Follow-ups
ZER-670 will supply the live snapshot and SSE contract plus a W3-to-
ActivityEntryadapter. A later product decision may add an explicit deep-diff viewer or artifact download for cases where bounded preview output is insufficient.Validation
pnpm lintpnpm testpnpm buildgit diff --checkReview follow-up addressed
The latest review identified two concrete defects and a contract validation failure. The update now preserves every diff item in real ACP
update.contentarrays and renders all of them in the tool card.FileDiffnow builds a bounded common-prefix/common-suffix hunk rather than constructing an O(n×m) LCS matrix or slicing away remote edits. Regression tests cover ACP multi-diff input, a 500-line change at line 400, new files, and oversized replacements. This description now includes the required Review Contract sections.