homura-runtime 0.3.7 → 0.3.9

data/lib/homura/runtime/cache.rb

@@ -74,8 +74,7 @@ module Cloudflare
     # don't re-open the handle.
     def self.open(name)
       name_str = name.to_s
-      js_promise =
-        `(typeof caches !== 'undefined' && caches && caches.open ? caches.open(#{name_str}) : Promise.resolve(null))`
+      js_promise = `(typeof caches !== 'undefined' && caches && caches.open ? caches.open(#{name_str}) : Promise.resolve(null))`
       js_cache = js_promise.__await__
       Cache.new(js_cache, name_str)
     end
@@ -113,22 +112,20 @@ module Cloudflare
       # JS Request. Derive the URL from the same shapes supported by
       # `request_to_js` so the `url` field on the returned
       # HTTPResponse is always a real URL.
-      url_str =
-        if request_or_url.is_a?(String)
-          request_or_url
-        elsif defined?(Cloudflare::HTTPResponse) &&
-              request_or_url.is_a?(Cloudflare::HTTPResponse)
-          request_or_url.url.to_s
-        elsif `(#{request_or_url} != null && typeof #{request_or_url} === 'object' && typeof #{request_or_url}.url === 'string')`
-          `String(#{request_or_url}.url)`
-        else
-          request_or_url.to_s
-        end
+      url_str = if request_or_url.is_a?(String)
+        request_or_url
+      elsif defined?(Cloudflare::HTTPResponse) &&
+          request_or_url.is_a?(Cloudflare::HTTPResponse)
+        request_or_url.url.to_s
+      elsif `(#{request_or_url} != null && typeof #{request_or_url} === 'object' && typeof #{request_or_url}.url === 'string')`
+        `String(#{request_or_url}.url)`
+      else
+        request_or_url.to_s
+      end
 
       # Single-line backtick IIFE — see `put` for the Opal multi-line
       # x-string quirk that silently drops the returned Promise.
-      js_promise =
-        `(async function(js, req, Kernel, err_klass) { if (js == null || js === Opal.nil) return null; var cached; try { cached = await js.match(req); } catch (e) { Kernel.$raise(err_klass.$new(e && e.message ? e.message : String(e), Opal.hash({ operation: 'match' }))); } if (cached == null) return null; var text = ''; try { text = await cached.text(); } catch (_) { text = ''; } var hk = []; var hv = []; if (cached.headers && typeof cached.headers.forEach === 'function') { cached.headers.forEach(function(v, k) { hk.push(String(k).toLowerCase()); hv.push(String(v)); }); } return { status: cached.status|0, text: text, hkeys: hk, hvals: hv }; })(#{js}, #{req}, #{Kernel}, #{err_klass})`
+      js_promise = `(async function(js, req, Kernel, err_klass) { if (js == null || js === Opal.nil) return null; var cached; try { cached = await js.match(req); } catch (e) { Kernel.$raise(err_klass.$new(e && e.message ? e.message : String(e), Opal.hash({ operation: 'match' }))); } if (cached == null) return null; var text = ''; try { text = await cached.text(); } catch (_) { text = ''; } var hk = []; var hv = []; if (cached.headers && typeof cached.headers.forEach === 'function') { cached.headers.forEach(function(v, k) { hk.push(String(k).toLowerCase()); hv.push(String(v)); }); } return { status: cached.status|0, text: text, hkeys: hk, hvals: hv }; })(#{js}, #{req}, #{Kernel}, #{err_klass})`
       js_result = js_promise.__await__
       return nil if `#{js_result} == null`
 
@@ -141,6 +138,7 @@ module Cloudflare
         h[`#{hkeys}[#{i}]`] = `#{hvals}[#{i}]`
         i += 1
       end
