Skip to content

Improve FRE Github Copilot install flow#262

Open
haonanttt wants to merge 5 commits into
mainfrom
dev/haonanttt/freOptimize
Open

Improve FRE Github Copilot install flow#262
haonanttt wants to merge 5 commits into
mainfrom
dev/haonanttt/freOptimize

Conversation

@haonanttt

Copy link
Copy Markdown
Contributor

Summary of the Pull Request

Prewarm winget and migrate winget installation from cli to COM API to capture detailed installation log for further diagnostic

References and Relevant Issues

#231

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

PR Checklist

  • Closes #xxx
  • Tests added/passed
  • Documentation updated
    • If checked, please file a pull request on our docs repo and link it here: #xxx
  • Schema updated (if necessary)

Copilot AI review requested due to automatic review settings June 10, 2026 06:22
Comment thread src/cascadia/TerminalApp/FreOverlay.cpp Fixed
Comment thread src/cascadia/TerminalApp/FreOverlay.cpp Fixed
Comment thread src/cascadia/TerminalApp/FreOverlay.h Fixed
Comment thread src/cascadia/TerminalApp/FreOverlay.h Fixed
…mize

# Conflicts:
#	src/cascadia/TerminalApp/FreOverlay.cpp

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

This PR improves the First Run Experience (FRE) bootstrap path for installing GitHub Copilot (and Node.js when needed) by pre-warming WinGet’s source cache and switching package installation from invoking winget.exe to the WinGet COM/WinRT API for more reliable diagnostics in packaged-app context.

Changes:

  • Add a single-flight, per-process “winget source update” pre-warm kicked off during FRE initialization and awaited before installs.
  • Replace winget install ... process execution with Microsoft.Management.Deployment.PackageManager-based installs (with structured status/error logging and timeouts).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
src/cascadia/TerminalApp/FreOverlay.h Adds static coordination state and helpers for winget source pre-warm.
src/cascadia/TerminalApp/FreOverlay.cpp Implements pre-warm execution + switches installs to WinGet COM API with detailed result logging and bounded wait.

Comment thread src/cascadia/TerminalApp/FreOverlay.cpp
Comment thread src/cascadia/TerminalApp/FreOverlay.cpp
@github-actions

This comment has been minimized.

Copilot AI review requested due to automatic review settings June 10, 2026 06:43

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Comment thread src/cascadia/TerminalApp/FreOverlay.h
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.

4 participants