Croway commented on PR #27960:
URL: https://github.com/apache/camel/pull/27960#issuecomment-6226661157

   Review follow-ups are in b66295875a5d, one commit on top of the three 
existing ones; the branch is rebased on main (the only conflict was the 4.24 
upgrade guide, both sides kept), hence the force push.
   
   - An operation without a `200` response is listed as answering HTTP 204 with 
an empty body, as the fixed `loadMockData` does; the PR description no longer 
presents CAMEL-25663 as out of scope.
   - `filePath` of a parameterized path carries the sample value 
(`camel-mock/pets/1.json`), `requestPath` keeps the template; the MCP page says 
so, and the component test writes the file at `filePath` as given.
   - The note and the mode explanation name the media type the example is taken 
for (Content-Type or Accept, `*/*` falling back to application/json then 
application/xml), otherwise HTTP 204.
   
   Tests: `OpenApiToolsTest` 21, `OpenApiScaffoldRestOpenApiTest` 2, whole 
camel-jbang-mcp module (468) and camel-jbang-core suite, all passing.
   
   _Claude Code on behalf of Croway_


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to