Skip to content

Conversation

@EricWittmann
Copy link
Contributor

This pull request adds a new blog post to announce the new quickjs4j extension in the quarkiverse. Links to quickjs4j and the new extension:

@github-actions
Copy link

github-actions bot commented Jul 14, 2025

🙈 The PR is closed and the preview is expired.

Copy link
Member

@cescoffier cescoffier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome!

I've made a few comment - just one blocking (as it messed up the rendering).

@cescoffier
Copy link
Member

Can we add a bit more content to the last three sections? At the moment, there are just a list of bullet points.

@cescoffier
Copy link
Member

Also, please update the date of publication.

@cescoffier
Copy link
Member

For my own understanding, are you creating one JS context per call?

What if you want to load JavaScript libraries?

@EricWittmann
Copy link
Contributor Author

For my own understanding, are you creating one JS context per call?

What if you want to load JavaScript libraries?

Currently yes, there is a single JS context per call. I've added a little bit of information to blog post in the "Future Plans" section about this. There are optimizations that will need to happen. Perhaps a discussion with Quarkus experts will be needed to really nail down the integration. I've focused mostly on the DevEx in this phase. The result is, I think, a really easy way to integrate JS functionality into a Quarkus app. But it's a pretty naive approach, without significant time spent optimizing.

As for javascript libraries, you can use whatever libraries you want (provided they don't use Browser or Node.js APIs that cannot be shimmed). You can even use typescript if you want. The idea is you could have a standard JS build pipeline for your script, with a bundler that produced a final .js file that would get loaded by this feature.

Quickjs4j even outputs a .mjs file for the context functions, so you can have some code completion and proper validation in your IDE.

@EricWittmann
Copy link
Contributor Author

@cescoffier Updated based on feedback. Thanks for the review.

@cescoffier cescoffier merged commit 1336c8c into quarkusio:main Jul 15, 2025
1 check passed
@cescoffier
Copy link
Member

Merged!

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.

2 participants