diff options
| author | nsfisis <nsfisis@gmail.com> | 2026-06-28 17:45:06 +0900 |
|---|---|---|
| committer | nsfisis <nsfisis@gmail.com> | 2026-06-28 17:45:26 +0900 |
| commit | 53f1fb395f33e0fb8db9aebd09ea9082f650f9f1 (patch) | |
| tree | f9e8bf0e9d4b1e98cce383574fb6a13b684fff08 /scripts/linters/cargo_workspace_dependencies.rb | |
| parent | 212f5cd75b1403ee75ffa44d7ebdb181174340c0 (diff) | |
| download | php-shirabe-53f1fb395f33e0fb8db9aebd09ea9082f650f9f1.tar.gz php-shirabe-53f1fb395f33e0fb8db9aebd09ea9082f650f9f1.tar.zst php-shirabe-53f1fb395f33e0fb8db9aebd09ea9082f650f9f1.zip | |
refactor: add linter
Diffstat (limited to 'scripts/linters/cargo_workspace_dependencies.rb')
| -rw-r--r-- | scripts/linters/cargo_workspace_dependencies.rb | 50 |
1 files changed, 50 insertions, 0 deletions
diff --git a/scripts/linters/cargo_workspace_dependencies.rb b/scripts/linters/cargo_workspace_dependencies.rb new file mode 100644 index 0000000..6e4278f --- /dev/null +++ b/scripts/linters/cargo_workspace_dependencies.rb @@ -0,0 +1,50 @@ +def cargo_workspace_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) + + find_non_workspace_deps(path, relative) + end + + return true if errors.empty? + + puts 'Found `[dependencies]` / `[dev-dependencies]` entries that do not use `workspace = true`.' + puts 'In a crate `Cargo.toml`, only `name.workspace = true` or `name = { workspace = true, ... }` is allowed:' + errors.each do |err| + puts " #{err}" + end + false +end + +def find_non_workspace_deps(path, relative) + errors = [] + current_section = nil + + File.read(path).each_line.with_index do |raw_line, idx| + stripped = raw_line.chomp.strip + + if stripped =~ /\A\[([^\]]+)\]\z/ + current_section = $1 + next + end + + next unless %w[dependencies dev-dependencies build-dependencies].include?(current_section) + next if stripped.empty? || stripped.start_with?('#') + + if stripped =~ /\A([A-Za-z0-9_-]+)\.workspace\s*=\s*true\b/ + next + elsif stripped =~ /\A([A-Za-z0-9_-]+)\s*=\s*\{(.+)\}\s*\z/ + name = $1 + inner = $2 + next if inner =~ /\bworkspace\s*=\s*true\b/ + + errors << "#{relative}:#{idx + 1}: `#{name}` does not use `workspace = true`" + elsif stripped =~ /\A([A-Za-z0-9_-]+)\s*=/ + name = $1 + errors << "#{relative}:#{idx + 1}: `#{name}` does not use `workspace = true`" + end + end + + errors +end |
