gamefic 2.2.3 → 2.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a57ab533bcfbcbd650cf3ee60c5a639b3ddd40ad3ad96d0d4e42e6658547f3d9
-  data.tar.gz: 904c5211a6bde65f870ae8bab1218423afd6618c6ddd79c44b01174bb069256b
+  metadata.gz: 2288262e86ac4a5cf259b3a37e80ba56a3a4ed334b8842770109e1930799585d
+  data.tar.gz: 542aaff9c3f4d0aae1a4471fba5d52ff35b569971c861af006ddc78f0d1cc568
 SHA512:
-  metadata.gz: afc3e6a06d6ac451827cea70db94b8c9e4b10c795dac9812df0c24f8c1e0ce654748dfae53c4c0565bbfff8abb5d2f853f866c1aac8b398c5cf8331ee3ea1795
-  data.tar.gz: dd790a5574ae81e91b61d01c7f18adba0abcd7e43650e72934329dfd84c9515adb54b6e1e0938814c9d6c6582d23801c66a470a28ae99700659fa0c462830660
+  metadata.gz: 6a89c7fb5978047517827e3cc2de5f172da5e6905a7e7efe0725763b4ce01d56725df46a443b903906effd962cebdbd238d81719b315eb26f3b8bf289d337089
+  data.tar.gz: 9976d0c58957309ba605e645a8c7e214d798c12e114b71e5d5fa2bb4b6d39878537612e670da6e42168dfd31ee5fce7304eee9983d1177d878262af630e6e582

data/.github/workflows/rspec.yml

@@ -0,0 +1,40 @@
+# This workflow uses actions that are not certified by GitHub.
+# They are provided by a third-party and are governed by
+# separate terms of service, privacy policy, and support
+# documentation.
+# This workflow will download a prebuilt Ruby version, install dependencies and run tests with rspec.
+# For more information see: https://github.com/marketplace/actions/setup-ruby-jruby-and-truffleruby
+
+name: RSpec
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+permissions:
+  contents: read
+
+jobs:
+  test:
+
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        ruby-version: ['2.5', '2.6', '2.7', '3.0', '3.1']
+
+    steps:
+    - uses: actions/checkout@v3
+    - name: Set up Ruby
+    # To automatically get bug fixes and new Ruby versions for ruby/setup-ruby,
+    # change this to (see https://github.com/ruby/setup-ruby#versioning):
+      uses: ruby/setup-ruby@v1
+      # uses: ruby/setup-ruby@2b019609e2b0f1ea1a2bc8ca11cb82ab46ada124
+      with:
+        ruby-version: ${{ matrix.ruby-version }}
+        bundler-cache: false
+    - name: Install dependencies
+      run: bundle install
+    - name: Run tests
+      run: bundle exec rspec

data/.rubocop.yml

@@ -10,4 +10,7 @@ Style/StringLiterals:
 
 Style/Documentation:
   Description: 'Document classes and non-namespace modules.'
-  Enabled: false
+  Enabled: false
+
+Style/MethodDefParentheses:
+  Enabled: false

data/CHANGELOG.md

@@ -1,5 +1,16 @@
+## 2.4.0 - February 11, 2023
+- Fix arity of delegated methods in scripts
+- Action hooks accept verb filters
+- Opal exception for delegated methods
+- Support separation of kwargs in Ruby >= 2.7
+
+## 2.3.0 - January 25, 2023
+- Remove unused Active#actions method
+- Add before_action and after_action hooks
+
 ## 2.2.3 - July 9, 2022
 - Fix Ruby version incompatibilities
+- Fix private attr_accessor call
 
 ## 2.2.2 - September 6, 2021
 - Darkroom indexes non-static elements

data/Gemfile

@@ -1,7 +1,3 @@
 source 'https://rubygems.org'
 
 gemspec
-
-group :test do
-  gem "simplecov"
-end

data/gamefic.gemspec

@@ -19,7 +19,7 @@ Gem::Specification.new do |s|
   end
   s.require_paths = ['lib']
 
