brainiac-fizzy 0.0.13 → 0.0.15

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0e09da26fb3e4d7a37ef7fff687ad5b5e094d6cad508227fac32467273541619
-  data.tar.gz: 4de9000657e8afd645e5dcd04b02c9522912fb2df1428ffca10e4235b9d1b462
+  metadata.gz: 554fe8796537111b5df0a3d5236cb457c28089097f75efb797fdb2a32fa20c6a
+  data.tar.gz: 5a1799ef50971dc53df499213fa00bae7d84c1913da5da070b4c4e96de8e113c
 SHA512:
-  metadata.gz: 3fe71aad4fec21bb3430d71bc9461156779b590c465bb49ec7199e6a4183f9321af16d98b3b6ffd9a01dc8505dd8d8708c9f0d1dfbdef98dd7e3b8c7fe3cea96
-  data.tar.gz: 22cff590e92a0455820feb9f40a0eacf9552f9989ccde9b155e193673c0260f108771469c4d45b739c72626a87927bd79108d2063b85c2fe0aab981681b2f0bd
+  metadata.gz: f2cc672d0cd0760c0c4319a9b5bca323877b8ca7284739a3cd8fbd4f6deeffb0735e7bd9c19d50726b62207453bffab2b228f972b55c1f0ac1289a0490155a12
+  data.tar.gz: 3e7e6d3da09df5f97bfbe47b7b152dca2381c58b3ed75d315ff8979f0341b62563dbae33043d568562c23de06e9cee10d07ce54e2b290bd8d13fbf9c0d322437

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

@@ -33,8 +33,8 @@ def move_card_to_column(card_number, column_name, project_config:, agent_name: n
   Brainiac::Plugins::Fizzy::Helpers.move_card_to_column(card_number, column_name, project_config: project_config, agent_name: agent_name)
 end
 
-def append_fizzy_comment_footer(card_number, project_config:, agent_name: nil)
-  Brainiac::Plugins::Fizzy::Helpers.append_fizzy_comment_footer(card_number, project_config: project_config, agent_name: agent_name)
+def append_fizzy_comment_footer(card_number, project_config:, agent_name: nil, since: nil)
+  Brainiac::Plugins::Fizzy::Helpers.append_fizzy_comment_footer(card_number, project_config: project_config, agent_name: agent_name, since: since)
 end
 
 def ensure_fizzy_yaml!(chdir, project_config)
@@ -136,6 +136,14 @@ PROMPT_MENTION = Brainiac::Plugins::Fizzy::Prompts::MENTION
 PROMPT_CROSS_AGENT_REVIEW = Brainiac::Plugins::Fizzy::Prompts::CROSS_AGENT_REVIEW
 PROMPT_PLANNING_MODE = Brainiac::Plugins::Fizzy::Prompts::PLANNING_MODE
 
+# Default Fizzy board column IDs — used as fallback when board_key is nil or
+# the board config doesn't specify column mappings.
+DEFAULT_COLUMN_IDS = {
+  "right_now" => "03f5xa5q9fog9592pa1279dts",
+  "needs_review" => "03f5ykobhpsd78hbuvajtn8g8",
+  "uat" => "03fsmglsr6az06ppyotawsti8"
+}.freeze
+
 # Render planning mode prompt — identical to render_prompt but inserts the planning
 # instructions between PROMPT_CORE and the channel rules. This was originally defined
 # in brainiac core's planning.rb but belongs here after the fizzy extraction.

data/lib/brainiac/plugins/fizzy/handlers/assignment.rb

@@ -152,7 +152,7 @@ def dispatch_assigned_card(card_number:, card_internal_id:, title:, tags:, branc
                             project_config: project_config, chdir: worktree_path,
                             log_name: "assigned-#{card_number}", model: model, effort: effort,
                             agent_name: agent_name, card_number: card_number, source: :fizzy,
-                            source_context: { card_number: card_number },
+                            source_context: { card_number: card_number, dispatched_at: Time.now },
                             cli_provider: cli_provider_override)
   register_session(card_key, pid, log_file: log_file, supersede_key: card_key, agent_name: agent_name)
 

data/lib/brainiac/plugins/fizzy/handlers/comments.rb

@@ -408,7 +408,7 @@ def dispatch_cross_agent_review(ctx, card_key:, card_number:, card_assigned_agen
                             log_name: "review-#{ctx.agent_name.downcase}-#{card_number || ctx.card_internal_id}",
                             model: ctx.model, effort: ctx.effort, agent_name: ctx.agent_name,
                             card_number: card_number, comment_id: ctx.comment_id,
-                            source: :fizzy, source_context: { card_number: card_number },
+                            source: :fizzy, source_context: { card_number: card_number, dispatched_at: Time.now },
                             cli_provider: ctx.cli_provider_override)
   return [200, { status: "dispatch_failed", agent: ctx.agent_name, card: card_number }.to_json] unless pid
 
@@ -514,7 +514,7 @@ def dispatch_new_mention(ctx, card_key:, card_number:, card_title:, branch:, wor
                             model: ctx.model, effort: ctx.effort, agent_name: ctx.agent_name,
                             card_number: card_number, comment_id: ctx.comment_id,
                             source: :fizzy, cli_provider: ctx.cli_provider_override,
-                            source_context: { card_number: card_number })
+                            source_context: { card_number: card_number, dispatched_at: Time.now })
   return [200, { status: "dispatch_failed", agent: ctx.agent_name, card: card_number }.to_json] unless pid
 
   register_session(card_key, pid, log_file: log_file, supersede_key: card_key, agent_name: ctx.agent_name)
