fastlane 2.182.0 → 2.184.1

data/deliver/lib/deliver/options.rb

@@ -109,11 +109,11 @@ module Deliver
                                      default_value: false,
                                      env_name: "DELIVER_EDIT_LIVE",
                                      description: "Modify live metadata, this option disables ipa upload and screenshot upload",
-                                     is_string: false),
+                                     type: Boolean),
         FastlaneCore::ConfigItem.new(key: :use_live_version,
                                      env_name: "DELIVER_USE_LIVE_VERSION",
                                      description: "Force usage of live version rather than edit version",
-                                     is_string: false,
+                                     type: Boolean,
                                      default_value: false),
 
         # paths
@@ -132,22 +132,22 @@ module Deliver
         FastlaneCore::ConfigItem.new(key: :skip_binary_upload,
                                      env_name: "DELIVER_SKIP_BINARY_UPLOAD",
                                      description: "Skip uploading an ipa or pkg to App Store Connect",
-                                     is_string: false,
+                                     type: Boolean,
                                      default_value: false),
         FastlaneCore::ConfigItem.new(key: :skip_screenshots,
                                      env_name: "DELIVER_SKIP_SCREENSHOTS",
                                      description: "Don't upload the screenshots",
-                                     is_string: false,
+                                     type: Boolean,
                                      default_value: false),
         FastlaneCore::ConfigItem.new(key: :skip_metadata,
                                      env_name: "DELIVER_SKIP_METADATA",
                                      description: "Don't upload the metadata (e.g. title, description). This will still upload screenshots",
-                                     is_string: false,
+                                     type: Boolean,
                                      default_value: false),
         FastlaneCore::ConfigItem.new(key: :skip_app_version_update,
                                      env_name: "DELIVER_SKIP_APP_VERSION_UPDATE",
                                      description: "Don’t create or update the app version that is being prepared for submission",
-                                     is_string: false,
+                                     type: Boolean,
                                      default_value: false),
 
         # how to operate
@@ -155,22 +155,22 @@ module Deliver
                                      short_option: "-f",
                                      env_name: "DELIVER_FORCE",
                                      description: "Skip verification of HTML preview file",
-                                     is_string: false,
+                                     type: Boolean,
                                      default_value: false),
         FastlaneCore::ConfigItem.new(key: :overwrite_screenshots,
                                      env_name: "DELIVER_OVERWRITE_SCREENSHOTS",
                                      description: "Clear all previously uploaded screenshots before uploading the new ones",
-                                     is_string: false,
+                                     type: Boolean,
                                      default_value: false),
         FastlaneCore::ConfigItem.new(key: :submit_for_review,
                                      env_name: "DELIVER_SUBMIT_FOR_REVIEW",
                                      description: "Submit the new version for Review after uploading everything",
-                                     is_string: false,
+                                     type: Boolean,
                                      default_value: false),
         FastlaneCore::ConfigItem.new(key: :reject_if_possible,
                                      env_name: "DELIVER_REJECT_IF_POSSIBLE",
                                      description: "Rejects the previously submitted build if it's in a state where it's possible",
-                                     is_string: false,
+                                     type: Boolean,
                                      default_value: false),
 
         # release
@@ -198,13 +198,13 @@ module Deliver
                                      env_name: "DELIVER_PHASED_RELEASE",
                                      description: "Enable the phased release feature of iTC",
                                      optional: true,
-                                     is_string: false,
+                                     type: Boolean,
                                      default_value: false),
         FastlaneCore::ConfigItem.new(key: :reset_ratings,
                                     env_name: "DELIVER_RESET_RATINGS",
                                     description: "Reset the summary rating when you release a new version of the application",
                                     optional: true,
-                                    is_string: false,
+                                    type: Boolean,
                                     default_value: false),
 
         # other app configuration
@@ -212,13 +212,12 @@ module Deliver
                                      short_option: "-r",
                                      env_name: "DELIVER_PRICE_TIER",
                                      description: "The price tier of this application",
