Skip to content

Add Codespaces Page - #296

Open
Noraa Junker (noraa-junker) wants to merge 9 commits into
microsoft:mainfrom
noraa-junker:addCodespaces
Open

Noraa Junker (noraa-junker) wants to merge 9 commits into
microsoft:mainfrom
noraa-junker:addCodespaces

Conversation

@noraa-junker

@noraa-junker Noraa Junker (noraa-junker) commented Jul 16, 2026 •

Copy link
Copy Markdown

Adds a codespaces page to the GitHub extension, where the logged in user sees all their with system information and current status.

image image

If VS Code is installed it shows an option to open the codespace in VS Code. Same for VS Code insiders.

As this requires additional permissions, existing users will see this error until they log out and log in again:
image

Copilot AI review requested due to automatic review settings July 16, 2026 22:11

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

Adds a new “Codespaces” top-level page to the GitHub extension so signed-in users can browse their Codespaces, see system/status details, and open a selected Codespace in VS Code / VS Code Insiders via protocol links. It also expands OAuth scopes to allow Codespaces access and introduces new UI strings/assets to support the feature.

Changes:

  • Introduces a new CodespacesPage and wires it into the extension’s top-level command set.
  • Adds the GitHub OAuth "codespace" scope to enable Codespaces API access.
  • Adds localized strings and VS Code / VS Code Insiders icon assets; updates test setup for the new command provider signature.

Reviewed changes

Copilot reviewed 6 out of 8 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
GitHubExtension/Strings/en-US/Resources.resw Adds new localized strings for the Codespaces page and related commands/messages.
GitHubExtension/Program.cs Instantiates CodespacesPage and passes it into the command provider.
GitHubExtension/GitHubExtensionCommandsProvider.cs Adds CodespacesPage to top-level commands and updates constructor signature/state.
GitHubExtension/DeveloperId/OAuthRequest.cs Requests additional OAuth scope needed for Codespaces access.
GitHubExtension/Controls/Pages/CodespacesPage.cs Implements the new Codespaces list page, VS Code open commands, and PATH-based VS Code detection.
GitHubExtension/Assets/vscode.svg Adds VS Code icon for command UI.
GitHubExtension/Assets/vscode-insiders.svg Adds VS Code Insiders icon for command UI.
GitHubExtension.Test/Helpers/TestSetupHelpers.cs Updates test composition to pass a CodespacesPage into GitHubExtensionCommandsProvider.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread GitHubExtension/Controls/Pages/CodespacesPage.cs
Comment thread GitHubExtension/Controls/Pages/CodespacesPage.cs
Comment thread GitHubExtension/Controls/Pages/CodespacesPage.cs
Comment thread GitHubExtension/Strings/en-US/Resources.resw Outdated
Comment thread GitHubExtension/Controls/Pages/CodespacesPage.cs

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 encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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 encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@michaeljolley

Copy link
Copy Markdown
Contributor

Noraa Junker (@noraa-junker) could you merge main and push? That should trigger the correct CI builds.

@noraa-junker

Copy link
Copy Markdown
Author

Noraa Junker (Noraa Junker (@noraa-junker)) could you merge main and push? That should trigger the correct CI builds.

Michael Jolley (@michaeljolley) Done. Could you trigger the CI again?

@michaeljolley

Copy link
Copy Markdown
Contributor

Noraa Junker (@noraa-junker), got some build failures.

@noraa-junker

Copy link
Copy Markdown
Author

Noraa Junker (Noraa Junker (@noraa-junker)), got some build failures.

Michael Jolley (@michaeljolley) Can you tell me why the pipeline failed. I don't even see where it failed in the log files...

@michaeljolley

Copy link
Copy Markdown
Contributor

Noraa Junker (Noraa Junker (Noraa Junker (@noraa-junker))), got some build failures.

Michael Jolley (Michael Jolley (@michaeljolley)) Can you tell me why the pipeline failed. I don't even see where it failed in the log files...

https://github.com/microsoft/CmdPalGitHubExtension/actions/runs/34212276204/job/102124462625?pr=296

@noraa-junker

Copy link
Copy Markdown
Author

Noraa Junker (Noraa Junker (Noraa Junker (Noraa Junker (@noraa-junker)))), got some build failures.

Michael Jolley (Michael Jolley (Michael Jolley (@michaeljolley))) Can you tell me why the pipeline failed. I don't even see where it failed in the log files...

https://github.com/microsoft/CmdPalGitHubExtension/actions/runs/34212276204/job/102124462625?pr=296

Thanks! I fixed it and hope that it works now.

@noraa-junker

Copy link
Copy Markdown
Author

This time it wasn't my fault:

image

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