Skip to content

refactor: simplify output structure for altitude and WFS describe type tools#102

Merged
mborne merged 1 commit into
masterfrom
86-remove-result
May 2, 2026
Merged

refactor: simplify output structure for altitude and WFS describe type tools#102
mborne merged 1 commit into
masterfrom
86-remove-result

Conversation

@esgn
Copy link
Copy Markdown
Member

@esgn esgn commented May 2, 2026

Closes #86

This PR removes the redundant result wrapper from singleton MCP tool outputs so their domain fields are exposed directly in structuredContent and in the serialized JSON text payload.

The change is intentionally limited to singleton object responses:

  • altitude now returns { lon, lat, altitude, accuracy }
  • gpf_wfs_describe_type now returns { id, namespace, name, title, description, properties }

The corresponding outputSchema definitions were flattened to match the new top-level shape, and the unit/integration tests were updated accordingly. List-style outputs using results are unchanged, and WFS FeatureCollection responses are explicitly out of scope.

This is an API cleanup and consistency change aligned with MCP output schema examples. It is a breaking response-shape change for any client currently reading structuredContent.result.

@esgn esgn marked this pull request as ready for review May 2, 2026 14:55
@esgn esgn requested a review from mborne May 2, 2026 14:55
@mborne mborne merged commit 11a635e into master May 2, 2026
3 checks passed
@mborne mborne deleted the 86-remove-result branch May 2, 2026 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove result wrapper from singleton structured tool outputs

2 participants