Skip to content

Fix help text, add missing descriptions for pipx/mas, add 3 built-in themes#39

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/brief-issues-and-priorities
Draft

Fix help text, add missing descriptions for pipx/mas, add 3 built-in themes#39
Copilot wants to merge 2 commits intomainfrom
copilot/brief-issues-and-priorities

Conversation

Copy link
Copy Markdown

Copilot AI commented Apr 3, 2026

Four top-priority gaps identified in the repo: stale CLI help listing only 13 of 36 managers, two managers with no description support (pipx, mas), and three themes listed in the roadmap but never shipped.

Changes

cmd/gpk/main.go — help text

Updated Supported managers: to list all 36 managers accurately.

internal/manager/pipx.go — add Describe()

Reads Summary from each tool's .dist-info/METADATA inside its pipx venv — same offline approach as uv.go. Respects PIPX_HOME env var.

func pipxLocalSummary(venvsDir, name string) string {
    pattern := filepath.Join(venvsDir, name, "lib", "python*", "site-packages", normalized+"-*.dist-info", "METADATA")
    // falls back to non-normalized name if no match
    return parseMetadataSummary(matches[0])
}

internal/manager/mas.go — add Describe()

Scan() now stores the numeric App Store ID in Package.Location. Describe() fans out concurrent iTunes lookup API calls (https://itunes.apple.com/lookup?id=<id>, max 5 goroutines) and trims descriptions to the first line, capped at 200 chars.

internal/config/themes/ — missing built-in themes

Added solarized-dark.toml, one-dark.toml, and rose-pine.toml — all three were listed in ROADMAP.md but absent from the themes directory.

tests/parsing/pipx_mas_describe_test.go

New tests covering METADATA Summary extraction, iTunes JSON response parsing, empty-result handling, and the updated mas.Scan() ID-capture logic.

Copilot AI changed the title [WIP] Update issue tracking with top priorities Fix help text, add missing descriptions for pipx/mas, add 3 built-in themes Apr 3, 2026
Copilot AI requested a review from neur0map April 3, 2026 01:11
Copy link
Copy Markdown
Contributor

@programmersd21 programmersd21 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@neur0map pls check it. lgtm :D

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