Skip to content

refactor(java): share runtime core for concore and concoredocker#544

Merged
pradeeban merged 1 commit intoControlCore-Project:devfrom
avinxshKD:fix/java-runtime-shared-core
Apr 14, 2026
Merged

refactor(java): share runtime core for concore and concoredocker#544
pradeeban merged 1 commit intoControlCore-Project:devfrom
avinxshKD:fix/java-runtime-shared-core

Conversation

@avinxshKD
Copy link
Copy Markdown
Contributor

Moved the duplicated Java runtime logic into one shared core and kept concore/concoredocker as thin wrappers with the same public API, then added local-wrapper parity test coverage in CI.

Did this because the old setup kept making us patch the same behavior in multiple files and drift was getting expensive.

closes #542

@avinxshKD
Copy link
Copy Markdown
Contributor Author

@pradeeban Pls take a look,. Java support is in a much better place now, so this just cleans up duplicated runtime paths and should make future fixes cheaper and safer.

If you want anything adjusted, I can update it quickly :p

@avinxshKD
Copy link
Copy Markdown
Contributor Author

Took a bit longer than expected, but the runtime path is cleaner now and behavior is still aligned with existing wrappers/tests.

@pradeeban
Copy link
Copy Markdown
Member

@avinxshKD is our new CJO (Chief Java Officer). :)

@pradeeban pradeeban merged commit 530324d into ControlCore-Project:dev Apr 14, 2026
8 checks passed
@GREENRAT-K405
Copy link
Copy Markdown
Contributor

Great work @avinxshKD :D let's keep contributing like this and helping each other out!

@pradeeban
Copy link
Copy Markdown
Member

The way we go with CONTROL-CORE multi-language support is funny. Where do we end? Now, Java and Julia. Maybe next is Erlang, Rust, Cairo, and Mojo? :D

@avinxshKD
Copy link
Copy Markdown
Contributor Author

The way we go with CONTROL-CORE multi-language support is funny. Where do we end? Now, Java and Julia. Maybe next is Erlang, Rust, Cairo, and Mojo? :D

Haha fair enough, seems like CONTROL-CORE is slowly becoming a multilingual runtime museum :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants