mirror of
https://github.com/anthropic-experimental/sandbox-runtime.git
synced 2026-07-04 11:04:32 +08:00
abb4d80442
* feat: support argv0 in RipgrepConfig
Adds an optional argv0 field to RipgrepConfig for invoking multicall
binaries that dispatch based on argv[0]. When set, spawn() is used
instead of execFile() since execFile doesn't support overriding argv[0].
The existing execFile code path is unchanged when argv0 is not provided.
Also bumps version to 0.0.40.
* refactor: unify ripGrep on spawn + stream/consumers
Drop execFile and use spawn for both argv0 and non-argv0 paths:
- text() from node:stream/consumers collects stdout/stderr as Promise<string>
- spawn({ timeout }) handles the 10s timeout natively
- Promise.all([stdout, stderr, close]) reads naturally as async/await
Drops the 20MB maxBuffer cap — the only caller scans for dangerous files
at bounded depth, so runaway output is not a realistic concern.
83 lines
2.6 KiB
TypeScript
83 lines
2.6 KiB
TypeScript
import { describe, it, expect } from 'bun:test'
|
|
import { writeFileSync, mkdtempSync, rmSync } from 'fs'
|
|
import { tmpdir } from 'os'
|
|
import { join } from 'path'
|
|
import { ripGrep } from '../../src/utils/ripgrep.js'
|
|
|
|
describe('ripGrep', () => {
|
|
it('finds matches with default config', async () => {
|
|
const dir = mkdtempSync(join(tmpdir(), 'rg-test-'))
|
|
try {
|
|
writeFileSync(join(dir, 'a.txt'), 'hello\nworld\nfoo')
|
|
const results = await ripGrep(
|
|
['-l', 'world'],
|
|
dir,
|
|
new AbortController().signal,
|
|
)
|
|
expect(results).toHaveLength(1)
|
|
expect(results[0]).toContain('a.txt')
|
|
} finally {
|
|
rmSync(dir, { recursive: true })
|
|
}
|
|
})
|
|
|
|
it('returns empty array on no matches (exit code 1)', async () => {
|
|
const dir = mkdtempSync(join(tmpdir(), 'rg-test-'))
|
|
try {
|
|
writeFileSync(join(dir, 'a.txt'), 'hello')
|
|
const results = await ripGrep(
|
|
['-l', 'nonexistent-pattern-xyz'],
|
|
dir,
|
|
new AbortController().signal,
|
|
)
|
|
expect(results).toEqual([])
|
|
} finally {
|
|
rmSync(dir, { recursive: true })
|
|
}
|
|
})
|
|
|
|
it('passes argv0 to spawn (multicall binary dispatch)', async () => {
|
|
// Spawn node with a script that echoes process.argv0 — shell scripts can't
|
|
// observe argv0 via $0 (shebang loader resets it), but native binaries can.
|
|
const dir = mkdtempSync(join(tmpdir(), 'rg-argv0-'))
|
|
try {
|
|
const script = join(dir, 'echo-argv0.cjs')
|
|
// ripGrep appends target as the last arg; ignore it and print argv0
|
|
writeFileSync(script, 'process.stdout.write(process.argv0)')
|
|
|
|
const results = await ripGrep([], dir, new AbortController().signal, {
|
|
command: process.execPath,
|
|
args: [script],
|
|
argv0: 'rg',
|
|
})
|
|
expect(results).toEqual(['rg'])
|
|
} finally {
|
|
rmSync(dir, { recursive: true })
|
|
}
|
|
})
|
|
|
|
it('uses execFile path when argv0 is not set', async () => {
|
|
const dir = mkdtempSync(join(tmpdir(), 'rg-noargv0-'))
|
|
try {
|
|
const script = join(dir, 'echo-argv0.cjs')
|
|
writeFileSync(script, 'process.stdout.write(process.argv0)')
|
|
|
|
const results = await ripGrep([], dir, new AbortController().signal, {
|
|
command: process.execPath,
|
|
args: [script],
|
|
})
|
|
// Without argv0 override, process.argv0 defaults to the executable path
|
|
expect(results[0]).not.toBe('rg')
|
|
expect(results[0]).toContain(process.execPath.split('/').pop())
|
|
} finally {
|
|
rmSync(dir, { recursive: true })
|
|
}
|
|
})
|
|
|
|
it('rejects on exit code > 1', () => {
|
|
expect(
|
|
ripGrep(['--invalid-flag-xyz'], '.', new AbortController().signal),
|
|
).rejects.toThrow(/ripgrep failed/)
|
|
})
|
|
})
|