vinter 0.6.1 → 0.6.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 18b99e927ea645baaa6bf8290f53625a800c64a36e7bcb681d8096f7591ab862
-  data.tar.gz: 35660a038fcc98c284731d75fe3644d41ff5a9212dd1ca7930d21d72b9f02764
+  metadata.gz: 24da5c73f3d23b6cf4102c37632877a090534aa5343e99b73aab613aa6e5a860
+  data.tar.gz: 7250550dedf15c7fe332ab7a048b833ff93c09bbc48433f50b0845aff7448db1
 SHA512:
-  metadata.gz: f9ed5e432fcdc6e0a88545e15da0996dda4c31176c88fdcbb46f2ea4228bc4c431fe1e569b3e7420f370ab8147ac290738ac27dd21f1f7bed0a0d2093da5b933
-  data.tar.gz: be685a411f5592425d122ea8a3080208c1cad58566dc88b7e5850a0e017ae9beebfff1c1a019955a47eb7e54a732ba94e4f2966d52e536404ce35ad1096465f9
+  metadata.gz: 26d2eb57e3ee43e442e786e76da8c06729c0f9253deca4ac991582a7595bdfe08580d4d9ac784eefa55d7619392725c5f818612744088fe49a2d87019261f29f
+  data.tar.gz: 380639a4f1fff2b3010317d8f1fa392ac6911a5fd3beb827590ce6220320d5c5a77209edce450032751e00030391a35743a6787fc51cb5a21ea1c5f7a70db0db

data/lib/vinter/cli.rb

@@ -6,21 +6,125 @@ module Vinter
 
     def run(args)
       if args.empty?
-        puts "Usage: vinter [file.vim|directory]"
+        puts "Usage: vinter [file.vim|directory] [--exclude=dir1,dir2] [--stdio]"
         return 1
       end
 
-      target_path = args[0]
+      # Parse args: first non-option argument is the target path.
+      exclude_value = nil
+      target_path = nil
+      stdio = false
+      format_value = nil
 
-      unless File.exist?(target_path)
-        puts "Error: File or directory not found: #{target_path}"
+      args.each_with_index do |a, i|
+        if a.start_with?("--exclude=")
+          exclude_value = a.split("=", 2)[1]
+        elsif a == "--exclude"
+          exclude_value = args[i + 1]
+        elsif a == "--stdio" || a.start_with?("--stdio=")
+          stdio = true
+        elsif a.start_with?("--format=")
+          format_value = a.split("=", 2)[1]
+        elsif a == "--format"
+          format_value = args[i + 1]
+        elsif !a.start_with?('-') && target_path.nil?
+          target_path = a
+        end
+      end
+
+      if target_path.nil? && !stdio
+        puts "Usage: vinter [file.vim|directory] [--exclude=dir1,dir2] [--stdio]"
         return 1
       end
 
