platformos-check 0.0.3 → 0.2.0

data/lib/platformos_check/file_system_storage.rb

@@ -31,10 +31,13 @@ module PlatformosCheck
 
       file(relative_path).dirname.mkpath unless file(relative_path).dirname.directory?
       file(relative_path).write(content, mode: 'w+b', encoding: 'UTF-8')
+      @platformos_app&.update([relative_path])
     end
 
     def remove(relative_path)
       file(relative_path).delete
+
+      @platformos_app&.update([relative_path], remove: true)
       reset_memoizers
     end
 
@@ -60,12 +63,6 @@ module PlatformosCheck
                       .map { |path| path.relative_path_from(@root).to_s }
     end
 
-    def directories
-      @directories ||= glob('**/')
-                       .select { |f| File.directory?(f) }
-                       .map { |f| f.relative_path_from(@root).to_s }
-    end
-
     private
 
     def file_exists?(relative_path)
@@ -74,7 +71,6 @@ module PlatformosCheck
 
     def reset_memoizers
       @file_array = nil
-      @directories = nil
     end
 
     def glob(pattern)

data/lib/platformos_check/graphql_file.rb

@@ -13,6 +13,12 @@ module PlatformosCheck
       @storage.write(@relative_path, content.gsub("\n", @eol))
       @source = content
       @rewriter = nil
+      @ast = nil
+      @variables = nil
+      @definition = nil
+      @parse = nil
+      @required_arguments = nil
+      @defined_arguments = nil
     end
 
     def rewriter
@@ -48,7 +54,7 @@ module PlatformosCheck
     end
 
     def warnings
-      @ast.warnings
+      parse.warnings
     end
 
     def root
@@ -59,8 +65,26 @@ module PlatformosCheck
       Struct.new(:warnings, :root)
     end
 
+    def required_arguments
+      @required_arguments ||= variables.each_with_object([]) do |v, vars|
+        vars << v.name if v.type.is_a?(GraphQL::Language::Nodes::NonNullType)
+      end
+    end
+
+    def defined_arguments
+      @defined_arguments ||= variables.map(&:name)
+    end
+
     private
 
+    def variables
+      @variables ||= definition&.variables || []
+    end
+
+    def definition
+      @definition ||= parse.definitions.detect { |d| d.is_a?(GraphQL::Language::Nodes::OperationDefinition) }
+    end
+
     def bounded(lower, x, upper)
       [lower, [x, upper].min].max
     end

data/lib/platformos_check/in_memory_storage.rb

@@ -22,47 +22,25 @@ module PlatformosCheck
     end
 
     def write(relative_path, content)
+      @platformos_app&.update([relative_path])
       @files[relative_path] = content
     end
 
     def remove(relative_path)
+      @platformos_app&.update([relative_path], remove: true)
       @files.delete(relative_path)
     end
 
     def mkdir(relative_path)
       @files[relative_path] = nil
-      reset_memoizers
     end
 
-    # TODO: Fix corrector
-    # def rename(old_path, new_path)
-    #   old_path += '/' if old_path[-1] != '/'
-    #   new_path += '/' if new_path[-1] != '/'
-    #   @files.transform_keys! { |k| k.sub(/\A#{old_path}/, new_path) }
-    #
-    #   reset_memoizers
-    # end
-
     def files
       @files.keys
     end
 
-    def directories
-      @directories ||= @files
-                       .keys
-                       .flat_map { |relative_path| Pathname.new(relative_path).ascend.to_a }
-                       .map(&:to_s)
-                       .uniq
-    end
-
     def relative_path(absolute_path)
       Pathname.new(absolute_path).relative_path_from(@root).to_s
     end
-
-    private
-
-    def reset_memoizers
-      @directories = nil
-    end
   end
 end

data/lib/platformos_check/language_server/completion_providers/background_partial_completion_provider.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+module PlatformosCheck
+  module LanguageServer
+    class BackgroundPartialCompletionProvider < CompletionProvider
+      include PartialCompletionProvider
+
+      private
+
+      def files
+        @storage.platformos_app.partials
+      end
+
+      def regexp
+        PARTIAL_BACKGROUND
+      end
+    end
+  end
+end

