vinter 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '0916f02e9f17998516fc27eb0d2b2ec2807d1f4ae3ab12050590556c6d78e0ba'
-  data.tar.gz: 21cec3329527181dde80ebe5e12d5dba094c4596c1bcb725a1ef7dc95505b43e
+  metadata.gz: 503bb79a2701783c22c7ef7f269bfc51dc93a72d68abc61286c4c0236aaa4d8f
+  data.tar.gz: f9f58293732eb9179228ec8e4c04bd87f279fc2e7edf65dc9969d1fe6f680b66
 SHA512:
-  metadata.gz: ce1879809bfc52ff91ad591bbd862ef347cfc8a6595a042cd7bc9efb0c9bb95824ee177844a5499b9264c9dc3c81f4e5bda1b7ccbdb9a85800ab2a87697d07d0
-  data.tar.gz: 10741136273e679ba2cf0c334e4ba815a9ba8fae25ba677fb0d7bd9f441bd912cff8c87e8919c27db110b6bab066102de5188f23305cbde76542793a4a4cb7c2
+  metadata.gz: 577a51ca98e7bd940ab48a6482978c5f098fbba5c5e6921105950086f92cc327e44edf10259a3aafdf6fe8ff068690fb5d8b387aed85f5e07f2fc57b7ea8ce5d
+  data.tar.gz: 5f5e7316bd62168356532c07186e0a3ff03c2c4a49c8d43d7dff51d64f261bf9368598b80acc69ebcc802af64d5ef0906f8422a0c78e2401d87429668721e3f6

data/README.md

@@ -1,6 +1,6 @@
 # Vinter
 
-A Ruby gem that provides linting capabilities for Vim9 script files. This linter helps identify syntax errors and enforce best practices for the new Vim9 script language introduced in Vim 9.0.
+A Ruby gem that provides linting capabilities for Vim9 script files. This linter helps identify syntax errors and enforce best practices for for Vim9 script.
 
 ## Features
 
@@ -24,7 +24,7 @@ gem install vinter
 Lint a Vim9 script file:
 
 ```bash
-vim9-lint path/to/your/script.vim
+vinter path/to/your/script.vim
 ```
 
 ### Ruby API
@@ -33,7 +33,7 @@ vim9-lint path/to/your/script.vim
 require 'vinter'
 
 content = File.read('path/to/your/script.vim')
-linter = Vim9Linter::Linter.new
+linter = Vinter::Linter.new
 issues = linter.lint(content)
 
 issues.each do |issue|

data/lib/vinter/parser.rb

@@ -180,6 +180,157 @@ module Vinter
       }
     end
 
+    def parse_while_statement
+      token = advance # Skip 'while'
+      line = token[:line]
+      column = token[:column]
+      condition = parse_expression
+
+      body = []
+
+      # Parse statements until we hit 'endwhile'
+      while @position < @tokens.length
+        if current_token[:type] == :keyword && current_token[:value] == 'endwhile'
+          break
+        end
+
+        stmt = parse_statement
+        body << stmt if stmt
+      end
+
+      # Expect endwhile
+      expect(:keyword) # This should be 'endwhile'
+
+      {
+        type: :while_statement,
+        condition: condition,
+        body: body,
+        line: line,
+        column: column
+      }
+    end
+
+    def parse_for_statement
+      token = advance # Skip 'for'
+      line = token[:line]
+      column = token[:column]
+
+      # Parse the loop variable(s)
+      if current_token && current_token[:type] == :paren_open
+        # Handle tuple assignment: for (key, val) in dict
+        advance # Skip '('
+
+        loop_vars = []
+
+        loop do
+          if current_token && current_token[:type] == :identifier
+            loop_vars << advance[:value]
+          else
+            @errors << {
+              message: "Expected identifier in for loop variables",
+              position: @position,
+              line: current_token ? current_token[:line] : 0,
+              column: current_token ? current_token[:column] : 0
+            }
+            break
+          end
+
+          if current_token && current_token[:type] == :comma
+            advance # Skip ','
+          else
+            break
+          end
+        end
+
+        expect(:paren_close) # Skip ')'
+
+        if !current_token || (current_token[:type] != :identifier || current_token[:value] != 'in')
+          @errors << {
+            message: "Expected 'in' after for loop variables",
+            position: @position,
+            line: current_token ? current_token[:line] : 0,
+            column: current_token ? current_token[:column] : 0
+          }
+        else
+          advance # Skip 'in'
+        end
+
+        iterable = parse_expression
+
+        # Parse the body until 'endfor'
+        body = []
+        while @position < @tokens.length
+          if current_token[:type] == :keyword && current_token[:value] == 'endfor'
+            break
+          end
+
+          stmt = parse_statement
+          body << stmt if stmt
+        end
+
+        # Expect endfor
+        expect(:keyword) # This should be 'endfor'
+
+        return {
+          type: :for_statement,
+          loop_vars: loop_vars,
+          iterable: iterable,
+          body: body,
+          line: line,
+          column: column
+        }
+      else
+        # Simple for var in list
+        if !current_token || current_token[:type] != :identifier
+          @errors << {
+            message: "Expected identifier as for loop variable",
+            position: @position,
+            line: current_token ? current_token[:line] : 0,
+            column: current_token ? current_token[:column] : 0
+          }
+          return nil
+        end
+
+        loop_var = advance[:value]
+
+        if !current_token || (current_token[:type] != :identifier || current_token[:value] != 'in')
+          @errors << {
+            message: "Expected 'in' after for loop variable",
+            position: @position,
+            line: current_token ? current_token[:line] : 0,
+            column: current_token ? current_token[:column] : 0
+          }
+        else
+          advance # Skip 'in'
+        end
+
+        iterable = parse_expression
+
+        # Parse the body until 'endfor'
+        body = []
+        while @position < @tokens.length
+          if current_token[:type] == :keyword && current_token[:value] == 'endfor'
+            break
+          end
+
+          stmt = parse_statement
+          body << stmt if stmt
+        end
+
+        # Expect endfor
+        expect(:keyword) # This should be 'endfor'
+
+        return {
+          type: :for_statement,
+          loop_var: loop_var,
+          iterable: iterable,
+          body: body,
+          line: line,
+          column: column
+        }
+      end
+    end
+
     def parse_def_function
       token = advance # Skip 'def'
       line = token[:line]
@@ -541,23 +692,23 @@ module Vinter
       token = advance # Skip 'import'
       line = token[:line]
       column = token[:column]
-      
+
       # Handle 'import autoload'
       is_autoload = false
       module_name = nil
       path = nil
-      
+
       if current_token && current_token[:type] == :identifier && current_token[:value] == 'autoload'
         is_autoload = true
         module_name = advance[:value]  # Store "autoload" as the module name
-        
+
         # After "autoload" keyword, expect a string path
         if current_token && current_token[:type] == :string
           path = current_token[:value]
           advance
         else
-          @errors << { 
-            message: "Expected string path after 'autoload'", 
+          @errors << {
+            message: "Expected string path after 'autoload'",
             position: @position,
             line: current_token ? current_token[:line] : 0,
             column: current_token ? current_token[:column] : 0
@@ -567,11 +718,11 @@ module Vinter
         # Regular import with a string path
         if current_token && current_token[:type] == :string
           path = current_token[:value]
-          
+
           # Extract module name from the path
           # This is simplified logic - you might need more complex extraction
           module_name = path.gsub(/['"]/, '').split('/').last.split('.').first
-          
+
           advance
         else
           # Handle other import formats
@@ -582,7 +733,7 @@ module Vinter
           end
         end
       end
-      
+
       # Handle 'as name'
       as_name = nil
       if current_token && current_token[:type] == :identifier && current_token[:value] == 'as'
@@ -590,17 +741,17 @@ module Vinter
         if current_token && current_token[:type] == :identifier
           as_name = advance[:value]
         else
-          @errors << { 
-            message: "Expected identifier after 'as'", 
+          @errors << {
+            message: "Expected identifier after 'as'",
             position: @position,
             line: current_token ? current_token[:line] : 0,
             column: current_token ? current_token[:column] : 0
           }
         end
       end
-      
-      { 
-        type: :import_statement, 
+
+      {
+        type: :import_statement,
         module: module_name,
         path: path,
         is_autoload: is_autoload,
@@ -608,7 +759,7 @@ module Vinter
         line: line,
         column: column
       }
-    end    
+    end
     def parse_export_statement
       token = advance # Skip 'export'
       line = token[:line]

data/lib/vinter.rb

@@ -4,5 +4,5 @@ require "vinter/linter"
 require "vinter/cli"
 
 module Vinter
-  VERSION = "0.1.0"
+  VERSION = "0.2.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: vinter
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - Dan Bradbury
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-04-04 00:00:00.000000000 Z
+date: 2025-04-05 00:00:00.000000000 Z
 dependencies: []
 description: A linter for the Vim9 script language, helping to identify issues and
   enforce best practices