+      unless stdio
+        unless File.exist?(target_path)
+          puts "Error: File or directory not found: #{target_path}"
+          return 1
+        end
+      end
+
+      excludes = Array(exclude_value).flat_map { |v| v.to_s.split(',') }.map(&:strip).reject(&:empty?)
+
+      # normalize format value
+      format_value = format_value.to_s.downcase if format_value
+
+      # Handle STDIN input mode
+      if stdio
+        content = STDIN.read
+
+        if content.nil? || content.empty?
+          puts "No input received on stdin"
+          return 0
+        end
+
+        issues = @linter.lint(content)
+        total_issues = issues.length
+
+        if format_value == 'json'
+          require 'json'
+
+          files = [
+            {
+              path: 'stdin',
+              offenses: issues.map do |issue|
+                {
+                  severity: (issue[:type] == :error ? 'fatal' : (issue[:type] == :warning ? 'warning' : 'convention')),
+                  message: issue[:message],
+                  cop_name: issue[:rule],
+                  corrected: false,
+                  correctable: false,
+                  location: {
+                    start_line: issue[:line] || 1,
+                    start_column: issue[:column] || 1,
+                    last_line: issue[:line] || 1,
+                    last_column: issue[:column] || 1,
+                    length: 0,
+                    line: issue[:line] || 1,
+                    column: issue[:column] || 1
+                  }
+                }
+              end
+            }
+          ]
+
+          metadata = {
+            'rubocop_version' => defined?(Vinter::VERSION) ? Vinter::VERSION : nil,
+            'ruby_engine' => defined?(RUBY_ENGINE) ? RUBY_ENGINE : RUBY_PLATFORM,
+            'ruby_version' => RUBY_VERSION,
+            'ruby_patchlevel' => RUBY_PATCHLEVEL.to_s,
+            'ruby_platform' => RUBY_PLATFORM
+          }
+
+          summary = {
+            'offense_count' => total_issues,
+            'target_file_count' => files.length,
+            'inspected_file_count' => files.length
+          }
+
+          output = { metadata: metadata, files: files, summary: summary }
+          options = {
+            indent: '',
+            space: '',
+            space_before: '',
+            object_nl: '',
+            array_nl: ''
+          }
+          puts JSON.pretty_generate(output, options)
+
+          return total_issues > 0 ? 1 : 0
+        end
+      end
+
       vim_files = if File.directory?(target_path)
-                    find_vim_files(target_path)
+                    find_vim_files(target_path, excludes)
                   else
-                    [target_path]
+                    # Check if single file is inside an excluded directory
+                    if excluded_file?(target_path, excludes, File.dirname(target_path))
+                      []
+                    else
+                      [target_path]
+                    end
                   end
 
       if vim_files.empty?
@@ -30,46 +134,125 @@ module Vinter
 
       total_issues = 0
       error_count = 0
+      json_files = []
 
       vim_files.each do |file_path|
         content = File.read(file_path)
         issues = @linter.lint(content)
         total_issues += issues.length
 
-        if issues.empty?
-          puts "No issues found in #{file_path}" if vim_files.length == 1
+        if format_value == 'json'
+          json_files << {
+            path: file_path,
+            offenses: issues.map do |issue|
+              {
+                severity: (issue[:type] == :error ? 'fatal' : (issue[:type] == :warning ? 'warning' : 'convention')),
+                message: issue[:message],
+                cop_name: issue[:rule],
+                corrected: false,
+                correctable: false,
+                location: {
+                  start_line: issue[:line] || 1,
+                  start_column: issue[:column] || 1,
+                  last_line: issue[:line] || 1,
+                  last_column: issue[:column] || 1,
+                  length: 0,
+                  line: issue[:line] || 1,
+                  column: issue[:column] || 1
+                }
+              }
+            end
+          }
         else
-          puts "Found #{issues.length} issues in #{file_path}:" if vim_files.length > 1
+          if issues.empty?
+            puts "No issues found in #{file_path}" if vim_files.length == 1
+          else
+            puts "Found #{issues.length} issues in #{file_path}:" if vim_files.length > 1
 
-          issues.each do |issue|
-            type_str = case issue[:type]
-                       when :error then "ERROR"
-                       when :warning then "WARNING"
-                       when :rule then "RULE(#{issue[:rule]})"
-                       else "UNKNOWN"
-                       end
+            issues.each do |issue|
+              type_str = case issue[:type]
+                         when :error then "ERROR"
+                         when :warning then "WARNING"
+                         when :rule then "RULE(#{issue[:rule]})"
+                         else "UNKNOWN"
+                         end
 
-            line = issue[:line] || 1
-            column = issue[:column] || 1
+              line = issue[:line] || 1
+              column = issue[:column] || 1
 
-            puts "#{file_path}:#{line}:#{column}: #{type_str}: #{issue[:message]}"
-          end
+              puts "#{file_path}:#{line}:#{column}: #{type_str}: #{issue[:message]}"
+            end
 
