fix: inline registration schema definitions

[Subharup-31]

Summary

• make registration output schemas a single self-contained JSON Schema resource
• merge tool/error $defs into the registration wrapper and strip nested $id/$defs from oneOf schemas
• update structured-output schema tests to assert wrapper-level $defs and no nested schema resources

Closes #419

Testing

• npm test -- src/core/tests/structured-output-schema.test.ts
• npm run generate:version && npm run typecheck
• pre-commit hook: npm run format:check, npm run lint, npm run build

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 5c5a61d. Configure here.}

[cursor]

Smoke test asserts opposite of new behavior

High Severity

The smoke test in e2e-mcp-discovery.test.ts asserts listSimsOutputSchema?.oneOf?.[0]?.$defs is defined, but the production code change in getMcpOutputSchemaForRegistrationJson now strips $defs from oneOf entries via inlineRegistrationSchemaResource. The unit test explicitly asserts the opposite: oneOf[0].$defs is undefined. This smoke test will fail at runtime because it was not updated to match the new inlining behavior.

Additional Locations (1)

• src/core/structured-output-schema.ts#L215-L218

 

^{Triggered by project rule: Bugbot Review Guide for XcodeBuildMCP}

^{Reviewed by Cursor Bugbot for commit 5c5a61d. Configure here.}