Skip to content

chore(deps): remove unused ink, react, and ora packages#10

Merged
tinsever merged 1 commit into
masterfrom
chore/remove-unused-ui-deps
May 25, 2026
Merged

chore(deps): remove unused ink, react, and ora packages#10
tinsever merged 1 commit into
masterfrom
chore/remove-unused-ui-deps

Conversation

@tinsever
Copy link
Copy Markdown
Member

@tinsever tinsever commented May 25, 2026

Summary

  • Removes unused production dependencies: ink, ink-select-input, ink-text-input, ora, and react
  • None of these were imported anywhere; the CLI uses commander and picocolors only
  • Supersedes Dependabot PR chore(deps): bump ink from 6.8.0 to 7.0.4 #9 (ink 6 → 7 bump)

Test plan

  • npm ci && npm test (97 tests)
  • npm run typecheck
  • npm install -g . && gfcli --version (CI covers this)

Summary by CodeRabbit

  • Chores
    • Removed several package dependencies.

Review Change Stack

These dependencies were not imported anywhere in the CLI. Dropping them
avoids carrying major-version bumps (e.g. ink 7) for dead weight.
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 25, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a3c4b25b-bbeb-4816-80ce-d2ce3c65a593

📥 Commits

Reviewing files that changed from the base of the PR and between be688b2 and bf2498d.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • package.json

📝 Walkthrough

Walkthrough

This PR removes unused runtime dependencies from the project. Five packages related to terminal UI components and React rendering (ink, ink-select-input, ink-text-input, ora, and react) are deleted from dependencies in package.json, while picocolors is retained.

Changes

Dependency cleanup

Layer / File(s) Summary
Remove terminal UI and React runtime dependencies
package.json
The dependencies section is updated to remove ink, ink-select-input, ink-text-input, ora, and react, leaving only picocolors as a runtime dependency.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A rabbit hops through the deps with care,
Five packages vanish, clean and fair—
UI tools and React fade away,
Colors remain to paint the day. 🐰✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/remove-unused-ui-deps

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

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

@tinsever tinsever merged commit 6ad81e1 into master May 25, 2026
7 of 9 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