Skip to content

Remove dashboard from CLI bundle#175

Merged
khaliqgant merged 1 commit intomainfrom
feature/cli-no-dashboard-default
Jan 14, 2026
Merged

Remove dashboard from CLI bundle#175
khaliqgant merged 1 commit intomainfrom
feature/cli-no-dashboard-default

Conversation

@khaliqgant
Copy link
Copy Markdown
Member

Summary

  • Remove dashboard-server from CLI build - dashboard now only available in cloud workspaces
  • Remove --dashboard-port option from wrapper command
  • Remove --no-dashboard and --port options from up command
  • Remove dashboard startup code from up command
  • Keep API-based commands (spawn, release, metrics, health) for cloud usage with default port 3888

Changes

  • src/cli/index.ts: Removed 76 lines of dashboard code
  • package.json: Removed build:dashboard script and dashboard copying
  • tsconfig.json: Added src/dashboard-server/**/* to exclude

Test plan

  • Build succeeds without dashboard
  • CLI help shows updated commands (no dashboard options)
  • Tests pass (1571/1574)
  • Code review verified no broken references

🤖 Generated with Claude Code

- Remove dashboard-server from CLI build (tsconfig exclude)
- Remove dashboard build scripts from package.json
- Remove --dashboard-port option from wrapper command
- Remove --no-dashboard and --port options from up command
- Remove dashboard startup code from up command
- Keep API-based commands (spawn, release, metrics, health) for cloud usage
- Dashboard functionality now only available in cloud workspaces

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@my-senior-dev-pr-review
Copy link
Copy Markdown

🤖 My Senior Dev — Analysis Complete

👤 For @khaliqgant

📁 Expert in src/dashboard/react-components/ (7 edits) • ⚡ 33rd PR this month

View your contributor analytics →


📊 3 files reviewed • 4 need attention

⚠️ Needs Attention:

  • src/cli/index.ts — Changes to the CLI impact user commands and documentation must be aligned with these removals.
  • package.json — Modifications to the build scripts and task removal could affect the overall build process.

🚀 Open Interactive Review →

The full interface unlocks features not available in GitHub:

  • 💬 AI Chat — Ask questions on any file, get context-aware answers
  • 🔍 Smart Hovers — See symbol definitions and usage without leaving the diff
  • 📚 Code Archeology — Understand how files evolved over time (/archeology)
  • 🎯 Learning Insights — See how this PR compares to similar changes

💬 Chat here: @my-senior-dev explain this change — or try @chaos-monkey @security-auditor @optimizer @skeptic @junior-dev

📖 View all 12 personas & slash commands

You can interact with me by mentioning @my-senior-dev in any comment:

In PR comments or on any line of code:

  • Ask questions about the code or PR
  • Request explanations of specific changes
  • Get suggestions for improvements

Slash commands:

  • /help — Show all available commands
  • /archeology — See the history and evolution of changed files
  • /profile — Performance analysis and suggestions
  • /expertise — Find who knows this code best
  • /personas — List all available AI personas

AI Personas (mention to get their perspective):

Persona Focus
@chaos-monkey 🐵 Edge cases & failure scenarios
@skeptic 🤨 Challenge assumptions
@optimizer Performance & efficiency
@security-auditor 🔒 Security vulnerabilities
@accessibility-advocate Inclusive design
@junior-dev 🌱 Simple explanations
@tech-debt-collector 💳 Code quality & shortcuts
@ux-champion 🎨 User experience
@devops-engineer 🚀 Deployment & scaling
@documentation-nazi 📚 Documentation gaps
@legacy-whisperer 🏛️ Working with existing code
@test-driven-purist Testing & TDD

For the best experience, view this PR on myseniordev.com — includes AI chat, file annotations, and interactive reviews.

@khaliqgant khaliqgant merged commit 75dd2ce into main Jan 14, 2026
6 checks passed
@khaliqgant khaliqgant deleted the feature/cli-no-dashboard-default branch January 14, 2026 02:43
@khaliqgant khaliqgant mentioned this pull request Jan 14, 2026
4 tasks
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