chore: remove unused apps/cli package

[AviPeltz]

Summary

• Remove the @superset/cli app — an unused prototype (v0.0.0, zero imports, never published)
• Clean up references in root package.json, CI workflow, and PR template

Why / Context

The CLI app was an early prototype built with Ink/Commander for local dev tooling. It was never adopted or imported by any other package in the monorepo. Removing it reduces dead code and avoids confusion.

What Changed

• Deleted apps/cli/ (49 files, ~7200 lines)
• package.json — removed dev:cli script
• .github/workflows/ci.yml — removed --filter=@superset/cli from build step
• .agents/commands/create-pr.md — removed CLI reference from testing checklist

Testing

• bun run lint:fix — passes, no fixes needed

Summary by CodeRabbit

• Chores
  • Removed CLI package from the repository, including all related source code, configuration files, documentation, and tests.
  • Simplified CI/CD workflow to build Desktop only, removing CLI from the build process.
  • Updated development scripts and PR templates to reflect the removal of CLI tooling.

[coderabbitai]

📝 Walkthrough

Walkthrough

This PR removes the entire CLI application from apps/cli directory, including all source code, type definitions, tests, configuration, and documentation. It also updates CI workflows and PR templates to focus on Desktop only, removing CLI build steps.

Changes

Cohort / File(s)	Summary
CLI Source & Components apps/cli/src/app.tsx, apps/cli/src/cli.tsx, apps/cli/src/commands/{agent,change,dashboard,env,init,panels,workspace}.tsx, apps/cli/src/commands/index.ts, apps/cli/src/components/{Spinner,Table}.tsx, apps/cli/src/server.ts	Deleted all React/Ink CLI UI components, entry points, command handlers, and utility components.
CLI Type Definitions apps/cli/src/types/{change,environment,process,workspace}.ts, apps/cli/src/types/index.ts	Removed all exported TypeScript interfaces and enums defining Change, Environment, Process, Agent, Workspace, and their orchestrator interfaces.
CLI Storage & Orchestration apps/cli/src/lib/storage/{adapter,config,lowdb-adapter,types}.ts, apps/cli/src/lib/storage/index.ts, apps/cli/src/lib/orchestrators/{change,environment,process,workspace}-orchestrator.ts, apps/cli/src/lib/orchestrators/index.ts	Removed persistence layer (StorageAdapter, LowdbAdapter), database schema types, and all CRUD orchestrator implementations with cascade delete logic.
CLI Launch & Process Management apps/cli/src/lib/launch/{config,run}.ts	Removed agent launch configuration, tmux-based session management, and process attachment logic.
CLI Tests & Documentation apps/cli/src/lib/orchestrators/*.test.ts, apps/cli/src/lib/storage/*.test.ts, apps/cli/src/lib/persistence_docs/{EXAMPLE,PLAN,SUMMARY}.md, apps/cli/README.md	Deleted all test suites for orchestrators and storage, plus persistence architecture documentation and CLI user guide.
CLI Configuration & Build apps/cli/.editorconfig, apps/cli/.gitattributes, apps/cli/.gitignore, apps/cli/.prettierignore, apps/cli/package.json, apps/cli/tsconfig.json	Removed all CLI project configuration files, package manifest, and TypeScript settings.
Build & CI Adjustments .agents/commands/create-pr.md, .github/workflows/ci.yml, package.json	Shortened PR template build command and simplified CI workflow to build Desktop only (removed CLI from build scope); removed dev:cli script.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~75 minutes

Possibly related PRs

• cli commands #99: Main PR effectively reverts/erases the CLI codebase that PR #99 added/modified across the same apps/cli files and symbols.
• finish ui #106: Directly related as this PR removes the orchestrators and persistence surface that PR #106 added for desktop/CLI integration.
• improve test quality and co-locate test files #161: Deletes the same CLI test files (orchestrator.test.ts, lowdb-adapter.test.ts) that PR #161 edits/moves.

Poem

🐰 A CLI once hopped through the code,
With agents and workspaces it stowed,
But now it must rest,
The Desktop's been blessed,
Simpler paths for us all on this road. ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'chore: remove unused apps/cli package' accurately and concisely summarizes the primary change—deletion of the unused CLI package from the monorepo.
Description check	✅ Passed	The PR description provides clear context (why), a detailed summary of changes, and testing validation, but deviates from the template by omitting the structured sections like Related Issues, Type of Change, and Screenshots.
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 remove-unused-cli-app

---

No actionable comments were generated in the recent review. 🎉

---

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.}

[github-actions]

🧹 Preview Cleanup Complete

The following preview resources have been cleaned up:

• ✅ Neon database branch
• ✅ Electric Fly.io app
• ✅ Streams Fly.io app

Thank you for your contribution! 🎉