Create Resource::BottleManifest.

Homebrew · Jul 13, 2024 · b5fef42 · b5fef42
1 parent 2601551
commit b5fef42
Show file tree

Hide file tree

Showing 2 changed files with 50 additions and 38 deletions.
diff --git a/Library/Homebrew/resource.rb b/Library/Homebrew/resource.rb
@@ -267,6 +267,53 @@ def stage(target, &block)
     end
   end
 
+  # A resource for a bottle manifest.
+  class BottleManifest < Resource
+    attr_reader :bottle
+
+    def initialize(bottle)
+      super("#{bottle.name}_bottle_manifest")
+      @bottle = bottle
+    end
+
+    def verify_download_integrity(_filename)
+      # no-op
+    end
+
+    def tab
+      json = begin
+        JSON.parse(cached_download.read)
+      rescue JSON::ParserError
+        raise "The downloaded GitHub Packages manifest was corrupted or modified (it is not valid JSON): " \
+              "\n#{github_packages_manifest_resource.cached_download}"
+      end
+
+      manifests = json["manifests"]
+      raise ArgumentError, "Missing 'manifests' section." if manifests.blank?
+
+      manifests_annotations = manifests.filter_map { |m| m["annotations"] }
+      raise ArgumentError, "Missing 'annotations' section." if manifests_annotations.blank?
+
+      bottle_digest = bottle.resource.checksum.hexdigest
+      image_ref = GitHubPackages.version_rebuild(bottle.resource.version, bottle.rebuild, bottle.tag.to_s)
+      manifest_annotations = manifests_annotations.find do |m|
+        next if m["sh.brew.bottle.digest"] != bottle_digest
+
+        m["org.opencontainers.image.ref.name"] == image_ref
+      end
+      raise ArgumentError, "Couldn't find manifest matching bottle checksum." if manifest_annotations.blank?
+
+      tab = manifest_annotations["sh.brew.tab"]
+      raise ArgumentError, "Couldn't find tab from manifest." if tab.blank?
+
+      begin
+        JSON.parse(tab)
+      rescue JSON::ParserError
+        raise ArgumentError, "Couldn't parse tab JSON."
+      end
+    end
+  end
+
   # A resource containing a patch.
   class PatchResource < Resource
     attr_reader :patch_files

diff --git a/Library/Homebrew/software_spec.rb b/Library/Homebrew/software_spec.rb
@@ -396,7 +396,7 @@ def fetch_tab
     github_packages_manifest_resource.fetch(verify_download_integrity: false)
 
     begin
-      github_packages_manifest_resource_tab(github_packages_manifest_resource)
+      github_packages_manifest_resource.tab
     rescue RuntimeError => e
       raise DownloadError.new(github_packages_manifest_resource, e)
     end
@@ -415,7 +415,7 @@ def fetch_tab
   def tab_attributes
     return {} unless github_packages_manifest_resource&.downloaded?
 
-    github_packages_manifest_resource_tab(github_packages_manifest_resource)
+    github_packages_manifest_resource.tab
   end
 
   sig { returns(Filename) }
@@ -425,46 +425,11 @@ def filename
 
   private
 
-  def github_packages_manifest_resource_tab(github_packages_manifest_resource)
-    manifest_json = github_packages_manifest_resource.cached_download.read
-
-    json = begin
-      JSON.parse(manifest_json)
-    rescue JSON::ParserError
-      raise "The downloaded GitHub Packages manifest was corrupted or modified (it is not valid JSON): " \
-            "\n#{github_packages_manifest_resource.cached_download}"
-    end
-
-    manifests = json["manifests"]
-    raise ArgumentError, "Missing 'manifests' section." if manifests.blank?
-
-    manifests_annotations = manifests.filter_map { |m| m["annotations"] }
-    raise ArgumentError, "Missing 'annotations' section." if manifests_annotations.blank?
-
-    bottle_digest = @resource.checksum.hexdigest
-    image_ref = GitHubPackages.version_rebuild(@resource.version, rebuild, @tag.to_s)
-    manifest_annotations = manifests_annotations.find do |m|
-      next if m["sh.brew.bottle.digest"] != bottle_digest
-
-      m["org.opencontainers.image.ref.name"] == image_ref
-    end
-    raise ArgumentError, "Couldn't find manifest matching bottle checksum." if manifest_annotations.blank?
-
-    tab = manifest_annotations["sh.brew.tab"]
-    raise ArgumentError, "Couldn't find tab from manifest." if tab.blank?
-
-    begin
-      JSON.parse(tab)
-    rescue JSON::ParserError
-      raise ArgumentError, "Couldn't parse tab JSON."
-    end
-  end
-
   def github_packages_manifest_resource
     return if @resource.download_strategy != CurlGitHubPackagesDownloadStrategy
 
     @github_packages_manifest_resource ||= begin
-      resource = Resource.new("#{name}_bottle_manifest")
+      resource = Resource::BottleManifest.new(self)
 
       version_rebuild = GitHubPackages.version_rebuild(@resource.version, rebuild)
       resource.version(version_rebuild)