slim_lint 0.26.0 → 0.28.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 007f6af5a15bd9858e04011b2fa0b604bc523148929b96852f44661b95fb1f66
-  data.tar.gz: 156ad192dc47014b863c9be9840f2280ee3174aa98bb96588b34017ffa7d470e
+  metadata.gz: b7be0fb0eb28f9ecd303cae98461185de3e6778c8ac50230adf67cb5b741d432
+  data.tar.gz: 584e97fc7dd83fa5e71aeec357e365cc5c5fd23f16d2504a7e7b62fbacb64b0e
 SHA512:
-  metadata.gz: 30e9a1af1ba6b00cdc64037fa862eb2b40789e81f6d846183d84755a37396de87a7606ef30b8c11955ca3dc41fe691a157b5b2e75b737b73a03dfaecde8279c3
-  data.tar.gz: 2664127ced343842a71564c9be8b146f39a5d0374c48bb4c6f41fe764c021507542a81ddc24c9df3b12fab79d62408c2971179f9b57b02e5781846b31922c63d
+  metadata.gz: 7922b5927ef482197e73669568b8dbe3c676a590f98164a52a085b5312853be43439a3fe36641301c68e5621ea6f40cefb566c523b9d6327a349a5fd5989e9cb
+  data.tar.gz: 110fb1466b24a89d8e2868afbb8e1bb1540a575bd406a075a63a29f8d5aae28f06ec1e62512d3cb0ccd252f2e04cbabc97a8793ea9cd7ee917bb2673f4ab704a

data/config/default.yml

@@ -32,6 +32,12 @@ linters:
     enabled: false
     max: 300
 
+  InstanceVariables:
+    enabled: false
+    include:
+      # Include only partial templates by default
+      - app/views/**/_*.html.slim
+
   LineLength:
     enabled: true
     max: 80
@@ -83,9 +89,19 @@ linters:
       - Style/WhileUntilDo
       - Style/WhileUntilModifier
 
+  StrictLocalsMissing:
+    enabled: false
+    include:
+      # Include only Rails partial templates by default
+      - app/views/**/_*.html.slim
+
   Tab:
     enabled: true
 
+  Tag:
+    enabled: false
+    forbidden_tags: []
+
   TagCase:
     enabled: true
 

data/lib/slim_lint/linter/embedded_engines.rb

@@ -12,7 +12,7 @@ module SlimLint
       dummy_node = Struct.new(:line)
       document.source_lines.each_with_index do |line, index|
         forbidden_engines.each do |forbidden_engine|
-          next unless line =~ /^#{forbidden_engine}.*:\s*$/
+          next unless line =~ /^\s*#{forbidden_engine}.*:\s*$/
 
           report_lint(dummy_node.new(index + 1), MESSAGE % forbidden_engine)
         end

data/lib/slim_lint/linter/instance_variables.rb

@@ -0,0 +1,34 @@
+# frozen_string_literal: true
+
+module SlimLint
+  # Searches for instance variables in partial or other templates.
+  class Linter::InstanceVariables < Linter
+    include LinterRegistry
+
+    on_start do |_sexp|
+      processed_sexp = SlimLint::RubyExtractEngine.new.call(document.source)
+
+      extractor = SlimLint::RubyExtractor.new
+      extracted_source = extractor.extract(processed_sexp)
+      next if extracted_source.source.empty?
+
+      parsed_ruby = parse_ruby(extracted_source.source)
+      next unless parsed_ruby
+
+      report_instance_variables(parsed_ruby, extracted_source.source_map)
+    end
+
+    private
+
+    def report_instance_variables(parsed_ruby, source_map)
+      parsed_ruby.each_node do |node|
+        next unless node.ivar_type?
+
+        dummy_node = Struct.new(:line)
+        report_lint(dummy_node.new(source_map[node.loc.line]),
+                    'Avoid instance variables in the configured ' \
+                    "view templates (found `#{node.source}`)")
+      end
+    end
+  end
+end

data/lib/slim_lint/linter/strict_locals_missing.rb

@@ -0,0 +1,15 @@
+# frozen_string_literal: true
+
+module SlimLint
+  # Reports on missing strict locals magic line in Slim templates.
+  class Linter::StrictLocalsMissing < Linter
+    include LinterRegistry
+
+    on_start do |_sexp|
+      unless document.source =~ %r{/#\s+locals:\s+\(.*\)}
+        dummy_node = Struct.new(:line)
+        report_lint(dummy_node.new(1), 'Strict locals magic line is missing')
+      end
+    end
+  end
+end

data/lib/slim_lint/linter/tag.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+module SlimLint
+  # Checks for forbidden tags.
+  class Linter::Tag < Linter
+    include LinterRegistry
+
+    on [:html, :tag] do |sexp|
+      _, _, name = sexp
+
+      forbidden_tags = config['forbidden_tags']
+      forbidden_tags.each do |forbidden_tag|
+        next unless name[/^#{forbidden_tag}$/i]
+
+        report_lint(sexp, "Forbidden tag `#{name}` found")
+      end
+    end
+  end
+end

data/lib/slim_lint/version.rb

@@ -2,5 +2,5 @@
 
 # Defines the gem version.
 module SlimLint
-  VERSION = '0.26.0'
+  VERSION = '0.28.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: slim_lint
 version: !ruby/object:Gem::Version
-  version: 0.26.0
+  version: 0.28.0
 platform: ruby
 authors:
 - Shane da Silva
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-01-16 00:00:00.000000000 Z
+date: 2024-08-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -114,10 +114,13 @@ files:
 - lib/slim_lint/linter/empty_control_statement.rb
 - lib/slim_lint/linter/empty_lines.rb
 - lib/slim_lint/linter/file_length.rb
+- lib/slim_lint/linter/instance_variables.rb
 - lib/slim_lint/linter/line_length.rb
 - lib/slim_lint/linter/redundant_div.rb
 - lib/slim_lint/linter/rubocop.rb
+- lib/slim_lint/linter/strict_locals_missing.rb
 - lib/slim_lint/linter/tab.rb
+- lib/slim_lint/linter/tag.rb
 - lib/slim_lint/linter/tag_case.rb
 - lib/slim_lint/linter/trailing_blank_lines.rb
 - lib/slim_lint/linter/trailing_whitespace.rb
@@ -164,7 +167,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.10
+rubygems_version: 3.5.9
 signing_key:
 specification_version: 4
 summary: Slim template linting tool