Allow dependency cycles

lib/librarian/puppet/action/install.rb

@@ -20,6 +20,10 @@ def check_specfile
           # don't fail if Puppetfile doesn't exist as we'll use the Modulefile or metadata.json
         end
 
+        def sorted_manifests
+          lock.manifests
+        end
+
       end
     end
   end

lib/librarian/puppet/action/resolve.rb

@@ -1,4 +1,5 @@
 require 'librarian/action/resolve'
+require 'librarian/puppet/resolver'
 
 module Librarian
   module Puppet
@@ -15,6 +16,10 @@ def run
           end
         end
 
+        def resolver
+          Resolver.new(environment, cyclic: true)
+        end
+
       end
     end
   end

lib/librarian/puppet/lockfile.rb

@@ -28,6 +28,20 @@ class << manifests_index
           super(lines, manifests_index)
         end
 
+        def compile(sources_ast)
+          manifests = compile_placeholder_manifests(sources_ast)
+          manifests = manifests.map do |name, manifest|
+            dependencies = manifest.dependencies.map do |d|
+              environment.dsl_class.dependency_type.new(d.name, d.requirement, manifests[d.name].source)
+            end
+            real = Manifest.new(manifest.source, manifest.name)
+            real.version = manifest.version
+            real.dependencies = manifest.dependencies
+            real
+          end
+          manifests.sort_by(&:name)
+        end
+
       end
 
       def load(string)

lib/librarian/puppet/resolver.rb

@@ -0,0 +1,14 @@
+require 'librarian/resolver'
+
+module Librarian
+  module Puppet
+    class Resolver < Librarian::Resolver
+
+      def sort(manifests)
+        manifests = manifests.values if Hash === manifests
+        manifests.sort_by(&:name)
+      end
+
+    end
+  end
+end