pindo 5.20.4 → 5.20.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 51a9e1f1176906c3f5cc98d5c8654bd194c2199eca4d056c5d24b84d321a0e7f
-  data.tar.gz: 983e11201e2b488800fb65a745300fc71184e068e01f94255ae29d2f469c3d02
+  metadata.gz: e05f84df2bd17ab7420a8975c8a6811f24080b11a4080391aff016f6f010b18f
+  data.tar.gz: 16c2db8376e2ec3e3f38ef5e0bd45eddaffb8434c65df25a95c27cdd265db469
 SHA512:
-  metadata.gz: 140c5a77bf8ad4e46547a60caef4587e04811450624a0c19213b978fae267078b8d6370486209531d4116ab202670ce12a707edc93c7299df1e4e244f389668d
-  data.tar.gz: f646d8f611a70c2ad39bdaa31ef69f55f4a0946ed3e1c90a27d806072ca2484c1bcf7ec8073d9e381031d2ab645aceaf5387cb13b8e90ed6e20070acee5f5a6a
+  metadata.gz: 14c3dd4dad433a639f118211ff83a2aed05b9ce63155c45c9f3293c1f52a6b39d1161ca1b1b175f1104fd51af54466e17d36c39c5fbafed8e6bfa0783827d143
+  data.tar.gz: d3675cbbfb84617cf00c13284c7450d69b4edf7bea3159c477d45cdb4db4327515bb50fba9c3b2fa4d31a9513f6222661f3652b9193350acfb959ada39bc9e5c

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

@@ -198,8 +198,6 @@ module Pindo
 
           puts "package_type #{package_type} manage_type: #{manage_type}"
 
-          Funlog.instance.fancyinfo_success("获取工作流列表成功，共 #{workflows.size} 个工作流")
-
           # 2. 根据 manage_type 或 package_type 过滤工作流
           if manage_type == "git"
             # manage_type 为 git 时，只筛选 manage_type == "git" 的工作流，不看 package_type
@@ -249,6 +247,10 @@ module Pindo
             display_workflows = filtered_workflows
           end
 
+          # 过滤后再报数：只报总数会让人以为剩下的工作流被漏掉了，
+          # 实际是按 manage_type / package_type 筛掉的，两个数字一起给出才不歧义
+          Funlog.instance.fancyinfo_success("获取工作流列表成功，共 #{workflows.size} 个工作流，其中 #{type_desc} #{display_workflows.size} 个")
+
           # 2.5 若指定了 workflow_name，按名称精确匹配（忽略大小写与首尾空格）确定性选择。
           # 只接受完整精确匹配，不做子串模糊匹配，避免 "Production" 误命中 "Legacy Production Upload"。
           if workflow_name && !workflow_name.to_s.strip.empty? && !display_workflows.empty?
@@ -849,6 +851,16 @@ module Pindo
 
                 # 计算上传耗时
                 upload_duration = Time.now - upload_start_time
+
+                # upload_file 失败时返回 nil。不在这里拦住的话，后面会照常打印「上传完成」
+                # 和一个拿完整文件大小除以失败耗时算出来的假速度，再去做无谓的附件上传，
+                # 直到提交包信息前才失败——日志里先 ❌ 后 ✅ 就是这么来的。
+                if upload_res.nil?
+                    Funlog.instance.fancyinfo_error("文件上传失败! 耗时: #{upload_duration.round(2)} 秒")
+                    puts "  失败原因见上方日志；加 --verbose 重跑可看到接口返回的 code/msg"
+                    raise Informative, "上传失败!"
+                end
+
                 upload_speed = file_size_mb / upload_duration
                 puts "✅ 文件上传完成! 耗时: #{upload_duration.round(2)} 秒, 平均速度: #{upload_speed.round(2)} MB/s"
 
@@ -865,8 +877,15 @@ module Pindo
 
                         # 计算附件上传耗时
                         attach_duration = Time.now - attach_start_time
-                        attach_speed = attach_size_mb / attach_duration
-                        puts "✅ 附件上传完成! 耗时: #{attach_duration.round(2)} 秒, 平均速度: #{attach_speed.round(2)} MB/s"
+
+                        # 同主文件：nil 即失败，不能照打「上传完成」。
+                        # 但附件失败不阻断主包流程（下方 rescue 也是这个取向），只如实报告。
+                        if attach_key_url.nil?
+                            Funlog.instance.fancyinfo_warning("附件上传失败! 耗时: #{attach_duration.round(2)} 秒（不影响主包上传）")
+                        else
+                            attach_speed = attach_size_mb / attach_duration
+                            puts "✅ 附件上传完成! 耗时: #{attach_duration.round(2)} 秒, 平均速度: #{attach_speed.round(2)} MB/s"
+                        end
                     end
 
                     if !attach_key_url.nil?