data/lib/platformos_check/language_server/completion_providers/function_partial_completion_provider.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+module PlatformosCheck
+  module LanguageServer
+    class FunctionPartialCompletionProvider < CompletionProvider
+      include PartialCompletionProvider
+
+      private
+
+      def files
+        @storage.platformos_app.partials
+      end
+
+      def regexp
+        PARTIAL_FUNCTION
+      end
+    end
+  end
+end

data/lib/platformos_check/language_server/completion_providers/graphql_partial_completion_provider.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+module PlatformosCheck
+  module LanguageServer
+    class GraphqlPartialCompletionProvider < CompletionProvider
+      include PartialCompletionProvider
+
+      private
+
+      def files
+        @storage.platformos_app.graphqls
+      end
+
+      def regexp
+        PARTIAL_GRAPHQL
+      end
+    end
+  end
+end

data/lib/platformos_check/language_server/completion_providers/include_partial_completion_provider.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+module PlatformosCheck
+  module LanguageServer
+    class IncludePartialCompletionProvider < CompletionProvider
+      include PartialCompletionProvider
+
+      private
+
+      def files
+        @storage.platformos_app.partials
+      end
+
+      def regexp
+        PARTIAL_INCLUDE
+      end
+    end
+  end
+end

data/lib/platformos_check/language_server/completion_providers/object_completion_provider.rb

@@ -13,6 +13,7 @@ module PlatformosCheck
 
         PlatformosLiquid::SourceIndex
           .objects
+          .select(&:global?)
           .select { |object| object.name.start_with?(partial(variable_lookup)) }
           .map { |object| object_to_completion(object) }
       end

data/lib/platformos_check/language_server/completion_providers/render_partial_completion_provider.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+module PlatformosCheck
+  module LanguageServer
+    class RenderPartialCompletionProvider < CompletionProvider
+      include PartialCompletionProvider
+
+      private
+
+      def files
+        @storage.platformos_app.partials
+      end
+
+      def regexp
+        PARTIAL_RENDER
+      end
+    end
+  end
+end

data/lib/platformos_check/language_server/completion_providers/tag_completion_provider.rb

@@ -5,34 +5,29 @@ module PlatformosCheck
     class TagCompletionProvider < CompletionProvider
       def completions(context)
         content = context.content
-        cursor = context.cursor
 
         return [] if content.nil?
-        return [] unless can_complete?(content, cursor)
+        return [] unless can_complete?(context)
 
-        partial = first_word(content) || ''
-        labels = PlatformosLiquid::Tag.labels
-        labels += PlatformosLiquid::Tag.end_labels
-        labels
-          .select { |w| w.start_with?(partial) }
-          .map { |tag| tag_to_completion(tag) }
+        partial = first_word(context.buffer.lines[context.line]) || ''
+        PlatformosLiquid::SourceIndex.tags.select { |tag| tag.name.start_with?(partial) }
+                                     .map { |tag| tag_to_completion(tag) }
       end
 
-      def can_complete?(content, cursor)
-        content.start_with?(Liquid::TagStart) && (
-          cursor_on_first_word?(content, cursor) ||
-          cursor_on_start_content?(content, cursor, Liquid::TagStart)
-        )
+      def can_complete?(context)
+        context.content.start_with?(Liquid::TagStart) && (cursor_on_first_word?(context.buffer.lines[context.line], context.col) || cursor_on_start_content?(context.buffer.lines[context.line], context.col, Liquid::TagStart))
       end
 
       private
 
       def tag_to_completion(tag)
-        content = PlatformosLiquid::Documentation.tag_doc(tag)
+        content = PlatformosLiquid::Documentation.tag_doc(tag.name)
 
         {
-          label: tag,
+          contents: content,
+          label: tag.name,
           kind: CompletionItemKinds::KEYWORD,
+          **format_hash(tag),
           **doc_hash(content)
         }
       end

data/lib/platformos_check/language_server/constants.rb

