hind 0.1.4 → 0.1.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 89ca540ebc5d2ae79689ed9eb4c4ad9a89cea5b66991adecb4de8262f8d50867
-  data.tar.gz: c1d5b39a3a772b1c4079376491361f67cb0dc2853a484e43ceca18b370798f1b
+  metadata.gz: 5fa041a7ffe7e916e1f3715500d21a9dea2a3a6ce60990be8cf4e7578f704702
+  data.tar.gz: '0779f87c520ddc1edddf0448c96d596194982d5b87fa5c380bb3a1132500bd9e'
 SHA512:
-  metadata.gz: cbb72f614d4355c2aa7f4d96b6c5f9babbd61565c7aa6e571c1f33096b5ed8154bda34cb20c9a55523bf5b9a628ec79f7901d0a04d71ceef992cd9930bd62f19
-  data.tar.gz: 3e2288b97ef3f4028060ff754b069bf5aa457a8429edce1e9177c75307d605a872b87eced63184cbda4578e17ffc5b39c2529ac0317d968e0a75d83e364a7b43
+  metadata.gz: 397847759b6c959511602d276eee22f7e89959c5858d840cf8398b874a39ef6ccfeec8d876e9c0539d5cc0bc0cce77cb1e297b7fabc30c9a4562bfb7e85ddb7b
+  data.tar.gz: 5c823067473c8b784a4868bf696b4374735b548eb8eb9341fb887dc78028e9223aeaac7172c621c624d0e8a3de221b50e101e908177626f9c316797e8ce2de70

data/exe/hind

@@ -1,5 +1,7 @@
 #!/usr/bin/env ruby
-require "hind"
-require "hind/cli"
+# frozen_string_literal: true
+
+require 'hind'
+require 'hind/cli'
 
 Hind::CLI.start(ARGV)

data/lib/hind/cli.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'thor'
 require 'json'
 require 'pathname'
@@ -26,7 +28,7 @@ module Hind
       begin
         generate_lsif(files, options)
         say "\nLSIF data has been written to: #{options[:output]}", :green if options[:verbose]
-      rescue StandardError => e
+      rescue => e
         abort "Error generating LSIF: #{e.message}"
       end
     end
@@ -49,7 +51,7 @@ module Hind
       begin
         generate_scip(files, options)
         say "\nSCIP data has been written to: #{options[:output]}", :green if options[:verbose]
-      rescue StandardError => e
+      rescue => e
         abort "Error generating SCIP: #{e.message}"
       end
     end
@@ -66,9 +68,7 @@ module Hind
     end
 
     def validate_output_file(output, force)
-      if File.exist?(output) && !force
-        abort "Error: Output file '#{output}' already exists. Use --force to overwrite."
-      end
+      abort "Error: Output file '#{output}' already exists. Use --force to overwrite." if File.exist?(output) && !force
 
       # Ensure output directory exists
       FileUtils.mkdir_p(File.dirname(output))
@@ -78,10 +78,8 @@ module Hind
       pattern = File.join(directory, glob)
       files = Dir.glob(pattern)
 
-      if exclude_patterns
-        exclude_patterns.each do |exclude|
-          files.reject! { |f| File.fnmatch?(exclude, f) }
-        end
+      exclude_patterns&.each do |exclude|
+        files.reject! { |f| File.fnmatch?(exclude, f) }
       end
 
       files
@@ -94,9 +92,7 @@ module Hind
 
       File.open(options[:output], 'w') do |output_file|
         files.each do |file|
-          if options[:verbose]
-            say "Processing file: #{file}", :cyan
-          end
+          say "Processing file: #{file}", :cyan if options[:verbose]
 
           relative_path = Pathname.new(file).relative_path_from(Pathname.new(options[:directory])).to_s
 
@@ -115,7 +111,7 @@ module Hind
             vertex_id = output.last[:id].to_i + 1
             output_file.puts(output.map(&:to_json).join("\n"))
             initial = false
-          rescue StandardError => e
+          rescue => e
             warn "Warning: Failed to process file '#{file}': #{e.message}"
             next
           end
@@ -124,7 +120,7 @@ module Hind
     end
 
     def generate_scip(files, options)
-      raise NotImplementedError, "SCIP generation not yet implemented"
+      raise NotImplementedError, 'SCIP generation not yet implemented'
       # Similar to generate_lsif but using SCIP generator
     end
 