@@ -1780,78 +1799,101 @@ module Pindo
 
         # 获取特定 commit 已绑定的包 ID
         #
-        # 走 project_package/bind_list（按 projectId 查项目下全部绑定），而不是
-        # commit_log/preview（按 workflowId 过滤）：写入接口 bind_commit 只认全局 commitId、
-        # 且是覆盖式的，查询作用域必须与之一致，否则查漏的平台会在覆盖时被删掉。
+        # 数据源是 commit_log/preview 的 bindVersions。
+        #
+        # 不能用 project_package/bind_list：它按 projectId 查、作用域看似更宽，
+        # 但实测（项目 930 条记录）每条的 commitId、branch、workflowId、commitLogs
+        # 全部为 nil——该接口只返回包本身，不携带 commit 绑定关系，
+        # 按 commitId 过滤会恒得空列表，进而让覆盖式写入误删该 commit 的已有绑定。
+        #
+        # preview 需要传 workflowId，但实测同项目下不同 workflowId（3457 / 2650）
+        # 返回的 bindVersions 完全一致——绑定按 commitId 全局记录，
+        # 不存在"按工作流查漏、覆盖时被删掉"的问题。
         #
         # 绑定接口是覆盖式写入，调用方需要把已绑定的包一起提交，因此必须区分
         # 「确认没有绑定过」和「查不出来」——后者返回空数组会导致已有绑定被静默覆盖丢失。
         #
         # @param commit_id [String] Git commit SHA（必需）
-        # @param project_id [String] JPS 项目 ID（必需）
+        # @param workflow_id [Integer, String] 工作流 ID（必需，接口入参）
         # @return [Array<String>, nil] 已绑定的包 ID 数组；查询失败或结构异常时返回 nil
-        def get_commit_bound_package_ids(commit_id:, project_id:)
+        def get_commit_bound_package_ids(commit_id:, workflow_id:)
             if commit_id.nil? || commit_id.empty?
                 Funlog.instance.fancyinfo_warning("commit_id 为空，无法查询已绑定的包")
                 return nil
             end
 
-            if project_id.nil? || project_id.to_s.empty?
-                Funlog.instance.fancyinfo_warning("project_id 为空，无法查询已绑定的包")
+            if workflow_id.nil? || workflow_id.to_s.empty?
+                Funlog.instance.fancyinfo_warning("workflow_id 为空，无法查询已绑定的包")
                 return nil
             end
 
             begin
-                result = @pgyer_client.get_project_package_bind_list(projectId: project_id)
+                result = @pgyer_client.get_commit_log_preview(
+                    workflowId: workflow_id,
+                    commitIds: [commit_id],
+                    params: { onlyCliff: false }
+                )
 
                 # 兼容两种响应格式
                 response_code = result&.dig("code") || result&.dig("meta", "code")
 
                 unless result && (response_code == 0 || response_code == 200)
                     error_msg = result&.dig("msg") || result&.dig("meta", "message") || "未知错误"
-                    Funlog.instance.fancyinfo_warning("查询项目 #{project_id} 的包绑定列表失败: #{error_msg}")
-                    debug_dump_bind_list_response(result)
+                    Funlog.instance.fancyinfo_warning("查询 Commit #{commit_id[0..7]} 的绑定信息失败: #{error_msg}")
                     return nil
                 end
 
                 data = result&.dig("data")
 
-                # 接口没有返回数据：项目下没有任何绑定，此时也就不存在会被覆盖丢失的绑定
-                return [] if data.nil?
-
-                # 接口定义返回 map<String, List<ProjectPackagePreviewInfo>>。
-                # 必须先校验类型再判空——[] 和 "" 都 respond_to?(:empty?)，
-                # 先判空会把结构异常的响应误判成「项目下无绑定」，随后覆盖写入清掉已有绑定。
-                # 分组键的语义（commitId / 平台类型）不影响结果：每条记录自带 commitId，按它过滤即可。
-                unless data.is_a?(Hash)
-                    Funlog.instance.fancyinfo_warning("包绑定列表结构异常（#{data.class}），无法确认已有绑定")
-                    debug_dump_bind_list_response(result)
-                    return nil
+                # code=0 且 data=nil 是接口表达「没有这条 commit」的正常响应（实测确认），
+                # 不是畸形响应。此时该 commit 还没进 commit log，不可能有已记录的绑定。
+                # 若误判为 nil，首次绑定（commit log 尚未生成）会被整体阻断。
+                if data.nil?
+                    puts "[PINDO_DEBUG] Commit #{commit_id[0..7]} 不在 commit log 中，视为未绑定" if ENV['PINDO_DEBUG']
+                    return []
                 end
 
