fastlane 2.232.2 → 2.233.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b51c417477b7551fedb50df2cee8534dff03c5d626bbc087593f0f9bd5016db8
-  data.tar.gz: 441b8c56863434b00e2853292d5995de56f3226ee779d00c0a1f7d466b2a410d
+  metadata.gz: a1e9ccdeab76489550801ba5eca3943ce3fb8b0f6a54e68281f39879469b5476
+  data.tar.gz: abd22788dfc2496906517916afbc50d18ae7789e5ba8430ccfdef5b979573f2b
 SHA512:
-  metadata.gz: 5985fb114fa5eeb03f19b81452e0d4094cba4c05684985280b04adf042eac10d80c43ebbcc264cf94ab3b30e1934f7639d9e6011f75278b50075158532e4c52a
-  data.tar.gz: 9d50eaca265539d2a64e073f6fed2032080dbbc40bc94f6b70ee23a2caeb122f6e03714deb7028d000b621b52de1b0fe4e62c5ce9b1b7e0d4adef9ba21a7a61c
+  metadata.gz: a801932f673957a2ce012624ad221efd34d7d0d04deca0f7df0f6dbc7eb976bb5ad1eaf6e0d005e1e60c7e01219665afa0b514e695dc063cf75e2834e79f0aaf
+  data.tar.gz: 81ff84ead2d0339f588af6fae06452092a97dac684513c5f5b02af084084bd727a6b140d17d01f3d85ed1f6d374f7063828dc6a86aa5ba139d0acbdd36ccb799

data/README.md

@@ -35,55 +35,55 @@ If the above doesn't help, please [submit an issue](https://github.com/fastlane/
 <!-- This table is regenerated and resorted on each release -->
 <table id='team'>
 <tr>
-<td id='manish-rathi'>
-<a href='https://github.com/crazymanish'>
-<img src='https://github.com/crazymanish.png' width='140px;'>
+<td id='manu-wallner'>
+<a href='https://github.com/milch'>
+<img src='https://github.com/milch.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/iammanishrathi'>Manish Rathi</a></h4>
+<h4 align='center'><a href='https://twitter.com/acrooow'>Manu Wallner</a></h4>
 </td>
-<td id='connor-tumbleson'>
-<a href='https://github.com/ibotpeaches'>
-<img src='https://github.com/ibotpeaches.png' width='140px;'>
+<td id='danielle-tomlinson'>
+<a href='https://github.com/endocrimes'>
+<img src='https://github.com/endocrimes.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/ibotpeaches'>Connor Tumbleson</a></h4>
+<h4 align='center'><a href='https://twitter.com/endocrimes'>Danielle Tomlinson</a></h4>
 </td>
-<td id='satoshi-namai'>
-<a href='https://github.com/ainame'>
-<img src='https://github.com/ainame.png' width='140px;'>
+<td id='aaron-brager'>
+<a href='https://github.com/getaaron'>
+<img src='https://github.com/getaaron.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/ainame'>Satoshi Namai</a></h4>
+<h4 align='center'><a href='https://twitter.com/getaaron'>Aaron Brager</a></h4>
 </td>
-<td id='jorge-revuelta-h'>
-<a href='https://github.com/minuscorp'>
-<img src='https://github.com/minuscorp.png' width='140px;'>
+<td id='jimmy-dee'>
+<a href='https://github.com/jdee'>
+<img src='https://github.com/jdee.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/minuscorp'>Jorge Revuelta H</a></h4>
+<h4 align='center'>Jimmy Dee</h4>
 </td>
-<td id='roger-oba'>
-<a href='https://github.com/rogerluan'>
-<img src='https://github.com/rogerluan.png' width='140px;'>
+<td id='kohki-miki'>
+<a href='https://github.com/giginet'>
+<img src='https://github.com/giginet.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/rogerluan_'>Roger Oba</a></h4>
+<h4 align='center'><a href='https://twitter.com/giginet'>Kohki Miki</a></h4>
 </td>
 </tr>
 <tr>
-<td id='max-ott'>
-<a href='https://github.com/max-ott'>
-<img src='https://github.com/max-ott.png' width='140px;'>
+<td id='matthew-ellis'>
+<a href='https://github.com/matthewellis'>
+<img src='https://github.com/matthewellis.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/ott_max'>Max Ott</a></h4>
+<h4 align='center'><a href='https://twitter.com/mellis1995'>Matthew Ellis</a></h4>
 </td>
