sas-linter 0.2.4 → 0.2.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b10cf15e62fe7e0a81d8f9d2e2c9eb61af36ee00d10f32e9ce8354a7ebe3e40f
-  data.tar.gz: 9abe8249ab9d968fdc8aefe558dcdad321ba2879abe4bf08374b9ec998eecb5f
+  metadata.gz: b12c05da17de3409a5eb163f3e3325cb7e2e24770d3de42e01efb786fc796d60
+  data.tar.gz: b14b81be912177959a039484c70ea7bf71f209910cf995822d8083c8d6ea2897
 SHA512:
-  metadata.gz: 38e745b6f3ec4c8a7cc2ce3c23883569bcd931d88d6f9dc0a6c2288ed7f0b39f199e066340e780e8556d0ceeaa7bfeb149185ef2d458f58574281c54dbbf60d3
-  data.tar.gz: 8e6b5f2751eb5b3b9e50b80f0476f953519763a321fba4dd17ce219d891a84a3f0fe3c1ea1fbc3c18f4a698bc0655ef794ff2ef71d1d859aa23195a9119d41bd
+  metadata.gz: 5fa28c6588e96bdd973197577e0e3fb1aa689fd641455ed10225f0354e0243e41fdb1e682b20d0de02d2a443b30ae9d263f419a31d42ea12763d91320bef7078
+  data.tar.gz: aa8eb2ffe598911984fd57acff5ea350136865ccc64da7d7003b23033840013ac5c544a9bfd56d7f57bcb67c1f01de5af74c5b32f59d87300eac3525ae9e9bfe

data/README.md

@@ -1,6 +1,6 @@
 # sas-linter
 