-  s.required_ruby_version = '>= 2.1.0'
+  s.required_ruby_version = '>= 2.3.0'
 
   s.add_development_dependency 'rake', '~> 12.3', '>= 12.3'
   s.add_development_dependency 'rspec', '~> 3.5', '>= 3.5.0'

data/lib/gamefic/action.rb

@@ -1,5 +1,8 @@
 module Gamefic
   class Action
+    # @return [Gamefic::Actor]
+    attr_reader :actor
+
     # An array of objects on which the action will operate, e.g., an entity
     # that is a direct object of a command.
     #
@@ -7,17 +10,34 @@ module Gamefic
     attr_reader :arguments
     alias parameters arguments
 
-    def initialize actor, arguments
+    # @param actor [Gamefic::Actor]
+    # @param arguments [Array<Object>]
+    def initialize actor, arguments, with_callbacks = false
       @actor = actor
       @arguments = arguments
       @executed = false
+      @with_callbacks = with_callbacks
     end
 
     # Perform the action.
     #
     def execute
-      @executed = true
+      return if @cancelled
+      run_before_actions
+      return if @cancelled
       self.class.executor.call(@actor, *arguments) unless self.class.executor.nil?
+      @executed = true
+      run_after_actions
+    end
+
+    # Cancel an action. This method can be called in a before_action hook to
+    # prevent subsequent hooks and the action itself from being executed.
+    # Cancelling an action in an after_action hook has no effect.
+    #
+    def cancel
+      # @todo Emit a warning for attempts to cancel an action after it's been
+      #   executed
+      @cancelled = true
     end
 
     # True if the #execute method has been called for this action.
@@ -27,6 +47,10 @@ module Gamefic
       @executed
     end
 
+    def cancelled?
+      !@executed && @cancelled
+    end
+
     # The verb associated with this action.
     #
     # @return [Symbol] The symbol representing the verb
@@ -68,6 +92,29 @@ module Gamefic
       act
     end
 
+    private
+
+    def run_before_actions
+      return unless @with_callbacks
+      @actor.playbooks
+            .flat_map(&:before_actions)
+            .each do |hook|
+              next unless hook.verb.nil? || hook.verb == verb
+              hook.block.call(self)
+              break if @cancelled
+            end
+    end
+
+    def run_after_actions
+      return unless @with_callbacks
+      @actor.playbooks
+            .flat_map(&:after_actions)
+            .each do |hook|
+              next unless hook.verb.nil? || hook.verb == verb
+              hook.block.call(self)
+            end
+    end
+
     class << self
       attr_reader :verb
 
@@ -135,7 +182,7 @@ module Gamefic
       # @param action [Gamefic::Entity]
       # @param command [Command]
       # @return [self, nil]
-      def attempt actor, command
+      def attempt actor, command, with_callbacks = false
         return nil if command.verb != verb
         tokens = command.arguments
         result = []
@@ -153,7 +200,7 @@ module Gamefic
             result.push accepted.first
           end
         end
-        new(actor, result)
+        new(actor, result, with_callbacks)
       end
 
       protected

data/lib/gamefic/active.rb

@@ -1,3 +1,7 @@
+# frozen_string_literal: true
+
+require 'set'
+
 module Gamefic
   class NotConclusionError < RuntimeError; end
 
@@ -37,7 +41,7 @@ module Gamefic
     end
 
     def syntaxes
-      playbooks.map(&:syntaxes).flatten
+      playbooks.flat_map(&:syntaxes)
     end
 
     # An array of actions waiting to be performed.
@@ -84,28 +88,18 @@ module Gamefic
     end
 
     # Perform a command.
-    # The command can be specified as a String or a verb with a list of
-    # parameters. Either form should yield the same result, but the
-    # verb/parameter form can yield better performance since it bypasses the
-    # parser.
     #
-    # The command will be executed immediately regardless of the entity's
-    # state.
+    # The command's action will be executed immediately regardless of the
+    # entity's state.
     #
     # @example Send a command as a string
     #   character.perform "take the key"
     #
