Skip to content

feat: add npm installer cli for skill distribution#1

Merged
AnxForever merged 1 commit into
mainfrom
cc/skill-npm-installer
Feb 24, 2026
Merged

feat: add npm installer cli for skill distribution#1
AnxForever merged 1 commit into
mainfrom
cc/skill-npm-installer

Conversation

@AnxForever
Copy link
Copy Markdown
Owner

@AnxForever AnxForever commented Feb 24, 2026

Summary by CodeRabbit

  • New Features

    • Added a command-line tool to manage skill installations with support for automatic tool detection, installation options, uninstallation, validation checks, and dry-run modes.
  • Documentation

    • Updated README with localized content including installation workflows, validation procedures, and npm publishing instructions.
    • Added MIT license file.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Feb 24, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 215dc0d and 93e84dd.

📒 Files selected for processing (4)
  • LICENSE
  • README.md
  • bin/stylekit-style-prompts-skill.js
  • package.json

📝 Walkthrough

Walkthrough

This pull request introduces a new npm package for StyleKit style-prompts skills, including an MIT license, a Node.js CLI installer tool for managing skill payloads across Codex and Claude integrations, package metadata, and localized Chinese documentation with installation and publishing workflows.

Changes

Cohort / File(s) Summary
Package Setup
LICENSE, package.json
Added MIT license text and package manifest with CLI bin entry, metadata, and Node ≥18 engine requirement.
Documentation
README.md
Rebranded repository name, localized documentation to Chinese, added multi-step installation flows (install, doctor, regression, publish), and expanded validation/publishing command examples.
CLI Tool
bin/stylekit-style-prompts-skill.js
Implemented standalone Node.js CLI script supporting install, uninstall, doctor, and help commands; includes payload validation, auto-detection of Codex/Claude targets, dry-run support, and detailed status reporting.

Sequence Diagram(s)

sequenceDiagram
    participant User
    participant CLI as CLI Script
    participant FS as File System
    participant Tool as Tool Directory<br/>(Codex/Claude)
    
    User->>CLI: Run install command<br/>(--tool, --target, --force)
    CLI->>CLI: Parse arguments
    CLI->>FS: Validate payload files<br/>in root directory
    alt Validation fails
        CLI->>User: Error: Missing files
    else Validation succeeds
        CLI->>CLI: Auto-detect tool target<br/>or use --target
        alt Dry-run mode
            CLI->>User: Report planned changes
        else Normal mode
            CLI->>FS: Check if target exists
            alt Target exists & !--force
                CLI->>User: Error: Already installed
            else OK to proceed
                CLI->>FS: Recursively copy<br/>payload to target
                FS->>Tool: Files installed
                CLI->>User: Success message
            end
        end
    end
Loading

Estimated Code Review Effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 A skill installer hopped into view,
With CLI commands shiny and new,
For Codex and Claude, a home to find,
Chinese words, and logic well-designed—
Pack it up, publish wide and true! 📦✨

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch cc/skill-npm-installer

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.

@AnxForever AnxForever merged commit 49099aa into main Feb 24, 2026
1 of 2 checks passed
@AnxForever AnxForever deleted the cc/skill-npm-installer branch February 25, 2026 09:21
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