[PR] CAMEL-23730: Export dry-run should add StubBeanRepository when stub pattern is component:* [camel]

["davsclaus (via GitHub)" <gi...@apache.org>]

davsclaus opened a new pull request, #23920:
URL: https://github.com/apache/camel/pull/23920

   ## Summary
   
   - Fix regression where `camel export` dry-run fails with `NoSuchBeanException` for bean references (e.g., `aggregationRepository=#event_aggregation`) when the route uses EIPs that resolve beans via `mandatoryLookup()`
   - The root cause: commit `8a185b5618f8` (CAMEL-23355) changed the stub pattern from `"*"` to `"component:*"` in `Run.java`, but the corresponding condition in `KameletMain.java:527` that adds `StubBeanRepository` to the registry was not updated to also handle `"component:*"`
   - Additionally, `StubBeanRepository` is now constructed with `"*"` pattern so it correctly matches bean names (a `"component:*"` pattern would not match bean names like `event_aggregation`)
   
   ## Changes
   
   **`KameletMain.java`** — Added `"component:*".equals(stubPattern)` to the condition that adds `StubBeanRepository`, and always pass `"*"` to `StubBeanRepository` since bean references should be stubbed regardless of the component stub pattern.
   
   **`StubBeanRepositoryTest.java`** — New test verifying that `StubBeanRepository("*")` stubs bean references and `StubBeanRepository("component:*")` does not.
   
   ## Test plan
   
   - [x] New `StubBeanRepositoryTest` verifies pattern matching behavior
   - [x] Existing tests pass
   - [ ] Manual: `camel export` with a route using `aggregationRepository=#myRepo` should complete without `NoSuchBeanException`
   
   _Claude Code on behalf of Claus Ibsen_
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] CAMEL-23730: Export dry-run should add StubBeanRepository when stub pattern is component:* [camel]

["davsclaus (via GitHub)" <gi...@apache.org>]

davsclaus merged PR #23920:
URL: https://github.com/apache/camel/pull/23920


-- 
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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] CAMEL-23730: Export dry-run should add StubBeanRepository when stub pattern is component:* [camel]

["github-actions[bot] (via GitHub)" <gi...@apache.org>]

github-actions[bot] commented on PR #23920:
URL: https://github.com/apache/camel/pull/23920#issuecomment-4672192510

   :star2: Thank you for your contribution to the Apache Camel project! :star2:
     :robot: CI automation will test this PR automatically.
   
     :camel: 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](https://camel.apache.org/community/team/#committers) have access to the summary.
     
     :warning: Be careful when sharing logs. Review their contents before sharing them publicly.


-- 
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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] CAMEL-23730: Export dry-run should add StubBeanRepository when stub pattern is component:* [camel]

["github-actions[bot] (via GitHub)" <gi...@apache.org>]

github-actions[bot] commented on PR #23920:
URL: https://github.com/apache/camel/pull/23920#issuecomment-4672394055

   <!-- ci-tested-modules -->
   :test_tube: **CI tested the following changed modules:**
   
   - `dsl/camel-jbang/camel-jbang-core`
   - `dsl/camel-kamelet-main`
   
   :warning: **Some tests are disabled on GitHub Actions** (`@DisabledIfSystemProperty(named = "ci.env.name")`) and require manual verification:
   
   - `dsl/camel-jbang/camel-jbang-core`: 1 test(s) disabled on GitHub Actions
   
   :bulb: **Manual integration tests recommended:**
   
   > You modified `dsl/camel-jbang/camel-jbang-core`. The related integration tests in `dsl/camel-jbang/camel-jbang-it` are excluded from CI. Consider running them manually:
   > ```
   > mvn verify -f dsl/camel-jbang/camel-jbang-it -Djbang-it-test
   > ```
   
   <details><summary>All tested modules (7 modules)</summary>
   
   - Camel :: JBang :: Core
   - Camel :: JBang :: MCP
   - Camel :: JBang :: Plugin :: Route Parser
   - Camel :: JBang :: Plugin :: TUI
   - Camel :: JBang :: Plugin :: Validate
   - Camel :: Kamelet Main
   - Camel :: Launcher :: Container
   
   </details>
   
   ---
   :gear: [View full build and test results](https://github.com/apache/camel/actions/runs/27288324986)


-- 
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: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org