-    # @example Send a command as a verb with parameters
-    #   character.perform :take, @key
-    #
-    # @todo Modify this method so it only accepts a single command.
-    #   Verbs with parameters should use Active#execute instead.
-    #   It might be necessary to support command splats with a deprecation
-    #   warning until version 3.
-    #
+    # @param command [String, Symbol]
     # @return [Gamefic::Action]
     def perform(*command)
       if command.length > 1
+        STDERR.puts "[WARN] #{caller[0]}: Passing a verb and arguments to #perform is deprecated. Use #execute instead."
         execute command.first, *command[1..-1]
       else
         dispatchers.push Dispatcher.dispatch(self, command.first.to_s)
@@ -118,16 +112,18 @@ module Gamefic
     # This method executes the command exactly as #perform does, except it
     # buffers the resulting output instead of sending it to the user.
     #
-    # @todo Modify this method so it only accepts a single command.
-    #   See Active#perform for more information.
-    #
+    # @param command [String, Symbol]
     # @return [String] The output that resulted from performing the command.
     def quietly(*command)
-      clear_buffer if buffer_stack == 0
-      set_buffer_stack buffer_stack + 1
-      self.perform *command
-      set_buffer_stack buffer_stack - 1
-      buffer
+      if command.length > 1
+        STDERR.puts "#{caller[0]}: Passing a verb and arguments to #quietly is deprecated. Use #execute instead"
+        execute command.first, *command[1..-1], quietly: true
+      else
+        dispatchers.push Dispatcher.dispatch(self, command.first.to_s)
+        result = proceed quietly: true
+        dispatchers.pop
+        result
+      end
     end
 
     # Perform an action.
@@ -141,6 +137,9 @@ module Gamefic
     # @example
     #   character.execute :take, @key
     #
+    # @param verb [Symbol]
+    # @param params [Array]
+    # @params quietly [Boolean]
     # @return [Gamefic::Action]
     def execute(verb, *params, quietly: false)
       dispatchers.push Dispatcher.dispatch_from_params(self, verb, params)
@@ -171,21 +170,15 @@ module Gamefic
     #     end
     #   end
     #
+    # @param quietly [Boolean] If true, return the action's output instead of appending it to #messages
+    # @return [String, nil]
     def proceed quietly: false
       return if dispatchers.empty?
       a = dispatchers.last.next
       return if a.nil?
-      if quietly
-        if buffer_stack == 0
-          @buffer = ""
-        end
-        set_buffer_stack(buffer_stack + 1)
-      end
+      prepare_buffer quietly
       a.execute
-      if quietly
-        set_buffer_stack(buffer_stack - 1)
-        @buffer
-      end
+      flush_buffer quietly
     end
 
     # Immediately start a new scene for the character.
@@ -251,8 +244,8 @@ module Gamefic
     # Track the entity's performance of a scene.
     #
     def entered scene
-      klass = (scene.kind_of?(Gamefic::Scene::Base) ? scene.class : scene)
-      entered_scenes.push klass unless entered_scenes.include?(klass)
+      klass = (scene.is_a?(Gamefic::Scene::Base) ? scene.class : scene)
+      entered_scenes.add klass
     end
 
     # Determine whether the entity has performed the specified scene.
@@ -265,9 +258,25 @@ module Gamefic
 
     private
 
-    # @return [Array<Gamefic::Scene::Base>]
+    def prepare_buffer quietly
+      if quietly
+        if buffer_stack == 0
+          @buffer = ""
+        end
+        set_buffer_stack(buffer_stack + 1)
+      end
+    end
+
+    def flush_buffer quietly
+      if quietly
+        set_buffer_stack(buffer_stack - 1)
+        @buffer
+      end
+    end
+
+    # @return [Set<Gamefic::Scene::Base>]
     def entered_scenes
-      @entered_scenes ||= []
+      @entered_scenes ||= Set.new
     end
 
     def buffer_stack

data/lib/gamefic/core_ext/array.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class Array
   # Get a subset of the array that matches the arguments.
   # If the argument is a Class or Module, the elements must be of the type.
@@ -12,7 +14,7 @@ class Array
   #
   # @return [Array]
   def that_are(*cls)
