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/sorted_dependencies.rb | 52 ---------------------------------- 1 file changed, 52 deletions(-) delete mode 100644 scripts/linters/sorted_dependencies.rb (limited to 'scripts/linters/sorted_dependencies.rb') diff --git a/scripts/linters/sorted_dependencies.rb b/scripts/linters/sorted_dependencies.rb deleted file mode 100644 index f7ae816e..00000000 --- a/scripts/linters/sorted_dependencies.rb +++ /dev/null @@ -1,52 +0,0 @@ -def sorted_dependencies(root_dir, excludes = []) - pattern = root_dir.join('crates', '*', 'Cargo.toml').to_s - errors = Dir.glob(pattern).sort.flat_map do |path| - relative = Pathname.new(path).relative_path_from(root_dir).to_s - next [] if excludes.include?(relative) - - sections = parse_dep_sections(File.read(path)) - - %w[dependencies dev-dependencies].filter_map do |section| - deps = sections[section] - next if deps.nil? || deps.empty? - - expected = sort_dep_names(deps) - next if deps == expected - - { path: relative, section: section, actual: deps, expected: expected } - end - end - - return true if errors.empty? - - puts 'Found unsorted `[dependencies]` / `[dev-dependencies]` in Cargo.toml.' - puts 'Entries must be alphabetical, with `shirabe-*` crates listed before others:' - errors.each do |err| - puts " #{err[:path]} [#{err[:section]}]" - puts " actual: #{err[:actual].join(', ')}" - puts " expected: #{err[:expected].join(', ')}" - end - false -end - -def parse_dep_sections(content) - sections = {} - current = nil - - content.each_line do |line| - stripped = line.chomp - if stripped =~ /\A\s*\[([^\]]+)\]\s*\z/ - current = $1 - sections[current] ||= [] - elsif current && stripped =~ /\A([A-Za-z0-9_-]+)\s*[.=]/ - sections[current] << $1 - end - end - - sections -end - -def sort_dep_names(deps) - shirabe, other = deps.partition { |d| d.start_with?('shirabe-') } - shirabe.sort + other.sort -end -- cgit v1.3.1