atiaomar1978-hub commented on PR #25202: URL: https://github.com/apache/camel/pull/25202#issuecomment-5108610353
### Review summary (Bugbot + Grok-style pass) **Addressed in commit `171194faacc`:** - Co-bind `--port` and management port when `--openapi-ui` is used (Swagger UI loads `/q/openapi.json` from the same host/port). - Run `prepareOpenApiUiServerOptions()` before persisting `camel.server.*` settings; write `camel.server.enabled`, `camel.management.enabled`, and `camel.rest.component=platform-http`. - Preserve absolute `http(s)://` values for `openapiUiSpecPath`; added `OpenApiUiSupportTest` + port alignment test in `RunOpenApiUiOptionsTest`. - Mark `camel.jbang.openapiUi` with `insecure:dev` metadata. **Not changed (noted):** - Idempotent Vert.x route registration on management server restart (pre-existing pattern for other `/q/*` consoles). - Optional jbang IT for end-to-end `--openapi-ui` (unit coverage deemed sufficient for now). **False positive / out of scope:** - Bugbot flagged corrupted `camel-xml-io.xsd` in the catalog — that was an **uncommitted local artifact**, not in the PR diff. Tests re-run locally: 6 tests in `camel-platform-http-main`, 2 in `camel-jbang-core` — all green. _AI-generated comment on behalf of [atiaomar1978-hub](https://github.com/atiaomar1978-hub)_ -- 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]