-                                     is_string: false,
+                                     type: Integer,
                                      optional: true),
         FastlaneCore::ConfigItem.new(key: :app_rating_config_path,
                                      short_option: "-g",
                                      env_name: "DELIVER_APP_RATING_CONFIG_PATH",
                                      description: "Path to the app rating's config",
-                                     is_string: true,
                                      optional: true,
                                      verify_block: proc do |value|
                                        UI.user_error!("Could not find config file at path '#{File.expand_path(value)}'") unless File.exist?(value)
@@ -227,7 +226,6 @@ module Deliver
         FastlaneCore::ConfigItem.new(key: :submission_information,
                                      short_option: "-b",
                                      description: "Extra information for the submission (e.g. compliance specifications, IDFA settings)",
-                                     is_string: false,
                                      type: Hash,
                                      optional: true),
 
@@ -237,7 +235,7 @@ module Deliver
                                      env_name: "DELIVER_TEAM_ID",
                                      description: "The ID of your App Store Connect team if you're in multiple teams",
                                      optional: true,
-                                     is_string: false, # as we also allow integers, which we convert to strings anyway
+                                     skip_type_validation: true, # as we also allow integers, which we convert to strings anyway
                                      code_gen_sensitive: true,
                                      default_value: CredentialsManager::AppfileConfig.try_fetch_value(:itc_team_id),
                                      default_value_dynamic: true,
@@ -260,7 +258,6 @@ module Deliver
                                      env_name: "DELIVER_DEV_PORTAL_TEAM_ID",
                                      description: "The short ID of your Developer Portal team, if you're in multiple teams. Different from your iTC team ID!",
                                      optional: true,
-                                     is_string: true,
                                      code_gen_sensitive: true,
                                      default_value: CredentialsManager::AppfileConfig.try_fetch_value(:team_id),
                                      default_value_dynamic: true,
@@ -293,13 +290,13 @@ module Deliver
                                      short_option: "-x",
                                      env_name: "DELIVER_RUN_PRECHECK_BEFORE_SUBMIT",
                                      description: "Run precheck before submitting to app review",
-                                     is_string: false,
+                                     type: Boolean,
                                      default_value: true),
         FastlaneCore::ConfigItem.new(key: :precheck_default_rule_level,
                                      short_option: "-d",
                                      env_name: "DELIVER_PRECHECK_DEFAULT_RULE_LEVEL",
                                      description: "The default precheck rule level unless otherwise configured",
-                                     is_string: false,
+                                     type: Symbol,
                                      default_value: :warn),
 
         # App Metadata
@@ -307,7 +304,6 @@ module Deliver
                                      env_name: "DELIVER_INDIVUDAL_METADATA_ITEMS",
                                      description: "An array of localized metadata items to upload individually by language so that errors can be identified. E.g. ['name', 'keywords', 'description']. Note: slow",
                                      deprecated: "Removed after the migration to the new App Store Connect API in June 2020",
-                                     is_string: false,
                                      type: Array,
                                      optional: true),
 
