config: eliminate duplicate body between UnsupportedField and InvalidValue

[Copilot]

UnsupportedField and InvalidValue in internal/config/validation_errors.go had byte-for-byte identical bodies — any future change to ValidationError construction (e.g. adding a Code field) would need to be applied twice.

Changes

• internal/config/validation_errors.go: UnsupportedField is now a documented alias that delegates to InvalidValue, removing the duplicate struct literal while preserving both names (both have active call sites in validation.go)

// Before: UnsupportedField had its own identical struct literal
func UnsupportedField(fieldName, message, jsonPath, suggestion string) *ValidationError {
    return &ValidationError{Field: fieldName, Message: message, JSONPath: jsonPath, Suggestion: suggestion}
}

// After: delegates to InvalidValue
func UnsupportedField(fieldName, message, jsonPath, suggestion string) *ValidationError {
    return InvalidValue(fieldName, message, jsonPath, suggestion)
}

No call sites changed — both function names are retained for semantic clarity (UnsupportedField = structural/unrecognized field; InvalidValue = constraint violation).

[Copilot]

Pull request overview

This PR removes duplicated ValidationError construction logic by making UnsupportedField a documented alias that delegates to InvalidValue, keeping both names available for clearer call-site semantics while ensuring future changes to ValidationError only need to be made once.

Changes:

• Refactored UnsupportedField to call InvalidValue(...) instead of duplicating the struct literal.
• Added doc comments clarifying UnsupportedField as an alias retained for semantic clarity.

Show a summary per file

File	Description
internal/config/validation_errors.go	Deduplicates ValidationError construction by delegating UnsupportedField to InvalidValue and documents the alias intent.

Copilot's findings

Tip

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

• Files reviewed: 1/1 changed files
• Comments generated: 0

[lpcox]

@copilot resolve the merge conflicts in this pull request

[Copilot]

@copilot resolve the merge conflicts in this pull request

Resolved. The only conflict was in the doc comment for UnsupportedField — both sides had made the same functional change (delegating to InvalidValue) but with slightly different wording. Merged with origin/main in 7da6d07, keeping the main-branch doc comment. All checks pass.