pindo 5.18.20 → 5.19.2

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}")