diver_down 0.0.1.alpha7 → 0.0.1.alpha9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3749e5acb9fae2d27b5267f30897c328b2a121fa1b25f89ab2dd0263af685492
-  data.tar.gz: 841226072e1c60faaacabcadabcf66c94bde318d7479d8090c064128be5556fe
+  metadata.gz: b7a272e39e974d543ceec5219e3b5505bca70d0a61afae0632107eb3cc5645fe
+  data.tar.gz: c6df48a9145a3d87c6b8816b6a37830bdbab3c6404d0d0347cb43bf3be636ec1
 SHA512:
-  metadata.gz: 80753b14f48811fa05d84b75f303d17335701f82fc04ead701c8e8b53473f0d9860bf7e78a25201e0c9bc9c58af89764350f023c3ea4310c7b3cb6c6b0ab93a9
-  data.tar.gz: 5ec6ec0814d825864fb062ec0db92f3f7d6df8949795ac5c4f6299b302219e5f50f6ba9f8f8590b8ac123c268f8a04ac268a95e0af6638084ff13b62859d603a
+  metadata.gz: 5a1d631fe52cab90a2c005f96480751007056414accfb50d3aaa59f337614c69cd5082ed77b456f68b4fce5f4ff17404296df2e84f82159b4abdcdf354d2b996
+  data.tar.gz: 3144ca2c364a66e8d20db470887f6d7499d05ffdb39a379b85d4aac093808971963baa76340863dc93f2e2480af4d2dba035a2874c2926f23bd015cff5b39f59

data/lib/diver_down/trace/session.rb

@@ -91,13 +91,7 @@ module DiverDown
 
               # Search is a heavy process and should be terminated early.
               # The position of the most recently found caller or the start of trace is used as the maximum value.
-              maximum_back_stack_size = if call_stack.empty_context_stack?
-                                          call_stack.stack_size
-                                        else
-                                          call_stack.stack_size - call_stack.context_stack_size[-1]
-                                        end
-
-              caller_location = find_neast_caller_location(maximum_back_stack_size)
+              caller_location = find_neast_caller_location(call_stack.stack_size)
 
               # `caller_location` is nil if it is filtered by caller_paths
               if caller_location

data/lib/diver_down/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module DiverDown
-  VERSION = '0.0.1.alpha7'
+  VERSION = '0.0.1.alpha9'
 end

data/lib/diver_down/web/action.rb

@@ -33,12 +33,19 @@ module DiverDown
         end
         # rubocop:enable Style/HashEachMethods
 
+        classified_sources_count = source_names.count { @module_store.classified?(_1) }
+
         json(
           sources: source_names.sort.map do |source_name|
             {
               source_name:,
+              memo: @module_store.get_memo(source_name),
+              modules: @module_store.get_modules(source_name).map do |module_name|
+                { module_name: }
+              end,
             }
-          end
+          end,
+          classified_sources_count:
         )
       end
 
@@ -50,7 +57,7 @@ module DiverDown
         # rubocop:disable Style/HashEachMethods
         @store.each do |_, definition|
           definition.sources.each do |source|
-            modules = @module_store.get(source.source_name)
+            modules = @module_store.get_modules(source.source_name)
             module_set.add(modules) unless modules.empty?
           end
         end
@@ -77,7 +84,7 @@ module DiverDown
         # rubocop:disable Style/HashEachMethods
         @store.each do |_, definition|
           definition.sources.each do |source|
-            source_module_names = @module_store.get(source.source_name)
+            source_module_names = @module_store.get_modules(source.source_name)
 
             next unless source_module_names[0..module_names.size - 1] == module_names
 
@@ -102,6 +109,7 @@ module DiverDown
           sources: source_names.sort.map do |source_name|
             {
               source_name:,
+              memo: @module_store.get_memo(source_name),
             }
           end,
           related_definitions: related_definitions.map do |definition|
@@ -131,6 +139,7 @@ module DiverDown
               definition_group: definition.definition_group,
               title: definition.title,
               sources_count: definition.sources.size,
+              unclassified_sources_count: definition.sources.reject { @module_store.classified?(_1.source_name) }.size,
             }
           end,
           pagination: pagination.to_h
@@ -188,7 +197,8 @@ module DiverDown
             sources: definition.sources.map do
               {
                 source_name: _1.source_name,
-                modules: @module_store.get(_1.source_name).map do |module_name|
+                memo: @module_store.get_memo(_1.source_name),
+                modules: @module_store.get_modules(_1.source_name).map do |module_name|
                   { module_name: }
                 end,
               }
@@ -239,11 +249,12 @@ module DiverDown
                          []
                        else
                          source = DiverDown::Definition::Source.combine(*found_sources)
