Ship Copilot as a built-in extension#302778
Merged
joaomoreno merged 3896 commits intomainfrom Apr 7, 2026
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Adds GitHub Copilot (as an extension source) to the repo by introducing it as a Git submodule under extensions/.
Changes:
- Added
extensions/copilotas a submodule pinned to a specific commit. - Added
.gitmodulesconfiguration to track the new submodule.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
extensions/copilot |
Adds a gitlink entry pinning the extension submodule to a specific commit. |
.gitmodules |
Registers the extensions/copilot submodule and defines its remote URL. |
joaomoreno
force-pushed
the
joao/copilot-builtin
branch
2 times, most recently
from
2bbdac8 to
4e4a578
Compare
joaomoreno
force-pushed
the
joao/copilot-builtin
branch
4 times, most recently
from
1e4512e to
a9eb901
Compare
* fix: address review feedback on OTel agent activity metrics * fix: guard recordEditAcceptance for accept/reject only, fix doc wording
…ead of the git extension API (#4799) * Sessions - update workspace tracker so that it uses file watcher instead of the git extension API * Also filter out events from the node_modules folder * Pull request feedback
- Exclude copilot from local extension compilation (excludedExtensions) - Copilot job now packages and publishes VSIX as pipeline artifact - All client jobs poll for and download the copilot VSIX artifact - downloadCopilotVsix.ts handles waiting, downloading, and extracting - Polling fails fast if the Copilot job itself fails - Added BUILDS_API_URL to CI jobs that were missing it
Extract shared step templates from product-copilot.yml into: - copilot/setup-steps.yml (node, auth, cache, emsdk, npm ci) - copilot/build-steps.yml (vscode-capi mixin, npm run build) - copilot/test-steps.yml (setup, typecheck, lint, compile, all tests) Add product-copilot-release.yml as a new pipeline entry point for publishing Copilot Chat to the VS Marketplace, extending the vscode-engineering stable.yml template. Update product-copilot.yml to use the shared templates.
…ations (#4805) * PromptsServiceImpl: avoid using openTextDocument to not trigger validations * Update src/platform/promptFiles/common/promptsServiceImpl.ts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * fix test --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Add transcript line count to compaction summary Expose getLineCount() on ISessionTranscriptService so the summarization prompt can tell the model how many lines the transcript had before compaction. This helps the model target the right region when reading the transcript file for context recovery. - Track lineCount on IActiveSession, increment in _bufferEntry - On session resume, count existing lines from the file on disk - Pass transcriptLineCount through SummarizedConversationHistory props to SummaryMessageElement - Include 'The transcript had N lines before compaction' in the hint * Update src/extension/prompts/node/agent/summarizedConversationHistory.tsx Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…sholds (#4812) * Fix premature compaction: decouple ratio from tool tokens, raise thresholds - Compute compaction ratios against baseBudget instead of budgetThreshold so users with many tools aren't penalized with earlier compaction - Reduce safety multiplier from 0.85 to 0.90 (10% covers observed tokenizer discrepancy without wasting 15% of context) - Raise background compaction kick-off from 75% to 80% Fixes #304475 * Address PR feedback: clamp messageBudget, clarify toolTokens guard comment
* Fix background compaction not applied between tool calls - Add addPendingSummary/pendingSummaries to Turn class for storing summaries before setResponse is called (mid-tool-call-loop) - Update normalizeSummariesOnRounds to fall back to pendingSummaries when resultMetadata isn't available yet * Add test for pendingSummaries fallback in normalizeSummariesOnRounds
…4785) * Attach response output message IDs only to phased assistant messages * noop * Remove stray probe file * noop2 * Remove stray probe file * noop3 * Remove stray probe file * adding correct messageID for the outgoing messages * adding correct messageID for the outgoing messages * adding correct messageID for the outgoing messages * Update defaultIntentRequestHandler snapshots
* Background - do not stage changes for folder sessions * Fix tests
This reverts commit 5a65c80.
joaomoreno
force-pushed
the
joao/copilot-builtin
branch
2 times, most recently
from
34cc7e8 to
d40d265
Compare
Move the 3 VS Code-dependent Copilot test steps (extension tests, completions core tests, sanity tests) from the standalone Copilot job into the Windows, Linux, and macOS platform test jobs. - Parameterize copilot/test-steps.yml with runIntegrationTests flag - Create copilot/test-integration-steps.yml for platform-specific setup - Add Copilot integration tests after core tests in each platform - product-copilot-release.yml is unaffected (uses default=true)
* initial commit fix request id for subagent and add parentToolCallId * change logging to info * fix headerRequestId missing * fix * fix... * remove debug logging * change back runSubagent changes --------- Co-authored-by: Yevhen Mohylevskyy <yevhenmohylevskyy@Yevhens-MacBook-Pro-2.local>
Co-authored-by: Copilot <copilot@github.com>
joaomoreno
requested review from
TylerLeonhardt,
alexdima,
hediet,
lszomoru and
rzhao271
as code owners
lszomoru
previously approved these changes
Apr 7, 2026
lszomoru
previously approved these changes
Apr 7, 2026
joaomoreno
force-pushed
the
joao/copilot-builtin
branch
from
8ce52a3 to
90fc912
Compare
Contributor
Screenshot ChangesBase: Changed (3)
|
| Before | After |
|---|---|
 |
 |
editor/suggestWidget/MethodCompletions/Dark
| Before | After |
|---|---|
 |
 |
editor/suggestWidget/MethodCompletions/Light
| Before | After |
|---|---|
 |
 |
alexdima
approved these changes
Apr 7, 2026
lszomoru
approved these changes
Apr 7, 2026
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.
No description provided.