brainiac-basecamp 0.0.6 → 0.0.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a07338e5677f448dd7b2fb15bb345009532fef8669fc375526738a8cf0eaf24d
-  data.tar.gz: 8d2aaabe74e5d02a770a3f38366287362aeab07dea91954f3ca54e9ae71ef17d
+  metadata.gz: f3b4d7d9e99a700f2a533b48110071025bde9eb69b55633acffd4125f88c5540
+  data.tar.gz: 63635c64597b65ed6e0a69e90511010d6975c418249c7a181d9a0349bded7dcc
 SHA512:
-  metadata.gz: a217e3701939b30771f21eb8696662052eb8abdb99ae78936a9f27b14044c263aff7a3291cf51b04467d24f1c4b4cab993945503b5ca8e63f004348e8ae0409d
-  data.tar.gz: 939aeb9b76e0e38bdb2132ea3f5e0ad14096600bab0a1cb844425ec2dd101c8222d79a20da29e2c31abfe850134eb2725b8d9b5820ddd26544d777852f6f0508
+  metadata.gz: 37d7d00ab786c8f99318d7098b8a700d8428d0c3db1da4ceaa1040aa924b627b0b886a928cb7f6f1661e35f578403ae4b8911b87a5c48c6bc67a9c0ed5f61c02
+  data.tar.gz: a798eb76376698539c16c54ecdfea9d597893e0f0630c035e9e473401e45b43c0357e4cb6ef8717df704c98543f2bace4754b5f2db6dc97677a46ee8782b2ff9

data/lib/brainiac/plugins/basecamp/review_gate.rb

@@ -203,6 +203,74 @@ module Brainiac
             dispatched
           end
 
+          # Dispatch only gate agents that have NOT yet responded (no approval, no changes_requested).
+          # Used during resume/health-check when some gates responded but others didn't.
+          #
+          # Tracks per-gate re-dispatch count to prevent infinite loops when a gate agent
+          # keeps crashing. After MAX_GATE_REDISPATCH_RETRIES, stops retrying that gate.
+          #
+          # IMPORTANT: Does NOT overwrite gates_dispatched_at — that timestamp represents
+          # the original dispatch and is used for stale detection. Overwriting it would
+          # reset the timeout window on every partial re-dispatch, causing infinite loops.
+          #
+          # @param epic [Hash] Epic state
+          # @param task [Hash] Task state
+          # @param pr_number [Integer, String] PR number
+          # @param repo_name [String] e.g. "stowzilla/brainiac-basecamp"
+          # @param repo_path [String] Local repo path
+          # @return [Array<String>] Agent names dispatched
+          def dispatch_missing_gates(epic:, task:, pr_number:, repo_name:, repo_path:)
+            responded_agents = Set.new
+            (task["gate_approvals"] || []).each { |a| responded_agents << a["agent"].downcase }
+            (task["changes_requested_by"] || []).each { |a| responded_agents << a.downcase }
+
+            missing_gates = gates.reject { |g| responded_agents.include?(g["agent"].downcase) }
+            return [] if missing_gates.empty?
+
+            # Track per-gate re-dispatch counts to prevent infinite loops
+            task["gate_redispatch_counts"] ||= {}
+
+            dispatched = []
+
+            missing_gates.each do |gate|
+              agent_name = gate["agent"]
+              role = gate["role"] || "review"
+
+              # Check retry count — stop if we've exceeded the max
+              retries = task["gate_redispatch_counts"][agent_name.downcase] || 0
+              if retries >= MAX_GATE_REDISPATCH_RETRIES
+                LOG.warn "[Basecamp:ReviewGate] Gate #{agent_name} exceeded max re-dispatch retries (#{retries}/#{MAX_GATE_REDISPATCH_RETRIES}) — skipping" if defined?(LOG)
+                next
+              end
+
+              LOG.info "[Basecamp:ReviewGate] Re-dispatching missing gate #{agent_name} (#{role}) to review PR ##{pr_number} (retry #{retries + 1}/#{MAX_GATE_REDISPATCH_RETRIES})" if defined?(LOG)
+
+              task["gate_redispatch_counts"][agent_name.downcase] = retries + 1
+
+              Thread.new do
+                dispatch_agent_for_review(
+                  agent_name: agent_name,
+                  role: role,
+                  pr_number: pr_number,
+                  repo_name: repo_name,
+                  repo_path: repo_path,
+                  card_number: task["fizzy_card"],
+                  epic: epic
+                )
+              rescue StandardError => e
+                LOG.error "[Basecamp:ReviewGate] Failed to dispatch #{agent_name}: #{e.message}" if defined?(LOG)
+              end
+
+              dispatched << agent_name
+            end
+
+            # Record when this partial re-dispatch happened (for diagnostics)
+            # but do NOT overwrite gates_dispatched_at — that's the original timestamp
+            task["last_redispatch_at"] = Time.now.iso8601
+
+            dispatched
+          end
+
           # Build the summary comment for Fizzy after all gates pass and merge completes.
           #
           # @param task [Hash] Task state

