bullet_train-super_scaffolding 1.6.16 → 1.6.17

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 756d0f0f80e99c6dae67cd43a18cb2c759b0aa8255d4671f25b3cdb415a3c318
-  data.tar.gz: 764cb6047ffcda07aebd78cc05e525ae4e658ff5fc866c428f9a3962308326eb
+  metadata.gz: ab9f39b31aacb0b95619b04fb4b3a32fef367608f8eb4115588dbaf596711c60
+  data.tar.gz: a1be9b35f77dfc9871e3909c653d59806519fcc08de8b6a9904a1721348b5b99
 SHA512:
-  metadata.gz: 56deea3c0cfb883f813acec3d63f92c6b56e9f51c539f03d93b177a1eff67b85a9504d1b61086e474689827f03db0f70c9e0185745b1ec206ad0e2b9bf62b82d
-  data.tar.gz: a20d36c4ffca88890982c106c23f31973968296f07da056248331384593bb0d64160527ec3fac1ad9150c7710f93a1618d86c1eb862006b5afb0f600ea7a7ac8
+  metadata.gz: 5e8cdceb4b5c68cd44d3425fa499f3f132eb51da96c867dd828c6374f8743e6fc3e1b4ccb78cba391fa8d197fd3bf87bcea87d8209976c1f8d7898dcd69f94af
+  data.tar.gz: '099142a8e33347d3c506c40780a65f376295d92ba58231beea2b63fe3a4da71766d44fe894002ecb55e941ebaef769818b56ca2ad07183f35ac17d052f8a99a3'

data/lib/bullet_train/super_scaffolding/version.rb

@@ -1,5 +1,5 @@
 module BulletTrain
   module SuperScaffolding
-    VERSION = "1.6.16"
+    VERSION = "1.6.17"
   end
 end

data/lib/scaffolding/routes_file_manipulator.rb

@@ -52,19 +52,28 @@ class Scaffolding::RoutesFileManipulator
   end
 
   def find_namespaces(namespaces, within = nil)
-    namespaces = namespaces.dup
     results = {}
-    block_end = Scaffolding::BlockManipulator.find_block_end(starting_from: within, lines: lines) if within
-    lines.each_with_index do |line, line_number|
-      if within
-        next unless line_number > within
-        return results if line_number >= block_end
-      end
-      if line.include?("namespace :#{namespaces.first} do")
-        results[namespaces.shift] = line_number
+    reinstantiate_masamune_object
+    namespace_nodes = @msmn.method_calls(token_value: "namespace")
+    namespace_nodes.map do |node|
+      # Get the first argument (a Symbol) without the colon from the list of arguments.
+      name = node.arguments.child_nodes.first.unescaped
+
+      # TODO: For some reason we use both strings and symbols for namespaces.
+      # i.e. - ["account"] and [:v1].
+      if namespaces.include?(name.to_sym)
+        results[name.to_sym] = node.line_number - 1
+      elsif namespaces.include?(name)
+        results[name] = node.line_number - 1
       end
-      return results unless namespaces.any?
     end
+
+    # `within` uses an Array index whereas Masamune nodes use the actual line number, so we write `within + 1` here.
+    if within
+      block_end = @msmn.method_calls.find { |node| node.line_number == within + 1 }.location.end_line
+      results.reject! { |name, line_number| line_number >= block_end }
+    end
+
     results
   end
 
@@ -418,4 +427,10 @@ class Scaffolding::RoutesFileManipulator
       lines[line_number].gsub!(/resources :(.*)$/, "resources :\\1, concerns: [#{existing_concerns.map { |e| ":#{e}" }.join(", ")}]")
     end
   end
+
+  # We have to do this because the `lines` object is constantly changing,
+  # so we reinstantiate this object wherever necessary.
+  def reinstantiate_masamune_object
+    @msmn = Masamune::AbstractSyntaxTree.new(lines.join)
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bullet_train-super_scaffolding
 version: !ruby/object:Gem::Version
-  version: 1.6.16
+  version: 1.6.17
 platform: ruby
 authors:
 - Andrew Culver
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-11-17 00:00:00.000000000 Z
+date: 2023-11-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: standard