brainiac-basecamp 0.0.16 → 0.0.18

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4d77f4d0787e023edeec79754c9958f4ee09f036250038ef7cc4ae1dc9961a20
-  data.tar.gz: 74d8aba9539c7526eed62e209b8a84cabe9ecf26482f8873f8c1581a2a70a86a
+  metadata.gz: c082cb39df24b56239de1b12de0772b25088854641d633afa7352a035e63b721
+  data.tar.gz: 2139e2c80a63636e2f3e427b9b784a4237d5f5abf2e52e8018407081d3b6ad23
 SHA512:
-  metadata.gz: 198ee07060302e70cf71612ede20b9232efda88f91090273dcee7606264364f381ad7646443a8965319a2437cb5a7f46d6e1e1b92e2683aa87fb176028fd97d2
-  data.tar.gz: 9194582cc89b6a9cb05721c0369f80762b656776b29e5ad5f97662a6ae460364d486c7d0d15b20bd36e131b9e41c74b9e66d71910155e30b206d0865bd03c967
+  metadata.gz: 16b4848a37f1d4a0b0e0a93940540cace70f020dd11c4cbad84410310f442cb6377a1bf7a8de0204326bf36c894a345aee88278908fffc5c2cd38ba0456b81d9
+  data.tar.gz: 2f2caa6a328819e0c27eb5f3b1da73b26e411b5a38f40d1bea2078ad4c3c9dfa1f381d25f83838675941a2cc06b599f50df99f75aec30bc024f891741d892d35

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

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require "json"
+require "open3"
 
 module Brainiac
   module Plugins
@@ -30,11 +31,19 @@ module Brainiac
               cmd_projects(args)
             when "set"
               cmd_set(args)
+            when "reset"
+              cmd_reset(args)
+            when "webhook", "webhooks"
+              cmd_webhook(args)
             else
               print_help
             end
           end
 
+          # The canonical set of webhook event types this plugin needs.
+          # Add new types here as features are added — webhook sync will pick them up.
+          REQUIRED_WEBHOOK_TYPES = %w[Todo Todolist Comment].freeze
+
           private
 
           def cmd_setup
@@ -86,7 +95,8 @@ module Brainiac
             puts "  2. Add bot accounts:     brainiac basecamp bot add <name> <person-id> <agent>"
             puts "  3. Map projects:         brainiac basecamp projects map <brainiac-key> <basecamp-id>"
             puts "  4. Set review gate:      brainiac basecamp set review-gate <on_complete|on_pr_merge>"
-            puts "  5. Set up webhooks:      basecamp webhooks create \"https://your-ngrok/basecamp\" --types \"Todo,Todolist\" --in <project>"
+            puts "  5. Set up webhooks:      basecamp webhooks create \"https://your-ngrok/basecamp\" " \
+                 "--types \"Todo,Todolist,Comment\" --in <project>"
             puts "  6. Restart brainiac:     brainiac restart"
           end
 
@@ -172,8 +182,23 @@ module Brainiac
 
               status_icon = epic["status"] == "active" ? "🚀" : "✅"
               puts "#{status_icon} #{epic['title']}"
-              puts "   Agent: #{epic['agent']} | Tasks: #{complete}/#{total} complete, #{in_flight} in-flight"
+              puts "   Todolist: #{epic['basecamp_todolist_id']} | Agent: #{epic['agent']}"
+              puts "   Tasks: #{complete}/#{total} complete, #{in_flight} in-flight"
               puts "   Started: #{epic['started_at']}"
+
+              # Show per-task detail if verbose or few tasks
+              if args.include?("--verbose") || args.include?("-v")
+                tasks.each do |t|
+                  icon = case t["status"]
+                         when "complete" then "✅"
+                         when "in_flight" then "🚀"
+                         when "in_review" then "👀"
+                         when "final_decision" then "⚡"
+                         else "⬜"
+                         end
+                  puts "     #{icon} ##{t['fizzy_card']} — #{t['title']} [#{t['status']}]"
+                end
+              end
               puts ""
             end
           end
@@ -200,11 +225,37 @@ module Brainiac
               person_id = args.shift
               agent = args.shift
 
