hiiro 0.1.289 → 0.1.291

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 37466eb01d7284c838b3ba958f35a41e0f33f4f4fd5c22036ce11f2daa13bfb4
-  data.tar.gz: 55b421c1b3c124cc265d14c93f7709d0e2c894d4497125884306fbe53fe29762
+  metadata.gz: 03a9ad967ffc6e338367882dd85b3305099e521dc7a0a2341acf71257ca22bde
+  data.tar.gz: 80f4746d3cdb168b9275e8ca4e55822d356966f4467f147b68741cde3045cfa6
 SHA512:
-  metadata.gz: dc41d3e7f2fc019d20336f0890dcff641d545a3bb25a6b83147fc1067b3a139a2548c498b8f9e825cbe7096c2a4d2d77706ed9e21d79b36b26459db54a2ea3f4
-  data.tar.gz: 403890caf1a18fe7b5c231b1fd12c2e4d07897867c8e46620441923f2860b4e5c0f7ab848105edb4c773a7d2724f5a4be4048e37155ec31bce78ea387f07fc1c
+  metadata.gz: 6f5076627ad3767f575122e566090160a35953059efb115e89509a855af089dec0f5f32336c33cf001cc3bc8d6fa3a364bde840ad4e1d7a2274c354c0e59b35f
+  data.tar.gz: 7b6cdcbe3c7640f6fbf27d42ee42aa72fcc6644d5f11c0ddec482d6ea98de8dc5f3712606d221e1c520acbbc66cb3c60fae1eea8ae2c8bf4d7175345465da2bd

data/CHANGELOG.md

@@ -1,9 +1,22 @@
 ```markdown
+## v0.1.291 (2026-03-26)
+
+### Added
+- `Options#mutual_exclusion(*names)` — star-topology mutual exclusion: first flag is the hub (clears all others when set); any other flag only clears the hub (spokes can coexist freely); last encountered in argv wins
+- `h pr ls`: new `--all`/`-a` flag (show all tracked PRs, no filter); all filter flags are declared mutually exclusive so `-oa`, `-ao`, `--all --active` etc. do the right thing
+
+## 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/bin/h-pr

@@ -238,7 +238,9 @@ Hiiro.run(*ARGV, plugins: [Pins]) do
       flag(:verbose,  short: 'v', desc: 'multi-line output per PR')
       flag(:checks,   short: 'C', desc: 'show individual check run details')
       flag(:diff,     short: 'd', desc: 'open diff for selected PR')
+      flag(:all,      short: 'a', desc: 'show all tracked PRs (no filter)')
       instance_eval(&FILTER_OPTS)
+      mutual_exclusion(:all, :active, :merged, :drafts, :red, :green, :pending, :conflicts)
     }
 
     if opts.help

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/options.rb

@@ -63,6 +63,19 @@ class Hiiro
       self
     end
 
+    # Declare a mutual exclusion group with star topology.
+    # The first name is the hub; the rest are spokes.
+    #   Setting the hub   clears all spokes.
+    #   Setting a spoke   clears only the hub.
+    #   Spokes can still be combined with each other freely.
+    # Two-member groups are fully symmetric (hub == spoke).
+    # Last flag encountered in argv always wins.
+    def mutual_exclusion(*names)
+      @mutex_groups ||= []
+      @mutex_groups << names.map(&:to_sym)
+      self
+    end
+
     private
 
     def deconflict_short(short)
@@ -82,7 +95,7 @@ class Hiiro
     end
 
     def parse(args)
-      Args.new(@definitions, args.flatten.compact)
+      Args.new(@definitions, args.flatten.compact, mutex_groups: @mutex_groups || [])
     end
 
     def parse!(args)
@@ -93,8 +106,9 @@ class Hiiro
       attr_reader :remaining_args, :original_args
       alias args remaining_args
 
-      def initialize(definitions, raw_args)
+      def initialize(definitions, raw_args, mutex_groups: [])
         @definitions = definitions
+        @mutex_groups = mutex_groups
         @original_args = raw_args.dup.freeze
         @values = {}
         @remaining_args = []
@@ -182,7 +196,7 @@ class Hiiro
         return unless defn
 
         if defn.flag? || defn.flag_active?(@values)
-          @values[defn.name] = !defn.default
+          set_flag(defn, !defn.default)
         else
           value ||= args.shift
           store_value(defn, value)
@@ -197,7 +211,7 @@ class Hiiro
           next unless defn
 
           if defn.flag? || defn.flag_active?(@values)
-            @values[defn.name] = !defn.default
+            set_flag(defn, !defn.default)
           elsif idx == chars.length - 1
             store_value(defn, args.shift)
           else
@@ -207,6 +221,28 @@ class Hiiro
         end
       end
 
+      def set_flag(defn, value)
+        # Star topology: group[0] is the hub.
+        #   Setting the hub   → clears all spokes (group[1..])
+        #   Setting a spoke   → clears only the hub (group[0])
+        # This lets spokes coexist with each other (e.g. --red --drafts is fine)
+        # while still preventing any spoke from combining with the hub (--all).
+        @mutex_groups.each do |group|
+          next unless group.include?(defn.name)
+          hub, *spokes = group
+          if defn.name == hub
+            spokes.each do |spoke|
+              spoke_defn = @definitions[spoke]
+              @values[spoke] = spoke_defn.default if spoke_defn
+            end
+          else
+            hub_defn = @definitions[hub]
+            @values[hub] = hub_defn.default if hub_defn
+          end
+        end
+        @values[defn.name] = value
+      end
+
       def store_value(defn, value)
         coerced = defn.coerce(value)
         if defn.multi

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.291"
 end

metadata

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