deps_grapher 1.0.0 → 1.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 31d04a84f91bd795f6c19d70a9de853963cd02d298cadb8cceaf68fae288e10e
-  data.tar.gz: 284e8006b5cd05e76e8446ca6732048267ee7e234d13d3320589c7e2bb79b38c
+  metadata.gz: 7a737245f2c63696cd2a7f5c492ddb9c049e8f8e9a3e51ee9ce897dc4fb8e593
+  data.tar.gz: dbb1e28028e37ef1d5127ce8e576012057b11518c7253c0dce047097aea8ad2d
 SHA512:
-  metadata.gz: 261f74611a3ac886afd45c7ec1487dcd4db6bb83ede88f6aadf70b1f338725b989bf546b6f0476cffb35251791ebc426c57ab5e887e2b129a82fe8511b625566
-  data.tar.gz: 752589ecdbc90718919f0fcdeefe18e75aa0cced56532aa7ed01f2bf220e4cfc26f0f79d0b3051d6d4016f9f5893a049518b3fe955eb77157c4acbffaa22aa7f
+  metadata.gz: 1e8d778b87a6c3e3ce815f2911c47fe7c636c31dc42d7000094e4c2415ab3628377eb235007641386099c07d126489d630bba7785c56318913d613706a0f58b8
+  data.tar.gz: f6f93769c92aaeb8111defca8a6ea1e4781ce711678c2d1e728be33c8fefbb4d6ceec8534078db6937add8942c954225da4867859357d4bc8f65cc7c81042856

data/CHANGELOG.md

@@ -1,3 +1,10 @@
-## [1.0.0] - 2024-03-22
+## [1.0.2] - 2024-04-03
+- Use `prism` gem instead of `parser` gem (parsing process is approximately 30% faster)
+
+## [1.0.1] - 2024-03-26
+- Fixed a cache issue
+- Fixed an issue that `__FILE__` or `__dir__` is nil in the configuration file
+
+## [1.0.0] - 2024-03-25
 
 - Initial release

data/README.md

@@ -5,7 +5,7 @@ A Tool to visualize Ruby class dependencies
 
 ## Description
 
-Deps Grapher is intended for a general understanding of class dependencies through static analysis. It uses `parser` gem.
+Deps Grapher is intended for a general understanding of class dependencies through static analysis. It uses `prism` gem.
 
 Please note the following:
 

data/deps_grapher.gemspec

@@ -31,4 +31,5 @@ Gem::Specification.new do |spec|
 
   spec.add_dependency "activesupport"
   spec.add_dependency "parser"
+  spec.add_dependency "prism"
 end

data/lib/deps_grapher/ast_processor.rb

@@ -40,7 +40,7 @@ module DepsGrapher
 
       log do
         source_buffer = Parser::Source::Buffer.new(@file_path)
-        parser = Parser::CurrentRuby.new
+        parser = Prism::Translation::Parser.new
         process parser.parse(source_buffer.read)
       end
     end

data/lib/deps_grapher/configuration.rb

@@ -12,6 +12,7 @@ module DepsGrapher
     class_attribute :target_path # target class name on graph
     class_attribute :clean, default: false
     class_attribute :logger
+    class_attribute :cache_key
     class_attribute :cache_dir, default: File.expand_path(File.join("..", "..", "tmp", "deps_grapher", "cache"), __dir__)
     class_attribute :cache_ttl, default: 60 * 5 # 5 minutes
     class_attribute :output_dir, default: File.expand_path(File.join("..", "..", "tmp", "deps_grapher", "graph"), __dir__)
@@ -75,11 +76,9 @@ module DepsGrapher
 
       content = File.read file
 
-      cache_key = Digest::MD5.hexdigest(content)
+      self.cache_key = Digest::MD5.hexdigest content
 
-      SourceCache::Registry.with_cache cache_key do
-        DSL.new(self).instance_eval content
-      end
+      DSL.new(self).instance_eval content, file
 
       return unless dump
 

data/lib/deps_grapher/graph.rb

@@ -38,14 +38,14 @@ module DepsGrapher
     private
 
     def dfs(current, target_path_matcher, visited, path, paths)