-    result = self.clone
+    result = clone
     cls.each do |c|
       _keep result, c, true
     end
@@ -24,7 +26,7 @@ class Array
   #
   # @return [Array]
   def that_are_not(*cls)
-    result = self.clone
+    result = clone
     cls.each do |c|
       _keep result, c, false
     end
@@ -34,7 +36,7 @@ class Array
   # Pop a random element from the array.
   #
   def pop_sample
-    delete_at(rand(self.length))
+    delete_at(rand(length))
   end
 
   # Get a string representation of the array that separates elements with
@@ -48,31 +50,32 @@ class Array
   # @param andSep [String] The separator for the last element
   # @param serial [Boolean] Use serial separators (e.g., serial commas)
   # @return [String]
-  def join_and(sep = ', ', andSep = ' and ', serial = true)
-    if self.length < 3
-      self.join(andSep)
+  def join_and(sep = ', ', and_sep = ' and ', serial = true)
+    if length < 3
+      join(and_sep)
     else
       start = self[0..-2]
-      start.join(sep) + "#{serial ? sep.strip : ''}#{andSep}#{self.last}"
+      start.join(sep) + "#{serial ? sep.strip : ''}#{and_sep}#{last}"
     end
   end
 
   # @see Array#join_and
   #
   # @return [String]
-  def join_or(sep = ', ', orSep = ' or ', serial = true)
-    join_and(sep, orSep, serial)
+  def join_or(sep = ', ', or_sep = ' or ', serial = true)
+    join_and(sep, or_sep, serial)
   end
 
   private
 
   def _keep(arr, cls, bool)
-    if (cls.kind_of?(Class) or cls.kind_of?(Module))
+    case cls
+    when Class, Module
       arr.keep_if { |i| i.is_a?(cls) == bool }
-    elsif cls.kind_of?(Symbol)
+    when Symbol
       arr.keep_if { |i| i.send(cls) == bool }
     else
-      arr.keep_if {|i| (i == cls) == bool}
+      arr.keep_if { |i| (i == cls) == bool }
     end
   end
 end

data/lib/gamefic/dispatcher.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Gamefic
   # The action selector for character commands.
   #
@@ -9,25 +11,27 @@ module Gamefic
       @actor = actor
       @commands = commands
       @actions = actions
+      @started = false
     end
 
+    # @param dispatcher [Dispatcher]
+    # @return [void]
     def merge dispatcher
       commands.concat dispatcher.commands
       actions.concat dispatcher.actions
     end
 
+    # Get the next executable action.
+    #
+    # @return [Action]
     def next
       instance = nil
       while instance.nil? && !@actions.empty?
         action = actions.shift
         commands.each do |cmd|
-          instance = action.attempt(actor, cmd)
+          instance = action.attempt(actor, cmd, !@started)
           if instance
-            unless instance.meta?
-              actor.playbooks.reverse.each do |playbook|
-                return nil unless validate_playbook(playbook, instance)
-              end
-            end
+            @started = true
             break
           end
         end
@@ -66,11 +70,5 @@ module Gamefic
 
     # @return [Array<Action>]
     attr_reader :actions
-
-    private
-
-    def validate_playbook playbook, action
-      playbook.validators.all? { |v| v.call(actor, action.verb, action.parameters) != false }
-    end
   end
 end

data/lib/gamefic/plot.rb

@@ -34,6 +34,10 @@ module Gamefic
       @static = [self] + scene_classes + entities
     end
 
+    def plot
+      self
+    end
+
     # Get an Array of the Plot's current Syntaxes.
     #
     # @return [Array<Syntax>]

data/lib/gamefic/query/base.rb

@@ -1,7 +1,7 @@
 module Gamefic
   module Query
     class Base
-      NEST_REGEXP = / in | on | of | from | inside /
+      NEST_REGEXP = / in | on | of | from | inside | from inside /
 
       attr_reader :arguments
 
@@ -31,6 +31,9 @@ module Gamefic
       # Get a collection of objects that exist in the subject's context and
       # match the provided token. The result is provided as a Matches object.
       #
