Repository navigation
chore: adopt the Marketplace skill updater - #5471
Conversation
…ll-actions-5397-99cb
…ll-actions-5397-99cb # Conflicts: # .github/workflows/update-skills.yaml
…ll-actions-5397-99cb
@coderabbitai full review |
|
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 41 minutes. View limit details
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 |
@codex review |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Codex Review: Didn't find any major issues. Keep it up! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Ready at88ce8e567f7e7e833c8ecda60a4480f219499ef2. CI40596878320 passed at this exact head. Codex completed it without findings (summary6456067642, receipt6456077628) after the primary provider's authenticated PR-scoped limit refusal. Complete thread, pending-review, review-body and human-direction sweeps are clear, and native merge state is CLEAN. Body/closing guards passed. All readable required gates passed; GitHub's ordinary protected queue enforces the opaque code-quality gate. Fresh live Platform health is OK. The change only redirects the existing updater call to verified Marketplace v1.0.1 at0e1d6010a3d8b04a59b64892a5879cd2b45c7432; inputs, pins, permissions and schedule are unchanged. Native source CI40594567919 exercised update behavior on Linux/macOS. The actual migrated plugin-consumer run40601029291 also succeeded and its native metadata proves the standalone workflow/commit binding. The local actionlint1.7.12 queue diagnostic reproduces unchanged on main. The real Platform maintenance run from main remains post-merge delivery verification. Testing the maintenance workflow on this feature branch would repurpose its existing updater branch/PR base, so ordinary protected merge precedes that live run. The queue will validate the candidate against its actual current base; there is no direct-merge bypass or production-health exception. |
Verified the protected queue merge at a4a3eb7. Candidate CI, main validation, CodeQL, and Code Quality passed. The signed queue candidate exactly preserves the reviewed caller change on its actual merge parent. The real main updater passed using the published immutable updater at 0e1d6010a3d8b04a59b64892a5879cd2b45c7432 (v1.0.1). It found no changes, so no update PR was needed. A transient public API quota refusal in the first main-validation attempt was classified and the failed-job-only retry passed; no checks or permissions changed. |
Why
The platform's skill updates need a maintained replacement before the internal updater is retired.
What
Adopts the independently released Marketplace updater while retaining the existing skill update behavior and safeguards.
Fixes #5397