hiiro 0.1.289 → 0.1.290

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 37466eb01d7284c838b3ba958f35a41e0f33f4f4fd5c22036ce11f2daa13bfb4
-  data.tar.gz: 55b421c1b3c124cc265d14c93f7709d0e2c894d4497125884306fbe53fe29762
+  metadata.gz: 39db2e632092be3971a6cb08da3a0a4a4ad3accfc42c6978bccee29ebd9c185f
+  data.tar.gz: 02e163c3f894838a6e1ef31d96d2b9f3011bf0ff53e50eee71f4f8c4c8ff1fb4
 SHA512:
-  metadata.gz: dc41d3e7f2fc019d20336f0890dcff641d545a3bb25a6b83147fc1067b3a139a2548c498b8f9e825cbe7096c2a4d2d77706ed9e21d79b36b26459db54a2ea3f4
-  data.tar.gz: 403890caf1a18fe7b5c231b1fd12c2e4d07897867c8e46620441923f2860b4e5c0f7ab848105edb4c773a7d2724f5a4be4048e37155ec31bce78ea387f07fc1c
+  metadata.gz: 67bdae476921c82c1aaf61195768b1145b235756ba73687e28ea918dfe43ea8d907471d38d1084dfa2caf82921fe594145f9183073480adc8b140a6d6c998723
+  data.tar.gz: 9a824284df02e880db1bb13496610b1ae39c2cbdbbae9f09176640ff8bb6e50d3e9c1a56b84491ef6d5b6ff0742db0cb9f84fe5c3c2a67abc58b42d581279c8d

data/CHANGELOG.md

