You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Verify skill text includes safety constraints, verify step, and idempotent rerun guidance
Related Issues
None
Summary by CodeRabbit
Documentation
Added specification for lark-cli-update skill enabling automated lark-cli binary updates with installation method detection and version comparison.
Includes safety constraints, comprehensive error handling, rollback guidance, and idempotent execution for failed or interrupted upgrades with skill refresh capabilities.
A new skill definition document lark-cli-update has been added, specifying a stateless v1 skill that upgrades the lark-cli main program and refreshes skills. The workflow covers installation detection, version comparison, safe upgrade execution, and verification with structured JSON output.
Changes
Cohort / File(s)
Summary
Skill Definition skills/lark-cli-update/SKILL.md
New skill specification defining a 4-step workflow: detect installation method (npm/source/not installed), compare versions, execute upgrades with safety constraints and explicit user confirmation for source mode, and verify with JSON output. Includes error handling, idempotency guarantees, and detailed branching logic.
Estimated code review effort
🎯 2 (Simple) | ⏱️ ~12 minutes
Suggested reviewers
liangshuo-1
liuxinyanglxy
Poem
🐰 Hop, hop! A skill springs to life,
To update lark-cli without strife,
Four careful steps, from first to last,
Safely forward, no backward blast!
Our CLI now grows with grace and care. ✨
The title 'feat(skill): add lark-cli-update skill' clearly and concisely summarizes the main change—adding a new skill for CLI self-maintenance.
Docstring Coverage
✅ Passed
Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Description check
✅ Passed
The PR description covers Summary, Changes, and Test Plan sections as required by the template, though some section content is abbreviated or uses escape sequences.
✏️ Tip: You can configure your own custom pre-merge checks in the settings.
✨ Finishing Touches🧪 Generate unit tests (beta)
Create PR with unit tests
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.
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
size/MSingle-domain feat or fix with limited business impact
2 participants
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
Add new
lark-cli-updateskill so AI agents can check for updates and upgrade lark-cli through one dedicated skill.Changes
skills/lark-cli-update/SKILL.mdTest Plan
skills/lark-cli-update/SKILL.mdand verify intent mapping covers upgrade / check / skills-only / reinstall flowsRelated Issues
Summary by CodeRabbit
lark-cli-updateskill enabling automated lark-cli binary updates with installation method detection and version comparison.