Skip to content
This repository was archived by the owner on May 20, 2026. It is now read-only.

clean up unused chat replay feature#4879

Merged
amunger merged 3 commits into
mainfrom
aamunger/deleteChatReplay
Mar 31, 2026
Merged

clean up unused chat replay feature#4879
amunger merged 3 commits into
mainfrom
aamunger/deleteChatReplay

Conversation

@amunger

@amunger amunger commented Mar 31, 2026

Copy link
Copy Markdown
Contributor

I think several things have broken with this feature with no complaints, and there are much easier ways to troubleshoot now, so we should just yank this out

@amunger amunger force-pushed the aamunger/deleteChatReplay branch from d92a0e3 to 63d9634 Compare March 31, 2026 19:56
@amunger amunger marked this pull request as ready for review March 31, 2026 20:26
Copilot AI review requested due to automatic review settings March 31, 2026 20:26

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes the unused “chat replay” feature (participant, notebook serializer, debug session, and tool) and updates request log exporting to a simpler JSON export format.

Changes:

  • Deleted the chat replay runtime implementation (participant/session provider/notebook serializer/debug adapter) and its associated tests.
  • Removed the tool_replay tool and all related tool name/registration/snapshot references.
  • Switched request log JSON export helpers from the replay module to chatLogExport and changed default export filenames to *.json.

Reviewed changes

Copilot reviewed 26 out of 27 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/extension/tools/node/toolReplayTool.tsx Deleted tool implementation used to replay tool results from recorded sessions.
src/extension/tools/node/allTools.ts Removed registration import for the deleted tool replay tool.
src/extension/tools/common/toolNames.ts Removed ToolReplay from tool enums and category mapping.
src/extension/replay/vscode-node/test/chatReplayNotebook.spec.ts Deleted notebook serializer/export tests for the removed replay feature.
src/extension/replay/vscode-node/replayDebugSession.ts Deleted debug adapter session used for stepping through replay logs.
src/extension/replay/vscode-node/chatReplaySessionProvider.ts Deleted chat session content provider for replay logs.
src/extension/replay/vscode-node/chatReplayParticipant.ts Deleted chat participant that executed replay steps and invoked the replay tool.
src/extension/replay/vscode-node/chatReplayNotebookSerializer.ts Deleted notebook serializer for *.chatreplay.json files.
src/extension/replay/vscode-node/chatReplayContrib.ts Deleted contribution wiring for the replay participant, notebook serializer, and debug adapter/commands.
src/extension/replay/node/replayParsing.spec.ts Deleted replay parser unit test.
src/extension/replay/node/replayParser.ts Deleted replay parser used to interpret exported replay JSON into steps.
src/extension/replay/node/chatReplayExport.ts Deleted replay export helpers (replaced by chat log export helpers).
src/extension/replay/common/chatReplayResponses.ts Deleted shared replay response queue/state container.
src/extension/prompts/node/agent/test/snapshots/agentPrompts-claude-sonnet-4.5/all_tools.spec.snap Updated snapshot to remove tool_replay from available tools list.
src/extension/prompts/node/agent/test/snapshots/agentPrompts-claude-sonnet-4.5/all_non_edit_tools.spec.snap Updated snapshot to remove tool_replay from available tools list.
src/extension/prompts/node/agent/test/snapshots/agentPrompts-claude-opus-4.6/all_tools.spec.snap Updated snapshot to remove tool_replay from available tools list.
src/extension/prompts/node/agent/test/snapshots/agentPrompts-claude-opus-4.6/all_non_edit_tools.spec.snap Updated snapshot to remove tool_replay from available tools list.
src/extension/prompts/node/agent/test/snapshots/agentPrompts-claude-opus-4.6-fast/all_tools.spec.snap Updated snapshot to remove tool_replay from available tools list.
src/extension/prompts/node/agent/test/snapshots/agentPrompts-claude-opus-4.6-fast/all_non_edit_tools.spec.snap Updated snapshot to remove tool_replay from available tools list.
src/extension/prompts/node/agent/test/snapshots/agentPrompts-claude-opus-4.5/all_tools.spec.snap Updated snapshot to remove tool_replay from available tools list.
src/extension/prompts/node/agent/test/snapshots/agentPrompts-claude-opus-4.5/all_non_edit_tools.spec.snap Updated snapshot to remove tool_replay from available tools list.
src/extension/log/vscode-node/requestLogTree.ts Switched export helpers to chatLogExport and changed default JSON export filenames.
src/extension/log/node/chatLogExport.ts Refactored export types/helpers to replace removed replay export module.
src/extension/extension/vscode-node/contributions.ts Removed registration of the deleted ChatReplayContribution.
package.nls.json Removed command titles for deleted edit tracing commands.
package.json Removed tool contribution, replay chat participant, replay commands/menus, notebook association, and debugger contribution for replay.
Comments suppressed due to low confidence (1)

src/extension/log/node/chatLogExport.ts:86

  • ExportedPrompt is exported (and imported by requestLogTree), but its logs property references ExportedLogEntry, which is not exported from this module. TypeScript will error on exporting a type that uses a private name. Export ExportedLogEntry (and any referenced types like ExportedLogMetadata), or inline those types into the exported interfaces.

@amunger amunger enabled auto-merge March 31, 2026 20:30
@amunger amunger added this pull request to the merge queue Mar 31, 2026
Merged via the queue into main with commit 9eb7bb4 Mar 31, 2026
24 checks passed
@amunger amunger deleted the aamunger/deleteChatReplay branch March 31, 2026 21:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants