aidp 0.21.0 → 0.22.0

data/lib/aidp/workflows/guided_agent.rb

@@ -1,5 +1,6 @@
 # frozen_string_literal: true
 
+require "fileutils"
 require_relative "../harness/provider_manager"
 require_relative "../harness/provider_factory"
 require_relative "../harness/config_manager"
@@ -32,7 +33,6 @@ module Aidp
         @provider_manager = Aidp::Harness::ProviderManager.new(@config_manager, prompt: @prompt)
         @conversation_history = []
         @user_input = {}
-        @invalid_planning_responses = 0
         @verbose = verbose
         @debug_env = ENV["DEBUG"] == "1" || ENV["DEBUG"] == "2"
       end
@@ -105,6 +105,7 @@ module Aidp
 
           # Ask questions
           question_response[:questions]&.each do |question|
+            puts "\n" + ("─" * 80) if iteration > 1 # Visual separator between Q&A pairs
             answer = @prompt.ask(question)
             @conversation_history << {role: "assistant", content: question}
             @conversation_history << {role: "user", content: answer}
@@ -115,7 +116,8 @@ module Aidp
 
           # Guard: break loop after 10 iterations to avoid infinite loop
           if iteration >= 10
-            display_message("[ERROR] Planning loop exceeded 10 iterations. Provider may be returning generic responses.", type: :error)
+            display_message("[WARNING] Planning loop exceeded 10 iterations. Provider may be returning generic responses.", type: :warning)
+            display_message("Continuing with the plan information gathered so far...", type: :info)
             break
           end
         end
@@ -135,7 +137,7 @@ module Aidp
         end
 
         response = call_provider_for_analysis(system_prompt, user_prompt)
-        parsed = safe_parse_planning_response(response)
+        parsed = parse_planning_response(response)
         # Attach raw response for debug
         parsed[:raw_response] = response
         emit_verbose_raw_prompt(system_prompt, user_prompt, response)
@@ -375,53 +377,20 @@ module Aidp
         json_match = response_text.match(/```json\s*(\{.*?\})\s*```/m) ||
           response_text.match(/(\{.*\})/m)
 
-        return {error: :invalid_format} unless json_match
-        JSON.parse(json_match[1], symbolize_names: true)
-      rescue JSON::ParserError
-        {error: :invalid_format}
-      end
-
-      # Provides structured fallback sequence when provider keeps returning invalid planning JSON.
-      # After exceeding sequence length, switches to manual entry question.
-      def safe_parse_planning_response(response_text)
-        parsed = parse_planning_response(response_text)
-        return parsed unless parsed.is_a?(Hash) && parsed[:error] == :invalid_format
-
-        @invalid_planning_responses += 1
-        fallback_sequence = [
-          "Provide scope (key features) and primary users.",
-          "List 3-5 key functional requirements and any technical constraints.",
-          "Supply any non-functional requirements (performance/security) or type 'skip'."
-        ]
-
-        if @invalid_planning_responses <= fallback_sequence.size
-          {complete: false, questions: [fallback_sequence[@invalid_planning_responses - 1]], reasoning: "Fallback due to invalid provider response (format)", error: :fallback}
-        else
-          display_message("[ERROR] Provider returned invalid planning JSON #{@invalid_planning_responses} times. Enter combined plan details manually.", type: :error)
-          {complete: false, questions: ["Enter plan details manually (features; users; requirements; constraints) or type 'skip'"], reasoning: "Manual recovery mode", error: :manual_recovery}
+        unless json_match
+          raise ConversationError, "Provider returned invalid format: no JSON found in response"
         end
+
+        JSON.parse(json_match[1], symbolize_names: true)
+      rescue JSON::ParserError => e
+        raise ConversationError, "Provider returned invalid JSON: #{e.message}"
       end
 
       def update_plan_from_answer(plan, question, answer)
         # Simple heuristic-based plan updates
         # In a more sophisticated implementation, use AI to categorize answers
 
