Stop relying on fugit (#2519)

* Stop relying on fugit

It is an overkill to have a (implicit) dependency
on fugit, especially that it doesn't even offer
a public interface to do what we need.

It's older version don't support timezones in cron
which I believe is the culprit in #2513.

Even if it's not, I still think it's a good change.

* Update CHANGELOG

Author: solnic

CHANGELOG.md

@@ -17,6 +17,7 @@
 
 - Test Ruby 3.4 in CI ([#2506](https://github.com/getsentry/sentry-ruby/pull/2506))
 - Upgrade actions workflows versions ([#2506](https://github.com/getsentry/sentry-ruby/pull/2506))
+- Stop relying on fugit ([#2519](https://github.com/getsentry/sentry-ruby/pull/2519))
 
 ## 5.22.1
 

sentry-sidekiq/lib/sentry/sidekiq-scheduler/scheduler.rb

@@ -38,16 +38,14 @@ def new_job(name, interval_type, config, schedule, options)
         monitor_config =
           case interval_type
           when "cron"
-            # fugit is a second order dependency of sidekiq-scheduler via rufus-scheduler
-            parsed_cron = ::Fugit.parse_cron(schedule)
-            timezone = parsed_cron.timezone
+            # Split schedule into cron and timezone parts (if timezone exists)
+            cron_parts = schedule.strip.split(" ")
 
-            # fugit supports having the timezone part of the cron string,
-            # so we need to pull that with some hacky stuff
-            if timezone
-              parsed_cron.instance_variable_set(:@timezone, nil)
-              cron_without_timezone = parsed_cron.to_cron_s
-              Sentry::Cron::MonitorConfig.from_crontab(cron_without_timezone, timezone: timezone.name)
+            if cron_parts.length > 5
+              timezone = cron_parts.pop
+              cron_without_timezone = cron_parts.join(" ")
+
+              Sentry::Cron::MonitorConfig.from_crontab(cron_without_timezone, timezone: timezone)
             else
               Sentry::Cron::MonitorConfig.from_crontab(schedule)
             end