Skip to content

CAMEL-23473: camel-jbang-mcp - drop verbose description from catalog list responses and lower default limit#23414

Merged
oscerd merged 1 commit into
apache:mainfrom
oscerd:fix/CAMEL-23473
May 21, 2026
Merged

CAMEL-23473: camel-jbang-mcp - drop verbose description from catalog list responses and lower default limit#23414
oscerd merged 1 commit into
apache:mainfrom
oscerd:fix/CAMEL-23473

Conversation

@oscerd
Copy link
Copy Markdown
Contributor

@oscerd oscerd commented May 21, 2026

Summary

Slims down the four catalog list tools (CAMEL-23473). Each list response previously
included a multi-sentence description per item and could return up to 50 items. The LLM rarely
needs the description in a list — name + title are enough to decide what to drill into, and
the matching *_doc tool already carries the description.

Changes

  • ComponentInfo, DataFormatInfo, LanguageInfo, EipInfo records: removed the description
    field. Mapping methods updated accordingly.
  • camel_catalog_components and camel_catalog_dataformats default limit lowered from 50 to 20.
    camel_catalog_languages and camel_catalog_eips did not previously expose a limit argument
    and are left as-is.
  • camel_catalog_components tool description updated to reflect the new payload shape and to
    point at camel_catalog_component_doc for descriptions.
  • Upgrade-guide entry added under camel-jbang-mcp in camel-4x-upgrade-guide-4_21.adoc.

Test plan

  • mvn -DskipTests install in dsl/camel-jbang/camel-jbang-mcp (formats + builds)
  • mvn test in dsl/camel-jbang/camel-jbang-mcp — 255 tests pass, including 2 new ones:
    • listInfoRecordsCarryNoDescription (reflective check across all four list records)
    • defaultLimitCapsResultsAtTwenty (asserts default limit returns ≤ 20 items)
  • componentInfoNullFieldsAreOmittedFromJson updated for the new ComponentInfo arity
  • mvn clean install -DskipTests from repo root — full reactor build green, no uncommitted
    regenerated artifacts

Claude Code on behalf of Andrea Cosentino

…list responses and lower default limit

The list-style catalog tools return up to 50 records, each carrying a
multi-sentence description. The LLM almost never needs the description
to decide what to drill into; name + title are enough, and the
matching *_doc tool already carries the description.

* Remove `description` from `ComponentInfo`, `DataFormatInfo`,
  `LanguageInfo`, `EipInfo`.
* Lower the default `limit` from 50 to 20 for
  `camel_catalog_components` and `camel_catalog_dataformats`.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Signed-off-by: Andrea Cosentino <ancosen@gmail.com>
@oscerd oscerd requested review from davsclaus and gnodet May 21, 2026 09:52
@github-actions
Copy link
Copy Markdown
Contributor

🌟 Thank you for your contribution to the Apache Camel project! 🌟
🤖 CI automation will test this PR automatically.

🐫 Apache Camel Committers, please review the following items:

  • First-time contributors require MANUAL approval for the GitHub Actions to run
  • You can use the command /component-test (camel-)component-name1 (camel-)component-name2.. to request a test from the test bot although they are normally detected and executed by CI.
  • You can label PRs using skip-tests and test-dependents to fine-tune the checks executed by this PR.
  • Build and test logs are available in the summary page. Only Apache Camel committers have access to the summary.

⚠️ Be careful when sharing logs. Review their contents before sharing them publicly.

@github-actions
Copy link
Copy Markdown
Contributor

🧪 CI tested the following changed modules:

  • docs
  • dsl/camel-jbang/camel-jbang-mcp

⚠️ Some tests are disabled on GitHub Actions (@DisabledIfSystemProperty(named = "ci.env.name")) and require manual verification:

  • dsl/camel-jbang/camel-jbang-mcp: 1 test(s) disabled on GitHub Actions

⚙️ View full build and test results

@oscerd oscerd merged commit b865ecb into apache:main May 21, 2026
7 checks passed
@oscerd oscerd deleted the fix/CAMEL-23473 branch May 21, 2026 11:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants