Repository navigation
chore: adopt the Marketplace skill updater - #613
Conversation
…ll-actions-609-99cb
…ll-actions-609-99cb
@coderabbitai full review |
✅ Action performed
|
|
No actionable comments were generated in the recent review.
|
| Check name | Status | Explanation | Resolution |
|---|---|---|---|
| Linked Issues check | Issue #609 has three checklist items, plus a dependency on devantler-tech/.github#775.1. Pin to a verified release at its full commit: Met in repository evidence. The head workflow `.github/workflow… |
Provide evidence for the following: - The verification result for devantler-tech/.github#775. It must confirm the standalone release v1.0.1 at 0e1d6010a3d8b04a59b64892a5879cd2b45c7432 and its Marketplace listing.- The reusable workflow… |

Passed checks: 3 passed
-
Full details: Out of Scope Changes check
-
Explanation
The supplied summary lists two changed files. Both changes match the linked issue
#609and the PR description..github/workflows/update-agent-skills.yaml: theupdatejob now points todevantler-tech/update-agent-skills-actionat the pinned commit0e1d6010a3d8b04a59b64892a5879cd2b45c7432(+1/-1). Inputs and secrets are unchanged according to the summary. This implements the updater adoption and reference update in#609.AGENTS.md: thegh skill update --alllink and theupdate-agent-skillsworkflow link now point to the new repository (+2/-2). The head file confirms these links. This implements the "update references to the relocated updater" item in#609.
No unrelated changes appear in the summary. The whole-PR diff could not be read, so the file list rests on the supplied summary.
-
Full details: Linked Issues check
-
Explanation
Issue
#609has three checklist items, plus a dependency on devantler-tech/.github#775.-
Pin to a verified release at its full commit: Met in repository evidence. The head workflow
.github/workflows/update-agent-skills.yamlcallsdevantler-tech/update-agent-skills-action/.github/workflows/update-agent-skills.yamlat the 40-character commit0e1d6010a3d8b04a59b64892a5879cd2b45c7432, annotated# v1.0.1. Release verification is not established from this repository. -
Preserve skill pins, update scope, signing, credentials, branch identities, and concurrency: Partly established. The caller keeps the
scopejob,dir,pr-branch,pins,pr-per-skill: true,use-app-token: true, and theAPP_PRIVATE_KEYsecret. Theconcurrencygroupagent-plugins-skill-sync-${{ github.repository }}withqueue: maxis unchanged in the head file. The signing and credential behavior of the external reusable workflow cannot be read here. The raw summary states that inputs and secrets are unchanged, but the base blob could not be read to confirm this. -
Update references and verify the native workflow interface before protected merge: References are met.
AGENTS.mdat head linksgh skill update --alland theupdate-agent-skillsworkflow todevantler-tech/update-agent-skills-action. The interface check against the external workflow cannot be performed from this repository. The issue also makes adoption wait on the standalone release and Marketplace listing in Publish skill maintenance actions on GitHub Marketplace .github#775, which is not verified here.
The whole-PR diff read was unavailable. The changed-file set and the base content rely on the supplied raw summary.
Resolution
Provide evidence for the following:
- The verification result for Publish skill maintenance actions on GitHub Marketplace .github#775. It must confirm the standalone release
v1.0.1at0e1d6010a3d8b04a59b64892a5879cd2b45c7432and its Marketplace listing. - The reusable workflow interface at
devantler-tech/update-agent-skills-actionfor that commit. It must show the accepted inputs (dir,pr-branch,pins,pr-per-skill,use-app-token) and theAPP_PRIVATE_KEYsecret. It must also show the signing and credential behavior that the issue requires to be preserved. - A readable whole-PR diff against merge base
5673211776964d51c60973b2ca1e7e28c68ed656. It must confirm that the previousdevantler-tech/.githubreference ata2403f8a2640dcf9d342bf7085a906d8e8ec094awas the only functional change in the workflow.
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
Ready at b1c402a. Native CI40596852339 passed. CodeRabbit completed a clean current-head review (summary6455761702); complete thread, pending-review and review-body sweeps are clear, no human direction remains and there is no merge conflict. The body and closing-keyword guards passed. All readable required gates passed; the normal protected merge enforces the opaque code-quality gate. The only runtime configuration change is the updater's source reference; its inputs, schedule, permissions and follow-up behavior are preserved. I verified the public Marketplace v1.0.1 release attestation resolves to0e1d6010a3d8b04a59b64892a5879cd2b45c7432 and traced its matching caller interface. Native source CI40594567919 exercised the actual updater on Linux/macOS, pins, no-op updates and per-skill mode. The local actionlint1.7.12 concurrency diagnostic reproduces on unchanged main and introduces no new diagnostic. The first real caller run on main remains post-merge delivery verification. Dispatching this maintenance workflow on a feature branch would repurpose existing updater branches and PR bases, so the live main run is observed after the ordinary protected merge. |
Post-merge verification passed at main6ed44e0a5ba652d580523419f13a93563360ec27. The real maintenance run 40601029291, using the declared engineering-practices scope, succeeded. GitHub's run metadata confirms it invoked the standalone updater workflow at verified Marketplace commit0e1d6010a3d8b04a59b64892a5879cd2b45c7432. The actual update job succeeded; there was no drift, so proposal and follow-up jobs correctly stayed idle. No disposable updater branch or PR was created. |
Why
Plugin skill updates need to keep working as the internal updater moves to GitHub Marketplace.
What
Adopts the standalone updater while retaining the existing update behavior and review controls, and points the maintenance guidance at its new owner.
Fixes #609