@@ -327,66 +323,56 @@ module Deliver
         FastlaneCore::ConfigItem.new(key: :copyright,
                                      env_name: "DELIVER_COPYRIGHT",
                                      description: "Metadata: The copyright notice",
-                                     optional: true,
-                                     is_string: true),
+                                     optional: true),
         FastlaneCore::ConfigItem.new(key: :primary_category,
                                      env_name: "DELIVER_PRIMARY_CATEGORY",
                                      description: "Metadata: The english name of the primary category (e.g. `Business`, `Books`)",
-                                     optional: true,
-                                     is_string: true),
+                                     optional: true),
         FastlaneCore::ConfigItem.new(key: :secondary_category,
                                      env_name: "DELIVER_SECONDARY_CATEGORY",
                                      description: "Metadata: The english name of the secondary category (e.g. `Business`, `Books`)",
-                                     optional: true,
-                                     is_string: true),
+                                     optional: true),
         FastlaneCore::ConfigItem.new(key: :primary_first_sub_category,
                                      env_name: "DELIVER_PRIMARY_FIRST_SUB_CATEGORY",
                                      description: "Metadata: The english name of the primary first sub category (e.g. `Educational`, `Puzzle`)",
-                                     optional: true,
-                                     is_string: true),
+                                     optional: true),
         FastlaneCore::ConfigItem.new(key: :primary_second_sub_category,
                                      env_name: "DELIVER_PRIMARY_SECOND_SUB_CATEGORY",
                                      description: "Metadata: The english name of the primary second sub category (e.g. `Educational`, `Puzzle`)",
-                                     optional: true,
-                                     is_string: true),
+                                     optional: true),
         FastlaneCore::ConfigItem.new(key: :secondary_first_sub_category,
                                      env_name: "DELIVER_SECONDARY_FIRST_SUB_CATEGORY",
                                      description: "Metadata: The english name of the secondary first sub category (e.g. `Educational`, `Puzzle`)",
-                                     optional: true,
-                                     is_string: true),
+                                     optional: true),
         FastlaneCore::ConfigItem.new(key: :secondary_second_sub_category,
                                      env_name: "DELIVER_SECONDARY_SECOND_SUB_CATEGORY",
                                      description: "Metadata: The english name of the secondary second sub category (e.g. `Educational`, `Puzzle`)",
-                                     optional: true,
-                                     is_string: true),
+                                     optional: true),
         FastlaneCore::ConfigItem.new(key: :trade_representative_contact_information,
                                      description: "Metadata: A hash containing the trade representative contact information",
                                      optional: true,
-                                     is_string: false,
+                                     deprecated: "This is no longer used by App Store Connect",
                                      type: Hash),
         FastlaneCore::ConfigItem.new(key: :app_review_information,
                                      description: "Metadata: A hash containing the review information",
                                      optional: true,
-                                     is_string: false,
                                      type: Hash),
         FastlaneCore::ConfigItem.new(key: :app_review_attachment_file,
                                      env_name: "DELIVER_APP_REVIEW_ATTACHMENT_FILE",
                                      description: "Metadata: Path to the app review attachment file",
-                                     optional: true,
-                                     is_string: true),
+                                     optional: true),
         # Localised
         FastlaneCore::ConfigItem.new(key: :description,
                                      description: "Metadata: The localised app description",
                                      optional: true,
-                                     is_string: false),
+                                     type: Hash),
         FastlaneCore::ConfigItem.new(key: :name,
                                      description: "Metadata: The localised app name",
                                      optional: true,
-                                     is_string: false),
+                                     type: Hash),
         FastlaneCore::ConfigItem.new(key: :subtitle,
                                      description: "Metadata: The localised app subtitle",
                                      optional: true,
-                                     is_string: false,
                                      type: Hash,
                                      verify_block: proc do |value|
                                        UI.user_error!(":subtitle must be a hash, with the language being the key") unless value.kind_of?(Hash)
@@ -394,7 +380,6 @@ module Deliver
         FastlaneCore::ConfigItem.new(key: :keywords,
                                      description: "Metadata: An array of localised keywords",
                                      optional: true,
-                                     is_string: false,
                                      type: Hash,
                                      verify_block: proc do |value|
                                        UI.user_error!(":keywords must be a hash, with the language being the key") unless value.kind_of?(Hash)
@@ -409,7 +394,6 @@ module Deliver
         FastlaneCore::ConfigItem.new(key: :promotional_text,
                                      description: "Metadata: An array of localised promotional texts",
                                      optional: true,
-                                     is_string: false,
                                      type: Hash,
                                      verify_block: proc do |value|
                                        UI.user_error!(":keywords must be a hash, with the language being the key") unless value.kind_of?(Hash)
@@ -417,23 +401,23 @@ module Deliver
         FastlaneCore::ConfigItem.new(key: :release_notes,
                                      description: "Metadata: Localised release notes for this version",
                                      optional: true,
-                                     is_string: false),
+                                     type: Hash),
         FastlaneCore::ConfigItem.new(key: :privacy_url,
                                      description: "Metadata: Localised privacy url",
                                      optional: true,
-                                     is_string: false),
+                                     type: Hash),
         FastlaneCore::ConfigItem.new(key: :apple_tv_privacy_policy,
                                      description: "Metadata: Localised Apple TV privacy policy text",
                                      optional: true,
-                                     is_string: false),
+                                     type: Hash),
         FastlaneCore::ConfigItem.new(key: :support_url,
                                      description: "Metadata: Localised support url",
                                      optional: true,
-                                     is_string: false),
+                                     type: Hash),
         FastlaneCore::ConfigItem.new(key: :marketing_url,
                                      description: "Metadata: Localised marketing url",
                                      optional: true,
-                                     is_string: false),
+                                     type: Hash),
         # The verify_block has been removed from here and verification now happens in Deliver::DetectValues
         # Verification needed Spaceship::Tunes.client which required the Deliver::Runner to already by started
         FastlaneCore::ConfigItem.new(key: :languages,
@@ -445,11 +429,11 @@ module Deliver
                                      env_name: "DELIVER_IGNORE_LANGUAGE_DIRECTORY_VALIDATION",
                                      description: "Ignore errors when invalid languages are found in metadata and screenshot directories",
                                      default_value: false,
-                                     is_string: false),
+                                     type: Boolean),
         FastlaneCore::ConfigItem.new(key: :precheck_include_in_app_purchases,
                                      env_name: "PRECHECK_INCLUDE_IN_APP_PURCHASES",
                                      description: "Should precheck check in-app purchases?",
-                                     is_string: false,
+                                     type: Boolean,
                                      optional: true,
                                      default_value: true),
 
