Skip to content

Support pinned GitHub Skill metadata#4

Merged
sheepxux merged 1 commit into
mainfrom
codex/v1.0.1-pin-metadata
Jul 11, 2026
Merged

Support pinned GitHub Skill metadata#4
sheepxux merged 1 commit into
mainfrom
codex/v1.0.1-pin-metadata

Conversation

@sheepxux

@sheepxux sheepxux commented Jul 11, 2026

Copy link
Copy Markdown
Owner

Fix

Allow the optional metadata.github-pinned string that GitHub CLI adds when a Skill is installed with gh skill install --pin.

The validator still requires the four standard GitHub source-tracking fields and rejects unknown metadata keys. The regression test now models an actual pinned installation, and the real pinned v1.0.0 installation validates successfully.

Validation

  • targeted validation suite passes
  • loopctl skill validate passes against a real gh skill install --pin v1.0.0 copy
  • loopctl doctor passes

Summary by cubic

Allow Skills installed with gh skill install --pin to validate by accepting the optional metadata.github-pinned string. Validation still requires github-path, github-ref, github-repo, and github-tree-sha, rejects unknown keys, and tests now include a pinned install case.

Written for commit a596e55. Summary will update on new commits.

Review in cubic

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 2 files

Re-trigger cubic

@sheepxux
sheepxux merged commit bc8ff15 into main Jul 11, 2026
4 checks passed
@sheepxux
sheepxux deleted the codex/v1.0.1-pin-metadata branch July 11, 2026 16:39
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