-          error_count += 1 if issues.any? { |i| i[:type] == :error }
+            error_count += 1 if issues.any? { |i| i[:type] == :error }
+          end
         end
       end
 
-      if vim_files.length > 1
-        puts "\nProcessed #{vim_files.length} files, found #{total_issues} total issues"
+      #if vim_files.length > 1
+        #puts "\nProcessed #{vim_files.length} files, found #{total_issues} total issues"
+      #end
+
+      if format_value == 'json'
+        require 'json'
+
+        metadata = {
+          'rubocop_version' => defined?(Vinter::VERSION) ? Vinter::VERSION : nil,
+          'ruby_engine' => defined?(RUBY_ENGINE) ? RUBY_ENGINE : RUBY_PLATFORM,
+          'ruby_version' => RUBY_VERSION,
+          'ruby_patchlevel' => RUBY_PATCHLEVEL.to_s,
+          'ruby_platform' => RUBY_PLATFORM
+        }
+
+        summary = {
+          'offense_count' => total_issues,
+          'target_file_count' => vim_files.length,
+          'inspected_file_count' => vim_files.length
+        }
+
+        output = { metadata: metadata, files: json_files, summary: summary }
+        options = {
+          indent: '',
+          space: '',
+          space_before: '',
+          object_nl: '',
+          array_nl: ''
+        }
+        puts JSON.pretty_generate(output, options)
+
+        return total_issues > 0 ? 1 : 0
       end
 
-      return error_count > 0 ? 1 : 0
+      return total_issues > 0 ? 1 : 0
     end
 
     private
 
-    def find_vim_files(directory)
-      Dir.glob(File.join(directory, "**", "*.vim")).sort
+    def find_vim_files(directory, excludes = [])
+      files = Dir.glob(File.join(directory, "**", "*.vim")).sort
+
+      return files if excludes.empty?
+
+      # Normalize exclude directories to absolute paths (relative to the target directory)
+      normalized = excludes.map { |e| File.expand_path(e, directory) }
+
+      files.reject do |f|
+        normalized.any? do |ex|
+          ex_with_slash = ex.end_with?(File::SEPARATOR) ? ex : ex + File::SEPARATOR
+          f.start_with?(ex_with_slash) || File.expand_path(f).start_with?(ex_with_slash)
+        end
+      end
+    end
+
+    def excluded_file?(file_path, excludes, base_dir)
+      return false if excludes.empty?
+
+      normalized = excludes.map { |e| File.expand_path(e, base_dir) }
+      file_abs = File.expand_path(file_path)
+
+      normalized.any? do |ex|
+        ex_with_slash = ex.end_with?(File::SEPARATOR) ? ex : ex + File::SEPARATOR
+        file_abs.start_with?(ex_with_slash)
+      end
     end
   end
 end

data/lib/vinter.rb

@@ -5,5 +5,5 @@ require "vinter/cli"
 require "vinter/ast_printer"
 
 module Vinter
-  VERSION = "0.6.1"
+  VERSION = "0.6.3"
 end

metadata

@@ -1,16 +1,16 @@
 --- !ruby/object:Gem::Specification
 name: vinter
 version: !ruby/object:Gem::Version
-  version: 0.6.1
+  version: 0.6.3
 platform: ruby
 authors:
 - Dan Bradbury
+autorequire:
 bindir: bin
 cert_chain: []
-date: 1980-01-02 00:00:00.000000000 Z
+date: 2025-12-01 00:00:00.000000000 Z
 dependencies: []
-description: A linter for the Vim9 script language, helping to identify issues and
-  enforce best practices
+description: A linter for vim9script
 email: dan.luckydaisy@gmail.com
 executables:
 - vinter
@@ -30,6 +30,7 @@ homepage: https://github.com/DanBradbury/vinter
 licenses:
 - MIT
 metadata: {}
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -44,7 +45,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.9
+rubygems_version: 3.5.11
+signing_key:
 specification_version: 4
-summary: A linter for leagacy + Vim9 script
+summary: A vim9script linter
 test_files: []