Revert: feat: add openspec dashboard command (#615)

[TabishB]

Summary

• Reverts [With New OpenSpec Version] feat: add openspec dashboard command for web-based browsing #615 which was accidentally merged

Test plan

• CI passes with the revert applied

🤖 Generated with Claude Code

Summary by CodeRabbit

• Revert

  • Removed the dashboard command from the CLI, eliminating the web interface for viewing project changes, specifications, and archive data.

• Tests

  • Removed dashboard test suites.

• Documentation

  • Removed dashboard project planning documentation.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

This PR removes the entire dashboard feature from the codebase, including the CLI command, core dashboard modules (data aggregation, markdown rendering, HTTP server), all corresponding tests, and the project task documentation.

Changes

Cohort / File(s)	Summary
Dashboard Core Modules src/core/dashboard/index.ts, src/core/dashboard/data.ts, src/core/dashboard/markdown.ts, src/core/dashboard/server.ts	Deleted all dashboard implementation including the DashboardCommand class, data aggregation logic, markdown-to-HTML renderer, and HTTP server with API endpoints and HTML client.
Dashboard Tests test/core/dashboard/data.test.ts, test/core/dashboard/markdown.test.ts, test/core/dashboard/server.test.ts	Removed all test suites for dashboard data retrieval, markdown rendering, and server integration functionality.
CLI and Documentation src/cli/index.ts, openspec/changes/add-dashboard-command/tasks.md	Removed the dashboard subcommand from CLI entry point and deleted the comprehensive project plan documentation with all task milestones and acceptance criteria.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

• [With New OpenSpec Version] feat: add openspec dashboard command for web-based browsing #615: Directly related—the inverse operation that reverses the dashboard feature addition including CLI command, data/markdown/server modules, and all tests.

Poem

🐰 The dashboard gleams no more, removed with care,
Nine files swept clean from the developer's lair,
Data, server, tests—all gone without a trace,
A feature retired from this codebase space! ✨

🚥 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 accurately describes the main change: reverting PR #615 that added the dashboard command feature.
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 docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch revert-pr-615

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[vibe-kanban-cloud]

Review Complete

Your review story is ready!

View Story

Comment !reviewfast on this PR to re-generate the story.

[greptile-apps]

Greptile Overview

Greptile Summary

This PR cleanly reverts #615, which introduced an openspec dashboard command. The revert removes all associated code, tests, and documentation files.

What was removed:

• Dashboard command registration from src/cli/index.ts (17 lines)
• Entire src/core/dashboard/ directory (4 implementation files, ~1,117 lines)
• Entire test/core/dashboard/ directory (3 test files, ~519 lines)
• Implementation tasks file from openspec/changes/

What remains intact:

• The existing openspec view command continues to provide terminal-based dashboard functionality
• Documentation references to "dashboard" in README files refer to the view command's functionality
• No orphaned imports or broken references detected

The revert is complete and does not leave any residual code or broken imports.

Confidence Score: 5/5

• This PR is safe to merge with no risk
• This is a straightforward revert that cleanly removes an accidentally merged feature. All code deletions are contained to the dashboard feature implementation, tests, and associated task files. No broken imports, no orphaned references, and no impact on existing functionality.
• No files require special attention

Important Files Changed

Filename	Overview
src/cli/index.ts	Removed dashboard command registration (17 lines deleted)
src/core/dashboard/data.ts	Deleted data gathering module for dashboard (250 lines)
src/core/dashboard/index.ts	Deleted dashboard command entry point (63 lines)
src/core/dashboard/server.ts	Deleted HTTP server implementation (591 lines)

Sequence Diagram

sequenceDiagram
    participant User
    participant Git
    participant Codebase
    
    User->>Git: Request to revert PR #615
    Git->>Codebase: Remove dashboard command registration
    Git->>Codebase: Delete src/core/dashboard/ directory
    Git->>Codebase: Delete test/core/dashboard/ directory
    Git->>Codebase: Delete tasks.md file
    Git->>User: Revert complete (9 files changed)
    
    Note over Codebase: Existing 'view' command remains<br/>Documentation references remain<br/>No imports to clean up

Loading