Croway commented on PR #27839: URL: https://github.com/apache/camel/pull/27839#issuecomment-6219360751
Review follow-ups are in faefe187b4bd, one commit on top of the two existing ones. The branch is rebased on main to resolve the conflict in the MCP `AuthoringTools` (two import lines), hence the force push; the review threads below are answered in place. - Prompt budget (blocking): I measured again after the rebase and tightened the shared `camel_catalog_doc` descriptions. The `AiPanelPromptBudgetTest` budgets are raised with comments: core 5000 -> 5100 (~5041), core+groups 5750 -> 5900 (~5820), full 10500 -> 10600 (~10485), core+semantic 6100 -> 6250 (~6170). - configKey: an option of both the endpoint and the component is listed once, with `scope: "endpoint,component"` and `configKey`. `matchedOptions` counts each name once. The kafka default answer now has `brokers` with its key. - platform-http: the doc paragraph and `mainConfig` now separate `camel run` (the server starts on its own, port 8080, `camel.server.port`/`--port`) from Camel Main (`camel.server.enabled=true`). The catalog mirror is regenerated. - Bot threads: section fences close only on their own delimiter (`////` included), `ignoringCase` is called once per lookup, and `capAnswer` has a last-resort text cut. The `optionsHint` now follows `listedOptions` when options are cut. - Tests: camel-jbang-core `CatalogDoc*Test,ToolRegistryTest,AuthoringToolsTest` 100/0 failures, camel-jbang-mcp 461/0, TUI `AiPanelPromptBudgetTest` 6/0. _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]