+      # @param subject [Entity]
+      # @param token [String]
+      # @param continued [Boolean]
       # @return [Gamefic::Query::Matches]
       def resolve(subject, token, continued: false)
         available = context_from(subject)
@@ -38,14 +41,13 @@ module Gamefic
         if continued
           return Matches.execute(available, token, continued: continued)
         elsif nested?(token)
-          drill = denest(available, token)
-          drill.keep_if{ |e| accept?(e) }
+          drill = denest(available, token).select { |e| accept?(e) }
           return Matches.new(drill, token, '') unless drill.length != 1
           return Matches.new([], '', token)
         end
-        result = available.select{ |e| e.specified?(token) }
-        result = available.select{ |e| e.specified?(token, fuzzy: true) } if result.empty?
-        result.keep_if{ |e| accept? e }
+        result = available.select { |e| e.specified?(token) }
+        result = available.select { |e| e.specified?(token, fuzzy: true) } if result.empty?
+        result.keep_if { |e| accept? e }
         Matches.new(result, (result.empty? ? '' : token), (result.empty? ? token : ''))
       end
 
@@ -82,6 +84,7 @@ module Gamefic
 
       # Determine whether the specified entity passes the query's arguments.
       #
+      # @param [Entity]
       # @return [Boolean]
       def accept?(entity)
         result = true
@@ -106,6 +109,7 @@ module Gamefic
       # recursively append its children.
       # The result will NOT include the original entity itself.
       #
+      # @param [Entity]
       # @return [Array<Object>]
       def subquery_accessible entity
         return [] if entity.nil?

data/lib/gamefic/query/external.rb

@@ -1,17 +1,38 @@
 module Gamefic
   module Query
     class External < Base
-      def initialize objects, *args
+      # @param container [Plot, Subplot, Array]
+      def initialize container, *args
         super(*args)
-        @objects = objects
+        @container = container
       end
 
       def context_from subject
-        @objects
+        Set.new
+           .merge(container_entities)
+           .merge(container_subplots_for(@container, subject))
+           .to_a
       end
 
-      def accept?(entity)
-        @objects.include?(entity) && super(entity)
+      private
+
+      # @return [Array<Entity>]
+      def container_entities
+        if @container.is_a?(World::Entities)
+          @container.entities
+        elsif @container.is_a?(Enumerable)
+          @container
+        else
+          raise ArgumentError, "Unable to derive entities from #{@container}"
+        end
+      end
+
+      # @return [Array<Entity>]
+      def container_subplots_for container, subject
+        return [] unless container.is_a?(Plot::Host)
+        container.subplots_featuring(subject).flat_map do |subplot|
+          subplot.entities + container_subplots_for(subplot, subject)
+        end
       end
     end
   end

data/lib/gamefic/scriptable.rb

@@ -76,9 +76,14 @@ Gamefic::Scriptable.module_exec do
       instance = self
       theater ||= Object.new
       theater.instance_exec do
-        define_singleton_method :method_missing do |symbol, *args, **splat, &block|
-          result = instance.public_send :public_send, symbol, *args, **splat, &block
-          result
+        if RUBY_ENGINE == 'opal' || RUBY_VERSION =~ /^2\.[456]\./
+          define_singleton_method :method_missing do |symbol, *args, &block|
+            instance.public_send :public_send, symbol, *args, &block
+          end
+        else
+          define_singleton_method :method_missing do |symbol, *args, **splat, &block|
+            instance.public_send :public_send, symbol, *args, **splat, &block
+          end
         end
       end
       theater.extend Gamefic::Serialize

data/lib/gamefic/subplot.rb

@@ -1,7 +1,10 @@
 require 'gamefic/plot'
 
 module Gamefic
-  class Subplot #< Container
+  # Subplots are disposable plots that run inside a parent plot. They can be
+  # started and concluded at any time during the parent plot's execution.
+  #
+  class Subplot
     include World
     include Scriptable
     include Gamefic::Serialize

data/lib/gamefic/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Gamefic
-  VERSION = '2.2.3'
+  VERSION = '2.4.0'
 end

data/lib/gamefic/world/commands.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'gamefic/action'
 
 module Gamefic
