hiiro 0.1.292 → 0.1.293

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b7bcfce2d0fd9d428a59696b9d95ffcc57a9ba875cf7cc865123303661905586
-  data.tar.gz: de38f5fe9cadba6b68d1dc7382f749b3c474372ea037da3d3fd4b2da9793d046
+  metadata.gz: 20e69313f458dee5ff24926e4c905ea46bcfa67146f45df57f08e5692c0c9005
+  data.tar.gz: cf1b62233a0c03e1bc024ec966b88b9687cbc384b2cbf593dae30b9587ade3ff
 SHA512:
-  metadata.gz: be11793dd32b0d8f95073004723fecc88793929b55c6f98134d3e571345850702d9acdb4d879ae2b2235d5d73dbae1aa910d0eea7372a1ed7e53fefb6cd349a1
-  data.tar.gz: 3c0bb0253dfa9171c22a64cef87fcb8e2bd14579101a9ab26b29f30dabcd4e0d8e7877f8dcadbc9064b158693729bf81909b15a34ee089076a069f814db36081
+  metadata.gz: d3db13bf516d8dfa5db32bd1480de76d886ed1dc51f9465749b4ee6f7b2ce75e420cc035bf3446828fe70383aa6b2fd315c6f76ae8f7bfb7dd2943250dc7a7ff
+  data.tar.gz: 402ff47ab1de9c80d226c853914ed7c2a9d21a03e2d275138545a73551bdbc62275a8e53bf884dafdfa8037b6f55b93ecb98cf90b63cf89eb32f2c512f6ad4dc

data/CHANGELOG.md

@@ -1,4 +1,9 @@
 ```markdown
+## v0.1.293 (2026-03-26)
+
+### Changed
+- `h pr update`: filter options now extracted to `PinnedPRManager` for better code organization and reusability
+
 ## v0.1.292 (2026-03-26)
 
 ### Added

data/bin/h-pr

@@ -7,20 +7,8 @@ require "yaml"
 require "json"
 require "tempfile"
 
-FILTER_OPTS = Proc.new {
-  flag(:red,       short: 'r', desc: 'filter: failing checks')
-  flag(:green,     short: 'g', desc: 'filter: passing checks')
-  flag(:conflicts, short: 'c', desc: 'filter: merge conflicts')
-  flag(:drafts,    short: 'D', desc: 'filter: draft PRs')
-  flag(:pending,   short: 'p', desc: 'filter: pending checks')
-  flag(:merged,    short: 'm', desc: 'filter: merged PRs')
-  flag(:active,    short: 'o', desc: 'filter: open (non-merged) PRs')
-  flag(:numbers,   short: 'n', desc: 'output PR numbers only (no #)')
-  option(:tag,     short: 't', desc: 'filter by tag (OR when multiple; AND with flag filters)', multi: true)
-}
-
 Hiiro.run(*ARGV, plugins: [Pins]) do
-  pinned_manager = Hiiro::PinnedPRManager.new
+  pm = pinned_manager = Hiiro::PinnedPRManager.new
   Hiiro::PinnedPRManager.add_resolvers(self)
 
   watch_block = ->(original_pr_number=nil, *watch_args) {
@@ -239,7 +227,7 @@ Hiiro.run(*ARGV, plugins: [Pins]) do
       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)
+      pm.add_options(self)
       mutual_exclusion(:all, :active, :merged, :drafts, :red, :green, :pending, :conflicts)
     }
 
@@ -329,7 +317,7 @@ Hiiro.run(*ARGV, plugins: [Pins]) do
     opts = Hiiro::Options.parse(update_args) {
       flag(:force_update, long: 'force-update', short: 'u', desc: 'force refresh even if recently checked')
       flag(:all, short: 'a', desc: 'show all tracked PRs (no filter)')
-      instance_eval(&FILTER_OPTS)
+      pm.add_options(self)
       mutual_exclusion(:all, :active, :merged, :drafts, :red, :green, :pending, :conflicts)
     }
 
@@ -350,7 +338,7 @@ Hiiro.run(*ARGV, plugins: [Pins]) do
   end
 
   add_subcmd(:green) do |*green_args|
-    opts = Hiiro::Options.parse(green_args, &FILTER_OPTS)
+    opts = Hiiro::Options.parse(green_args) { |o| pm.add_options(o) }
     pinned = pinned_manager.load_pinned
     results = pinned_manager.apply_filters(pinned, opts, forced: [:green])
 
@@ -367,7 +355,7 @@ Hiiro.run(*ARGV, plugins: [Pins]) do
   end
 
   add_subcmd(:red) do |*red_args|
-    opts = Hiiro::Options.parse(red_args, &FILTER_OPTS)
+    opts = Hiiro::Options.parse(red_args) { |o| pm.add_options(o) }
     pinned = pinned_manager.load_pinned
     results = pinned_manager.apply_filters(pinned, opts, forced: [:red])
 
@@ -384,7 +372,7 @@ Hiiro.run(*ARGV, plugins: [Pins]) do
   end
 
   add_subcmd(:old) do |*old_args|
-    opts = Hiiro::Options.parse(old_args, &FILTER_OPTS)
+    opts = Hiiro::Options.parse(old_args) { |o| pm.add_options(o) }
     pinned = pinned_manager.load_pinned
     results = pinned_manager.apply_filters(pinned, opts, forced: [:merged])
 
@@ -428,7 +416,7 @@ Hiiro.run(*ARGV, plugins: [Pins]) do
   end
 
   add_subcmd(:draft) do |*draft_args|
-    opts = Hiiro::Options.parse(draft_args, &FILTER_OPTS)
+    opts = Hiiro::Options.parse(draft_args) { |o| pm.add_options(o) }
     pinned = pinned_manager.load_pinned
     results = pinned_manager.apply_filters(pinned, opts, forced: [:drafts])
 

data/lib/hiiro/pinned_pr_manager.rb

@@ -55,14 +55,26 @@ class Hiiro
       end
     end
 
-    def pr_repo(pr)
-      pr.repo || Hiiro::Git::Pr.repo_from_url(pr.url)
-    end
-
     def initialize
       ensure_file
     end
 
+    def add_options(opts)
+      opts.flag(:red,       short: 'r', desc: 'filter: failing checks')
+      opts.flag(:green,     short: 'g', desc: 'filter: passing checks')
+      opts.flag(:conflicts, short: 'c', desc: 'filter: merge conflicts')
+      opts.flag(:drafts,    short: 'D', desc: 'filter: draft PRs')
+      opts.flag(:pending,   short: 'p', desc: 'filter: pending checks')
+      opts.flag(:merged,    short: 'm', desc: 'filter: merged PRs')
+      opts.flag(:active,    short: 'o', desc: 'filter: open (non-merged) PRs')
+      opts.flag(:numbers,   short: 'n', desc: 'output PR numbers only (no #)')
+      opts.option(:tag,     short: 't', desc: 'filter by tag (OR when multiple; AND with flag filters)', multi: true)
+    end
+
+    def pr_repo(pr)
+      pr.repo || Hiiro::Git::Pr.repo_from_url(pr.url)
+    end
+
     def ensure_file
       dir = File.dirname(Hiiro::Git::Pr::PINNED_FILE)
       FileUtils.mkdir_p(dir) unless Dir.exist?(dir)

data/lib/hiiro/version.rb

@@ -1,3 +1,3 @@
 class Hiiro
-  VERSION = "0.1.292"
+  VERSION = "0.1.293"
 end

metadata

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