Skip to content

JavaScript: Fixing cross-module recipe calls#6894

Merged
jkschneider merged 3 commits intomainfrom
greg-js-cross-module-recipe
Mar 11, 2026
Merged

JavaScript: Fixing cross-module recipe calls#6894
jkschneider merged 3 commits intomainfrom
greg-js-cross-module-recipe

Conversation

@greg-at-moderne
Copy link
Contributor

@greg-at-moderne greg-at-moderne commented Mar 9, 2026

What's changed?

When preparing a recipe, making sure it's subrecipes (from recipeList) are installed too.

What's your motivation?

Fix:

Caused by io.moderne.jsonrpc.JsonRpcException: {code=-32603, message='Request PrepareRecipe failed with message: Could not find recipe with id org.openrewrite.javascript.change-import'}
  io.moderne.jsonrpc.JsonRpc$1.compute(JsonRpc.java:78)
  java.base/java.util.concurrent.RecursiveAction.exec(RecursiveAction.java:194)

when running recipes from rewrite-angular.

@greg-at-moderne greg-at-moderne marked this pull request as ready for review March 10, 2026 09:15
@jkschneider jkschneider merged commit edc6367 into main Mar 11, 2026
1 check passed
@jkschneider jkschneider deleted the greg-js-cross-module-recipe branch March 11, 2026 16:07
@github-project-automation github-project-automation bot moved this from In Progress to Done in OpenRewrite Mar 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants