brainiac-github 0.2.3 → 0.2.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 43962acb8ab8344a4e420e841550114d53082045252252ad340944dad73d267f
-  data.tar.gz: eb8769a6f13771b1d057133fe8c72c3f3482602528ad0494475eb14022ea0d33
+  metadata.gz: 4cb6c5132e15d851261b150a636b895a0d8d8f6487b51a5049290a1ce2f12274
+  data.tar.gz: c6bf867e0bb5dbca4008e0971dc0568fb95755c2b296b7865e889e2d09645120
 SHA512:
-  metadata.gz: bf3b0e3b90397dfd8232b2bd1fefab1f228dbb2b206d4f9b2766cbd064b6b2c9f4007bd651a7f3ab4ae91bd8934d674e814c931ef4549cf38eaa70b58d74b182
-  data.tar.gz: b22dae8eacf0a45f9032eaadf63a33e945a17d4647ced3195254eb48d17962c2cf459a52214948d29407066c5217a09f21c093876cdd4dbde5e177e1296cabd2
+  metadata.gz: ed2481547416f7bd7d0194aff3ec96da00811c6fe2a4ee8478b4f90c63820fde0318e7ea2427ba07646bd5755abcba3ac2a3c32af78170abb6eebb461f54a4dc
+  data.tar.gz: 3df84f31506b7fbeeaf7636f84b8d8602a9e6fee9d6d8a6a4b18b4f5b76a9de666df79e5c2dccb9c81d1f90727bc8ffd60cc8b784776ea8f55ef0385e3234791

data/lib/brainiac/plugins/github/handler.rb

@@ -16,7 +16,8 @@ module Brainiac
             default_branch = payload.dig("repository", "default_branch") || "main"
 
             # Extract card number from branch name (e.g., "fizzy-1182-add-goodbye-method" → 1182)
-            card_number_from_branch = branch&.match(/^fizzy-(\d+)-/)&.[](1)&.to_i
+            card_number_from_branch = branch&.match(/^fizzy-(\d+)-/)
+            card_number_from_branch = card_number_from_branch[1].to_i if card_number_from_branch
 
             # Emit a hook for ALL merges (including epic branches) so consumers can decide
             if card_number_from_branch
@@ -126,6 +127,14 @@ module Brainiac
             review_state = review["state"]
             reviewer = review.dig("user", "login")
 
+            # If the PR was opened by a remote agent bot (not local to this machine),
+            # skip dispatch — the other machine that owns the agent will handle it.
+            unless pr_owned_locally?(pr)
+              pr_author = pr.dig("user", "login")
+              LOG.info "[GitHub] PR ##{pr_number} owned by remote agent (#{pr_author}) — skipping dispatch"
+              return [200, { status: "ignored", reason: "PR owned by remote agent" }.to_json]
+            end
+
             project_result = identify_project_by_repo(repo_name)
             return [200, { status: "ignored", reason: "no matching project" }.to_json] unless project_result
 
@@ -192,6 +201,16 @@ module Brainiac
               return [200, { status: "ignored", reason: "not a PR comment" }.to_json]
             end
 
+            # If the PR was opened by a remote agent bot (not local to this machine),
+            # skip dispatch — the other machine that owns the agent will handle it.
+            pr_author = issue.dig("user", "login")
+            pr_author_type = issue.dig("user", "type")
+            unless pr_author_owned_locally?(pr_author, pr_author_type)
+              pr_number = issue["number"]
+              LOG.info "[GitHub] PR ##{pr_number} owned by remote agent (#{pr_author}) — skipping dispatch"
+              return [200, { status: "ignored", reason: "PR owned by remote agent" }.to_json]
+            end
+
             project_result = identify_project_by_repo(repo_name)
             unless project_result
               LOG.info "No project found for GitHub repo #{repo_name}"
@@ -317,6 +336,52 @@ module Brainiac
             nil
           end
 
+          # Determine if this machine should handle a PR based on who opened it.
+          # If the PR was opened by a bot that maps to a known agent, only the
+          # machine where that agent is local should dispatch. Human-opened PRs
+          # are always considered local (handled by normal routing logic).
+          #
+          # This prevents both machines from dispatching when they share the same
+          # GitHub webhook — e.g. Adam's Kaylee opens a PR, only Adam's machine handles it.
+          def pr_owned_locally?(pull_request)
+            pr_author = pull_request.dig("user", "login")
+            pr_author_type = pull_request.dig("user", "type")
+            pr_author_owned_locally?(pr_author, pr_author_type)
+          end
+
+          # Check PR ownership given the author login and account type.
+          # Used by both pull_request_review (has full PR object) and issue_comment
+          # (has issue.user which is the PR author on PR-linked issues).
+          def pr_author_owned_locally?(pr_author, pr_author_type)
+            # Human-opened PRs are always handled (normal routing applies)
+            return true unless pr_author_type == "Bot"
+
+            # Extract agent name from bot login (e.g. "kaylee-brainiac[bot]" → "Kaylee")
+            agent_from_bot = agent_name_from_bot_login(pr_author)
+            return true unless agent_from_bot
+
+            # If the bot's agent is local to this machine, we own it
+            local_agent_names.include?(agent_from_bot)
+          end
+
+          # Extract a Brainiac agent display name from a GitHub bot login.
+          # Bot logins follow the pattern "<name>-brainiac[bot]" or "<name>-brainiac".
+          # Returns the matched agent display name, or nil if not a known agent bot.
+          def agent_name_from_bot_login(bot_login)
+            return nil unless bot_login
+
+            # Normalize: remove [bot] suffix, lowercase
+            normalized = bot_login.to_s.downcase.delete_suffix("[bot]").strip
+
+            # Match against known agents — bot logins are "<agent>-brainiac"
+            all_agent_names.each do |name|
+              return name if normalized == "#{name.downcase}-brainiac"
+              return name if normalized == name.downcase
+            end
+
+            nil
+          end
+
           # Generate a GH_TOKEN env var for the agent process so `gh` CLI
           # authenticates as the app bot instead of the user's personal account.
           # Falls back to the shared "brainiac" app if the agent's own app isn't configured.

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

@@ -3,7 +3,7 @@
 module Brainiac
   module Plugins
     module Github
-      VERSION = "0.2.3"
+      VERSION = "0.2.4"
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: brainiac-github
 version: !ruby/object:Gem::Version
-  version: 0.2.3
+  version: 0.2.4
 platform: ruby
 authors:
 - Andy Davis