-              unless name && person_id && agent
-                puts "Usage: brainiac basecamp bot add <name> <basecamp-person-id> <default-agent>"
+              # If person_id is omitted but agent is given, or only agent name given,
+              # try to auto-resolve from Basecamp
+              if name && !person_id
+                # Single arg: treat as agent name, auto-resolve
+                agent = name
+                person_id = auto_resolve_person_id(agent)
+                unless person_id
+                  puts "❌ Could not find '#{agent}' in Basecamp people list."
+                  puts "   Try: basecamp people list --jq '.data[] | {id, name}'"
+                  return
+                end
+                name = agent.downcase
+              elsif name && person_id && !agent
+                # Two args: name + agent, auto-resolve person_id
+                agent = person_id
+                person_id = auto_resolve_person_id(agent)
+                unless person_id
+                  puts "❌ Could not find '#{agent}' in Basecamp people list."
+                  puts "   Provide the person_id manually:"
+                  puts "   brainiac basecamp bot add <name> <person-id> <agent>"
+                  return
+                end
+              elsif !(name && person_id && agent)
+                puts "Usage: brainiac basecamp bot add <agent-name>"
+                puts "       brainiac basecamp bot add <name> <agent-name>"
+                puts "       brainiac basecamp bot add <name> <basecamp-person-id> <agent-name>"
                 puts ""
-                puts "Example:"
-                puts "  brainiac basecamp bot add andy-server 12345 Galen"
+                puts "Examples:"
+                puts "  brainiac basecamp bot add Galen                    # auto-resolves person_id"
+                puts "  brainiac basecamp bot add andy-server Galen        # auto-resolves person_id"
+                puts "  brainiac basecamp bot add andy-server 52992796 Galen"
                 return
               end
 
@@ -217,7 +268,68 @@ module Brainiac
               save_config(config)
               puts "✓ Added bot account '#{name}' (person_id: #{person_id}, agent: #{agent})"
 
-            when "list"
+            when "sync"
+              # Auto-discover agents from ~/.brainiac/agents.json and resolve their
+              # Basecamp person IDs. Creates bot_account entries for any that match.
+              config = load_config
+              config["bot_accounts"] ||= {}
+
+              agents = load_agents_registry
+              if agents.empty?
+                puts "No agents found in ~/.brainiac/agents.json"
+                return
+              end
+
+              puts "Syncing bot accounts from agents registry..."
+              puts "Found #{agents.size} agent(s): #{agents.map { |k, v| v['display_name'] || k }.join(', ')}"
+              puts ""
+
+              added = 0
+              updated = 0
+              not_found = 0
+
+              agents.each do |key, agent_config|
+                display_name = agent_config["display_name"] || key.capitalize
+                # Skip the meta "brainiac" agent
+                next if key == "brainiac"
+
+                resolved_id = auto_resolve_person_id(display_name)
+
+                if resolved_id
+                  existing = config["bot_accounts"].find { |_k, v| v["default_agent"] == display_name }
+
+                  if existing
+                    _existing_key, existing_account = existing
+                    if existing_account["person_id"].to_s == resolved_id.to_s
+                      puts "  \u00b7 #{display_name}: #{resolved_id} (unchanged)"
+                    else
+                      existing_account["person_id"] = resolved_id
+                      updated += 1
+                      puts "  \u2191 #{display_name}: updated person_id \u2192 #{resolved_id}"
+                    end
+                  else
+                    # Create new bot_account entry using agent key as the account name
+                    config["bot_accounts"][key] = {
+                      "person_id" => resolved_id,
+                      "default_agent" => display_name
+                    }
+                    added += 1
+                    puts "  + #{display_name}: #{resolved_id} (new)"
+                  end
+                else
+                  puts "  \u2717 #{display_name}: not found in Basecamp"
+                  not_found += 1
+                end
+              end
+
+              if (added + updated).positive?
+                save_config(config)
+                puts ""
+                puts "\u2713 Added #{added}, updated #{updated} bot account(s)" if added.positive? || updated.positive?
+              else
+                puts "\nAll bot accounts already up to date."
+              end
+              puts "  (#{not_found} agent(s) not found in Basecamp)" if not_found.positive?
               config = load_config
               bots = config["bot_accounts"] || {}
               if bots.empty?