@@ -1,9 +1,16 @@
 ```markdown
+## v0.1.290 (2026-03-26)
+
+### Fixed
+- `h pr ls`/`h pr update`: revert `statusCheckRollup` contexts limit to 100 and add pagination to retrieve all checks beyond the first 100; GitHub's GraphQL API silently returns null when limit is exceeded, causing all checks to vanish
+
 ## v0.1.289 (2026-03-26)
 
 ### Fixed
 - `h queue run`/`watch`: frontmatter `session_name` now directly controls which tmux session the task launches in; working directory seeded from that session's active pane when no tree is specified
 - `h queue sadd`: now immediately launches a new tmux window in the target session after adding to queue, consistent with `hadd`/`vadd` behavior (previously just added to pending with no launch)
+- `h pr ls`: revert `statusCheckRollup` contexts limit from 250 back to 100; GitHub's GraphQL API caps connections at 100 — exceeding it silently returns null for the entire field, causing all checks to vanish from the list
+- `h pr update`: paginate beyond the first 100 checks for PRs that hit the limit; show `❓` status for PRs where pagination still couldn't retrieve all checks
 
 ## v0.1.288 (2026-03-26)
 

data/lib/hiiro/git/pr.rb

@@ -106,7 +106,8 @@ class Hiiro
 
       # Summarizes raw statusCheckRollup contexts into { total, success, pending, failed, frozen }.
       # frozen = number of failed contexts that are specifically the ISC code freeze check.
-      def self.summarize_checks(rollup)
+      # truncated: true is added when pagination couldn't retrieve all checks.
+      def self.summarize_checks(rollup, truncated: false)
         return nil unless rollup
 
         contexts = rollup.is_a?(Array) ? rollup : []
@@ -126,7 +127,9 @@ class Hiiro
             (FAILED_CONCLUSIONS.include?(c['conclusion']) || %w[FAILURE ERROR].include?(c['state']))
         end
 
-        { 'total' => total, 'success' => success, 'pending' => pending, 'failed' => failed, 'frozen' => frozen }
+        result = { 'total' => total, 'success' => success, 'pending' => pending, 'failed' => failed, 'frozen' => frozen }
+        result['truncated'] = true if truncated
+        result
       end
 
       # Summarizes raw review nodes into { approved, changes_requested, commented, reviewers }.

data/lib/hiiro/pinned_pr_manager.rb

@@ -262,7 +262,7 @@ class Hiiro
         pr.state           = info['state']
         pr.title           = info['title']
         pr.check_runs      = rollup
-        pr.checks          = Hiiro::Git::Pr.summarize_checks(rollup)
+        pr.checks          = Hiiro::Git::Pr.summarize_checks(rollup, truncated: info['checksTruncated'])
         pr.reviews         = Hiiro::Git::Pr.summarize_reviews(info['reviews'])
         pr.review_decision = info['reviewDecision']
         pr.is_draft        = info['isDraft']
@@ -317,6 +317,8 @@ class Hiiro
             only_frozen ? "　❄️" : "　❌"
           elsif has_pending
             "⏳　"
+          elsif c['truncated']
+            "　❓"
           else
             "　✅"
           end
@@ -494,44 +496,24 @@ class Hiiro
     def fetch_batch_for_repo(owner, name, pr_numbers)
       return {} if pr_numbers.empty?
 
+      context_fragment = <<~GRAPHQL.strip
+        __typename
+        ... on CheckRun { __typename name conclusion status detailsUrl }
+        ... on StatusContext { __typename context state targetUrl }
+      GRAPHQL
+
       pr_queries = pr_numbers.map.with_index do |num, idx|
         <<~GRAPHQL.strip
           pr#{idx}: pullRequest(number: #{num}) {
-            number
-            title
-            url
-            headRefName
-            state
-            isDraft
-            mergeable
-            reviewDecision
+            number title url headRefName state isDraft mergeable reviewDecision
             statusCheckRollup {
-              contexts(last: 250) {
-                nodes {
-                  ... on CheckRun {
-                    __typename
-                    name
-                    conclusion
-                    status
-                    detailsUrl
-                  }
-                  ... on StatusContext {
-                    __typename
-                    context
-                    state
-                    targetUrl
-                  }
-                }
-              }
-            }
-            reviews(last: 50) {
-              nodes {
-                author {
-                  login
-                }
-                state
+              contexts(last: 100) {
+                totalCount
+                pageInfo { hasPreviousPage startCursor }
+                nodes { #{context_fragment} }
               }
             }
+            reviews(last: 50) { nodes { author { login } state } }
           }
         GRAPHQL
       end
@@ -557,18 +539,36 @@ class Hiiro
         pr_data = repo_data["pr#{idx}"]
         next unless pr_data
 
+        contexts_data = pr_data.dig('statusCheckRollup', 'contexts')
+        nodes       = contexts_data&.[]('nodes') || []
+        total_count = contexts_data&.[]('totalCount').to_i
+        page_info   = contexts_data&.[]('pageInfo') || {}
+
+        # Paginate backwards to collect all checks beyond the first 100
+        all_nodes = nodes.dup
+        cursor = page_info['startCursor']
+        while page_info['hasPreviousPage'] && cursor
+          extra_nodes, page_info = fetch_contexts_page(owner, name, num, cursor, context_fragment)
+          break unless extra_nodes
+          all_nodes = extra_nodes + all_nodes
+          cursor = page_info&.[]('startCursor')
+        end
+
+        truncated = total_count > 0 && all_nodes.length < total_count
+
         pr_info_by_key[[num, repo_path]] = {
-          'number' => pr_data['number'],
-          'title' => pr_data['title'],
-          'url' => pr_data['url'],
-          'headRefName' => pr_data['headRefName'],
-          'state' => pr_data['state'],
-          'isDraft' => pr_data['isDraft'],
-          'mergeable' => pr_data['mergeable'],
-          'reviewDecision' => pr_data['reviewDecision'],
-          'statusCheckRollup' => pr_data.dig('statusCheckRollup', 'contexts', 'nodes'),
-          'reviews' => pr_data.dig('reviews', 'nodes') || [],
-          'repo' => repo_path
+          'number'           => pr_data['number'],
+          'title'            => pr_data['title'],
+          'url'              => pr_data['url'],
+          'headRefName'      => pr_data['headRefName'],
+          'state'            => pr_data['state'],
+          'isDraft'          => pr_data['isDraft'],
+          'mergeable'        => pr_data['mergeable'],
+          'reviewDecision'   => pr_data['reviewDecision'],
+          'statusCheckRollup'=> all_nodes.any? ? all_nodes : nil,
+          'checksTruncated'  => truncated,
+          'reviews'          => pr_data.dig('reviews', 'nodes') || [],
+          'repo'             => repo_path
         }
       end
 
@@ -577,6 +577,33 @@ class Hiiro
       {}
     end
 
+    def fetch_contexts_page(owner, name, pr_number, before_cursor, context_fragment)
+      query = <<~GRAPHQL
+        query {
+          repository(owner: "#{owner}", name: "#{name}") {
+            pullRequest(number: #{pr_number}) {
+              statusCheckRollup {
+                contexts(last: 100, before: "#{before_cursor}") {
+                  pageInfo { hasPreviousPage startCursor }
+                  nodes { #{context_fragment} }
+                }
+              }
+            }
+          }
+        }
+      GRAPHQL
+
+      result = `gh api graphql -f query='#{query.gsub("'", "'\\''")}' 2>/dev/null`
+      return [nil, nil] if result.empty?
+
+      contexts = JSON.parse(result).dig('data', 'repository', 'pullRequest', 'statusCheckRollup', 'contexts')
+      return [nil, nil] unless contexts
+
+      [contexts['nodes'] || [], contexts['pageInfo'] || {}]
+    rescue JSON::ParserError, StandardError
+      [nil, nil]
+    end
+
     def check_run_emoji(conclusion, status)
       return "⏳" if %w[QUEUED IN_PROGRESS PENDING REQUESTED WAITING].include?(status) && conclusion.nil?
       case conclusion

data/lib/hiiro/version.rb

@@ -1,3 +1,3 @@
 class Hiiro
-  VERSION = "0.1.289"
+  VERSION = "0.1.290"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hiiro
 version: !ruby/object:Gem::Version
-  version: 0.1.289
+  version: 0.1.290
 platform: ruby
 authors:
 - Joshua Toyota