Skip to content

docs: use npx skills to install the skill#585

Merged
njzjz merged 3 commits intodeepmodeling:masterfrom
njzjz:docs-install-skill
Mar 29, 2026
Merged

docs: use npx skills to install the skill#585
njzjz merged 3 commits intodeepmodeling:masterfrom
njzjz:docs-install-skill

Conversation

@njzjz
Copy link
Copy Markdown
Member

@njzjz njzjz commented Mar 29, 2026

Summary by CodeRabbit

  • Documentation
    • Updated agent installation instructions to use a single Skill CLI command for adding the dispatcher skill: npx -y skills add deepmodeling/dpdispatcher -a <your-agent-name> -y.
    • Retains the final session refresh/restart step to reload skills after installation.

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@ustc.edu.cn>
Copilot AI review requested due to automatic review settings March 29, 2026 12:08
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Mar 29, 2026
@dosubot dosubot bot added the documentation Improvements or additions to documentation label Mar 29, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 29, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 48.33%. Comparing base (c605e72) to head (6bf28c5).
⚠️ Report is 7 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #585   +/-   ##
=======================================
  Coverage   48.33%   48.33%           
=======================================
  Files          40       40           
  Lines        3958     3958           
=======================================
  Hits         1913     1913           
  Misses       2045     2045           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates agent-facing documentation to install DPDispatcher skills via the skills CLI (npx skills) instead of manual cloning/copying, making installation more reproducible for agent workspaces.

Changes:

  • Replace manual skill installation steps with an npx skills add ... command.
  • Add brief explanation of the agent argument and what the CLI does.
  • Keep the existing “refresh/restart session” step.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 29, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 56b6206a-e8d8-445e-af1b-a16d6e033106

📥 Commits

Reviewing files that changed from the base of the PR and between 42dbb3e and 6bf28c5.

📒 Files selected for processing (1)
  • doc/agent-skills.md
✅ Files skipped from review due to trivial changes (1)
  • doc/agent-skills.md

📝 Walkthrough

Walkthrough

Replaced manual three-step agent skill installation instructions with a single Skill CLI command: npx -y skills add deepmodeling/dpdispatcher -a <your-agent-name> -y. The command now performs cloning and skill directory copy; users still must refresh/restart the agent session to reload skills.

Changes

Cohort / File(s) Summary
Documentation
doc/agent-skills.md
Rewrote installation steps for agent skills to use the Skill CLI command (npx -y skills add deepmodeling/dpdispatcher -a <your-agent-name> -y) instead of instructing users to manually clone deepmodeling/dpdispatcher and copy skills/ directories; retained final refresh/restart step.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and specifically describes the main change: updating documentation to use the npx skills command for skill installation instead of manual steps.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ 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.

❤️ Share

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

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Jinzhe Zeng <njzjz@qq.com>
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. and removed size:S This PR changes 10-29 lines, ignoring generated files. labels Mar 29, 2026
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:XS This PR changes 0-9 lines, ignoring generated files. labels Mar 29, 2026
@njzjz njzjz enabled auto-merge (squash) March 29, 2026 17:07
@njzjz njzjz merged commit c4788d3 into deepmodeling:master Mar 29, 2026
29 checks passed
@njzjz njzjz deleted the docs-install-skill branch March 29, 2026 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants