pindo 5.18.17 → 5.19.1

data/lib/pindo/config/ios_config_parser.rb

@@ -1,5 +1,6 @@
 require 'singleton'
 require 'json'
+require 'pindo/base/informative'
 
 module Pindo
   # iOS 配置解析器（单例模式）
@@ -21,7 +22,32 @@ module Pindo
   class IosConfigParser
     include Singleton
 
-    attr_reader :config_file_path, :config_json, :app_repo_name
+    # AdHoc 合并时需要替换前缀的所有 Bundle ID 字段
+    ADHOC_BUNDLE_ID_KEYS = %w[
+      app_identifier
+      app_identifier_pushcontent
+      app_identifier_pushservice
+      app_identifier_keyboard
+      app_identifier_imessage
+      app_identifier_extension
+      app_identifier_siri
+      app_identifier_siriui
+      app_identifier_widget
+      app_identifier_extensionad
+      app_identifier_extensionporn
+      app_identifier_watchapp
+      app_identifier_watchapp_extension
+    ].freeze
+
+    # AdHoc 合并 app_setting 时需要保留原值（服务器/URL）的 key
+    ADHOC_EXCLUDED_APP_SETTING_KEYS = %w[
+      kGUKeyAppClientHost
+      kGUKeyResBaseUrl
+      app_web_host
+      app_client_url
+    ].freeze
+
+    attr_reader :config_file_path, :config_json, :app_repo_name, :adhoc_config_dir
 
     def initialize
       @config_file_path = nil
@@ -29,6 +55,8 @@ module Pindo
       @config_data = {}
       @overrides = {}
       @app_repo_name = nil  # 保存原始 Bundle ID，用于配置仓库路径
+      @adhoc_merged = false # 是否已合并 AdHoc 配置（幂等标志）
+      @adhoc_config_dir = nil # 已合并的 AdHoc 配置仓库目录（缓存，避免重复克隆）
     end
 
     # 加载配置文件
@@ -47,6 +75,11 @@ module Pindo
         raise "配置文件格式错误: #{e.message}"
       end
 
+      # 从磁盘重新加载的配置是未合并的，重置 AdHoc 标志，
+      # 否则后续 merge 会被幂等短路错误跳过，导致用原始配置构建
+      @adhoc_merged = false
+      @adhoc_config_dir = nil
+
       # 解析配置
       parse_config_data
 
@@ -69,6 +102,14 @@ module Pindo
       @config_data = {}
       @overrides = {}
       @app_repo_name = nil
+      @adhoc_merged = false
+      @adhoc_config_dir = nil
+    end
+
+    # AdHoc 配置是否已合并（幂等短路用）
+    # @return [Boolean]
+    def adhoc_merged?
+      @adhoc_merged == true
     end
 
     # ==================== 账号信息 ====================
@@ -270,6 +311,34 @@ module Pindo
       @config_data[:ios_deployment_target]
     end
 
+    # ==================== Unity AppConfig 映射 ====================
+
+    # 提取需要写入 Unity 工程 AppConfig（TextAsset JSON）的字段
+    # 仅包含在 config.json 中有明确来源的项，已去除 nil/空值
+    # @return [Hash{String=>String}] AppConfig 键 => 值
+    def unity_appconfig_values
+      return {} unless loaded?
+
+      app_info = @config_json['app_info'] || {}
+      app_setting = @config_json['app_setting'] || {}
+
+      {
+        "apple_app_id"        => app_info['app_id_ios'],
+        "appsflyer_dev_key"   => app_setting['kGUKeyAppsFlyerDevKey'],
+        "support_email"       => app_setting['kGUKeyAppSupportEmail'],
+        "fb_app_id"           => app_info['facebook_app_id'],
+        "fb_client_token"     => app_info['facebook_client_token'],
+        "fb_app_label"        => app_info['app_display_name'],
+        "mecha_host"          => app_setting['kGUKeyAppClientHost'],
+        "mecha_app_id"        => app_setting['kGUKeyAppPlatform'],
+        "google_web_client_id" => app_setting['kGUKeyGoogleWebClientId'],
+        "google_ios_client_id" => app_setting['kGUKeyGoogleIosClientId']
+      }.each_with_object({}) do |(key, value), result|
+        next if value.nil? || value.to_s.strip.empty?
+        result[key] = value.to_s.strip
+      end
+    end
+
     # ==================== 配置验证 ====================
 
     # 验证必需的配置项是否存在
@@ -297,151 +366,38 @@ module Pindo
     # ==================== AdHoc 发布配置合并 ====================
 
     # 合并 AdHoc 发布配置
-    # 1. 替换所有 Bundle ID 为 AdHoc 配置的 Bundle ID（确保使用 AdHoc 证书）
-    # 2. 合并第三方 SDK 配置（Facebook、AdMob、AppLovin）
-    # 3. 合并 app_setting（排除主机和 URL 配置）
+    # 1. 替换 Apple ID / 公司名为 AdHoc 配置值
+    # 2. 替换所有 Bundle ID（含 Extension、App Group、iCloud）为 AdHoc 前缀（确保使用 AdHoc 证书）
+    # 3. 合并第三方 SDK 配置（Facebook、AdMob、AppLovin）
+    # 4. 合并 app_setting（排除主机和 URL 配置）
+    #
+    # 幂等：已合并则直接短路，不再二次改写。
+    # 失败采用 fail-fast：配置文件缺失/解析失败/缺关键字段时抛 Informative，
+    # 由调用方决定是否中断，避免静默退化为"原始配置"产出错误身份的包。
+    #
     # @param adhoc_config_dir [String] AdHoc 配置仓库目录
-    # @return [Boolean] 是否成功
+    # @return [Boolean] 是否成功合并
+    # @raise [Pindo::Informative] AdHoc 配置无效或缺失关键字段
     def modify_config_with_adhoc(adhoc_config_dir:)
