Repository navigation
chore: the examples move to Camel 4.23.0 - #126
Merged
Merged
Conversation
…p-server and contracts/openapi-server use what it brings CI pins Camel 4.23.0 for the CLI and for the integrations the Citrus tests start, as camel@apache/camel defaults to the latest LTS release (4.22.0). ai/mcp-server sets the server title, description and instructions, its README says that the server serves ai-resource routes next to the tools, and the log lines it shows are those of 4.23.0. contracts/openapi-server keeps its /api base path, but its README no longer says the servers entry must have a path: on 4.23 an entry without one serves the operations at the root. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Signed-off-by: Claus Ibsen <claus.ibsen@gmail.com>
…assifier writes at runtime The catalog was generated in a directory where the example had run, so it listed tickets-out/*/T-*.json, which are not in the repository; fetching them from GitHub answers 404. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Signed-off-by: Claus Ibsen <claus.ibsen@gmail.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Moves the examples to Camel 4.23.0, the 4.23 items of #86.
camel@apache/cameldefaults to the latest LTS release (4.22.0), so the workflow pinsCAMEL_VERSIONwith-Dcamel.jbang.versionfor the CLI, andCITRUS_CAMEL_JBANG_VERSION/CITRUS_CAMEL_CLI_VERSIONfor the integrations the Citrus tests start. The same change is in contracts/rest-code-first - the REST DSL with JSON binding to a Java record and a generated API description #118; whichever merges second rebases cleanly.mcp-server-title,mcp-server-descriptionandmcp-instructions. The README explains them, says the server servesai-resourceroutes next to tools, and shows the 4.23.0 log lines ("serving tools and resources", and the current line numbers)./api, but its README no longer says theserversentry must have a path. On 4.23.0 an entry without one serves the operations at the root, checked with a copy of the contract.--camel-version. The 4.23.0 CLI showsteachesincamel run --example=<group>, andciSkipas the 🤖 icon.ai/ticket-classifier/tickets-out/*/T-*.json. These are files the example writes when it runs, not repository files, and fetching them from GitHub returns 404. Regenerated.The circuit-breaker (#81) and LangChain4j provider (#91) items are in their own PRs, stacked on this one.
Tested locally with Camel 4.23.0: the mcp-server Citrus test passes when run with the same command and environment as CI (the integration logs "Apache Camel 4.23.0"), and
initializereturns the title, description and instructions.Claude Code on behalf of davsclaus
🤖 Generated with Claude Code