Skip to content

fix: catch scheduler tick rejections#15

Merged
yahorbarkouski merged 1 commit intomainfrom
codex/issue-6-scheduler-tick-errors
Mar 29, 2026
Merged

fix: catch scheduler tick rejections#15
yahorbarkouski merged 1 commit intomainfrom
codex/issue-6-scheduler-tick-errors

Conversation

@yahorbarkouski
Copy link
Copy Markdown
Owner

Summary

  • catch and log rejected scheduler tick promises in the scheduler interval callback
  • add a regression test that forces a scheduled capture failure and verifies it is logged
  • confirm issue Scheduler tick errors cause unhandled promise rejections #6 is valid: startScheduler() previously invoked tick() without handling the returned promise

Testing

  • npm test -- electron/main/features/scheduler/tests/SchedulerService.test.ts
  • npx biome check electron/main/features/scheduler/SchedulerService.ts electron/main/features/scheduler/tests/SchedulerService.test.ts
  • npm run typecheck

Closes #6

@yahorbarkouski yahorbarkouski merged commit 9705a89 into main Mar 29, 2026
1 check passed
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 1.30.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Scheduler tick errors cause unhandled promise rejections

1 participant