Sort toolsnaps JSON keys recursively to eliminate noop churn

[Copilot]

Summary

Toolsnaps now write JSON with consistent alphabetical key ordering at all nesting levels, eliminating diff churn from Go's randomized map iteration.

Why

Toolsnap diffs showed field reordering noise even when schemas hadn't changed. Example from a typical regeneration:

 "order": {
-  "type": "string",
-  "description": "Sort order",
+  "description": "Sort order",
   "enum": ["asc", "desc"]
+  "type": "string"
 }

What changed

• Added sortJSONKeys() in internal/toolsnaps/toolsnaps.go: unmarshals and remarshals JSON to leverage Go's built-in alphabetical map key sorting
• Updated writeSnap() to sort before writing
• Regenerated all 94 toolsnap files with consistent ordering
• Added tests for sorting behavior and idempotency

MCP impact

• No tool or API changes - Only affects snapshot file formatting, not runtime behavior or schemas

Prompts tested (tool changes only)

N/A

Security / limits

• No security or limits impact - Internal testing infrastructure only

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 - Internal change to test infrastructure

Original prompt

Sort the fields of toolsnaps so that we never see noop churn of them before writing them. Might even need to be recursive, https://github.com/github/github-mcp-server/pull/1776/changes is a good example

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[Copilot]

Pull request overview

This PR addresses JSON key ordering instability in toolsnap files caused by Go's randomized map iteration. The solution adds a sortJSONKeys() function that recursively sorts all JSON object keys alphabetically by unmarshaling and remarshaling JSON data, leveraging Go's built-in alphabetical key sorting for maps.

Changes:

• Added sortJSONKeys() function to recursively sort JSON keys at all nesting levels
• Modified writeSnap() to sort JSON before writing
• Regenerated all 94 toolsnap files with consistent alphabetical key ordering
• Added comprehensive tests for sorting behavior and idempotency

Reviewed changes

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

File	Description
internal/toolsnaps/toolsnaps.go	Added sortJSONKeys() function and integrated it into writeSnap()
internal/toolsnaps/toolsnaps_test.go	Added comprehensive test coverage for JSON key sorting and idempotency
pkg/github/toolsnaps/*.snap	Regenerated all 94 toolsnap files with alphabetically sorted keys