-<td id='helmut-januschka'>
-<a href='https://github.com/hjanuschka'>
-<img src='https://github.com/hjanuschka.png' width='140px;'>
+<td id='fumiya-nakamura'>
+<a href='https://github.com/nafu'>
+<img src='https://github.com/nafu.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/hjanuschka'>Helmut Januschka</a></h4>
+<h4 align='center'><a href='https://twitter.com/nafu003'>Fumiya Nakamura</a></h4>
 </td>
-<td id='danielle-tomlinson'>
-<a href='https://github.com/endocrimes'>
-<img src='https://github.com/endocrimes.png' width='140px;'>
+<td id='luka-mirosevic'>
+<a href='https://github.com/lmirosevic'>
+<img src='https://github.com/lmirosevic.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/endocrimes'>Danielle Tomlinson</a></h4>
+<h4 align='center'><a href='https://twitter.com/lmirosevic'>Luka Mirosevic</a></h4>
 </td>
 <td id='josh-holtz'>
 <a href='https://github.com/joshdholtz'>
@@ -91,89 +91,89 @@ If the above doesn't help, please [submit an issue](https://github.com/fastlane/
 </a>
 <h4 align='center'><a href='https://twitter.com/joshdholtz'>Josh Holtz</a></h4>
 </td>
-<td id='matthew-ellis'>
-<a href='https://github.com/matthewellis'>
-<img src='https://github.com/matthewellis.png' width='140px;'>
+<td id='jérôme-lacoste'>
+<a href='https://github.com/lacostej'>
+<img src='https://github.com/lacostej.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/mellis1995'>Matthew Ellis</a></h4>
+<h4 align='center'><a href='https://twitter.com/lacostej'>Jérôme Lacoste</a></h4>
 </td>
 </tr>
 <tr>
-<td id='andrew-mcburney'>
-<a href='https://github.com/armcburney'>
-<img src='https://github.com/armcburney.png' width='140px;'>
-</a>
-<h4 align='center'><a href='https://twitter.com/armcburney'>Andrew McBurney</a></h4>
-</td>
 <td id='daniel-jankowski'>
 <a href='https://github.com/mollyIV'>
 <img src='https://github.com/mollyIV.png' width='140px;'>
 </a>
 <h4 align='center'><a href='https://twitter.com/mollyIV'>Daniel Jankowski</a></h4>
 </td>
-<td id='łukasz-grabowski'>
-<a href='https://github.com/lucgrabowski'>
-<img src='https://github.com/lucgrabowski.png' width='140px;'>
+<td id='helmut-januschka'>
+<a href='https://github.com/hjanuschka'>
+<img src='https://github.com/hjanuschka.png' width='140px;'>
 </a>
-<h4 align='center'>Łukasz Grabowski</h4>
+<h4 align='center'><a href='https://twitter.com/hjanuschka'>Helmut Januschka</a></h4>
 </td>
-<td id='jimmy-dee'>
-<a href='https://github.com/jdee'>
-<img src='https://github.com/jdee.png' width='140px;'>
+<td id='andrew-mcburney'>
+<a href='https://github.com/armcburney'>
+<img src='https://github.com/armcburney.png' width='140px;'>
 </a>
-<h4 align='center'>Jimmy Dee</h4>
+<h4 align='center'><a href='https://twitter.com/armcburney'>Andrew McBurney</a></h4>
 </td>
-<td id='kohki-miki'>
-<a href='https://github.com/giginet'>
-<img src='https://github.com/giginet.png' width='140px;'>
+<td id='iulian-onofrei'>
+<a href='https://github.com/revolter'>
+<img src='https://github.com/revolter.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/giginet'>Kohki Miki</a></h4>
+<h4 align='center'><a href='https://twitter.com/Revolt666'>Iulian Onofrei</a></h4>
+</td>
+<td id='jorge-revuelta-h'>
+<a href='https://github.com/minuscorp'>
+<img src='https://github.com/minuscorp.png' width='140px;'>
+</a>
+<h4 align='center'><a href='https://twitter.com/minuscorp'>Jorge Revuelta H</a></h4>
 </td>
 </tr>
 <tr>
