Skip to content

Misc fixes#440

Merged
bl-ue merged 1 commit intomainfrom
misc-fixes
Jan 30, 2026
Merged

Misc fixes#440
bl-ue merged 1 commit intomainfrom
misc-fixes

Conversation

@bl-ue
Copy link
Member

@bl-ue bl-ue commented Jan 30, 2026 •
edited by coderabbitai bot
Loading

Summary by CodeRabbit

  • New Features

    • Patches now track and display when they're skipped due to unmet conditions, providing better visibility into the evaluation process

  • Documentation

    • Updated startup message with clarified instructions on using the --apply flag when reapplying changes via the command line

✏️ Tip: You can customize this high-level summary in your review settings.

@bl-ue bl-ue enabled auto-merge (squash) January 30, 2026 16:29
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 30, 2026 •
edited
Loading

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The PR renames the PatchGroup.NEW_FEATURES group to PatchGroup.FEATURES, introduces a skipped-tracking mechanism to record patches with unmet conditions, makes the Patch interface's description field required, and updates the startup notification message to use invocationCommand with --apply CLI instruction.

Changes

Cohort / File(s) Summary
Patch System Updates
src/patches/index.ts, src/index.tsx		 Renamed PatchGroup.NEW_FEATURES to PatchGroup.FEATURES; introduced skipped-tracking mechanism to record patches with false conditions as skipped; made description field required in Patch interface; added skipped?: boolean to PatchResult interface.
UI Message Update
src/ui/App.tsx		 Updated startup notification message to instruct reapplying changes via CLI using invocationCommand with --apply flag, replacing previous static guidance.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~15 minutes

Possibly related PRs

Poem

🐰 A clever hop through patch designs,
Where skipped conditions now align,
FEATURES gleams where NEW once shone,
And CLI wisdom, clearly shown—
The tweaks are sharp, the changes bright! ✨

✨ 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 misc-fixes

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.

@bl-ue bl-ue merged commit e16d0e1 into main Jan 30, 2026
1 of 2 checks passed
@bl-ue bl-ue deleted the misc-fixes branch January 30, 2026 16:30
This was referenced Feb 3, 2026
Merged
Merged
Merged
Merged
Merged
Closed
Merged
Merged
Merged
This was referenced Mar 1, 2026
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@bl-ue