@@ -133,7 +129,7 @@ module Hind
 
       begin
         YAML.load_file(config_path) || {}
-      rescue StandardError => e
+      rescue => e
         abort "Error loading config file: #{e.message}"
       end
     end

data/lib/hind/lsif/edge.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Hind
   module LSIF
     class Edge
@@ -12,7 +14,7 @@ module Hind
         @document = document
       end
 
-      def to_json
+      def to_json(*_args)
         json = {
           id: @id,
           type: 'edge',

data/lib/hind/lsif/generator.rb

@@ -91,7 +91,7 @@ module Hind
           }
         })
 
-        @global_state.project_id = emit_vertex('project', { kind: 'ruby' })
+        @global_state.project_id = emit_vertex('project', {kind: 'ruby'})
       end
 
       def setup_document
@@ -110,9 +110,9 @@ module Hind
         file_path = File.join(@metadata[:projectRoot], @metadata[:uri])
         ranges = @global_state.ranges[file_path]
 
-        if ranges&.any?
-          emit_edge('contains', @document_id, ranges)
-        end
+        return unless ranges&.any?
+
+        emit_edge('contains', @document_id, ranges)
       end
 
       def update_cross_file_references
@@ -150,15 +150,16 @@ module Hind
       def valid_in_v?(in_v)
         return false unless in_v
         return in_v.any? if in_v.is_a?(Array)
+
         true
       end
 
       def edge_document(label)
-        label == 'item' ? @document_id : nil
+        (label == 'item') ? @document_id : nil
       end
 
       def path_to_uri(path)
