Skip to content

Conversation

mschwarzl
Copy link

Copy link

google-cla bot commented Aug 27, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

Copy link
Collaborator

@saelo saelo left a comment

Choose a reason for hiding this comment

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

Oh nice, thanks! Some comments about splitting the PR but otherwise this looks good

.gitignore Outdated
package-lock.json

# results dir and workerd corpus
workerd*
Copy link
Collaborator

Choose a reason for hiding this comment

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

If you really need an in-place directory for temporary files, it should have a fixed name and should only be ignored on the top level, e.g.

Suggested change
workerd*
/workerd

This will otherwise match any subdirectory that starts with workerd.

Copy link
Author

Choose a reason for hiding this comment

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

sorry this wasn't meant to be in here! reverted it!

@mschwarzl
Copy link
Author

mschwarzl commented Sep 9, 2025

Hi @saelo, @Liedtke,

thank you for the comments!
Did the split now such that the top-level await feature is in #527
and this PR only contains the workerd target. The changes in the REPRL main were reverted.

1. Follow the instructions [here](https://github.com/cloudflare/workerd/blob/main/README.md#getting-started)
2. Run the fuzzbuild.sh script in the workerd root directory to build workerd with the fuzzili configuration
3. Test if REPRL works:
`swift run REPRLRun <path-to-workerd> fuzzilli <path-to-capnp-config> --experimental`
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is it obvious what capnp-config is after doing the previous steps?

Copy link
Author

Choose a reason for hiding this comment

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

It is if you look through the spec but there'll be a dedicated config in the workerd repository once I get https://github.com/cloudflare/workerd/pulls merged, I can update the description here pointing to the exact file.

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