Simplify GenerateOutputSchema to use idiomatic For[T]() pattern

[Copilot]

The GenerateOutputSchema function used manual reflection indirection when the jsonschema-go library provides a cleaner generic For[T]() function.

Changes

• Replace reflect.TypeOf() indirection with direct jsonschema.For[T](nil) call
• Remove unused fmt and reflect imports
• Reduce function from 11 lines to 1 line

Before/After

// Before: manual reflection
func GenerateOutputSchema[T any]() (*jsonschema.Schema, error) {
	var zero T
	typ := reflect.TypeOf(zero)
	schema, err := jsonschema.ForType(typ, &jsonschema.ForOptions{})
	if err != nil {
		return nil, fmt.Errorf("failed to generate schema: %w", err)
	}
	return schema, nil
}

// After: idiomatic generic pattern
func GenerateOutputSchema[T any]() (*jsonschema.Schema, error) {
	return jsonschema.For[T](nil)
}

The For[T]() function internally uses reflect.TypeFor[T]() which is more efficient than instantiating a zero value and calling reflect.TypeOf().

Original prompt

---

This section details on the original issue you should resolve

<issue_title>[plan] Simplify GenerateOutputSchema to use idiomatic ForT pattern</issue_title>
<issue_description>## Objective

Refactor GenerateOutputSchema function to use the more idiomatic For[T]() pattern instead of manual reflection.

Context

The current implementation uses reflect.TypeOf() indirection, but the library provides a cleaner generic For[T]() function.

Current Implementation

func GenerateOutputSchema[T any]() (*jsonschema.Schema, error) {
    var zero T
    typ := reflect.TypeOf(zero)
    return jsonschema.ForType(typ, &jsonschema.ForOptions{})
}

Proposed Implementation

func GenerateOutputSchema[T any]() (*jsonschema.Schema, error) {
    return jsonschema.For[T](nil)
}

Benefits

• Cleaner code (3 lines → 1 line)
• No reflect.TypeOf indirection
• More idiomatic library usage
• Better error messages with type names

Files to Modify

• pkg/cli/mcp_schema.go - Update GenerateOutputSchema function

Acceptance Criteria

• Function uses jsonschema.For[T](nil) pattern
• Existing tests still pass
• Code is more concise and readable
  Related to [plan] Upgrade and enhance google/jsonschema-go integration #6830

AI generated by Plan Command for discussion #6818

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

• Fixes [plan] Simplify GenerateOutputSchema to use idiomatic For[T]() pattern #6832

---

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