-        # IMPORTANT: Check manual recovery sentinel prompt first so it isn't misclassified
-        # by broader keyword heuristics (e.g., it contains the word 'users').
-        if question.start_with?("Enter plan details manually")
-          unless answer.strip.downcase == "skip"
-            parts = answer.split(/;|\|/).map(&:strip).reject(&:empty?)
-            features, users, requirements, constraints = parts
-            plan[:scope][:included] ||= []
-            plan[:scope][:included] << features if features
-            plan[:users][:personas] ||= []
-            plan[:users][:personas] << users if users
-            plan[:requirements][:functional] ||= []
-            plan[:requirements][:functional] << requirements if requirements
-            plan[:constraints][:technical] ||= []
-            plan[:constraints][:technical] << constraints if constraints
-          end
-        elsif question.downcase.include?("scope") || question.downcase.include?("include")
+        if question.downcase.include?("scope") || question.downcase.include?("include")
           plan[:scope][:included] ||= []
           plan[:scope][:included] << answer
         elsif question.downcase.include?("user") || question.downcase.include?("who")
@@ -532,7 +501,9 @@ module Aidp
           Generated by AIDP Plan & Execute workflow on #{Time.now.strftime("%Y-%m-%d")}
         PRD
 
-        File.write(File.join(@project_dir, "docs", "prd.md"), prd_content)
+        docs_dir = File.join(@project_dir, "docs")
+        FileUtils.mkdir_p(docs_dir) unless Dir.exist?(docs_dir)
+        File.write(File.join(docs_dir, "prd.md"), prd_content)
       end
 
       def generate_nfr_from_plan(plan)
@@ -548,7 +519,9 @@ module Aidp
           Generated by AIDP Plan & Execute workflow on #{Time.now.strftime("%Y-%m-%d")}
         NFR
 
-        File.write(File.join(@project_dir, "docs", "nfrs.md"), nfr_content)
+        docs_dir = File.join(@project_dir, "docs")
+        FileUtils.mkdir_p(docs_dir) unless Dir.exist?(docs_dir)
+        File.write(File.join(docs_dir, "nfrs.md"), nfr_content)
       end
 
       def generate_style_guide_from_plan(plan)
@@ -563,7 +536,9 @@ module Aidp
           Generated by AIDP Plan & Execute workflow on #{Time.now.strftime("%Y-%m-%d")}
         STYLE
 
-        File.write(File.join(@project_dir, "docs", "LLM_STYLE_GUIDE.md"), style_guide_content)
+        docs_dir = File.join(@project_dir, "docs")
+        FileUtils.mkdir_p(docs_dir) unless Dir.exist?(docs_dir)
+        File.write(File.join(docs_dir, "LLM_STYLE_GUIDE.md"), style_guide_content)
       end
 
       def format_hash_for_doc(hash)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: aidp
 version: !ruby/object:Gem::Version
-  version: 0.21.0
+  version: 0.22.0
 platform: ruby
 authors:
 - Bart Agapinan
@@ -245,6 +245,7 @@ files:
 - lib/aidp/analyze/tree_sitter_grammar_loader.rb
 - lib/aidp/analyze/tree_sitter_scan.rb
 - lib/aidp/cli.rb
+- lib/aidp/cli/devcontainer_commands.rb
 - lib/aidp/cli/enhanced_input.rb
 - lib/aidp/cli/first_run_wizard.rb
 - lib/aidp/cli/issue_importer.rb
@@ -354,6 +355,11 @@ files:
 - lib/aidp/providers/github_copilot.rb
 - lib/aidp/providers/opencode.rb
 - lib/aidp/rescue_logging.rb
+- lib/aidp/safe_directory.rb
+- lib/aidp/setup/devcontainer/backup_manager.rb
+- lib/aidp/setup/devcontainer/generator.rb
+- lib/aidp/setup/devcontainer/parser.rb
+- lib/aidp/setup/devcontainer/port_manager.rb
 - lib/aidp/setup/wizard.rb
 - lib/aidp/skills.rb
 - lib/aidp/skills/composer.rb