volcano-sdk 0.27.0 → 0.28.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d244570ed2e33751efeb30aa873400168dd80c0c490cf4536ce5dbb6a33f057b
-  data.tar.gz: ef977dcbbff49ebcb7e43ab1cfdf9aa292a58816281a48988af2de203c8c4dca
+  metadata.gz: 47568709329b06a2dd8a84c4ab0058accc47d749d6cc773de6a87c0fc08a6ce1
+  data.tar.gz: d9685709f29037f6db0c5598bae32a84537500f2025337dbf6497a729a315774
 SHA512:
-  metadata.gz: c8f7c362b7f9c3c7c00dc91409e8816c941e2c2244b2b2e1f2035c3c4c40f2aef9a91eec84701cb4230c4ad741ec3a4fe80063a42592012ce8024532b753f698
-  data.tar.gz: f1c26e610f4739303885d5a1d4aa03f86117bc74e4228329e21ef91e379e9f39c55bbee46569dc73ceecb4a68e03d13853e15dcf4cec712ea44fe7d7e6d47bff
+  metadata.gz: 776a324b0d6330c0152e4ef6f46f87d4145f5605c8ed9c4f1fbe648418b8d806e67c668af5f1d41e75ecd5825aa58b4b8d894cb6941c75413f3991153e021f84
+  data.tar.gz: 2b3a2bc60a3cd4e77c32c08f44fc2c8529cc886688381eec5abcdb8ca6bd4f2160fa8bddbfba75544fd865b99b44160a055fece6b65f78c4c6ca1fd79a8bddaf

data/lib/volcano/function_resolution.rb

@@ -16,11 +16,7 @@ module Volcano
     # A cached resolve result. A nil resolution is a remembered miss.
     Outcome = Data.define(:resolution, :failure)
 
-    Failure = Data.define(:message, :code, :retry_after) do
-      def exception
-        Error::NotFoundError.new(message.dup, status: 404, code: code&.dup, retry_after: retry_after)
-      end
-    end
+    Failure = Data.define(:message, :code, :retry_after)
     private_constant :Failure
 
     Entry = Data.define(:outcome, :expires_at)
@@ -51,21 +47,23 @@ module Volcano
       # when the API itself is plaintext, so a resolve response cannot
       # downgrade a credential that is otherwise protected in transit.
       def valid_invoke_url(value, api_url)
-        return nil unless nonempty_url?(value)
+        url = nonempty_url(value)
+        return nil unless url
 
-        uri = URI.parse(value)
+        uri = URI.parse(url)
         return nil unless usable_address?(uri)
 
-        usable_scheme?(uri.scheme, api_url) ? value : nil
+        usable_scheme?(uri.scheme, api_url) ? url : nil
       rescue URI::InvalidURIError
         nil
       end
 
       # Returns the cached Outcome for a name, or nil when it must be resolved.
       def lookup(api_url, authorization, name)
+        # @type var key: [String, String, String]
         key = [api_url, authorization, name]
         @lock.synchronize do
-          entry = @entries[key]
+          entry = @entries.fetch(key, nil)
           next nil if entry.nil?
 
           if entry.expires_at <= now
@@ -89,6 +87,11 @@ module Volcano
         write([api_url, authorization, name], Outcome.new(resolution: nil, failure: failure), NEGATIVE_TTL_SECONDS)
       end
 
+      def error_for(failure)
+        Error::NotFoundError.new(failure.message.dup, status: 404, code: failure.code&.dup,
+                                                      retry_after: failure.retry_after)
+      end
+
       # Drops one cached resolution that turned out to be stale.
       def forget(api_url, authorization, name)
         @lock.synchronize { @entries.delete([api_url, authorization, name]) }
@@ -106,8 +109,8 @@ module Volcano
 
       private
 
-      def nonempty_url?(value)
-        value.is_a?(String) && !value.empty?
+      def nonempty_url(value)
+        value if value.is_a?(String) && !value.empty?
       end
 
       def usable_address?(uri)
@@ -126,15 +129,15 @@ module Volcano
       def write(key, outcome, ttl_seconds)
         @lock.synchronize do
           @entries[key] = Entry.new(outcome: outcome, expires_at: now + ttl_seconds)
-          next if @entries.size <= MAX_ENTRIES
-
-          evict_expired_entries
+          evict_expired_entries if @entries.size > MAX_ENTRIES
         end
       end
 
       def evict_expired_entries
         @entries.delete_if { |_, entry| entry.expires_at <= now }
