brainiac-fizzy 0.0.28 → 0.0.29

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e5d55859eafd2aeccf63a2083999340a1ce4a77c01b81a8297adf136b0e553b3
-  data.tar.gz: b1dbff19165dec2cf391f4c258ae2a207457cdc53088ee44062ffda050530a48
+  metadata.gz: 93dfc73c8a3984f834883fbc59057446f220cf640ef14feecdc3882eab0f5e79
+  data.tar.gz: 16dccf767e981ab8b7ec4f4c9ad570a6af713fe9deb410f0e3163f8745def891
 SHA512:
-  metadata.gz: 9438b9143636ab612da09a19be5b672ada809095317ea7ce72d9045acbebe6601d5485ba4bfd234b85edcdb54a6d676edd330a74b4241cc85c0605d590fe523d
-  data.tar.gz: 93fbe26b2a546b1b72e1025f7af334ac6539efe0c4144338bd6d2929b336e2ed21752cc8a1d4ef2949d380cc72635e945e0dad819afe043570c483ddc889d2d7
+  metadata.gz: dfabf4b7239f9fa9181fb327c24660e7c39c0a1558151362330ff67c8c334ba9565239e5da35c1335c67bd5dfdd246cacf8295509e6e61c86b9afb0f2f6be43d
+  data.tar.gz: ebafbeceb274b07e70f23f69d8b7123af573d646e58d92eb756b693bf2cf4d87a12ab7fc79ddca3ced785c54d3beca9c8c602a4c655d0158067cc952d441a90f

data/README.md

@@ -37,6 +37,8 @@ Fizzy configuration lives in `~/.brainiac/fizzy.json` (same as before):
     { "id": "user-id-1", "name": "Andy", "human": true },
     { "id": "agent-id-1", "name": "Galen", "human": false }
   ],
