hiiro 0.1.230 → 0.1.232

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: deca910b037607a912c6aee097095ec1598dd13a449e403a8958cf94abfe675b
-  data.tar.gz: 17dd925fba07d3e8b4b827c9fa986f605c7cb37bea25441d5a084d5b2feef701
+  metadata.gz: a0d1ba68ed577241b03e05bbc748dbc880b0c411547a646bdc7bb9c77733f8c3
+  data.tar.gz: e25ca589b0609094596456dbef9f9afcb83963045dea1e2d809a50c227f4d7cd
 SHA512:
-  metadata.gz: 0ef54e118391b0f7c99a238626ebe005db813cf21fbab0136075d0927c52c464af117368a487c410f786113203b055c0c9a4f8f184f87a639e9b32214a6c728c
-  data.tar.gz: f88f14553901994056540f67ba493f2b1b0247fb46355baa66da87856ace0ac949106a68331f0cedd97d6a5496cb910d9451e74a4ed6dea1b9f970fd8b6fa95c
+  metadata.gz: 5c3558fd44e96b1b68ead9f8526f05c7de18e56fd0cd7345c77a3d8c649f6a236a82133aecd94676efd988e90dd192804a33120aca15aca73d1394d3c4043f76
+  data.tar.gz: e1ca686b11cdf7f0d795eb0a03a4275e660af825cf30f01a36563586f5843a75e5a4148bba648cf8b7c9bac264bcbac1b2f2dbb79c7324fedc349621f8e17c39

data/bin/h-notify