-      return false unless loaded?
-      return false if adhoc_config_dir.nil? || adhoc_config_dir.empty?
-
-      begin
-        # 1. 读取 AdHoc 配置
-        adhoc_config_file = File.join(adhoc_config_dir, "config.json")
-        unless File.exist?(adhoc_config_file)
-          puts "AdHoc 配置文件不存在: #{adhoc_config_file}"
-          return false
-        end
-
-        adhoc_config_json = JSON.parse(File.read(adhoc_config_file))
-
-        # 1.5 替换 Apple ID（使用 AdHoc 配置的 Apple ID）
-        if adhoc_config_json.dig('account_info', 'apple_acount_id')
-          original_apple_id = @config_json.dig('account_info', 'apple_acount_id')&.strip  # 去除前后空格
-          adhoc_apple_id = adhoc_config_json['account_info']['apple_acount_id']&.strip  # 去除前后空格
-
-          if original_apple_id && adhoc_apple_id && !adhoc_apple_id.empty? && original_apple_id != adhoc_apple_id
-            @config_json['account_info']['apple_acount_id'] = adhoc_apple_id
-            puts "  替换 Apple ID:"
-            puts "    原始 Apple ID: #{original_apple_id}"
-            puts "    AdHoc Apple ID: #{adhoc_apple_id}"
-          end
-
-          # 如果 AdHoc 配置包含 company_name，也一并替换
-          if adhoc_config_json.dig('account_info', 'acount_company_name')
-            company_name = adhoc_config_json['account_info']['acount_company_name']&.strip  # 去除前后空格
-            @config_json['account_info']['acount_company_name'] = company_name if company_name && !company_name.empty?
-          end
-        end
-
-        # 2. 替换所有 Bundle ID（使用 AdHoc 配置的 Bundle ID）
-        original_bundle_id = @config_json.dig('app_info', 'app_identifier')
-        adhoc_bundle_id = adhoc_config_json.dig('app_info', 'app_identifier')
-
-        if original_bundle_id && adhoc_bundle_id && original_bundle_id != adhoc_bundle_id
-          puts "  替换 Bundle ID 前缀:"
-          puts "    原始前缀: #{original_bundle_id}"
-          puts "    AdHoc 前缀: #{adhoc_bundle_id}"
-
-          # Bundle ID 字段映射表
-          bundle_id_keys = [
-            'app_identifier',              # 主应用
-            'app_identifier_pushcontent',  # Push Content Extension
-            'app_identifier_pushservice',  # Push Service Extension
-            'app_identifier_keyboard',     # Keyboard Extension
-            'app_identifier_imessage',     # iMessage Extension
-            'app_identifier_extension',    # 通用 Extension
-            'app_identifier_siri',         # Siri Extension
-            'app_identifier_siriui',       # Siri UI Extension
-            'app_identifier_widget',       # Widget Extension
-            'app_identifier_extensionad',  # 广告 Extension
-            'app_identifier_extensionporn',# Porn Extension
-            'app_identifier_watchapp',     # Watch App
-            'app_identifier_watchapp_extension' # Watch App Extension
-          ]
-
-          # 遍历所有 Bundle ID 字段并替换前缀
-          bundle_id_keys.each do |key|
-            current_value = @config_json.dig('app_info', key)
-            next if current_value.nil? || current_value.empty?
-
-            # 如果当前值以原始主 Bundle ID 开头，替换为 AdHoc Bundle ID
-            if current_value.start_with?(original_bundle_id)
-              # 计算后缀部分（如 .extension、.keyboard 等）
-              suffix = current_value[original_bundle_id.length..-1]
-              new_value = adhoc_bundle_id + suffix
-
-              @config_json['app_info'][key] = new_value
-              puts "    ✓ #{key}: #{current_value} → #{new_value}"
-            end
-          end
-        end
-
-        # 2.5 替换 App Group ID 和 iCloud Container ID
-        if original_bundle_id && adhoc_bundle_id && original_bundle_id != adhoc_bundle_id
-          # 处理 app_group_id (格式: group.com.example.app)
-          if @config_json.dig('app_setting', 'app_group_id')
-            current_group_id = @config_json['app_setting']['app_group_id']
-            if current_group_id.include?(original_bundle_id)
-              new_group_id = current_group_id.gsub(original_bundle_id, adhoc_bundle_id)
-              @config_json['app_setting']['app_group_id'] = new_group_id
-              puts "    ✓ app_group_id: #{current_group_id} → #{new_group_id}"
-            end
-          end
-
-          # 处理 app_icloud_id (格式: iCloud.com.example.app)
-          if @config_json.dig('app_setting', 'app_icloud_id')
-            current_icloud_id = @config_json['app_setting']['app_icloud_id']
-            if current_icloud_id.include?(original_bundle_id)
-              new_icloud_id = current_icloud_id.gsub(original_bundle_id, adhoc_bundle_id)
-              @config_json['app_setting']['app_icloud_id'] = new_icloud_id
-              puts "    ✓ app_icloud_id: #{current_icloud_id} → #{new_icloud_id}"
-            end
-          end
-        end
-
-        # 3. 合并第三方 SDK 配置
-        # Facebook 配置
-        if adhoc_config_json.dig('app_info', 'facebook_app_id')
-          @config_json['app_info']['facebook_app_id'] = adhoc_config_json['app_info']['facebook_app_id']
-          @config_json['app_info']['facebook_client_token'] = adhoc_config_json['app_info']['facebook_client_token']
-        end
-
-        # AdMob 配置
-        if @config_json.dig('app_info', 'admob_app_id') && adhoc_config_json.dig('app_info', 'admob_app_id')
-          @config_json['app_info']['admob_app_id'] = adhoc_config_json['app_info']['admob_app_id']
-        end
-
-        # AppLovin 配置
-        if @config_json.dig('app_info', 'applovin_app_id') && adhoc_config_json.dig('app_info', 'applovin_app_id')
-          @config_json['app_info']['applovin_app_id'] = adhoc_config_json['app_info']['applovin_app_id']
-        end
+      raise Informative, "配置未加载，无法合并 AdHoc 配置" unless loaded?
+      return true if adhoc_merged?  # 幂等短路
+      raise Informative, "AdHoc 配置目录无效" if adhoc_config_dir.nil? || adhoc_config_dir.empty?
 
-        # 4. 合并 app_setting（排除主机和 URL 配置）
-        excluded_keys = ['kGUKeyAppClientHost', 'kGUKeyResBaseUrl', 'app_web_host', 'app_client_url']
+      adhoc_config_json = read_adhoc_config(adhoc_config_dir)
 
-        if @config_json['app_setting'] && adhoc_config_json['app_setting']
-          @config_json['app_setting'].each_key do |key|
-            next if excluded_keys.include?(key)
+      merge_adhoc_apple_account(adhoc_config_json)
+      replace_adhoc_bundle_identifiers(adhoc_config_json)
+      merge_adhoc_sdk_config(adhoc_config_json)
+      merge_adhoc_app_setting(adhoc_config_json)
 
-            if adhoc_config_json['app_setting'][key]
-              @config_json['app_setting'][key] = adhoc_config_json['app_setting'][key]
-            end
-          end
-        end
-
-        # 5. 重新解析配置数据，同步更新 @config_data
-        # 这样 bundle_id、bundle_id_pushcontent 等属性方法才能返回替换后的值
-        parse_config_data
+      # 重新解析配置数据，同步更新 @config_data
+      # 这样 bundle_id、bundle_id_pushcontent 等属性方法才能返回替换后的值
+      parse_config_data
 
-        puts "✅ 已替换 Apple ID、Bundle ID 并合并 AdHoc 配置（Facebook、AdMob、AppLovin）"
-        true
-      rescue => e
-        puts "合并发布配置失败: #{e.message}"
-        puts e.backtrace.join("\n") if ENV['PINDO_DEBUG']
-        false
-      end
+      @adhoc_merged = true
+      @adhoc_config_dir = adhoc_config_dir
+      puts "✅ 已替换 Apple ID、Bundle ID 并合并 AdHoc 配置（Facebook、AdMob、AppLovin）"
+      true
     end
 
     # ==================== 调试和信息 ====================
@@ -483,6 +439,122 @@ module Pindo
 
     private
 
+    # ==================== AdHoc 合并辅助方法 ====================
+
+    # 读取并解析 AdHoc 配置仓库的 config.json（失败 fail-fast）
+    # @raise [Pindo::Informative] 文件不存在或 JSON 解析失败
+    def read_adhoc_config(adhoc_config_dir)
+      adhoc_config_file = File.join(adhoc_config_dir, "config.json")
+      unless File.exist?(adhoc_config_file)
+        raise Informative, "AdHoc 配置文件不存在: #{adhoc_config_file}"
+      end
+
+      JSON.parse(File.read(adhoc_config_file))
+    rescue JSON::ParserError => e
+      raise Informative, "AdHoc 配置文件解析失败: #{e.message}"
+    end
+
+    # 替换 Apple ID 与公司名
+    # 注：原配置无 account_info 容器则跳过（原代码此处会 NoMethodError，这里改为安全跳过）
+    def merge_adhoc_apple_account(adhoc_config_json)
+      return unless @config_json['account_info']
+
+      adhoc_account = adhoc_config_json['account_info'] || {}
+
+      adhoc_apple_id = adhoc_account['apple_acount_id']&.strip
+      if adhoc_apple_id && !adhoc_apple_id.empty?
+        original_apple_id = @config_json['account_info']['apple_acount_id']&.strip
+        if original_apple_id != adhoc_apple_id
+          @config_json['account_info']['apple_acount_id'] = adhoc_apple_id
+          puts "  替换 Apple ID:"
+          puts "    原始 Apple ID: #{original_apple_id}"
+          puts "    AdHoc Apple ID: #{adhoc_apple_id}"
+        end
+      end
+
+      company_name = adhoc_account['acount_company_name']&.strip
+      if company_name && !company_name.empty?
+        @config_json['account_info']['acount_company_name'] = company_name
+      end
+    end
+
+    # 替换主 Bundle ID、所有 Extension Bundle ID、App Group、iCloud Container
+    # @raise [Pindo::Informative] AdHoc 配置缺少主 Bundle ID
+    def replace_adhoc_bundle_identifiers(adhoc_config_json)
+      original_bundle_id = @config_json.dig('app_info', 'app_identifier')
+      adhoc_bundle_id = adhoc_config_json.dig('app_info', 'app_identifier')
+
+      if adhoc_bundle_id.nil? || adhoc_bundle_id.empty?
+        raise Informative, "AdHoc 配置缺少 app_identifier（主 Bundle ID），无法替换证书身份"
+      end
+      return if original_bundle_id.nil? || original_bundle_id == adhoc_bundle_id
+
+      puts "  替换 Bundle ID 前缀:"
+      puts "    原始前缀: #{original_bundle_id}"
+      puts "    AdHoc 前缀: #{adhoc_bundle_id}"
+
+      # 替换所有 app_identifier_* 字段：凡以原始主 Bundle ID 开头的，替换前缀保留后缀
+      ADHOC_BUNDLE_ID_KEYS.each do |key|
+        current_value = @config_json.dig('app_info', key)
+        next if current_value.nil? || current_value.empty?
+        next unless current_value.start_with?(original_bundle_id)
+
+        new_value = adhoc_bundle_id + current_value[original_bundle_id.length..-1]
+        @config_json['app_info'][key] = new_value
+        puts "    ✓ #{key}: #{current_value} → #{new_value}"
+      end
+
+      # App Group ID（group.xxx）与 iCloud Container ID（iCloud.xxx）内嵌的 Bundle ID
+      replace_adhoc_app_setting_id('app_group_id', original_bundle_id, adhoc_bundle_id)
+      replace_adhoc_app_setting_id('app_icloud_id', original_bundle_id, adhoc_bundle_id)
+    end
+
+    # 替换 app_setting 中内嵌了原始 Bundle ID 的标识符（app_group_id / app_icloud_id）
+    def replace_adhoc_app_setting_id(key, original_bundle_id, adhoc_bundle_id)
+      current_value = @config_json.dig('app_setting', key)
+      return if current_value.nil? || !current_value.include?(original_bundle_id)
+
+      new_value = current_value.gsub(original_bundle_id, adhoc_bundle_id)
+      @config_json['app_setting'][key] = new_value
+      puts "    ✓ #{key}: #{current_value} → #{new_value}"
+    end
+
+    # 合并第三方 SDK 配置（Facebook / AdMob / AppLovin）
+    # 统一规则：仅当 AdHoc 配置存在对应值时才覆盖，避免用 nil 抹掉原有有效值
+    def merge_adhoc_sdk_config(adhoc_config_json)
+      app_info = adhoc_config_json['app_info'] || {}
+
+      if app_info['facebook_app_id']
+        @config_json['app_info']['facebook_app_id'] = app_info['facebook_app_id']
+        # token 存在才覆盖，否则保留原有值
+        if app_info['facebook_client_token']
+          @config_json['app_info']['facebook_client_token'] = app_info['facebook_client_token']
+        end
+      end
+
+      if @config_json.dig('app_info', 'admob_app_id') && app_info['admob_app_id']
+        @config_json['app_info']['admob_app_id'] = app_info['admob_app_id']
+      end
+
+      if @config_json.dig('app_info', 'applovin_app_id') && app_info['applovin_app_id']
+        @config_json['app_info']['applovin_app_id'] = app_info['applovin_app_id']
+      end
+    end
+
+    # 合并 app_setting，逐 key 用 AdHoc 值覆盖原配置已有的 key
+    # 排除服务器/URL 相关 key，保留原环境后端地址（让测试包连真实后端）
+    def merge_adhoc_app_setting(adhoc_config_json)
+      adhoc_app_setting = adhoc_config_json['app_setting']
+      return unless @config_json['app_setting'] && adhoc_app_setting
+
+      @config_json['app_setting'].each_key do |key|
+        next if ADHOC_EXCLUDED_APP_SETTING_KEYS.include?(key)
+        next unless adhoc_app_setting[key]
+
+        @config_json['app_setting'][key] = adhoc_app_setting[key]
+      end
+    end
+
     # 解析配置数据
     def parse_config_data
       return unless @config_json

data/lib/pindo/module/android/android_build_helper.rb

@@ -6,6 +6,7 @@ require_relative 'java_env_helper'
 require_relative 'keystore_helper'
 require_relative 'workflow_gradle_injector'
 require 'fileutils'
+require 'find'
 
 module Pindo
   class AndroidBuildHelper
@@ -24,6 +25,10 @@ module Pindo
       raise ArgumentError, "项目目录不能为空" if project_dir.nil?
       raise ArgumentError, "项目目录不存在" unless File.directory?(project_dir)
 
+      # 构建前先清理 macOS 自动生成的 .DS_Store
+      # 避免其被带入 base / asset pack 模块的 assets/ 导致冲突，或引起 Gradle clean 在遍历删除时出现竞态
+      prune_ds_store_files!(project_dir)
+
       # 设置 Java Home 环境变量
       Pindo::JavaEnvHelper.setup_java_home_from_project(project_dir)
 
@@ -396,20 +401,19 @@ module Pindo
       puts '  AAB 构建前先执行 Gradle clean'
       gradle_clean_resilient!(project_path)
 
-      cmd = "cd \"#{project_path}\" && ./gradlew --no-daemon #{bundle_task}"
-      system(cmd)
+      # 数组参数 + chdir，避免 project_path 拼接进 shell
+      system("./gradlew", "--no-daemon", bundle_task, chdir: project_path)
     end
 
     # clean 失败（占用、.DS_Store 竞态等）时：停 Daemon → 再删 .DS_Store → 物理删除各模块 build/，与 Gradle clean 产物一致，不中断后续 bundle
     private def gradle_clean_resilient!(project_path)
-      prune_ds_store_before_gradle_clean!(project_path)
-      cmd = %(cd "#{project_path}" && ./gradlew --no-daemon clean)
-      return if system(cmd)
+      prune_ds_store_files!(project_path)
+      return if system("./gradlew", "--no-daemon", "clean", chdir: project_path)
 
       puts '  ⚠ Gradle clean 失败，正在自动恢复：--stop、清理 .DS_Store、删除各模块 build/ ...'
