Skip to content

Remove usage of update-notifier from cli.js#71

Open
igorsantos07 wants to merge 2 commits intojasonkneen:masterfrom
igorsantos07:patch-1
Open

Remove usage of update-notifier from cli.js#71
igorsantos07 wants to merge 2 commits intojasonkneen:masterfrom
igorsantos07:patch-1

Conversation

@igorsantos07
Copy link

@igorsantos07 igorsantos07 commented Nov 18, 2025

Damn plugin always causes issues from time to time anyway, and it isn't like this package is getting new important releases anyway 😂

I tried installing the lib in multiple node versions (14, 16, 20) and they all caused different issues with that lib, the latest simply being the lib wasn't being found, even if it got correctly installed 🤷‍♂️

Summary by CodeRabbit

  • Chores
    • Removed automatic update checking and notifications from the CLI.

@coderabbitai
Copy link

coderabbitai bot commented Nov 18, 2025

Walkthrough

The update-notifier dependency and all related notification checks have been removed from the CLI codebase. All require statements, notifier instance creation, and update notification calls are eliminated while preserving core CLI functionality and command handling logic.

Changes

Cohort / File(s) Summary
Update-notifier removal
cli.js, package.json
Removed all update-notifier integration from CLI initialization and command handlers; eliminated the corresponding dependency from package.json

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

  • Straightforward dependency and feature removal with no complex logic changes
  • Affected only 2 files with repetitive removal patterns
  • No control-flow modifications or public API changes

Poem

🐰 Hop, hop—no more checks for what's new,
The notifier's gone, the code runs true!
Cleaner, lighter, swift we bound,
Updates can wait—less noise around! 🥕

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: removing update-notifier usage from cli.js, which is reflected in both cli.js and package.json modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

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