Co-locate WithSchemaCache with other With functions

[Copilot]

Summary

Moved WithSchemaCache to be grouped with other HandlerOption functions in pkg/http/handler.go.

Why

Code organization feedback from PR #1993. Option builder functions should be co-located.

What changed

• Moved WithSchemaCache from top of file (after imports) to line 78, grouped with WithScopeFetcher, WithGitHubMCPServerFactory, WithInventoryFactory, WithOAuthConfig, and WithFeatureChecker

MCP impact

• No tool or API changes

Pure refactor, no behavioral changes.

Prompts tested (tool changes only)

N/A

Security / limits

• No security or limits impact

Tool renaming

• I am not renaming tools as part of this PR

Lint & tests

• Linted locally with ./script/lint
• Tested locally with ./script/test

Docs

• Not needed

Original prompt

Pull Request: #1993

Shouldn't WithSchemaCache be co-located with the other With functions in the file?

---

✨ 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 is a small refactor in the HTTP handler to improve code organization by grouping the WithSchemaCache HandlerOption with the other With* option builders in pkg/http/handler.go.

Changes:

• Moved WithSchemaCache from near the imports to the existing block of HandlerOption functions (no logic changes).