pindo 5.18.20 → 5.19.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 004b03010300629da1e55b7c6b34eada7cd76e55247deb8f76865d7446ef3572
-  data.tar.gz: 44757caa7ef3e6ae5253bd17d4a0c35ebe0c4cb6120d7ee2840c1a010b6800d2
+  metadata.gz: db02bf00df627b07423ebd53475c538b60a4d5d8d567b2bb81a61390aaae73bc
+  data.tar.gz: 5252fb968e7bde86a095a78dce92599c9614061c1da9666e8cf2bea03927c8c6
 SHA512:
-  metadata.gz: 05bd2c5264163a1ec12aa545e165f2bee0d832e2c038cd30d506c932842c024f9664d6ecd51c7aa80a71f061ffcaf4747a56caac6223b2c34fe4887845681fc6
-  data.tar.gz: 03d477b345f080c1dfe28e5367786d2147029e86aac185c4aa698b8d17acc98064bee17958060be7d1f44ebb5a41e2790a228a9a4b4699d5d9e2bb6f99709a1d
+  metadata.gz: fa95f45065b4a8635c4d12c5a4c1d3a95e678df97724103857e055d9088d8f8613fddef8786229163bbd8ad01c87242e396e5b79e1c5f4d92411b61192e3bde1
+  data.tar.gz: 931bd858133452f7cc5a3edafc60cbe7c4f56b5d0f8da6d720f24c23f7974cfe2c823e0bbc7e1d1a04bc8f99faa581ef7c52ae99b0676ec6cfc4e407411e3e05

data/lib/pindo/base/git_handler.rb

@@ -86,8 +86,7 @@ module Pindo
       end
 
       def add_branch(local_repo_dir: nil, branch: nil)
-
-          current=Dir.pwd
+          # 命令已带 -C，无需 Dir.chdir（并发模式下 chdir 会污染全局 cwd）
           result = false
 
           if !local_branch_exists?(local_repo_dir: local_repo_dir, branch: branch)