+  "uat_agent": false,
+  "uat_agent_tag": "uat",
   "boards": {
     "development": {
       "board_id": "your-board-id",
@@ -51,6 +53,25 @@ Fizzy configuration lives in `~/.brainiac/fizzy.json` (same as before):
 }
 ```
 
+### UAT agent (`uat_agent`)
+
+When a PR is merged and its card moves to the UAT column, Brainiac can dispatch
+an LLM agent to write manual testing steps as a card comment. This spends tokens
+on every merge, so it's **disabled by default**. Set `"uat_agent": true` at the
+top level of `fizzy.json` to opt back in globally. The card still moves to UAT
+and gets the PR-merged comment either way — only the agent dispatch is gated.
+
+**Per-card opt-in:** Even with `uat_agent` off globally, you can trigger the UAT
+agent for a single card by adding the `uat` tag to it. When the card's PR merges,
+Brainiac fetches the card's live tags and dispatches the UAT agent if the tag is
+present. Customize the tag name with `"uat_agent_tag": "your-tag"`, or set it to
+`false`/`null` to disable per-card overrides entirely. This gives you three
+states:
+
+- `uat_agent: true` — dispatch on every merge
+- `uat_agent: false` (default) + `uat` tag on a card — dispatch for that card only
+- `uat_agent: false` + no tag — never dispatch (pure bookkeeping, zero tokens)
+
 ## What This Plugin Handles
 
 | Event | Action |

data/lib/brainiac/plugins/fizzy/config.rb

@@ -82,6 +82,26 @@ module Brainiac
             nil
           end
 
+          # Whether to dispatch an LLM agent to write UAT testing steps when a
+          # PR is merged and the card moves to UAT. Disabled by default to save
+          # tokens — set "uat_agent": true in ~/.brainiac/fizzy.json to enable
+          # globally. Individual cards can still opt in via the UAT tag even when
+          # this is false (see #uat_agent_tag).
+          def uat_agent_enabled?
+            @config.fetch("uat_agent", false) == true
+          end
+
+          # The card tag that opts a single card into UAT agent dispatch even
+          # when uat_agent is globally disabled. Defaults to "uat". Set
+          # "uat_agent_tag": "some-tag" in fizzy.json to customize, or false/null
+          # to disable per-card overrides entirely.
+          def uat_agent_tag
+            value = @config.fetch("uat_agent_tag", "uat")
+            return nil if value == false || value.nil? || value.to_s.strip.empty?
+
+            value.to_s
+          end
+
           def authorized?(payload)
             creator_id = payload.dig("creator", "id")
             @authorized_user_ids.include?(creator_id)

data/lib/brainiac/plugins/fizzy/hooks.rb

@@ -134,36 +134,7 @@ module Brainiac
           # PR merged — post comment on Fizzy card, move to UAT, dispatch UAT agent
           def register_pr_merged
             Brainiac.on(:pr_merged) do |ctx|
-              card_number = ctx[:card_number]
-              next unless card_number
-
-              card_info = ctx[:card_info]
-              card_agent = card_info["agent"]
-              env = Helpers.fizzy_env_for(card_agent)
-              repo_path = ctx[:repo_path]
-              board_key = card_info.dig("sources", "fizzy", "board_key") || Config.board_key_for_project(ctx[:project_config])
-
-              # Post PR link comment
-              pr_url = ctx[:pr_url]
-              pr_title = ctx[:pr_title]
-              branch = ctx[:branch]
-              comment_body = "<p>PR merged into main: <a href=\"#{pr_url}\">#{pr_title}</a></p>" \
-                             "<p>Branch: <code>#{branch}</code></p>"
-              run_cmd("fizzy", "comment", "create", "--card", card_number.to_s, "--body", comment_body,
-                      chdir: repo_path, env: env)
-
-              # Move card to UAT
-              Helpers.move_card_to_column(card_number, "uat",
-                                          project_config: ctx[:project_config],
-                                          agent_name: card_agent,
-                                          board_key: board_key)
-              record_self_move(card_number)
-
-              # Clear deployment tracking
-              clear_deployment_for_card(card_number) if respond_to?(:clear_deployment_for_card)
-
-              # Dispatch UAT agent
-              dispatch_fizzy_uat_agent(ctx)
+              handle_pr_merged(ctx)
             rescue StandardError => e
               LOG.error "[Fizzy] Error in pr_merged hook: #{e.message}" if defined?(LOG)
             end
@@ -292,6 +263,75 @@ module Brainiac
 
           # --- Private helpers ---
 
+          # Body of the :pr_merged hook. Posts the PR link comment, moves the
+          # card to UAT, clears deployment tracking, and dispatches the UAT
+          # agent when enabled. Extracted from the hook block to keep the
+          # registration method within complexity limits.
+          def handle_pr_merged(ctx)
+            card_number = ctx[:card_number]
+            return unless card_number
+
+            card_info = ctx[:card_info]
+            card_agent = card_info["agent"]
+            env = Helpers.fizzy_env_for(card_agent)
+            repo_path = ctx[:repo_path]
+            board_key = card_info.dig("sources", "fizzy", "board_key") || Config.board_key_for_project(ctx[:project_config])
+
+            # Post PR link comment
+            comment_body = "<p>PR merged into main: <a href=\"#{ctx[:pr_url]}\">#{ctx[:pr_title]}</a></p>" \
+                           "<p>Branch: <code>#{ctx[:branch]}</code></p>"
+            run_cmd("fizzy", "comment", "create", "--card", card_number.to_s, "--body", comment_body,
+                    chdir: repo_path, env: env)
+
+            # Move card to UAT
+            Helpers.move_card_to_column(card_number, "uat",
+                                        project_config: ctx[:project_config],
+                                        agent_name: card_agent,
+                                        board_key: board_key)
+            record_self_move(card_number)
+
+            # Clear deployment tracking
+            clear_deployment_for_card(card_number) if respond_to?(:clear_deployment_for_card)
+
+            # Dispatch UAT agent — off by default to save tokens. Globally
+            # enabled via "uat_agent": true, or per-card via the UAT tag
+            # (default "uat") even when the global flag is off.
+            if uat_agent_dispatch?(card_number, repo_path: repo_path, env: env)
+              dispatch_fizzy_uat_agent(ctx)
+            else
+              log_uat_dispatch_skipped(card_number)
+            end
+          end
+
+          # Log that UAT dispatch was skipped for a card (global flag off, no tag).
+          def log_uat_dispatch_skipped(card_number)
+            return unless defined?(LOG)
+
+            LOG.info "[Fizzy] UAT agent disabled (set \"uat_agent\": true in fizzy.json, or add the " \
+                     "\"#{Config.uat_agent_tag}\" tag to this card) — skipping dispatch for card ##{card_number}"
+          end
+
+          # Decide whether to dispatch the UAT agent for a merged card.
+          # True when globally enabled, OR when the card carries the configured
+          # UAT tag (per-card opt-in). Card tags aren't in the pr_merged ctx, so
+          # we fetch them live — only when the global flag is off, to avoid an
+          # extra API call when it's already enabled.
+          def uat_agent_dispatch?(card_number, repo_path:, env: nil)
+            return true if Config.uat_agent_enabled?
+
+            tag = Config.uat_agent_tag
+            return false unless tag && card_number && repo_path
+
+            tags = Helpers.fetch_card_tags(card_number, repo_path: repo_path, env: env)
+            return false unless tags
+
+            has_tag = Helpers.card_has_tag?(tags, tag)
+            if has_tag && defined?(LOG)
+              LOG.info "[Fizzy] Card ##{card_number} has \"#{tag}\" tag — dispatching UAT agent despite global flag being off"
+            end
+            has_tag
+          end
+
           def dispatch_fizzy_uat_agent(ctx)
             card_number = ctx[:card_number]
             card_info = ctx[:card_info]

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

@@ -3,7 +3,7 @@
 module Brainiac
   module Plugins
     module Fizzy
-      VERSION = "0.0.28"
+      VERSION = "0.0.29"
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: brainiac-fizzy
 version: !ruby/object:Gem::Version
-  version: 0.0.28
+  version: 0.0.29
 platform: ruby
 authors:
 - Andy Davis