feat: enhance gitignore check and update analysis documentation handling

Author: hasansino

pkg/agentenv/agentenv.go

@@ -1,6 +1,7 @@
 package agentenv
 
 import (
+	"bytes"
 	"context"
 	"fmt"
 	"log/slog"
@@ -157,14 +158,12 @@ func updateGitIgnore(outputPath string) error {
 	}
 
 	if stat.Size() > 0 {
-		// Check if the marker already exists
+		// Check if the marker already exists anywhere in the file
 		content, err := os.ReadFile(gitignorePath)
 		if err != nil {
 			return fmt.Errorf("failed to read .gitignore: %w", err)
 		}
-		contentLen := len(content)
-		markerLen := len(gitignoreMarker)
-		if contentLen >= markerLen && string(content[contentLen-markerLen:]) == gitignoreMarker {
+		if bytes.Contains(content, []byte(gitignoreMarker)) {
 			// Marker already exists, no need to add again
 			return nil
 		}

pkg/agentenv/analyse.md

@@ -9,7 +9,7 @@ Analyze this project and extract STABLE, HIGH-LEVEL information that rarely chan
 - Explain the mental model, not the code structure
 - Wrap your analysis in tags: `### BEGIN ANALYSIS ###` and `### END ANALYSIS ###` (without the backticks)
 
-Example output structure (use Markdown):
+## Example output:
 
 ```Markdown
 
@@ -73,9 +73,13 @@ The architecture follows a modular, service-oriented design philosophy where dis
     - Backward compatibility approach
     - NOT: Current roadmap or pending features
 
+8. **Folder Structure Philosophy**
+    - Rationale behind the organization of code into folders
+    - Document every case with examples
+    - For every example draw tree diagrams
+
 ## Exclude From Analysis
 
-Do not include these volatile implementation details:
 - Current file paths and function names
 - Specific dependencies and versions
 - Implementation details and code snippets

pkg/agentenv/analyser.go

@@ -20,7 +20,7 @@ const (
 	endMarker   = "### END ANALYSIS ###"
 
 	providerClaude = "claude"
-	modelClaude    = "claude-opus-4-1"
+	modelClaude    = "claude-sonnet-4-0"
 
 	providerGemini = "gemini"
 	modelGemini    = "gemini-2.5-pro"
@@ -92,13 +92,25 @@ func (a *analyser) Run(ctx context.Context, provider string, model string) error
 
 	if err != nil {
 		a.logger.Error("Analysis command failed", "error", err, "stderr", errorOutput)
-		if output == "" && errorOutput != "" {
-			output = errorOutput
+		errorDetails := fmt.Sprintf(
+			"# Analysis Failed\n\n## Command Error\n%v\n\n## Standard Error Output\n%s\n\n## Standard Output\n%s\n",
+			err,
+			errorOutput,
+			output,
+		)
+		if output == "" {
+			output = errorDetails
+		} else {
+			output = errorDetails + "\n\n## Original Output\n" + output
 		}
 	}
 
 	if output == "" {
-		return fmt.Errorf("no output from analysis")
+		output = fmt.Sprintf(
+			"# Analysis Failed\n\nNo output generated from the analysis command.\nCommand: %s\nError: %v\n",
+			cmd.String(),
+			err,
+		)
 	}
 
 	output = a.extractAnalysis(output)

pkg/agentenv/template/chunks/20-operation.tpl.md

@@ -1,4 +1,4 @@
-### Operation
+    ### Operation
 
 - Ignore anything between `[IGNORE]` and `[/IGNORE]` tags - it used for internal notes
 - When possible always prefer executing actions (tools) in parallel.