-<td id='olivier-halligon'>
-<a href='https://github.com/AliSoftware'>
-<img src='https://github.com/AliSoftware.png' width='140px;'>
+<td id='connor-tumbleson'>
+<a href='https://github.com/ibotpeaches'>
+<img src='https://github.com/ibotpeaches.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/aligatr'>Olivier Halligon</a></h4>
+<h4 align='center'><a href='https://twitter.com/ibotpeaches'>Connor Tumbleson</a></h4>
 </td>
-<td id='aaron-brager'>
-<a href='https://github.com/getaaron'>
-<img src='https://github.com/getaaron.png' width='140px;'>
+<td id='jan-piotrowski'>
+<a href='https://github.com/janpio'>
+<img src='https://github.com/janpio.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/getaaron'>Aaron Brager</a></h4>
+<h4 align='center'><a href='https://twitter.com/Sujan'>Jan Piotrowski</a></h4>
 </td>
-<td id='joshua-liebowitz'>
-<a href='https://github.com/taquitos'>
-<img src='https://github.com/taquitos.png' width='140px;'>
+<td id='stefan-natchev'>
+<a href='https://github.com/snatchev'>
+<img src='https://github.com/snatchev.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/taquitos'>Joshua Liebowitz</a></h4>
+<h4 align='center'><a href='https://twitter.com/snatchev'>Stefan Natchev</a></h4>
 </td>
-<td id='manu-wallner'>
-<a href='https://github.com/milch'>
-<img src='https://github.com/milch.png' width='140px;'>
+<td id='manish-rathi'>
+<a href='https://github.com/crazymanish'>
+<img src='https://github.com/crazymanish.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/acrooow'>Manu Wallner</a></h4>
+<h4 align='center'><a href='https://twitter.com/iammanishrathi'>Manish Rathi</a></h4>
 </td>
-<td id='luka-mirosevic'>
-<a href='https://github.com/lmirosevic'>
-<img src='https://github.com/lmirosevic.png' width='140px;'>
+<td id='olivier-halligon'>
+<a href='https://github.com/AliSoftware'>
+<img src='https://github.com/AliSoftware.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/lmirosevic'>Luka Mirosevic</a></h4>
+<h4 align='center'><a href='https://twitter.com/aligatr'>Olivier Halligon</a></h4>
 </td>
 </tr>
 <tr>
-<td id='jérôme-lacoste'>
-<a href='https://github.com/lacostej'>
-<img src='https://github.com/lacostej.png' width='140px;'>
+<td id='roger-oba'>
+<a href='https://github.com/rogerluan'>
+<img src='https://github.com/rogerluan.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/lacostej'>Jérôme Lacoste</a></h4>
+<h4 align='center'><a href='https://twitter.com/rogerluan_'>Roger Oba</a></h4>
 </td>
-<td id='iulian-onofrei'>
-<a href='https://github.com/revolter'>
-<img src='https://github.com/revolter.png' width='140px;'>
+<td id='maksym-grebenets'>
+<a href='https://github.com/mgrebenets'>
+<img src='https://github.com/mgrebenets.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/Revolt666'>Iulian Onofrei</a></h4>
+<h4 align='center'><a href='https://twitter.com/mgrebenets'>Maksym Grebenets</a></h4>
 </td>
 <td id='felix-krause'>
 <a href='https://github.com/KrauseFx'>
@@ -181,31 +181,31 @@ If the above doesn't help, please [submit an issue](https://github.com/fastlane/
 </a>
 <h4 align='center'><a href='https://twitter.com/KrauseFx'>Felix Krause</a></h4>
 </td>
-<td id='stefan-natchev'>
-<a href='https://github.com/snatchev'>
-<img src='https://github.com/snatchev.png' width='140px;'>
+<td id='łukasz-grabowski'>
+<a href='https://github.com/lucgrabowski'>
+<img src='https://github.com/lucgrabowski.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/snatchev'>Stefan Natchev</a></h4>
+<h4 align='center'>Łukasz Grabowski</h4>
 </td>
-<td id='maksym-grebenets'>
-<a href='https://github.com/mgrebenets'>
-<img src='https://github.com/mgrebenets.png' width='140px;'>
+<td id='joshua-liebowitz'>
+<a href='https://github.com/taquitos'>
+<img src='https://github.com/taquitos.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/mgrebenets'>Maksym Grebenets</a></h4>
+<h4 align='center'><a href='https://twitter.com/taquitos'>Joshua Liebowitz</a></h4>
 </td>
 </tr>
 <tr>