@@ -301,6 +413,102 @@ module Brainiac
             end
           end
 
+          def cmd_webhook(args)
+            action = args.shift
+
+            case action
+            when "sync"
+              cmd_webhook_sync
+            else
+              puts "Usage: brainiac basecamp webhook <command>"
+              puts ""
+              puts "Commands:"
+              puts "  sync    Ensure all project webhooks have the required event types"
+            end
+          end
+
+          def cmd_webhook_sync
+            config = load_config
+            mappings = config["project_mappings"] || {}
+
+            if mappings.empty?
+              puts "No project mappings configured. Add one first:"
+              puts "  brainiac basecamp projects map <key> <basecamp-project-id>"
+              return
+            end
+
+            updated = 0
+            skipped = 0
+            not_found = 0
+
+            mappings.each do |key, mapping|
+              project_id = mapping["basecamp_project_id"]
+              puts "Checking project '#{key}' (Basecamp #{project_id})..."
+
+              # List webhooks for this project
+              output, status = Open3.capture2("basecamp", "webhooks", "list", "--in", project_id.to_s, "--json")
+              unless status.success?
+                puts "  ✗ Failed to list webhooks"
+                not_found += 1
+                next
+              end
+
+              data = JSON.parse(output)
+              webhooks = data["data"] || []
+
+              if webhooks.empty?
+                puts "  ⚠ No webhooks found"
+                not_found += 1
+                next
+              end
+
+              # Find webhooks that point to a /basecamp endpoint (ours)
+              our_webhooks = webhooks.select { |wh| wh["payload_url"]&.include?("/basecamp") }
+
+              if our_webhooks.empty?
+                puts "  ⚠ No webhooks with /basecamp endpoint found"
+                not_found += 1
+                next
+              end
+
+              our_webhooks.each do |wh|
+                current_types = wh["types"] || []
+                missing_types = REQUIRED_WEBHOOK_TYPES - current_types
+
+                if missing_types.empty?
+                  puts "  · Webhook #{wh['id']} (#{wh['payload_url']}): up to date"
+                  skipped += 1
+                  next
+                end
+
+                new_types = (current_types + missing_types).uniq
+                puts "  ↑ Webhook #{wh['id']}: adding #{missing_types.join(', ')}"
+
+                _, stderr, update_status = Open3.capture3(
+                  "basecamp", "webhooks", "update", wh["id"].to_s,
+                  "--types", new_types.join(","),
+                  "--in", project_id.to_s
+                )
+
+                if update_status.success?
+                  puts "    ✓ Updated: #{new_types.join(', ')}"
+                  updated += 1
+                else
+                  puts "    ✗ Failed: #{stderr.strip}"
+                end
+              end
+            end
+
+            puts ""
+            if updated.positive?
+              puts "✓ Updated #{updated} webhook(s)"
+            elsif skipped.positive? && updated.zero?
+              puts "All webhooks already have required types: #{REQUIRED_WEBHOOK_TYPES.join(', ')}"
+            else
+              puts "No webhooks updated."
+            end
+          end
+
           def print_help
             puts <<~HELP
               Usage: brainiac basecamp <command>
@@ -311,15 +519,21 @@ module Brainiac
                 status                                  Check plugin status
                 epics [--all]                           List active epics (--all for completed too)
                 link <card> <url>                       Link a Fizzy card to a Basecamp todo
-                bot add <name> <person-id> <agent>      Add a bot account mapping
+                bot add <agent-name>                    Add bot (auto-resolves person_id from Basecamp)
+                bot add <name> <person-id> <agent>      Add bot with explicit person_id
+                bot sync                                Re-resolve all bot person IDs from Basecamp
                 bot list                                List bot accounts
                 bot remove <name>                       Remove a bot account
                 projects map <key> <basecamp-id>        Map a Brainiac project to Basecamp
                 projects list                           List project mappings
                 projects unmap <key>                    Remove a project mapping