@@ -554,7 +554,7 @@ def dispatch_followup_comment(ctx, card_key:, card_number:, work_dir:)
                             source: :fizzy, cli_provider: ctx.cli_provider_override, resume: should_resume,
                             source_context: {
                               card_number: card_number, card_internal_id: ctx.card_internal_id,
-                              deploy_intent: ctx.deploy_intent
+                              deploy_intent: ctx.deploy_intent, dispatched_at: Time.now
                             })
   return [200, { status: "dispatch_failed", agent: ctx.agent_name, card: card_number }.to_json] unless pid
 

data/lib/brainiac/plugins/fizzy/handlers/summarize.rb

@@ -0,0 +1,154 @@
+# frozen_string_literal: true
+
+require "time"
+
+module Brainiac
+  module Plugins
+    module Fizzy
+      # Handles re-dispatch summarize sessions and fallback comment posting.
+      # Extracted from Hooks to keep module length manageable.
+      module Summarize
+        class << self
+          # Re-dispatch an agent to post a summary comment when it completed work
+          # but didn't comment on the card. Fires async with a 60s timeout.
+          # If this session also fails to comment, posts a minimal fallback.
+          def dispatch_summarize_session(ctx)
+            card_number = ctx[:card_number]
+            agent_name = ctx[:agent_name]
+            project_config = ctx[:project_config]
+            repo_path = project_config["repo_path"]
+            session_started_at = ctx[:source_context]&.dig(:dispatched_at)
+
+            map = load_work_item_map
+            work_item = find_work_item(map, card_number)
+
+            branch = work_item&.dig("branch") || "unknown"
+            card_title = work_item&.dig("title") || ctx[:source_context]&.dig(:card_title) || "untitled"
+            worktree_path = work_item&.dig("worktree") || repo_path
+
+            if defined?(LOG)
+              LOG.info "[Fizzy] Agent #{agent_name} completed card ##{card_number} without commenting — " \
+                       "dispatching summarize session"
+            end
+
+            pid, log_file = dispatch_summarize_agent(card_number, card_title, branch, agent_name,
+                                                     project_config, worktree_path)
+            register_session("card-#{card_number}-summarize", pid, log_file: log_file, agent_name: agent_name)
+
+            monitor_summarize_session(pid, card_number, agent_name, branch, repo_path, session_started_at,
+                                      project_config)
+          rescue StandardError => e
+            LOG.error "[Fizzy] Failed to dispatch summarize session for card ##{card_number}: #{e.message}" if defined?(LOG)
+            post_fallback_comment(card_number, work_item&.dig("branch") || "unknown", agent_name, project_config)
+          end
+
+          # Check if the agent posted a comment on the card since a given time.
+          # If no `since` is provided, checks for any agent comment (legacy behavior).
+          # Subtracts a 30s buffer from `since` to account for clock skew between
+          # the local server and the Fizzy API.
+          CLOCK_SKEW_BUFFER = 30
+
+          def agent_commented_on_card?(card_number, agent_name, repo_path:, since: nil)
+            env = Helpers.fizzy_env_for(agent_name)
+            output = run_cmd("fizzy", "comment", "list", "--card", card_number.to_s, chdir: repo_path, env: env)
+            comments = JSON.parse(output)["data"] || []
+            agent_display = agent_display_name(agent_name)
+
+            agent_comments = comments.select { |c| c["creator_name"]&.downcase == agent_display.downcase }
+            return agent_comments.any? unless since
+
+            buffered_since = since - CLOCK_SKEW_BUFFER
+            agent_comments.any? do |c|
+              comment_time = c["created_at"] && Time.parse(c["created_at"])
+              comment_time && comment_time > buffered_since
+            end
+          rescue StandardError
+            false
+          end
+
+          # Post a minimal server-generated comment when all else fails.
+          def post_fallback_comment(card_number, branch, agent_name, project_config)
+            repo_path = project_config["repo_path"]
+            env = Helpers.fizzy_env_for(agent_name)
+
+            pr_url = Helpers.send(:detect_pr_url, branch, project_config)
+            body = "<p>✅ Work completed on this card.</p>"
+            body += "<p><a href=\"#{pr_url}\">View PR</a></p>" if pr_url
+            body += "<p><strong>Branch:</strong> <code>#{branch}</code></p>"
+
+            run_cmd("fizzy", "comment", "create", "--card", card_number.to_s, "--body", body,
+                    chdir: repo_path, env: env)
+            LOG.info "[Fizzy] Posted fallback comment on card ##{card_number}" if defined?(LOG)
+          rescue StandardError => e
+            LOG.error "[Fizzy] Failed to post fallback comment on card ##{card_number}: #{e.message}" if defined?(LOG)
+          end
+
+          private
+
+          def find_work_item(map, card_number)
+            map.values.find do |info|
+              info.is_a?(Hash) &&
+                (info.dig("sources", "fizzy", "card_number").to_s == card_number.to_s ||
+                 info["number"].to_s == card_number.to_s)
+            end
+          end
+
+          def dispatch_summarize_agent(card_number, card_title, branch, agent_name, project_config, worktree_path)
+            prompt = render_prompt(Prompts::SUMMARIZE_WORK,
+                                   { "CARD_NUMBER" => card_number, "CARD_TITLE" => card_title, "BRANCH" => branch },
+                                   brain_context: "", agent_name: agent_name, channel: :fizzy,
+                                   board_key: Config.board_key_for_project(project_config))
+
+            run_agent(prompt,
+                      project_config: project_config, chdir: worktree_path,
+                      log_name: "summarize-#{card_number}", agent_name: agent_name,
+                      source: :fizzy,
+                      source_context: { card_number: card_number, skip_summarize_redispatch: true },
+                      skip_column_move: true)
+          end
+
+          def monitor_summarize_session(pid, card_number, agent_name, branch, repo_path, session_started_at,
+                                        project_config)
+            Thread.new do
+              wait_for_process_or_timeout(pid, timeout: 60)
+              sleep 2 # Brief pause to let any comment propagate
+
+              unless agent_commented_on_card?(card_number, agent_name, repo_path: repo_path, since: session_started_at)
+                post_fallback_comment(card_number, branch, agent_name, project_config)
+              end
+            rescue Errno::ECHILD
+              # Process already reaped by session manager — check for comment
+              sleep 2
+              unless agent_commented_on_card?(card_number, agent_name, repo_path: repo_path, since: session_started_at)
+                post_fallback_comment(card_number, branch, agent_name, project_config)
+              end
+            rescue StandardError => e
+              LOG.error "[Fizzy] Summarize monitor error for card ##{card_number}: #{e.message}" if defined?(LOG)
+            end
+          end
+
+          def wait_for_process_or_timeout(pid, timeout: 60)
+            deadline = Time.now + timeout
+            loop do
+              result = Process.waitpid2(pid, Process::WNOHANG)
+              break if result
+
+              if Time.now > deadline
+                safe_kill(pid)
+                break
+              end
+              sleep 1
+            end
+          end
+
+          def safe_kill(pid)
+            Process.kill("TERM", pid)
+            Process.waitpid2(pid)
+          rescue Errno::ESRCH, Errno::ECHILD
+            # Process already gone
+          end
+        end
+      end
+    end
+  end
+end

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

