Skip to content

fix: UI list and show#1210

Merged
Crunchyman-ralph merged 6 commits intonextfrom
ralph/fix.ui.list.show
Sep 17, 2025
Merged

fix: UI list and show#1210
Crunchyman-ralph merged 6 commits intonextfrom
ralph/fix.ui.list.show

Conversation

@Crunchyman-ralph
Copy link
Collaborator

@Crunchyman-ralph Crunchyman-ralph commented Sep 17, 2025

  • What type of PR is this?

  • 🐛 Bug fix
  • ✨ Feature
  • 🔌 Integration
  • 📝 Docs
  • 🧹 Refactor
  • Other:

Description

  • implement ui tweaks for list and show

Related Issues

How to Test This

# Example commands or steps

Expected result:

Contributor Checklist

  • Created changeset: npm run changeset
  • Tests pass: npm test
  • Format check passes: npm run format-check (or npm run format to fix)
  • Addressed CodeRabbit comments (if any)
  • Linked related issues (if any)
  • Manually tested the changes

Changelog Entry


For Maintainers

  • PR title follows conventional commits
  • Target branch correct
  • Labels added
  • Milestone assigned (if applicable)
  • add typescript
  • add npm workspaces

Summary by CodeRabbit

  • New Features

    • List view now shows project and dependency dashboards with progress, status, and priority breakdowns.
    • Adds a “Recommended Next Task” panel with actions and optional description.
    • Displays “Suggested Next Steps” guidance after listings.
    • Revamped task detail view with styled header, properties table, markdown-rendered details/test strategy, and subtasks table.
  • Enhancements

    • Consistent header showing current tag and source path.
    • Task table includes a Complexity column, simplified dependency display, and refreshed status icons.
  • Chores

    • Build and development tooling updates; new markdown rendering dependencies.

@changeset-bot
Copy link

changeset-bot bot commented Sep 17, 2025

⚠️ No Changeset found

Latest commit: 099e72a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 17, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

Introduces 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

Cohort / File(s) Summary
CLI commands integration
apps/cli/src/commands/list.command.ts, apps/cli/src/commands/show.command.ts
Refactors output to UI helpers; adds statistics, dashboards, next-task selection/display, suggested steps; migrates single-task rendering to new task-detail components.
UI components (new)
apps/cli/src/ui/components/*.ts, apps/cli/src/ui/components/index.ts, apps/cli/src/ui/index.ts
Adds header, dashboard, next-task, suggested-steps, and task-detail components; provides a barrel and UI entrypoint; exposes types and rendering utilities.
CLI packaging
apps/cli/package.json
Changes exports to source path, adds typesVersions mapping, removes build/dev scripts.
Core package exports/build
packages/tm-core/package.json, packages/tm-core/tsup.config.ts
Switches exports to source files; removes dist import targets; deletes tsup config; drops build/dev scripts.
Root build/tooling
package.json, tsconfig.json, tsup.config.ts, turbo.json
Adds marked/marked-terminal deps; rewires dev watch; defines TS path aliases; overhauls tsup (env injection, aliasing, externals, entry change); removes turbo dev dependsOn.
Entrypoints/runtime
bin/task-master.js, src/utils/getVersion.js
Replaces bin wrapper with direct runCLI call; simplifies version retrieval to JSON import.

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()
Loading
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)
Loading
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
Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~75 minutes

Possibly related PRs

Suggested reviewers

  • eyaltoledano
✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch ralph/fix.ui.list.show

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 799d1d2 and 099e72a.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (19)
  • apps/cli/package.json (2 hunks)
  • apps/cli/src/commands/list.command.ts (3 hunks)
  • apps/cli/src/commands/show.command.ts (3 hunks)
  • apps/cli/src/ui/components/dashboard.component.ts (1 hunks)
  • apps/cli/src/ui/components/header.component.ts (1 hunks)
  • apps/cli/src/ui/components/index.ts (1 hunks)
  • apps/cli/src/ui/components/next-task.component.ts (1 hunks)
  • apps/cli/src/ui/components/suggested-steps.component.ts (1 hunks)
  • apps/cli/src/ui/components/task-detail.component.ts (1 hunks)
  • apps/cli/src/ui/index.ts (1 hunks)
  • apps/cli/src/utils/ui.ts (4 hunks)
  • bin/task-master.js (1 hunks)
  • package.json (4 hunks)
  • packages/tm-core/package.json (1 hunks)
  • packages/tm-core/tsup.config.ts (0 hunks)
  • src/utils/getVersion.js (1 hunks)
  • tsconfig.json (1 hunks)
  • tsup.config.ts (1 hunks)
  • turbo.json (0 hunks)

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.

  • Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
  • Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

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 @coderabbitai help to get the list of available commands and usage tips.

Copy link
Collaborator Author

@Crunchyman-ralph Crunchyman-ralph left a comment

Choose a reason for hiding this comment

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

lgtm

@Crunchyman-ralph Crunchyman-ralph merged commit 0e8c42c into next Sep 17, 2025
3 of 4 checks passed
github-actions bot added a commit that referenced this pull request Sep 17, 2025
  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>
@coderabbitai coderabbitai bot mentioned this pull request Sep 17, 2025
16 tasks
Crunchyman-ralph added a commit that referenced this pull request Sep 17, 2025
@coderabbitai coderabbitai bot mentioned this pull request Nov 26, 2025
16 tasks
sfc-gh-dflippo pushed a commit to sfc-gh-dflippo/task-master-ai that referenced this pull request Dec 4, 2025
sfc-gh-dflippo pushed a commit to sfc-gh-dflippo/task-master-ai that referenced this pull request Dec 4, 2025
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