chore: refactor configure_extensions_dialog to reduce line count

[baxen]

Summary

Refactors the configure_extensions_dialog function from 353 lines down to 36 lines by extracting common functionality into focused helper functions.

Changes

Split the monolithic function into smaller, reusable helpers:

• prompt_extension_timeout(): Prompts for extension timeout
• prompt_extension_description(): Prompts for extension description
• prompt_extension_name(): Prompts for extension name with validation
• collect_env_vars(): Collects environment variables interactively
• collect_headers(): Collects custom headers interactively
• configure_builtin_extension(): Handles built-in extension setup
• configure_stdio_extension(): Handles stdio extension setup
• configure_sse_extension(): Handles SSE extension setup
• configure_streamable_http_extension(): Handles streaming HTTP setup

Result

• Before: 353 lines
• After: 36 lines (main function)
• Target: Under 200 lines ✅

The refactoring improves code readability and maintainability while preserving all existing functionality.

Testing

• cargo build -p goose-cli ✅
• cargo fmt ✅
• cargo clippy -p goose-cli -- -D warnings ✅
• ./scripts/clippy-lint.sh ✅

[Copilot]

Pull request overview

This PR successfully refactors the configure_extensions_dialog function from 353 lines to 36 lines by extracting common prompting and configuration logic into focused helper functions. The refactoring improves code maintainability and readability while preserving all existing functionality.

Key changes:

• Extracted 9 helper functions for reusable prompting and configuration logic
• Standardized description validation across all extension types (fixing a validation bug in the process)
• Maintained identical behavior for all extension configuration flows

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.