@@ -102,7 +102,7 @@ module Brainiac
             run_cmd("fizzy", "card", "column", card_number.to_s, "--column", column_id, chdir: repo_path, env: env)
           end
 
-          def append_fizzy_comment_footer(card_number, project_config:, agent_name: nil)
+          def append_fizzy_comment_footer(card_number, project_config:, agent_name: nil, since: nil)
             repo_path = project_config["repo_path"]
             env = fizzy_env_for(agent_name || AI_AGENT_NAME)
 
@@ -110,29 +110,40 @@ module Brainiac
             comments = JSON.parse(output)["data"] || []
             agent_display = agent_display_name(agent_name || AI_AGENT_NAME)
 
-            last_agent_comment = comments.reverse.find do |c|
-              c["creator_name"]&.downcase == agent_display.downcase
+            # Find the most recent agent comment, scoped to this session if `since` is provided.
+            # Subtracts 30s buffer from `since` to account for clock skew between local server and Fizzy API.
+            agent_comments = comments.select { |c| c["creator_name"]&.downcase == agent_display.downcase }
+            if since
+              buffered_since = since - 30
+              agent_comments = agent_comments.select do |c|
+                comment_time = c["created_at"] && Time.parse(c["created_at"])
+                comment_time && comment_time > buffered_since
+              end
             end