@@ -13,6 +13,25 @@ TYPE_PRESETS = {
 
 LOG_FILE = Hiiro::Config.data_path('notify_log.yml')
 
+TMUX_CONF       = File.join(Dir.home, '.tmux.conf')
+NOTIFY_CONF     = File.join(Dir.home, '.config', 'tmux', 'h-notify.tmux.conf')
+CLAUDE_SETTINGS = File.join(Dir.home, '.claude', 'settings.json')
+
+NOTIFY_TMUX_HOOKS = %w[after-kill-pane window-unlinked session-closed].freeze
+
+NOTIFY_TMUX_CONF_CONTENT = <<~'CONF'
+  # --- Notify hooks (clean up notifications when panes/windows/sessions close) ---
+  set-hook -g after-kill-pane  "run-shell -b 'h notify remove_pane \#{hook_pane_id}'"
+  set-hook -g window-unlinked  "run-shell -b 'h notify remove_window \#{hook_window_id}'"
+  set-hook -g session-closed   "run-shell -b 'h notify remove_session \#{hook_session_name}'"
+
+  # --- Notify keybinding (prefix + N to open notification menu) ---
+  bind-key N run-shell "h notify menu"
+CONF
+
+CLAUDE_NOTIFICATION_CMD = %q(MSG=$(cat | jq -r '.message // "Needs input"'); h alert -t 'Claude Code' -m "$MSG" -s k -c "echo switchc -t '$TMUX_PANE' | pbcopy"; h notify push -t info "$MSG")
+CLAUDE_STOP_CMD         = %q(h alert -t 'Claude Code' -m 'Work completed' -s b -c "echo switchc -t '$TMUX_PANE' | pbcopy"; h notify push -t success 'Work completed')
+
 def read_log
   return {} unless File.exist?(LOG_FILE)
   YAML.safe_load(File.read(LOG_FILE)) || {}
@@ -26,6 +45,16 @@ def write_log(data)
   File.write(LOG_FILE, data.to_yaml)
 end
 
+def current_pane
+  # Prefer $TMUX_PANE env var — it's reliably set even in hook subprocesses
+  env_id = ENV['TMUX_PANE'].to_s
+  if !env_id.empty?
+    Hiiro::Tmux::Panes.fetch(all: true).find_by_id(env_id)
+  else
+    Hiiro::Tmux::Pane.current
+  end
+end
+
 def pane_exists?(pane_id)
   Hiiro::Tmux::Panes.fetch(all: true).find_by_id(pane_id)
 end
@@ -44,7 +73,7 @@ Hiiro.run(*ARGV, tasks: true) do
       next
     end
 
-    pane    = Hiiro::Tmux::Pane.current
+    pane    = current_pane
     session = pane&.session_name || Hiiro::Tmux::Session.current&.name
 
     unless pane && session
@@ -64,7 +93,7 @@ Hiiro.run(*ARGV, tasks: true) do
 
     data = read_log
     data[session] ||= []
-    # One entry per pane — replace existing entry for this pane, newest first
+    # One entry per pane — replace existing, newest first
     data[session].reject! { |e| e['pane_id'] == pane.id }
     data[session].unshift(entry)
     write_log(data)
@@ -167,7 +196,7 @@ Hiiro.run(*ARGV, tasks: true) do
     tmux_client.display_message('Notifications cleared')
   end
 
-  # Called by tmux hook: after-kill-pane / pane-died
+  # Called by tmux hook: after-kill-pane
   add_subcmd(:remove_pane) do |pane_id = nil|
     next unless pane_id
     data = read_log
@@ -175,7 +204,7 @@ Hiiro.run(*ARGV, tasks: true) do
     write_log(data)
   end
 
-  # Called by tmux hook: window-closed
+  # Called by tmux hook: window-unlinked
   add_subcmd(:remove_window) do |window_id = nil|
     next unless window_id
     data = read_log
@@ -191,78 +220,134 @@ Hiiro.run(*ARGV, tasks: true) do
     write_log(data)
   end
 
-  add_subcmd(:setup) do
-    conf_dir    = File.join(Dir.home, '.config', 'tmux')
-    conf_file   = File.join(conf_dir, 'h-notify.tmux.conf')
-    tmux_conf   = File.join(Dir.home, '.tmux.conf')
-    source_line = "source-file #{conf_file}"
-
-    FileUtils.mkdir_p(conf_dir)
-
-    notify_conf = <<~'CONF'
-      # --- Notify hooks (clean up notifications when panes/windows/sessions close) ---
-      set-hook -g after-kill-pane  "run-shell -b 'h notify remove_pane \#{hook_pane_id}'"
-      set-hook -g window-unlinked  "run-shell -b 'h notify remove_window \#{hook_window_id}'"
-      set-hook -g session-closed   "run-shell -b 'h notify remove_session \#{hook_session_name}'"
-
-      # --- Notify keybinding (prefix + N to open notification menu) ---
-      bind-key N run-shell "h notify menu"
-    CONF
-
-    File.write(conf_file, notify_conf)
-    puts "Wrote #{conf_file}"
-
-    if File.exist?(tmux_conf)
-      existing = File.read(tmux_conf)
-      if existing.include?(source_line)
-        puts "#{tmux_conf} already sources #{conf_file}"
-      else
-        File.open(tmux_conf, 'a') { |f| f.puts; f.puts source_line }
-        puts "Added '#{source_line}' to #{tmux_conf}"
+  # --- h notify tmux ---
+
+  add_subcmd(:tmux) do
+    make_child {
+      add_subcmd(:setup) do
+        FileUtils.mkdir_p(File.dirname(NOTIFY_CONF))
+        File.write(NOTIFY_CONF, NOTIFY_TMUX_CONF_CONTENT)
+        puts "Wrote #{NOTIFY_CONF}"
       end
-    else
-      File.write(tmux_conf, source_line + "\n")
-      puts "Created #{tmux_conf} with source line"
-    end
 
-    puts
-    puts "Reload tmux config:"
-    puts "  tmux source-file ~/.tmux.conf"
-    puts
-    puts "─────────────────────────────────────────────────"
-    puts "Claude Code hook setup"
-    puts "─────────────────────────────────────────────────"
-    puts
-    puts "Run this to auto-update ~/.claude/settings.json:"
-    puts "  h notify update_hooks"
-    puts
-    puts "Or manually update hooks to:"
-    puts
-    puts '  Notification: h alert ... ; h notify push -t info "$MSG"'
-    puts '  Stop:         h alert ... ; h notify push -t success "Work completed"'
+      add_subcmd(:add_hooks) do
+        source_line = "source-file #{NOTIFY_CONF}"
+
+        if File.exist?(TMUX_CONF)
+          existing = File.read(TMUX_CONF)
+          if existing.include?(source_line)
+            puts "#{TMUX_CONF} already sources #{NOTIFY_CONF}"
+          else
+            File.open(TMUX_CONF, 'a') { |f| f.puts; f.puts source_line }
+            puts "Added '#{source_line}' to #{TMUX_CONF}"
+          end
+        else
+          File.write(TMUX_CONF, source_line + "\n")
+          puts "Created #{TMUX_CONF} with source line"
+        end
+
+        system('tmux', 'source-file', TMUX_CONF)
+        puts "Reloaded #{TMUX_CONF}"
+      end
+
+      add_subcmd(:reset_hooks) do
+        NOTIFY_TMUX_HOOKS.each do |hook|
+          system('tmux', 'set-hook', '-gu', hook)
+          puts "Unset tmux hook: #{hook}"
+        end
+      end
+
+      add_subcmd(:load_hooks) do
+        system('tmux', 'source-file', TMUX_CONF)
+        puts "Sourced #{TMUX_CONF}"
+      end
+    }.run
   end
 
-  add_subcmd(:update_hooks) do
-    settings_file = File.join(Dir.home, '.claude', 'settings.json')
+  # --- h notify claude ---
 
-    unless File.exist?(settings_file)
-      puts "#{settings_file} not found"
-      next
+  add_subcmd(:claude) do
+    require 'json'
+
+    read_claude_settings = lambda do
+      return {} unless File.exist?(CLAUDE_SETTINGS)
+      JSON.parse(File.read(CLAUDE_SETTINGS))
+    rescue JSON::ParserError
+      {}
     end
 
-    require 'json'
-    settings = JSON.parse(File.read(settings_file))
-    settings['hooks'] ||= {}
+    write_claude_settings = lambda do |settings|
+      File.write(CLAUDE_SETTINGS, JSON.pretty_generate(settings))
+    end
+
+    make_child {
+      add_subcmd(:setup) do
+        settings = read_claude_settings.call
+        settings['hooks'] ||= {}
+        settings['hooks']['Notification'] = [{ 'hooks' => [{ 'type' => 'command', 'command' => CLAUDE_NOTIFICATION_CMD }] }]
+        settings['hooks']['Stop']         = [{ 'hooks' => [{ 'type' => 'command', 'command' => CLAUDE_STOP_CMD }] }]
+        write_claude_settings.call(settings)
+        puts "Updated #{CLAUDE_SETTINGS}"
+        puts "  Notification -> h alert + h notify push -t info"
+        puts "  Stop         -> h alert + h notify push -t success"
+      end
 
-    notification_cmd = %q(MSG=$(cat | jq -r '.message // "Needs input"'); h alert -t 'Claude Code' -m "$MSG" -s k -c "echo switchc -t '$TMUX_PANE' | pbcopy"; h notify push -t info "$MSG")
-    stop_cmd         = %q(h alert -t 'Claude Code' -m 'Work completed' -s b -c "echo switchc -t '$TMUX_PANE' | pbcopy"; h notify push -t success 'Work completed')
+      add_subcmd(:add_hooks) do
+        settings = read_claude_settings.call
+        unless File.exist?(CLAUDE_SETTINGS)
+          puts "#{CLAUDE_SETTINGS} not found"
+          next
+        end
+
+        settings['hooks'] ||= {}
+
+        # Inject h notify push into existing commands if not already present,
+        # otherwise write fresh hooks
+        %w[Notification Stop].each do |event|
+          existing_hooks = settings.dig('hooks', event, 0, 'hooks') || []
+          if existing_hooks.any? { |h| h['command']&.include?('h notify push') }
+            puts "#{event} hook already includes h notify push — skipping"
+            next
+          end
+
+          cmd = event == 'Notification' ? CLAUDE_NOTIFICATION_CMD : CLAUDE_STOP_CMD
+          settings['hooks'][event] = [{ 'hooks' => [{ 'type' => 'command', 'command' => cmd }] }]
+          puts "Set #{event} hook -> h alert + h notify push"
+        end
+
+        write_claude_settings.call(settings)
+      end
 
-    settings['hooks']['Notification'] = [{ 'hooks' => [{ 'type' => 'command', 'command' => notification_cmd }] }]
-    settings['hooks']['Stop']         = [{ 'hooks' => [{ 'type' => 'command', 'command' => stop_cmd }] }]
+      add_subcmd(:reset_hooks) do
+        unless File.exist?(CLAUDE_SETTINGS)
+          puts "#{CLAUDE_SETTINGS} not found"
+          next
+        end
+
+        settings = read_claude_settings.call
+        settings['hooks'] ||= {}
+
+        %w[Notification Stop].each do |event|
+          hooks = settings.dig('hooks', event, 0, 'hooks') || []
+          hooks.each do |h|
+            next unless h['command']&.include?('h notify push')
+            # Strip the h notify push portion from the command
+            h['command'] = h['command']
+              .split(';')
+              .reject { |part| part.strip.start_with?('h notify push') }
+              .join(';')
+              .strip
+          end
+          puts "Stripped h notify push from #{event} hook"
+        end
+
+        write_claude_settings.call(settings)
+      end
 
-    File.write(settings_file, JSON.pretty_generate(settings))
-    puts "Updated #{settings_file}"
-    puts "  Notification -> h alert + h notify push -t info"
-    puts "  Stop         -> h alert + h notify push -t success"
+      add_subcmd(:load_hooks) do
+        puts "Claude Code settings are loaded automatically on startup."
+        puts "Restart claude to pick up changes to #{CLAUDE_SETTINGS}"
+      end
+    }.run
   end
 end

data/bin/h-pr

@@ -1071,6 +1071,81 @@ Hiiro.run(*ARGV, plugins: [Pins]) do
     system('gh', 'pr', 'merge', pr_number.to_s, *merge_args)
   end
 
+  add_subcmd(:fix) do |*fix_args|
+    opts = Hiiro::Options.parse(fix_args) do
+      flag(:red, short: :r, desc: 'Fix all PRs with failing checks')
+      flag(:run, short: :R, desc: 'Launch queue tasks immediately after queuing')
+    end
+
+    pinned = pinned_manager.load_pinned
+
+    prs_to_fix = if opts.red
+      failing = pinned.select { |pr| (c = pr['checks']) && c['failed'].to_i > 0 }
+      if failing.empty?
+        puts "No PRs with failing checks"
+        next
+      end
+      failing
+    elsif opts.args.empty?
+      # Default: fuzzy-select from failing PRs, fall back to all tracked
+      failing = pinned.select { |pr| (c = pr['checks']) && c['failed'].to_i > 0 }
+      candidates = failing.empty? ? pinned : failing
+      if candidates.empty?
+        puts "No tracked PRs"
+        next
+      end
+      lines = candidates.each_with_index.each_with_object({}) do |(pr, idx), h|
+        h[pinned_manager.display_pinned(pr, idx)] = pr
+      end
+      selected = fuzzyfind_from_map(lines)
+      next unless selected
+      [selected]
+    else
+      # Resolve each positional arg as a PR ref
+      opts.args.map { |ref|
+        pr_num = resolve_pr.call(ref)
+        next nil unless pr_num
+        pinned.find { |p| p['number'].to_s == pr_num.to_s }
+      }.compact
+    end
+
+    if prs_to_fix.empty?
+      puts "No PRs to fix"
+      next
+    end
+
+    q = Hiiro::Queue.current(self)
+    queued_names = []
+
+    prs_to_fix.each do |pr|
+      task_info = {
+        task_name:    pr['task'],
+        tree_name:    pr['worktree'],
+        session_name: pr['tmux_session'],
+      }.compact
+      task_info = nil if task_info.empty?
+
+      result = q.add_with_frontmatter("/pr:fix #{pr['number']}", task_info: task_info)
+      if result
+        puts "Queued fix for ##{pr['number']}: #{pr['title']}"
+        queued_names << result[:name]
+      else
+        STDERR.puts "Failed to queue fix for ##{pr['number']}"
+      end
+    end
+
+    unless queued_names.empty?
+      puts
+      puts "Queued #{queued_names.length} fix task(s)."
+      if opts.run
+        puts "Launching..."
+        queued_names.each { |name| q.launch_task(name) }
+      else
+        puts "Run 'h queue run' to start."
+      end
+    end
+  end
+
   add_subcmd(:comment) do |ref = nil|
     pr_number = resolve_pr.call(ref)
     next unless pr_number

data/lib/hiiro/version.rb

@@ -1,3 +1,3 @@
 class Hiiro
-  VERSION = "0.1.230"
+  VERSION = "0.1.232"
 end

metadata

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