data/lib/brainiac/plugins/basecamp/version.rb

@@ -3,7 +3,7 @@
 module Brainiac
   module Plugins
     module Basecamp
-      VERSION = "0.0.6"
+      VERSION = "0.0.9"
     end
   end
 end

data/lib/brainiac/plugins/basecamp.rb

@@ -16,6 +16,15 @@ require_relative "basecamp/cli"
 module Brainiac
   module Plugins
     module Basecamp
+      # Maximum seconds to wait for a dispatched agent/gate to respond before
+      # considering it stale and re-dispatching. Used across resume, health-check,
+      # and dispatch_missing_gates.
+      STALE_DISPATCH_TIMEOUT = 300 # 5 minutes
+
+      # Maximum number of times a gate will be re-dispatched for the same review
+      # cycle before giving up (prevents infinite re-dispatch loops).
+      MAX_GATE_REDISPATCH_RETRIES = 3
+
       class << self
         # Called by Brainiac plugin system during server startup.
         #
@@ -107,22 +116,38 @@ module Brainiac
               end
 
             when "in_flight"
-              # Check if impl agent is assigned when changes were requested
-              if task["changes_requested_by"]&.any?
-                impl_agent = epic["agent"]
-                card_json, = Open3.capture2("fizzy", "card", "show", card_number.to_s, "--json")
-                card_data = JSON.parse(card_json) rescue {}
-                assignees = card_data.dig("data", "assignees") || []
-                assignee_names = assignees.map { |a| a["name"]&.downcase }
-
-                unless assignee_names.include?(impl_agent.downcase)
-                  LOG.info "[Basecamp:HealthCheck] Task ##{card_number} in_flight with changes_requested but #{impl_agent} not assigned — healing" if defined?(LOG)
+              # Check if impl agent session is likely dead after a restart.
+              # Instead of just checking Fizzy assignment (stale artifact), use
+              # dispatched_at as a liveness signal — if dispatched too long ago
+              # without completion, assume the session died and re-dispatch.
+              dispatched_at = task["dispatched_at"]
+              if dispatched_at
+                elapsed = Time.now - Time.parse(dispatched_at)
+                # If task has been in_flight for longer than the stale timeout
+                # AND no agent is actively running (we can't check PID, so use elapsed time)
+                if elapsed > STALE_DISPATCH_TIMEOUT
+                  impl_agent = epic["agent"]
+                  LOG.info "[Basecamp:HealthCheck] Task ##{card_number} in_flight for #{elapsed.round}s — re-dispatching #{impl_agent}" if defined?(LOG)
+
+                  # Re-dispatch by re-assigning the card (triggers webhook)
+                  task["dispatched_at"] = Time.now.iso8601
                   fizzy_user_id = Orchestrator.send(:resolve_fizzy_user_id, impl_agent)
                   if fizzy_user_id
-                    Open3.capture3("fizzy", "card", "assign", card_number.to_s, "--user", fizzy_user_id)
+                    Hooks.send(:safe_assign_card, card_number, fizzy_user_id)
                     healed_any = true
                   end
                 end