-                # 空 map：确认项目下没有绑定
-                return [] if data.empty?
+                list = data.is_a?(Array) ? data : [data]
+                matched = list.select { |item| item.is_a?(Hash) && same_commit?(item["commitId"], commit_id) }
 
+                # 接口正常返回但没有这条 commit：该 commit 还没进 commit log，
+                # 也就不可能有已记录的绑定
+                if matched.empty?
+                    puts "[PINDO_DEBUG] Commit #{commit_id[0..7]} 不在 commit log 中，视为未绑定" if ENV['PINDO_DEBUG']
+                    return []
+                end
+
+                # 同一个 commitId 在 commit log 里可能对应多条记录（push 一条、tag 一条，
+                # 实测 commit_log/list 可见），哪条带 bindVersions 没有契约保证。
+                # 只取第一条匹配会漏读绑定，进而让覆盖式写入删掉已有绑定，
+                # 因此合并所有匹配记录的绑定。
                 package_ids = []
-                data.each do |group_key, packages|
-                    unless packages.is_a?(Array)
-                        Funlog.instance.fancyinfo_warning("包绑定列表分组 #{group_key} 结构异常（#{packages.class}），无法确认已有绑定")
+                matched.each do |commit_info|
+                    bind_versions = commit_info["bindVersions"]
+
+                    # 该记录没有绑定信息：未绑定的 commit 就是 null，属正常情况
+                    next if bind_versions.nil?
+
+                    unless bind_versions.is_a?(Hash)
+                        Funlog.instance.fancyinfo_warning("bindVersions 结构异常（#{bind_versions.class}），无法确认已有绑定")
                         return nil
                     end
 
-                    packages.each do |pkg|
-                        unless pkg.is_a?(Hash)
-                            Funlog.instance.fancyinfo_warning("包绑定列表分组 #{group_key} 中存在异常条目，无法确认已有绑定")
+                    # bindVersions 形如 { "ipa" => { "projectPackageId" => "..." }, ... }，
+                    # 每个平台一个包
+                    bind_versions.each do |platform, version_info|
+                        next if version_info.nil?
+
+                        unless version_info.is_a?(Hash)
+                            Funlog.instance.fancyinfo_warning("bindVersions[#{platform}] 结构异常（#{version_info.class}），无法确认已有绑定")
                             return nil
                         end
 
-                        next unless pkg["commitId"] == commit_id
-
-                        # 命中目标 commit 却拿不到包 id：静默跳过会得到残缺列表，
-                        # 覆盖写入时等于把那个包的绑定删掉
-                        package_id = pkg["id"]
+                        # 有平台条目却拿不到包 id：静默跳过会得到残缺列表，
+                        # 覆盖写入时等于把那个平台的绑定删掉
+                        package_id = version_info["projectPackageId"]
                         if package_id.nil? || package_id.to_s.strip.empty?
-                            Funlog.instance.fancyinfo_warning("Commit #{commit_id[0..7]} 的绑定记录缺少包 id，无法确认已有绑定")
+                            Funlog.instance.fancyinfo_warning("Commit #{commit_id[0..7]} 的 #{platform} 绑定缺少 projectPackageId，无法确认已有绑定")
                             return nil
                         end
 
@@ -1859,7 +1901,7 @@ module Pindo
                     end
                 end
 
-                puts "[PINDO_DEBUG] Commit #{commit_id[0..7]} 已绑定 #{package_ids.size} 个包" if ENV['PINDO_DEBUG']
+                puts "[PINDO_DEBUG] Commit #{commit_id[0..7]} 已绑定 #{package_ids.size} 个包: #{package_ids.inspect}" if ENV['PINDO_DEBUG']
                 package_ids.uniq
             rescue => e
                 Funlog.instance.fancyinfo_warning("查询 commit 已绑定的包异常: #{e.message}")
@@ -1868,25 +1910,25 @@ module Pindo
             end
         end
 
