Skip to content

Commit 69ff02c

Browse files
committed
chore: apply prettier
1 parent 7399fec commit 69ff02c

5 files changed

Lines changed: 21 additions & 36 deletions

File tree

README.md

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -33,18 +33,3 @@ Runnable examples live in [`examples/`](./examples/) and are indexed in [`exampl
3333

3434
- [Execbox Architecture Overview](./docs/architecture/README.md)
3535
- [QuickJS Pooling Baseline](./docs/performance/quickjs-pooling-baseline.md)
36-
37-
## Development
38-
39-
```bash
40-
npm install
41-
npm test
42-
npm run test:security
43-
npm run lint
44-
npm run build
45-
npm run typecheck
46-
npm run examples
47-
npm run benchmark:quickjs-pooling
48-
```
49-
50-
Use `npm run verify:isolated-vm` when working on the native executor package. The required CI security lane also runs `npm run test:isolated-vm` on Node 24 with `--no-node-snapshot`.

docs/performance/quickjs-pooling-baseline.md

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -16,24 +16,24 @@ As of `2026-04-06`, pooled mode is the default for both packages. The pooled col
1616

1717
### short-script-no-tools
1818

19-
| Executor | Ephemeral median / p95 | Pooled median / p95 | Median delta |
20-
| --- | --- | --- | --- |
21-
| `process` | `241.20ms / 252.79ms` | `1.98ms / 2.22ms` | `-99.2%` |
22-
| `worker` | `125.11ms / 170.48ms` | `1.78ms / 2.06ms` | `-98.6%` |
19+
| Executor | Ephemeral median / p95 | Pooled median / p95 | Median delta |
20+
| --------- | ---------------------- | ------------------- | ------------ |
21+
| `process` | `241.20ms / 252.79ms` | `1.98ms / 2.22ms` | `-99.2%` |
22+
| `worker` | `125.11ms / 170.48ms` | `1.78ms / 2.06ms` | `-98.6%` |
2323

2424
### short-script-one-tool
2525

26-
| Executor | Ephemeral median / p95 | Pooled median / p95 | Median delta |
27-
| --- | --- | --- | --- |
28-
| `process` | `247.01ms / 273.24ms` | `3.26ms / 3.87ms` | `-98.7%` |
29-
| `worker` | `128.04ms / 131.16ms` | `3.23ms / 3.79ms` | `-97.5%` |
26+
| Executor | Ephemeral median / p95 | Pooled median / p95 | Median delta |
27+
| --------- | ---------------------- | ------------------- | ------------ |
28+
| `process` | `247.01ms / 273.24ms` | `3.26ms / 3.87ms` | `-98.7%` |
29+
| `worker` | `128.04ms / 131.16ms` | `3.23ms / 3.79ms` | `-97.5%` |
3030

3131
### short-script-two-tools
3232

33-
| Executor | Ephemeral median / p95 | Pooled median / p95 | Median delta |
34-
| --- | --- | --- | --- |
35-
| `process` | `245.69ms / 256.12ms` | `4.44ms / 5.08ms` | `-98.2%` |
36-
| `worker` | `131.17ms / 136.03ms` | `4.24ms / 4.78ms` | `-96.8%` |
33+
| Executor | Ephemeral median / p95 | Pooled median / p95 | Median delta |
34+
| --------- | ---------------------- | ------------------- | ------------ |
35+
| `process` | `245.69ms / 256.12ms` | `4.44ms / 5.08ms` | `-98.2%` |
36+
| `worker` | `131.17ms / 136.03ms` | `4.24ms / 4.78ms` | `-96.8%` |
3737

3838
## Notes
3939

packages/core/test-support/runExecutorContractSuite.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -425,7 +425,7 @@ export function runExecutorContractSuite(
425425
if (supportsPooling) {
426426
it("does not leak guest global state across pooled executions", async () => {
427427
const executor = createExecutor({
428-
...( { pool: { maxSize: 1 } } as object),
428+
...({ pool: { maxSize: 1 } } as object),
429429
} as ExecutorContractOptions);
430430
await (
431431
executor as { prewarm?(count?: number): Promise<void> }

packages/process/src/processExecutor.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -106,9 +106,7 @@ function createProcessShell(): ProcessShell {
106106
function createProcessTransport(child: ChildProcess): HostTransport {
107107
let terminated = false;
108108
let closeReason: TransportCloseReason | undefined;
109-
const closeHandlers = new Set<
110-
(reason?: TransportCloseReason) => void
111-
>();
109+
const closeHandlers = new Set<(reason?: TransportCloseReason) => void>();
112110
const errorHandlers = new Set<(error: Error) => void>();
113111
const messageHandlers = new Set<(message: RunnerMessage) => void>();
114112

@@ -237,7 +235,9 @@ function resolvePoolOptions(
237235
}
238236

239237
function isReusableResult(result: ExecuteResult): boolean {
240-
return result.ok || !["internal_error", "timeout"].includes(result.error.code);
238+
return (
239+
result.ok || !["internal_error", "timeout"].includes(result.error.code)
240+
);
241241
}
242242

243243
/**

packages/worker/src/workerExecutor.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -75,9 +75,7 @@ function createBorrowedTransport(transport: HostTransport): HostTransport {
7575
function createWorkerTransport(worker: Worker): HostTransport {
7676
let terminated = false;
7777
let closeReason: TransportCloseReason | undefined;
78-
const closeHandlers = new Set<
79-
(reason?: TransportCloseReason) => void
80-
>();
78+
const closeHandlers = new Set<(reason?: TransportCloseReason) => void>();
8179
const errorHandlers = new Set<(error: Error) => void>();
8280
const messageHandlers = new Set<(message: RunnerMessage) => void>();
8381

@@ -195,7 +193,9 @@ function resolvePoolOptions(
195193
}
196194

197195
function isReusableResult(result: ExecuteResult): boolean {
198-
return result.ok || !["internal_error", "timeout"].includes(result.error.code);
196+
return (
197+
result.ok || !["internal_error", "timeout"].includes(result.error.code)
198+
);
199199
}
200200

201201
/**

0 commit comments

Comments
 (0)