@roj-ai/sdk 0.2.18 → 0.2.20

package/src/core/image/types.ts

@@ -9,6 +9,11 @@ export interface ImageResizeOptions {
 	signal?: AbortSignal
 }
 
+/**
+ * Paths are on the host's `FileSystem`: the caller reads the result through it and unlinks
+ * `tempFile` there once the image is encoded. When an image cannot be shrunk, return the original
+ * path: the LLM path treats a throw as an unreadable image.
+ */
 export interface ImageResizer {
 	resize(filePath: string, mimeType: string, options?: ImageResizeOptions): Promise<ImageResizeResult>
 }
@@ -16,6 +21,7 @@ export interface ImageResizer {
 export interface ImageResizeResult {
 	path: string
 	mimeType: string
+	/** A file the resizer created; the caller owns it from here and deletes it. */
 	tempFile?: string
 }
 

package/src/core/sessions/session-manager.ts

@@ -83,6 +83,7 @@ export interface SessionManagerOptions {
 	onUserOutput?: UserOutputCallback
 	preprocessorRegistry?: PreprocessorRegistry
 	uploadArchiveLimits?: ArchiveLimitOverrides
+	uploadMaxFileSize?: number
 	resourceArchiveLimits?: ArchiveLimitOverrides
 	llmLogger?: LLMLogger
 	portPool?: PortPool
@@ -142,6 +143,7 @@ export class SessionManager {
 	private readonly onUserOutput?: UserOutputCallback
 	private readonly preprocessorRegistry?: PreprocessorRegistry
 	private readonly uploadArchiveLimits?: ArchiveLimitOverrides
+	private readonly uploadMaxFileSize?: number
 	private readonly resourceArchiveLimits?: ArchiveLimitOverrides
 	private readonly llmLogger?: LLMLogger
 	private readonly portPool?: PortPool
@@ -171,6 +173,7 @@ export class SessionManager {
 		this.onUserOutput = options.onUserOutput
 		this.preprocessorRegistry = options.preprocessorRegistry
 		this.uploadArchiveLimits = options.uploadArchiveLimits
+		this.uploadMaxFileSize = options.uploadMaxFileSize
 		this.resourceArchiveLimits = options.resourceArchiveLimits
 		this.llmLogger = options.llmLogger
 		this.portPool = options.portPool
@@ -1079,6 +1082,7 @@ export class SessionManager {
 			dataFileStore: this.dataFileStore,
 			preprocessorRegistry: this.preprocessorRegistry,
 			archiveLimits: this.uploadArchiveLimits,
+			maxFileSize: this.uploadMaxFileSize,
 		})
 
 		if (this.resourceArchiveLimits) {

package/src/core/system.ts

@@ -121,6 +121,7 @@ export interface CreateSystemOptions<TPlugins extends readonly PluginDefinition<
 	onUserOutput?: UserOutputCallback
 	preprocessorRegistry?: PreprocessorRegistry
 	uploadArchiveLimits?: ArchiveLimitOverrides
+	uploadMaxFileSize?: number
 	resourceArchiveLimits?: ArchiveLimitOverrides
 	llmLogger?: LLMLogger
 	portPool?: PortPool
@@ -155,6 +156,7 @@ export function createSystem<const TPlugins extends readonly PluginDefinition<st
 		onUserOutput,
 		preprocessorRegistry,
 		uploadArchiveLimits,
+		uploadMaxFileSize,
 		resourceArchiveLimits,
 		llmLogger,
 		portPool,
@@ -181,6 +183,7 @@ export function createSystem<const TPlugins extends readonly PluginDefinition<st
 		onUserOutput,
 		preprocessorRegistry,
 		uploadArchiveLimits,
+		uploadMaxFileSize,
 		resourceArchiveLimits,
 		llmLogger,
 		portPool,

package/src/index.ts

@@ -196,6 +196,9 @@ export type {
 	ExecFileResult,
 	FileHandle,
 	FileSystem,
+	ImageResizeOptions,
+	ImageResizer,
+	ImageResizeResult,
 	Platform,
 	ProcessRunner,
 	SpawnOptions,

package/src/platform/index.ts

@@ -6,6 +6,7 @@
  * runtime packages (e.g. `@roj-ai/sdk/bun-platform`) and wired in at bootstrap time.
  */
 
+import type { ImageResizer } from '../core/image/types.js'
 import type { FileSystem } from './fs.js'
 import type { FsRevision } from './fs-revision.js'
 import type { GitClient } from './git.js'
@@ -39,6 +40,7 @@ export type { LiveScheduler, Scheduler, WakeHandler } from './scheduler.js'
 export type { SessionLogPage, SessionLogStore } from './session-log.js'
 export type { ShellConfinement, ShellGrant, ShellLimits, ShellRunner, ShellRunOptions, ShellRunResult } from './shell.js'
 
+export type { ImageResizeOptions, ImageResizer, ImageResizeResult } from '../core/image/types.js'
 export type { ServiceExecutorPort } from '../plugins/services/service.js'
 export type { PortPool } from '../plugins/services/port-pool.js'
 export type { ServiceConfig, ServiceStatus, ServiceStopSource } from '../plugins/services/schema.js'
@@ -77,6 +79,11 @@ export interface Platform {
 	 * writing `sessions/<id>/calls/<callId>.json` and listing that directory.
 	 */
 	llmCallLog?: LLMCallStore
+	/**
+	 * Shrinks images before they reach a model. Absent on hosts with a process table,
+	 * which keep resizing with the `vipsheader` / `vipsthumbnail` binaries.
+	 */
+	imageResizer?: ImageResizer
 	/** Absolute path to the OS temp directory (equivalent to `os.tmpdir()` on Node/Bun). */
 	tmpDir: string
 }

package/src/plugins/limits-guard/plugin.ts

@@ -122,18 +122,11 @@ export type BudgetExceededEvent = (typeof limitsEvents)['Events']['budget_exceed
 // ============================================================================
 
 /**
- * Resolve tool name from agent state for tool_completed/tool_failed events.
- * Must run BEFORE core reducer (which clears executingToolCall).
+ * Name of the tool behind a tool_completed/tool_failed event. Plugin reducers run after the
+ * core reducer, which has already cleared the running call and recorded the named result.
  */
-function resolveToolName(
-	sessionState: { agents: Map<AgentId, { executingToolCall?: { toolName: string }; pendingToolCalls: Array<{ id: string; name: string }> }> },
-	agentId: AgentId,
-	toolCallId: string,
-): string {
-	const agent = sessionState.agents.get(agentId)
-	if (!agent) return 'unknown'
-	return agent.executingToolCall?.toolName
-		?? agent.pendingToolCalls.find((tc) => tc.id === toolCallId)?.name
+function resolveToolName(sessionState: SessionState, agentId: AgentId, toolCallId: ToolCallId): string {
+	return sessionState.agents.get(agentId)?.pendingToolResults.find((result) => result.toolCallId === toolCallId)?.toolName
 		?? 'unknown'
 }
 
@@ -279,7 +272,6 @@ export const limitsGuardPlugin = definePlugin('limits-guard')
 					const counters = limits.get(event.agentId)
 					if (!counters) return limits
 
-					// Resolve tool name from agent state (must run before core clears executingToolCall)
 					const toolName = resolveToolName(sessionState, event.agentId, event.toolCallId)
 					const { [toolName]: _, ...restFailures } = counters.consecutiveToolFailures
 
@@ -296,7 +288,6 @@ export const limitsGuardPlugin = definePlugin('limits-guard')
 					const counters = limits.get(event.agentId)
 					if (!counters) return limits
 
-					// Resolve tool name from agent state (must run before core clears executingToolCall)
 					const toolName = resolveToolName(sessionState, event.agentId, event.toolCallId)
 					const currentEntry = counters.consecutiveToolFailures[toolName]
 

package/src/plugins/uploads/plugin.ts

@@ -31,7 +31,7 @@ const statusChangedSchema = z.object({
 // Constants
 // ============================================================================
 
-const MAX_FILE_SIZE = 10 * 1024 * 1024 // 10MB
+const DEFAULT_MAX_FILE_SIZE = 10 * 1024 * 1024 // 10MB
 
 /** Allowed MIME type patterns */
 const ALLOWED_MIME_TYPES = [
@@ -68,6 +68,8 @@ const uploadStorageError = (): DomainError => createDomainError('upload_storage_
 export interface UploadsPluginConfig {
 	dataFileStore: FileStore
 	preprocessorRegistry?: PreprocessorRegistry
+	/** Largest accepted file in bytes. Defaults to 10 MB. */
+	maxFileSize?: number
 	processingTimeoutMs?: number
 	processingAbortGraceMs?: number
 	/** Bound in ms on the close-time wait for in-flight uploads and mutations. Must be positive; anything else uses the 30s default. */
@@ -181,9 +183,9 @@ function toStoredFilename(filename: string): string {
 	return filename.normalize('NFC')
 }
 
-function validateUploadInput(input: Pick<UploadInput, 'size' | 'mimeType' | 'filename'>): string | undefined {
+function validateUploadInput(input: Pick<UploadInput, 'size' | 'mimeType' | 'filename'>, maxFileSize: number): string | undefined {
 	if (!isBasename(input.filename)) return 'Invalid filename'
-	if (input.size > MAX_FILE_SIZE) return `File too large: max ${MAX_FILE_SIZE / (1024 * 1024)}MB`
+	if (input.size > maxFileSize) return `File too large: max ${maxFileSize / (1024 * 1024)}MB`
 	if (!isAllowedMimeType(input.mimeType)) return `Unsupported file type: ${input.mimeType}`
 	return undefined
 }
@@ -290,11 +292,12 @@ function runTrackedMutation<T>(args: {
 async function prepareUpload(args: {
 	pluginContext: UploadsPluginContext
 	dataFileStore: FileStore
+	maxFileSize: number
 	logger: Logger
 	input: UploadInput
 }): Promise<Result<PreparedUpload, DomainError>> {
 	const filename = toStoredFilename(args.input.filename)
-	const validationError = validateUploadInput({ ...args.input, filename })
+	const validationError = validateUploadInput({ ...args.input, filename }, args.maxFileSize)
 	if (validationError) return Err(ValidationErrors.invalid(validationError))
 
 	const uploadId = generateUploadId()
@@ -1013,6 +1016,7 @@ export const uploadsPlugin = definePlugin('uploads')
 			const preparedResult = await prepareUpload({
 				pluginContext: ctx.pluginContext,
 				dataFileStore: ctx.pluginConfig.dataFileStore,
+				maxFileSize: ctx.pluginConfig.maxFileSize ?? DEFAULT_MAX_FILE_SIZE,
 				logger: ctx.logger,
 				input,
 			})
@@ -1076,6 +1080,7 @@ export const uploadsPlugin = definePlugin('uploads')
 			const preparedResult = await prepareUpload({
 				pluginContext: ctx.pluginContext,
 				dataFileStore: ctx.pluginConfig.dataFileStore,
+				maxFileSize: ctx.pluginConfig.maxFileSize ?? DEFAULT_MAX_FILE_SIZE,
 				logger: ctx.logger,
 				input,
 			})

package/src/testing/test-harness.ts

@@ -58,6 +58,7 @@ export class TestHarness {
 		eventStore?: MemoryEventStore
 		/** Resident session idle timeout. Zero or absent disables eviction. */
 		sessionIdleTimeoutMs?: number
+		uploadMaxFileSize?: number
 		/** Extra notification sink, run after the recorder. Throwing here simulates a hostile embedder. */
 		onUserOutput?: (notification: PluginNotification) => void
 	}) {
@@ -116,6 +117,7 @@ export class TestHarness {
 			llmLogger: options.llmLogger,
 			platform,
 			sessionIdleTimeoutMs: options.sessionIdleTimeoutMs,
+			uploadMaxFileSize: options.uploadMaxFileSize,
 			systemPlugins: mergeSystemPlugins([...fullPlugins, ...(options.systemPlugins ?? [])]),
 		})
 	}