aidp 0.8.3 → 0.9.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 60b61133b8f3b07ef7fbdd76bce1485bd89b75f78c4130f195c786331f383ded
-  data.tar.gz: e81f6387d6c6941da8b2ae1ff2b3153a3911fd2bb0405c4982f52b166681fc64
+  metadata.gz: c1d22317d935a3fc16c21b5113f8c5f4e91c1f246445796ea705c63e75715b24
+  data.tar.gz: c63cb58339c15dc07c74e86c33eb5620e92f305ec5fe84517051f9c3b8503eb7
 SHA512:
-  metadata.gz: 42b8c458bd6dcbfce249d12bb7a1e7eab1d09a61259ea231b935fd4d28c39170baf57d099bb3a5fd3037a3501a4d5daa9abc3696e3c7b91e07137ffce24b22a6
-  data.tar.gz: e2f038cc38c7b32f9bda535ac6bfb8a0bc39c54ef252dc9d7f333f66c71de41dc8e88a6479e8319bad64e5e9dc8adffb2d4dd03febddb82081df2ddf91e8efe0
+  metadata.gz: 7b0976481771d0bdf32b3a3b7191d5275a13182fc8f145ea89085ec6f64071c8add854e7cf70a055dba01cb47df105c36e63b0183ef0852b3862d52f54185b35
+  data.tar.gz: 925699c5ecbfeec13f24e597340f343765881182f264957b80ade1c3b218f0b22b4936c3553129abf95ce8348624f4c3068aa72b29ad114c9a732466528fe8aa

data/README.md