@@ -97,13 +96,11 @@ module Pindo
 
           git_remote!(local_repo_dir, %W(-C #{local_repo_dir} push origin #{branch}:#{branch}))
 
-          Dir.chdir(current)
           return result
       end
 
       def remove_branch(local_repo_dir: nil, branch: nil)
-          current=Dir.pwd
-
+          # 命令已带 -C，无需 Dir.chdir（并发模式下 chdir 会污染全局 cwd）
           result = false
           if File.exist?(local_repo_dir)
             current_branch = git!(%W(-C #{local_repo_dir} rev-parse --abbrev-ref HEAD)).strip
@@ -120,51 +117,40 @@ module Pindo
           else
             result = false
           end
-          Dir.chdir(current)
           return result
       end
 
 
       def local_branch_exists?(local_repo_dir: nil, branch: nil)
-          current=Dir.pwd
+          # 命令改带 -C，去掉 Dir.chdir（并发模式下 chdir 会污染全局 cwd）
           result = false
 
           if File.exist?(local_repo_dir)
-            Dir.chdir(local_repo_dir)
-
-            res_data = Executable.capture_command('git', %W(rev-parse --verify #{branch}), :capture => :out)
-            # puts "=====1"
-            # puts res_data
-            # res_data = git!(%W(-C #{local_repo_dir} --no-pager branch --list origin/#{branch} --no-color -r))
+            res_data = Executable.capture_command('git', %W(-C #{local_repo_dir} rev-parse --verify #{branch}), :capture => :out)
             result = !res_data.nil? && !res_data.empty?
           else
             result = false
           end
-          Dir.chdir(current)
           return result
       end
 
       def remote_branch_exists?(local_repo_dir: nil, branch: nil)
-          current=Dir.pwd
+          # 命令已带 -C，无需 Dir.chdir（并发模式下 chdir 会污染全局 cwd）
           result = false
           if File.exist?(local_repo_dir)
-            Dir.chdir(local_repo_dir)
             res_data = git_remote!(local_repo_dir, %W(-C #{local_repo_dir} ls-remote --heads origin refs/heads/#{branch}))
             result = !res_data.nil? && !res_data.empty?
           else
             result = false
           end
-          Dir.chdir(current)
           return result
       end
 
 
       def add_tag(local_repo_dir: nil, tag_name: nil)
-
-          current=Dir.pwd
+          # 命令已带 -C，无需 Dir.chdir（并发模式下 chdir 会污染全局 cwd）
           result = false
           if File.exist?(local_repo_dir)
-            Dir.chdir(local_repo_dir)
             if !local_tag_exists?(local_repo_dir: local_repo_dir, tag_name: tag_name)
                 git!(%W(-C #{local_repo_dir} tag #{tag_name}))
                 result = true
@@ -173,7 +159,6 @@ module Pindo
           else
             result = false
           end
-          Dir.chdir(current)
           return result
       end
 
@@ -207,11 +192,9 @@ module Pindo
       end
 
       def remove_tag(local_repo_dir: nil, tag_name: nil)
-
+          # 命令已带 -C，无需 Dir.chdir（并发模式下 chdir 会污染全局 cwd）
           result = false
-          current=Dir.pwd
           if File.exist?(local_repo_dir)
-            Dir.chdir(local_repo_dir)
             if local_tag_exists?(local_repo_dir: local_repo_dir, tag_name: tag_name)
                 git!(%W(-C #{local_repo_dir} tag -d #{tag_name}))
                 result = true
@@ -223,7 +206,6 @@ module Pindo
           else
             result = false
           end
-          Dir.chdir(current)
           return result
 
       end
@@ -253,9 +235,7 @@ module Pindo
       end
 
       def git_latest_commit_id(local_repo_dir:nil)
-
-        current=Dir.pwd
-
+        # 命令已带 -C，无需 Dir.chdir（并发模式下 chdir 会污染全局 cwd）
         unless File.exist?(File::join(local_repo_dir, ".git"))
           return nil
         end
@@ -263,16 +243,9 @@ module Pindo
         commit_id = nil
 
         if File.exist?(local_repo_dir)
-          Dir.chdir(local_repo_dir)
           current_branch = git!(%W(-C #{local_repo_dir} rev-parse --abbrev-ref HEAD)).strip
-          # puts "current_branch: #{current_branch}"
-
-          # git log -n 1 --pretty=format:"commit %H"
-          # format_str = "commit %H"
-          # latest_log_info = git!(%W(-C #{current} log -n 1 --pretty=#{format_str} #{current_branch})).strip
           commit_id = git!(%W(-C #{local_repo_dir} rev-parse #{current_branch})).strip
         end
-        Dir.chdir(current)
 
         return commit_id
       end
@@ -362,9 +335,11 @@ module Pindo
         rescue StandardError => e
           Funlog.instance.fancyinfo_error("仓库#{local_repo_dir}更新失败!")
           raise Informative, e.to_s
+        ensure
+          # 无论成功失败都还原 cwd，避免异常路径下污染全局工作目录
+          Dir.chdir(current)
         end
 
-        Dir.chdir(current)
         return local_repo_dir
       end
 
@@ -464,12 +439,8 @@ module Pindo
       end
 
       def git_addpush_repo(path:nil, message:"res", commit_file_params:nil)
-        current=Dir.pwd
-        Dir.chdir(path)
-
-        # gitee.com 仓库自动去除代理
+        # gitee.com 仓库自动去除代理；is_gitee_repo? 命令已带 -C，无需 chdir
         if is_gitee_repo?(local_repo_dir: path)
-          Dir.chdir(current)
           return without_proxy { git_addpush_repo_impl(path: path, message: message, commit_file_params: commit_file_params) }
         end
 
@@ -512,10 +483,10 @@ module Pindo
         rescue => error
             # puts(error.to_s)
             raise Informative, "\n#{path}\n 仓库失败 ！！！"
+        ensure
+            # 无论成功失败都还原 cwd，避免异常路径下污染全局工作目录
+            Dir.chdir(current)
         end
-
-
-        Dir.chdir(current)
       end
 
 
@@ -548,8 +519,11 @@ module Pindo
                     return nil
                 end
 
-                # 将字符串转换为数组
-                files_list.is_a?(String) ? files_list.split("\n").reject(&:empty?) : files_list
+                # git!/IO#readpartial 返回的是 ASCII-8BIT 字符串，含非 ASCII 字节（如中文路径、
+                # core.quotePath=false 时的原始多字节）会与下游 UTF-8 字面量拼接失败，这里统一
+                # 标成 UTF-8 并 scrub 无效字节，避免上层 `puts "...#{file}"` 抛 Encoding::CompatibilityError
+                raw_list = files_list.is_a?(String) ? files_list.split("\n") : Array(files_list)
+                raw_list.map { |f| f.dup.force_encoding('UTF-8').scrub }.reject(&:empty?)
             rescue => e
                 Funlog.instance.fancyinfo_error("获取未提交文件列表失败: #{e.message}")
                 nil
@@ -624,14 +598,13 @@ module Pindo
             puts
 
             # 将文件列表分行并用红色显示
-            if files.is_a?(String)
-                files.split("\n").each do |file|
-                    puts "  • #{file}".red unless file.empty?
-                end
-            else
-                files.each do |file|
-                    puts "  • #{file}".red unless file.empty?
-                end
+            # 防御性编码归一化：git! 返回的是 ASCII-8BIT，与 UTF-8 字面量 "  • " 拼接会抛
+            # Encoding::CompatibilityError；此处再兜一层，避免其它调用方传入 BINARY 字符串时崩溃
+            file_lines = files.is_a?(String) ? files.split("\n") : Array(files)
+            file_lines.each do |file|
+                next if file.nil? || file.empty?
+                safe_file = file.dup.force_encoding('UTF-8').scrub
+                puts "  • #{safe_file}".red
             end
 
             puts

data/lib/pindo/command/repo/create.rb

@@ -43,7 +43,7 @@ module Pindo
                 def initialize(argv)
 
                     @args_repo_name = argv.shift_argument 
-                    @args_login_flag = argv.flag?('force', false)
+                    @args_login_flag = argv.flag?('login', false)
                     @repo_name = argv.option('repo')
                     @org_name = argv.option('org')
 

data/lib/pindo/command/repo/search.rb

@@ -48,7 +48,7 @@ module Pindo
                 def initialize(argv)
 
                     @args_repo_name = argv.shift_argument 
-                    @args_login_flag = argv.flag?('force', false)
+                    @args_login_flag = argv.flag?('login', false)
                     @repo_name = argv.option('repo')
                     @org_name = argv.option('org')
 

data/lib/pindo/command/utils/installskills.rb

@@ -159,8 +159,8 @@ module Pindo
           Funlog.fancyinfo_start("克隆仓库: #{url}...")
           Funlog.info("仓库 URL: #{url}") if ENV['PINDO_DEBUG']
 
-          # 克隆仓库
-          success = system("git clone #{url} #{temp_dir} --quiet 2>/dev/null")
+          # 克隆仓库（数组参数形式，避免 URL 中的 shell 元字符注入）
+          success = system("git", "clone", url, temp_dir, "--quiet", err: File::NULL)
 
           unless success && File.exist?(File.join(temp_dir, '.git'))
             Funlog.fancyinfo_error("仓库克隆失败: #{url}")

data/lib/pindo/command/utils/renewproj.rb

@@ -142,7 +142,6 @@ module Pindo
                     new_project_dir = File.join(File::expand_path(current_dir + "/../") , temp_dir_name)
 
                     if File.exist?(new_project_dir)
-                        system 'sh rm -rf ' + new_project_dir
                         FileUtils.rm_rf(new_project_dir)
                     end
                     FileUtils.mkdir(new_project_dir)

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

data/lib/pindo/module/cert/cert_helper.rb

@@ -165,9 +165,9 @@ module Pindo
         puts "找到 #{identity_ids.size} 个证书:"
         puts identity_ids
 
-        # 删除证书
+        # 删除证书（数组参数形式，避免 identity_id 拼接进 shell）
         identity_ids.each do |identity_id|
-          system "security delete-certificate -Z #{identity_id}"
+          system("security", "delete-certificate", "-Z", identity_id)
         end
 
         # 清理 Provisioning Profiles

data/lib/pindo/module/pgyer/pgyerhelper.rb

@@ -710,14 +710,10 @@ module Pindo
                         FileUtils.rm_rf(ipa_file_upload)
                     end
 
-                    current_dir = Dir.pwd
                     if File.exist?(ipa_base_dir)
-                        Dir.chdir(ipa_base_dir)
                         base_name = File.basename(mac_app_path)
-                        command = "zip -qry \"#{ipa_file_upload}\" \"#{base_name}\""
-                        puts command
-                        system command
-                        Dir.chdir(current_dir)
+                        # 数组参数 + chdir，避免 shell 注入与 cwd 污染
+                        system("zip", "-qry", ipa_file_upload, base_name, chdir: ipa_base_dir)
                     end
                 end
                 if !ipa_file_upload.nil? &&
@@ -738,19 +734,18 @@ module Pindo
                     rescue StandardError => e
 
                     end
-                    current_project_dir = Dir.pwd
                     web_build_name = web_build_name.downcase.strip.gsub(/[\s\-_]/, '')
                     web_res_zip_fullname = File.join(web_zip_dir, web_build_name + "_" + web_build_version + ".zip")
                     if File.exist?(web_res_zip_fullname)
                         FileUtils.rm_rf(web_res_zip_fullname)
                     end
                     Zip::File.open(web_res_zip_fullname, Zip::File::CREATE) do |zipfile|
-                        Dir.chdir web_res_path
-                        Dir.glob("**/*").reject {|fn| File.directory?(fn) }.each do |file|
-                            zipfile.add(file.sub(web_res_path + '/', ''), file)
+                        Dir.chdir(web_res_path) do
+                            Dir.glob("**/*").reject {|fn| File.directory?(fn) }.each do |file|
+                                zipfile.add(file.sub(web_res_path + '/', ''), file)
+                            end
                         end
                     end
-                    Dir.chdir(current_project_dir)
                     ipa_file_upload = web_res_zip_fullname
                 end
                 if !ipa_file_upload.nil? &&
@@ -767,12 +762,11 @@ module Pindo
                         FileUtils.rm_rf(windows_zip_fullname)
                     end
 
-                    current_project_dir = Dir.pwd
                     Zip::File.open(windows_zip_fullname, Zip::File::CREATE) do |zipfile|
-                        Dir.chdir windows_build_dir
-                        zipfile.add(windows_exe_name, windows_exe_name)
+                        Dir.chdir(windows_build_dir) do
+                            zipfile.add(windows_exe_name, windows_exe_name)
+                        end
                     end
-                    Dir.chdir(current_project_dir)
                     ipa_file_upload = windows_zip_fullname
                 end
                 unless !ipa_file_upload.nil? && File.exist?(ipa_file_upload)
@@ -800,14 +794,14 @@ module Pindo
                     end
 
                     Zip::File.open(server_zipfile_name, Zip::File::CREATE) do |zipfile|
-                        Dir.chdir server_file_directory
-                        Dir.glob("**/*").reject {|fn| File.directory?(fn) }.each do |file|
-                            zipfile.add(file.sub(server_file_directory + '/', ''), file)
+                        Dir.chdir(server_file_directory) do
+                            Dir.glob("**/*").reject {|fn| File.directory?(fn) }.each do |file|
+                                zipfile.add(file.sub(server_file_directory + '/', ''), file)
+                            end
                         end
                     end
 
                     addtach_file = server_zipfile_name
-                    Dir.chdir current_project_dir
                 end
 
                 puts
@@ -1539,6 +1533,7 @@ module Pindo
                 if git_cliff_installed
                     temp_dir = Dir.pwd
                     Dir.chdir(current_git_root_path)
+                    begin
 
                     # 检查 HEAD 是否有 tag
                     head_tag = `git describe --exact-match --tags HEAD 2>/dev/null`.strip
@@ -1605,7 +1600,10 @@ module Pindo
                         description = "版本更新"
                     end
 
-                    Dir.chdir(temp_dir)
+                    ensure
+                        # ensure 保证异常路径也还原 cwd，避免污染全局工作目录
+                        Dir.chdir(temp_dir)
+                    end
                 else
                     # git-cliff 未安装，提示用户并退出
                     puts "\n\e[31m错误: git-cliff 未安装\e[0m"

data/lib/pindo/module/task/model/build/android_build_dev_task.rb

@@ -58,43 +58,7 @@ module Pindo
         build_android_project
       end
 
-      def find_output
-        # 搜索 APK 和 AAB 文件
-        search_paths = []
-        search_paths.concat(TaskConfig::BUILD_OUTPUT_PATTERNS[:apk].map { |p| File.join(@project_path, p) })
-        search_paths.concat(TaskConfig::BUILD_OUTPUT_PATTERNS[:aab].map { |p| File.join(@project_path, p) })
-
-        package_files = []
-        search_paths.each do |pattern|
-          package_files.concat(Dir.glob(pattern))
-        end
-
-        # 过滤测试包和未签名包
-        package_files.reject! do |f|
-          basename = File.basename(f).downcase
-          TaskConfig::EXCLUDED_PATTERNS.any? { |pattern| basename.include?(pattern) }
-        end
-
-        if package_files.any?
-          # 优先返回 AAB，其次 APK
-          aab_files = package_files.select { |f| f.end_with?(".aab") }
-          apk_files = package_files.select { |f| f.end_with?(".apk") }
-
-          latest_package = if aab_files.any?
-                            aab_files.max_by { |f| File.mtime(f) }
-                          elsif apk_files.any?
-                            apk_files.max_by { |f| File.mtime(f) }
-                          else
-                            package_files.max_by { |f| File.mtime(f) }
-                          end
-
-          puts "  找到 Android 包文件: #{latest_package}"
-          latest_package
-        else
-          puts "  警告: 未找到 APK/AAB 文件"
-          nil
-        end
-      end
+      # find_output 已上移到 AndroidBuildTask 基类（dev/adhoc/gplay 共用）
 
       private
 

data/lib/pindo/module/task/model/build/android_build_task.rb

@@ -1,4 +1,5 @@
 require_relative '../build_task'
+require_relative '../../task_config'
 
 module Pindo
   module TaskSystem
@@ -12,6 +13,45 @@ module Pindo
       def required_resources
         (super || []) + [{ type: :gradle, directory: @project_path }]
       end
+
+      # 查找构建产物（APK/AAB），三个子类（dev/adhoc/gplay）共用同一套 glob 逻辑
+      # 搜索路径与过滤规则与 pindo jps upload 查找 Android 包保持一致；优先 AAB，其次 APK
+      def find_output
+        search_paths = []
+        search_paths.concat(TaskConfig::BUILD_OUTPUT_PATTERNS[:apk].map { |p| File.join(@project_path, p) })
+        search_paths.concat(TaskConfig::BUILD_OUTPUT_PATTERNS[:aab].map { |p| File.join(@project_path, p) })
+
+        package_files = []
+        search_paths.each do |pattern|
+          package_files.concat(Dir.glob(pattern))
+        end
+
+        # 过滤测试包和未签名包
+        package_files.reject! do |f|
+          basename = File.basename(f).downcase
+          TaskConfig::EXCLUDED_PATTERNS.any? { |pattern| basename.include?(pattern) }
+        end
+
+        if package_files.any?
+          # 优先返回 AAB，其次 APK
+          aab_files = package_files.select { |f| f.end_with?(".aab") }
+          apk_files = package_files.select { |f| f.end_with?(".apk") }
+
+          latest_package = if aab_files.any?
+                            aab_files.max_by { |f| File.mtime(f) }
+                          elsif apk_files.any?
+                            apk_files.max_by { |f| File.mtime(f) }
+                          else
+                            package_files.max_by { |f| File.mtime(f) }
+                          end
+
+          puts "  找到 Android 包文件: #{latest_package}"
+          latest_package
+        else
+          # 构建已执行但产物缺失，视为失败，不静默返回 nil
+          raise Informative, "构建完成但未找到 APK/AAB 文件"
+        end
+      end
     end
   end
 end

data/lib/pindo/module/task/model/build/ios_build_adhoc_task.rb

@@ -41,7 +41,7 @@ module Pindo
 
       #准备构建
       def prepare_build
-        Dir.chdir(@project_path)
+        # Dir.chdir(@project_path) # Removed for thread safety（下游均传入绝对路径）
 
         # 0. 清理 Firebase Shell Script
         cleanup_firebase_shell
@@ -98,8 +98,8 @@ module Pindo
           puts "  找到 IPA 文件: #{latest_ipa}"
           latest_ipa
         else
-          puts "  警告: 未找到 IPA 文件"
-          nil
+          # 构建已执行但产物缺失，视为失败，不静默返回 nil
+          raise Informative, "构建完成但未找到 IPA 文件 (#{build_path})"
         end
       end
 
@@ -289,7 +289,7 @@ module Pindo
         puts "  处理 Quark/Swark..."
 
         require 'pindo/module/xcode/xcode_swark_helper'
-        Dir.chdir(@project_path)
+        # Dir.chdir(@project_path) # Removed for thread safety（下游均传入绝对路径）
 
         if xcode_build_type.include?("quark")
           Pindo::XcodeSwarkHelper.quark_run(
@@ -377,7 +377,7 @@ module Pindo
 
       # 编译 iOS 工程
       def build_ios_project
-        Dir.chdir(@project_path)
+        # Dir.chdir(@project_path) # Removed for thread safety（下游均传入绝对路径）
 
         # 使用 XcodeBuildHelper 进行构建
         ipa_file = Pindo::XcodeBuildHelper.build_project(

data/lib/pindo/module/task/model/build/ios_build_appstore_task.rb

@@ -40,7 +40,7 @@ module Pindo
 
       #准备构建
       def prepare_build
-        Dir.chdir(@project_path)
+        # Dir.chdir(@project_path) # Removed for thread safety（下游均传入绝对路径）
 
         # 0. 清理 Firebase Shell Script
         cleanup_firebase_shell
@@ -81,8 +81,8 @@ module Pindo
           puts "  找到 IPA 文件: #{latest_ipa}"
           latest_ipa
         else
-          puts "  警告: 未找到 IPA 文件"
-          nil
+          # 构建已执行但产物缺失，视为失败，不静默返回 nil
+          raise Informative, "构建完成但未找到 IPA 文件 (#{build_path})"
         end
       end
 
@@ -262,7 +262,7 @@ module Pindo
         puts "  处理 Quark/Swark..."
 
         require 'pindo/module/xcode/xcode_swark_helper'
-        Dir.chdir(@project_path)
+        # Dir.chdir(@project_path) # Removed for thread safety（下游均传入绝对路径）
 
         if xcode_build_type.include?("quark")
           Pindo::XcodeSwarkHelper.quark_run(
@@ -350,7 +350,7 @@ module Pindo
 
       # 编译 iOS 工程
       def build_ios_project
-        Dir.chdir(@project_path)
+        # Dir.chdir(@project_path) # Removed for thread safety（下游均传入绝对路径）
 
         # 使用 XcodeBuildHelper 进行构建
         ipa_file = Pindo::XcodeBuildHelper.build_project(

data/lib/pindo/module/task/model/build/ios_build_dev_task.rb

@@ -91,8 +91,8 @@ module Pindo
           return latest_output
         end
 
-        puts "  警告: 未找到 IPA / APP / 可执行文件"
-        nil
+        # 构建已执行但产物缺失，视为失败，不静默返回 nil
+        raise Informative, "构建完成但未找到 IPA / APP / 可执行文件 (#{build_dir})"
       end
 
       # 处理 CocoaPods 依赖（Dev 构建）

data/lib/pindo/module/task/model/build_task.rb

@@ -180,6 +180,8 @@ module Pindo
         execute_build
 
         # 查找输出文件
+        # 注意：是否「找不到产物即失败」由各子类 find_output 自行决定（已实现的子类会 raise；
+        # 未实现的 TODO 桩 AndroidBuildAdhocTask/AndroidBuildGPlayTask 仍返回 nil，保持原行为）
         @output_path = find_output
 
         {

data/lib/pindo/module/task/model/git/git_commit_task.rb

@@ -13,7 +13,7 @@ module Pindo
         :git_commit
       end
 
-      # 重试配置：失败后重试 1 次
+      # 重试配置：失败后重试 2 次（共 3 次尝试）
       def self.default_retry_count
         2
       end
@@ -69,6 +69,16 @@ module Pindo
         root_dir = @git_root_path
         git_repo_helper = Pindo::GitRepoHelper.share_instance
 
+        # 0. 先拉取远程 tag，确保本地 tag 列表是最新的
+        # 否则本地 tag 落后于远程时（如远程被 CI/其他机器打了更高 tag），
+        # 后续基于本地 tag 计算的版本会偏低，导致本该新建的版本 tag 与远程已有 tag 冲突而被漏打。
+        # 尽力而为：网络不可达时仅告警，用本地 tag 降级计算，不阻断构建。
+        begin
+          Pindo::GitHandler.git_remote!(root_dir, %W(-C #{root_dir} fetch --tags --force origin))
+        rescue => e
+          Funlog.instance.fancyinfo_warning("拉取远程 tag 失败，将使用本地 tag 计算版本: #{e.message.lines.first&.strip}")
+        end
+
         # 1. 如果 fixed_version 为 nil，尝试从 HEAD 的 tag 获取版本
         # 必须在 check_gitignore 之前检查，因为 check_gitignore 可能创建提交导致 HEAD 移动
         if (@fixed_version.nil? || @fixed_version.empty?) &&

data/lib/pindo/module/task/model/git/git_tag_task.rb

@@ -12,7 +12,7 @@ module Pindo
         :git_tag
       end
 
-      # 重试配置：失败后重试 1 次
+      # 重试配置：失败后重试 2 次（共 3 次尝试）
       def self.default_retry_count
         2
       end
@@ -39,7 +39,7 @@ module Pindo
       protected
 
       def do_work
-        # 检查是否为 Git 仓库，如果不是则直接成功返回
+        # 检查是否为 Git 仓库，如果不是则直接成功返回（非 Git 仓库目录下跳过打 tag）
         unless valid_git_repository?
           Funlog.instance.fancyinfo_warning("当前目录不是Git仓库，跳过Git打Tag")
           return {

data/lib/pindo/module/task/model/jps/jps_workflow_message_task.rb

@@ -15,6 +15,11 @@ module Pindo
         :jps_workflow_message
       end
 
+      # 工作流消息失败重试 1 次（覆盖 JPSTask 默认的重试 3 次）
+      def self.default_retry_count
+        1
+      end
+
       # 初始化工作流消息发送任务
       # @param options [Hash] 选项
       # @option options [String] :git_commit_id Git commit SHA（可选，从依赖任务获取）
@@ -70,17 +75,10 @@ module Pindo
             workflow_id: @workflow_id
           }
         rescue => e
-          # 外层错误保护：消息发送失败不应影响整体流程
-          puts "  ⚠️  JPS 工作流消息发送发生错误: #{e.message}"
+          # 工作流消息是终端任务，失败需明确报错中断（不再伪造 success）
+          puts "  ❌ JPS 工作流消息发送失败: #{e.message}"
           puts "  ⚠️  错误堆栈: #{e.backtrace.first(3).join("\n           ")}" if ENV['PINDO_DEBUG'] && e.backtrace
-          puts "  ℹ️  消息发送失败不影响主流程\n"
-
-          # 消息发送失败不抛出异常，返回成功但带警告
-          {
-            success: true,
-            warning: e.message,
-            git_commit_id: @git_commit_id
-          }
+          raise Informative, "JPS 工作流消息发送失败: #{e.message}"
         end
       end
 

data/lib/pindo/module/task/pindo_task.rb

@@ -446,9 +446,9 @@ module Pindo
         end
       end
 
-      # 计算当前重试次数
+      # 计算当前重试次数（调用时 @retry_count 已递减，无需再 +1）
       def current_retry_attempt
-        @max_retry_count - @retry_count + 1
+        @max_retry_count - @retry_count
       end
 
       # 立即重试

data/lib/pindo/module/task/task_manager.rb

@@ -118,12 +118,19 @@ module Pindo
         # 输出执行摘要
         @reporter.print_execution_summary
 
-        # 检查失败任务，有失败则抛出异常（确保进程返回非零退出码）
-        report = execution_report
-        if report[:failed] > 0
-          failed_tasks = @queue.completed_snapshot.select { |t| t.status == TaskStatus::FAILED }
-          failed_names = failed_tasks.map(&:name).join(', ')
-          raise Informative, "#{report[:failed]} 个任务执行失败: #{failed_names}"
+        # 失败闸门：FAILED / CANCELLED / 残留 PENDING（依赖阻塞等导致从未执行）都视为
+        # 「未成功完成」，统一抛异常确保进程非零退出码，避免任务没跑成却静默成功
+        completed = @queue.completed_snapshot
+        failed_tasks = completed.select { |t| t.status == TaskStatus::FAILED }
+        cancelled_tasks = completed.select { |t| t.status == TaskStatus::CANCELLED }
+        unfinished_tasks = @queue.pending_snapshot
+
+        if failed_tasks.any? || cancelled_tasks.any? || unfinished_tasks.any?
+          parts = []
+          parts << "失败: #{failed_tasks.map(&:name).join(', ')}" if failed_tasks.any?
+          parts << "取消: #{cancelled_tasks.map(&:name).join(', ')}" if cancelled_tasks.any?
+          parts << "未执行: #{unfinished_tasks.map(&:name).join(', ')}" if unfinished_tasks.any?
+          raise Informative, "存在未成功完成的任务 —— #{parts.join('；')}"
         end
       end
 
@@ -138,6 +145,7 @@ module Pindo
           completed: completed.count,
           success: completed.count { |t| t.status == TaskStatus::SUCCESS },
           failed: completed.count { |t| t.status == TaskStatus::FAILED },
+          cancelled: completed.count { |t| t.status == TaskStatus::CANCELLED },
           tasks: (pending + completed).map do |task|
             {
               id: task.id,

data/lib/pindo/module/unity/nuget_helper.rb

@@ -505,9 +505,8 @@ module Pindo
         puts "   使用工具: nuget pack"
         puts
 
-        cmd = "cd \"#{package_dir}\" && nuget pack \"#{File.basename(nuspec_file)}\" -OutputDirectory \"#{output_dir}\""
-
-        success = system(cmd)
+        # 数组参数 + chdir，避免路径中的 shell 元字符注入
+        success = system("nuget", "pack", File.basename(nuspec_file), "-OutputDirectory", output_dir, chdir: package_dir)
 
         unless success
           raise Informative, "NuGet 打包失败，请检查 .nuspec 文件格式"
@@ -573,10 +572,8 @@ module Pindo
         begin
           # 查找是否存在与 nuspec 版本匹配的 tag（支持 v1.1.3, V1.1.3, 1.1.3 等格式）
           matching_tag = nil
-          temp_dir = Dir.pwd
-          Dir.chdir(git_root)
 
-          # 获取所有 tags
+          # 获取所有 tags（命令已带 -C，无需 chdir）
           all_tags = Pindo::GitHandler.git!(%W(-C #{git_root} tag -l)).split("\n")
 
           # 查找匹配当前版本的 tag（不区分大小写，支持 v 前缀）
@@ -585,17 +582,12 @@ module Pindo
             tag_version.downcase == nuspec_version.downcase
           end
 
-          Dir.chdir(temp_dir)
-
           # 确定获取哪个范围的 commits
           git_range = if matching_tag
             # 找到匹配的 tag，获取前一个 tag 到这个 tag 的 commits
             puts "📝 找到版本 tag: #{matching_tag}，生成该版本的 release notes..."
 
-            temp_dir = Dir.pwd
-            Dir.chdir(git_root)
-
-            # 获取所有 tags 并按版本号排序
+            # 获取所有 tags 并按版本号排序（命令已带 -C，无需 chdir）
             all_tags = Pindo::GitHandler.git!(%W(-C #{git_root} tag -l)).split("\n")
             sorted_tags = all_tags.sort_by do |tag|
               version_str = tag.gsub(/^(v|V|release[\s_-]*)/i, '')
@@ -609,8 +601,6 @@ module Pindo
               prev_tag = sorted_tags[tag_index - 1]
             end
 
-            Dir.chdir(temp_dir)
-
             if prev_tag
               "#{prev_tag}..#{matching_tag}"
             else
@@ -635,10 +625,7 @@ module Pindo
             end
           end
 
-          # 获取 commit messages（包含完整的 body）
-          temp_dir = Dir.pwd
-          Dir.chdir(git_root)
-
+          # 获取 commit messages（包含完整的 body；命令已带 -C，无需 chdir）
           # 使用特殊分隔符来区分不同的 commits
           separator = "---COMMIT-SEPARATOR---"
           commits_raw = Pindo::GitHandler.git!(%W(-C #{git_root} log #{git_range} --pretty=format:%B#{separator}))
@@ -646,8 +633,6 @@ module Pindo
           # 按分隔符拆分成单个 commits
           commits = commits_raw.split(separator).map(&:strip).reject(&:empty?)
 
-          Dir.chdir(temp_dir)
-
           # 过滤只保留符合规范的 commits（支持带 scope 和不带 scope 格式）
           # 匹配格式: feat:, feat(scope):, fix:, fix(api): 等
           conventional_commit_regex = /^(feat|fix|docs|doc|perf|refactor|style|test|chore|revert)(\([^)]+\))?:/i

data/lib/pindo/module/utils/git_repo_helper.rb

@@ -330,53 +330,34 @@ module Pindo
       raise ArgumentError, "项目目录不能为空" if project_dir.nil?
       raise ArgumentError, "tag名称不能为空" if tag_name.nil? || tag_name.empty?
 
+      local_exists = Pindo::GitHandler.local_tag_exists?(local_repo_dir: project_dir, tag_name: tag_name)
+
+      # 建本地 tag 是纯本地操作，不依赖远程可达，网络故障也能留下本地 tag；
+      # 删远程残留与推送一律交给 push_tag_with_verify，推送失败时本地 tag 仍保留，可事后手动补推。
       case tag_type
       when Pindo::CreateTagType::NONE
+        # 不创建 tag
         Funlog.instance.fancyinfo_success("跳过创建 Tag")
-        return nil
-      when Pindo::CreateTagType::RECREATE
-        # 删除旧 tag（本地和远程）
-        if Pindo::GitHandler.remote_tag_exists?(local_repo_dir: project_dir, tag_name: tag_name)
-          Funlog.instance.fancyinfo_update("删除远程 Tag: #{tag_name}")
-          Pindo::GitHandler.git_remote!(project_dir, %W(-C #{project_dir} push origin :refs/tags/#{tag_name}))
-        end
-        if Pindo::GitHandler.local_tag_exists?(local_repo_dir: project_dir, tag_name: tag_name)
-          Funlog.instance.fancyinfo_update("删除本地 Tag: #{tag_name}")
-          Pindo::GitHandler.git!(%W(-C #{project_dir} tag -d #{tag_name}))
-        end
-      when Pindo::CreateTagType::NEW
-        local_exists = Pindo::GitHandler.local_tag_exists?(local_repo_dir: project_dir, tag_name: tag_name)
-        remote_exists = Pindo::GitHandler.remote_tag_exists?(local_repo_dir: project_dir, tag_name: tag_name)
-
-        # 如果 tag 在 HEAD 上，不需要重新创建
+        nil
+      when Pindo::CreateTagType::NEW, Pindo::CreateTagType::RECREATE
+        # 本地 tag 已在 HEAD：无需重建，仅确保推送到远程（RECREATE 在 HEAD 原地重建也是空转，同样适用）
         if local_exists && Pindo::GitHandler.is_tag_at_head?(git_root_dir: project_dir, tag_name: tag_name)
-          if !remote_exists
-            # 本地存在且在 HEAD，但远程不存在，推送到远程
-            push_tag_with_verify(project_dir: project_dir, tag_name: tag_name)
-            Funlog.instance.fancyinfo_success("推送 Tag 到远程: #{tag_name}")
-          else
-            Funlog.instance.fancyinfo_success("Tag 已存在且在 HEAD: #{tag_name}")
+          push_tag_with_verify(project_dir: project_dir, tag_name: tag_name)
+          tag_name
+        else
+          # 本地 tag 不存在或不在 HEAD：删旧重建后推送
+          if local_exists
+            Funlog.instance.fancyinfo_update("删除本地旧 Tag: #{tag_name}")
+            Pindo::GitHandler.git!(%W(-C #{project_dir} tag -d #{tag_name}))
           end
-          return tag_name
-        end
-
-        # tag 不在 HEAD 上，需要删除后重新创建
-        if remote_exists
-          Funlog.instance.fancyinfo_update("删除远程 Tag: #{tag_name}")
-          Pindo::GitHandler.git_remote!(project_dir, %W(-C #{project_dir} push origin :refs/tags/#{tag_name}))
-        end
-        if local_exists
-          Funlog.instance.fancyinfo_update("删除本地 Tag: #{tag_name}")
-          Pindo::GitHandler.git!(%W(-C #{project_dir} tag -d #{tag_name}))
+          Pindo::GitHandler.git!(%W(-C #{project_dir} tag #{tag_name}))
+          Funlog.instance.fancyinfo_success("已创建本地 Tag: #{tag_name}")
+          push_tag_with_verify(project_dir: project_dir, tag_name: tag_name)
+          tag_name
         end
-        # 继续创建新 tag
+      else
+        raise ArgumentError, "未知的 tag_type: #{tag_type}"
       end
-
-      # 创建并推送 tag
-      Pindo::GitHandler.git!(%W(-C #{project_dir} tag #{tag_name}))
-      push_tag_with_verify(project_dir: project_dir, tag_name: tag_name)
-      Funlog.instance.fancyinfo_success("创建并推送 Tag: #{tag_name}")
-      tag_name
     end
 
     # 推送 tag 到远程并校验是否真正推送成功，失败则重试
@@ -393,23 +374,37 @@ module Pindo
       loop do
         attempt += 1
         begin
+          # 远程若残留指向不同 commit 的同名 tag，先删除再推送，避免 non-fast-forward 被拒
+          remote_commit = Pindo::GitHandler.remote_tag_commit(local_repo_dir: project_dir, tag_name: tag_name)
+          if remote_commit == local_commit
+            Funlog.instance.fancyinfo_success("已确认 Tag 推送到远程: #{tag_name}")
+            return true
+          elsif remote_commit
+            Funlog.instance.fancyinfo_update("删除远程残留 Tag（远程 #{remote_commit[0..7]} ≠ 本地 #{local_commit[0..7]}）: #{tag_name}")
+            Pindo::GitHandler.git_remote!(project_dir, %W(-C #{project_dir} push origin :refs/tags/#{tag_name}))
+          end
+
           Pindo::GitHandler.git_remote!(project_dir, %W(-C #{project_dir} push origin #{tag_name}))
         rescue => e
           first_line = e.message.lines.first&.strip
           Funlog.instance.fancyinfo_warning("推送 Tag 失败（第 #{attempt}/#{max_retries} 次）: #{first_line}")
         end
 
-        # 校验远程 tag 指向的 commit 是否与本地一致，确认推送成功
-        remote_commit = Pindo::GitHandler.remote_tag_commit(local_repo_dir: project_dir, tag_name: tag_name)
+        # 校验远程 tag 指向的 commit 是否与本地一致，确认推送成功（探测失败按未确认处理，不中断重试）
+        remote_commit = begin
+          Pindo::GitHandler.remote_tag_commit(local_repo_dir: project_dir, tag_name: tag_name)
+        rescue
+          nil
+        end
         if remote_commit && remote_commit == local_commit
           Funlog.instance.fancyinfo_success("已确认 Tag 推送到远程: #{tag_name}")
           return true
-        elsif remote_commit
-          Funlog.instance.fancyinfo_warning("远程 Tag 指向不同 commit（远程 #{remote_commit[0..7]} ≠ 本地 #{local_commit[0..7]}），疑似残留旧 Tag: #{tag_name}")
         end
 
         if attempt >= max_retries
-          raise Informative, "Tag 推送失败，已重试 #{max_retries} 次仍未在远程确认: #{tag_name}"
+          # 本地 tag 已创建，仅远程未确认 —— 保留本地 tag，提示可手动补推
+          Funlog.instance.fancyinfo_warning("本地 Tag 已创建并保留，可稍后手动执行: git push origin #{tag_name}")
+          raise Informative, "Tag 推送失败，已重试 #{max_retries} 次仍未在远程确认: #{tag_name}（本地 Tag 已保留，可手动补推）"
         end
 
         Funlog.instance.fancyinfo_update("远程未确认 Tag，#{attempt}s 后重试推送: #{tag_name}")

data/lib/pindo/module/xcode/ipa_resign_helper.rb

@@ -86,10 +86,10 @@ module Pindo
                 tmp_dir = File.join(ipa_dir, tmp_time)
                 FileUtils.mkdir(tmp_dir) unless File.exist?(tmp_dir)
 
-                # 解压 IPA
-                command = "unzip -q \"#{resign_ipa_full_name}\" -d #{tmp_dir}"
-                puts command
-                system command
+                # 解压 IPA（数组参数避免注入；失败立即报错，不静默继续）
+                unless system("unzip", "-q", resign_ipa_full_name, "-d", tmp_dir)
+                    raise Informative, "IPA 解压失败: #{resign_ipa_full_name}"
+                end
 
                 payload_path = File.join(tmp_dir, "Payload")
                 if File.exist?(payload_path)
@@ -129,14 +129,13 @@ module Pindo
                 # 修改 Info.plist 中的 Bundle ID
                 modify_ipa_info_plist(modify_content_path: modify_content_path, bundle_id: new_bundle_id)
 
-                # 重新打包 IPA
-                current_dir = Dir.pwd
+                # 重新打包 IPA（数组参数 + chdir，避免 shell；* 由 Ruby 展开；失败立即报错）
                 if File.exist?(payload_path)
-                    Dir.chdir(tmp_dir)
-                    command = "zip -qry \"#{resign_ipa_full_name}\" * "
-                    puts command
-                    system command
-                    Dir.chdir(current_dir)
+                    # 排除点文件，对齐原 shell `*` 语义（避免把 .DS_Store 等打进 IPA 根目录）
+                    entries = Dir.children(tmp_dir).reject { |e| e.start_with?('.') }
+                    unless system("zip", "-qry", resign_ipa_full_name, *entries, chdir: tmp_dir)
+                        raise Informative, "IPA 重新打包失败: #{resign_ipa_full_name}"
+                    end
                 end
 
                 # 清理临时目录
@@ -160,24 +159,21 @@ module Pindo
 
                 # 确保 PlistBuddy 可用
                 unless File.exist?("/usr/local/bin/PlistBuddy")
-                    command = 'ln -s /usr/libexec/PlistBuddy /usr/local/bin/PlistBuddy'
-                    system command
+                    system("ln", "-s", "/usr/libexec/PlistBuddy", "/usr/local/bin/PlistBuddy")
                 end
 
-                # 读取旧的 Bundle ID
-                old_bundle_id_command = '/usr/local/bin/PlistBuddy -c "Print :CFBundleIdentifier" ' + main_info_plist
-                puts old_bundle_id_command
+                # 读取旧的 Bundle ID（数组参数，避免路径拼接进 shell）
                 old_bundle_id = ""
-                IO.popen(old_bundle_id_command) { |f| old_bundle_id = f.gets }
-                old_bundle_id = old_bundle_id.strip
+                IO.popen(["/usr/local/bin/PlistBuddy", "-c", "Print :CFBundleIdentifier", main_info_plist]) { |f| old_bundle_id = f.gets }
+                old_bundle_id = old_bundle_id.to_s.strip
                 puts "旧 Bundle ID: #{old_bundle_id}"
 
-                # 修改主应用的 Bundle ID
+                # 修改主应用的 Bundle ID（失败立即报错）
                 new_main_bundle_id = bundle_id
 
-                exchange_bundleid_command = '/usr/local/bin/PlistBuddy -c "Set :CFBundleIdentifier ' + new_main_bundle_id + '" ' + main_info_plist
-                puts exchange_bundleid_command
-                system exchange_bundleid_command
+                unless system("/usr/local/bin/PlistBuddy", "-c", "Set :CFBundleIdentifier #{new_main_bundle_id}", main_info_plist)
+                    raise Informative, "修改主应用 Bundle ID 失败: #{main_info_plist}"
+                end
 
                 # 如果 Bundle ID 发生变化，更新 URL Schemes
                 if old_bundle_id != new_main_bundle_id
@@ -193,10 +189,8 @@ module Pindo
                         appex_path = File.join(plugin_path, file)
                         tmp_info_plist = File.join(appex_path, "Info.plist")
 
-                        old_bundle_id_command = '/usr/local/bin/PlistBuddy -c "Print :CFBundleIdentifier" ' + tmp_info_plist
-                        puts old_bundle_id_command
                         old_plugin_bundle_id = ""
-                        IO.popen(old_bundle_id_command) { |f| old_plugin_bundle_id = f.gets }
+                        IO.popen(["/usr/local/bin/PlistBuddy", "-c", "Print :CFBundleIdentifier", tmp_info_plist]) { |f| old_plugin_bundle_id = f.gets }
                         puts "old_bundle_id = #{old_plugin_bundle_id}"
 
                         # 根据插件类型修改 Bundle ID
@@ -219,9 +213,9 @@ module Pindo
                 plugin_types.each do |suffix|
                     if old_bundle_id.include?(suffix)
                         new_plugin_bundle_id = new_base_bundle_id + suffix
-                        exchange_bundleid_command = '/usr/local/bin/PlistBuddy -c "Set :CFBundleIdentifier ' + new_plugin_bundle_id + '" ' + plist_path
-                        puts exchange_bundleid_command
-                        system exchange_bundleid_command
+                        unless system("/usr/local/bin/PlistBuddy", "-c", "Set :CFBundleIdentifier #{new_plugin_bundle_id}", plist_path)
+                            puts "  ⚠ 修改插件 Bundle ID 失败: #{plist_path}"
+                        end
                         break
                     end
                 end
@@ -322,49 +316,26 @@ module Pindo
                 puts "  旧 Scheme: #{old_scheme} (基于 #{old_bundle_id})"
                 puts "  新 Scheme: #{new_scheme} (基于 #{new_bundle_id})"
 
-                # 将 binary plist 转换为 XML 格式
-                convert_command = "plutil -convert xml1 \"#{plist_path}\" 2>/dev/null"
-                unless system(convert_command)
+                # 将 binary plist 转换为 XML 格式（数组参数，避免 plist_path 拼接进 shell）
+                unless system("plutil", "-convert", "xml1", plist_path, err: File::NULL)
                     puts "  ✗ 转换 plist 为 XML 失败"
                     return
                 end
 
-                # 检查文件中是否存在旧的 scheme
-                check_command = "grep -q \"#{old_scheme}\" \"#{plist_path}\""
-                scheme_exists = system(check_command)
-
-                unless scheme_exists
+                # 用 Ruby 文件操作做字面量替换，彻底规避 sed/grep 的 shell 注入
+                content = File.read(plist_path)
+                unless content.include?(old_scheme)
                     puts "  ✓ 没有找到需要更新的 URL Scheme"
-                    # 转换回 binary 格式
-                    convert_back_command = "plutil -convert binary1 \"#{plist_path}\" 2>/dev/null"
-                    system(convert_back_command)
+                    system("plutil", "-convert", "binary1", plist_path, err: File::NULL)
                     return
                 end
 
-                # 使用 sed 进行全文替换
-                # 使用临时文件避免 sed -i 在不同系统上的兼容性问题
-                temp_file = "#{plist_path}.tmp"
-                sed_command = "sed 's/#{old_scheme}/#{new_scheme}/g' \"#{plist_path}\" > \"#{temp_file}\" && mv \"#{temp_file}\" \"#{plist_path}\""
+                replace_count = content.scan(old_scheme).size
+                File.write(plist_path, content.gsub(old_scheme, new_scheme))
 
-                if system(sed_command)
-                    # 统计替换次数
-                    count_command = "grep -o \"#{new_scheme}\" \"#{plist_path}\" | wc -l"
-                    replace_count = `#{count_command}`.strip.to_i
-
-                    # 转换回 binary 格式
-                    convert_back_command = "plutil -convert binary1 \"#{plist_path}\" 2>/dev/null"
-                    system(convert_back_command)
-
-                    puts "  ✓ 已替换 #{replace_count} 处 URL Scheme: #{old_scheme} → #{new_scheme}"
-                else
-                    puts "  ✗ 替换 URL Scheme 失败"
-                    # 清理临时文件
-                    FileUtils.rm_f(temp_file) if File.exist?(temp_file)
-
-                    # 转换回 binary 格式（即使失败也要转回）
-                    convert_back_command = "plutil -convert binary1 \"#{plist_path}\" 2>/dev/null"
-                    system(convert_back_command)
-                end
+                # 转换回 binary 格式
+                system("plutil", "-convert", "binary1", plist_path, err: File::NULL)
+                puts "  ✓ 已替换 #{replace_count} 处 URL Scheme: #{old_scheme} → #{new_scheme}"
             end
 
         end

data/lib/pindo/version.rb

@@ -6,7 +6,7 @@ require 'time'
 
 module Pindo
 
-  VERSION = "5.18.20"
+  VERSION = "5.19.1"
 
   class VersionCheck
     RUBYGEMS_API = 'https://rubygems.org/api/v1/gems/pindo.json'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pindo
 version: !ruby/object:Gem::Version
-  version: 5.18.20
+  version: 5.19.1
 platform: ruby
 authors:
 - wade