lib/nanoc/base/entities/action_sequence.rb in nanoc-4.7.7 vs lib/nanoc/base/entities/action_sequence.rb in nanoc-4.7.8

@@ -1,64 +1,47 @@
 module Nanoc::Int
   class ActionSequence
     include Nanoc::Int::ContractsSupport
     include Enumerable
+    extend Nanoc::Int::Memoization
 
     attr_reader :item_rep
     attr_reader :actions
 
     def initialize(item_rep, actions: [])
       @item_rep = item_rep
       @actions = actions
     end
 
+    def self.build(rep)
+      builder = Nanoc::Int::ActionSequenceBuilder.new(rep)
+      yield(builder)
+      builder.action_sequence
+    end
+
     contract C::None => Numeric
     def size
       @actions.size
     end
 
     contract Numeric => C::Maybe[Nanoc::Int::ProcessingAction]
     def [](idx)
       @actions[idx]
     end
 
-    contract Symbol, Hash => self
-    def add_filter(filter_name, params)
-      @actions << Nanoc::Int::ProcessingActions::Filter.new(filter_name, params)
-      self
-    end
-
-    contract String, C::Maybe[Hash] => self
-    def add_layout(layout_identifier, params)
-      @actions << Nanoc::Int::ProcessingActions::Layout.new(layout_identifier, params)
-      self
-    end
-
-    contract Symbol, C::Maybe[String] => self
-    def add_snapshot(snapshot_name, path)
-      will_add_snapshot(snapshot_name)
-      @actions << Nanoc::Int::ProcessingActions::Snapshot.new([snapshot_name], path ? [path] : [])
-      self
-    end
-
     contract C::None => C::ArrayOf[Nanoc::Int::ProcessingAction]
     def snapshot_actions
       @actions.select { |a| a.is_a?(Nanoc::Int::ProcessingActions::Snapshot) }
     end
 
-    contract C::None => C::Bool
-    def any_layouts?
-      @actions.any? { |a| a.is_a?(Nanoc::Int::ProcessingActions::Layout) }
-    end
-
     contract C::None => Array
     def paths
       snapshot_actions.map { |a| [a.snapshot_names, a.paths] }
     end
 
     # TODO: Add contract
-    def serialize
+    memoized def serialize
       to_a.map(&:serialize)
     end
 
     contract C::Func[Nanoc::Int::ProcessingAction => C::Any] => self
     def each
@@ -88,19 +71,8 @@
           is_binary = Nanoc::Filter.named!(action.filter_name).to_binary?
         end
       end
 
       snapshot_defs
-    end
-
-    private
-
-    def will_add_snapshot(name)
-      @_snapshot_names ||= Set.new
-      if @_snapshot_names.include?(name)
-        raise Nanoc::Int::Errors::CannotCreateMultipleSnapshotsWithSameName.new(@item_rep, name)
-      else
-        @_snapshot_names << name
-      end
     end
   end
 end