@@ -15,6 +15,20 @@ cd /your/project
 aidp
 ```
 
+### First-Time Setup
+
+On the first run in a project without an `aidp.yml`, AIDP now launches a **First-Time Setup Wizard** instead of failing with a configuration error. You'll be prompted to choose one of:
+
+1. Minimal (single provider: cursor)
+2. Development template (multiple providers, safe defaults)
+3. Production template (full-feature example – review before committing)
+4. Full example (verbose documented config)
+5. Custom (interactive prompts for providers and defaults)
+
+Non-interactive environments (CI, scripts, pipes) automatically receive a minimal `aidp.yml` so workflows can proceed without manual intervention.
+
+You can re-run the wizard manually by removing `aidp.yml` and starting `aidp` again.
+
 ## Enhanced TUI
 
 AIDP features a rich terminal interface that transforms it from a step-by-step tool into an intelligent development assistant. The enhanced TUI provides beautiful, interactive terminal components while running complete workflows automatically.

data/lib/aidp/cli/first_run_wizard.rb

@@ -0,0 +1,346 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+require "yaml"
+require "tty-prompt"
+
+module Aidp
+  class CLI
+    # Handles interactive first-time project setup when no aidp.yml exists
+    class FirstRunWizard
+      TEMPLATES_DIR = File.expand_path(File.join(__dir__, "..", "..", "..", "templates"))
+
+      def self.ensure_config(project_dir, input: $stdin, output: $stdout, non_interactive: false)
+        return true if Aidp::Config.config_exists?(project_dir)
+
+        wizard = new(project_dir, input: input, output: output)
+
+        if non_interactive || !input.tty? || !output.tty?
+          # Non-interactive environment - create minimal config silently
+          path = wizard.send(:write_minimal_config, project_dir)
+          output.puts "Created minimal configuration at #{wizard.send(:relative, path)} (non-interactive default)"
+          return true
+        end
+
+        wizard.run
+      end
+
+      def self.setup_config(project_dir, input: $stdin, output: $stdout, non_interactive: false)
+        wizard = new(project_dir, input: input, output: output)
+
+        if non_interactive || !input.tty? || !output.tty?
+          # Non-interactive environment - skip setup
+          output.puts "Configuration setup skipped in non-interactive environment"
+          return true
+        end
+
+        wizard.run_setup_config
+      end
+
+      def initialize(project_dir, input: $stdin, output: $stdout)
+        @project_dir = project_dir
+        @input = input
+        @output = output
+        @prompt = TTY::Prompt.new
+      end
+
+      def run
+        banner
+        loop do
+          choice = ask_choice
+          case choice
+          when "1" then return finish(write_minimal_config(@project_dir))
+          when "2" then return finish(copy_template("aidp-development.yml.example"))
+          when "3" then return finish(copy_template("aidp-production.yml.example"))
+          when "4" then return finish(write_example_config(@project_dir))
+          when "5" then return finish(run_custom)
+          when "q", "Q" then @output.puts("Exiting without creating configuration.")
+                             return false
+          else
+            @output.puts "Invalid selection. Please choose one of the listed options."
+          end
+        end
+      end
+
+      def run_setup_config
+        @prompt.say("🔧 Configuration Setup", color: :blue)
+        @prompt.say("Setting up your configuration file with current values as defaults.")
+        @prompt.say("")
+
+        # Load existing config to use as defaults (if it exists)
+        existing_config = load_existing_config
+
+        if existing_config
+          # Run custom configuration with existing values as defaults
+          finish(run_custom_with_defaults(existing_config))
+        else
+          # No existing config, run the normal setup flow
+          @prompt.say("No existing configuration found. Running first-time setup...")
+          @prompt.say("")
+          run
+        end
+      end
+
+      private
+
+      def banner
+        @output.puts "\n🚀 First-time setup detected"
+        @output.puts "No 'aidp.yml' configuration file found in #{relative(@project_dir)}."
+        @output.puts "Let's create one so you can start using AI Dev Pipeline."
+        @output.puts
+      end
+
+      def ask_choice
+        @output.puts "Choose a configuration style:" unless @asking
+        @output.puts <<~MENU
+          1) Minimal (single provider: cursor)
+          2) Development template (multiple providers, safe defaults)
+          3) Production template (full features, review required)
+          4) Full example (verbose example config)
+          5) Custom (interactive prompts)
+          q) Quit
+        MENU
+        @output.print "Enter choice [1]: "
+        @output.flush
+        ans = @input.gets&.strip
+        ans = "1" if ans.nil? || ans.empty?
+        ans
+      end
+
+      def finish(path)
+        if path
+          @output.puts "\n✅ Configuration created at #{relative(path)}"
+          @output.puts "You can edit this file anytime. Continuing startup...\n"
+          true
+        else
+          @output.puts "❌ Failed to create configuration file."
+          false
+        end
+      end
+
+      def copy_template(filename)
+        src = File.join(TEMPLATES_DIR, filename)
+        unless File.exist?(src)
+          @output.puts "Template not found: #{filename}"
+          return nil
+        end
+        dest = File.join(@project_dir, "aidp.yml")
+        File.write(dest, File.read(src))
+        dest
+      end
+
+      def write_minimal_config(project_dir)
+        dest = File.join(project_dir, "aidp.yml")
+        return dest if File.exist?(dest)
+        data = {
+          harness: {
+            max_retries: 2,
+            default_provider: "cursor",
+            fallback_providers: ["cursor"],
+            no_api_keys_required: false
+          },
+          providers: {
+            cursor: {
+              type: "package",
+              default_flags: []
+            }
+          }
+        }
+        File.write(dest, YAML.dump(data))
+        dest
+      end
+
+      def write_example_config(project_dir)
+        Aidp::Config.create_example_config(project_dir)
+        File.join(project_dir, "aidp.yml")
+      end
+
+      def run_custom
+        dest = File.join(@project_dir, "aidp.yml")
+        return dest if File.exist?(dest)
+
+        @prompt.say("Interactive custom configuration: press Enter to accept defaults shown in [brackets].")
+        @prompt.say("")
+
+        # Get available providers for validation
+        available_providers = get_available_providers
+
+        # Use TTY::Prompt select for primary provider
+        # Find the formatted string that matches the default
+        default_option = available_providers.find { |option| option.start_with?("cursor -") } || available_providers.first
+        default_provider = @prompt.select("Default provider?", available_providers, default: default_option)
+
+        # Extract just the provider name from the formatted string
+        provider_name = default_provider.split(" - ").first
+
+        # Validate fallback providers
+        fallback_input = @prompt.ask("Fallback providers (comma-separated)?", default: provider_name) do |q|
+          q.validate(/^[a-zA-Z0-9_,\s]+$/, "Invalid characters. Use only letters, numbers, commas, and spaces.")
+          q.validate(->(input) { validate_provider_list(input, available_providers) }, "One or more providers are not supported.")
+        end
+
+        restrict = @prompt.yes?("Only use providers that don't require API keys?", default: false)
+
+        # Process the inputs
+        fallback_providers = fallback_input.split(/\s*,\s*/).map(&:strip).reject(&:empty?)
+        providers = [provider_name] + fallback_providers
+        providers.uniq!
+
+        provider_section = {}
+        providers.each do |prov|
+          provider_section[prov] = {"type" => (prov == "cursor") ? "package" : "usage_based", "default_flags" => []}
+        end
+
+        data = {
+          "harness" => {
+            "max_retries" => 2,
+            "default_provider" => provider_name,
+            "fallback_providers" => fallback_providers,
+            "no_api_keys_required" => restrict
+          },
+          "providers" => provider_section
+        }
+        File.write(dest, YAML.dump(data))
+        dest
+      end
+
+      def run_custom_with_defaults(existing_config)
+        dest = File.join(@project_dir, "aidp.yml")
+
+        # Extract current values from existing config
+        harness_config = existing_config[:harness] || existing_config["harness"] || {}
+        providers_config = existing_config[:providers] || existing_config["providers"] || {}
+
+        current_default = harness_config[:default_provider] || harness_config["default_provider"] || "cursor"
+        current_fallbacks = harness_config[:fallback_providers] || harness_config["fallback_providers"] || [current_default]
+        current_restrict = harness_config[:no_api_keys_required] || harness_config["no_api_keys_required"] || false
+
+        # Use TTY::Prompt for interactive configuration
+        @prompt.say("Interactive configuration update: press Enter to keep current values shown in [brackets].")
+        @prompt.say("")
+
+        # Get available providers for validation
+        available_providers = get_available_providers
+
+        # Use TTY::Prompt select for primary provider
+        # Find the formatted string that matches the current default
+        default_option = available_providers.find { |option| option.start_with?("#{current_default} -") } || available_providers.first
+        default_provider = @prompt.select("Default provider?", available_providers, default: default_option)
+
+        # Extract just the provider name from the formatted string
+        provider_name = default_provider.split(" - ").first
+
+        # Validate fallback providers
+        fallback_input = @prompt.ask("Fallback providers (comma-separated)?", default: current_fallbacks.join(", ")) do |q|
+          q.validate(/^[a-zA-Z0-9_,\s]+$/, "Invalid characters. Use only letters, numbers, commas, and spaces.")
+          q.validate(->(input) { validate_provider_list(input, available_providers) }, "One or more providers are not supported.")
+        end
+
+        restrict_input = @prompt.yes?("Only use providers that don't require API keys?", default: current_restrict)
+
+        # Process the inputs
+        fallback_providers = fallback_input.split(/\s*,\s*/).map(&:strip).reject(&:empty?)
+        providers = [provider_name] + fallback_providers
+        providers.uniq!
+
+        # Build provider section
+        provider_section = {}
+        providers.each do |prov|
+          # Try to preserve existing provider config if it exists
+          existing_provider = providers_config[prov.to_sym] || providers_config[prov.to_s]
+          if existing_provider
+            # Convert existing provider config to string keys
+            converted_provider = {}
+            existing_provider.each { |k, v| converted_provider[k.to_s] = v }
+            provider_section[prov] = converted_provider
+          else
+            provider_section[prov] = {"type" => (prov == "cursor") ? "package" : "usage_based", "default_flags" => []}
+          end
+        end
+
+        # Build the new config
+        data = {
+          "harness" => {
+            "max_retries" => harness_config[:max_retries] || harness_config["max_retries"] || 2,
+            "default_provider" => provider_name,
+            "fallback_providers" => fallback_providers,
+            "no_api_keys_required" => restrict_input
+          },
+          "providers" => provider_section
+        }
+
+        File.write(dest, YAML.dump(data))
+        dest
+      end
+
+      def load_existing_config
+        config_file = File.join(@project_dir, "aidp.yml")
+        return nil unless File.exist?(config_file)
+
+        begin
+          YAML.load_file(config_file) || {}
+        rescue => e
+          @prompt.say("❌ Failed to load existing configuration: #{e.message}", color: :red)
+          nil
+        end
+      end
+
+      def ask(prompt, default: nil)
+        if default
+          @output.print "#{prompt} [#{default}]: "
+        else
+          @output.print "#{prompt}: "
+        end
+        @output.flush
+        ans = @input.gets&.strip
+        return default if (ans.nil? || ans.empty?) && default
+        ans
+      end
+
+      def relative(path)
+        pn = Pathname.new(path)
+        wd = Pathname.new(@project_dir)
+        rel = pn.relative_path_from(wd).to_s
+        rel.start_with?("..") ? path : rel
+      rescue
+        path
+      end
+
+      # Get available providers for validation
+      def get_available_providers
+        # Define the available providers based on the system
+        available = ["cursor", "anthropic", "gemini", "macos", "opencode"]
+
+        # Add descriptions for better UX
+        available.map do |provider|
+          case provider
+          when "cursor"
+            "cursor - Cursor AI (no API key required)"
+          when "anthropic"
+            "anthropic - Anthropic Claude (requires API key)"
+          when "gemini"
+            "gemini - Google Gemini (requires API key)"
+          when "macos"
+            "macos - macOS UI Automation (no API key required)"
+          when "opencode"
+            "opencode - OpenCode (no API key required)"
+          else
+            provider
+          end
+        end
+      end
+
+      # Validate provider list input
+      def validate_provider_list(input, available_providers)
+        return true if input.nil? || input.empty?
+
+        # Extract provider names from the input
+        providers = input.split(/\s*,\s*/).map(&:strip).reject(&:empty?)
+
+        # Check if all providers are valid
+        valid_providers = available_providers.map { |p| p.split(" - ").first }
+        providers.all? { |provider| valid_providers.include?(provider) }
+      end
+    end
+  end
+end

data/lib/aidp/cli.rb

@@ -6,6 +6,7 @@ require_relative "execute/workflow_selector"
 require_relative "harness/ui/enhanced_tui"
 require_relative "harness/ui/enhanced_workflow_selector"
 require_relative "harness/enhanced_runner"
+require_relative "cli/first_run_wizard"
 
 module Aidp
   # CLI interface for AIDP
@@ -122,6 +123,21 @@ module Aidp
         puts "   Press Ctrl+C to stop\n"
         $stdout.flush
 
+        # Handle configuration setup
+        if options[:setup_config]
+          # Force setup/reconfigure even if config exists
+          unless Aidp::CLI::FirstRunWizard.setup_config(Dir.pwd, input: $stdin, output: $stdout, non_interactive: ENV["CI"] == "true")
+            puts "Configuration setup cancelled. Aborting startup."
+            return 1
+          end
+        else
+          # First-time setup wizard (before TUI to avoid noisy errors)
+          unless Aidp::CLI::FirstRunWizard.ensure_config(Dir.pwd, input: $stdin, output: $stdout, non_interactive: ENV["CI"] == "true")
+            puts "Configuration required. Aborting startup."
+            return 1
+          end
+        end
+
         # Initialize the enhanced TUI
         tui = Aidp::Harness::UI::EnhancedTUI.new
         workflow_selector = Aidp::Harness::UI::EnhancedWorkflowSelector.new(tui)
@@ -175,6 +191,7 @@ module Aidp
 
           opts.on("-h", "--help", "Show this help message") { options[:help] = true }
           opts.on("-v", "--version", "Show version information") { options[:version] = true }
+          opts.on("--setup-config", "Setup or reconfigure config file with current values as defaults") { options[:setup_config] = true }
         end
 
         parser.parse!(args)

data/lib/aidp/config.rb

@@ -11,7 +11,7 @@ module Aidp
         max_retries: 2,
         default_provider: "cursor",
         fallback_providers: ["cursor"],
-        restrict_to_non_byok: false,
+        no_api_keys_required: false,
         provider_weights: {
           "cursor" => 3,
           "anthropic" => 2,
@@ -204,10 +204,16 @@ module Aidp
         require_relative "harness/config_validator"
         validator = Aidp::Harness::ConfigValidator.new(project_dir)
 
-        original_providers.each do |provider_name, _provider_config|
-          validation_result = validator.validate_provider(provider_name)
-          unless validation_result[:valid]
-            errors.concat(validation_result[:errors])
+        # Only validate if the config file exists
+        # Skip validation if we're validating a simple test config (no project_dir specified or simple config)
+        should_validate = validator.config_exists? &&
+          (project_dir != Dir.pwd || config[:harness]&.keys&.size.to_i > 2)
+        if should_validate
+          original_providers.each do |provider_name, _provider_config|
+            validation_result = validator.validate_provider(provider_name)
+            unless validation_result[:valid]
+              errors.concat(validation_result[:errors])
+            end
           end
         end
       end
@@ -256,7 +262,7 @@ module Aidp
           max_retries: 2,
           default_provider: "cursor",
           fallback_providers: ["cursor"],
-          restrict_to_non_byok: false
+          no_api_keys_required: false
         },
         providers: {
           cursor: {

data/lib/aidp/harness/config_schema.rb

@@ -33,7 +33,7 @@ module Aidp
                 pattern: /^[a-zA-Z0-9_-]+$/
               }
             },
-            restrict_to_non_byok: {
+            no_api_keys_required: {
               type: :boolean,
               required: false,
               default: false
@@ -616,7 +616,7 @@ module Aidp
             max_retries: 2,
             default_provider: "cursor",
             fallback_providers: ["cursor"],
-            restrict_to_non_byok: false,
+            no_api_keys_required: false,
             provider_weights: {
               "cursor" => 3,
               "anthropic" => 2,

data/lib/aidp/harness/configuration.rb

@@ -42,9 +42,9 @@ module Aidp
         harness_config[:max_retries]
       end
 
-      # Check if restricted to non-BYOK providers
-      def restrict_to_non_byok?
-        harness_config[:restrict_to_non_byok]
+      # Check if restricted to providers that don't require API keys
+      def no_api_keys_required?
+        harness_config[:no_api_keys_required]
       end
 
       # Get provider type (api, package, etc.)
@@ -237,7 +237,7 @@ module Aidp
           default_provider: default_provider,
           fallback_providers: fallback_providers.size,
           max_retries: max_retries,
-          restrict_to_non_byok: restrict_to_non_byok?,
+          no_api_keys_required: no_api_keys_required?,
           load_balancing_enabled: load_balancing_config[:enabled],
           model_switching_enabled: model_switching_config[:enabled],
           circuit_breaker_enabled: circuit_breaker_config[:enabled],
@@ -298,7 +298,9 @@ module Aidp
         end
 
         # Validate each provider configuration using config_validator
-        configured_providers.each do |provider|
+        # Only validate providers that are actually referenced in the harness configuration
+        providers_to_validate = [default_provider] + fallback_providers
+        providers_to_validate.uniq.each do |provider|
           require_relative "config_validator"
           validator = ConfigValidator.new(@project_dir)
           validation_result = validator.validate_provider(provider)

data/lib/aidp/harness/ui/enhanced_tui.rb

@@ -2,7 +2,6 @@
 
 require "tty-cursor"
 require "tty-screen"
-require "tty-reader"
 require "tty-box"
 require "tty-table"
 require "tty-progressbar"
@@ -22,7 +21,6 @@ module Aidp
         def initialize
           @cursor = TTY::Cursor
           @screen = TTY::Screen
-          @reader = TTY::Reader.new
           @pastel = Pastel.new
           @prompt = TTY::Prompt.new
           # Headless (non-interactive) detection for test/CI environments:
@@ -35,43 +33,20 @@ module Aidp
 
           @jobs = {}
           @jobs_visible = false
-          @input_mode = false
-          @input_prompt = ""
-          @input_buffer = ""
-          @input_position = 0
-          @display_active = false
-          @display_thread = nil
 
           setup_signal_handlers
         end
 
-        # Smart display loop - only shows input overlay when needed
+        # Simple display initialization - no background threads
         def start_display_loop
-          # Display loop is no longer needed since we use TTY::Prompt for input
-          # Keep this method for compatibility but don't start the loop
-          @display_active = true
-          start_key_listener
-          # Always emit a visible menu header once so outer harness/system tests
-          # (tmux sessions that may appear TTY) can detect readiness reliably.
-          puts "Choose your mode"
+          # Display loop is now just a no-op for compatibility
         end
 
         def stop_display_loop
-          @display_active = false
-          @display_thread&.join
-          @key_thread&.kill
-          @key_thread = nil
+          # Simple cleanup - no background threads to stop
           restore_screen
         end
 
-        def pause_display_loop
-          @input_mode = false
-        end
-
-        def resume_display_loop
-          @input_mode = true
-        end
-
         # Job monitoring methods
         def add_job(job_id, job_data)
           @jobs[job_id] = {
@@ -84,90 +59,53 @@ module Aidp
             provider: job_data[:provider] || "unknown"
           }
           @jobs_visible = true
-          add_message("🔄 Started job: #{@jobs[job_id][:name]}", :info)
         end
 
         def update_job(job_id, updates)
           return unless @jobs[job_id]
 
-          old_status = @jobs[job_id][:status]
           @jobs[job_id].merge!(updates)
           @jobs[job_id][:updated_at] = Time.now
-
-          # Show status change messages
-          if old_status != @jobs[job_id][:status]
-            case @jobs[job_id][:status]
-            when :completed
-              add_message("✅ Completed job: #{@jobs[job_id][:name]}", :success)
-            when :failed
-              add_message("❌ Failed job: #{@jobs[job_id][:name]}", :error)
-            when :running
-              add_message("🔄 Running job: #{@jobs[job_id][:name]}", :info)
-            end
-          end
         end
 
         def remove_job(job_id)
-          job_name = @jobs[job_id]&.dig(:name)
           @jobs.delete(job_id)
           @jobs_visible = @jobs.any?
-          add_message("🗑️ Removed job: #{job_name}", :info) if job_name
         end
 
-        # Input methods using TTY
+        # Input methods using TTY::Prompt only - no background threads
         def get_user_input(prompt = "💬 You: ")
-          # Use TTY::Prompt for better input handling - no display loop needed
           @prompt.ask(prompt)
-        rescue TTY::Reader::InputInterrupt
-          # Clean exit without error trace
-          puts "\n\n👋 Goodbye!"
-          exit(0)
         end
 
         def get_confirmation(message, default: true)
-          # Use TTY::Prompt for better input handling - no display loop needed
           @prompt.yes?(message)
-        rescue TTY::Reader::InputInterrupt
-          # Clean exit without error trace
-          puts "\n\n👋 Goodbye!"
-          exit(0)
         end
 
-        # Single-select interface using TTY::Prompt (much better!)
+        # Single-select interface using TTY::Prompt
         def single_select(title, items, default: 0)
           @prompt.select(title, items, default: default, cycle: true)
-        rescue TTY::Reader::InputInterrupt
-          # Clean exit without error trace
-          puts "\n\n👋 Goodbye!"
-          exit(0)
         end
 
-        # Multiselect interface using TTY::Prompt (much better!)
+        # Multiselect interface using TTY::Prompt
         def multiselect(title, items, selected: [])
           @prompt.multi_select(title, items, default: selected)
-        rescue TTY::Reader::InputInterrupt
-          # Clean exit without error trace
-          puts "\n\n👋 Goodbye!"
-          exit(0)
         end
 
-        # Display methods using TTY
+        # Display methods using TTY::Prompt
         def show_message(message, type = :info)
           case type
           when :info
-            puts @pastel.blue("ℹ") + " #{message}"
+            @prompt.say("ℹ #{message}", color: :blue)
           when :success
-            puts @pastel.green("✓") + " #{message}"
+            @prompt.say("✓ #{message}", color: :green)
           when :warning
-            puts @pastel.yellow("⚠") + " #{message}"
+            @prompt.say("⚠ #{message}", color: :yellow)
           when :error
-            puts @pastel.red("✗") + " #{message}"
+            @prompt.say("✗ #{message}", color: :red)
           else
-            puts message
+            @prompt.say(message)
           end
-
-          # Add to main content for history
-          add_message(message, type)
         end
 
         # Called by CLI after mode selection in interactive flow (added helper)
@@ -175,8 +113,8 @@ module Aidp
           @current_mode = mode
           if @headless
             header = (mode == :analyze) ? "Analyze Mode" : "Execute Mode"
-            puts header
-            puts "Select workflow"
+            @prompt.say(header)
+            @prompt.say("Select workflow")
           end
         end
 
@@ -186,74 +124,9 @@ module Aidp
           @workflow_active = true
           @current_step = step_name
           questions = extract_questions_for_step(step_name)
-          questions.each { |q| puts q }
+          questions.each { |q| @prompt.say(q) }
           # Simulate quick completion
-          puts "#{step_name.split("_").first} completed" if step_name.start_with?("00_PRD")
-        end
-
-        def add_message(message, type = :info)
-          # Just add to a simple message log - no recursion
-          # This method is used by job monitoring, not for display
-        end
-
-        def show_progress(message, progress = 0)
-          if progress > 0
-            progress_bar = TTY::ProgressBar.new(
-              "⏳ #{message} [:bar] :percent",
-              total: 100,
-              width: 40
-            )
-            progress_bar.current = progress
-            progress_bar.render
-          else
-            # Use the unified spinner helper for indeterminate progress
-            @current_spinner = TTY::Spinner.new("⏳ #{message} :spinner", format: :pulse)
-            @current_spinner.start
-          end
-        end
-
-        def hide_progress
-          @current_spinner&.stop
-          @current_spinner = nil
-        end
-
-        # Job display methods
-        def show_jobs_dashboard
-          return unless @jobs_visible && @jobs.any?
-
-          # Create jobs table
-          table = TTY::Table.new(header: ["Status", "Job", "Provider", "Elapsed", "Message"])
-
-          @jobs.each do |job_id, job|
-            status_icon = case job[:status]
-            when :running then @pastel.green("●")
-            when :completed then @pastel.blue("●")
-            when :failed then @pastel.red("●")
-            when :pending then @pastel.yellow("●")
-            else @pastel.white("●")
-            end
-
-            elapsed = format_elapsed_time(Time.now - job[:started_at])
-            status_text = "#{status_icon} #{job[:status].to_s.capitalize}"
-
-            table << [
-              status_text,
-              job[:name],
-              job[:provider],
-              elapsed,
-              job[:message]
-            ]
-          end
-
-          # Display in a box
-          box = TTY::Box.frame(
-            width: 80,  # Fixed width instead of @screen.width
-            height: @jobs.length + 3,
-            title: {top_left: "🔄 Background Jobs"},
-            border: {type: :thick}
-          )
-
-          puts box.render(table.render(:unicode, padding: [0, 1]))
+          @prompt.say("#{step_name.split("_").first} completed") if step_name.start_with?("00_PRD")
         end
 
         # Enhanced workflow display
@@ -379,68 +252,6 @@ module Aidp
 
         private
 
-        # Very lightweight key listener just for spec expectations (F1 help, Ctrl shortcuts)
-        def start_key_listener
-          return if @key_thread
-          return unless $stdin&.tty? || @headless
-
-          @key_thread = Thread.new do
-            while @display_active
-              begin
-                if IO.select([$stdin], nil, nil, 0.1)
-                  ch = $stdin.getc
-                  next unless ch
-                  code = ch.ord
-                  case code
-                  when 16 # Ctrl+P
-                    if @workflow_active
-                      puts "Workflow Paused"
-                    end
-                  when 18 # Ctrl+R
-                    if @workflow_active
-                      puts "Workflow Resumed"
-                    end
-                  when 19 # Ctrl+S
-                    if @workflow_active
-                      puts "Workflow Stopped"
-                      @workflow_active = false
-                    end
-                  when 27 # ESC - re-show menu header hint
-                    puts "Choose your mode"
-                  else
-                    # Detect simple F1 sequence variants: some tmux sends ESC O P, or just O then P in tests
-                    if ch == "O"
-                      # Peek next char non blocking
-                      nxt = begin
-                        $stdin.read_nonblock(1)
-                      rescue
-                        nil
-                      end
-                      if nxt == "P"
-                        show_help_overlay
-                      end
-                    elsif ch == "\e"
-                      seq = begin
-                        $stdin.read_nonblock(2)
-                      rescue
-                        ""
-                      end
-                      show_help_overlay if seq.include?("OP")
-                    end
-                  end
-                end
-              rescue IOError
-                # ignore
-              end
-            end
-          end
-        end
-
-        def show_help_overlay
-          puts "Keyboard Shortcuts"
-          puts "Ctrl+P Pause | Ctrl+R Resume | Ctrl+S Stop | Esc Back"
-        end
-
         def extract_questions_for_step(step_name)
           return [] unless @headless
           root = ENV["AIDP_ROOT"] || Dir.pwd
@@ -465,55 +276,12 @@ module Aidp
           []
         end
 
-        def initialize_display
-          @cursor.hide
-        end
-
         def restore_screen
           @cursor.show
           @cursor.clear_screen
           @cursor.move_to(1, 1)
         end
 
-        def refresh_display
-          return unless @input_mode
-
-          @cursor.save
-          @cursor.move_to(1, @screen.height)
-
-          # Clear the bottom line
-          print " " * @screen.width
-
-          # Draw input overlay at the bottom
-          draw_input_overlay
-
-          @cursor.restore
-        end
-
-        def draw_input_overlay
-          # Get terminal width and ensure we don't exceed it
-          width = @screen.width
-          max_width = width - 4  # Leave some margin
-
-          # Create the input line
-          input_line = @input_prompt + @input_buffer
-
-          # Truncate if too long
-          if input_line.length > max_width
-            input_line = input_line[0...max_width] + "..."
-          end
-
-          # Draw the input overlay at the bottom
-          @cursor.move_to(1, @screen.height)
-          print @pastel.blue("┌") + "─" * (width - 2) + @pastel.blue("┐")
-
-          @cursor.move_to(1, @screen.height + 1)
-          print @pastel.blue("│") + input_line + " " * (width - input_line.length - 2) + @pastel.blue("│")
-
-          @cursor.move_to(1, @screen.height + 2)
-          print @pastel.blue("└") + "─" * (width - 2) + @pastel.blue("┘")
-        end
-
         def setup_signal_handlers
           Signal.trap("INT") do
             stop_display_loop

data/lib/aidp/harness/ui/enhanced_workflow_selector.rb

@@ -125,26 +125,13 @@ module Aidp
         end
 
         def collect_project_info_interactive
-          @tui.show_message("📋 Project Setup", :info)
-          @tui.show_message("Let's set up your development workflow", :info)
-
           @user_input[:project_description] = @tui.get_user_input("What do you want to build? (Be specific about features and goals)")
-          @tui.show_message("✅ Project description captured", :success)
-
           @user_input[:tech_stack] = @tui.get_user_input("What technology stack are you using? (e.g., Ruby/Rails, Node.js, Python/Django) [optional]")
-          @tui.show_message("✅ Tech stack captured", :success)
-
           @user_input[:target_users] = @tui.get_user_input("Who are the target users? (e.g., developers, end users, internal team) [optional]")
-          @tui.show_message("✅ Target users captured", :success)
-
           @user_input[:success_criteria] = @tui.get_user_input("How will you know this is successful? (e.g., performance metrics, user adoption) [optional]")
-          @tui.show_message("✅ Success criteria captured", :success)
         end
 
         def choose_workflow_type_interactive
-          @tui.show_message("🛠️ Workflow Selection", :info)
-          @tui.show_message("Choose your development approach:", :info)
-
           workflow_options = [
             "🔬 Exploration/Experiment - Quick prototype or proof of concept",
             "🏗️ Full Development - Production-ready feature or system"
@@ -154,10 +141,8 @@ module Aidp
           @user_input[:workflow_type] = selected
 
           if selected.include?("Exploration")
-            @tui.show_message("🔬 Using exploration workflow - fast iteration, minimal documentation", :info)
             :exploration
           else
-            @tui.show_message("🏗️ Using full development workflow - comprehensive planning and documentation", :info)
             :full
           end
         end
@@ -174,7 +159,6 @@ module Aidp
         end
 
         def generate_exploration_steps
-          @tui.show_message("🔬 Using exploration workflow - fast iteration, minimal documentation", :info)
           [
             "00_PRD",           # Generate PRD from user input (no manual gate)
             "10_TESTING_STRATEGY", # Ensure we have tests
@@ -184,9 +168,6 @@ module Aidp
         end
 
         def generate_full_steps_interactive
-          @tui.show_message("🏗️ Customize Full Workflow", :info)
-          @tui.show_message("Customizing your full development workflow", :info)
-
           available_steps = [
             "00_PRD - Product Requirements Document (required)",
             "01_NFRS - Non-Functional Requirements (optional)",
@@ -212,8 +193,6 @@ module Aidp
 
           # Add implementation at the end
           selected_steps << "16_IMPLEMENTATION"
-
-          @tui.show_message("✅ Selected #{selected_steps.length} steps for your workflow", :success)
           selected_steps
         end
 

data/lib/aidp/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Aidp
-  VERSION = "0.8.3"
+  VERSION = "0.9.1"
 end

data/templates/README.md

@@ -102,7 +102,7 @@ The `harness` section controls the overall behavior of the harness system:
 
 The `providers` section defines individual provider settings:
 
-- `type`: Provider type (package, api, byok)
+- `type`: Provider type (package, api, passthrough)
 - `priority`: Provider priority (higher = more preferred)
 - `models`: Available models for the provider
 - `features`: Provider capabilities
@@ -186,11 +186,11 @@ time_based:
 - **Examples**: Claude, Gemini
 - **Configuration**: Requires API keys
 
-### BYOK Providers
+### Passthrough Providers
 
-- **Type**: `byok`
-- **Pricing**: User provides their own API key
-- **Examples**: OpenAI, custom APIs
+- **Type**: `passthrough`
+- **Pricing**: Uses underlying service pricing
+- **Examples**: macOS UI automation, custom integrations
 - **Configuration**: User manages API keys
 
 ## Best Practices
@@ -198,7 +198,7 @@ time_based:
 ### Security
 
 - Store API keys in environment variables, not in the config file
-- Use `restrict_to_non_byok: true` to avoid BYOK providers
+- Use `no_api_keys_required: true` to avoid providers that require API keys
 - Enable SSL verification in production
 - Configure allowed/blocked hosts appropriately
 

data/templates/aidp.yml.example

@@ -13,8 +13,8 @@ harness:
   # Fallback providers in order of preference
   fallback_providers: ["claude", "gemini"]
 
-  # Restrict to non-BYOK (Bring Your Own Key) providers only
-  restrict_to_non_byok: true
+  # Only use providers that don't require API keys
+  no_api_keys_required: true
 
   # Provider weights for load balancing (higher = more preferred)
   provider_weights:
@@ -127,7 +127,7 @@ harness:
 providers:
   # Cursor provider (package-based)
   cursor:
-    type: "package"           # package, api, or byok
+    type: "package"           # package, api, or passthrough
     priority: 1               # Provider priority (higher = more preferred)
     default_flags: []         # Default command-line flags for this provider
 
@@ -395,9 +395,9 @@ providers:
       timeout: 30
       max_redirects: 5
 
-  # Example BYOK provider
+  # Example passthrough provider
   # openai:
-  #   type: "byok"
+  #   type: "passthrough"
   #   priority: 4
   #   default_flags: ["--model", "gpt-4"]
   #   models: ["gpt-4", "gpt-3.5-turbo"]
@@ -428,7 +428,7 @@ providers:
 # Provider types explained:
 # - "package": Uses package-based pricing (e.g., Cursor Pro)
 # - "api": Uses API-based pricing with token limits
-# - "byok": Bring Your Own Key (user provides API key)
+# - "passthrough": Uses underlying service (user manages API keys)
 
 # Environment-specific configurations
 environments:
@@ -584,7 +584,7 @@ users:
 # - Set max_tokens based on your API plan limits
 # - Use default_flags to customize provider behavior
 # - Configure fallback_providers for automatic failover
-# - Set restrict_to_non_byok: true to avoid BYOK providers
+# - Set no_api_keys_required: true to avoid providers that require API keys
 # - Adjust provider_weights to control load balancing
 # - Configure model_weights for model selection within providers
 # - Set appropriate timeouts for different models

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: aidp
 version: !ruby/object:Gem::Version
-  version: 0.8.3
+  version: 0.9.1
 platform: ruby
 authors:
 - Bart Agapinan
@@ -248,6 +248,7 @@ files:
 - lib/aidp/analyze/runner.rb
 - lib/aidp/analyze/steps.rb
 - lib/aidp/cli.rb
+- lib/aidp/cli/first_run_wizard.rb
 - lib/aidp/cli/jobs_command.rb
 - lib/aidp/cli/terminal_io.rb
 - lib/aidp/config.rb