-                set fizzy-account-id <id>                Set Fizzy account ID (for card URLs)
+                set fizzy-account-id <id>               Set Fizzy account ID (for card URLs)
                 set review-gate <mode>                  Set review gate (on_complete or on_pr_merge)
                 set epic-prefix <prefix>                Set epic todolist prefix (default: "Epic:")
+                reset task <card-number> [--to <status>] Reset a task to a given status (default: pending)
+                reset epic <todolist-id> [--to <status>] Reset entire epic or all tasks within it
+                reset gates <card-number>               Clear gate approvals and re-dispatch gates
+                webhook sync                            Ensure webhooks have all required event types
 
               Config file: ~/.brainiac/basecamp.json
               Epics state: ~/.brainiac/basecamp_epics.json
@@ -327,6 +541,13 @@ module Brainiac
               Review gate modes:
                 on_complete   — Advance to next task as soon as agent finishes (default)
                 on_pr_merge   — Wait for PR to be merged before advancing (review gate)
+
+              Task statuses (for reset --to):
+                pending         — Not yet started, waiting for dependencies
+                in_flight       — Agent is actively working on it
+                in_review       — PR open, gates reviewing
+                final_decision  — Gates passed, awaiting implementation agent merge decision
+                complete        — Done
             HELP
           end
 
@@ -385,6 +606,309 @@ module Brainiac
             end
           end
 