+              elsif task["changes_requested_by"]&.any?
+                # No dispatched_at recorded but changes were requested — legacy state.
+                # Always re-dispatch in this case.
+                impl_agent = epic["agent"]
+                LOG.info "[Basecamp:HealthCheck] Task ##{card_number} in_flight with changes_requested but no dispatched_at — re-dispatching #{impl_agent}" if defined?(LOG)
+                task["dispatched_at"] = Time.now.iso8601
+                fizzy_user_id = Orchestrator.send(:resolve_fizzy_user_id, impl_agent)
+                if fizzy_user_id
+                  Hooks.send(:safe_assign_card, card_number, fizzy_user_id)
+                  healed_any = true
+                end
               end
 
             when "in_review"
@@ -131,10 +156,11 @@ module Brainiac
                 projects_file = File.join(ENV.fetch("BRAINIAC_DIR", File.join(Dir.home, ".brainiac")), "projects.json")
                 projects = File.exist?(projects_file) ? JSON.parse(File.read(projects_file)) : {}
                 repo_path = projects.dig(project_key, "repo_path")
+                github_repo = projects.dig(project_key, "github_repo")
 
                 if repo_path
                   # Sync from GitHub to catch any missed approvals
-                  sync_result = ReviewGate.sync_from_github(task, repo_path: repo_path)
+                  ReviewGate.sync_from_github(task, repo_path: repo_path)
 
                   if ReviewGate.all_gates_passed?(task)
                     # All approved — advance to final_decision
@@ -164,6 +190,42 @@ module Brainiac
                       end
                     end
                     healed_any = true
+                  elsif (task["gate_approvals"] || []).empty? && !task["changes_requested_by"]&.any?
+                    # No gate responses at all — gates may never have been dispatched or agents crashed
+                    gates_dispatched_at = task["gates_dispatched_at"]
+                    should_redispatch = gates_dispatched_at.nil? || (Time.now - Time.parse(gates_dispatched_at)) > STALE_DISPATCH_TIMEOUT
+
+                    if should_redispatch && github_repo
+                      LOG.info "[Basecamp:HealthCheck] No gate responses for ##{card_number} — re-dispatching review gates" if defined?(LOG)
+                      ReviewGate.dispatch_gates(
+                        epic: epic,
+                        task: task,
+                        pr_number: pr_number,
+                        repo_name: github_repo,
+                        repo_path: repo_path
+                      )
+                      Hooks.send(:save_epic_state, epic)
+                      healed_any = true
+                    end
+                  else
+                    # Partial responses — some gates responded but others haven't
+                    gates_dispatched_at = task["gates_dispatched_at"]
+                    if gates_dispatched_at && (Time.now - Time.parse(gates_dispatched_at)) > STALE_DISPATCH_TIMEOUT
+                      responded_count = (task["gate_approvals"]&.size || 0) + (task["changes_requested_by"]&.size || 0)
+                      total_gates = ReviewGate.gates.size
+                      if responded_count.positive? && responded_count < total_gates && github_repo
+                        LOG.info "[Basecamp:HealthCheck] Partial gate response for ##{card_number} (#{responded_count}/#{total_gates}) — re-dispatching missing gates" if defined?(LOG)
+                        ReviewGate.dispatch_missing_gates(
+                          epic: epic,
+                          task: task,
+                          pr_number: pr_number,
+                          repo_name: github_repo,
+                          repo_path: repo_path
+                        )
+                        Hooks.send(:save_epic_state, epic)
+                        healed_any = true
+                      end
+                    end
                   end
                 end
               end
@@ -260,30 +322,71 @@ module Brainiac
             Hooks.send(:save_epic_state, epic)
             Hooks.send(:dispatch_final_decision, epic, task, {})
           elsif task["changes_requested_by"]&.any?
-            # Gates requested changes — check if impl agent is assigned
+            # Gates requested changes — after a restart, no agent session is running even if assigned.
+            # Always re-dispatch the impl agent to address the feedback.
             impl_agent = epic["agent"]
-            LOG.info "[Basecamp] Resume: ##{card_number} has changes requested, checking assignment" if defined?(LOG)
+            LOG.info "[Basecamp] Resume: ##{card_number} has changes requested — re-dispatching #{impl_agent}" if defined?(LOG)
 
