Merge pull request Homebrew#16826 from Homebrew/cask_installed_versio…

…n_plist

cask: read bundle version from `Info.plist` when sensible.

Library/Homebrew/cask/cask.rb

@@ -2,6 +2,7 @@
 # frozen_string_literal: true
 
 require "attrable"
+require "bundle_version"
 require "cask/cask_loader"
 require "cask/config"
 require "cask/dsl"
@@ -176,8 +177,20 @@ def installed_caskfile
 
     sig { returns(T.nilable(String)) }
     def installed_version
+      return unless (installed_caskfile = self.installed_caskfile)
+
       # <caskroom_path>/.metadata/<version>/<timestamp>/Casks/<token>.{rb,json} -> <version>
-      installed_caskfile&.dirname&.dirname&.dirname&.basename&.to_s
+      installed_caskfile.dirname.dirname.dirname.basename.to_s
+    end
+
+    sig { returns(T.nilable(String)) }
+    def bundle_short_version
+      bundle_version&.short_version
+    end
+
+    sig { returns(T.nilable(String)) }
+    def bundle_long_version
+      bundle_version&.version
     end
 
     def config_path
@@ -326,6 +339,8 @@ def to_h
         "version"              => version,
         "installed"            => installed_version,
         "installed_time"       => install_time&.to_i,
+        "bundle_version"       => bundle_long_version,
+        "bundle_short_version" => bundle_short_version,
         "outdated"             => outdated?,
         "sha256"               => sha256,
         "artifacts"            => artifacts_list,
@@ -386,6 +401,14 @@ def to_hash_with_variations
 
     private
 
+    sig { returns(T.nilable(Homebrew::BundleVersion)) }
+    def bundle_version
+      @bundle_version ||= if (bundle = artifacts.find { |a| a.is_a?(Artifact::App) }&.target) &&
+                             (plist = Pathname("#{bundle}/Contents/Info.plist")) && plist.exist?
+        Homebrew::BundleVersion.from_info_plist(plist)
+      end
+    end
+
     def api_to_local_hash(hash)
       hash["token"] = token
       hash["installed"] = installed_version

Library/Homebrew/test/support/fixtures/cask/everything.json

@@ -21,6 +21,8 @@
   "version": "1.2.3",
   "installed": null,
   "installed_time": null,
+  "bundle_version": null,
+  "bundle_short_version": null,
   "outdated": false,
   "sha256": "c64c05bdc0be845505d6e55e69e696a7f50d40846e76155f0c85d5ff5e7bbb84",
   "artifacts": [

Library/Homebrew/version.rb

@@ -94,6 +94,9 @@ def numeric?
     def null?
       false
     end
+
+    sig { returns(T::Boolean) }
+    def blank? = null?
   end
 
   # A pseudo-token representing the absence of a token.
@@ -127,6 +130,9 @@ def null?
       true
     end
 
+    sig { returns(T::Boolean) }
+    def blank? = true
+
     sig { override.returns(String) }
     def inspect
       "#<#{self.class.name}>"