-        normalized_path = path.gsub('\\', '/')
+        normalized_path = path.tr('\\', '/')
         normalized_path = normalized_path.sub(%r{^file://}, '')
         absolute_path = File.expand_path(normalized_path)
         "file://#{absolute_path}"

data/lib/hind/lsif/global_state.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Hind
   module LSIF
     class GlobalState
@@ -18,12 +20,12 @@ module Hind
       end
 
       def add_definition(qualified_name, file_path, range_id)
-        @definitions[qualified_name] = { file: file_path, range_id: range_id }
+        @definitions[qualified_name] = {file: file_path, range_id: range_id}
       end
 
       def add_reference(qualified_name, file_path, range_id)
         @references[qualified_name] ||= []
-        @references[qualified_name] << { file: file_path, range_id: range_id }
+        @references[qualified_name] << {file: file_path, range_id: range_id}
       end
     end
   end

data/lib/hind/lsif/vertex.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Hind
   module LSIF
     class Vertex
@@ -9,7 +11,7 @@ module Hind
         @data = data
       end
 
-      def to_json
+      def to_json(*_args)
         json = {
           id: @id,
           type: 'vertex',

data/lib/hind/lsif/visitor.rb

@@ -1,3 +1,6 @@
+# lib/hind/lsif/visitor.rb
+# frozen_string_literal: true
+
 module Hind
   module LSIF
     class Visitor
@@ -9,7 +12,7 @@ module Hind
       def visit(node)
         return unless node
 
-        method_name = "visit_#{node.class.name.split('::').last.downcase}"
+        method_name = "visit_#{node.class.name.split("::").last.downcase}"
         if respond_to?(method_name)
           send(method_name, node)
         else
@@ -22,6 +25,7 @@ module Hind
       end
 
       def visit_defnode(node)
+        # Handle method definitions
         method_name = node.name.to_s
         qualified_name = current_scope_name.empty? ? method_name : "#{current_scope_name}##{method_name}"
 
@@ -33,7 +37,7 @@ module Hind
         @generator.emit_edge('textDocument/definition', result_set_id, def_result_id)
         @generator.emit_edge('item', def_result_id, [range_id], 'definitions')
 
-        # Add hover information
+        # Generate method signature for hover
         sig = []
         sig << "def #{qualified_name}"
         sig << "(#{node.parameters.slice})" if node.parameters
@@ -51,13 +55,214 @@ module Hind
         visit_children(node)
       end
 
-      # Additional visitor methods...
+      def visit_classnode(node)
+        @current_scope.push(node.constant_path.slice)
+        class_name = current_scope_name
+
+        range_id = @generator.create_range(node.location, node.location)
+        result_set_id = @generator.emit_vertex('resultSet')
+        @generator.emit_edge('next', range_id, result_set_id)
+
+        def_result_id = @generator.emit_vertex('definitionResult')
+        @generator.emit_edge('textDocument/definition', result_set_id, def_result_id)
+        @generator.emit_edge('item', def_result_id, [range_id], 'definitions')
+
+        # Generate hover with inheritance info
+        hover = []
+        class_def = "class #{class_name}"
+        class_def += " < #{node.superclass.slice}" if node.superclass
+
+        hover << if node.superclass
+          "#{class_def}\n\nInherits from: #{node.superclass.slice}"
+        else
+          class_def
+        end
+
+        hover_id = @generator.emit_vertex('hoverResult', {
+          contents: [{
+            language: 'ruby',
+            value: hover.join("\n")
+          }]
+        })
+        @generator.emit_edge('textDocument/hover', result_set_id, hover_id)
+
+        @generator.add_to_global_state(class_name, result_set_id, range_id)
+
+        # Handle inheritance
+        visit_inheritance(node.superclass) if node.superclass
+
+        visit_children(node)
+        @current_scope.pop
+      end
+
+      def visit_modulenode(node)
+        @current_scope.push(node.constant_path.slice)
+        module_name = current_scope_name
+
+        range_id = @generator.create_range(node.location, node.location)
+        result_set_id = @generator.emit_vertex('resultSet')
+        @generator.emit_edge('next', range_id, result_set_id)
+
+        def_result_id = @generator.emit_vertex('definitionResult')
+        @generator.emit_edge('textDocument/definition', result_set_id, def_result_id)
+        @generator.emit_edge('item', def_result_id, [range_id], 'definitions')
+
+        hover_id = @generator.emit_vertex('hoverResult', {
+          contents: [{
+            language: 'ruby',
+            value: "module #{module_name}"
+          }]
+        })
+        @generator.emit_edge('textDocument/hover', result_set_id, hover_id)
+
+        @generator.add_to_global_state(module_name, result_set_id, range_id)
+
+        visit_children(node)
+        @current_scope.pop
+      end
+
+      def visit_callnode(node)
+        return unless node.name && node.location
+
+        method_name = node.name.to_s
+        qualified_names = []
+
+        # Try with current scope first
+        qualified_names << "#{current_scope_name}##{method_name}" unless current_scope_name.empty?
+
+        # Try with receiver's type if available
+        if node.receiver
+          case node.receiver
+          when Prism::ConstantReadNode
+            qualified_names << "#{node.receiver.name}##{method_name}"
+          when Prism::CallNode
+            # Handle method chaining
+            qualified_names << "#{node.receiver.name}##{method_name}" if node.receiver.name
+          when Prism::InstanceVariableReadNode
+            # Handle instance variable calls
+            qualified_names << "#{current_scope_name}##{method_name}" if current_scope_name
+          end
+        end
+
+        # Try as a standalone method
+        qualified_names << method_name
+
+        # Add references for matching qualified names
+        qualified_names.each do |qualified_name|
+          next unless @generator.global_state.result_sets[qualified_name]
+
+          range_id = @generator.create_range(node.location, node.location)
+          @generator.global_state.add_reference(qualified_name, @generator.metadata[:uri], range_id)
+          @generator.emit_edge('next', range_id, @generator.global_state.result_sets[qualified_name])
+          break # Stop after finding first match
+        end
+
+        visit_children(node)
+      end
+
+      def visit_constantreadnode(node)
+        return unless node.name
+
+        constant_name = node.name.to_s
+        qualified_name = @current_scope.empty? ? constant_name : "#{current_scope_name}::#{constant_name}"
+
+        return unless @generator.global_state.result_sets[qualified_name]
+
+        range_id = @generator.create_range(node.location, node.location)
+        @generator.global_state.add_reference(qualified_name, @generator.metadata[:uri], range_id)
+        @generator.emit_edge('next', range_id, @generator.global_state.result_sets[qualified_name])
+      end
+
+      def visit_constantwritenode(node)
+        return unless node.name
+
+        constant_name = node.name.to_s
+        qualified_name = @current_scope.empty? ? constant_name : "#{current_scope_name}::#{constant_name}"
+
+        range_id = @generator.create_range(node.location, node.location)
+        result_set_id = @generator.emit_vertex('resultSet')
+        @generator.emit_edge('next', range_id, result_set_id)
+
+        def_result_id = @generator.emit_vertex('definitionResult')
+        @generator.emit_edge('textDocument/definition', result_set_id, def_result_id)
+        @generator.emit_edge('item', def_result_id, [range_id], 'definitions')
+
+        hover_id = @generator.emit_vertex('hoverResult', {
+          contents: [{
+            language: 'ruby',
+            value: "#{qualified_name} = ..."
+          }]
+        })
+        @generator.emit_edge('textDocument/hover', result_set_id, hover_id)
+
+        @generator.add_to_global_state(qualified_name, result_set_id, range_id)
+
+        visit_children(node)
+      end
+
+      def visit_instancevariablereadnode(node)
+        return unless node.name && current_scope_name
+
+        var_name = node.name.to_s
+        qualified_name = "#{current_scope_name}##{var_name}"
+
+        return unless @generator.global_state.result_sets[qualified_name]
+
+        range_id = @generator.create_range(node.location, node.location)
+        @generator.global_state.add_reference(qualified_name, @generator.metadata[:uri], range_id)
+        @generator.emit_edge('next', range_id, @generator.global_state.result_sets[qualified_name])
+      end
+
+      def visit_instancevariablewritenode(node)
+        return unless node.name && current_scope_name
+
+        var_name = node.name.to_s
+        qualified_name = "#{current_scope_name}##{var_name}"
+
+        range_id = @generator.create_range(node.location, node.location)
+        result_set_id = @generator.emit_vertex('resultSet')
+        @generator.emit_edge('next', range_id, result_set_id)
+
+        def_result_id = @generator.emit_vertex('definitionResult')
+        @generator.emit_edge('textDocument/definition', result_set_id, def_result_id)
+        @generator.emit_edge('item', def_result_id, [range_id], 'definitions')
+
+        hover_id = @generator.emit_vertex('hoverResult', {
+          contents: [{
+            language: 'ruby',
+            value: "Instance variable #{var_name} in #{current_scope_name}"
+          }]
+        })
+        @generator.emit_edge('textDocument/hover', result_set_id, hover_id)
+
+        @generator.add_to_global_state(qualified_name, result_set_id, range_id)
+
+        visit_children(node)
+      end
 
       private
 
       def current_scope_name
         @current_scope.join('::')
       end
+
+      def visit_inheritance(node)
+        case node
+        when Prism::ConstantReadNode, Prism::ConstantPathNode
+          range_id = @generator.create_range(node.location, node.location)
+          qualified_name = case node
+          when Prism::ConstantReadNode
+            node.name.to_s
+          when Prism::ConstantPathNode
+            node.slice
+          end
+
+          return unless @generator.global_state.result_sets[qualified_name]
+
+          @generator.global_state.add_reference(qualified_name, @generator.metadata[:uri], range_id)
+          @generator.emit_edge('next', range_id, @generator.global_state.result_sets[qualified_name])
+        end
+      end
     end
   end
 end

data/lib/hind/lsif.rb

@@ -1,9 +1,10 @@
-require_relative "lsif/global_state"
-require_relative "lsif/edge"
-require_relative "lsif/generator"
-require_relative "lsif/vertex"
-require_relative "lsif/visitor"
+# frozen_string_literal: true
 
+require_relative 'lsif/global_state'
+require_relative 'lsif/edge'
+require_relative 'lsif/generator'
+require_relative 'lsif/vertex'
+require_relative 'lsif/visitor'
 
 module Hind
   module LSIF

data/lib/hind/parser.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Hind
   class Parser
     def initialize(code)
@@ -7,6 +9,7 @@ module Hind
     def parse
       result = Prism.parse(@code)
       raise "Parse error: #{result.errors}" unless result.success?
+
       result.value
     end
   end

data/lib/hind/scip/generator.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Hind
   module SCIP
     class Generator

data/lib/hind/scip/visitor.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Hind
   module SCIP
     class Visitor

data/lib/hind/scip.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Hind
   module SCIP
   end

data/lib/hind/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Hind
-  VERSION = "0.1.4"
+  VERSION = '0.1.5'
 end

data/lib/hind.rb

@@ -1,9 +1,9 @@
 # frozen_string_literal: true
 
-require_relative "hind/version"
-require_relative "hind/lsif"
-require_relative "hind/scip"
-require_relative "hind/parser"
+require_relative 'hind/version'
+require_relative 'hind/lsif'
+require_relative 'hind/scip'
+require_relative 'hind/parser'
 
 module Hind
   class Error < StandardError; end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hind
 version: !ruby/object:Gem::Version
-  version: 0.1.4
+  version: 0.1.5
 platform: ruby
 authors:
 - Aboobacker MK