@@ -15,7 +15,7 @@ module PlatformosCheck
 
     def self.partial_tag_with_result(tag)
       /
-        \{%-?\s*#{tag}\s+(?<var>[\w]+)+\s*=\s*'(?<partial>[^']*)'|
+        \{%-?\s*#{tag}\s+(?<var>[\w]+)\s*=\s*'(?<partial>[^']*)'|
         \{%-?\s*#{tag}\s+(?<var>[\w]+)\s*=\s*"(?<partial>[^"]*)"|
 
         # in liquid tags the whole line is white space until the tag

data/lib/platformos_check/language_server/diagnostics_engine.rb

@@ -21,13 +21,12 @@ module PlatformosCheck
       def analyze_and_send_offenses(absolute_path_or_paths, config, force: false, only_single_file: false)
         return unless @diagnostics_lock.try_lock
 
-        platformos_app = PlatformosCheck::App.new(storage)
-        analyzer = PlatformosCheck::Analyzer.new(platformos_app, config.enabled_checks)
+        analyzer = PlatformosCheck::Analyzer.new(storage.platformos_app, config.enabled_checks)
 
         if !only_single_file && (@diagnostics_manager.first_run? || force)
           run_full_platformos_check(analyzer)
         else
-          run_partial_platformos_check(absolute_path_or_paths, platformos_app, analyzer, only_single_file)
+          run_partial_platformos_check(absolute_path_or_paths, storage.platformos_app, analyzer, only_single_file)
         end
 
         @diagnostics_lock.unlock

data/lib/platformos_check/language_server/document_link_engine.rb

@@ -12,10 +12,8 @@ module PlatformosCheck
         buffer = @storage.read(relative_path)
         return [] unless buffer
 
-        platformos_app = PlatformosCheck::App.new(@storage)
-
         @providers.flat_map do |p|
-          p.document_links(buffer, platformos_app)
+          p.document_links(buffer, @storage.platformos_app)
         end
       end
     end

data/lib/platformos_check/language_server/handler.rb

@@ -24,7 +24,7 @@ module PlatformosCheck
 
       CAPABILITIES = {
         completionProvider: {
-          triggerCharacters: ['.', '{{ ', '{% '],
+          triggerCharacters: ['.', '{{ ', '{% ', '/'],
           context: true
         },
         codeActionProvider: {

data/lib/platformos_check/language_server/hover_providers/filter_hover_provider.rb

@@ -18,7 +18,10 @@ module PlatformosCheck
         variable_lookup = VariableLookupFinder.lookup(context)
         denied_filters = denied_filters_for(variable_lookup)
         available_filters_for(determine_input_type(variable_lookup))
-          .select { |filter| (filter.name == partial(content, cursor)) && denied_filters.none?(filter.name) }
+          .select do |filter|
+            partial_name = partial(content, cursor)
+            (filter.name == partial_name || filter.aliases.any? { |a| a == partial_name }) && denied_filters.none?(filter.name)
+          end
           .map { |filter| filter_to_completion(filter) }
           .first
       end
@@ -88,23 +91,14 @@ module PlatformosCheck
         partial_match[1]
       end
 
-      def param_to_doc(param)
-        "#{param&.name || 'object'}:#{param&.return_type&.downcase || 'untyped'}"
-      end
-
       def filter_to_completion(filter)
         content = PlatformosLiquid::Documentation.render_doc(filter)
-        first_param, *other_params = filter.parameters
-        other_params = other_params.map { |param| param_to_doc(param) }
-        other_params = other_params.any? ? ": #{other_params.join(', ')}" : ""
-        content += "  \n\n#{param_to_doc(first_param)} | #{filter.name}#{other_params} => #{filter.return_type}"
-
         {
-          contents: content
-          # label: filter.name,
-          # kind: CompletionItemKinds::FUNCTION,
-          # **format_hash(filter),
-          # **doc_hash(content)
+          contents: content,
+          label: filter.name,
+          kind: CompletionItemKinds::FUNCTION,
+          **format_hash(filter),
+          **doc_hash(content)
         }
       end
     end

data/lib/platformos_check/language_server/hover_providers/tag_hover_provider.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+module PlatformosCheck
+  module LanguageServer
+    class TagHoverProvider < HoverProvider
+      def completions(context)
+        content = context.content
+
+        return [] if content.nil?
+        return [] unless can_complete?(context)
+
+        partial = first_word(context.buffer.lines[context.line]) || ''
+        PlatformosLiquid::SourceIndex.tags.select { |tag| tag.name.start_with?(partial) }
+                                     .map { |tag| tag_to_completion(tag) }.first
+      end
+
+      def can_complete?(context)
+        context.content.start_with?(Liquid::TagStart) && (cursor_on_first_word?(context.buffer.lines[context.line], context.col) || cursor_on_start_content?(context.buffer.lines[context.line], context.col, Liquid::TagStart))
+      end
+
+      private
+
+      def tag_to_completion(tag)
+        content = PlatformosLiquid::Documentation.tag_doc(tag.name)
+
+        {
+          contents: content,
+          label: tag.name,
+          kind: CompletionItemKinds::KEYWORD,
+          **format_hash(tag),
+          **doc_hash(content)
+        }
+      end
+    end
+  end
+end

data/lib/platformos_check/language_server/partial_completion_provider.rb

@@ -0,0 +1,71 @@
+# frozen_string_literal: true
+
+module PlatformosCheck
+  module LanguageServer
+    module PartialCompletionProvider
+      def completions(context)
+        content = context.buffer.lines[context.line]
+        cursor = context.col
+        @file_name = nil
+
+        return [] if content.nil?
+        return [] unless cursor_on_quoted_argument?(content, cursor)
+
+        files
+          .select { |x| x.name.start_with?(@file_name) }
+          .map { |x| file_to_completion(x, context) }
+      end
+
+      private
+
+      def cursor_on_quoted_argument?(content, cursor)
+        @match = content.match(regexp)
+        return false if @match.nil?
+
+        return false unless @match.begin(:partial) <= cursor && cursor <= @match.end(:partial)
+
+        @file_name = @match[:partial][0, cursor - @match.begin(:partial)]
+        true
+      end
+
+      def files
+        raise NotImplementedError
+      end
+
+      def regexp
+        raise NotImplementedError
+      end
+
+      def file_to_completion(file, context)
+        {
+          label: file.name,
+          kind: CompletionItemKinds::TEXT,
+          detail: file.source,
+          textEdit: {
+            newText: file.name,
+            insert: {
+              start: {
+                line: context.line,
+                character: @match.begin(:partial)
+              },
+              end: {
+                line: context.line,
+                character: @match.end(:partial)
+              }
+            },
+            replace: {
+              start: {
+                line: context.line,
+                character: @match.begin(:partial)
+              },
+              end: {
+                line: context.line,
+                character: @match.end(:partial)
+              }
+            }
+          }
+        }
+      end
+    end
+  end
+end

data/lib/platformos_check/language_server.rb

@@ -32,6 +32,7 @@ require_relative "language_server/completion_context"
 require_relative "language_server/completion_helper"
 require_relative "language_server/completion_provider"
 require_relative "language_server/completion_engine"
+require_relative "language_server/partial_completion_provider"
 Dir[__dir__ + "/language_server/completion_providers/*.rb"].each do |file|
   require file
 end

data/lib/platformos_check/liquid_visitor.rb

@@ -12,7 +12,7 @@ module PlatformosCheck
     def visit_liquid_file(liquid_file)
       visit(LiquidNode.new(liquid_file.root, nil, liquid_file))
     rescue Liquid::Error => e
-      e.template_name = liquid_file.name
+      e.template_name = liquid_file.relative_path
       call_checks(:on_error, e)
     end
 

data/lib/platformos_check/platformos_liquid/source_index/filter_entry.rb

@@ -1,12 +1,45 @@
 # frozen_string_literal: true
 
+require 'cgi'
+
 module PlatformosCheck
   module PlatformosLiquid
     class SourceIndex
       class FilterEntry < BaseEntry
         def parameters
-          (hash['parameters'] || [])
-            .map { |hash| ParameterEntry.new(hash) }
+          @parameters ||= (hash['parameters'] || []).map { |hash| ParameterEntry.new(hash) }
+        end
+
+        def summary
+          hash['summary']&.strip == 'returns' ? nil : hash['summary']
+        end
+
+        def description
+          @descritpion = begin
+            desc = hash['description']&.strip || ''
+            desc = '' if desc == 'returns'
+            if parameters.any?
+              desc += "Parameters:"
+              parameters.each { |p| desc += "\n- #{p.full_summary}" }
+            end
+            if hash['return_type']&.any?
+              rt = hash['return_type'].first
+              rt['description'] = nil if rt['description']&.strip == ''
+              desc += "\n\nReturns:"
+              desc += "\n- #{[rt['type'], rt['description']].compact.join(': ')}\n"
+            end
+            if hash['examples']
+              desc += "\n\n---\n\n"
+              hash['examples'].each_with_index do |e, i|
+                example = e['raw_liquid'].gsub(/[\n]+/, "\n").strip.split('=>')
+                input = example[0].strip
+                output = example[1]&.strip
+                desc += "\n  - Example #{i}:\n\n```liquid\n#{input}\n```"
+                desc += "\n##\nOutput: #{output}" if output
+              end
+            end
+          end
+          desc
         end
 
         def aliases

data/lib/platformos_check/platformos_liquid/source_index/object_entry.rb

@@ -14,6 +14,10 @@ module PlatformosCheck
         def platformos_documentation_url
           "#{PLATFORMOS_DOCUMENTATION_URL}/developer-guide/variables/context-variable##{hash['name']}"
         end
+
+        def global?
+          hash.dig('access', 'global')
+        end
       end
     end
   end

data/lib/platformos_check/platformos_liquid/source_index/parameter_entry.rb

@@ -12,6 +12,10 @@ module PlatformosCheck
           "#{PLATFORMOS_DOCUMENTATION_URL}/api-reference/liquid/filters/#{hash['name']}"
         end
 
+        def full_summary
+          "#{hash['name']} - #{hash['description']}"
+        end
+
         private
 
         def return_type_hash

data/lib/platformos_check/platformos_liquid/source_index/tag_entry.rb

@@ -15,6 +15,34 @@ module PlatformosCheck
           }
         end
 
+        def description
+          @descritpion = begin
+            desc = hash['description']&.strip || ''
+            desc = '' if desc == 'returns'
+            if parameters.any?
+              desc += "\n\n---\n\nParameters:"
+              parameters.each { |p| desc += "\n- #{p.full_summary}" }
+            end
+            if hash['return_type']&.any?
+              rt = hash['return_type'].first
+              rt['description'] = nil if rt['description']&.strip == ''
+              desc += "\n\nReturns:"
+              desc += "\n- #{[rt['type'], rt['description']].compact.join(': ')}\n"
+            end
+            if hash['examples']
+              desc += "\n\n---\n\n"
+              hash['examples'].each_with_index do |e, i|
+                example = e['raw_liquid'].gsub(/[\n]+/, "\n").strip.split('=>')
+                input = example[0]&.strip
+                output = example[1]&.strip
+                desc += "\n  - Example #{i}:\n\n```liquid\n#{input}\n```"
+                desc += "\n##\nOutput: #{output}" if output
+              end
+            end
+          end
+          desc
+        end
+
         def platformos_documentation_url
           "#{PLATFORMOS_DOCUMENTATION_URL}/api-reference/liquid/tags/#{hash['name']}"
         end

data/lib/platformos_check/storage.rb

@@ -18,8 +18,8 @@ module PlatformosCheck
       raise NotImplementedError
     end
 
-    def directories
-      raise NotImplementedError
+    def platformos_app
+      @platformos_app ||= PlatformosCheck::App.new(self)
     end
 
     def versioned?

data/lib/platformos_check/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module PlatformosCheck
-  VERSION = "0.0.3"
+  VERSION = "0.2.0"
 end