@@ -91,15 +93,29 @@ module Gamefic
       # @param command [Symbol] An imperative verb for the command
       # @param queries [Array<Query::Base>] Filters for the command's tokens
       # @yieldparam [Gamefic::Actor]
-      def meta(command, *queries, &proc)
-        playbook.meta command, *queries, &proc
+      def meta(command, *queries, &block)
+        playbook.meta command, *queries, &block
+      end
+
+      # Add a proc to be evaluated before a character executes an action.
+      # When a verb is specified, the proc will only be evaluated if the
+      # action's verb matches it.
+      #
+      # @param verb [Symbol, nil]
+      # @yieldparam [Gamefic::Action]
+      def before_action verb = nil, &block
+        playbook.before_action verb, &block
       end
+      alias validate before_action
 
-      # Validate an order before a character can execute its command.
+      # Add a proc to be evaluated after a character executes an action.
+      # When a verb is specified, the proc will only be evaluated if the
+      # action's verb matches it.
       #
-      # @yieldparam [Gamefic::Director::Order]
-      def validate &block
-        playbook.validate &block
+      # @param [Symbol, nil]
+      # @yieldparam [Gamefic::Action]
+      def after_action verb = nil, &block
+        playbook.after_action verb, &block
       end
 
       # Create an alternate Syntax for an Action.
@@ -156,7 +172,7 @@ module Gamefic
           elsif q.is_a?(Gamefic::Element) || (q.is_a?(Class) && q <= Gamefic::Element)
             get_default_query.new(q)
           else
-            raise ArgumentError.new("Invalid argument for response: #{q}")
+            raise ArgumentError, "Invalid argument for response: #{q.inspect}"
           end
         end
       end

data/lib/gamefic/world/playbook.rb

@@ -5,24 +5,33 @@ module Gamefic
     # A collection of rules for performing commands.
     #
     class Playbook
+      ActionHook = Struct.new(:verb, :block)
+
       # An array of available syntaxes.
       #
       # @return [Array<Gamefic::Syntax>]
       attr_reader :syntaxes
 
-      # An array of defined validators.
+      # An array of blocks to execute before actions.
+      #
+      # @return [Array<Proc>]
+      attr_reader :before_actions
+
+      # An array of blocks to execute after actions.
       #
       # @return [Array<Proc>]
-      attr_reader :validators
+      attr_reader :after_actions
 
       # @param commands [Hash]
       # @param syntaxes [Array<Syntax>, Set<Syntax>]
-      # @param validators [Array]
-      def initialize commands: {}, syntaxes: [], validators: []
+      # @param before_actions [Array]
+      # @param after_actions [Array]
+      def initialize commands: {}, syntaxes: [], before_actions: [], after_actions: []
         @commands = commands
         @syntax_set = syntaxes.to_set
         sort_syntaxes
-        @validators = validators
+        @before_actions = before_actions
+        @after_actions = after_actions
       end
 
       # An array of available actions.
@@ -39,10 +48,25 @@ module Gamefic
         @commands.keys
       end
 
-      # Add a block that determines whether an action can be executed.
+      # Add a proc to be evaluated before a character executes an action.
+      # When a verb is specified, the proc will only be evaluated if the
+      # action's verb matches it.
+      #
+      # @param verb [Symbol, nil]
+      # @yieldparam [Gamefic::Action]
+      def before_action verb = nil, &block
+        @before_actions.push ActionHook.new(verb, block)
+      end
+      alias validate before_action
+
+      # Add a proc to be evaluated after a character executes an action.
+      # When a verb is specified, the proc will only be evaluated if the
+      # action's verb matches it.
       #
-      def validate &block
-        @validators.push block
+      # @param verb [Symbol, nil]
+      # @yieldparam [Gamefic::Action]
+      def after_action verb = nil, &block
+        @after_actions.push ActionHook.new(verb, block)
       end
 
       # Get an Array of all Actions associated with the specified verb.

data/lib/gamefic/world/scenes.rb

@@ -1,6 +1,9 @@
 module Gamefic
   module World
     module Scenes
