Skip to content

Remove workspace context provider - #8929

Merged
Alex Ross (alexr00) merged 1 commit into
mainfrom
alexr00/small-planarian
Sep 4, 2026
Merged

Remove workspace context provider#8929
Alex Ross (alexr00) merged 1 commit into
mainfrom
alexr00/small-planarian

Conversation

@alexr00

Copy link
Copy Markdown
Member

No description provided.

Copilot AI lite review requested due to automatic review settings September 4, 2026 13:09
@alexr00
Alex Ross (alexr00) enabled auto-merge (squash) September 4, 2026 13:09
@alexr00 Alex Ross (alexr00) self-assigned this Sep 4, 2026

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.

Copilot review overview

🟢 Approval recommended

The workspace context provider removal is complete (no remaining references/usages found) and the remaining chat context providers continue to be registered as before.

Review tier: Lite
Findings: None

What changed in this PR

This PR removes the extension’s chat workspace context provider so the extension no longer contributes repository/branch/active PR info via registerChatWorkspaceContextProvider, while keeping the PR attach/tab context providers intact.

Changes:

  • Deleted WorkspaceContextProvider implementation (and its Git/Logger/event wiring) from src/lm/pullRequestContextProvider.ts.
  • Removed registration/initialization of the workspace chat context provider from extension activation in src/extension.ts.
File Description
src/​lm/​pullRequestContextProvider.ts Removes the workspace chat context provider class and its related imports.
src/​extension.ts Stops instantiating/registering the workspace chat context provider during initialization.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@alexr00
Alex Ross (alexr00) merged commit c0f7541 into main Sep 4, 2026
7 checks passed
@alexr00
Alex Ross (alexr00) deleted the alexr00/small-planarian branch September 4, 2026 13:16
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