hiiro 0.1.321 → 0.1.323

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 155f21221068bb182837aa80c7b920edfb592d46003cc697761441fca7e104d4
-  data.tar.gz: c911e69d3f63924790edd1103f299450ee19bbf70587f54ab205817a0ed9956b
+  metadata.gz: 3f0e6f5a6312065c5ed74198ae2c2eb9e2a00b313a716946c920722805908ecf
+  data.tar.gz: 88a43b3e4225ef3ce0d99b7eeb7f31d0aa3b4adc521ea7563d31f3ef4bed203c
 SHA512:
-  metadata.gz: e7f3f13b81a41a625160d27c7e90a093f20678cd835f5ef1eadc95999fa00c17b41eaac793135389573746e233120b41f94363f87e59bbffac1357c9a573db47
-  data.tar.gz: 195f053e63a71b2975b4fb29addb85893e91012b1482e08547169e762d4651d9397c0a228b9e29735b3e16928d37f2cee906281ed0e674155366806590e7bdaa
+  metadata.gz: b79f44eab5cd1d65e6fe231242cd82492d280356d5ca571ec3d012b0b36c5d06e21b1d914bd37a6b7a98b1ad7f108e2fbefcffb127ff6efa19fa1c869e449410
+  data.tar.gz: ddf1662f78450f2167db30dba65f582a5cf863ca123c69dc5a7be66917f3741bc179f7e6437970914b17714bd78c3776b7509d8fc226e7bf83a5103d53f4930d

data/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## [0.1.323] - 2026-04-01
+
+### Changed
+- Simplify `h pr ls` status refresh logic: always call `refresh_all_status` with `force:` parameter instead of conditional block
+- Add optional `verbose:` parameter to `refresh_all_status` to control "already checked" message output
+- Update `h pr update` to pass `verbose: true` when refreshing active PR status
+
+## [0.1.322] - 2026-04-01
+
+### Changed
+- Refactor `h link tags` filtering logic to use new `Hiiro::Tag.tags_by_type` helper
+- Simplify tag query to use `Hiiro::Link.where(id:)` instead of manual filtering
+- Extract `tags_by_type(type)` singleton method to `Hiiro::Tag` for code reuse
+
 ## [0.1.321] - 2026-04-01
 
 ### Changed
@@ -271,4 +285,4 @@
 ## [0.1.295]
 
 ### Changed
-- Filter logic changes for PR management
+- Filter logic changes for PR management

data/bin/h-link

@@ -528,35 +528,21 @@ Hiiro.run(*ARGV, plugins: [Pins], links_file: lm.links_file) do
   end
 
   add_subcmd(:tags) do |*tag_args|
-    all_known = Hiiro::Tag.where(taggable_type: 'Link').distinct.pluck(:name).sort
+    all_known = Hiiro::Tag.tags_by_type('Link')
+    filtered = all_known.select{|t|
+      tag_args.any?{|ta| t.start_with?(ta) }
+    }
 
-    if tag_args.empty?
-      if all_known.empty?
-        puts "No tags found."
-      else
-        all_known.each { |t| puts t }
-      end
-      next
-    end
-
-    # Prefix-match the given args against known tags
-    matched_tags = tag_args.flat_map { |arg|
-      all_known.select { |t| t.start_with?(arg) }
-    }.uniq
-
-    if matched_tags.empty?
-      puts "No tags match: #{tag_args.join(', ')}"
-      next
-    end
-
-    # Batch-load all link tags to avoid N+1
-    tagged_ids = Hiiro::Tag.where(taggable_type: 'Link', name: matched_tags).map { |t| t.taggable_id.to_i }.uniq
-    links = lm.load_links.select { |link| tagged_ids.include?(link.id) }
-
-    if links.empty?
-      puts "No links tagged with: #{matched_tags.join(', ')}"
+    if filtered.empty?
+      puts "No tags found."
     else
-      links.each_with_index { |link, idx| puts link.display_string(idx) }
+      filtered.each do |tag_name|
+        ids = Hiiro::Tag.where(taggable_type: 'Link', name: tag_name).select(:taggable_id).map(&:taggable_id).map(&:to_i)
+        # links = lm.load_links.select  |link| tagged_ids.include?(link.id) }
+        links = Hiiro::Link.where(id: ids)
+        puts "#{tag_name}:"
+        links.each { |link| puts "  #{link.display_string(exclude_tags: [tag_name])}" }
+      end
     end
   end
 

data/bin/h-pr

@@ -242,13 +242,8 @@ Hiiro.run(*ARGV, plugins: [Pins]) do
     pinned = pinned_manager.load_pinned
     next puts "No tracked PRs" if pinned.empty?
 
-    if opts.update
-      active = pinned.select(&:active?)
-      puts "Updating status for #{active.length} active PR(s)..."
-      pinned_manager.refresh_all_status(pinned, force: true)
-      pinned_manager.save_pinned(pinned)
-      puts
-    end
+    pinned_manager.refresh_all_status(pinned, force: opts.update)
+    pinned_manager.save_pinned(pinned)
 
     results = pinned_manager.apply_filters(pinned, opts)
 
@@ -334,7 +329,7 @@ Hiiro.run(*ARGV, plugins: [Pins]) do
 
     active = pinned.select(&:active?)
     puts "Updating status for #{active.length} active PR(s) (#{pinned.length - active.length} closed/merged skipped)..."
-    pinned_manager.refresh_all_status(pinned, force: opts.force_update)
+    pinned_manager.refresh_all_status(pinned, force: opts.force_update, verbose: true)
     pinned_manager.save_pinned(pinned)
     puts "Done."
 

data/lib/hiiro/link.rb

@@ -40,11 +40,11 @@ class Hiiro
       terms.all? { |term| searchable.include?(term.downcase) }
     end
 
-    def display_string(index = nil)
+    def display_string(index = nil, exclude_tags: [])
       num = index ? "#{(index + 1).to_s.rjust(3)}." : ""
       shorthand_str = shorthand ? " [#{shorthand}]" : ""
       desc_str = description.to_s.empty? ? "" : " - #{description}"
-      link_tags = tags
+      link_tags = tags - Array(exclude_tags)
       tags_str = link_tags.any? ? " \e[30;104m#{link_tags.join(' ')}\e[0m" : ""
       "#{num}#{shorthand_str} #{url}#{desc_str}#{tags_str}".strip
     end

data/lib/hiiro/pinned_pr_manager.rb

@@ -271,11 +271,11 @@ class Hiiro
       result
     end
 
-    def refresh_all_status(prs, force: false)
+    def refresh_all_status(prs, force: false, verbose: false)
       prs_to_refresh = prs.select { |pr| pr.active? && needs_refresh?(pr, force: force) }
 
       if prs_to_refresh.empty?
-        puts "All PRs recently checked (within last 2 minutes). Use -U to force update." unless force
+        puts "All PRs recently checked (within last 2 minutes). Use -U to force update." if verbose
         return prs
       end
 

data/lib/hiiro/tags.rb

@@ -21,6 +21,11 @@ class Hiiro
       klass&.[](taggable_id)
     end
 
+    # Returns all Tag rows for a given object
+    def self.tags_by_type(type)
+      where(taggable_type: type.to_s).select(:name).distinct.map(&:name).sort
+    end
+
     # Returns all Tag rows for a given object
     def self.for(obj)
       t = obj.class.name.split('::').last

data/lib/hiiro/version.rb

@@ -1,3 +1,3 @@
 class Hiiro
-  VERSION = "0.1.321"
+  VERSION = "0.1.323"
 end

metadata

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