brainiac-github 0.3.1 → 0.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fa974a43197bb85272bc7a2b53c44b2dc763203d6f5ef7c1286059b50aa2cd97
-  data.tar.gz: 15a725f96507c8cf78dc37e73e1e91505b03ba1c5ee21b7f4c5a8c583f353503
+  metadata.gz: 675324c048d76415c0f4cd440ff81ed8eea72aa2806451882ea2bbe24e26303a
+  data.tar.gz: 5d30ef4b03cfdf213331c70b17ad0d275a07e3b8e7f1b869cde62665852cf0c5
 SHA512:
-  metadata.gz: c9f0b4c4c6c453a6c146a9ca5e3fa8aea3657a2aa79345a44e0803121cda492f1ff598256d05de2d4e649058fdfe128249499533c8dc9410c5880194795b1d56
-  data.tar.gz: 2a1c93fb97fe48873c4dbffb654d4b4c305929404c70336704c509a06ff0408a33bbb674dcd66bb98356caf6e1ec223b81022445c5b3901da5775342c95df777
+  metadata.gz: bef506b5391c0344d6b7f3cb25edc9c0cb4843b708fc3ddf7f2ec3fa37426855ed54d201872f41901e9e2773928ddf34527f25dbd7075b8575f7f3ed5b6538ec
+  data.tar.gz: 6e5f83d6f1598ce872505ae9eb1bad25296868846f1e93cdfea77aa9012fe956e479577b0adaa90d43a4525f6961da9ec5d3fbb3fb0fe1673a1aa16b2b78cee6

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

@@ -181,13 +181,14 @@ module Brainiac
             result = find_work_item_by_branch(branch)
 
             if result
-              _internal_id, card_info = result
+              work_item_id, card_info = result
               card_number = extract_card_number(card_info)
               unless card_number
                 LOG.warn "Card has no number — can't dispatch review"
                 return [200, { status: "ignored", reason: "card has no number" }.to_json]
               end
             else
+              work_item_id = nil
               card_info = {}
               card_number = nil
             end
@@ -236,7 +237,7 @@ module Brainiac
             card_context = card_number ? " for card ##{card_number}" : ""
             LOG.info "PR review submitted by #{reviewer} on PR ##{pr_number}#{card_context} (project: #{project_key})"
             dispatch_pr_review(card_number, card_key, card_info, pr_number, review, reviewer,
-                               repo_name, project_key, project_config, repo_path)
+                               repo_name, project_key, project_config, repo_path, work_item_id: work_item_id)
 
             [200, { status: "processed", card: card_number, pr: pr_number, reviewer: reviewer, project: project_key }.to_json]
           rescue StandardError => e
@@ -744,12 +745,12 @@ module Brainiac
             agent_env = github_agent_env(agent_name, repo_name)
             run_agent_opts = { project_config: project_config, chdir: worktree,
                                log_name: "pr-comment-#{pr_number}",
-                               model: detect_model(project_config, text: comment_body),
-                               effort: detect_effort(project_config, text: comment_body),
                                agent_name: agent_name, source: :github,
                                source_context: { pr_number: pr_number, repo_name: repo_name, work_dir: worktree },
                                message: comment_body, channel: "GitHub PR comment",
                                context: intent_ctx }
+            work_item_id = find_work_item_by_worktree(worktree)&.first if respond_to?(:find_work_item_by_worktree, true)
+            run_agent_opts.merge!(dispatch_override_opts(project_config, comment_body, agent_name, work_item_id))
             run_agent_opts[:env] = agent_env if method(:run_agent).parameters.flatten.include?(:env)
             pid, log_file = run_agent(prompt, **run_agent_opts)
             return unless pid
@@ -758,7 +759,7 @@ module Brainiac
           end
 
           def dispatch_pr_review(card_number, card_key, card_info, pr_number, review, reviewer,
-                                 repo_name, project_key, project_config, repo_path)
+                                 repo_name, project_key, project_config, repo_path, work_item_id: nil)
             review_id = review["id"]
             agent_name = resolve_work_item_agent(card_info, project_config)
 
@@ -801,6 +802,7 @@ module Brainiac
                                source_context: { pr_number: pr_number, repo_name: repo_name, work_dir: work_dir },
                                message: review["body"], channel: "GitHub PR review",
                                context: fetch_pr_intent_context(pr_number, repo_name, agent_name) }
+            run_agent_opts.merge!(dispatch_override_opts(project_config, review["body"].to_s, agent_name, work_item_id))
             run_agent_opts[:env] = agent_env if method(:run_agent).parameters.flatten.include?(:env)
             pid, log_file = run_agent(prompt, **run_agent_opts)
             return unless pid
@@ -808,6 +810,36 @@ module Brainiac
             register_session(card_key, pid, log_file: log_file, agent_name: agent_name)
           end
 
+          # run_agent options for the CLI provider, model, effort and profile of a dispatch.
+          # Core's resolve_dispatch_overrides parses the [cli:X]/[model]/[effort:X]/[p:X] tags,
+          # falls back to the work item's stored overrides (so a card switched to claude in
+          # Fizzy stays on claude for PR comments), resolves the model through the provider
+          # that will actually run, and persists new tags. Older core builds lack it, so fall
+          # back to the per-tag detectors there.
+          def dispatch_override_opts(project_config, text, agent_name, work_item_id)
+            supported = method(:run_agent).parameters.flatten
+            opts = if respond_to?(:resolve_dispatch_overrides, true)
+                     overrides = resolve_dispatch_overrides(project_config, text: text, agent_name: agent_name,
+                                                                            work_item_id: work_item_id)
+                     { cli_provider: overrides[:cli_provider], model: overrides[:model], effort: overrides[:effort],
+                       profile: overrides[:profile], explicit_model: overrides[:explicit_model] }
+                   else
+                     legacy_override_opts(project_config, text)
+                   end
+            opts.compact.slice(*supported)
+          end
+
+          def legacy_override_opts(project_config, text)
+            cli_provider = respond_to?(:detect_cli_provider, true) ? detect_cli_provider(text: text) : nil
+            if respond_to?(:detect_model_explicit, true)
+              model, explicit = detect_model_explicit(project_config, text: text, cli_provider_override: cli_provider)
+            else
+              model = detect_model(project_config, text: text)
+            end
+            { cli_provider: cli_provider, model: model, explicit_model: explicit,
+              effort: detect_effort(project_config, text: text) }
+          end
+
           def build_review_context(reviewer, review, pr_number, repo_name, agent_key = nil)
             context = "GitHub PR Review from @#{reviewer}:\n\n"
             context += "Review body:\n#{review["body"]}\n\n" if review["body"] && !review["body"].empty?

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

@@ -3,7 +3,7 @@
 module Brainiac
   module Plugins
     module Github
-      VERSION = "0.3.1"
+      VERSION = "0.3.3"
     end
   end
 end

metadata

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