pindo 5.20.5 → 5.20.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e05f84df2bd17ab7420a8975c8a6811f24080b11a4080391aff016f6f010b18f
-  data.tar.gz: 16c2db8376e2ec3e3f38ef5e0bd45eddaffb8434c65df25a95c27cdd265db469
+  metadata.gz: c0edb06546cd16285c138448253905ee5649e4e171af1a13cfbb61942f169e03
+  data.tar.gz: 88760601ffcc0a1863d537a8e1bf5a7f26f7ddb840b96831825e3699a26d2f5c
 SHA512:
-  metadata.gz: 14c3dd4dad433a639f118211ff83a2aed05b9ce63155c45c9f3293c1f52a6b39d1161ca1b1b175f1104fd51af54466e17d36c39c5fbafed8e6bfa0783827d143
-  data.tar.gz: d3675cbbfb84617cf00c13284c7450d69b4edf7bea3159c477d45cdb4db4327515bb50fba9c3b2fa4d31a9513f6222661f3652b9193350acfb959ada39bc9e5c
+  metadata.gz: c35bf1529656b6a5a24a2e727db177943ce3ab78a3fcc837a985b4584a62e1512897430e6520c4eaf146bc2a7b3c8ba6fca8d9e1af270826b31f0a070f81a8da
+  data.tar.gz: 4a9ecf6b0f98d988ca930c660d7082f0fe266698934926238482577cef0cd4c1f594ef686532a61aafa7a3cbb046aead11fa776da7a444344362b881d6d0e607

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

@@ -1125,10 +1125,11 @@ module Pindo
         # @param git_commit_id [String] Git commit SHA
         # @param workflow_id [Integer] 工作流ID（可选）
         # @return [Hash, nil] commit_log 记录或 nil
-        def find_commit_log_by_git_commit_id(git_commit_id, workflow_id = nil)
+        def find_commit_log_by_git_commit_id(git_commit_id, workflow_id = nil, branch = nil)
             params = {
                 pageNo: 1,
-                pageSize: 50  # 查询最近的 50 条记录
+                pageSize: 50,  # 查询最近的 50 条记录
+                onlyCliff: false
             }
 
             # 如果指定了 workflow_id，添加到筛选条件
@@ -1140,26 +1141,16 @@ module Pindo
                 if list_result && list_result["data"] && list_result["data"]["details"]
                     details = list_result["data"]["details"]
 
-                    # 遍历查找匹配的 commitId
-                    # 支持完整匹配或前缀匹配（git 短 SHA）
-                    commit_log = details.find do |item|
-                        item_commit_id = item["commitId"]
-                        next false unless item_commit_id
-
-                        # 完整匹配
-                        if item_commit_id == git_commit_id
-                            true
-                        # 前缀匹配（支持 git 短 SHA，如 9e959ffd）
-                        elsif git_commit_id.length >= 7 && item_commit_id.start_with?(git_commit_id)
-                            true
-                        elsif git_commit_id.length >= 7 && git_commit_id.start_with?(item_commit_id)
-                            true
-                        else
-                            false
-                        end
+                    matched_logs = details.select { |item| same_commit?(item["commitId"], git_commit_id) }
+                    return nil if matched_logs.empty?
+
+                    if !branch.nil?
+                        branch_text = branch.to_s
+                        branch_matched = matched_logs.find { |item| item["branch"].to_s == branch_text }
+                        return branch_matched if branch_matched
                     end
 
-                    return commit_log
+                    return matched_logs.first
                 end
             rescue => e
                 Funlog.instance.warning("查询 commit_log 列表失败: #{e.message}")
@@ -1168,6 +1159,30 @@ module Pindo
             nil
         end
 
+        # 从 commit_log 记录中提取发送消息需要的 indexNo。
+        #
+        # commit_log/list 的真实响应里 indexNo 不在根节点，而在 bindVersions 的各平台包里。
+        # 同一 commit 可能绑定多个平台，send_message 实测使用其中一个包的 indexNo；
+        # 取最大值可匹配当前线上请求数据，并稳定指向最近生成的包序号。
+        #
+        # @param commit_log [Hash, nil] commit_log/list 的单条记录
+        # @return [Integer, nil] indexNo
+        def commit_log_message_index_no(commit_log)
+            return nil unless commit_log.is_a?(Hash)
+
+            root_index_no = commit_log["indexNo"]
+            return root_index_no if root_index_no
+
+            bind_versions = commit_log["bindVersions"]
+            return nil unless bind_versions.is_a?(Hash)
+
+            bind_versions.values.map do |version_info|
+                next nil unless version_info.is_a?(Hash)
+
+                version_info["indexNo"]
+            end.compact.max
+        end
+
         def get_user_local_wechat_url( )
 
             wechat_msg_url = nil
@@ -1799,24 +1814,21 @@ module Pindo
 
         # 获取特定 commit 已绑定的包 ID
         #
-        # 数据源是 commit_log/preview 的 bindVersions。
+        # 数据源是 commit_log/list 的 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 workflow_id [Integer, String] 工作流 ID（必需，接口入参）
+        # @param excluded_package_types [Array<String>] 要排除的平台类型（本次新包会替换这些旧绑定）
         # @return [Array<String>, nil] 已绑定的包 ID 数组；查询失败或结构异常时返回 nil
