Skip to content

Extract skill catalog parsing and rename legacy manifest - #26

Closed
git-hulk wants to merge 1 commit into
skills-snapshot-cachefrom
skills-catalog-refactor
Closed

Extract skill catalog parsing and rename legacy manifest#26
git-hulk wants to merge 1 commit into
skills-snapshot-cachefrom
skills-catalog-refactor

Conversation

@git-hulk

@git-hulk git-hulk commented Aug 31, 2026

Copy link
Copy Markdown
Owner

Prepare for the versioned skill manifest by carving two mechanical
refactors out of the upcoming switch; behavior is unchanged.

  • Move catalog parsing (skills.yaml, marketplace.json, plugin.json,
    SKILL.md frontmatter) from github.go into catalog.go as ReadCatalog
    with a Catalog type, so the versioned flow can read catalogs from
    immutable snapshots through the same code path.

  • Rename the current manifest types to LegacyManifest /
    LoadLegacyManifest in legacy_manifest.go, freeing the Manifest name
    for the repository-keyed desired-state manifest that lands next.

🤖 Generated with Claude Code

Prepare for the versioned skill manifest by carving two mechanical
refactors out of the upcoming switch; behavior is unchanged.

  - Move catalog parsing (skills.yaml, marketplace.json, plugin.json,
    SKILL.md frontmatter) from github.go into catalog.go as ReadCatalog
    with a Catalog type, so the versioned flow can read catalogs from
    immutable snapshots through the same code path.

  - Rename the current manifest types to LegacyManifest /
    LoadLegacyManifest in legacy_manifest.go, freeing the Manifest name
    for the repository-keyed desired-state manifest that lands next.

Assistant By Fable 5
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@git-hulk
git-hulk force-pushed the skills-catalog-refactor branch from ab64d82 to 4574e3e Compare September 1, 2026 02:18
@git-hulk git-hulk closed this Sep 1, 2026
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.

1 participant