2026-03-16 21:34:48 +00:00
|
|
|
/**
|
|
|
|
|
* @license
|
|
|
|
|
* Copyright 2026 Google LLC
|
|
|
|
|
* SPDX-License-Identifier: Apache-2.0
|
|
|
|
|
*/
|
|
|
|
|
|
2026-03-24 04:04:17 +00:00
|
|
|
import { describe, it, expect, vi, beforeEach } from 'vitest';
|
2026-03-16 21:34:48 +00:00
|
|
|
import { LinuxSandboxManager } from './LinuxSandboxManager.js';
|
|
|
|
|
import type { SandboxRequest } from '../../services/sandboxManager.js';
|
2026-03-24 04:04:17 +00:00
|
|
|
import fs from 'node:fs';
|
|
|
|
|
|
|
|
|
|
vi.mock('node:fs', async () => {
|
|
|
|
|
const actual = await vi.importActual<typeof import('node:fs')>('node:fs');
|
|
|
|
|
return {
|
|
|
|
|
...actual,
|
|
|
|
|
default: {
|
|
|
|
|
// @ts-expect-error - Property 'default' does not exist on type 'typeof import("node:fs")'
|
|
|
|
|
...actual.default,
|
|
|
|
|
existsSync: vi.fn(() => true),
|
|
|
|
|
realpathSync: vi.fn((p: string | Buffer) => p.toString()),
|
|
|
|
|
mkdirSync: vi.fn(),
|
|
|
|
|
openSync: vi.fn(),
|
|
|
|
|
closeSync: vi.fn(),
|
|
|
|
|
writeFileSync: vi.fn(),
|
|
|
|
|
},
|
|
|
|
|
existsSync: vi.fn(() => true),
|
|
|
|
|
realpathSync: vi.fn((p: string | Buffer) => p.toString()),
|
|
|
|
|
mkdirSync: vi.fn(),
|
|
|
|
|
openSync: vi.fn(),
|
|
|
|
|
closeSync: vi.fn(),
|
|
|
|
|
writeFileSync: vi.fn(),
|
|
|
|
|
};
|
|
|
|
|
});
|
2026-03-16 21:34:48 +00:00
|
|
|
|
|
|
|
|
describe('LinuxSandboxManager', () => {
|
|
|
|
|
const workspace = '/home/user/workspace';
|
2026-03-23 11:43:58 -04:00
|
|
|
let manager: LinuxSandboxManager;
|
2026-03-16 21:34:48 +00:00
|
|
|
|
2026-03-23 11:43:58 -04:00
|
|
|
beforeEach(() => {
|
2026-03-24 04:04:17 +00:00
|
|
|
vi.clearAllMocks();
|
|
|
|
|
vi.mocked(fs.existsSync).mockReturnValue(true);
|
|
|
|
|
vi.mocked(fs.realpathSync).mockImplementation((p) => p.toString());
|
2026-03-23 11:43:58 -04:00
|
|
|
manager = new LinuxSandboxManager({ workspace });
|
|
|
|
|
});
|
2026-03-16 21:34:48 +00:00
|
|
|
|
2026-03-23 11:43:58 -04:00
|
|
|
const getBwrapArgs = async (req: SandboxRequest) => {
|
2026-03-16 21:34:48 +00:00
|
|
|
const result = await manager.prepareCommand(req);
|
2026-03-17 20:29:13 +00:00
|
|
|
expect(result.program).toBe('sh');
|
|
|
|
|
expect(result.args[0]).toBe('-c');
|
|
|
|
|
expect(result.args[1]).toBe(
|
|
|
|
|
'bpf_path="$1"; shift; exec bwrap "$@" 9< "$bpf_path"',
|
|
|
|
|
);
|
|
|
|
|
expect(result.args[2]).toBe('_');
|
|
|
|
|
expect(result.args[3]).toMatch(/gemini-cli-seccomp-.*\.bpf$/);
|
2026-03-23 11:43:58 -04:00
|
|
|
return result.args.slice(4);
|
|
|
|
|
};
|
|
|
|
|
|
|
|
|
|
it('correctly outputs bwrap as the program with appropriate isolation flags', async () => {
|
|
|
|
|
const bwrapArgs = await getBwrapArgs({
|
|
|
|
|
command: 'ls',
|
|
|
|
|
args: ['-la'],
|
|
|
|
|
cwd: workspace,
|
|
|
|
|
env: {},
|
|
|
|
|
});
|
2026-03-17 20:29:13 +00:00
|
|
|
|
|
|
|
|
expect(bwrapArgs).toEqual([
|
2026-03-16 21:34:48 +00:00
|
|
|
'--unshare-all',
|
|
|
|
|
'--new-session',
|
|
|
|
|
'--die-with-parent',
|
|
|
|
|
'--ro-bind',
|
|
|
|
|
'/',
|
|
|
|
|
'/',
|
|
|
|
|
'--dev',
|
|
|
|
|
'/dev',
|
|
|
|
|
'--proc',
|
|
|
|
|
'/proc',
|
|
|
|
|
'--tmpfs',
|
|
|
|
|
'/tmp',
|
|
|
|
|
'--bind',
|
|
|
|
|
workspace,
|
|
|
|
|
workspace,
|
2026-03-24 04:04:17 +00:00
|
|
|
'--ro-bind',
|
|
|
|
|
`${workspace}/.gitignore`,
|
|
|
|
|
`${workspace}/.gitignore`,
|
|
|
|
|
'--ro-bind',
|
|
|
|
|
`${workspace}/.geminiignore`,
|
|
|
|
|
`${workspace}/.geminiignore`,
|
|
|
|
|
'--ro-bind',
|
|
|
|
|
`${workspace}/.git`,
|
|
|
|
|
`${workspace}/.git`,
|
2026-03-17 20:29:13 +00:00
|
|
|
'--seccomp',
|
|
|
|
|
'9',
|
2026-03-16 21:34:48 +00:00
|
|
|
'--',
|
|
|
|
|
'ls',
|
|
|
|
|
'-la',
|
|
|
|
|
]);
|
|
|
|
|
});
|
|
|
|
|
|
|
|
|
|
it('maps allowedPaths to bwrap binds', async () => {
|
2026-03-23 11:43:58 -04:00
|
|
|
const bwrapArgs = await getBwrapArgs({
|
2026-03-16 21:34:48 +00:00
|
|
|
command: 'node',
|
|
|
|
|
args: ['script.js'],
|
|
|
|
|
cwd: workspace,
|
|
|
|
|
env: {},
|
2026-03-23 11:43:58 -04:00
|
|
|
policy: {
|
|
|
|
|
allowedPaths: ['/tmp/cache', '/opt/tools', workspace],
|
|
|
|
|
},
|
|
|
|
|
});
|
2026-03-16 21:34:48 +00:00
|
|
|
|
2026-03-23 11:43:58 -04:00
|
|
|
// Verify the specific bindings were added correctly
|
|
|
|
|
const bindsIndex = bwrapArgs.indexOf('--seccomp');
|
|
|
|
|
const binds = bwrapArgs.slice(bwrapArgs.indexOf('--bind'), bindsIndex);
|
2026-03-16 21:34:48 +00:00
|
|
|
|
2026-03-23 11:43:58 -04:00
|
|
|
expect(binds).toEqual([
|
2026-03-16 21:34:48 +00:00
|
|
|
'--bind',
|
|
|
|
|
workspace,
|
|
|
|
|
workspace,
|
2026-03-24 04:04:17 +00:00
|
|
|
'--ro-bind',
|
|
|
|
|
`${workspace}/.gitignore`,
|
|
|
|
|
`${workspace}/.gitignore`,
|
|
|
|
|
'--ro-bind',
|
|
|
|
|
`${workspace}/.geminiignore`,
|
|
|
|
|
`${workspace}/.geminiignore`,
|
|
|
|
|
'--ro-bind',
|
|
|
|
|
`${workspace}/.git`,
|
|
|
|
|
`${workspace}/.git`,
|
2026-03-23 11:43:58 -04:00
|
|
|
'--bind-try',
|
2026-03-16 21:34:48 +00:00
|
|
|
'/tmp/cache',
|
|
|
|
|
'/tmp/cache',
|
2026-03-23 11:43:58 -04:00
|
|
|
'--bind-try',
|
2026-03-16 21:34:48 +00:00
|
|
|
'/opt/tools',
|
|
|
|
|
'/opt/tools',
|
|
|
|
|
]);
|
|
|
|
|
});
|
2026-03-23 11:43:58 -04:00
|
|
|
|
2026-03-24 04:04:17 +00:00
|
|
|
it('protects real paths of governance files if they are symlinks', async () => {
|
|
|
|
|
vi.mocked(fs.realpathSync).mockImplementation((p) => {
|
|
|
|
|
if (p.toString() === `${workspace}/.gitignore`)
|
|
|
|
|
return '/shared/global.gitignore';
|
|
|
|
|
return p.toString();
|
|
|
|
|
});
|
|
|
|
|
|
|
|
|
|
const bwrapArgs = await getBwrapArgs({
|
|
|
|
|
command: 'ls',
|
|
|
|
|
args: [],
|
|
|
|
|
cwd: workspace,
|
|
|
|
|
env: {},
|
|
|
|
|
});
|
|
|
|
|
|
|
|
|
|
expect(bwrapArgs).toContain('--ro-bind');
|
|
|
|
|
expect(bwrapArgs).toContain(`${workspace}/.gitignore`);
|
|
|
|
|
expect(bwrapArgs).toContain('/shared/global.gitignore');
|
|
|
|
|
|
|
|
|
|
// Check that both are bound
|
|
|
|
|
const gitignoreIndex = bwrapArgs.indexOf(`${workspace}/.gitignore`);
|
|
|
|
|
expect(bwrapArgs[gitignoreIndex - 1]).toBe('--ro-bind');
|
|
|
|
|
expect(bwrapArgs[gitignoreIndex + 1]).toBe(`${workspace}/.gitignore`);
|
|
|
|
|
|
|
|
|
|
const realGitignoreIndex = bwrapArgs.indexOf('/shared/global.gitignore');
|
|
|
|
|
expect(bwrapArgs[realGitignoreIndex - 1]).toBe('--ro-bind');
|
|
|
|
|
expect(bwrapArgs[realGitignoreIndex + 1]).toBe('/shared/global.gitignore');
|
|
|
|
|
});
|
|
|
|
|
|
|
|
|
|
it('touches governance files if they do not exist', async () => {
|
|
|
|
|
vi.mocked(fs.existsSync).mockReturnValue(false);
|
|
|
|
|
|
|
|
|
|
await getBwrapArgs({
|
|
|
|
|
command: 'ls',
|
|
|
|
|
args: [],
|
|
|
|
|
cwd: workspace,
|
|
|
|
|
env: {},
|
|
|
|
|
});
|
|
|
|
|
|
|
|
|
|
expect(fs.mkdirSync).toHaveBeenCalled();
|
|
|
|
|
expect(fs.openSync).toHaveBeenCalled();
|
|
|
|
|
});
|
|
|
|
|
|
2026-03-23 11:43:58 -04:00
|
|
|
it('should not bind the workspace twice even if it has a trailing slash in allowedPaths', async () => {
|
|
|
|
|
const bwrapArgs = await getBwrapArgs({
|
|
|
|
|
command: 'ls',
|
|
|
|
|
args: ['-la'],
|
|
|
|
|
cwd: workspace,
|
|
|
|
|
env: {},
|
|
|
|
|
policy: {
|
|
|
|
|
allowedPaths: [workspace + '/'],
|
|
|
|
|
},
|
|
|
|
|
});
|
|
|
|
|
|
|
|
|
|
const bindsIndex = bwrapArgs.indexOf('--seccomp');
|
|
|
|
|
const binds = bwrapArgs.slice(bwrapArgs.indexOf('--bind'), bindsIndex);
|
|
|
|
|
|
2026-03-24 04:04:17 +00:00
|
|
|
// Should only contain the primary workspace bind and governance files, not the second workspace bind with a trailing slash
|
|
|
|
|
expect(binds).toEqual([
|
|
|
|
|
'--bind',
|
|
|
|
|
workspace,
|
|
|
|
|
workspace,
|
|
|
|
|
'--ro-bind',
|
|
|
|
|
`${workspace}/.gitignore`,
|
|
|
|
|
`${workspace}/.gitignore`,
|
|
|
|
|
'--ro-bind',
|
|
|
|
|
`${workspace}/.geminiignore`,
|
|
|
|
|
`${workspace}/.geminiignore`,
|
|
|
|
|
'--ro-bind',
|
|
|
|
|
`${workspace}/.git`,
|
|
|
|
|
`${workspace}/.git`,
|
|
|
|
|
]);
|
2026-03-23 11:43:58 -04:00
|
|
|
});
|
2026-03-16 21:34:48 +00:00
|
|
|
});
|