+
       response_klass.new(
         status: `#{js_result}.status`,
         headers: h,
@@ -212,15 +210,19 @@ module Cloudflare
     def request_to_js(request_or_url)
       return `new Request(#{request_or_url})` if request_or_url.is_a?(String)
       if defined?(Cloudflare::HTTPResponse) &&
-           request_or_url.is_a?(Cloudflare::HTTPResponse)
+          request_or_url.is_a?(Cloudflare::HTTPResponse)
         url_str = request_or_url.url.to_s
         return `new Request(#{url_str})`
       end
+
       if `(#{request_or_url} != null && typeof #{request_or_url} === 'object' && typeof #{request_or_url}.url === 'string')`
         return request_or_url
       end
-      raise ArgumentError,
-            "Cloudflare::Cache request must be a String URL, Cloudflare::HTTPResponse, or JS Request (got #{request_or_url.class})"
+
+      raise(
+        ArgumentError,
+        "Cloudflare::Cache request must be a String URL, Cloudflare::HTTPResponse, or JS Request (got #{request_or_url.class})"
+      )
     end
 
     def ruby_headers_to_js(hash)
@@ -230,6 +232,7 @@ module Cloudflare
         vs = v.to_s
         `#{js_obj}[#{ks}] = #{vs}`
       end
+
       js_obj
     end
   end

data/lib/homura/runtime/durable_object.rb

@@ -167,8 +167,7 @@ module Cloudflare
 
       # Single-line IIFE — see `lib/homura/runtime/cache.rb#put`
       # for why Opal can silently drop a multi-line x-string Promise.
-      js_promise =
-        `(async function(stub, url_str, method_str, js_headers, js_body, Kernel, err_klass, do_class_label) { var init = { method: method_str, headers: js_headers }; if (js_body !== null && js_body !== undefined && js_body !== Opal.nil) { init.body = js_body; } var resp; try { resp = await stub.fetch(url_str, init); } catch (e) { Kernel.$raise(err_klass.$new(e && e.message ? e.message : String(e), Opal.hash({ operation: 'stub.fetch', do_class: do_class_label }))); } var text = ''; try { text = await resp.text(); } catch (_) { text = ''; } var hk = []; var hv = []; if (resp.headers && typeof resp.headers.forEach === 'function') { resp.headers.forEach(function(v, k) { hk.push(String(k).toLowerCase()); hv.push(String(v)); }); } return { status: resp.status|0, text: text, hkeys: hk, hvals: hv }; })(#{js_stub}, #{url_str}, #{method_str}, #{js_headers}, #{js_body}, #{Kernel}, #{err_klass}, #{do_class_label})`
+      js_promise = `(async function(stub, url_str, method_str, js_headers, js_body, Kernel, err_klass, do_class_label) { var init = { method: method_str, headers: js_headers }; if (js_body !== null && js_body !== undefined && js_body !== Opal.nil) { init.body = js_body; } var resp; try { resp = await stub.fetch(url_str, init); } catch (e) { Kernel.$raise(err_klass.$new(e && e.message ? e.message : String(e), Opal.hash({ operation: 'stub.fetch', do_class: do_class_label }))); } var text = ''; try { text = await resp.text(); } catch (_) { text = ''; } var hk = []; var hv = []; if (resp.headers && typeof resp.headers.forEach === 'function') { resp.headers.forEach(function(v, k) { hk.push(String(k).toLowerCase()); hv.push(String(v)); }); } return { status: resp.status|0, text: text, hkeys: hk, hvals: hv }; })(#{js_stub}, #{url_str}, #{method_str}, #{js_headers}, #{js_body}, #{Kernel}, #{err_klass}, #{do_class_label})`
 
       js_result = js_promise.__await__
       hkeys = `#{js_result}.hkeys`
@@ -198,6 +197,7 @@ module Cloudflare
           hdrs["content-type"] = "application/json"
         end
       end
+
       fetch(path, method: method, headers: hdrs, body: request_body)
     end
 
@@ -258,11 +258,11 @@ module Cloudflare
       unless class_name.is_a?(String)
         raise ArgumentError, "class_name must be a String"
       end
+
       @handlers ||= {}
       # Wrap via define_method so Opal's `# await: true` picks it up as
       # async (same trick Sinatra::Scheduled uses for its jobs).
-      method_name =
-        "__do_handler_#{class_name.gsub(/[^A-Za-z0-9_]/, "_")}".to_sym
+      method_name = "__do_handler_#{class_name.gsub(/[^A-Za-z0-9_]/, "_")}".to_sym
       DurableObjectRequestContext.send(:define_method, method_name, &block)
       unbound = DurableObjectRequestContext.instance_method(method_name)
       DurableObjectRequestContext.send(:remove_method, method_name)
@@ -317,9 +317,7 @@ module Cloudflare
         body = {
           "error" => "no Ruby handler for DurableObject class #{class_name}"
         }.to_json
-        return(
-          build_js_response(500, { "content-type" => "application/json" }, body)
-        )
+        return (build_js_response(500, {"content-type" => "application/json"}, body))
       end
 
       state = DurableObjectState.new(js_state)
@@ -354,7 +352,7 @@ module Cloudflare
           (result["body"] || result[:body] || "").to_s
         ]
       else
