Tianyi Cui
c4647a8609
Implements the execa Agent Note's four sub-changes:
- execa (root devDep + loader-smoke dep) replaces the hand-rolled
spawn-collect-timeout choreography in loader-smoke, apps/cli and
cli-demo/acp-demo built-bin e2e, lsp-local and code-runtime-worker
built-lib e2e, the tui pty-harness outer collector, the jsonrpc
keyless smoke, and crash-recovery's child spawn. Genuinely custom
parts stay custom: cli-demo's interrupt-on-marker, jsonrpc's
line-predicate protocol driving, crash-recovery's SIGKILL-at-failpoint.
The two loader-smoke /* v8 ignore */ OS-error branches are gone.
- llm-mock-server CLI tokenizes via node:util parseArgs; numeric
coercion/bounds/cross-option constraints stay manual; pinned
error-message tests updated to the parseArgs texts.
- both loadRootEnv copies in apps/web/tests are deleted: the owning
vitest configs (web unconditionally, snapshot in record mode)
already load the repo-root .env before these files run.
- the four poll loops (acp-snapshot harness waits + crash-recovery
waitForFile) ride vi.waitFor with explicit {interval, timeout}.
58 lines
2.5 KiB
TypeScript
58 lines
2.5 KiB
TypeScript
import { existsSync } from 'node:fs'
|
|
import { join } from 'node:path'
|
|
import { fileURLToPath } from 'node:url'
|
|
import { execa } from 'execa'
|
|
import { describe, expect, it } from 'vitest'
|
|
|
|
/**
|
|
* Keyless built-artifact smoke: plain Node imports the package by name through its exports map,
|
|
* then exercises type stripping, sibling `worker.cjs` loading, bindings, and logs. Unit tests use
|
|
* `src/worker.ts`; this pins the downstream `lib/index.js` path. It skips when `lib/` is absent,
|
|
* and CI runs it after the build.
|
|
*/
|
|
|
|
const pkgDir = fileURLToPath(new URL('..', import.meta.url))
|
|
const built = ['lib/index.js', 'lib/worker.cjs'].every(file => existsSync(join(pkgDir, file)))
|
|
&& existsSync(join(pkgDir, '../code-runtime/lib/index.js'))
|
|
|
|
describe.skipIf(!built)('built lib real load path (plain node)', () => {
|
|
it('runs a TypeScript program with a binding through lib/index.js and its lib/worker.cjs entry', async () => {
|
|
const script = `
|
|
const { Context } = await import('cordis')
|
|
const { WorkerCodeRuntime } = await import('@deepseek-ai/dsh-code-runtime-worker')
|
|
const ctx = new Context()
|
|
await ctx.plugin(WorkerCodeRuntime, {})
|
|
const result = await ctx.codeRuntime.run({
|
|
program: 'const doubled: number = await tools.double({ n: 21 }); console.log("halfway", doubled); let failure; try { await tools.fail({}) } catch (error) { failure = { typed: error instanceof ToolCallError, name: error.name, toolName: error.toolName, message: error.message } } return { doubled, failure };',
|
|
bindings: [{
|
|
global: 'tools',
|
|
functions: {
|
|
double: async args => args.n * 2,
|
|
fail: async () => { throw new Error('denied') },
|
|
},
|
|
errorClass: { name: 'ToolCallError', memberNameProperty: 'toolName' },
|
|
}],
|
|
})
|
|
console.log(JSON.stringify(result))
|
|
process.exit(0)
|
|
`
|
|
const { exitCode, stdout, stderr } = await execa(process.execPath, ['--input-type=module', '-e', script], {
|
|
cwd: pkgDir,
|
|
stdin: 'ignore',
|
|
timeout: 55_000,
|
|
killSignal: 'SIGKILL',
|
|
reject: false,
|
|
})
|
|
|
|
expect(exitCode, `stderr:\n${stderr}`).toBe(0)
|
|
const lastLine = stdout.trim().split('\n').at(-1) ?? ''
|
|
const result = JSON.parse(lastLine) as { value?: unknown; logs: string[]; error?: unknown }
|
|
expect(result.error).toBeUndefined()
|
|
expect(result.value).toEqual({
|
|
doubled: 42,
|
|
failure: { typed: true, name: 'ToolCallError', toolName: 'fail', message: 'denied' },
|
|
})
|
|
expect(result.logs).toContain('halfway 42')
|
|
})
|
|
})
|