+          def cmd_reset(args)
+            subcommand = args.shift
+
+            case subcommand
+            when "task"
+              cmd_reset_task(args)
+            when "epic"
+              cmd_reset_epic(args)
+            when "gates"
+              cmd_reset_gates(args)
+            else
+              puts <<~HELP
+                Usage: brainiac basecamp reset <subcommand>
+
+                Subcommands:
+                  task <card-number> [--to <status>]    Reset a task's status (default: pending)
+                  epic <todolist-id> [--to <status>]    Reset entire epic or all its tasks
+                  gates <card-number>                   Clear gate approvals and re-dispatch
+
+                Task statuses:
+                  pending, in_flight, in_review, final_decision, complete
+
+                Examples:
+                  brainiac basecamp reset task 1234                  # Reset card #1234 to pending
+                  brainiac basecamp reset task 1234 --to in_flight   # Reset to in_flight
+                  brainiac basecamp reset gates 1234                 # Clear gates, re-dispatch reviewers
+                  brainiac basecamp reset epic 10233212224            # Reset all tasks to pending
+                  brainiac basecamp reset epic 10233212224 --to pending  # Same as above
+              HELP
+            end
+          end
+
+          def cmd_reset_task(args)
+            card_number = args.shift&.gsub(/^#/, "")
+            unless card_number
+              puts "Usage: brainiac basecamp reset task <card-number> [--to <status>]"
+              return
+            end
+
+            target_status = parse_to_flag(args) || "pending"
+            unless valid_task_status?(target_status)
+              puts "Error: Invalid status '#{target_status}'"
+              puts "Valid statuses: pending, in_flight, in_review, final_decision, complete"
+              return
+            end
+
+            epics = load_epics
+            epic = epics.find do |e|
+              e["status"] == "active" &&
+                e["tasks"]&.any? { |t| t["fizzy_card"] == card_number.to_i }
+            end
+
+            unless epic
+              puts "Error: Card ##{card_number} not found in any active epic"
+              return
+            end
+
+            task = epic["tasks"].find { |t| t["fizzy_card"] == card_number.to_i }
+            old_status = task["status"]
+
+            # Reset the task
+            reset_task_to(task, target_status)
+
+            save_epics(epics)
+            puts "✓ Reset card ##{card_number} from '#{old_status}' → '#{target_status}'"
+            puts "  Epic: #{epic['title']}"
+
+            return unless target_status == "pending"
+
+            puts "  The task will be picked up on next dispatch cycle (restart brainiac or wait for next event)"
+          end
+
+          def cmd_reset_gates(args)
+            card_number = args.shift&.gsub(/^#/, "")
+            unless card_number
+              puts "Usage: brainiac basecamp reset gates <card-number>"
+              return
+            end
+
+            epics = load_epics
+            epic = epics.find do |e|
+              e["status"] == "active" &&
+                e["tasks"]&.any? { |t| t["fizzy_card"] == card_number.to_i }
+            end
+
+            unless epic
+              puts "Error: Card ##{card_number} not found in any active epic"
+              return
+            end
+
+            task = epic["tasks"].find { |t| t["fizzy_card"] == card_number.to_i }
+
+            # Clear all gate-related state
+            old_approvals = task["gate_approvals"]&.size || 0
+            old_changes = task["changes_requested_by"]&.size || 0
+
+            task["gate_approvals"] = []
+            task["changes_requested_by"] = []
+            task["gates_dispatched_at"] = nil
+            task["awaiting_final_decision"] = nil
+            task["changes_debounce_started"] = nil
+            task["gate_redispatch_counts"] = {}
+            task["last_redispatch_at"] = nil
+            task["status"] = "in_review"
+            epic["updated_at"] = Time.now.iso8601
+
+            save_epics(epics)
+            puts "✓ Cleared gates for card ##{card_number}"
+            puts "  Removed #{old_approvals} approval(s), #{old_changes} changes_requested"
+            puts "  Status set to 'in_review' — gates will be re-dispatched on next health check"
+            puts ""
+            puts "  To force immediate re-dispatch, restart brainiac:"
+            puts "    brainiac restart"
+          end
+
+          def cmd_reset_epic(args)
+            todolist_id = args.shift
+            unless todolist_id
+              puts "Usage: brainiac basecamp reset epic <todolist-id> [--to <status>]"
+              puts ""
+              puts "Options:"
+              puts "  --to <status>    Reset all tasks to this status (default: pending)"
+              puts "  --reactivate     Reset a completed epic back to active"
+              puts ""
+              puts "Find your todolist ID with: brainiac basecamp epics --all"
+              return
+            end
+
+            target_status = parse_to_flag(args) || "pending"
+            reactivate = args.include?("--reactivate")
+
+            unless valid_task_status?(target_status)
+              puts "Error: Invalid status '#{target_status}'"
+              puts "Valid statuses: pending, in_flight, in_review, final_decision, complete"
+              return
+            end
+
+            epics = load_epics
+            epic = epics.find { |e| e["basecamp_todolist_id"] == todolist_id.to_s }
+
+            unless epic
+              puts "Error: No epic found with todolist ID #{todolist_id}"
+              puts ""
+              puts "Active epics:"
+              epics.select { |e| e["status"] == "active" }.each do |e|
+                puts "  #{e['basecamp_todolist_id']} — #{e['title']}"
+              end
+              return
+            end
+
+            if epic["status"] != "active" && !reactivate
+              puts "Error: Epic '#{epic['title']}' is #{epic['status']} (not active)"
+              puts "  Use --reactivate to set it back to active"
+              return
+            end
+
+            # Reactivate if needed
+            if epic["status"] != "active" && reactivate
+              epic["status"] = "active"
+              epic["completed_at"] = nil
+              puts "✓ Reactivated epic '#{epic['title']}'"
+            end
+
+            # Reset all non-complete tasks (unless resetting to pending, in which case reset all)
+            tasks_to_reset = if target_status == "pending"
+                               epic["tasks"]
+                             else
+                               epic["tasks"].reject { |t| t["status"] == "complete" }
+                             end
+
+            tasks_to_reset.each { |t| reset_task_to(t, target_status) }
+            epic["updated_at"] = Time.now.iso8601
+
+            save_epics(epics)
+            puts "✓ Reset #{tasks_to_reset.size} task(s) → '#{target_status}' in epic '#{epic['title']}'"
+            puts ""
+            epic["tasks"].each do |t|
+              icon = case t["status"]
+                     when "complete" then "✅"
+                     when "in_flight" then "🚀"
+                     when "in_review" then "👀"
+                     when "final_decision" then "⚡"
+                     else "⬜"
+                     end
+              puts "  #{icon} ##{t['fizzy_card']} — #{t['title']} [#{t['status']}]"
+            end
+          end
+
+          # --- Reset helpers ---
+
+          def valid_task_status?(status)
+            %w[pending in_flight in_review final_decision complete].include?(status)
+          end
+
+          def parse_to_flag(args)
+            idx = args.index("--to")
+            return nil unless idx
+
+            args.delete_at(idx) # remove --to
+            args.delete_at(idx) # remove the value (now at same index)
+          end
+
+          def reset_task_to(task, status)
+            task["status"] = status
+
+            case status
+            when "pending"
+              # Clear all progress state
+              task["dispatched_at"] = nil
+              task["completed_at"] = nil
+              task["pr_number"] = nil
+              task["pr_repo"] = nil
+              task["gate_approvals"] = []
+              task["changes_requested_by"] = []
+              task["gates_dispatched_at"] = nil
+              task["awaiting_final_decision"] = nil
+              task["changes_debounce_started"] = nil
+              task["gate_redispatch_counts"] = {}
+              task["last_redispatch_at"] = nil
+              task["last_reviewed_sha"] = nil
+            when "in_flight"
+              # Keep PR info but clear gate state
+              task["completed_at"] = nil
+              task["gate_approvals"] = []
+              task["changes_requested_by"] = []
+              task["gates_dispatched_at"] = nil
+              task["awaiting_final_decision"] = nil
+              task["changes_debounce_started"] = nil
+              task["gate_redispatch_counts"] = {}
+              task["last_redispatch_at"] = nil
+              task["dispatched_at"] ||= Time.now.iso8601
+            when "in_review"
+              # Clear gate approvals but keep PR info
+              task["completed_at"] = nil
+              task["gate_approvals"] = []
+              task["changes_requested_by"] = []
+              task["gates_dispatched_at"] = nil
+              task["awaiting_final_decision"] = nil
+              task["changes_debounce_started"] = nil
+              task["gate_redispatch_counts"] = {}
+              task["last_redispatch_at"] = nil
+            when "final_decision"
+              task["completed_at"] = nil
+              task["awaiting_final_decision"] = true
+            when "complete"
+              task["completed_at"] ||= Time.now.iso8601
+            end
+          end
+
+          def load_epics
+            epics_file = File.join(BRAINIAC_DIR, "basecamp_epics.json")
+            return [] unless File.exist?(epics_file)
+
+            data = JSON.parse(File.read(epics_file))
+            data["epics"] || []
+          rescue JSON::ParserError
+            []
+          end
+
+          def save_epics(epics)
+            epics_file = File.join(BRAINIAC_DIR, "basecamp_epics.json")
+            File.write(epics_file, JSON.pretty_generate({
+                                                          "epics" => epics,
+                                                          "updated_at" => Time.now.iso8601
+                                                        }))
+          end
+
+          def auto_resolve_person_id(agent_name)
+            output, status = Open3.capture2(
+              "basecamp", "people", "list", "--jq",
+              ".data[] | select(.name | ascii_downcase | contains(\"#{agent_name.downcase}\")) | {id, name}"
+            )
+            return nil unless status.success? && !output.strip.empty?
+
+            # Parse each JSON line — prefer exact match
+            candidates = []
+            output.each_line do |line|
+              person = JSON.parse(line.strip)
+              candidates << person
+            rescue JSON::ParserError
+              next
+            end
+
+            # Exact name match first
+            exact = candidates.find { |p| p["name"]&.downcase == agent_name.downcase }
+            return exact["id"].to_s if exact
+
+            # Otherwise first contains-match
+            candidates.first&.dig("id")&.to_s
+          end
+
+          # Load the agents registry from ~/.brainiac/agents.json
+          #
+          # @return [Hash] Agent key => config hash
+          def load_agents_registry
+            agents_file = File.join(BRAINIAC_DIR, "agents.json")
+            return {} unless File.exist?(agents_file)
+
+            JSON.parse(File.read(agents_file))
+          rescue JSON::ParserError
+            {}
+          end
+
           def load_config
             if File.exist?(CONFIG_FILE)
               JSON.parse(File.read(CONFIG_FILE))
@@ -404,7 +928,7 @@ module Brainiac
       end
 
       def self.completions
-        %w[setup config status epics link bot projects set]
+        %w[setup config status epics link bot projects set reset webhook]
       end
     end
   end