improve: Only load async workers if deps available

[jylamont]

This PR aims to further insulate queue workers implementations if your app doesn't have this dependency. Zeitwerk will only load the code if a constant like Sidekiq is defined.

[jylamont]

• improve: Only load async workers if deps available #89 : 2 dependent PRs (#92 , #93 ) 👈 (View in Graphite)
• refactor: Refactor #88
• bug: Fixed RestClient logger issue #91
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

[Copilot]

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (2)

lib/vero/workers/resque_worker.rb:9

• The removal of the options.transform_keys call in ResqueWorker may affect the expected key format. Please verify that Vero::Senders::Base.new.call handles options consistently without requiring symbolized keys.

Vero::Senders::Base.new.call(api_class, domain, options)

lib/vero.rb:29

• Since the SuckerPunchWorker has been relocated from lib/vero/sucker_punch_worker.rb to lib/vero/workers/sucker_punch_worker.rb, please confirm that the ignore path is correct and matches the new file structure.

loader.ignore("#{__dir__}/vero/workers/sucker_punch_worker") unless defined?(SuckerPunch)