Move validation functions from compiler.go to validation.go

[Copilot]

Part of #2388. Semantic function clustering analysis identified validation functions scattered across files. Three validation methods in compiler.go belong in validation.go with related validation logic.

Changes

• Moved validateHTTPTransportSupport() from compiler.go (line 2949) to validation.go (line 451)
• Moved validateMaxTurnsSupport() from compiler.go (line 2968) to validation.go (line 470)
• Moved validateWebSearchSupport() from compiler.go (line 2992) to validation.go (line 494)

All functions retain their *Compiler receiver and exact signatures. All required imports (fmt, os, console) already present in validation.go. Function calls in compiler.go unchanged.

Net change: -60 lines compiler.go, +60 lines validation.go

Original prompt

---

This section details on the original issue you should resolve

<issue_title>[task] Move validation functions from compiler.go to validation.go</issue_title>
<issue_description>## Objective
Relocate 3 validation functions from pkg/workflow/compiler.go to pkg/workflow/validation.go to improve code organization and separation of concerns.

Context

Part of issue #2388. The semantic function clustering analysis identified that validation functions are scattered across files. Three validation functions are currently misplaced in compiler.go when they should be in validation.go with other validation logic.

Approach

1. Move validateHTTPTransportSupport() from line 2949 in compiler.go to validation.go
2. Move validateMaxTurnsSupport() from line 2968 in compiler.go to validation.go
3. Move validateWebSearchSupport() from line 2992 in compiler.go to validation.go
4. Ensure all method receivers and imports are properly maintained
5. Update any references if needed

Files to Modify

• Move from: pkg/workflow/compiler.go (lines 2949, 2968, 2992)
• Move to: pkg/workflow/validation.go

Acceptance Criteria

• All 3 validation functions moved to validation.go
• Functions maintain same signatures and behavior
• All tests pass (make test)
• Code compiles successfully (make build)
• No import errors or broken references
  Related to [refactor] 🔧 Semantic Function Clustering Analysis - Refactoring Opportunities Identified #2388

AI generated by Plan Command for #2388</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes [task] Move validation functions from compiler.go to validation.go #2390

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[Copilot]

Pull Request Overview

This PR relocates three validation functions from compiler.go to validation.go to improve code organization and separation of concerns. The functions are moved without modification, maintaining their *Compiler receivers and exact signatures.

• Moved validateHTTPTransportSupport(), validateMaxTurnsSupport(), and validateWebSearchSupport() to validation.go
• All required imports (fmt, os, console) already exist in validation.go
• Function calls in compiler.go remain unchanged

Reviewed Changes

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

File	Description
pkg/workflow/validation.go	Added three validation functions for HTTP transport, max-turns, and web-search support
pkg/workflow/compiler.go	Removed the same three validation functions that were relocated to validation.go

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[github-actions]

Agentic Changeset Generator triggered by this pull request.