Conversation
|
|
Caution Review failedThe pull request is closed. WalkthroughIntroduces a new UI layer for the CLI (dashboards, headers, task details, next task, suggested steps) and rewires list/show commands to use it. Updates build and packaging: consolidates exports to source, adds path aliases, adjusts tsup config with env injection and aliases, simplifies bin entry, and modifies watch/dev scripts. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
actor User
participant CLI as ListTasksCommand
participant Core as @tm/core
participant UI as UI Components
User->>CLI: tm list [--tag]
CLI->>Core: load tasks
CLI->>UI: displayHeader({ tag, filePath })
CLI->>UI: calculateTaskStatistics/tasks, subtasks
CLI->>UI: calculateDependencyStatistics/tasks
CLI->>UI: getPriorityBreakdown/tasks
CLI->>CLI: findNextTask(tasks)
CLI->>UI: displayDashboards(stats, substats, priorities, depStats, nextTask)
CLI->>UI: displayRecommendedNextTask(nextTask + description)
CLI->>UI: displaySuggestedNextSteps()
sequenceDiagram
autonumber
actor User
participant CLI as ShowTaskCommand
participant Core as @tm/core
participant UI as Task Detail Components
User->>CLI: tm show <id>
CLI->>Core: fetch task + subtasks
CLI->>UI: displayTaskHeader(id, title)
CLI->>UI: displayTaskProperties(task)
alt if task.details
CLI->>UI: displayImplementationDetails(details)
end
alt if task.testStrategy
CLI->>UI: displayTestStrategy(testStrategy)
end
CLI->>UI: displaySubtasks(filteredSubtasks, task.id)
CLI->>UI: displaySuggestedActions(task.id)
sequenceDiagram
autonumber
participant Bin as bin/task-master.js
participant Runner as runCLI
Bin->>Runner: execute()
note over Bin,Runner: Legacy flag parsing/handlers removed
Estimated code review effort🎯 4 (Complex) | ⏱️ ~75 minutes Possibly related PRs
Suggested reviewers
✨ Finishing touches
🧪 Generate unit tests
📜 Recent review detailsConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (19)
Tip 👮 Agentic pre-merge checks are now available in preview!Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.
Please see the documentation for more information. Example: reviews:
pre_merge_checks:
custom_checks:
- name: "Undocumented Breaking Changes"
mode: "warning"
instructions: |
Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).Please share your feedback with us on this Discord post. Comment |
05a1e13 to
099e72a
Compare
This PR was automatically generated to update documentation based on recent changes. Original commit: fix: UI list and show (#1210)\n\n\n Co-authored-by: Claude <claude-assistant@anthropic.com>
What type of PR is this?
Description
Related Issues
How to Test This
# Example commands or stepsExpected result:
Contributor Checklist
npm run changesetnpm testnpm run format-check(ornpm run formatto fix)Changelog Entry
For Maintainers
Summary by CodeRabbit
New Features
Enhancements
Chores