fix: correct CLI command names for expand operations in llms-install.md#1659
fix: correct CLI command names for expand operations in llms-install.md#1659withsivram wants to merge 1 commit intoeyaltoledano:mainfrom
Conversation
…md (fixes eyaltoledano#1640) The llms-install.md "Next Steps" section incorrectly listed `expand-task` and `expand-all` as CLI command names. These are internal identifiers — the actual CLI commands are `task-master expand --id=<id>` and `task-master expand --all`. The `expand_task` MCP tool maps to the `expand` CLI command (not `expand-task`), and `expand_all` maps to `expand --all`. The `assets/AGENTS.md` (which generates `.taskmaster/CLAUDE.md`) already correctly documented this mapping; only llms-install.md had the wrong command names in a user-facing context. No other files contained incorrect CLI command references for these operations — other occurrences of `expand-task` in the codebase are internal command identifiers used for telemetry, schema lookup, and prompt management (via `commandName: 'expand-task'`), not CLI commands. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughDocumentation update correcting CLI command references in installation instructions. The changes replace outdated command names with the proper Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Pull request overview
Updates the LLM installation guide to reflect the correct Task Master CLI syntax for task expansion, aligning the doc with the actual expand command used by the CLI and MCP tooling.
Changes:
- Replaced incorrect
expand-all/expand-taskCLI references withtask-master expand --allandtask-master expand --id=<id>in the “Next Steps” section.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| - Create new tasks with `add-task` or parse a PRD (scripts/prd.txt) into tasks with `parse-prd` | ||
| - Set up model preferences with `models` tool | ||
| - Expand tasks into subtasks with `expand-all` and `expand-task` | ||
| - Expand tasks into subtasks with `task-master expand --all` or `task-master expand --id=<id>` |
There was a problem hiding this comment.
In the "Next Steps" list, this bullet now uses the fully qualified CLI form (task-master expand ...) while the surrounding bullets use bare subcommand names (add-task, parse-prd, models). For consistency and to avoid confusing users about whether these are CLI subcommands or MCP tool names, please standardize the formatting (either prefix all of them with task-master, or remove the prefix here and keep only the subcommand/flags).
| - Expand tasks into subtasks with `task-master expand --all` or `task-master expand --id=<id>` | |
| - Expand tasks into subtasks with `expand --all` or `expand --id=<id>` |
| - Expand tasks into subtasks with `task-master expand --all` or `task-master expand --id=<id>` | ||
| - Explore advanced features like research mode and complexity analysis | ||
|
|
||
| For detailed documentation, refer to the Task Master docs directory.`` |
There was a problem hiding this comment.
There appear to be two stray backticks at the end of the sentence `For detailed documentation, refer to the Task Master docs directory.```. This breaks Markdown formatting; please remove the extra backticks so the paragraph renders correctly.
| For detailed documentation, refer to the Task Master docs directory.`` | |
| For detailed documentation, refer to the Task Master docs directory. |
Summary
expand-taskandexpand-allreferences inllms-install.mdto the actual CLI syntax:task-master expand --id=<id>andtask-master expand --allexpand_taskmaps to the CLIexpandcommand (notexpand-task) —assets/AGENTS.mdalready documented this correctly, onlyllms-install.mdhad the wrong namesTest plan
llms-install.md"Next Steps" section shows correct CLI commandsexpand-taskas a CLI commandFixes #1640
🤖 Generated with Claude Code
Summary by CodeRabbit
task-master expandformat with flag-based options.