-                         @module_store.get(source.source_name)
+                         @module_store.get_modules(source.source_name)
                        end
 
         json(
           source_name:,
+          memo: @module_store.get_memo(source_name),
           modules: module_names.map do
             { module_name: _1 }
           end,
@@ -280,7 +291,28 @@ module DiverDown
         end
 
         if found_source
-          @module_store.set(source_name, modules)
+          @module_store.set_modules(source_name, modules)
+          @module_store.flush
+
+          json({})
+        else
+          not_found
+        end
+      end
+
+      # POST /api/sources/:source_name/memo.json
+      #
+      # @param source_name [String]
+      # @param memo [String]
+      def set_memo(source_name, memo)
+        found_source = @store.any? do |_, definition|
+          definition.sources.any? do |source|
+            source.source_name == source_name
+          end
+        end
+
+        if found_source
+          @module_store.set_memo(source_name, memo)
           @module_store.flush
 
           json({})

data/lib/diver_down/web/definition_to_dot.rb

@@ -31,7 +31,7 @@ module DiverDown
           private
 
           def source_to_h
-            modules = module_store.get(data.source_name).map do
+            modules = module_store.get_modules(data.source_name).map do
               {
                 module_name: _1,
               }
@@ -41,6 +41,7 @@ module DiverDown
               id:,
               type: 'source',
               source_name: data.source_name,
+              memo: module_store.get_memo(data.source_name),
               modules:,
             }
           end
@@ -185,11 +186,11 @@ module DiverDown
         dependency_map = Hash.new { |h, k| h[k] = Hash.new { |hi, ki| hi[ki] = [] } }
 
         definition.sources.sort_by(&:source_name).each do |source|
-          source_modules = module_store.get(source.source_name)
+          source_modules = module_store.get_modules(source.source_name)
           next if source_modules.empty?
 
           source.dependencies.each do |dependency|
-            dependency_modules = module_store.get(dependency.source_name)
+            dependency_modules = module_store.get_modules(dependency.source_name)
             next if dependency_modules.empty?
 
             dependency_map[source_modules][dependency_modules].push(dependency)
@@ -268,7 +269,7 @@ module DiverDown
 
       def render_sources
         by_modules = definition.sources.group_by do |source|
-          module_store.get(source.source_name)
+          module_store.get_modules(source.source_name)
         end
 
         # Remove duplicated prefix modules
@@ -326,8 +327,8 @@ module DiverDown
       def insert_dependencies(source)
         source.dependencies.each do
           attributes = {}
-          ltail = module_label(*module_store.get(source.source_name))
-          lhead = module_label(*module_store.get(_1.source_name))
+          ltail = module_label(*module_store.get_modules(source.source_name))
+          lhead = module_label(*module_store.get_modules(_1.source_name))
 
           if @compound && (ltail || lhead)
             # Rendering of dependencies between modules is done only once

data/lib/diver_down/web/module_store.rb

@@ -6,6 +6,7 @@ module DiverDown
   class Web
     class ModuleStore
       BLANK_ARRAY = [].freeze
+      BLANK_MEMO = ''
       BLANK_RE = /\A\s*\z/
 
       private_constant(:BLANK_RE)
@@ -17,16 +18,42 @@ module DiverDown
 
       # @param source_name [String]
       # @param module_names [Array<String>]
-      def set(source_name, module_names)
-        @store[source_name] = module_names.dup.reject do
+      def set_modules(source_name, module_names)
+        source = (@store[source_name] ||= {})
+
+        source[:modules] = module_names.dup.reject do
           BLANK_RE.match?(_1)
         end.freeze
       end
 
       # @param source_name [String]
       # @return [Array<Module>]
-      def get(source_name)
-        @store[source_name] || BLANK_ARRAY
+      def get_modules(source_name)
+        return BLANK_ARRAY unless @store.key?(source_name)
+
+        @store[source_name][:modules] || BLANK_ARRAY
+      end
+
+      # @param source_name [String]
+      # @param memo [String]
+      # @return [void]
+      def set_memo(source_name, memo)
+        source = (@store[source_name] ||= {})
+        source[:memo] = memo.strip
+      end
+
+      # @param source_name [String]
+      # @return [String]
+      def get_memo(source_name)
+        return BLANK_MEMO unless @store.key?(source_name)
+
+        @store[source_name][:memo] || BLANK_MEMO
+      end
+
+      # @param source_name [String]
+      # @return [Boolean]
+      def classified?(source_name)
+        get_modules(source_name).any?
       end
 
       # @return [Hash]

data/lib/diver_down/web.rb

@@ -53,7 +53,7 @@ module DiverDown
       in ['GET', %r{\A/api/modules\.json\z}]
         action.modules
       in ['GET', %r{\A/api/modules/(?<module_names>.+)\.json\z}]
-        module_names = Regexp.last_match[:module_names].split('/')
+        module_names = CGI.unescape(Regexp.last_match[:module_names]).split('/')
         action.module(module_names)
       in ['GET', %r{\A/api/definitions/(?<bit_id>\d+)\.json\z}]
         bit_id = Regexp.last_match[:bit_id].to_i
@@ -68,6 +68,10 @@ module DiverDown
         source = Regexp.last_match[:source]
         modules = request.params['modules'] || []
         action.set_modules(source, modules)
+      in ['POST', %r{\A/api/sources/(?<source>[^/]+)/memo.json\z}]
+        source = Regexp.last_match[:source]
+        memo = request.params['memo'] || ''
+        action.set_memo(source, memo)
       in ['GET', %r{\A/api/pid\.json\z}]
         action.pid
       in ['GET', %r{\A/api/initialization_status\.json\z}]