@roj-ai/sdk 0.1.35 → 0.2.1

package/src/plugins/context-compact/history-offloader.test.ts

@@ -1,100 +0,0 @@
-import { afterAll, beforeAll, describe, expect, it } from 'bun:test'
-import { existsSync } from 'node:fs'
-import { mkdir, readFile, rm } from 'node:fs/promises'
-import { join } from 'node:path'
-import { AgentId } from '~/core/agents/schema.js'
-import { createNodeFileSystem } from '~/testing/node-platform.js'
-import { FileHistoryOffloader } from './history-offloader.js'
-
-// ============================================================================
-// Test Constants
-// ============================================================================
-
-const TEST_SESSION_DIR = '/tmp/roj-test-history-offloader'
-
-// ============================================================================
-// Test Setup
-// ============================================================================
-
-beforeAll(async () => {
-	await mkdir(TEST_SESSION_DIR, { recursive: true })
-})
-
-afterAll(async () => {
-	if (existsSync(TEST_SESSION_DIR)) {
-		await rm(TEST_SESSION_DIR, { recursive: true })
-	}
-})
-
-// ============================================================================
-// Tests
-// ============================================================================
-
-describe('FileHistoryOffloader', () => {
-	it('creates history file with content', async () => {
-		const offloader = new FileHistoryOffloader(TEST_SESSION_DIR, createNodeFileSystem())
-		const agentId = AgentId('test-agent-1')
-		const content = 'User: Hello\n\nAgent: Hi there!'
-
-		const path = await offloader.offload(agentId, content, '/session/.history/')
-
-		// Verify returned path
-		expect(path).toBe('/session/.history/test-agent-1/history.md')
-
-		// Verify file was created
-		const absolutePath = join(TEST_SESSION_DIR, '.history', agentId, 'history.md')
-		expect(existsSync(absolutePath)).toBe(true)
-
-		// Verify content
-		const fileContent = await readFile(absolutePath, 'utf-8')
-		expect(fileContent).toContain('## Summarized at')
-		expect(fileContent).toContain('User: Hello')
-		expect(fileContent).toContain('Agent: Hi there!')
-		expect(fileContent).toContain('---')
-	})
-
-	it('appends to existing history file', async () => {
-		const offloader = new FileHistoryOffloader(TEST_SESSION_DIR, createNodeFileSystem())
-		const agentId = AgentId('test-agent-2')
-
-		// First offload
-		await offloader.offload(agentId, 'First conversation', '/session/.history/')
-
-		// Second offload
-		await offloader.offload(agentId, 'Second conversation', '/session/.history/')
-
-		// Verify both are in the file
-		const absolutePath = join(TEST_SESSION_DIR, '.history', agentId, 'history.md')
-		const fileContent = await readFile(absolutePath, 'utf-8')
-
-		// Should contain both conversations
-		expect(fileContent).toContain('First conversation')
-		expect(fileContent).toContain('Second conversation')
-
-		// Should have two "Summarized at" headers
-		const summaryHeaders = fileContent.match(/## Summarized at/g)
-		expect(summaryHeaders?.length).toBe(2)
-	})
-
-	it('creates parent directories if they do not exist', async () => {
-		const offloader = new FileHistoryOffloader(TEST_SESSION_DIR, createNodeFileSystem())
-		const agentId = AgentId('test-agent-3')
-
-		// Use a deep path prefix
-		const path = await offloader.offload(agentId, 'Content', '/session/.deep/nested/history/')
-
-		expect(path).toBe('/session/.deep/nested/history/test-agent-3/history.md')
-
-		const absolutePath = join(TEST_SESSION_DIR, '.deep/nested/history', agentId, 'history.md')
-		expect(existsSync(absolutePath)).toBe(true)
-	})
-
-	it('handles path prefix without trailing slash', async () => {
-		const offloader = new FileHistoryOffloader(TEST_SESSION_DIR, createNodeFileSystem())
-		const agentId = AgentId('test-agent-4')
-
-		const path = await offloader.offload(agentId, 'Content', '/session/.history')
-
-		expect(path).toBe('/session/.history/test-agent-4/history.md')
-	})
-})

package/src/plugins/filesystem/filesystem.integration.test.ts

@@ -1,529 +0,0 @@
-import { afterAll, beforeAll, describe, expect, it } from 'bun:test'
-import * as fs from 'node:fs'
-import * as path from 'node:path'
-import { contentToString } from '~/core/llm/llm-log-types.js'
-import { MockLLMProvider } from '~/core/llm/mock.js'
-import { ToolCallId } from '~/core/tools/schema.js'
-import { createTestPreset, TestHarness } from '~/testing/index.js'
-import { filesystemPlugin } from './index.js'
-import type { FilesystemPresetConfig } from './plugin.js'
-
-// ============================================================================
-// Test fixtures
-// ============================================================================
-
-let fixtureDir: string
-
-beforeAll(() => {
-	fixtureDir = `/tmp/roj-fs-test-${Math.random().toString(36).slice(2)}`
-	fs.mkdirSync(fixtureDir, { recursive: true })
-
-	// Create test files
-	fs.writeFileSync(path.join(fixtureDir, 'hello.txt'), 'Hello, world!')
-	fs.writeFileSync(path.join(fixtureDir, 'multiline.txt'), Array.from({ length: 20 }, (_, i) => `Line ${i + 1}`).join('\n'))
-
-	// Create a minimal 1x1 PNG for image tests
-	const onePixelPng = Buffer.from(
-		'89504e470d0a1a0a0000000d49484452000000010000000108060000001f15c489000000'
-			+ '0a49444154789c6300010000000500010d0a2db40000000049454e44ae426082',
-		'hex',
-	)
-	fs.writeFileSync(path.join(fixtureDir, 'pixel.png'), onePixelPng)
-
-	// Create subdirectory with files
-	fs.mkdirSync(path.join(fixtureDir, 'subdir'), { recursive: true })
-	fs.writeFileSync(path.join(fixtureDir, 'subdir', 'nested.txt'), 'Nested content')
-
-	// Create a denied path directory
-	fs.mkdirSync(path.join(fixtureDir, '.events'), { recursive: true })
-	fs.writeFileSync(path.join(fixtureDir, '.events', 'secret.txt'), 'Should not be accessible')
-})
-
-afterAll(() => {
-	fs.rmSync(fixtureDir, { recursive: true, force: true })
-})
-
-// ============================================================================
-// Helpers
-// ============================================================================
-
-function createFsPreset(pluginConfig?: Partial<FilesystemPresetConfig>, overrides?: Parameters<typeof createTestPreset>[0]) {
-	const preset = createTestPreset({
-		...overrides,
-		plugins: [
-			filesystemPlugin.configure({
-				deniedPaths: ['.events'],
-				...pluginConfig,
-			}),
-			...(overrides?.plugins ?? []),
-		],
-	})
-	// Set workspaceDir so file paths within fixtureDir are allowed
-	preset.workspaceDir = fixtureDir
-	return preset
-}
-
-function createFsHarness(options: Omit<ConstructorParameters<typeof TestHarness>[0], 'systemPlugins'>) {
-	return new TestHarness(options)
-}
-
-// ============================================================================
-// Tests
-// ============================================================================
-
-describe('filesystem plugin', () => {
-	// =========================================================================
-	// read_file tool
-	// =========================================================================
-
-	describe('read_file tool', () => {
-		it('read existing file → correct content', async () => {
-			const filePath = path.join(fixtureDir, 'hello.txt')
-			const harness = createFsHarness({
-				presets: [createFsPreset()],
-				llmProvider: MockLLMProvider.withSequence([
-					{
-						toolCalls: [{
-							id: ToolCallId('tc1'),
-							name: 'read_file',
-							input: { path: filePath },
-						}],
-					},
-					{ content: 'Done', toolCalls: [] },
-				]),
-			})
-
-			const session = await harness.createSession('test')
-			await session.sendAndWaitForIdle('Read file')
-
-			const callHistory = harness.llmProvider.getCallHistory()
-			expect(callHistory).toHaveLength(2)
-			const toolMessages = callHistory[1].messages.filter((m) => m.role === 'tool')
-			expect(toolMessages).toHaveLength(1)
-			const result = JSON.parse(contentToString(toolMessages[0].content))
-			expect(result.content).toBe('Hello, world!')
-
-			await harness.shutdown()
-		})
-
-		it('read with offset and maxLines → correct slice', async () => {
-			const filePath = path.join(fixtureDir, 'multiline.txt')
-			const harness = createFsHarness({
-				presets: [createFsPreset()],
-				llmProvider: MockLLMProvider.withSequence([
-					{
-						toolCalls: [{
-							id: ToolCallId('tc1'),
-							name: 'read_file',
-							input: { path: filePath, offset: 2, maxLines: 3 },
-						}],
-					},
-					{ content: 'Done', toolCalls: [] },
-				]),
-			})
-
-			const session = await harness.createSession('test')
-			await session.sendAndWaitForIdle('Read file')
-
-			const callHistory = harness.llmProvider.getCallHistory()
-			const toolMessages = callHistory[1].messages.filter((m) => m.role === 'tool')
-			const result = JSON.parse(contentToString(toolMessages[0].content))
-			expect(result.content).toBe('Line 3\nLine 4\nLine 5')
-			expect(result.totalLines).toBe(20)
-
-			await harness.shutdown()
-		})
-
-		it('read non-existent file → error', async () => {
-			const filePath = path.join(fixtureDir, 'does-not-exist.txt')
-			const harness = createFsHarness({
-				presets: [createFsPreset()],
-				llmProvider: MockLLMProvider.withSequence([
-					{
-						toolCalls: [{
-							id: ToolCallId('tc1'),
-							name: 'read_file',
-							input: { path: filePath },
-						}],
-					},
-					{ content: 'Done', toolCalls: [] },
-				]),
-			})
-
-			const session = await harness.createSession('test')
-			await session.sendAndWaitForIdle('Read file')
-
-			const callHistory = harness.llmProvider.getCallHistory()
-			const toolMessages = callHistory[1].messages.filter((m) => m.role === 'tool')
-			expect(toolMessages).toHaveLength(1)
-			const content = toolMessages[0].content
-			expect(content).toContain('Not found')
-
-			await harness.shutdown()
-		})
-
-		it('read image file → file:// URL uses agent-visible input path, not resolved real path', async () => {
-			// Regression: previously read_file returned file://<realPath>, which the
-			// sandboxed FileStore then rejected when re-resolving on the next inference
-			// (it only accepts agent-visible paths like /home/user/session/...). The URL
-			// must echo input.path so it stays resolvable through fileStore.realPath().
-			const filePath = path.join(fixtureDir, 'pixel.png')
-			const harness = createFsHarness({
-				presets: [createFsPreset()],
-				llmProvider: MockLLMProvider.withSequence([
-					{
-						toolCalls: [{
-							id: ToolCallId('tc1'),
-							name: 'read_file',
-							input: { path: filePath },
-						}],
-					},
-					{ content: 'Done', toolCalls: [] },
-				]),
-			})
-
-			const session = await harness.createSession('test')
-			await session.sendAndWaitForIdle('Read image')
-
-			const callHistory = harness.llmProvider.getCallHistory()
-			const toolMessages = callHistory[1].messages.filter((m) => m.role === 'tool')
-			expect(toolMessages).toHaveLength(1)
-			const content = toolMessages[0].content
-			expect(Array.isArray(content)).toBe(true)
-			const blocks = content as Array<{ type: string; text?: string; imageUrl?: { url: string } }>
-			const imageBlock = blocks.find((b) => b.type === 'image_url')
-			expect(imageBlock).toBeDefined()
-			expect(imageBlock?.imageUrl?.url).toBe(`file://${filePath}`)
-
-			await harness.shutdown()
-		})
-
-		it('read a directory path → "is not a file" error', async () => {
-			const dirPath = path.join(fixtureDir, 'subdir')
-			const harness = createFsHarness({
-				presets: [createFsPreset()],
-				llmProvider: MockLLMProvider.withSequence([
-					{
-						toolCalls: [{
-							id: ToolCallId('tc1'),
-							name: 'read_file',
-							input: { path: dirPath },
-						}],
-					},
-					{ content: 'Done', toolCalls: [] },
-				]),
-			})
-
-			const session = await harness.createSession('test')
-			await session.sendAndWaitForIdle('Read file')
-
-			const callHistory = harness.llmProvider.getCallHistory()
-			const toolMessages = callHistory[1].messages.filter((m) => m.role === 'tool')
-			const content = toolMessages[0].content
-			expect(content).toContain('is not a file')
-
-			await harness.shutdown()
-		})
-	})
-
-	// =========================================================================
-	// write_file tool
-	// =========================================================================
-
-	describe('write_file tool', () => {
-		it('write file → file created with correct content', async () => {
-			const filePath = path.join(fixtureDir, 'written.txt')
-			const harness = createFsHarness({
-				presets: [createFsPreset()],
-				llmProvider: MockLLMProvider.withSequence([
-					{
-						toolCalls: [{
-							id: ToolCallId('tc1'),
-							name: 'write_file',
-							input: { path: filePath, content: 'Written content' },
-						}],
-					},
-					{ content: 'Done', toolCalls: [] },
-				]),
-			})
-
-			const session = await harness.createSession('test')
-			await session.sendAndWaitForIdle('Write file')
-
-			const content = fs.readFileSync(filePath, 'utf-8')
-			expect(content).toBe('Written content')
-
-			fs.unlinkSync(filePath)
-			await harness.shutdown()
-		})
-
-		it('write to nested path → parent directories created', async () => {
-			const filePath = path.join(fixtureDir, 'new-dir', 'deep', 'file.txt')
-			const harness = createFsHarness({
-				presets: [createFsPreset()],
-				llmProvider: MockLLMProvider.withSequence([
-					{
-						toolCalls: [{
-							id: ToolCallId('tc1'),
-							name: 'write_file',
-							input: { path: filePath, content: 'Deep content' },
-						}],
-					},
-					{ content: 'Done', toolCalls: [] },
-				]),
-			})
-
-			const session = await harness.createSession('test')
-			await session.sendAndWaitForIdle('Write file')
-
-			const content = fs.readFileSync(filePath, 'utf-8')
-			expect(content).toBe('Deep content')
-
-			fs.rmSync(path.join(fixtureDir, 'new-dir'), { recursive: true })
-			await harness.shutdown()
-		})
-
-		it('write to denied path → access denied', async () => {
-			const filePath = path.join(fixtureDir, '.events', 'blocked.txt')
-			const harness = createFsHarness({
-				presets: [createFsPreset()],
-				llmProvider: MockLLMProvider.withSequence([
-					{
-						toolCalls: [{
-							id: ToolCallId('tc1'),
-							name: 'write_file',
-							input: { path: filePath, content: 'Should fail' },
-						}],
-					},
-					{ content: 'Done', toolCalls: [] },
-				]),
-			})
-
-			const session = await harness.createSession('test')
-			await session.sendAndWaitForIdle('Write file')
-
-			const callHistory = harness.llmProvider.getCallHistory()
-			const toolMessages = callHistory[1].messages.filter((m) => m.role === 'tool')
-			const content = toolMessages[0].content
-			expect(content).toContain('denied')
-
-			await harness.shutdown()
-		})
-	})
-
-	// =========================================================================
-	// list_directory tool
-	// =========================================================================
-
-	describe('list_directory tool', () => {
-		it('list directory → returns files and subdirectories', async () => {
-			const harness = createFsHarness({
-				presets: [createFsPreset()],
-				llmProvider: MockLLMProvider.withSequence([
-					{
-						toolCalls: [{
-							id: ToolCallId('tc1'),
-							name: 'list_directory',
-							input: { path: fixtureDir, includeHidden: false },
-						}],
-					},
-					{ content: 'Done', toolCalls: [] },
-				]),
-			})
-
-			const session = await harness.createSession('test')
-			await session.sendAndWaitForIdle('List dir')
-
-			const callHistory = harness.llmProvider.getCallHistory()
-			const toolMessages = callHistory[1].messages.filter((m) => m.role === 'tool')
-			const content = toolMessages[0].content
-			expect(content).toContain('hello.txt')
-			expect(content).toContain('subdir')
-
-			await harness.shutdown()
-		})
-
-		it('recursive: true → nested entries included', async () => {
-			const harness = createFsHarness({
-				presets: [createFsPreset()],
-				llmProvider: MockLLMProvider.withSequence([
-					{
-						toolCalls: [{
-							id: ToolCallId('tc1'),
-							name: 'list_directory',
-							input: { path: fixtureDir, recursive: true, includeHidden: false },
-						}],
-					},
-					{ content: 'Done', toolCalls: [] },
-				]),
-			})
-
-			const session = await harness.createSession('test')
-			await session.sendAndWaitForIdle('List dir')
-
-			const callHistory = harness.llmProvider.getCallHistory()
-			const toolMessages = callHistory[1].messages.filter((m) => m.role === 'tool')
-			const content = toolMessages[0].content
-			expect(content).toContain('nested.txt')
-
-			await harness.shutdown()
-		})
-
-		it('list non-existent path → error', async () => {
-			const dirPath = path.join(fixtureDir, 'nonexistent-dir')
-			const harness = createFsHarness({
-				presets: [createFsPreset()],
-				llmProvider: MockLLMProvider.withSequence([
-					{
-						toolCalls: [{
-							id: ToolCallId('tc1'),
-							name: 'list_directory',
-							input: { path: dirPath },
-						}],
-					},
-					{ content: 'Done', toolCalls: [] },
-				]),
-			})
-
-			const session = await harness.createSession('test')
-			await session.sendAndWaitForIdle('List dir')
-
-			const callHistory = harness.llmProvider.getCallHistory()
-			const toolMessages = callHistory[1].messages.filter((m) => m.role === 'tool')
-			const content = toolMessages[0].content
-			expect(content).toContain('Not found')
-
-			await harness.shutdown()
-		})
-
-		it('list file path → "is not a directory" error', async () => {
-			const filePath = path.join(fixtureDir, 'hello.txt')
-			const harness = createFsHarness({
-				presets: [createFsPreset()],
-				llmProvider: MockLLMProvider.withSequence([
-					{
-						toolCalls: [{
-							id: ToolCallId('tc1'),
-							name: 'list_directory',
-							input: { path: filePath },
-						}],
-					},
-					{ content: 'Done', toolCalls: [] },
-				]),
-			})
-
-			const session = await harness.createSession('test')
-			await session.sendAndWaitForIdle('List dir')
-
-			const callHistory = harness.llmProvider.getCallHistory()
-			const toolMessages = callHistory[1].messages.filter((m) => m.role === 'tool')
-			const content = toolMessages[0].content
-			expect(content).toContain('is not a directory')
-
-			await harness.shutdown()
-		})
-	})
-
-	// =========================================================================
-	// Auto-truncation
-	// =========================================================================
-
-	describe('auto-truncation', () => {
-		it('large file without explicit range → auto-truncated with head+tail', async () => {
-			const largeFilePath = path.join(fixtureDir, 'large.txt')
-			const lines = Array.from({ length: 1000 }, (_, i) => `Line ${i + 1}: ${'x'.repeat(100)}`)
-			fs.writeFileSync(largeFilePath, lines.join('\n'))
-
-			const harness = createFsHarness({
-				presets: [createFsPreset({
-					maxTokens: 100,
-				})],
-				llmProvider: MockLLMProvider.withSequence([
-					{
-						toolCalls: [{
-							id: ToolCallId('tc1'),
-							name: 'read_file',
-							input: { path: largeFilePath },
-						}],
-					},
-					{ content: 'Done', toolCalls: [] },
-				]),
-			})
-
-			const session = await harness.createSession('test')
-			await session.sendAndWaitForIdle('Read file')
-
-			const callHistory = harness.llmProvider.getCallHistory()
-			const toolMessages = callHistory[1].messages.filter((m) => m.role === 'tool')
-			const result = JSON.parse(contentToString(toolMessages[0].content))
-			expect(result.truncated).toBe(true)
-			expect(result.content).toContain('truncated')
-			expect(result.content).toContain('Line 1:')
-			expect(result.totalLines).toBe(1000)
-
-			fs.unlinkSync(largeFilePath)
-			await harness.shutdown()
-		})
-	})
-
-	// =========================================================================
-	// Denied paths
-	// =========================================================================
-
-	describe('denied paths', () => {
-		it('read in denied path → access denied', async () => {
-			const filePath = path.join(fixtureDir, '.events', 'secret.txt')
-			const harness = createFsHarness({
-				presets: [createFsPreset()],
-				llmProvider: MockLLMProvider.withSequence([
-					{
-						toolCalls: [{
-							id: ToolCallId('tc1'),
-							name: 'read_file',
-							input: { path: filePath },
-						}],
-					},
-					{ content: 'Done', toolCalls: [] },
-				]),
-			})
-
-			const session = await harness.createSession('test')
-			await session.sendAndWaitForIdle('Read file')
-
-			const callHistory = harness.llmProvider.getCallHistory()
-			const toolMessages = callHistory[1].messages.filter((m) => m.role === 'tool')
-			const content = toolMessages[0].content
-			expect(content).toContain('denied')
-
-			await harness.shutdown()
-		})
-	})
-
-	// =========================================================================
-	// Disabled
-	// =========================================================================
-
-	describe('disabled', () => {
-		it('enabled: false at agent level → no filesystem tools', async () => {
-			const harness = createFsHarness({
-				presets: [createFsPreset(undefined, {
-					orchestratorPlugins: [
-						filesystemPlugin.configureAgent({ enabled: false }),
-					],
-				})],
-				llmProvider: MockLLMProvider.withFixedResponse({ content: 'Ok', toolCalls: [] }),
-			})
-
-			const session = await harness.createSession('test')
-			await session.sendAndWaitForIdle('Hi')
-
-			const lastRequest = harness.llmProvider.getLastRequest()
-			const toolNames = lastRequest?.tools?.map((t) => t.name) ?? []
-			expect(toolNames).not.toContain('read_file')
-			expect(toolNames).not.toContain('write_file')
-			expect(toolNames).not.toContain('list_directory')
-
-			await harness.shutdown()
-		})
-	})
-})

package/src/plugins/filesystem/schema.ts

@@ -1,6 +0,0 @@
-export interface DirectoryEntry {
-	name: string
-	path: string
-	type: 'file' | 'directory' | 'symlink' | 'other'
-	size?: number
-}