Walkthrough

Adds guarded initialization of plugin schedules when RUN == "schedule": computes schedule via Cron(run_sch).schedule(...), validates non-None before appending to conf.mySchedules, raises ValueError with specific error log on invalid schedules, and adds a catch-all exception handler for unexpected scheduling errors.

Changes

Sequence Diagram(s)

sequenceDiagram
  autonumber
  participant Init as initialise.py
  participant Cron as Cron(run_sch)
  participant Conf as conf.mySchedules
  participant Log as Logger

  rect rgb(240,245,255)
  note over Init: RUN == "schedule"
  Init->>Cron: schedule(plugin, offset, ...)
  Cron-->>Init: scheduleObj or None / throws
  end

  alt Valid scheduleObj
    Init->>Conf: append(scheduleObj)
    Log-->>Init: info: schedule set
  else Invalid (None)
    Init->>Log: [ERROR] Invalid schedule
    Init->>Init: raise ValueError
  else Exception thrown
    Init->>Log: [ERROR] Unexpected scheduling error
  end

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I twitch my whiskers at the clockwork chime,
Now schedules hop in validated time.
If None appears, I thump—abort the run!
With guarded paths, the race is neatly won.
Logs nibble errors; carrots for success—yum!

Pre-merge checks and finishing touches

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.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}