@@ -458,7 +442,8 @@ module Deliver
                                      short_option: "-p",
                                      env_name: "DELIVER_APP_ID",
                                      description: "The (spaceship) app ID of the app you want to use/modify",
-                                     is_string: false) # don't add any verification here, as it's used to store a spaceship ref
+                                     optional: true,
+                                     type: Integer)
       ]
     end
   end

data/deliver/lib/deliver/runner.rb

@@ -26,9 +26,11 @@ module Deliver
     end
 
     def login
-      if api_token
+      if (api_token = Spaceship::ConnectAPI::Token.from(hash: options[:api_key], filepath: options[:api_key_path]))
         UI.message("Creating authorization token for App Store Connect API")
         Spaceship::ConnectAPI.token = api_token
+      elsif !Spaceship::ConnectAPI.token.nil?
+        UI.message("Using existing authorization token for App Store Connect API")
       else
         # Username is now optional since addition of App Store Connect API Key
         # Force asking for username to prompt user if not already set
@@ -42,12 +44,6 @@ module Deliver
       end
     end
 
-    def api_token
-      @api_token ||= Spaceship::ConnectAPI::Token.create(**options[:api_key]) if options[:api_key]
-      @api_token ||= Spaceship::ConnectAPI::Token.from_json_file(options[:api_key_path]) if options[:api_key_path]
-      return @api_token
-    end
-
     def run
       verify_version if options[:app_version].to_s.length > 0 && !options[:skip_app_version_update]
 
@@ -118,7 +114,7 @@ module Deliver
       app_version = options[:app_version]
       UI.message("Making sure the latest version on App Store Connect matches '#{app_version}'...")
 
-      app = options[:app]
+      app = Deliver.cache[:app]
 
       platform = Spaceship::ConnectAPI::Platform.map(options[:platform])
       changed = app.ensure_version!(app_version, platform: platform)