-        [200, { "content-type" => "text/plain; charset=utf-8" }, result.to_s]
+        [200, {"content-type" => "text/plain; charset=utf-8"}, result.to_s]
       end
     end
 
@@ -365,6 +363,7 @@ module Cloudflare
         vs = v.to_s
         `#{js_headers}[#{ks}] = #{vs}`
       end
+
       status_int = status.to_i
       body_str = body.to_s
       `new Response(#{body_str}, { status: #{status_int}, headers: #{js_headers} })`
@@ -475,10 +474,12 @@ module Cloudflare
           ts = t.to_s
           `#{js_tags}.push(#{ts})`
         end
+
         `#{js_state}.acceptWebSocket(#{js_ws}, #{js_tags})`
       else
         `#{js_state}.acceptWebSocket(#{js_ws})`
       end
+
       nil
     end
 
@@ -487,13 +488,13 @@ module Cloudflare
     # `getWebSockets(tag)`.
     def web_sockets(tag: nil)
       js_state = @js_state
-      js_arr =
-        if tag
-          ts = tag.to_s
-          `(#{js_state}.getWebSockets ? #{js_state}.getWebSockets(#{ts}) : [])`
-        else
-          `(#{js_state}.getWebSockets ? #{js_state}.getWebSockets() : [])`
-        end
+      js_arr = if tag
+        ts = tag.to_s
+        `(#{js_state}.getWebSockets ? #{js_state}.getWebSockets(#{ts}) : [])`
+      else
+        `(#{js_state}.getWebSockets ? #{js_state}.getWebSockets() : [])`
+      end
+
       out = []
       len = `#{js_arr}.length`
       i = 0
@@ -501,6 +502,7 @@ module Cloudflare
         out << `#{js_arr}[#{i}]`
         i += 1
       end
+
       out
     end
   end
@@ -567,8 +569,7 @@ module Cloudflare
       `#{js_opts}.reverse = #{!!reverse}` unless reverse.nil?
       `#{js_opts}.start   = #{start.to_s}` unless start.nil?
       `#{js_opts}.end     = #{end_key.to_s}` unless end_key.nil?
-      js_promise =
-        `#{js}.list(#{js_opts}).catch(function(e) { #{Kernel}.$raise(#{err_klass}.$new(e && e.message ? e.message : String(e), Opal.hash({ operation: 'storage.list' }))); })`
+      js_promise = `#{js}.list(#{js_opts}).catch(function(e) { #{Kernel}.$raise(#{err_klass}.$new(e && e.message ? e.message : String(e), Opal.hash({ operation: 'storage.list' }))); })`
       js_result = js_promise.__await__
       out = {}
       return out if `#{js_result} == null`
@@ -646,9 +647,11 @@ module Cloudflare
     def storage
       @state.storage
     end
+
     def cf_env
       env["cloudflare.env"]
     end
+
     def cf_ctx
       env["cloudflare.ctx"]
     end
@@ -656,24 +659,31 @@ module Cloudflare
     def d1
       env["cloudflare.DB"]
     end
+
     def db
       d1
     end
+
     def kv
       env["cloudflare.KV"]
     end
+
     def bucket
       env["cloudflare.BUCKET"]
     end
+
     def ai
       Cloudflare::Bindings.ai(env)
     end
+
     def send_email
       env["cloudflare.SEND_EMAIL"]
     end
+
     def jobs_queue
       env["cloudflare.QUEUE_JOBS"]
     end
+
     def durable_object(name, id_or_name = nil)
       Cloudflare::Bindings.durable_object(env, name, id_or_name)
     end

data/lib/homura/runtime/email.rb

@@ -52,20 +52,18 @@ module Cloudflare
       has_html = !(html.nil? || html.to_s.empty?)
       raise Error, "text or html is required" unless has_text || has_html
 
-      payload =
-        build_send_payload(
-          to: to,
-          from: from,
-          subject: subject.to_s,
-          text: text,
-          html: html,
-          reply_to: reply_to
-        )
+      payload = build_send_payload(
+        to: to,
+        from: from,
+        subject: subject.to_s,
+        text: text,
+        html: html,
+        reply_to: reply_to
+      )
 
       cf = Cloudflare
       # 多行 x-string をメソッド末尾に置くと Opal が Promise を返さない出力になることがあるため return を明示する。
