tap_auditor: validate pypi_formula_mappings.json

Signed-off-by: Michael Cho <[email protected]>

Library/Homebrew/Gemfile

@@ -30,9 +30,6 @@ end
 group :man, optional: true do
  gem "ronn", require: false
 end
-group :pr_upload, optional: true do
- gem "json_schemer", require: false
-end
 group :prof, optional: true do
  gem "ruby-prof", require: false
  gem "stackprof", require: false
@@ -71,6 +68,9 @@ end
 group :audit, :bump_unversioned_casks, :livecheck, optional: true do
  gem "rexml", require: false
 end
+group :audit, :pr_upload, optional: true do
+ gem "json_schemer", require: false
+end
 
 # vendored gems (no group)
 gem "addressable"

Library/Homebrew/data/schemas/pypi_formula_mappings.schema.json

@@ -0,0 +1,25 @@
+{
+ "type": "object",
+ "additionalProperties": {
+ "type": ["string", "object"],
+ "properties": {
+ "package_name": {"type": "string"},
+ "exclude_packages": {
+ "type": "array",
+ "items": {"type": "string"},
+ "minItems": 1
+ },
+ "extra_packages": {
+ "type": "array",
+ "items": {"type": "string"},
+ "minItems": 1
+ },
+ "dependencies": {
+ "type": "array",
+ "items": {"type": "string"},
+ "minItems": 1
+ }
+ },
+ "additionalProperties": false
+ }
+}

Library/Homebrew/tap_auditor.rb

@@ -45,13 +45,29 @@ def audit_json_files
  rescue JSON::ParserError
  problem "#{file.to_s.delete_prefix("#{@path}/")} contains invalid JSON"
  end
+
+ require "json_schemer"
+ schemer = JSONSchemer.schema(HOMEBREW_DATA_PATH/"schemas/pypi_formula_mappings.schema.json")
+ return if schemer.valid?(@tap_pypi_formula_mappings)
+
+ problem <<~EOS
+ pypi_formula_mappings.json schema validation failed with following errors:
+ #{schemer.validate(@tap_pypi_formula_mappings).map { |error| "* #{error["error"]}" }.join("\n")}
+ EOS
  end
 
  sig { void }
  def audit_tap_formula_lists
  check_formula_list_directory "audit_exceptions", @tap_audit_exceptions
  check_formula_list_directory "style_exceptions", @tap_style_exceptions
  check_formula_list "pypi_formula_mappings", @tap_pypi_formula_mappings
+
+ @tap_pypi_formula_mappings.each_value do |formula_mapping|
+ next unless formula_mapping.is_a?(Hash)
+ next unless formula_mapping.key?("dependencies")
+
+ check_formula_list "pypi_formula_mappings", formula_mapping["dependencies"]
+ end
  end
 
  sig { void }