Skip to content

JavaScript Type reading of cyclic types from Java -> TypeScript process#6126

Merged
jkschneider merged 13 commits intomainfrom
javascript-type-visiting
Oct 13, 2025
Merged

JavaScript Type reading of cyclic types from Java -> TypeScript process#6126
jkschneider merged 13 commits intomainfrom
javascript-type-visiting

Conversation

@jkschneider
Copy link
Copy Markdown
Member

@jkschneider jkschneider commented Oct 12, 2025

What's changed?

Working on solving the remaining issues with type transmission affecting Moderne CLI recipe runs.

The use of immer in JavaReceiver/JavaScriptReceiver in the typescript implementation failed on a "max call stack size exceeded" because of cyclic type attribution.

@github-project-automation github-project-automation Bot moved this to In Progress in OpenRewrite Oct 12, 2025
@jkschneider jkschneider marked this pull request as draft October 12, 2025 01:02
@jkschneider jkschneider force-pushed the javascript-type-visiting branch from d415e12 to 3a55054 Compare October 12, 2025 22:39
@jkschneider jkschneider force-pushed the javascript-type-visiting branch from 3adff3b to 3dbf671 Compare October 13, 2025 02:41
@jkschneider jkschneider marked this pull request as ready for review October 13, 2025 11:19
@jkschneider jkschneider changed the title Working on JavaScript Type RPC Working on JavaScript Type RPC Oct 13, 2025
@jkschneider jkschneider changed the title Working on JavaScript Type RPC JavaScript Type reading of cyclic types from Java -> Typescript process Oct 13, 2025
@jkschneider jkschneider changed the title JavaScript Type reading of cyclic types from Java -> Typescript process JavaScript Type reading of cyclic types from Java -> TypeScript process Oct 13, 2025
@jkschneider jkschneider merged commit 186c617 into main Oct 13, 2025
1 of 2 checks passed
@jkschneider jkschneider deleted the javascript-type-visiting branch October 13, 2025 11:50
@github-project-automation github-project-automation Bot moved this from In Progress to Done in OpenRewrite Oct 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

1 participant