-<td id='fumiya-nakamura'>
-<a href='https://github.com/nafu'>
-<img src='https://github.com/nafu.png' width='140px;'>
+<td id='satoshi-namai'>
+<a href='https://github.com/ainame'>
+<img src='https://github.com/ainame.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/nafu003'>Fumiya Nakamura</a></h4>
+<h4 align='center'><a href='https://twitter.com/ainame'>Satoshi Namai</a></h4>
 </td>
-<td id='jan-piotrowski'>
-<a href='https://github.com/janpio'>
-<img src='https://github.com/janpio.png' width='140px;'>
+<td id='max-ott'>
+<a href='https://github.com/max-ott'>
+<img src='https://github.com/max-ott.png' width='140px;'>
 </a>
-<h4 align='center'><a href='https://twitter.com/Sujan'>Jan Piotrowski</a></h4>
+<h4 align='center'><a href='https://twitter.com/ott_max'>Max Ott</a></h4>
 </td>
 </table>
 

data/credentials_manager/lib/credentials_manager/appfile_config.rb

@@ -123,6 +123,10 @@ module CredentialsManager
       setter(:itc_provider, *args, &block)
     end
 
+    def provider_public_id(*args, &block)
+      setter(:provider_public_id, *args, &block)
+    end
+
     # Android
     def json_key_file(*args, &block)
       setter(:json_key_file, *args, &block)

data/deliver/lib/deliver/detect_values.rb

@@ -58,11 +58,13 @@ module Deliver
     def find_folders(options)
       containing = Helper.fastlane_enabled? ? FastlaneCore::FastlaneFolder.path : '.'
       options[:screenshots_path] ||= File.join(containing, 'screenshots')
+      options[:app_previews_path] ||= File.join(containing, 'app-previews')
       options[:metadata_path] ||= File.join(containing, 'metadata')
     end
 
     def ensure_folders_created(options)
       FileUtils.mkdir_p(options[:screenshots_path])
+      FileUtils.mkdir_p(options[:app_previews_path])
       FileUtils.mkdir_p(options[:metadata_path])
     end
 

data/deliver/lib/deliver/options.rb

@@ -128,6 +128,22 @@ module Deliver
                                      description: "Path to the folder containing the screenshots",
                                      optional: true),
 
+        # app previews (videos)
+        FastlaneCore::ConfigItem.new(key: :app_previews_path,
+                                     env_name: "DELIVER_APP_PREVIEWS_PATH",
+                                     description: "Path to the folder containing localized App Preview videos",
+                                     optional: true),
+        FastlaneCore::ConfigItem.new(key: :preview_frame_time_code,
+                                     env_name: "DELIVER_PREVIEW_FRAME_TIME_CODE",
+                                     description: "Time code for the App Preview still frame written as hour:minute:second:centisecond (e.g. 00:00:00:01)",
+                                     optional: true,
+                                     default_value: "00:00:05:00"),
+        FastlaneCore::ConfigItem.new(key: :overwrite_preview_videos,
+                                     env_name: "DELIVER_OVERWRITE_PREVIEW_VIDEOS",
+                                     description: "Clear all previously uploaded App Preview videos before uploading the new ones",
+                                     type: Boolean,
+                                     default_value: false),
+
         # skip
         FastlaneCore::ConfigItem.new(key: :skip_binary_upload,
                                      env_name: "DELIVER_SKIP_BINARY_UPLOAD",
@@ -307,6 +323,13 @@ module Deliver
                                      code_gen_sensitive: true,
                                      default_value: CredentialsManager::AppfileConfig.try_fetch_value(:itc_provider),
                                      default_value_dynamic: true),
+        FastlaneCore::ConfigItem.new(key: :provider_public_id,
+                                     env_name: "DELIVER_PROVIDER_PUBLIC_ID",
+                                     description: "The provider public ID to be used with altool (--provider-public-id). This value will override the automatically detected provider value for altool uploads. Required after Xcode 26 when your account is associated with multiple providers and using username/app-password authentication",
+                                     optional: true,
+                                     code_gen_sensitive: true,
+                                     default_value: CredentialsManager::AppfileConfig.try_fetch_value(:provider_public_id),
+                                     default_value_dynamic: true),
         # rubocop:enable Layout/LineLength
 
         # precheck

data/deliver/lib/deliver/runner.rb

