Skip to content

feat(skills): add update command to check and refresh skills#47

Merged
BlackHole1 merged 1 commit into
mainfrom
add-skills-update
Mar 30, 2026
Merged

feat(skills): add update command to check and refresh skills#47
BlackHole1 merged 1 commit into
mainfrom
add-skills-update

Conversation

@BlackHole1
Copy link
Copy Markdown
Member

Introduce oo skills update [skills...] that checks installed oo-managed published skills against the registry and updates them when a newer version is available.

Key changes:

  • Extract registry skill publication logic into a shared module so both install and update can reuse it
  • Add variadic argument support to the CLI adapter
  • Include interactive progress reporter with spinner animation
  • Add comprehensive tests for the update workflow

Introduce `oo skills update [skills...]` that checks installed
oo-managed published skills against the registry and updates them
when a newer version is available.

Key changes:
- Extract registry skill publication logic into a shared module
  so both install and update can reuse it
- Add variadic argument support to the CLI adapter
- Include interactive progress reporter with spinner animation
- Add comprehensive tests for the update workflow

Signed-off-by: Kevin Cui <bh@bugs.cc>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 30, 2026

Caution

Review failed

Pull request was closed or merged during review

Summary by CodeRabbit

  • New Features
    • Added oo skills update [skills...] command to update installed managed skills
    • Run without arguments to update all skills, or specify particular skills to update
    • Includes real-time progress reporting in interactive terminals
    • Bundled skill automatically excluded from updates

Walkthrough

This PR implements a new skills update CLI command for managing published Codex skills. It adds documentation describing the command behavior (when no skills are specified, all installed oo-managed published skills are updated; when specific skills are named, only those are updated), excludes the bundled oo skill from updates, and explains the update workflow for registry skills. The implementation includes a new SkillsUpdateProgressReporter class for rendering terminal progress display, refactors the registry skill installation flow to use new reusable prepareRegistrySkillPublication and publishPreparedRegistrySkillPublication helpers, introduces support for variadic CLI arguments, and adds comprehensive test coverage validating end-to-end behavior including concurrent multi-skill updates and error handling.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~60 minutes

Possibly related PRs

Suggested labels

codex

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title follows the required format (): , uses English, and directly describes the main change—adding an update command to check and refresh skills.
Description check ✅ Passed The description is clearly related to the changeset, outlining the introduction of the skills update command and key refactoring/enhancement changes that align with the file modifications.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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