-A configurable lint engine for SAS source files. Built on the [`sas-lexer`](https://github.com/mes-amis/sas-lexer-rb) gem (a Ruby FFI binding to Misha Perlov's Rust [`sas-lexer`](https://github.com/mishamsk/sas-lexer)) and ships with fourteen pluggable rules covering structural defects, cosmetic issues, and source-header conventions.
+A configurable lint engine for SAS source files. Built on the [`sas-lexer`](https://github.com/mes-amis/sas-lexer-rb) gem (a Ruby FFI binding to Misha Perlov's Rust [`sas-lexer`](https://github.com/mishamsk/sas-lexer)) and ships with fifteen pluggable rules covering structural defects, cosmetic issues, and source-header conventions.
 
 ## Installation
 
@@ -63,6 +63,10 @@ rules:
     enabled: true
     autofix: false             # rule supports autofix; off by default
 
+  unterminated_comment:
+    enabled: true
+    autofix: false             # append `;` to `**…**` lines whose missing terminator eats the next line
+
   inconsistent_variable_case:
     enabled: true
     autofix: false             # rewrite every minority casing to the most-common form
@@ -141,6 +145,7 @@ findings = linter.lint_file("path/to/source.sas")
 | `encoding_issues` | Smart-quote / em-dash / Win-1252 byte sequences that confuse downstream tooling. |
 | `malformed_if_condition` | Empty conditions, missing operators, orphan `then`, unbalanced parens, etc. |
 | `missing_assignment_semicolon` | Assignment statements followed by an inline `**` comment but no terminating `;`. |
+| `unterminated_comment` | A standalone `** … **` comment whose missing `;` lets the SAS lexer extend the comment into the next line of real code, silently swallowing it. Autofix appends the `;`. |
 | `variable_value_out_of_known_range` | `if VAR = N` / `if VAR in (...)` literals fall outside the variable's documented acceptable values. Loads the catalog from one or more CSVs with configurable column names and column separator (`,`, `;`, tab). |
 | `inconsistent_variable_case` | Identifier appears with more than one casing in the same file (`myVar` vs `MyVar`). SAS treats both as the same variable; autofix rewrites every minority spelling to the most-common form. Skips proc-format definitions and `format.` / `lib.member` references. |
 | `format_for_unknown_variable` | `format` / `informat` / `attrib` statement assigns a format to a variable that's referenced nowhere else in the file — almost always a typo. Skipped on files that pull in columns via `set` / `merge` / `update` / `infile` / `input`. |

data/lib/sas_linter/rules/unterminated_comment.rb

@@ -0,0 +1,101 @@
+# frozen_string_literal: true
+
+require_relative "../../sas_linter"
+require "sas_lexer"
+
+class SasLinter
+  module Rules
+    # Flag a `** ... **` comment line whose missing `;` causes the
+    # SAS lexer to extend the comment statement into the following
+    # line(s) of real code, silently swallowing it.
+    #
+    # SAS `*` / `**` comment statements are terminated by the next
+    # `;` — the closing `**` is just prose. So
+    #
+    #     ** SOME COMMENT **
+    #     y = x + 1;
+    #
+    # lexes as a single comment token covering both lines, and the
+    # `y = x + 1;` assignment never executes.
+    #
+    # Detection: a COMMENT-channel `PREDICTED_COMMENT_STAT` token
+    # whose `start_line != end_line` AND whose first source line,
+    # rstripped, ends with `**`. That shape is the boxed-comment
+    # closer the user clearly intended — they only forgot the `;`.
+    # Legitimate multi-line `*...;` prose ends its first line with
+    # plain text, not `**`, so it's left alone.
+    #
+    # Autofix: append `;` to the end of each flagged first line.
+    class UnterminatedComment < Rule
+      rule_id :unterminated_comment
+      description "`**` comment missing its terminating `;` — consumes following code."
+      severity :warning
+
+      TT = SasLexer::Lexer::TokenType
+      COMMENT_CHANNEL = SasLexer::Lexer::TokenChannel::COMMENT
+
+      def self.supports_autofix?
+        true
+      end
+
+      def check(_tokens, path:, all_tokens: nil, source: nil)
+        return [] unless all_tokens && source
+
+        lines = source.split("\n", -1)
+        unterminated_comment_lines(all_tokens, lines).map do |i|
+          finding_for_line(lines[i], i, path)
+        end
+      end
+
+      def autofix(source)
+        return source if source.nil? || source.empty?
+
+        lines = source.split("\n", -1)
+        bad = unterminated_comment_lines(tokenize(source), lines)
+        return source if bad.empty?
+
+        bad.each do |i|
+          lines[i] = "#{lines[i].rstrip};"
+        end
+        lines.join("\n")
+      end
+
+      private
+
+      def finding_for_line(line, idx, path)
+        finding(
+          line: idx + 1,
+          column: line.length - line.lstrip.length + 1,
+          message: "`**` comment missing `;` — consumes the next line of code as comment text.",
+          path: path
+        )
+      end
+
+      def tokenize(source)
+        lexer = SasLexer::Lexer.new
+        begin
+          lexer.tokenize(source)
+        ensure
+          lexer.free
+        end
+      end
+
+      # 0-indexed source lines that hold a `** ... **` comment whose
+      # missing `;` made the lexer extend it into the next line.
+      def unterminated_comment_lines(all_tokens, lines)
+        bad = []
+        all_tokens.each do |t|
+          next unless t[:channel] == COMMENT_CHANNEL
+          next unless t[:type] == TT::PREDICTED_COMMENT_STAT
+          next unless t[:start_line] < t[:end_line]
+
+          first = lines[t[:start_line] - 1] or next
+          next unless first.rstrip.end_with?("**")
+
+          bad << (t[:start_line] - 1)
+        end
+        bad
+      end
+    end
+  end
+end

data/lib/sas_linter/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 class SasLinter
-  VERSION = "0.2.4"
+  VERSION = "0.2.5"
 end

data/lib/sas_linter.rb

@@ -311,3 +311,4 @@ require_relative "sas_linter/rules/variable_value_out_of_known_range"
 require_relative "sas_linter/rules/invalid_numeric_literal"
 require_relative "sas_linter/rules/inconsistent_variable_case"
 require_relative "sas_linter/rules/format_for_unknown_variable"
+require_relative "sas_linter/rules/unterminated_comment"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sas-linter
 version: !ruby/object:Gem::Version
-  version: 0.2.4
+  version: 0.2.5
 platform: ruby
 authors:
 - Craig McNamara
@@ -74,6 +74,7 @@ files:
 - lib/sas_linter/rules/tab_expansion.rb
 - lib/sas_linter/rules/trailing_whitespace.rb
 - lib/sas_linter/rules/unreachable_inner_branch_value.rb
+- lib/sas_linter/rules/unterminated_comment.rb
 - lib/sas_linter/rules/variable_value_out_of_known_range.rb
 - lib/sas_linter/version.rb
 homepage: https://github.com/mes-amis/sas-linter