Function parseAndDisplayPoutineOutputForDirectory

pkg/cli/poutine.go:385–525

parseAndDisplayPoutineOutputForDirectory parses poutine JSON output and displays all findings Returns the total number of warnings found across all files

(stdout string, verbose bool, gitRoot string)

// parseAndDisplayPoutineOutputForDirectory parses poutine JSON output and displays all findings
// Returns the total number of warnings found across all files
func parseAndDisplayPoutineOutputForDirectory(stdout string, verbose bool, gitRoot string) (int, error) {
	// Parse JSON output from stdout
	var output poutineOutput
	if stdout == "" {
		return 0, nil // No output means no findings
	}

	trimmed := strings.TrimSpace(stdout)
	if !strings.HasPrefix(trimmed, "{") {
		// Non-JSON output, likely an error
		if len(trimmed) > 0 {
			return 0, fmt.Errorf("unexpected poutine output format: %s", trimmed)
		}
		return 0, nil
	}

	if err := json.Unmarshal([]byte(stdout), &output); err != nil {
		return 0, fmt.Errorf("failed to parse poutine JSON output: %w", err)
	}

	// Display all findings (no filtering by file)
	totalWarnings := len(output.Findings)

	// Skip if no warnings
	if totalWarnings == 0 {
		return 0, nil
	}

	// Group findings by file for better readability
	findingsByFile := make(map[string][]poutineFinding)
	for _, finding := range output.Findings {
		findingsByFile[finding.Meta.Path] = append(findingsByFile[finding.Meta.Path], finding)
	}

	// Display findings for each file
	for filePath, findings := range findingsByFile {
		// Validate and sanitize file path to prevent path traversal
		cleanPath := filepath.Clean(filePath)

		// Convert to absolute path if relative
		absPath := cleanPath
		if !filepath.IsAbs(cleanPath) {
			absPath = filepath.Join(gitRoot, cleanPath)
		}

		// Ensure the file is within gitRoot to prevent path traversal
		absGitRoot, err := filepath.Abs(gitRoot)
		if err != nil {
			poutineLog.Printf("Failed to get absolute path for git root: %v", err)
			continue
		}

		absPath, err = filepath.Abs(absPath)
		if err != nil {
			poutineLog.Printf("Failed to get absolute path for %s: %v", filePath, err)
			continue
		}