Skip to content

chore: remove CLI package#72

Merged
arek-e merged 2 commits into
mainfrom
chore/remove-cli
Apr 3, 2026
Merged

chore: remove CLI package#72
arek-e merged 2 commits into
mainfrom
chore/remove-cli

Conversation

@arek-e
Copy link
Copy Markdown
Owner

@arek-e arek-e commented Apr 3, 2026

Summary

Remove packages/cli/ — 4,368 lines deleted, zero functionality lost.

The CLI duplicates what the dashboard and API already provide. It was the most broken package in the monorepo (missing react/ink types, untested). No users.

Dashboard is the primary UI. curl + SDK cover programmatic access. Can be re-added if a real CLI use case emerges.

Test plan

  • Removed from tsconfig.json references
  • No remaining imports of @paws/cli anywhere

🤖 Generated with Claude Code

Summary by CodeRabbit

Release Notes

  • Chores
    • Updated CI pipeline to include CLI package in automated testing
    • Updated project configuration references

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 3, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a881da7a-25d2-4f15-a15b-d295f73dabc5

📥 Commits

Reviewing files that changed from the base of the PR and between 42a10da and 16f1a1d.

📒 Files selected for processing (2)
  • .github/workflows/ci.yml
  • tsconfig.json

📝 Walkthrough

Walkthrough

The PR removes the CLI package from TypeScript project references in tsconfig.json and updates the CI workflow to include the CLI package in test execution by removing its exclusion filter.

Changes

Cohort / File(s) Summary
CLI Package Configuration
tsconfig.json, .github/workflows/ci.yml
Removes packages/cli from TypeScript project references and removes the --filter='!@paws/cli' exclusion from the CI test step to include CLI tests in the build pipeline.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🐰 The CLI hops back into the fold,
No longer filtered from the tests so bold,
TypeScript references now let it run free,
In the pipeline's embrace, where it's meant to be! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: removing the CLI package. It is specific, accurate, and directly reflects the primary objective of the pull request.
Description check ✅ Passed The description addresses core sections (What, Why) with clear rationale for removal and mentions test verification steps, though it lacks explicit checklist completion markers and deviates from the standard template format.
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 unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/remove-cli

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

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Apr 3, 2026

Deploying getpaws with  Cloudflare Pages  Cloudflare Pages

Latest commit: 16f1a1d
Status:⚡️  Build in progress...

View logs

arek-e and others added 2 commits April 4, 2026 01:44
The CLI duplicates dashboard + API functionality with zero users and
the most broken types in the monorepo. The dashboard is the primary UI,
curl/SDK cover programmatic access. Can be re-added later if needed.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@arek-e arek-e force-pushed the chore/remove-cli branch from 45bc638 to 16f1a1d Compare April 3, 2026 23:44
@arek-e arek-e merged commit 8f64d99 into main Apr 3, 2026
1 of 3 checks passed
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