Skip to content

Conversation

@fluentcrafter
Copy link
Contributor

@fluentcrafter fluentcrafter commented Nov 20, 2025

This PR implements issue(s) #

Checklist

  • I wrote new tests for my new core changes.
  • I have successfully ran tests, style checker and build against my new changes locally.
  • I have informed the team of any breaking changes if there are any.

Note

Refactors liveness bundle selection in LineaTransactionSelectorFactory to early-return on missing/empty services, streamline evaluation, and centralize uptime metric updates with commit/rollback.

  • Sequencer – Transaction Selection:
    • Streamlines checkAndSendLivenessBundle in LineaTransactionSelectorFactory:
      • Early-return if livenessService is absent or no liveness bundle is built.
      • Replaces nested Optional usage with direct local reference to livenessService.
      • Consolidates evaluation flow: logs, detects interrupt, updates uptime metrics, and commits/rolls back accordingly.
      • Uses livenessService.updateUptimeMetrics(...) consistently after success/failure.

Written by Cursor Bugbot for commit e406a3a. This will update automatically on new commits. Configure here.

@fluentcrafter fluentcrafter changed the base branch from main to fix/interrupt-bundle-selection-on-timeout November 20, 2025 14:43
Base automatically changed from fix/interrupt-bundle-selection-on-timeout to main November 20, 2025 14:43
@fluentcrafter fluentcrafter requested review from ahamlat, fab-10 and jonesho and removed request for fab-10 November 20, 2025 14:43
@fluentcrafter fluentcrafter force-pushed the chore-sequencer-plugin-core-readability branch from 80c6bb2 to e406a3a Compare November 20, 2025 14:45
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.

3 participants