Skip to content

Add support for other platforms#14

Merged
rajbos merged 6 commits intorajbos:mainfrom
readefries:feat/add-support-for-other-platforms
Sep 28, 2025
Merged

Add support for other platforms#14
rajbos merged 6 commits intorajbos:mainfrom
readefries:feat/add-support-for-other-platforms

Conversation

@readefries
Copy link
Contributor

@readefries readefries commented Sep 27, 2025

We (Claude and I) updated the renamed dependency (vcse -> @azure/vcse) as I don't like known security issues which was in the old source
and added support for multiple platforms.

This resolves #7

Copilot AI review requested due to automatic review settings September 28, 2025 11:51
Copy link
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 adds cross-platform support for tracking GitHub Copilot token usage and improves extension robustness by migrating from the deprecated vsce package to @vscode/vsce and enhancing logging capabilities.

  • Cross-platform path resolution for Windows, macOS, and Linux/Unix systems
  • Enhanced logging system with dedicated output channel
  • Improved extension initialization with smart delays for GitHub Codespaces
  • Better error handling and debugging capabilities

Reviewed Changes

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

File Description
src/extension.ts Major refactoring to add cross-platform support, logging system, and improved Copilot extension detection
package.json Updated dependency from deprecated vsce to @vscode/vsce

@rajbos
Copy link
Owner

rajbos commented Sep 28, 2025

Awesome @readefries ! I added some more logging options to be able to test on different platforms on what folders it tries to use or when it cannot find them.

@rajbos rajbos merged commit 74064b7 into rajbos:main Sep 28, 2025
9 checks passed
@readefries readefries deleted the feat/add-support-for-other-platforms branch September 28, 2025 13:28
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.

Test on different OS-es

2 participants

Comments