-            return unless last_agent_comment
-
-            # Check if footer already exists
-            body = last_agent_comment.dig("body", "html") || ""
-            return if body.include?("<em>Branch:")
 
-            # Detect branch from comment content or card map
-            branch = detect_branch_from_comment(body, card_number)
-            return unless branch
+            last_agent_comment = agent_comments.last
+            return false unless last_agent_comment
 
-            pr_url = detect_pr_url(branch, project_config)
-            footer = "<p><em>Branch: <code>#{branch}</code>"
-            footer += " | <a href=\"#{pr_url}\">PR</a>" if pr_url
-            footer += "</em></p>"
+            # Append footer if applicable
+            body = last_agent_comment.dig("body", "html") || ""
+            unless body.include?("<em>Branch:")
+              branch = detect_branch_from_comment(body, card_number)
+              if branch
+                pr_url = detect_pr_url(branch, project_config)
+                footer = "<p><em>Branch: <code>#{branch}</code>"
+                footer += " | <a href=\"#{pr_url}\">PR</a>" if pr_url
+                footer += "</em></p>"
+
+                updated_body = body + footer
+                run_cmd("fizzy", "comment", "update", last_agent_comment["id"], "--card", card_number.to_s,
+                        "--body", updated_body, chdir: repo_path, env: env)
+              end
+            end
 
-            updated_body = body + footer
-            run_cmd("fizzy", "comment", "update", last_agent_comment["id"], "--card", card_number.to_s,
-                    "--body", updated_body, chdir: repo_path, env: env)
+            true
           rescue StandardError => e
             LOG.warn "[Fizzy] Could not append footer to card ##{card_number}: #{e.message}" if defined?(LOG)
+            false
           end
 
           def ensure_fizzy_yaml!(chdir, project_config)

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

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require "time"
+
 module Brainiac
   module Plugins
     module Fizzy
@@ -25,6 +27,7 @@ module Brainiac
           private
 
           # After an agent session completes — move card to needs_review, append footer
+          # If the agent didn't post a comment, re-dispatch for a summary.
           def register_agent_completed
             Brainiac.on(:agent_completed) do |ctx|
               next unless ctx[:source] == :fizzy
@@ -39,9 +42,18 @@ module Brainiac
                 record_self_move(card_number)
               end
 
-              Helpers.append_fizzy_comment_footer(card_number,
-                                                  project_config: ctx[:project_config],
-                                                  agent_name: ctx[:agent_name])
+              found_comment = Helpers.append_fizzy_comment_footer(card_number,
+                                                                  project_config: ctx[:project_config],
+                                                                  agent_name: ctx[:agent_name],
+                                                                  since: ctx[:source_context]&.dig(:dispatched_at))
+
+              # If the agent didn't post any comment during this session, re-dispatch to summarize from memory.
+              # Uses dispatched_at from source_context to only check for comments made during THIS session,
+              # not previous sessions on the same card.
+              if !found_comment &&
+                 !ctx[:source_context]&.dig(:skip_summarize_redispatch)
+                dispatch_summarize_session(ctx)
+              end
 
               # Planning mode finalization
               Planning.finalize_if_needed(ctx[:prompt_file], ctx[:agent_name], ctx[:project_config])