-        # 按白名单输出包绑定列表响应的诊断信息
+        # 判断两个 commit SHA 是否指向同一个提交
         #
-        # ProjectPackagePreviewInfo 里含 cdnUrl、plistUrl、attachFiles、qrCodeUrl 等
-        # 可直接下载安装包的链接，整体 inspect 会把它们写进 CI 日志，
-        # 因此只输出定位问题所需的字段。
-        # @param result [Hash, nil] 接口原始响应
-        def debug_dump_bind_list_response(result)
-            return unless ENV['PINDO_DEBUG']
-
-            code = result&.dig("code") || result&.dig("meta", "code")
-            message = result&.dig("msg") || result&.dig("meta", "message")
-            data = result&.dig("data")
-            summary = if data.is_a?(Hash)
-                data.map { |k, v| [k, v.is_a?(Array) ? "#{v.size} 条" : v.class.to_s] }.to_h
-            else
-                data.class.to_s
-            end
-
-            puts "[PINDO_DEBUG] bind_list 响应: code=#{code.inspect} msg=#{message.inspect} data=#{summary.inspect}"
+        # 一端是全长 40 位 SHA、另一端是短 SHA（或大小写不同）时，严格 == 会永不命中。
+        # 对覆盖式写入的 bind_commit 来说这不是「查不到」而是「查成空」，
+        # 会把该 commit 上已有的绑定一起删掉，因此按前缀 + 忽略大小写比对。
+        # 要求较短的一方至少 7 字符，避免过短或异常值产生误匹配。
+        #
+        # @param sha_a [String, nil] 待比较的 SHA
+        # @param sha_b [String, nil] 待比较的 SHA
+        # @return [Boolean] 是否同一个提交
+        def same_commit?(sha_a, sha_b)
+            a = sha_a.to_s.strip.downcase
+            b = sha_b.to_s.strip.downcase
+            return false if a.empty? || b.empty?
+
+            shorter, longer = a.length <= b.length ? [a, b] : [b, a]
+            return false if shorter.length < 7
+
+            longer.start_with?(shorter)
         end
 
         # 获取工作流中已绑定的包 ID（已废弃，请使用 get_commit_bound_package_ids）

data/lib/pindo/module/source_files_guard.rb

@@ -26,6 +26,12 @@ module Pindo
       '**/src/main/res'
     ].freeze
 
