Skip to content

Conversation

@jakubno
Copy link
Member

@jakubno jakubno commented Aug 29, 2025

Setup linting and formatting in all packages

@jakubno jakubno force-pushed the require-lint branch 4 times, most recently from 06b06d8 to f5a7417 Compare August 31, 2025 15:44
@jakubno jakubno changed the title Require lint Add pipeline for linting and formatting Aug 31, 2025
@jakubno jakubno marked this pull request as ready for review August 31, 2025 17:36
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Bug: Example File Uses Unavailable Template ID

The example.mts file was updated to use a hardcoded template ID 'bwyvo5fk343pbvxst536' and beta functionality. This ID likely belongs to an internal test template, making the example unreliable for general users without access to it. This change also seems unrelated to the commit's stated purpose.

js/example.mts#L1-L35

import { config } from 'dotenv'
import { Sandbox } from './dist'
function log(...args: any[]) {
console.log(...args)
}
config()
const sbx = await Sandbox.create('bwyvo5fk343pbvxst536')
log('ℹ️ sandbox created', sbx.sandboxId)
await sbx.runCode('x = 1')
log('Sandbox code executed')
const sandboxId = await sbx.betaPause()
log('Sandbox paused', sandboxId)
// Resume the sandbox from the same state
const sameSbx = await Sandbox.connect(sbx.sandboxId)
log('Sandbox resumed', sameSbx.sandboxId)
const execution = await sameSbx.runCode('x+=1; x')
// Output result
log(execution.text)
log(execution.error)
if (execution.text !== '2') {
log('Test failed:', 'Failed to resume sandbox')
throw new Error('Failed to resume sandbox')
}
log('Sandbox resumed successfully')
await sbx.kill()
log('Sandbox deleted')

Fix in Cursor Fix in Web


Comment @cursor review or bugbot run to trigger another review on this PR

@jakubno jakubno merged commit 09efac2 into main Aug 31, 2025
9 checks passed
@jakubno jakubno deleted the require-lint branch August 31, 2025 19:01
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