-        @entries.delete(@entries.min_by { |_, entry| entry.expires_at }.first) while @entries.size > MAX_ENTRIES
+        while @entries.size > MAX_ENTRIES
+          @entries.min_by(1) { |_, entry| entry.expires_at }.each { |pair| @entries.delete(pair.first) }
+        end
       end
     end
   end

data/lib/volcano/function_response.rb

@@ -2,14 +2,15 @@
 
 module Volcano
   # Immutable result of one function invocation.
-  FunctionResponse = Data.define(:data, :status, :headers, :version) do
+  FunctionResponse = Data.define(:data, :status, :headers, :version)
+
+  # Reopen the generated Data class so Steep can check its initializer.
+  class FunctionResponse
     def initialize(data:, status:, headers:, version:)
-      super(
-        data: immutable_json(data),
-        status: status,
-        headers: immutable_json(headers),
-        version: version&.dup&.freeze
-      )
+      data = immutable_json(data)
+      headers = immutable_json(headers)
+      version = version&.dup&.freeze
+      super
     end
 
     private

data/lib/volcano/functions.rb

@@ -98,7 +98,7 @@ module Volcano
     end
 
     def cached_resolution(cached)
-      cached.resolution || raise(cached.failure.exception)
+      cached.resolution || raise(FunctionResolution.error_for(cached.failure))
     end
 
     def validate_invocation(name, payload)

data/lib/volcano/lock_guard.rb

@@ -84,7 +84,7 @@ module Volcano
         wait = wait_duration(deadline)
         return false unless wait
 
-        @changed.wait(@mutex, wait)
+        @changed.wait(@mutex, Float(wait))
       end
     end
 
@@ -129,7 +129,11 @@ module Volcano
     def remaining_time = @lease_clock.remaining
 
     def wait_deadline(timeout)
-      timeout && (@lease_clock.monotonic_now + timeout)
+      return unless timeout
+
+      raise ArgumentError, 'invalid timeout' unless timeout.is_a?(Numeric) && timeout.real? && timeout.finite?
+
+      @lease_clock.monotonic_now + timeout
     end
   end
 end

data/lib/volcano/lock_renewer.rb

@@ -22,10 +22,11 @@ module Volcano
 
     def stop
       request_stop
-      return unless @thread
+      thread = @thread
+      return unless thread
 
       # Renew requires current token ownership, so a late request cannot resurrect a released lease.
-      @thread.join(@config.shutdown_timeout)
+      thread.join(@config.shutdown_timeout)
     end
 
     private

data/lib/volcano/lock_session.rb

@@ -44,6 +44,7 @@ module Volcano
     end
 
     def cleanup(completed)
+      # @type var failure: StandardError?
       failure = nil
       @guard.stop_expiry_watch
       failure = @guard.failure

data/lib/volcano/log_responses.rb

@@ -20,24 +20,28 @@ module Volcano
   end
   private_constant :ImmutableLogJSON
 
+  LogSearchResponse = Data.define(:data, :limit, :has_more, :next_cursor)
+
   # Immutable page returned by a project log search.
-  LogSearchResponse = Data.define(:data, :limit, :has_more, :next_cursor) do
+  class LogSearchResponse
     include ImmutableLogJSON
 
     def initialize(data:, limit:, has_more:, next_cursor: nil)
-      super(
-        data: immutable_json(data), limit: limit, has_more: has_more,
-        next_cursor: next_cursor&.dup&.freeze
-      )
+      data = immutable_json(data)
+      next_cursor = next_cursor&.dup&.freeze
+      super
     end
   end
 
+  LogActivityResponse = Data.define(:data, :total)
+
   # Immutable bucketed project log activity.
-  LogActivityResponse = Data.define(:data, :total) do
+  class LogActivityResponse
     include ImmutableLogJSON
 
     def initialize(data:, total:)
-      super(data: immutable_json(data), total: total)
+      data = immutable_json(data)
+      super
     end
   end
 end

data/lib/volcano/realtime/blocking_call.rb

@@ -4,26 +4,25 @@ module Volcano
   class Realtime
     # Runs one blocking operation off the Async reactor and returns its result.
     module BlockingCall
-      Result = Data.define(:value, :error)
-      private_constant :Result
+      Success = Data.define(:value)
+      Failure = Data.define(:error)
+      private_constant :Success, :Failure
 
