Skip to content

feat: align discovery with mppx#184

Merged
brendanjryan merged 2 commits intotempoxyz:mainfrom
stevencartavia:service-metadata-2
Apr 8, 2026
Merged

feat: align discovery with mppx#184
brendanjryan merged 2 commits intotempoxyz:mainfrom
stevencartavia:service-metadata-2

Conversation

@stevencartavia
Copy link
Copy Markdown
Contributor

@stevencartavia stevencartavia commented Mar 31, 2026

Simplified to_llms_txt_with() to match mppx toLlmsTxt(): title heading, description blockquote, service list, OpenAPI discovery link.

Part 2 of 3:

  • Step 1: add service metadata fields 164
  • **Step 2: align discovery with mppx (this PR)
  • Step 3: add generate_openapi() + wire GET /openapi.json route

@brendanjryan
Copy link
Copy Markdown
Collaborator

@stevencartavia -- what is the motivation for this change? I would prefer we match the equivalent shape and functionality mppx if possible instead of adding features only to the rust SDK unless explicitly needed

@stevencartavia
Copy link
Copy Markdown
Contributor Author

I've removed the features that were deleted in (wevm/mppx@b4e1a3d) (wants_markdown, markdown renderers, DiscoveryResponse::Markdown) and simplifies to_llms_txt_with() to match mppx's toLlmsTxt(). Follow-up PR will add generate_openapi() matching mppx's generateProxy().

Copy link
Copy Markdown
Collaborator

@brendanjryan brendanjryan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@stevencartavia stevencartavia changed the title feat: add discovery rendering functions feat: align discovery with mppx Apr 7, 2026
@brendanjryan brendanjryan merged commit 0be1965 into tempoxyz:main Apr 8, 2026
7 checks passed
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.

2 participants