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 }