-      module_function
-
-      def call(&operation)
+      def self.call(&operation)
         results = ::Queue.new
         worker = Thread.new { execute(results, operation) }
         worker.report_on_exception = false
         result = results.pop
         worker.join
-        raise result.error if result.error
+        raise result.error if result.is_a?(Failure)
 
         result.value
       end
 
-      def execute(results, operation)
-        results << Result.new(value: operation.call, error: nil)
+      def self.execute(results, operation)
+        results << Success.new(value: operation.call)
       rescue StandardError => e
-        results << Result.new(value: nil, error: e)
+        results << Failure.new(error: e)
       end
       private_class_method :execute
     end

data/lib/volcano/realtime/protocol_recovery_position.rb

@@ -21,7 +21,7 @@ module Volcano
       end
 
       def complete_publication(channel, publication)
-        current = @stream_positions[channel]
+        current = @stream_positions.fetch(channel, nil)
         return unless current
 
         completed = publication_position(publication, current)
@@ -49,7 +49,7 @@ module Volcano
       end
 
       def mark_publication_gap(channel, publication)
-        current = @stream_positions[channel]
+        current = @stream_positions.fetch(channel, nil)
         return unless current
 
         gap = publication_position(publication, current)
@@ -68,7 +68,7 @@ module Volcano
       def store_earliest_gap(channel, gap)
         existing = @position_gaps[channel]
         return if existing == true
-        return if existing && existing.fetch(:offset) <= gap.fetch(:offset)
+        return if existing.is_a?(Hash) && existing.fetch(:offset) <= gap.fetch(:offset)
 
         @position_gaps[channel] = gap
       end
@@ -84,7 +84,9 @@ module Volcano
         return unless epoch.is_a?(String) && !epoch.empty?
         return unless offset.is_a?(Integer) && offset >= 0
 
-        { epoch: epoch.dup.freeze, offset: offset.freeze }.freeze
+        # @type var position: position
+        position = { epoch: epoch.dup.freeze, offset: offset.freeze }
+        position.freeze
       end
 
       def same_epoch?(first, second) = first.fetch(:epoch) == second.fetch(:epoch)
@@ -95,7 +97,10 @@ module Volcano
 
       def completion_before_gap?(channel, completed)
         gap = @position_gaps[channel]
-        gap != true && (!gap || completed.fetch(:offset) < gap.fetch(:offset))
+        return false if gap == true
+        return true unless gap.is_a?(Hash)
+
+        completed.fetch(:offset) < gap.fetch(:offset)
       end
 
       def mark_unknown_gap(channel) = @position_gaps[channel] = true

data/lib/volcano/sign_up_result.rb

@@ -1,7 +1,12 @@
 # frozen_string_literal: true
 
 module Volcano
-  SignUpResult = Data.define(:confirmation_required, :message, :session) do
+  SignUpResult = Data.define(:confirmation_required, :message, :session)
+
+  # Reopen the generated Data class so Steep can check its initializer.
+  class SignUpResult
+    # @dynamic confirmation_required, message, session, members, with, to_h, deconstruct, deconstruct_keys
+    # @dynamic self.[], self.members
     def initialize(confirmation_required:, message:, session: nil)
       super
     end

data/lib/volcano/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Volcano
-  VERSION = '0.27.0'
+  VERSION = '0.28.0'
 end

data/sig/lock_guard.rbs

@@ -0,0 +1,21 @@
+module Volcano
+  interface _LeaseStartedAt
+    def monotonic: () -> Float
+    def wall: () -> Time
+  end
+
+  class LockGuard
+    EXPIRY_MESSAGE: String
+    MAX_EXPIRY_POLL_SECONDS: Float
+    UNSAFE_RENEWAL_MESSAGE: String
+
+    attr_reader ttl: Integer
+
+    def initialize: (LockLease lease, ttl: Integer, started_at: _LeaseStartedAt) -> void
+    def lease: () -> LockLease
+    def lost?: () -> bool
+    # The timeout must be a finite real number.
+    def wait_lost: (?timeout: Numeric?) -> bool
+    def failure: () -> StandardError?
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: volcano-sdk
 version: !ruby/object:Gem::Version
-  version: 0.27.0
+  version: 0.28.0
 platform: ruby
 authors:
 - Kong
@@ -609,6 +609,7 @@ files:
 - sig/email_change_result.rbs
 - sig/errors.rbs
 - sig/function_response.rbs
+- sig/lock_guard.rbs
 - sig/lock_records.rbs
 - sig/log_responses.rbs
 - sig/oauth_records.rbs