@@ -240,13 +252,28 @@ module Brainiac
 
             pid, log_file = run_agent(prompt, project_config: project_config, chdir: repo_path,
                                               log_name: "uat-#{card_number}", agent_name: agent_name,
-                                              source: :fizzy, source_context: { card_number: card_number }, skip_column_move: true)
+                                              source: :fizzy,
+                                              source_context: { card_number: card_number, dispatched_at: Time.now },
+                                              skip_column_move: true)
             register_session("card-#{card_number}", pid, log_file: log_file, agent_name: agent_name)
             LOG.info "[Fizzy] Dispatched #{agent_name} for UAT on card ##{card_number}" if defined?(LOG)
           rescue StandardError => e
             LOG.error "[Fizzy] Failed to dispatch UAT agent: #{e.message}" if defined?(LOG)
           end
 
+          # Delegated to Summarize module for manageable file length.
+          def dispatch_summarize_session(ctx)
+            Summarize.dispatch_summarize_session(ctx)
+          end
+
+          def agent_commented_on_card?(card_number, agent_name, repo_path:, since: nil)
+            Summarize.agent_commented_on_card?(card_number, agent_name, repo_path: repo_path, since: since)
+          end
+
+          def post_fallback_comment(card_number, branch, agent_name, project_config)
+            Summarize.post_fallback_comment(card_number, branch, agent_name, project_config)
+          end
+
           def close_uat_cards(card_list, repo_path)
             closed = []
             map = load_work_item_map

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

@@ -55,7 +55,9 @@ module Brainiac
           You have been assigned Fizzy card #{{CARD_NUMBER}}: "{{CARD_TITLE}}".
           You are on branch "{{BRANCH}}" in a fresh worktree.
           Implement the task, commit, push, and open a PR (link back to Fizzy).
-          When you're done, post a comment on the card with a concise summary, PR link, and branch name.
+
+          **Response destination: Post your response as a comment on Fizzy card #{{CARD_NUMBER}}.**
+          Your comment MUST include a concise summary of what you did, a PR link, and the branch name.
 
           **MANDATORY: Always include the branch name in your comment.** Use this format:
           `<p><strong>Branch:</strong> <code>{{BRANCH}}</code></p>`
@@ -111,6 +113,18 @@ module Brainiac
           **IMPORTANT: Do NOT @mention any other agents in your response.**
         PROMPT
 
+        SUMMARIZE_WORK = <<~'PROMPT'
+          You completed work on Fizzy card #{{CARD_NUMBER}} ("{{CARD_TITLE}}") but did not post a comment.
+          Read your memory file for this card and write a brief summary comment.
+
+          **Response destination: Post your response as a comment on Fizzy card #{{CARD_NUMBER}}.**
+          Include what you did, any PR link, and the branch name.
+          Keep it concise — this is a summary, not a full report.
+
+          **MANDATORY: Always include the branch name in your comment.** Use this format:
+          `<p><strong>Branch:</strong> <code>{{BRANCH}}</code></p>`
+        PROMPT
+
         UAT_TESTING = <<~'PROMPT'
           PR for card #{{CARD_NUMBER}} ("{{CARD_TITLE}}") has been merged to main (PR #{{PR_NUMBER}}).
           Run any UAT testing steps defined in the card or acceptance criteria.

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

@@ -3,7 +3,7 @@
 module Brainiac
   module Plugins
     module Fizzy
-      VERSION = "0.0.13"
+      VERSION = "0.0.15"
     end
   end
 end

data/lib/brainiac/plugins/fizzy.rb

@@ -16,6 +16,7 @@ require_relative "fizzy/handlers/dedup"
 require_relative "fizzy/handlers/deploy"
 require_relative "fizzy/handlers/card_index"
 require_relative "fizzy/handlers/deployments"
+require_relative "fizzy/handlers/summarize"
 
 module Brainiac
   module Plugins

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: brainiac-fizzy
 version: !ruby/object:Gem::Version
-  version: 0.0.13
+  version: 0.0.15
 platform: ruby
 authors:
 - Andy Davis
@@ -98,6 +98,7 @@ files:
 - lib/brainiac/plugins/fizzy/handlers/dedup.rb
 - lib/brainiac/plugins/fizzy/handlers/deploy.rb
 - lib/brainiac/plugins/fizzy/handlers/deployments.rb
+- lib/brainiac/plugins/fizzy/handlers/summarize.rb
 - lib/brainiac/plugins/fizzy/helpers.rb
 - lib/brainiac/plugins/fizzy/hooks.rb
 - lib/brainiac/plugins/fizzy/metadata.rb