Skip to content

feat: Add compare-scopes command to verify PAT scope requirements Co-authored-by: JoannaaKL <67866556+JoannaaKL@users.noreply.github.com>#1

Open
jm2velez-maker wants to merge 4 commits intomainfrom
copilot/add-compare-pat-scopes-command
Open

feat: Add compare-scopes command to verify PAT scope requirements Co-authored-by: JoannaaKL <67866556+JoannaaKL@users.noreply.github.com>#1
jm2velez-maker wants to merge 4 commits intomainfrom
copilot/add-compare-pat-scopes-command

Conversation

@jm2velez-maker
Copy link
Owner

No description provided.

Copilot AI and others added 4 commits January 8, 2026 09:05
- Created compare_scopes.go with compare-scopes command
- Fetches token scopes from GitHub API using pkg/scopes/fetcher
- Compares with required scopes from inventory
- Reports missing and extra scopes
- Supports text and json output formats
- Added comprehensive unit tests
- Created script/compare-scopes wrapper script

Co-authored-by: JoannaaKL <67866556+JoannaaKL@users.noreply.github.com>
- Created compare_scopes.go with complete implementation
  - Fetches token scopes from GitHub API using pkg/scopes/fetcher
  - Compares with required scopes from server inventory
  - Handles scope hierarchy (parent scopes cover child scopes)
  - Reports missing and extra scopes intelligently
  - Supports text and JSON output formats
  - Works with GitHub Enterprise via --gh-host flag
- Added comprehensive unit tests in compare_scopes_test.go
  - Tests scope comparison logic
  - Tests scope hierarchy handling
  - Tests edge cases (empty scopes, etc.)
  - All tests passing
- Created script/compare-scopes wrapper script for easy CLI usage
- Added detailed documentation in docs/compare-scopes.md
  - Usage examples with various configurations
  - Output format examples
  - Scope hierarchy explanation
  - GitHub Enterprise support
  - Common workflows and troubleshooting scenarios

Co-authored-by: JoannaaKL <67866556+JoannaaKL@users.noreply.github.com>
Co-authored-by: JoannaaKL <67866556+JoannaaKL@users.noreply.github.com>
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.

2 participants