@@ -11,6 +11,7 @@ require_relative 'upload_price_tier'
 require_relative 'upload_metadata'
 require_relative 'upload_screenshots'
 require_relative 'sync_screenshots'
+require_relative 'sync_app_previews'
 require_relative 'detect_values'
 
 module Deliver
@@ -158,6 +159,17 @@ module Deliver
         upload_screenshots.upload(options, screenshots)
       end
 
+      if options[:app_previews_path]
+        previews = Deliver::SyncAppPreviews.new(
+          app: Deliver.cache[:app],
+          platform: Spaceship::ConnectAPI::Platform.map(options[:platform]),
+          app_previews_path: options[:app_previews_path],
+          preview_frame_time_code: options[:preview_frame_time_code],
+          overwrite_preview_videos: options[:overwrite_preview_videos]
+        )
+        previews.sync_from_path
+      end
+
       UploadPriceTier.new.upload(options)
     end
 
@@ -265,7 +277,7 @@ module Deliver
 
     # If App Store Connect API token, use token.
     # If api_key is specified and it is an Individual API Key, don't use token but use username.
-    # If itc_provider was explicitly specified, use it.
+    # If itc_provider or provider_public_id was explicitly specified, use it.
     # If there are multiple teams, infer the provider from the selected team name.
     # If there are fewer than two teams, don't infer the provider.
     def transporter_for_selected_team
@@ -281,23 +293,16 @@ module Deliver
                   api_key
                 end
 
-      # Currently no kind of transporters accept an Individual API Key. Use username and app-specific password instead.
-      # See https://github.com/fastlane/fastlane/issues/22115
-      is_individual_key = !api_key.nil? && api_key[:issuer_id].nil?
-      if is_individual_key
-        api_key = nil
-        api_token = nil
-      end
-
       unless api_token.nil?
         api_token.refresh! if api_token.expired?
-        return FastlaneCore::ItunesTransporter.new(nil, nil, false, nil, api_token.text, altool_compatible_command: true, api_key: api_key)
+        return FastlaneCore::ItunesTransporter.new(nil, nil, false, nil, api_token.text, altool_compatible_command: true, api_key: api_key, provider_public_id: nil)
       end
 
       tunes_client = Spaceship::ConnectAPI.client.tunes_client
 
-      generic_transporter = FastlaneCore::ItunesTransporter.new(options[:username], nil, false, options[:itc_provider], altool_compatible_command: true, api_key: api_key)
-      return generic_transporter unless options[:itc_provider].nil? && tunes_client.teams.count > 1
+      generic_transporter = FastlaneCore::ItunesTransporter.new(options[:username], nil, false, options[:itc_provider], altool_compatible_command: true, api_key: api_key, provider_public_id: options[:provider_public_id])
+      return generic_transporter if options[:itc_provider] || options[:provider_public_id] || tunes_client.nil?
+      return generic_transporter unless tunes_client.teams.count > 1
 
       begin
         team = tunes_client.teams.find { |t| t['providerId'].to_s == tunes_client.team_id }

data/deliver/lib/deliver/sync_app_previews.rb

