docs(skills): document the skills installer and dhq skills command#29
Conversation
PRs #25 (dhq launch) and #26 (skills installer) merged, but only launch made it into the docs. Bring the skills installer up to date: - Quick Start: note that dhq hello now offers to install the DeployHQ skill into detected agents, and add a standalone 'dhq skills install'. - Command list: add 'dhq skills list | install'. - Agent Integration: lead with 'dhq skills install' (auto-detects 12 agents, native formats, user vs project scope), and frame 'dhq setup' as the narrower per-agent alternative. - CLAUDE.md: add internal/skillinstaller/ to the architecture map. - hello.go: extend the command's long description to mention the skill install step so --help matches the new behaviour. Docs-only (plus the one help string); no behaviour change. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✅ Files skipped from review due to trivial changes (1)
🚧 Files skipped from review as they are similar to previous changes (1)
WalkthroughDocumentation and CLI help text were updated to describe Changesdhq skills documentation and hello command update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes 🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@internal/commands/hello.go`:
- Line 24: The `hello` command description in `hello` overstates what the
workflow does by saying it installs the skill into “any” detected AI coding
agents. Update the `Long` description to reflect the actual behavior: it guides
login/account/project setup and offers skill installs for detected agents, while
project-scope targets are skipped unless `--agent` is provided.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: d3e2440c-0764-4e1c-9013-c4db26589690
📒 Files selected for processing (3)
CLAUDE.mdREADME.mdinternal/commands/hello.go
Address Review Council findings on PR #29: - Quick Start + hello.go no longer imply bare 'dhq skills install' / 'dhq hello' cover project-scope agents. Copilot is ScopeProject and is skipped by the bare command (opt-in via --agent), which the prose now reflects instead of contradicting the Agent Integration section. - Align the supported-agent list with the CLI display names: Continue -> Continue.dev, Kiro -> Kiro CLI (as shown by 'dhq skills list'). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Summary
PRs #25 (
dhq launch) and #26 (skills installer) merged, but onlydhq launchmade it into the docs. This brings the skills installer up to date.dhq hellonow notes it offers to install the DeployHQ skill into detected agents, plus a standalonedhq skills installline.dhq skills list | install.dhq skills install(auto-detects 12 agents, installs into each native format, user-scope vs project-scope--agent), and reframesdhq setupas the narrower per-agent alternative.internal/skillinstaller/to the architecture map.--helpmatches the new onboarding behaviour.Docs-only, plus the one
--helpstring — no behaviour change.Note on
dhq setupvsdhq skillsThese overlap:
dhq setup <agent>installs the skill for 4 agents;dhq skills installauto-detects and covers 12. This PR documentsdhq skillsas the primary path and keepsdhq setupas a documented alternative. Whether to formally deprecatedhq setupin code is left as a follow-up decision.🤖 Generated with Claude Code
Summary by CodeRabbit
Documentation
dhq signup,dhq auth login, anddhq configure.dhq skillscommand to list and install the DeployHQ skill for detected AI agents.--agentrequirement for project-scoped installs, plus updated agent helper examples.Refactor
dhq hellohelp text to include the DeployHQ skill installation step as part of guided setup.