-            # Check current assignees via Fizzy CLI
-            card_json, = Open3.capture2("fizzy", "card", "show", card_number.to_s, "--json")
-            card_data = JSON.parse(card_json) rescue {}
-            assignees = card_data.dig("data", "assignees") || []
-            assignee_names = assignees.map { |a| a["name"]&.downcase }
+            # Transition to in_flight so the agent gets the right context
+            task["status"] = "in_flight"
+            Hooks.send(:save_epic_state, epic)
 
-            if assignee_names.include?(impl_agent.downcase)
-              LOG.info "[Basecamp] Resume: ##{card_number} already assigned to #{impl_agent} — waiting for fixes" if defined?(LOG)
-            else
-              # Re-assign to trigger dispatch
-              LOG.info "[Basecamp] Resume: ##{card_number} not assigned to #{impl_agent} — re-assigning" if defined?(LOG)
-              fizzy_user_id = Orchestrator.send(:resolve_fizzy_user_id, impl_agent)
-              if fizzy_user_id
-                Open3.capture3("fizzy", "card", "assign", card_number.to_s, "--user", fizzy_user_id)
-              end
+            # Re-assign card to trigger a fresh dispatch via the Fizzy webhook
+            fizzy_user_id = Orchestrator.send(:resolve_fizzy_user_id, impl_agent)
+            if fizzy_user_id
+              Hooks.send(:safe_assign_card, card_number, fizzy_user_id)
             end
           else
-            # No changes requested yet — just waiting for reviews
+            # No changes requested yet — check if gates need (re-)dispatching
             approvals = task["gate_approvals"]&.size || 0
-            LOG.info "[Basecamp] Resume: ##{card_number} has #{approvals} approvals, waiting for more reviews" if defined?(LOG)
+            gates_dispatched_at = task["gates_dispatched_at"]
+
+            # Calculate how many gates have responded (approved or requested changes)
+            responded_count = approvals + (task["changes_requested_by"]&.size || 0)
+            total_gates = ReviewGate.gates.size
+
+            # Re-dispatch gates if:
+            # 1. Gates were never dispatched (missed webhook), OR
+            # 2. Gates were dispatched but no responses received after 5 minutes, OR
+            # 3. Some gates responded but others didn't after 5 minutes (partial response)
+            should_redispatch = if gates_dispatched_at.nil?
+                                  true
+                                else
+                                  elapsed = Time.now - Time.parse(gates_dispatched_at)
+                                  elapsed > STALE_DISPATCH_TIMEOUT && responded_count < total_gates
+                                end
+
+            if should_redispatch && ReviewGate.enabled?
+              github_repo = projects.dig(project_key, "github_repo")
+              if github_repo
+                if responded_count.zero?
+                  # No responses at all — dispatch all gates
+                  LOG.info "[Basecamp] Resume: ##{card_number} has 0/#{total_gates} gate responses — re-dispatching all review gates" if defined?(LOG)
+                  ReviewGate.dispatch_gates(
+                    epic: epic,
+                    task: task,
+                    pr_number: pr_number,
+                    repo_name: github_repo,
+                    repo_path: repo_path
+                  )
+                else
+                  # Partial responses — dispatch only the missing gates
+                  LOG.info "[Basecamp] Resume: ##{card_number} has #{responded_count}/#{total_gates} gate responses — re-dispatching missing gates" if defined?(LOG)
+                  ReviewGate.dispatch_missing_gates(
+                    epic: epic,
+                    task: task,
+                    pr_number: pr_number,
+                    repo_name: github_repo,
+                    repo_path: repo_path
+                  )
+                end
+                Hooks.send(:save_epic_state, epic)
+              else
+                LOG.warn "[Basecamp] Resume: ##{card_number} — cannot re-dispatch gates, no github_repo for #{project_key}" if defined?(LOG)
+              end
+            else
+              LOG.info "[Basecamp] Resume: ##{card_number} has #{responded_count}/#{total_gates} gate responses, waiting for more reviews" if defined?(LOG)
+            end
           end
         end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: brainiac-basecamp
 version: !ruby/object:Gem::Version
-  version: 0.0.6
+  version: 0.0.9
 platform: ruby
 authors:
 - Andy Davis