@@ -0,0 +1,204 @@
+require "fastlane_core"
+require "fastlane_core/video_utils"
+require "digest/md5"
+
+require_relative 'module'
+
+module Deliver
+  class SyncAppPreviews
+    UploadPreviewJob = Struct.new(:localization, :preview_set, :video_path, :frame_time_code)
+
+    def initialize(app:, platform:, app_previews_path:, preview_frame_time_code: nil, overwrite_preview_videos: false)
+      @app = app
+      @platform = platform
+      @app_previews_path = app_previews_path
+      @frame_time_code = preview_frame_time_code
+      @overwrite = overwrite_preview_videos
+    end
+
+    def sync_from_path
+      UI.important("Uploading App Preview videos...")
+      validate_path!
+
+      localizations = editable_version.get_app_store_version_localizations
+      locale_by_code = localizations.each_with_object({}) { |l, h| h[l.locale] = l }
+
+      video_previews_per_locale = discover_videos(@app_previews_path)
+      if video_previews_per_locale.empty?
+        UI.message("No preview videos found under '#{@app_previews_path}'.")
+        return
+      end
+
+      jobs = []
+      video_previews_per_locale.each do |locale, previews|
+        process_locale_videos(locale: locale, previews: previews, locale_by_code: locale_by_code, jobs: jobs)
+      end
+
+      if jobs.empty?
+        UI.message("No new preview videos to upload.")
+        return
+      end
+
+      UI.message("Queueing #{jobs.size} preview video upload job(s) across #{video_previews_per_locale.keys.size} locale(s)...")
+
+      upload_errors = []
+      worker = FastlaneCore::QueueWorker.new do |job|
+        begin
+          UI.message("Uploading preview video '#{File.basename(job.video_path)}' for locale #{job.localization.locale} (set #{job.preview_set.preview_type})...")
+          job.preview_set.upload_preview(path: job.video_path, frame_time_code: job.frame_time_code)
+          UI.message("Uploaded preview video '#{File.basename(job.video_path)}' for locale #{job.localization.locale} (set #{job.preview_set.preview_type}).")
+        rescue => e
+          UI.error("Failed to upload '#{job.video_path}': #{e.class} - #{e.message}")
+          upload_errors << e
+        end
+      end
+
+      jobs.each { |j| worker.enqueue(j) }
+      worker.start
+      UI.message("All upload jobs finished. Sorting previews by filename...")
+
+      # sort previews in each set by file name
+      localizations.each do |loc|
+        loc.get_app_preview_sets(includes: "appPreviews").each do |set|
+          next unless set.app_previews && set.app_previews.length > 1
+          ordered_ids = set.app_previews.sort_by { |preview| preview.file_name.to_s }.map(&:id)
+          set.reorder_previews(app_preview_ids: ordered_ids)
+        end
+      end
+
+      unless upload_errors.empty?
+        UI.user_error!("#{upload_errors.size} App Preview upload error(s) occurred. First error: #{upload_errors.first.class} - #{upload_errors.first.message}")
+      end
+
+      UI.success("Successfully uploaded and sorted App Preview videos.")
+    end
+
+    private
+
+    # process videos for a single locale: enforce limits, create/reuse sets, enqueue upload jobs
+    def process_locale_videos(locale:, previews:, locale_by_code:, jobs:)
+      localization = locale_by_code[locale]
+      unless localization
+        UI.important("Locale '#{locale}' does not exist on App Store Connect for this version. Skipping its videos.")
+        return
+      end
+
+      sets_by_preview_type = localization
+                             .get_app_preview_sets(includes: "appPreviews")
+                             .each_with_object({}) { |set, h| h[set.preview_type] = set }
+
+      if @overwrite
+        delete_existing_previews(localization, sets_by_preview_type.values)
+        # re-fetch sets after deletes
+        sets_by_preview_type = localization
+                               .get_app_preview_sets(includes: "appPreviews")
+                               .each_with_object({}) { |set, h| h[set.preview_type] = set }
+      end
+
+      # group videos by preview type to enforce a max of 3 per locale AND type
+      videos_by_preview_type = Hash.new { |h, k| h[k] = [] }
+      previews.each { |item| videos_by_preview_type[item[:preview_type]] << item[:path] }
+
+      videos_by_preview_type.each do |preview_type, video_paths|
+        video_paths.sort!
+        if video_paths.size > 3
+          UI.important("[#{locale}] Found #{video_paths.size} '#{preview_type}' videos. Limiting to first 3 by filename.")
+          video_paths = video_paths.first(3)
+        end
+
+        preview_set = sets_by_preview_type[preview_type] || begin
+          UI.message("[#{locale}] Creating App Preview Set for type #{preview_type}...")
+          created = localization.create_app_preview_set(attributes: { previewType: preview_type })
+          sets_by_preview_type[preview_type] = created
+        end
+
+        video_paths.each do |video_path|
+          already_exist = (preview_set.app_previews || []).any? { |preview| preview.source_file_checksum == Digest::MD5.file(video_path).hexdigest }
+          if already_exist
+            UI.message("[#{locale}] Preview '#{File.basename(video_path)}' already uploaded (matching checksum). Skipping upload.")
+            next
+          end
+
+          jobs << UploadPreviewJob.new(localization, preview_set, video_path, @frame_time_code)
+        end
+      end
+    end
+
+    def validate_path!
+      UI.user_error!("app_previews_path is required") if @app_previews_path.to_s.empty?
+      UI.user_error!("app_previews_path '#{@app_previews_path}' does not exist") unless Dir.exist?(@app_previews_path)
+    end
+
+    def editable_version
+      @app.get_edit_app_store_version(platform: @platform)
+    end
+
+    def discover_videos(root)
+      extensions = %w[mp4 mov m4v]
+      locales = Dir.children(root).select { |subdir| File.directory?(File.join(root, subdir)) }
+      all_videos = {}
+      locales.each do |locale|
+        dir = File.join(root, locale)
+        video_paths = Dir.children(dir)
+                         .select { |filename| extensions.include?(File.extname(filename).delete(".").downcase) }
+                         .map { |filename| File.join(dir, filename) }
+                         .sort
+        valid_previews = []
+        video_paths.each do |path|
+          # require filename to contain a known preview type token
+          preview_type = Spaceship::ConnectAPI::AppPreviewSet.preview_type_from_filename(File.basename(path))
+          unless preview_type
+            UI.important("[#{locale}] '#{File.basename(path)}' does not contain any known preview device type. Skipping.")
+            next
+          end
+
+          # enforce size constraint (under 500MB)
+          size_mb = File.size(path) / (1024.0 * 1024.0)
+          if size_mb > 500
+            UI.important("[#{locale}] '#{File.basename(path)}' is #{size_mb.round(1)}MB (> 500MB). Skipping.")
+            next
+          end
+
+          # enforce duration constraints [15s..30s]. warn if duration can't be determined
+          duration = FastlaneCore::VideoUtils.read_video_duration_seconds(path)
+          if duration
+            if duration < 15.0 || duration > 30.0
+              UI.important("[#{locale}] '#{File.basename(path)}' duration is #{duration.round(2)}s (allowed: 15–30s). Skipping.")
+              next
+            end
+          else
+            UI.important("[#{locale}] Could not determine duration for '#{File.basename(path)}'. Proceeding anyway.")
+          end
+
+          # validate resolution against accepted canonical sizes; warn if resolution can't be determined
+          res = FastlaneCore::VideoUtils.read_video_resolution(path)
+          if res
+            unless Spaceship::ConnectAPI::AppPreviewSet.validate_video_resolution(res[0], res[1], preview_type)
+              UI.important("[#{locale}] '#{File.basename(path)}' has invalid resolution #{res.join('x')}. Skipping.")
+              next
+            end
+          else
+            UI.important("[#{locale}] Could not determine resolution for '#{File.basename(path)}'. Proceeding anyway.")
+          end
+          valid_previews << { path: path, preview_type: preview_type }
+        end
+        all_videos[locale] = valid_previews unless valid_previews.empty?
+      end
+      all_videos
+    end
+
+    def delete_existing_previews(localization, sets)
+      sets.each do |set|
+        next unless set.app_previews && set.app_previews.any?
+        UI.message("Deleting #{set.app_previews.size} existing previews from set #{set.preview_type} for locale #{localization.locale} due to overwrite...")
+        set.app_previews.each do |preview|
+          begin
+            preview.delete!
+          rescue => e
+            UI.error("Failed to delete preview '#{preview.file_name}': #{e.class} - #{e.message}")
+          end
+        end
+      end
+    end
+  end
+end

data/fastlane/lib/fastlane/actions/app_store_connect_api_key.rb

@@ -56,7 +56,7 @@ module Fastlane
                                        description: "The key ID"),
           FastlaneCore::ConfigItem.new(key: :issuer_id,
                                        env_name: "APP_STORE_CONNECT_API_KEY_ISSUER_ID",
-                                       description: "The issuer ID. It can be nil if the key is individual API key",
+                                       description: "The issuer ID. It should be nil if the key is individual API key",
                                        optional: true),
           FastlaneCore::ConfigItem.new(key: :key_filepath,
                                        env_name: "APP_STORE_CONNECT_API_KEY_KEY_FILEPATH",
@@ -137,6 +137,10 @@ module Fastlane
             key_id: "D83848D23",
             issuer_id: "227b0bbf-ada8-458c-9d62-3d8022b7d07f",
             key_content: "-----BEGIN EC PRIVATE KEY-----\nfewfawefawfe\n-----END EC PRIVATE KEY-----"
+          )',
+          'app_store_connect_api_key(
+            key_id: "D83848D23", # no issuer_id if the key is individual
+            key_content: "-----BEGIN EC PRIVATE KEY-----\nfewfawefawfe\n-----END EC PRIVATE KEY-----"
           )'
         ]
       end