Repository navigation
chore: remove pull-based skills sync, add Sync PR release chores (PD-126336) - #17
Merged
Merged
Conversation
Skills are now pushed to the Sync PR by the producer repo, so the nightly workflow that pulled them with a personal token is no longer needed. Drops the script, its hash helper and the .sync-notes.md ignore entry. skills.lock.json is kept because the push-based sync still uses it. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
The push-based sync only writes skills/. This workflow runs on the automated/skills-sync PR, bumps every plugin manifest to base+1, updates CHANGELOG.md and runs plugin validate --strict before pushing the bump. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Only the removed pull-based sync read and wrote this file. The push-based sync commits touch skills/ only, and nothing else references it. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Skills are now pushed to the rolling
automated/skills-syncPR by the producer repo, so the old pull-based sync can go. This PR removes it and adds a small workflow for the release chores the old sync used to do.Ticket: PD-126336
Removed
scripts/sync-skills.mjs: clonedpandadoc-studio/agentic-hub-skillswith a personal token and vendoredskills/.scripts/lib/hash.mjs: only used by the script above..sync-notes.mdentry in.gitignore: leftover from the old sync.skills.lock.json: only the old sync read and wrote it. The push-based sync commits touchskills/only, and no docs or scripts reference it..github/workflows/sync-skills.ymlwas already deleted onmain(#9d98e52).Added
sync-release-chores.yml+scripts/bump-plugin-version.mjs. The push-based pipeline only writesskills/, so this covers what the old sync did on top of that. On PRs fromautomated/skills-syncintomainit:CHANGELOG.mdentry (Added / Updated / Removed skills).claude plugin validate . --strict(@latest, unpinned on purpose).The bump is derived from the base branch, so re-runs on a re-synced PR are idempotent.
Testing
origin/automated/skills-syncbranch in a throwaway worktree: 1.0.3 → 1.0.4 across all manifests, changelog entry correct, identical output on a second run.claude plugin validate . --strictpasses on this branch and on the sync branch after the bump.sync-skills,lib/hash,skills.lock.jsonorSKILLS_REPO_TOKENremain.Before merging
RELEASE_BOT_APP_IDandRELEASE_BOT_PRIVATE_KEYrepo secrets exist, and the App is installed on this repo with contents write access (and allowed to push toautomated/skills-syncif a ruleset covers it).Heads-up
plugin.json,.cursor-plugin/plugin.json,.github/plugin/plugin.json,gemini-extension.jsonandserver.jsonare at 1.0.0 while.claude-plugin/plugin.jsonis at 1.0.3. The first run moves all of them to base + 1, so expect a larger manifest diff on the next sync PR (chore(skills): sync skills v0.2.5 #16), includingserver.json.Follow-ups from the ticket (not in this PR)
SKILLS_REPO_TOKENActions secret and revoke the underlying personal token.🤖 Generated with Claude Code