Skip to content

Conversation

@chsami
Copy link
Owner

@chsami chsami commented Jan 18, 2026

Update breakhandler to allow to stop a plugin when taking a break

Also added config to allow to plugin to stop early before break
Added config to allow plugin to start after a certain amount of time after break ended.

@chsami chsami merged commit b6c1391 into main Jan 18, 2026
2 of 3 checks passed
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 18, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

This PR updates the microbot version and refactors the BreakHandlerV2 plugin-stop functionality from enum-based to string-based storage. It introduces two new utility classes (PluginStopOption and PluginStopHelper) to manage plugin resolution across built-in Microbot plugins and external Plugin Hub plugins. Configuration adds lead-time and restart-delay options for controlling when plugins stop and restart relative to breaks. The UI is updated with a dynamic combo box for plugin selection, and BreakHandlerV2Script implements pre-break plugin stopping with configurable lead time and state management through new timing fields.

Possibly related PRs

  • PR #1622: Directly extends and modifies the BreakHandlerV2 codepath that was initially added in that PR, implementing the plugin-stop feature infrastructure.
  • PR #1653: Adds the initial MicrobotPluginChoice-based stop-plugin feature across the same BreakHandlerV2 files (Config, Overlay, Script), which this PR refactors to use string-based storage with PluginStopOption.
  • PR #1624: Modifies the same BreakHandler V2 codebase files including MicrobotPluginChoice, new PluginStopHelper and PluginStopOption classes, and Microbot UI configuration panel changes.
✨ Finishing touches
  • 📝 Generate docstrings

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.

2 participants