Don't diff on arbitrary repack

[bl-ue]

Summary by CodeRabbit

• Chores

  • Updated changelog with latest unreleased changes

• Refactor

  • Repack operation now applies changes directly without displaying a diff preview or requiring user confirmation before writing the changes.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

---

📝 Walkthrough

Walkthrough

This pull request removes the diff approval workflow from the repack handler in src/commands.ts, allowing direct writing of new JavaScript without displaying a diff preview or requiring user confirmation. The changelog is updated to document this change under the Unreleased section.

Changes

Cohort / File(s)	Summary
Documentation CHANGELOG.md	Added unreleased entry documenting the removal of diff confirmation from arbitrary repack operations.
Repack Logic src/commands.ts	Removed diff approval workflow from handleRepack function, eliminating the code that read current JS, prompted for user confirmation, and conditionally wrote the new JS. Function now writes directly to installation.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• Add unpack, repack, and adhoc-patch commands #481 — Adds handleRepack with diff-approval step; this PR reverses that flow by removing the confirmation logic.
• Show lightning fast formatted patch changes via oxfmt before applying patch scripts #489 — Implements full diff-preview/approval flow in the same command handler; this PR simplifies that workflow.

Poem

Hop hop, the code now flows so free,
No diff prompts holding back with glee,
The repack's simplified its way,
Straight through without delay,
Installing swift and clean today! 🐰

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The pull request title 'Don't diff on arbitrary repack' directly and clearly describes the main change: removing the diff preview/approval workflow from the repack operation.
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 (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch dont-diff-on-arbitrary-repack

---

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