-      return(
-        `(async function(binding, payload, Kernel, Err, cf) {
+      return (`(async function(binding, payload, Kernel, Err, cf) {
         try {
           var r = await binding.send(payload);
           if (r == null || r === undefined) {
@@ -83,8 +81,7 @@ module Cloudflare
           var msg = (e && e.message) ? String(e.message) : String(e);
           Kernel.$raise(Err.$new(msg, Opal.hash({ code: code })));
         }
-      })(#{js}, #{payload}, #{Kernel}, #{err_klass}, #{cf})`
-      )
+      })(#{js}, #{payload}, #{Kernel}, #{err_klass}, #{cf})`)
     end
 
     private
@@ -128,9 +125,11 @@ module Cloudflare
           if e[:name] && !e[:name].to_s.strip.empty?
             `#{js}.name = #{e[:name].to_s}`
           end
+
           `#{arr}.push(#{js})`
         end
       end
+
       arr
     end
 
@@ -149,8 +148,9 @@ module Cloudflare
         if nm.nil? || nm.to_s.strip.empty?
           [em.to_s.strip]
         else
-          [{ email: em.to_s.strip, name: nm.to_s }]
+          [{email: em.to_s.strip, name: nm.to_s}]
         end
+
       when Array
         raw.flat_map { |x| flatten_recipients(x) }
       else

data/lib/homura/runtime/http.rb

@@ -76,7 +76,8 @@ module Cloudflare
       response_klass = Cloudflare::HTTPResponse
       err_klass = Cloudflare::HTTPError
       headers_to_hash = method(:js_headers_to_hash)
-      _ = headers_to_hash # silence opal lint
+      # silence opal lint
+      _ = headers_to_hash
 
       # NOTE: the multi-line backtick below returns a Promise BECAUSE it
       # is assigned to `js_promise` (and `js_promise.__await__` awaits
@@ -87,8 +88,7 @@ module Cloudflare
       # expression. See the single-line IIFE pattern used in
       # lib/homura/runtime/{cache,queue,durable_object}.rb#put for
       # the alternative that survives either position. (Phase 11B audit.)
-      js_promise =
-        `
+      js_promise = `
         (async function() {
           var init = { method: #{method_str}, headers: #{js_headers}, redirect: 'follow' };
           if (#{js_body} !== nil && #{js_body} != null) { init.body = #{js_body}; }
@@ -144,6 +144,7 @@ module Cloudflare
         vs = v.to_s
         `#{js_obj}[#{ks}] = #{vs}`
       end
+
       js_obj
     end
 

data/lib/homura/runtime/multipart.rb

@@ -56,6 +56,7 @@ module Cloudflare
     def size
       @bytes_binstr.length
     end
+
     alias_method :bytesize, :size
 
     # Convenience accessor matching the CRuby Rack shape.
@@ -106,6 +107,7 @@ module Cloudflare
         tempfile: self
       }
     end
+
     alias_method :to_hash, :to_h
 
     # `#[]` so `file[:filename]` works on the UploadedFile itself
@@ -133,9 +135,11 @@ module Cloudflare
       if (m = ct.match(/boundary="([^"]+)"/i))
         return m[1]
       end
+
       if (m = ct.match(/boundary=([^;,\s]+)/i))
         return m[1]
       end
+
       nil
     end
 
@@ -154,7 +158,8 @@ module Cloudflare
       sep = "--" + boundary
       term = "--" + boundary + "--"
       sep_line = sep + CRLF
-      sep_last = sep + CRLF # the very first boundary may skip the leading CRLF
+      # the very first boundary may skip the leading CRLF
+      sep_last = sep + CRLF
       body = body_binstr.to_s
 
       # Skip any preamble before the first boundary.
@@ -251,13 +256,14 @@ module Cloudflare
       # Quoted `key="value"`
       q_re = /(?:^|[;\s])#{k}\s*=\s*"((?:\\"|[^"])*)"/i
       if (m = disposition.match(q_re))
-        return m[1].gsub('\\"', '"')
+        return m[1].gsub("\\\"", "\"")
       end
       # Bare `key=value`
       b_re = /(?:^|[;\s])#{k}\s*=\s*([^;]+)/i
       if (m = disposition.match(b_re))
         return m[1].strip
       end
+
       nil
     end
 
@@ -276,10 +282,10 @@ module Cloudflare
       return cached if cached
 
       ct = env["CONTENT_TYPE"]
-      return({}) unless ct && ct.to_s.downcase.include?("multipart/")
+      return ({}) unless ct && ct.to_s.downcase.include?("multipart/")
 
       io = env["rack.input"]
-      return({}) if io.nil?
+      return ({}) if io.nil?
 
       # `rack.input` is normally a StringIO wrapping the body_binstr
       # we staged in src/worker.mjs. Read the full body; it's already
@@ -292,6 +298,7 @@ module Cloudflare
           # some stubs don't support rewind — ignore
         end
       end
+
       body = io.respond_to?(:read) ? io.read.to_s : ""
 
       parsed = parse(body, ct)

data/lib/homura/runtime/queue.rb

@@ -82,11 +82,13 @@ module Cloudflare
       qname = @name
       err_klass = Cloudflare::QueueError
       unless available?
-        raise QueueError.new(
-                "queue binding not bound",
-                operation: "send",
-                queue: qname
-              )
+        raise(
+          QueueError.new(
+            "queue binding not bound",
+            operation: "send",
+            queue: qname
+          )
+        )
       end
 
       js_body = ruby_to_js(body)
@@ -108,11 +110,13 @@ module Cloudflare
       qname = @name
       err_klass = Cloudflare::QueueError
       unless available?
-        raise QueueError.new(
-                "queue binding not bound",
-                operation: "send_batch",
-                queue: qname
-              )
+        raise(
+          QueueError.new(
+            "queue binding not bound",
+            operation: "send_batch",
+            queue: qname
+          )
+        )
       end
 
       js_msgs = `([])`
@@ -131,6 +135,7 @@ module Cloudflare
           `#{js_msgs}.push({ body: #{js_body} })`
         end
       end
+
       js_opts = `({})`
       `#{js_opts}.delaySeconds = #{delay_seconds.to_i}` if delay_seconds
 
@@ -143,10 +148,14 @@ module Cloudflare
     # Same Ruby→JS conversion that Cloudflare::AI.ruby_to_js uses, but
     # local to avoid cross-wrapper coupling.
     def ruby_to_js(val)
-      if val.is_a?(String) || val.is_a?(Numeric) || val == true ||
-           val == false || val.nil?
+      if val.is_a?(String) ||
+          val.is_a?(Numeric) ||
+          val == true ||
+          val == false ||
+          val.nil?
         return val
       end
+
       return val.to_s if val.is_a?(Symbol)
       if val.is_a?(Hash)
         obj = `({})`
@@ -155,16 +164,20 @@ module Cloudflare
           jv = ruby_to_js(v)
           `#{obj}[#{ks}] = #{jv}`
         end
+
         return obj
       end
+
       if val.is_a?(Array)
         arr = `([])`
         val.each do |v|
           jv = ruby_to_js(v)
           `#{arr}.push(#{jv})`
         end
+
         return arr
       end
+
       val
     end
   end
@@ -185,8 +198,7 @@ module Cloudflare
 
     def timestamp
       js = @js
-      ms =
-        `(#{js} && #{js}.timestamp && typeof #{js}.timestamp.getTime === 'function' ? #{js}.timestamp.getTime() : null)`
+      ms = `(#{js} && #{js}.timestamp && typeof #{js}.timestamp.getTime === 'function' ? #{js}.timestamp.getTime() : null)`
       return nil if `#{ms} == null`
       Time.at(ms.to_f / 1000.0)
     end
@@ -234,6 +246,7 @@ module Cloudflare
       else
         `(#{js} && typeof #{js}.retry === 'function' ? #{js}.retry() : null)`
       end
+
       nil
     end
 
@@ -244,6 +257,7 @@ module Cloudflare
       if `typeof #{v} === 'string' || typeof #{v} === 'number' || typeof #{v} === 'boolean'`
         return v
       end
+
       if `Array.isArray(#{v})`
         out = []
         len = `#{v}.length`
@@ -252,8 +266,10 @@ module Cloudflare
           out << js_to_ruby(`#{v}[#{i}]`)
           i += 1
         end
+
         return out
       end
+
       if `typeof #{v} === 'object'`
         h = {}
         keys = `Object.keys(#{v})`
@@ -264,8 +280,10 @@ module Cloudflare
           h[k] = js_to_ruby(`#{v}[#{k}]`)
           i += 1
         end
+
         return h
       end
+
       v
     end
   end
@@ -296,6 +314,7 @@ module Cloudflare
         out << QueueMessage.new(`#{arr}[#{i}]`)
         i += 1
       end
+
       @messages = out
     end
 
@@ -317,6 +336,7 @@ module Cloudflare
       else
         `(#{js} && typeof #{js}.retryAll === 'function' ? #{js}.retryAll() : null)`
       end
+
       nil
     end
   end
@@ -353,15 +373,15 @@ module Cloudflare
       queue_name = batch.queue
       handler = handler_for(queue_name)
       if handler.nil?
-        warn "[Cloudflare::QueueConsumer] no handler registered for queue #{queue_name.inspect}; messages will time out and retry"
-        return(
-          {
-            "queue" => queue_name,
-            "handled" => false,
-            "size" => batch.size,
-            "reason" => "no_handler"
-          }
+        warn(
+          "[Cloudflare::QueueConsumer] no handler registered for queue #{queue_name.inspect}; messages will time out and retry"
         )
+        return ({
+          "queue" => queue_name,
+          "handled" => false,
+          "size" => batch.size,
+          "reason" => "no_handler"
+        })
       end
 
       ctx = QueueContext.new(batch, js_env, js_ctx)
@@ -369,6 +389,7 @@ module Cloudflare
       if `(#{result} != null && typeof #{result}.then === 'function')`
         result = result.__await__
       end
+
       {
         "queue" => queue_name,
         "handled" => true,
@@ -403,30 +424,39 @@ module Cloudflare
     def d1
       env["cloudflare.DB"]
     end
+
     def db
       d1
     end
+
     def cf_env
       env["cloudflare.env"]
     end
+
     def cf_ctx
       env["cloudflare.ctx"]
     end
+
     def kv
       env["cloudflare.KV"]
     end
+
     def bucket
       env["cloudflare.BUCKET"]
     end
+
     def ai
       Cloudflare::Bindings.ai(env)
     end
+
     def send_email
       env["cloudflare.SEND_EMAIL"]
     end
+
     def jobs_queue
       env["cloudflare.QUEUE_JOBS"]
     end
+
     def durable_object(name, id_or_name = nil)
       Cloudflare::Bindings.durable_object(env, name, id_or_name)
     end
@@ -446,7 +476,7 @@ module Cloudflare
       Cloudflare::Bindings.build_env(
         js_env,
         @js_ctx,
-        { "cloudflare.queue" => true }
+        {"cloudflare.queue" => true}
       )
     end
   end

data/lib/homura/runtime/scheduled.rb

@@ -63,16 +63,14 @@ module Cloudflare
       return new(cron: "", scheduled_time: Time.now) if `#{js_event} == null`
 
       cron = `(#{js_event}.cron == null ? '' : String(#{js_event}.cron))`
-      type =
-        `(#{js_event}.type == null ? 'scheduled' : String(#{js_event}.type))`
+      type = `(#{js_event}.type == null ? 'scheduled' : String(#{js_event}.type))`
       has_sched = `(#{js_event}.scheduledTime != null)`
-      sched_t =
-        if has_sched
-          sched_ms = `Number(#{js_event}.scheduledTime)`
-          Time.at(sched_ms.to_f / 1000.0)
-        else
-          Time.now
-        end
+      sched_t = if has_sched
+        sched_ms = `Number(#{js_event}.scheduledTime)`
+        Time.at(sched_ms.to_f / 1000.0)
+      else
+        Time.now
+      end
 
       new(cron: cron, scheduled_time: sched_t, type: type, raw: js_event)
     end
@@ -121,11 +119,10 @@ module Cloudflare
       event = ScheduledEvent.from_js(js_event)
       target = resolve_app
       if target.nil?
-        warn "[Cloudflare::Scheduled] no app registered; ignoring scheduled event"
-        return(
-          { "fired" => 0, "total" => 0, "results" => [], "error" => "no_app" }
-        )
+        warn("[Cloudflare::Scheduled] no app registered; ignoring scheduled event")
+        return ({"fired" => 0, "total" => 0, "results" => [], "error" => "no_app"})
       end
+
       target.dispatch_scheduled(event, js_env, js_ctx).__await__
     end
 
@@ -171,6 +168,7 @@ module Cloudflare
       if candidate.nil? && defined?(::Rack::Handler::Homura)
         candidate = ::Rack::Handler::Homura.app
       end
+
       return nil if candidate.nil?
       # Sinatra app classes respond to `dispatch_scheduled` (added by
       # Sinatra::Scheduled). Plain Rack apps would be instances and
@@ -180,7 +178,7 @@ module Cloudflare
       if candidate.respond_to?(:dispatch_scheduled)
         candidate
       elsif candidate.respond_to?(:class) &&
-            candidate.class.respond_to?(:dispatch_scheduled)
+          candidate.class.respond_to?(:dispatch_scheduled)
         candidate.class
       else
         candidate