-      system(%(cd "#{project_path}" && ./gradlew --stop), out: File::NULL, err: File::NULL)
+      system("./gradlew", "--stop", chdir: project_path, out: File::NULL, err: File::NULL)
       sleep 0.3
-      prune_ds_store_before_gradle_clean!(project_path)
+      prune_ds_store_files!(project_path)
       remove_android_module_build_dirs!(project_path)
     end
 
@@ -445,18 +449,37 @@ module Pindo
       puts "  ⚠ 物理删除 build 目录时出现异常（将继续尝试 bundle）: #{e.message}"
     end
 
-    # macOS 在 clean 遍历删除 build 目录时 Finder 可能写入 .DS_Store，Gradle 会报 Unable to delete / New files were found
-    private def prune_ds_store_before_gradle_clean!(project_path)
-      return unless project_path && File.directory?(project_path)
-
-      system('find', project_path, '-name', '.DS_Store', '-type', 'f', '-delete',
-             out: File::NULL, err: File::NULL)
+    # 清理 macOS 自动生成的 .DS_Store 文件
+    # 作用：
+    # 1) 避免被带入 base 与 asset pack 模块的 assets/ 造成打包冲突
+    # 2) 避免 Gradle clean 遍历删除 build 目录时 Finder 回写 .DS_Store 触发
+    #    Unable to delete / New files were found 的竞态错误
+    private def prune_ds_store_files!(project_path)
+      return 0 unless project_path && File.directory?(project_path)
+
+      count = 0
+      Find.find(project_path) do |path|
+        next unless File.basename(path) == '.DS_Store'
+        next unless File.file?(path)
+
+        begin
+          File.delete(path)
+          count += 1
+        rescue StandardError
+          # 个别文件删除失败不影响整体流程
+          next
+        end
+      end
+      puts "  已清理 #{count} 个 .DS_Store 文件" if count.positive?
+      count
+    rescue StandardError => e
+      puts "  ⚠ 清理 .DS_Store 异常：#{e.message}"
+      0
     end
 
     def build_so_library(project_path)
-      # 编译so库 (Thread Safe)
-      cmd = "cd \"#{project_path}\" && ./gradlew unityLibrary:BuildIl2CppTask"
-      system(cmd)
+      # 编译so库 (Thread Safe)；数组参数 + chdir，避免 project_path 拼接进 shell
+      system("./gradlew", "unityLibrary:BuildIl2CppTask", chdir: project_path)
     end
 
     def copy_so_files(source_path, target_path)

data/lib/pindo/module/android/gradle_helper.rb

@@ -518,18 +518,16 @@ module Pindo
           return false
         end
 
-        # 3. 使用unzip -t测试ZIP文件完整性
-        test_cmd = "unzip -t '#{zip_file}' > /dev/null 2>&1"
-        system(test_cmd)
+        # 3. 使用unzip -t测试ZIP文件完整性（数组参数，避免路径注入）
+        system("unzip", "-t", zip_file, out: File::NULL, err: File::NULL)
       rescue => e
         false
       end
 
       # 使用curl下载文件
       def download_with_curl(url, output_path)
-        # 简化的curl命令，避免特殊字符问题
-        cmd = "curl -L -o '#{output_path}' '#{url}'"
-        system(cmd)
+        # 数组参数形式，避免 URL/路径中的 shell 元字符注入
+        system("curl", "-L", "-o", output_path, url)
       rescue => e
         false
       end
@@ -537,9 +535,8 @@ module Pindo
       # 解压Gradle分发包
       def extract_gradle_distribution(zip_file, extract_dir, gradle_version)
         begin
-          # 使用unzip命令解压
-          extract_cmd = "cd '#{File.dirname(extract_dir)}' && unzip -q '#{zip_file}'"
-          if system(extract_cmd)
+          # 使用unzip命令解压（数组参数 + chdir，避免路径注入）
+          if system("unzip", "-q", zip_file, chdir: File.dirname(extract_dir))
             # 设置执行权限
             gradle_script = File.join(extract_dir, "bin/gradle")
             if File.exist?(gradle_script)
@@ -577,11 +574,8 @@ module Pindo
 
       # 使用官方Gradle命令生成wrapper
       def generate_gradle_wrapper_official(project_path, gradle_version)
-        Dir.chdir(project_path) do
-          # 使用gradle wrapper命令
-          cmd = "gradle wrapper"
-          system(cmd)
-        end
+        # 数组参数 + chdir，避免 shell 解析
+        system("gradle", "wrapper", chdir: project_path)
       rescue => e
         false
       end