-      return if visited[current]
-
       visited[current] = true
 
       if target_path_matcher.match?(current.class_name)
         paths << path
       else
         @graph[current]&.each do |node|
+          next if visited[node]
+
           dfs node, target_path_matcher, visited, path + [node], paths
         end
       end

data/lib/deps_grapher/input.rb

@@ -9,32 +9,34 @@ module DepsGrapher
     def files
       FileUtils.rm_rf File.dirname(config.cache_dir) if config.clean
 
-      set_layers_visibility!
-      extract_files_from_layers
-    end
+      layer_visibilities = config.visualizer_options[:layers]
 
-    private
+      files = []
 
-    attr_reader :config
+      SourceCache::Registry.with_cache config.cache_key do |restored|
+        config.layers.each_value do |layer|
+          name = layer.name
+          source = layer.source
 
-    def set_layers_visibility!
-      layer_visibility = config.visualizer_options[:layers]
+          layer.visible = layer_visibilities.include? name
 
-      config.layers.each_value do |layer|
-        layer.visible = layer_visibility.include? layer.name
-      end
-    end
+          SourceCache.register! name, source unless restored
 
-    def extract_files_from_layers
-      config.layers.values.select(&:visible).each_with_object([]) do |layer, files|
-        source = layer.source
+          next unless layer.visible
 
-        source.files.each do |file|
-          next if file == config.path
+          source.files.each do |file|
+            next if file == config.path
 
-          files << file
+            files << file
+          end
         end
       end
+
+      files
     end
+
+    private
+
+    attr_reader :config
   end
 end

data/lib/deps_grapher/layer.rb

@@ -36,8 +36,6 @@ module DepsGrapher
       source.files.each do |file|
         Registry.register file, self
       end
-
-      SourceCache.register! name, source
     end
 
     def source(&block)

data/lib/deps_grapher/source_cache/class_name_extractor.rb

@@ -16,7 +16,7 @@ module DepsGrapher
 
       def extract!(file_path)
         source_buffer = Parser::Source::Buffer.new(file_path)
-        parser = Parser::CurrentRuby.new
+        parser = Prism::Translation::Parser.new
         process parser.parse(source_buffer.read)
       end
 

data/lib/deps_grapher/source_cache/registry.rb

@@ -26,35 +26,30 @@ module DepsGrapher
         end
 
         def with_cache(key)
-          restore_cache! key
-          yield
-          persist_cache! key
-        end
-
-        def persist_cache!(key)
           cache_file = DepsGrapher.cache_file key
-          cache_file.write @registry
+          restore_cache! cache_file
+          yield @cache_restoration
+          persist_cache! cache_file
         end
 
-        def restore_cache!(key)
-          cache_file = DepsGrapher.cache_file key
+        private
+
+        def restore_cache!(cache_file)
           loaded = cache_file.read
 
           unless loaded
-            @restored_cache = false
+            @cache_restoration = false
             return
           end
 
           @registry = loaded
-          @restored_cache = true
+          @cache_restoration = true
         end
 
-        def restored_cache?
-          @restored_cache
+        def persist_cache!(cache_file)
+          cache_file.write @registry
         end
 
-        private
-
         def registry
           @registry ||= {}
         end

data/lib/deps_grapher/source_cache.rb

@@ -21,8 +21,6 @@ module DepsGrapher
     end
 
     def register!
-      return if Registry.restored_cache?
-
       class_name_extractor = ClassNameExtractor.new do |class_name, location|
         @cache_by_const_name[class_name] = location
         @cache_by_location[location] ||= class_name

data/lib/deps_grapher/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module DepsGrapher
-  VERSION = "1.0.0"
+  VERSION = "1.0.2"
 end

data/lib/deps_grapher.rb

@@ -5,7 +5,7 @@ require "active_support"
 require "active_support/core_ext"
 require "json"
 require "fileutils"
-require "parser/current"
+require "prism"
 require "net/https"
 require "logger"
 require "erb"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: deps_grapher
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.2
 platform: ruby
 authors:
 - jk-es335
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-03-25 00:00:00.000000000 Z
+date: 2024-04-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -38,6 +38,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: prism
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: 
 email:
 - dev@hacomono.co.jp