@@ -167,14 +163,14 @@ module Deliver
 
       if upload_ipa
         package_path = FastlaneCore::IpaUploadPackageBuilder.new.generate(
-          app_id: options[:app].id,
+          app_id: Deliver.cache[:app].id,
           ipa_path: options[:ipa],
           package_path: "/tmp",
           platform: options[:platform]
         )
       elsif upload_pkg
         package_path = FastlaneCore::PkgUploadPackageBuilder.new.generate(
-          app_id: options[:app].id,
+          app_id: Deliver.cache[:app].id,
           pkg_path: options[:pkg],
           package_path: "/tmp",
           platform: options[:platform]
@@ -191,7 +187,7 @@ module Deliver
     end
 
     def reject_version_if_possible
-      app = options[:app]
+      app = Deliver.cache[:app]
       platform = Spaceship::ConnectAPI::Platform.map(options[:platform])
       if app.reject_version_if_possible!(platform: platform)
         UI.success("Successfully rejected previous version!")
@@ -210,6 +206,7 @@ module Deliver
     # If there are fewer than two teams, don't infer the provider.
     def transporter_for_selected_team
       # Use JWT auth
+      api_token = Spaceship::ConnectAPI.token
       unless api_token.nil?
         api_token.refresh! if api_token.expired?
         return FastlaneCore::ItunesTransporter.new(nil, nil, false, nil, api_token.text)

data/deliver/lib/deliver/setup.rb

@@ -39,7 +39,7 @@ module Deliver
     # This method takes care of creating a new 'deliver' folder, containing the app metadata
     # and screenshots folders
     def generate_deliver_file(deliver_path, options)
-      app = options[:app]
+      app = Deliver.cache[:app]
 
       platform = Spaceship::ConnectAPI::Platform.map(options[:platform])
       v = app.get_latest_app_store_version(platform: platform)

data/deliver/lib/deliver/submit_for_review.rb

@@ -7,7 +7,7 @@ require 'fastlane_core/pkg_file_analyser'
 module Deliver
   class SubmitForReview
     def submit!(options)
-      app = options[:app]
+      app = Deliver.cache[:app]
 
       platform = Spaceship::ConnectAPI::Platform.map(options[:platform])
       version = app.get_edit_app_store_version(platform: platform)
@@ -57,7 +57,7 @@ module Deliver
 
     def update_export_compliance(options, app, build)
       submission_information = options[:submission_information] || {}
-      submission_information = submission_information.collect { |k, v| [k.to_sym, v] }.to_h
+      submission_information = submission_information.transform_keys(&:to_sym)
 
       uses_encryption = submission_information[:export_compliance_uses_encryption]
 
@@ -87,7 +87,7 @@ module Deliver
 
     def update_idfa(options, app, version)
       submission_information = options[:submission_information] || {}
-      submission_information = submission_information.collect { |k, v| [k.to_sym, v] }.to_h
+      submission_information = submission_information.transform_keys(&:to_sym)
 
       uses_idfa = submission_information[:add_id_info_uses_idfa]
 
@@ -163,7 +163,7 @@ module Deliver
 
     def update_submission_information(options, app)
       submission_information = options[:submission_information] || {}
-      submission_information = submission_information.collect { |k, v| [k.to_sym, v] }.to_h
+      submission_information = submission_information.transform_keys(&:to_sym)
 
       content_rights = submission_information[:content_rights_contains_third_party_content]
 

data/deliver/lib/deliver/upload_metadata.rb

@@ -82,7 +82,7 @@ module Deliver
     def upload(options)
       return if options[:skip_metadata]
 
-      app = options[:app]
+      app = Deliver.cache[:app]
 
       platform = Spaceship::ConnectAPI::Platform.map(options[:platform])
 
@@ -342,7 +342,7 @@ module Deliver
 
       set_review_information(version, options)
       set_review_attachment_file(version, options)
-      set_app_rating(version, options)
+      set_app_rating(app_info, options)
     end
 
     # rubocop:enable Metrics/PerceivedComplexity
@@ -595,7 +595,7 @@ module Deliver
       info = options[:app_review_information]
       return if info.nil? || info.empty?
 
-      info = info.collect { |k, v| [k.to_sym, v] }.to_h
+      info = info.transform_keys(&:to_sym)
       UI.user_error!("`app_review_information` must be a hash", show_github_issues: true) unless info.kind_of?(Hash)
 
       attributes = {}
@@ -642,7 +642,7 @@ module Deliver
       end
     end
 
-    def set_app_rating(version, options)
+    def set_app_rating(app_info, options)
       return unless options[:app_rating_config_path]
 
       require 'json'
@@ -675,9 +675,23 @@ module Deliver
         has_mapped_values = true
         UI.deprecated("Age rating '#{k}' from iTunesConnect has been deprecated. Please replace with '#{v}'")
       end
-      UI.deprecated("You can find more info at https://docs.fastlane.tools/actions/deliver/#reference") if has_mapped_values
 
-      age_rating_declaration = version.fetch_age_rating_declaration
+      # Handle App Store Connect deprecation/migrations of keys/values if possible
+      attributes, deprecation_messages, errors = Spaceship::ConnectAPI::AgeRatingDeclaration.map_deprecation_if_possible(attributes)
+      deprecation_messages.each do |message|
+        UI.deprecated(message)
+      end
+
+      unless errors.empty?
+        errors.each do |error|
+          UI.error(error)
+        end
+        UI.user_error!("There are Age Rating deprecation errors that cannot be solved automatically... Please apply any fixes and try again")
+      end
+
+      UI.deprecated("You can find more info at https://docs.fastlane.tools/actions/deliver/#reference") if has_mapped_values || !deprecation_messages.empty?
+
+      age_rating_declaration = app_info.fetch_age_rating_declaration
       age_rating_declaration.update(attributes: attributes)
     end
   end