From c899a4675ca90507a420d901ef7fa26d8b285f23 Mon Sep 17 00:00:00 2001 From: nsfisis Date: Tue, 21 Jul 2026 08:08:15 +0900 Subject: refactor(lint): rewrite structural linters from Ruby to PHP Fold scripts/lint and scripts/linters/*.rb into a standalone Composer project under scripts/linters/, matching the scripts/plugin-class-classifier/ convention. Uses no external packages, only PHP + Composer autoloading. Verified byte-for-byte identical output against the original Ruby implementation, both on the current repo (all linters pass) and on a synthetic fixture exercising every violation type. Entry point moves from `scripts/lint` to `scripts/linters/lint`. --- scripts/linters/no_format_trailing_comma.rb | 27 --------------------------- 1 file changed, 27 deletions(-) delete mode 100644 scripts/linters/no_format_trailing_comma.rb (limited to 'scripts/linters/no_format_trailing_comma.rb') diff --git a/scripts/linters/no_format_trailing_comma.rb b/scripts/linters/no_format_trailing_comma.rb deleted file mode 100644 index 3e474a76..00000000 --- a/scripts/linters/no_format_trailing_comma.rb +++ /dev/null @@ -1,27 +0,0 @@ -def no_format_trailing_comma(root_dir, excludes = []) - crates_dir = root_dir.join('crates') - raw = `grep -rn --include='*.rs' -e ',)' #{crates_dir}` - - errors = [] - raw.each_line do |line| - line = line.chomp - path, lineno, content = line.split(':', 3) - next if content.nil? - - relative = Pathname.new(path).relative_path_from(root_dir).to_s - next if excludes.include?(relative) - - # `(,)` is a macro repetition fragment (e.g. `$(,)?`). - next if content.match?(/\(,\)/) - - errors << "#{relative}:#{lineno}: trailing `,)` before a closing paren" - end - - return true if errors.empty? - - puts 'Found `,)` introduced by formatting. Remove it.' - errors.each do |err| - puts " #{err}" - end - false -end -- cgit v1.3.1