Skip to content

Move files into Copilot CLI specific folders#312266

Merged
DonJayamanne merged 4 commits intomainfrom
don/due-frog
Apr 24, 2026
Merged

Move files into Copilot CLI specific folders#312266
DonJayamanne merged 4 commits intomainfrom
don/due-frog

Conversation

@DonJayamanne
Copy link
Copy Markdown
Contributor

Co-authored-by: Copilot copilot@github.com

Co-authored-by: Copilot <copilot@github.com>
Copilot AI review requested due to automatic review settings April 24, 2026 00:23
@DonJayamanne DonJayamanne self-assigned this Apr 24, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Copilot extension’s Copilot CLI chat-session implementation to reflect a “Copilot CLI specific folders” layout by fixing relative import paths after the re-organization.

Changes:

  • Updated relative imports in Copilot CLI vscode-node sources to reference platform/, util/, and shared chatSessions/common modules correctly.
  • Updated Vitest specs’ imports and vi.mock(...) module paths to match the new folder structure.
Show a summary per file
File Description
extensions/copilot/src/extension/chatSessions/copilotcli/vscode-node/test/copilotCLISDKUpgrade.spec.ts Adjusts test imports to correct platform and Copilot CLI node shim paths.
extensions/copilot/src/extension/chatSessions/copilotcli/vscode-node/test/copilotCLICustomizationProvider.spec.ts Fixes test imports to point at moved platform, util, and Copilot CLI node modules.
extensions/copilot/src/extension/chatSessions/copilotcli/vscode-node/test/chatSessionMetadataStoreImpl.spec.ts Updates test imports and vi.mock target path for cliHelpers after folder changes.
extensions/copilot/src/extension/chatSessions/copilotcli/vscode-node/test/chatSessionInitializer.spec.ts Updates initializer test imports to correct platform, util, and shared chatSessions paths.
extensions/copilot/src/extension/chatSessions/copilotcli/vscode-node/test/askUserQuestionHandler.spec.ts Fixes tool/test imports and typed mock import path for ILogService.
extensions/copilot/src/extension/chatSessions/copilotcli/vscode-node/copilotCLIPromptReferences.ts Updates util/* and vscodeTypes imports to match the new relative depth.
extensions/copilot/src/extension/chatSessions/copilotcli/vscode-node/copilotCLICustomizationProvider.ts Fixes platform/*, util/*, and Copilot CLI node imports for the new folder layout.
extensions/copilot/src/extension/chatSessions/copilotcli/vscode-node/copilotCLIChatSessionInitializer.ts Adjusts imports for platform/*, util/*, shared chatSessions/common, and Copilot CLI node modules.
extensions/copilot/src/extension/chatSessions/copilotcli/vscode-node/chatSessionMetadataStoreImpl.ts Updates imports for platform/*, util/*, shared chatSessions/common, and Copilot CLI node helpers.
extensions/copilot/src/extension/chatSessions/copilotcli/vscode-node/askUserQuestionHandler.ts Fixes imports for platform/*, util/*, tools, and Copilot CLI node user input helpers.

Copilot's findings

  • Files reviewed: 10/10 changed files
  • Comments generated: 0

@DonJayamanne DonJayamanne marked this pull request as ready for review April 24, 2026 02:13
@DonJayamanne DonJayamanne enabled auto-merge (squash) April 24, 2026 02:13
@DonJayamanne DonJayamanne merged commit d90a8a7 into main Apr 24, 2026
26 checks passed
@DonJayamanne DonJayamanne deleted the don/due-frog branch April 24, 2026 02:52
@vs-code-engineering vs-code-engineering Bot added this to the 1.118.0 milestone Apr 24, 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.

3 participants