+    # Unity 工程判定即要求该文件存在（BuildHelper#unity_project?），故用精确路径而非 glob，
+    # 避免匹配到 Packages 下嵌入工程里的同名副本。
+    UNITY_PATTERNS = [
+      'ProjectSettings/ProjectSettings.asset'
+    ].freeze
+
     EXCLUDED_PATH_COMPONENTS = %w[
       .git
       .gradle
@@ -56,9 +62,12 @@ module Pindo
         source_paths(project_path, ANDROID_PATTERNS)
       end
 
-      # Unity 平台工程导出到已忽略的 GoodPlatform，不提交 Unity 源码配置。
-      def unity(_project_path)
-        []
+      # 导出的平台工程落在已忽略的 GoodPlatform 里，不需要提交。
+      # 但 ProjectSettings.asset 是 Unity 源码配置，构建期间会被 Unity 写入
+      # （版本号、编译宏等），性质与 iOS 的 project.pbxproj / Info.plist 相同，
+      # 不提交就会在构建后残留为未提交改动，且 tag 上拿不到构建实际用的配置。
+      def unity(project_path)
+        source_paths(project_path, UNITY_PATTERNS)
       end
 
       private

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

@@ -27,9 +27,9 @@ module Pindo
       # @option options [String] :git_commit_time Git commit 时间（可选）
       # @option options [String] :git_commit_desc Git commit 描述（可选）
       # @option options [String] :project_dir 项目目录（可选，用于自动获取 commit 信息）
-      # @option options [Hash] :app_info_obj JPS 应用信息对象（必需，用于取 project_id；
-      #   绑定与查询都以项目为作用域，不再依赖 workflow_info）
-      # @option options [Hash] :workflow_info 工作流信息（可选，继承自基类，本任务不使用）
+      # @option options [Hash] :app_info_obj JPS 应用信息对象（可选，继承自基类，本任务不使用）
+      # @option options [Hash] :workflow_info 工作流信息（必需，用于取 workflow_id；
+      #   回读绑定走 commit_log/preview，该接口以 workflowId 为入参）
       # @option options [String] :project_name 项目名称（可选，继承自基类）
       def initialize(app_version_list = nil, options = {})
         # 兼容性处理：如果传入的是单个 Hash，转为数组
@@ -142,16 +142,18 @@ module Pindo
           raise "无法登录 JPS，请检查配置"
         end
 
-        # 6. 获取项目 ID（绑定与查询都以项目为作用域）
-        project_id = @app_info_obj&.dig("id") || @app_info_obj&.dig(:id)
-        unless project_id
-          raise "缺少 app_info_obj，无法获取 project_id"
+        # 6. 获取工作流 ID（commit_log/preview 的入参；绑定本身按 commitId 全局记录，
+        # 同项目下任一 workflowId 查到的 bindVersions 都相同）
+        workflow_id = @workflow_info&.dig(:workflow_id) || @workflow_info&.dig('workflow_id') ||
+                      @workflow_info&.dig(:id) || @workflow_info&.dig('id')
+        unless workflow_id
+          raise "缺少 workflow_info，无法获取 workflow_id"
         end
 
         # 7. 获取当前 commit 已绑定的包 ID
         existing_bound_ids = pgyer_helper.get_commit_bound_package_ids(
           commit_id: @git_commit_id,
-          project_id: project_id
+          workflow_id: workflow_id
         )
 
         # 查询失败（nil）时中止：绑定接口是覆盖式写入，带着不完整的列表提交
@@ -196,7 +198,7 @@ module Pindo
           # 根治需要服务端支持，二选一即可：
           #   1) bind_commit 增加 append 语义（只追加不覆盖）
           #   2) 返回并校验 revision，做乐观并发控制
-          verify_bound_packages(pgyer_helper, project_id, merged_package_ids)
+          verify_bound_packages(pgyer_helper, workflow_id, merged_package_ids)
 
           puts ""
 
@@ -225,12 +227,12 @@ module Pindo
       # 只有确认「期望的包没在列表里」才判定为被并发覆盖，抛异常交给任务重试。
       #
       # @param pgyer_helper [PgyerHelper] 已登录的 helper
-      # @param project_id [String] 项目 ID
+      # @param workflow_id [Integer, String] 工作流 ID
       # @param expected_ids [Array<String>] 本次写入的包 ID 列表
-      def verify_bound_packages(pgyer_helper, project_id, expected_ids)
+      def verify_bound_packages(pgyer_helper, workflow_id, expected_ids)
         actual_ids = pgyer_helper.get_commit_bound_package_ids(
           commit_id: @git_commit_id,
-          project_id: project_id
+          workflow_id: workflow_id
         )
 
         if actual_ids.nil?

data/lib/pindo/module/task/model/nuget/nuget_upload_task.rb

@@ -152,13 +152,17 @@ module Pindo
 
         # 计算上传耗时
         upload_duration = Time.now - upload_start_time
-        upload_speed = file_size_mb / upload_duration
-        puts "✅ 文件上传完成! 耗时: #{upload_duration.round(2)} 秒, 平均速度: #{upload_speed.round(2)} MB/s"
 
+        # 判空必须放在打印「上传完成」之前：upload_file 失败返回 nil，
+        # 先打印会输出一句假的完成提示，外加一个用完整文件大小除以失败耗时算出的假速度
         if file_url.nil?
+          puts "❌ 文件上传失败! 耗时: #{upload_duration.round(2)} 秒"
           raise Informative, "文件上传失败"
         end
 
+        upload_speed = file_size_mb / upload_duration
+        puts "✅ 文件上传完成! 耗时: #{upload_duration.round(2)} 秒, 平均速度: #{upload_speed.round(2)} MB/s"
+
         puts "✅ 文件上传成功"
         puts
 

data/lib/pindo/version.rb

@@ -6,7 +6,7 @@ require 'time'
 
 module Pindo
 
-  VERSION = "5.20.4"
+  VERSION = "5.20.5"
 
   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.20.4
+  version: 5.20.5
 platform: ruby
 authors:
 - wade
@@ -112,7 +112,7 @@ dependencies:
         version: '2.7'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.7.1
+        version: 2.7.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -122,7 +122,7 @@ dependencies:
         version: '2.7'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.7.1
+        version: 2.7.2
 - !ruby/object:Gem::Dependency
   name: rqrcode
   requirement: !ruby/object:Gem::Requirement
@@ -531,7 +531,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: 3.0.0
 requirements: []
-rubygems_version: 4.0.11
+rubygems_version: 4.0.10
 specification_version: 4
 summary: easy work
 test_files: []