Proposal: cwd support to sandbox class

[tlgimenes]

Hi folks,

I'm building an online bundler. The idea is that I'll spin a container to write files, npm install and bundle.

Currently, I'm creating a new folder for each build with the following code

const sandbox = getSandbox(c.SANDBOX, 'my-sandbox')

// copy files with sanbox.writeFile

// install deps and build
await sandbox.exec('bun', ['install', --cwd, "path/to/the/build/folder"])
await sandbox.exec('bun', ['build', --cwd, 'path/to/the/build/folder'])

This works great, however, I need to, somehow, have a way to cwd to a given folder for building so I can remove this --cwd to the bun command.

Is this the intended usage? Should I spin one container for each build instead or this folder approach is ok?

thanks!

[ItsWendell]

Since the resources available for a given instance are limited (bigger instances available after beta), I would recommend a single instance for each build.

You could pass a random uuid crypto.randomUuid() as an build id for - example for each build.

I would also recommend discussions here on GitHub or the Cloudflare Discord for questions like these.

https://discord.gg/cloudflaredev

[threepointone]

we already have lightweight support for sessions, we haven't exposed it correctly in the api just yet. so you should be able to call multipole commands in a row, including cd