-        def get_commit_bound_package_ids(commit_id:, workflow_id:)
+        def get_commit_bound_package_ids(commit_id:, workflow_id:, excluded_package_types: [])
             if commit_id.nil? || commit_id.empty?
                 Funlog.instance.fancyinfo_warning("commit_id 为空，无法查询已绑定的包")
                 return nil
@@ -1828,10 +1840,13 @@ module Pindo
             end
 
             begin
-                result = @pgyer_client.get_commit_log_preview(
-                    workflowId: workflow_id,
-                    commitIds: [commit_id],
-                    params: { onlyCliff: false }
+                result = @pgyer_client.get_commit_log_list(
+                    params: {
+                        workflowIds: [workflow_id],
+                        pageNo: 1,
+                        pageSize: 50,
+                        onlyCliff: false
+                    }
                 )
 
                 # 兼容两种响应格式
@@ -1839,22 +1854,26 @@ module Pindo
 
                 unless result && (response_code == 0 || response_code == 200)
                     error_msg = result&.dig("msg") || result&.dig("meta", "message") || "未知错误"
-                    Funlog.instance.fancyinfo_warning("查询 Commit #{commit_id[0..7]} 的绑定信息失败: #{error_msg}")
+                    Funlog.instance.fancyinfo_warning("查询 Commit #{commit_id[0..7]} 的提交记录失败: #{error_msg}")
                     return nil
                 end
 
-                data = result&.dig("data")
+                details = result&.dig("data", "details")
 
-                # code=0 且 data=nil 是接口表达「没有这条 commit」的正常响应（实测确认），
+                # code=0 且 details=nil 是接口表达「没有这条 commit」的正常响应（实测确认），
                 # 不是畸形响应。此时该 commit 还没进 commit log，不可能有已记录的绑定。
                 # 若误判为 nil，首次绑定（commit log 尚未生成）会被整体阻断。
-                if data.nil?
+                if details.nil?
                     puts "[PINDO_DEBUG] Commit #{commit_id[0..7]} 不在 commit log 中，视为未绑定" if ENV['PINDO_DEBUG']
                     return []
                 end
 
-                list = data.is_a?(Array) ? data : [data]
-                matched = list.select { |item| item.is_a?(Hash) && same_commit?(item["commitId"], commit_id) }
+                unless details.is_a?(Array)
+                    Funlog.instance.fancyinfo_warning("commit_log/list details 结构异常（#{details.class}），无法确认已有绑定")
+                    return nil
+                end
+
+                matched = details.select { |item| item.is_a?(Hash) && same_commit?(item["commitId"], commit_id) }
 
                 # 接口正常返回但没有这条 commit：该 commit 还没进 commit log，
                 # 也就不可能有已记录的绑定
@@ -1867,8 +1886,14 @@ module Pindo
                 # 实测 commit_log/list 可见），哪条带 bindVersions 没有契约保证。
                 # 只取第一条匹配会漏读绑定，进而让覆盖式写入删掉已有绑定，
                 # 因此合并所有匹配记录的绑定。
+                excluded_types = excluded_package_types.map { |item| normalize_package_type(item) }.compact
                 package_ids = []
                 matched.each do |commit_info|
+                    unless commit_info.key?("bindVersions")
+                        Funlog.instance.fancyinfo_warning("commit_log/list 缺少 bindVersions 字段，无法确认已有绑定")
+                        return nil
+                    end
+
                     bind_versions = commit_info["bindVersions"]
 
                     # 该记录没有绑定信息：未绑定的 commit 就是 null，属正常情况
@@ -1882,6 +1907,7 @@ module Pindo
                     # bindVersions 形如 { "ipa" => { "projectPackageId" => "..." }, ... }，
                     # 每个平台一个包
                     bind_versions.each do |platform, version_info|
+                        next if excluded_types.include?(normalize_package_type(platform))
                         next if version_info.nil?
 
                         unless version_info.is_a?(Hash)
@@ -1910,6 +1936,21 @@ module Pindo
             end
         end
 
+        # 归一化包平台类型，用于比较上传包类型和 bindVersions 平台 key。
+        #
+        # 服务端不同接口可能返回 "ipa"、"IPA" 或 "ipa#packageName" 这类变体。
+        # 绑定接口是覆盖式写入，若同类型旧包没被排除，会出现 ipa 新旧包一起提交，
+        # 最终回读仍是旧 ipa，导致校验缺失新包。
+        #
+        # @param package_type [Object] 平台类型
+        # @return [String, nil] 归一化后的平台类型
+        def normalize_package_type(package_type)
+            value = package_type.to_s.strip.downcase
+            return nil if value.empty?
+
+            value.split('#', 2).first
+        end
+
         # 判断两个 commit SHA 是否指向同一个提交
         #
         # 一端是全长 40 位 SHA、另一端是短 SHA（或大小写不同）时，严格 == 会永不命中。
@@ -2070,8 +2111,7 @@ module Pindo
         # @param project_id [String] 项目 ID
         # @param workflow_id [Integer] 工作流 ID
         # @param commit_id [String] Git commit SHA
-        # @param branch [String, nil] 分支名（可选，nil 时不发送分支字段——传错的分支名
-        #   会让 commit 与分支错误关联，比不传更糟）
+        # @param branch [String, nil] 分支名（可选，空值时按接口真实参数传空字符串）
         # @param single [Boolean] 是否单个提交（可选，默认 true）
         # @return [Hash] 发送结果
         def send_workflow_message(project_id:, workflow_id:, commit_id:, branch: nil, single: true)
@@ -2091,15 +2131,22 @@ module Pindo
             end
 
             begin
+                commit_log = find_commit_log_by_git_commit_id(commit_id, workflow_id, branch)
+                commit_log_id = commit_log && commit_log["id"]
+                index_no = commit_log_message_index_no(commit_log)
+
+                body_params = {
+                    single: single,
+                    ids: commit_log_id ? [commit_log_id] : nil,
+                    commitIds: [commit_id],
+                    branch: branch.to_s,
+                    indexNo: index_no
+                }.compact
+
                 result = @pgyer_client.send_commit_log_message(
                     projectId: project_id,
                     workflowId: workflow_id,
-                    params: {
-                        single: single,
-                        # jpsclient 内部归一化为服务端要求的 branches 数组；nil 时不带该字段
-                        branch: branch,
-                        commitIds: [commit_id]
-                    }.compact
+                    params: body_params
                 )
 
                 # 兼容两种响应格式

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

@@ -118,6 +118,7 @@ module Pindo
         if project_package_ids.empty?
           raise "app_version_list 中的包都缺少 id 字段"
         end
+        package_types = @app_version_list.map { |pkg| package_type_from(pkg) }.compact.uniq
 
         # 4. 打印绑定信息
         puts ""
@@ -125,7 +126,7 @@ module Pindo
         puts "     Git Commit: #{@git_commit_id[0..7]}"
         puts "     包数量: #{project_package_ids.size} 个"
         @app_version_list.each do |pkg|
-          package_type = pkg['nativePackageType'] || pkg['originalType'] || 'unknown'
+          package_type = raw_package_type_from(pkg) || 'unknown'
           puts "     [#{package_type}] ID: #{pkg['id']}, Version: #{pkg['projectVersion']}, Build: #{pkg['build']}"
         end
         if @git_commit_desc
@@ -153,7 +154,8 @@ module Pindo
         # 7. 获取当前 commit 已绑定的包 ID
         existing_bound_ids = pgyer_helper.get_commit_bound_package_ids(
           commit_id: @git_commit_id,
-          workflow_id: workflow_id
+          workflow_id: workflow_id,
+          excluded_package_types: package_types
         )
 
         # 查询失败（nil）时中止：绑定接口是覆盖式写入，带着不完整的列表提交
@@ -166,8 +168,8 @@ module Pindo
           puts "  📋 当前 Commit 已绑定的包: #{existing_bound_ids.size} 个"
         end
 
-        # 8. 合并当前 commit 已绑定的 ID 和新上传的包 ID（去重）
-        merged_package_ids = (existing_bound_ids + project_package_ids).uniq
+        # 8. 合并新上传的包 ID 和其它平台已绑定的 ID（同平台旧包已在查询时排除）
+        merged_package_ids = (project_package_ids + existing_bound_ids).uniq
 
         puts "  📦 合并后的包 ID 数量: #{merged_package_ids.size} 个"
         if merged_package_ids.size > project_package_ids.size
@@ -221,6 +223,28 @@ module Pindo
 
       private
 
+      def package_type_from(pkg)
+        normalize_package_type(raw_package_type_from(pkg))
+      end
+
+      def raw_package_type_from(pkg)
+        return nil unless pkg.respond_to?(:[])
+
+        pkg['nativePackageType'] || pkg[:nativePackageType] ||
+          pkg['originalType'] || pkg[:originalType] ||
+          pkg['packageType'] || pkg[:packageType] ||
+          pkg['native_package_type'] || pkg[:native_package_type] ||
+          pkg['original_type'] || pkg[:original_type] ||
+          pkg['type'] || pkg[:type]
+      end
+
+      def normalize_package_type(package_type)
+        value = package_type.to_s.strip.downcase
+        return nil if value.empty?
+
+        value.split('#', 2).first
+      end
+
       # 写后回读校验：确认期望绑定的包都在服务端的绑定列表里
       #
       # 查询失败时只告警不失败——包已经写进去了，回读失败不代表绑定失败。

data/lib/pindo/version.rb

@@ -6,7 +6,7 @@ require 'time'
 
 module Pindo
 
-  VERSION = "5.20.5"
+  VERSION = "5.20.6"
 
   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.5
+  version: 5.20.6
 platform: ruby
 authors:
 - wade
@@ -112,7 +112,7 @@ dependencies:
         version: '2.7'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.7.2
+        version: 2.7.3
   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.2
+        version: 2.7.3
 - !ruby/object:Gem::Dependency
   name: rqrcode
   requirement: !ruby/object:Gem::Requirement