fix(vscode-ide-companion): Fix UI display issues with server-side timestamp and file path extraction - #1682
Merged
Merged
Conversation
- 在 sessionUpdateMetaSchema 中添加 timestamp 字段 - 为 MessageEmitter 和 ToolCallEmitter 方法添加 timestamp 参数 - 在 HistoryReplayer 中传递并使用记录的 timestamp - 为用户消息、代理消息和工具调用添加 _meta.timestamp - 添加 toEpochMs 工具方法处理 ISO 字符串和 epoch ms - 更新所有相关测试用例验证 timestamp 传递 Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>
- 在 SessionUpdateMeta 中添加 timestamp 字段 - 为 UserMessageChunkUpdate 和 ToolCallUpdate 添加 _meta 字段 - 在 QwenAgentManager 中使用 server timestamp 而非客户端时间 - 在 QwenSessionUpdateHandler 中解析并传递 tool call timestamp - 在 useToolCalls 中改进 timestamp 解析逻辑 - 修正 App.tsx 中 timestamp 处理,使用 nullish-coalescing operator - 为 ToolCallUpdateData 和 ToolCallUpdate 类型添加 timestamp 和 _meta feat(integration-tests): add file location extraction for tool calls feat(webui): enhance ReadToolCall component with content preview and expand/collapse refactor(webui): simplify SaveMemoryToolCall component UI Co-authored-by: Qwen-Coder <qwen-coder@alibabacloud.com>
yiliang114
requested review from
DennisYu07,
LaZzyMan,
Mingholy,
gwinthis,
pomelo-nwu and
tanzhenxin
as code owners
February 1, 2026 05:16
Contributor
📋 Review SummaryThis PR addresses important UI display issues in the VSCode IDE Companion extension by introducing server-side timestamps for correct message ordering and fixing missing file path information in exported Session HTML. The changes span across CLI/ACP layer, VSCode IDE Companion, and WebUI components with comprehensive test updates. 🔍 General Feedback
🎯 Specific Feedback🟡 High
🟢 Medium
🔵 Low
✅ Highlights
|
Contributor
Code Coverage Summary
CLI Package - Full Text ReportCore Package - Full Text ReportFor detailed HTML reports, please see the 'coverage-reports-22.x-ubuntu-latest' artifact from the main CI run. |
xaelistic
pushed a commit
to xaelistic/qwen-code
that referenced
this pull request
Jun 7, 2026
…-show-bugs fix(vscode-ide-companion): Fix UI display issues with server-side timestamp and file path extraction
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 fixes UI display issues in the VSCode IDE Companion extension:
Problem Background
Issue 1: Message Ordering Problems
The previous implementation relied on client-side timestamps for message ordering, which could cause:
Issue 2: Missing File Paths in Exported Sessions
Exported Session HTML was missing file path information for tool calls (Read, Write, Edit, etc.), resulting in:
Changes
1. CLI/ACP Layer - Server-Side Timestamp Support
schema.ts: AddserverTimestampfield definition to message and tool call schemastypes.ts: Extend type definitions to supportserverTimestamppropertyBaseEmitter.ts: Add timestamp generation logic to base emitterMessageEmitter.ts: Integrate server-side timestamp in message emitterToolCallEmitter.ts: Integrate server-side timestamp in tool call emitterHistoryReplayer.ts: Support sorting by serverTimestamp in history replayer, with corresponding unit tests2. VSCode IDE Companion Layer - UI Ordering Fix
acpTypes.ts: Add serverTimestamp type definitionchatTypes.ts: Extend chat types to support new fieldqwenAgentManager.ts: Handle timestamp logic in agent managerqwenSessionUpdateHandler.ts: Sort messages by serverTimestamp in session update handlerApp.tsx: Adapt main app component to new sorting logicuseToolCalls.ts: Use serverTimestamp for tool call sortinguseWebViewMessages.ts: Adapt WebView messages hook3. WebUI Layer - Component Fixes
ReadToolCall.tsx: Fix Read tool call componentSaveMemoryToolCall.tsx: Fix SaveMemory tool call component4. Integration Test Tools - Fix Missing File Paths in Exported Sessions
export-html-from-chatrecord-jsonl.js:extractLocationsfunction to extract file location information fromrawInputortoolCallResultabsolute_path(read_file, write_file),file_path(edit),path(generic)Impact
Test Plan
export-html-from-chatrecord-jsonl.jsand verify file paths and links display correctly