+      include Commands
+      include Players
+
       # @return [Class<Gamefic::Scene::Activity>]
       def default_scene
         @default_scene ||= Scene::Activity
@@ -29,13 +32,12 @@ module Gamefic
       # This method is typically called by the Engine that manages game execution.
       #
       # @param [Gamefic::Actor]
+      # @return [void]
       def introduce(player)
         player.playbooks.push playbook unless player.playbooks.include?(playbook)
         player.cue default_scene
         players.push player
-        @introduction.call(player) unless @introduction.nil?
-        # @todo Find a better way to persist player characters
-        # Gamefic::Index.stick
+        @introduction&.call(player)
       end
 
       # Create a multiple-choice scene.
@@ -53,7 +55,7 @@ module Gamefic
       # @yieldparam [Gamefic::Scene::MultipleChoice]
       # @return [Class<Gamefic::Scene::MultipleChoice>]
       def multiple_choice *choices, &block
-        s = Scene::MultipleChoice.subclass do |actor, scene|
+        s = Scene::MultipleChoice.subclass do |_actor, scene|
           scene.options.concat choices
           scene.on_finish &block
         end
@@ -73,12 +75,12 @@ module Gamefic
       #     end
       #   end
       #
-      # @param prompt [String]
+      # @param prompt [String, nil]
       # @yieldparam [Gamefic::Actor]
       # @yieldparam [Gamefic::Scene::YesOrNo]
       # @return [Class<Gamefic::Scene::YesOrNo>]
       def yes_or_no prompt = nil, &block
-        s = Scene::YesOrNo.subclass do |actor, scene|
+        s = Scene::YesOrNo.subclass do |_actor, scene|
           scene.prompt = prompt
           scene.on_finish &block
         end
@@ -98,7 +100,7 @@ module Gamefic
       # @yieldparam [Gamefic::Scene::Base]
       # @return [Class<Gamefic::Scene::Base>]
       def question prompt = 'What is your answer?', &block
-        s = Scene::Base.subclass do |actor, scene|
+        s = Scene::Base.subclass do |_actor, scene|
           scene.prompt = prompt
           scene.on_finish &block
         end
@@ -116,13 +118,13 @@ module Gamefic
       #     actor.prepare default_scene
       #   end
       #
-      # @param prompt [String] The text to display when prompting the user to continue.
+      # @param prompt [String, nil] The text to display when prompting the user to continue
       # @yieldparam [Gamefic::Actor]
       # @return [Class<Gamefic::Scene::Pause>]
       def pause prompt = nil, &block
         s = Scene::Pause.subclass do |actor, scene|
           scene.prompt = prompt unless prompt.nil?
-          block.call(actor, scene) unless block.nil?
+          block&.call(actor, scene)
         end
         scene_classes.push s
         s
@@ -163,7 +165,7 @@ module Gamefic
       #     end
       #   end
       #
-      # @param cls [Class] The class of scene to be instantiated.
+      # @param cls [Class<Scene::Base>] The class of scene to be instantiated.
       # @yieldparam [Gamefic::Actor]
       # @return [Class<Gamefic::Scene::Base>]
       def custom cls = Scene::Base, &block
@@ -208,10 +210,10 @@ module Gamefic
       # @return [Class<Gamefic::Scene::MultipleScene>]
       def multiple_scene map = {}, &block
         s = Scene::MultipleScene.subclass do |actor, scene|
-          map.each_pair { |k, v|
+          map.each_pair do |k, v|
             scene.map k, v
-          }
-          block.call actor, scene unless block.nil?
+          end
+          block&.call actor, scene
         end
         scene_classes.push s
         s

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gamefic
 version: !ruby/object:Gem::Version
-  version: 2.2.3
+  version: 2.4.0
 platform: ruby
 authors:
 - Fred Snyder
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-07-09 00:00:00.000000000 Z
+date: 2023-02-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -70,6 +70,7 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/rspec.yml"
 - ".gitignore"
 - ".rspec"
 - ".rubocop.yml"
@@ -142,7 +143,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.1.0
+      version: 2.3.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="