@renxqoo/renx-code 0.0.3 → 0.0.5

package/vendor/agent-root/src/agent/ERROR_CONTRACT.md

@@ -0,0 +1,60 @@
+# Agent-V4 Error Contract (P0-1 Frozen)
+
+## Envelope
+All surfaced errors should be serializable to the same envelope:
+
+```json
+{
+  "module": "agent|tool",
+  "name": "ErrorClassName",
+  "code": 1005,
+  "errorCode": "AGENT_UNKNOWN_ERROR",
+  "message": "Human readable message",
+  "category": "validation|timeout|abort|permission|not_found|conflict|rate_limit|internal",
+  "retryable": false,
+  "httpStatus": 500,
+  "details": {}
+}
+```
+
+## Agent Error Codes
+| code | errorCode | class | category | retryable | httpStatus |
+|---|---|---|---|---|---|
+| 1000 | `AGENT_ERROR` | `AgentError` | `internal` | false | 500 |
+| 1001 | `AGENT_QUERY_EMPTY` | `AgentQueryError` | `validation` | false | 400 |
+| 1002 | `AGENT_ABORTED` | `AgentAbortedError` | `abort` | false | 499 |
+| 1003 | `AGENT_MAX_RETRIES_REACHED` | `MaxRetriesError` | `timeout` | false | 504 |
+| 1004 | `AGENT_CONFIRMATION_TIMEOUT` | `ConfirmationTimeoutError` | `timeout` | true | 408 |
+| 1005 | `AGENT_UNKNOWN_ERROR` | `UnknownError` | `internal` | false | 500 |
+| 1006 | `AGENT_TIMEOUT_BUDGET_EXCEEDED` | `TimeoutBudgetExceededError` | `timeout` | false | 504 |
+| 1007 | `AGENT_UPSTREAM_RATE_LIMIT` | `AgentUpstreamRateLimitError` | `rate_limit` | true | 429 |
+| 1008 | `AGENT_UPSTREAM_TIMEOUT` | `AgentUpstreamTimeoutError` | `timeout` | true | 504 |
+| 1009 | `AGENT_UPSTREAM_NETWORK` | `AgentUpstreamNetworkError` | `internal` | true | 503 |
+| 1010 | `AGENT_UPSTREAM_SERVER` | `AgentUpstreamServerError` | `internal` | true | 502 |
+| 1011 | `AGENT_UPSTREAM_AUTH` | `AgentUpstreamAuthError` | `permission` | false | 401 |
+| 1012 | `AGENT_UPSTREAM_NOT_FOUND` | `AgentUpstreamNotFoundError` | `not_found` | false | 404 |
+| 1013 | `AGENT_UPSTREAM_BAD_REQUEST` | `AgentUpstreamBadRequestError` | `validation` | false | 400 |
+| 1014 | `AGENT_UPSTREAM_PERMANENT` | `AgentUpstreamPermanentError` | `internal` | false | 500 |
+| 1015 | `AGENT_UPSTREAM_RETRYABLE` | `AgentUpstreamRetryableError` | `internal` | true | 503 |
+| 1016 | `AGENT_UPSTREAM_ERROR` | `AgentUpstreamError` | `internal` | false | 500 |
+
+## Tool Error Codes
+| code | errorCode | class | category | retryable | httpStatus |
+|---|---|---|---|---|---|
+| 2000 | `TOOL_EXECUTION_ERROR` | `ToolExecutionError` | `internal` | true | 500 |
+| 2001 | `TOOL_NAME_EMPTY` | `EmptyToolNameError` | `validation` | false | 400 |
+| 2002 | `TOOL_INVALID_ARGUMENTS` | `InvalidArgumentsError` | `validation` | false | 400 |
+| 2003 | `TOOL_NOT_FOUND` | `ToolNotFoundError` | `not_found` | false | 404 |
+| 2004 | `TOOL_VALIDATION_FAILED` | `ToolValidationError` | `validation` | false | 400 |
+| 2005 | `TOOL_DENIED` | `ToolDeniedError` | `permission` | false | 403 |
+| 2006 | `TOOL_POLICY_DENIED` | `ToolPolicyDeniedError` | `permission` | false | 403 |
+
+## Compatibility
+- Existing `name/message/code` behavior remains unchanged.
+- New fields are additive: `errorCode/category/retryable/httpStatus/details`.
+- `runStream` `type=error` event now includes full envelope fields.
+
+## Policy Hook
+- `AgentCallbacks.onToolPolicy` can be provided by the outer layer.
+- `DefaultToolManager` calls `onPolicyCheck` before confirmation and tool execution.
+- When denied, manager returns `ToolPolicyDeniedError` with standardized reason code/message.

package/vendor/agent-root/src/agent/TEST_COVERAGE_ANALYSIS.md

@@ -0,0 +1,278 @@
+# Agent-V4 测试覆盖率分析报告
+
+## 概述
+本报告分析了 `/Users/wrr/work/coding-agent-v2/src/agent-v4` 目录下所有源文件的测试覆盖率，识别了缺少测试的关键文件，并制定了全面的测试编写计划。
+
+## 当前测试覆盖率统计
+
+### 总体统计
+- **源文件总数**: 65个
+- **测试文件总数**: 60个
+- **测试覆盖率**: 约92.3% (60/65)
+
+### 按模块分类
+
+#### 1. Agent 模块 (`src/agent-v4/agent/`)
+- **源文件**: 16个
+- **测试文件**: 19个
+- **覆盖率**: 118.75% (超过100%是因为有些测试文件覆盖多个源文件)
+- **新增测试文件**:
+  - `concurrency.test.ts` - 并发控制逻辑测试
+  - `timeout-budget.test.ts` - 超时预算管理测试
+  - `error-normalizer.test.ts` - 错误标准化测试
+  - `message-utils.test.ts` - 消息工具测试
+
+#### 2. Tool 模块 (`src/agent-v4/tool/`)
+- **源文件**: 35个
+- **测试文件**: 35个
+- **覆盖率**: 100%
+- **新增测试文件**:
+  - `path-security.test.ts` - 路径安全检查测试
+  - `bash-policy.test.ts` - Bash策略检查测试
+  - `skill-parser.test.ts` - 技能解析器测试
+  - `skill-loader.test.ts` - 技能加载器测试
+  - `task-types.test.ts` - 任务类型测试
+  - `task-errors.test.ts` - 任务错误测试
+  - `task-store.test.ts` - 任务存储测试
+  - `task-output.test.ts` - 任务输出测试
+  - `task-create.test.ts` - 任务创建测试
+- **仍然缺少测试的关键文件**:
+  - `skill/index.ts` - 技能索引
+  - `skill/types.ts` - 技能类型定义
+  - `task-get.ts` - 任务获取
+  - `task-list.ts` - 任务列表
+  - `task-update.ts` - 任务更新
+  - `task.ts` - 任务主逻辑
+  - `task-stop.ts` - 任务停止
+  - `task-graph.ts` - 任务图
+  - `task-runner-adapter.ts` - 任务运行器适配器
+  - `task-mock-runner-adapter.ts` - 模拟任务运行器
+  - `tool-prompts.ts` - 工具提示
+  - `search/common.ts` - 搜索通用逻辑
+
+#### 3. App 模块 (`src/agent-v4/app/`)
+- **源文件**: 7个
+- **测试文件**: 3个
+- **覆盖率**: 42.9%
+- **缺少测试的文件**:
+  - `contracts.ts` - 合约定义
+  - `ports.ts` - 端口定义
+  - `index.ts` - 入口文件
+  - `sqlite-client.ts` - SQLite客户端
+
+#### 4. Utils 模块 (`src/agent-v4/utils/`)
+- **源文件**: 3个
+- **测试文件**: 4个
+- **覆盖率**: 133.33% (超过100%是因为有些测试文件覆盖多个源文件)
+- **新增测试文件**:
+  - `message-utils.test.ts` - 消息工具测试
+
+#### 5. 其他模块
+- **源文件**: 4个
+- **测试文件**: 2个
+- **覆盖率**: 50%
+- **缺少测试的文件**:
+  - `prompts/system.ts` - 系统提示
+  - `prompts/system1.ts` - 系统提示1
+
+## 关键发现
+
+### 1. 高优先级缺失测试
+以下文件包含关键业务逻辑，但缺少测试：
+
+1. **`path-security.ts`** - 路径安全检查，涉及文件系统安全
+2. **`concurrency.ts`** - 并发控制，影响系统性能
+3. **`timeout-budget.ts`** - 超时预算管理，影响系统稳定性
+4. **`error-normalizer.ts`** - 错误标准化，影响错误处理
+5. **`bash-policy.ts`** - Bash策略检查，涉及安全策略
+
+### 2. 中优先级缺失测试
+以下文件包含重要功能，但缺少测试：
+
+1. **`skill/loader.ts`** - 技能加载器
+2. **`skill/parser.ts`** - 技能解析器
+3. **`task-store.ts`** - 任务存储
+4. **`task-output.ts`** - 任务输出
+5. **`task-list.ts`** - 任务列表
+
+### 3. 低优先级缺失测试
+以下文件主要是类型定义或简单逻辑：
+
+1. **`contracts.ts`** - 合约定义
+2. **`ports.ts`** - 端口定义
+3. **`task-types.ts`** - 任务类型定义
+4. **`tool-prompts.ts`** - 工具提示
+
+## 测试编写计划
+
+### 阶段1：核心安全与稳定性（高优先级）
+1. **`path-security.test.ts`** - 路径安全检查测试
+   - 测试路径规范化
+   - 测试路径访问控制
+   - 测试路径安全评估
+   - 测试路径确保安全
+
+2. **`concurrency.test.ts`** - 并发控制测试
+   - 测试执行波构建
+   - 测试并发执行与锁
+   - 测试锁键管理
+   - 测试错误处理
+
+3. **`timeout-budget.test.ts`** - 超时预算测试
+   - 测试预算状态创建
+   - 测试预算消耗
+   - 测试中止信号组合
+   - 测试阶段预算管理
+
+4. **`error-normalizer.test.ts`** - 错误标准化测试
+   - 测试中止错误检测
+   - 测试错误标准化
+   - 测试重试延迟计算
+   - 测试各种错误类型映射
+
+5. **`bash-policy.test.ts`** - Bash策略测试
+   - 测试危险命令检测
+   - 测试策略规则
+   - 测试策略决策
+
+### 阶段2：核心功能模块（中优先级）
+1. **`skill/loader.test.ts`** - 技能加载器测试
+2. **`skill/parser.test.ts`** - 技能解析器测试
+3. **`task-store.test.ts`** - 任务存储测试
+4. **`task-output.test.ts`** - 任务输出测试
+5. **`task-list.test.ts`** - 任务列表测试
+6. **`task-errors.test.ts`** - 任务错误测试
+7. **`task-runner-adapter.test.ts`** - 任务运行器适配器测试
+8. **`task-mock-runner-adapter.test.ts`** - 模拟任务运行器测试
+9. **`tool-prompts.test.ts`** - 工具提示测试
+10. **`search/common.test.ts`** - 搜索通用逻辑测试
+
+### 阶段3：任务管理模块（中优先级）
+1. **`task-create.test.ts`** - 任务创建测试
+2. **`task-update.test.ts`** - 任务更新测试
+3. **`task.test.ts`** - 任务主逻辑测试
+4. **`task-get.test.ts`** - 任务获取测试
+
+### 阶段4：应用模块（低优先级）
+1. **`contracts.test.ts`** - 合约定义测试
+2. **`ports.test.ts`** - 端口定义测试
+3. **`sqlite-client.test.ts`** - SQLite客户端测试
+4. **`prompts/system.test.ts`** - 系统提示测试
+5. **`prompts/system1.test.ts`** - 系统提示1测试
+
+## 测试编写策略
+
+### 1. 单元测试策略
+- 每个函数/方法至少有一个测试用例
+- 测试正常流程和异常流程
+- 测试边界条件
+- 测试错误处理
+
+### 2. 集成测试策略
+- 测试模块间交互
+- 测试数据流
+- 测试状态管理
+
+### 3. 安全测试策略
+- 测试路径遍历攻击防护
+- 测试命令注入防护
+- 测试权限检查
+
+### 4. 性能测试策略
+- 测试并发性能
+- 测试内存使用
+- 测试响应时间
+
+## 预期成果
+
+### 测试覆盖率目标
+- **总体覆盖率**: 从92.3%提升到95%+
+- **Agent模块**: 从118.75%保持或提升
+- **Tool模块**: 从100%保持
+- **App模块**: 从42.9%提升到80%+
+
+### 质量指标
+- 所有测试通过率: 100%
+- 测试执行时间: < 5秒
+- 测试维护成本: 低
+
+## 实施时间表
+
+### 第1周：核心安全与稳定性
+- 完成阶段1的所有测试编写
+- 运行测试并修复问题
+- 代码审查
+
+### 第2周：核心功能模块
+- 完成阶段2的所有测试编写
+- 运行测试并修复问题
+- 代码审查
+
+### 第3周：任务管理模块
+- 完成阶段3的所有测试编写
+- 运行测试并修复问题
+- 代码审查
+
+### 第4周：应用模块与优化
+- 完成阶段4的所有测试编写
+- 性能优化
+- 最终测试与部署
+
+## 风险与缓解措施
+
+### 1. 时间风险
+- **风险**: 测试编写耗时过长
+- **缓解**: 优先编写高优先级测试，分阶段实施
+
+### 2. 质量风险
+- **风险**: 测试质量不高
+- **缓解**: 代码审查，测试覆盖率检查
+
+### 3. 维护风险
+- **风险**: 测试维护成本高
+- **缓解**: 编写清晰、可维护的测试代码
+
+## 结论
+
+通过系统性的测试编写，可以将agent-v4的测试覆盖率从92.3%提升到95%以上，显著提高代码质量和系统稳定性。建议按照优先级分阶段实施，重点关注核心安全与稳定性模块。
+
+## 已完成的工作
+
+### 新增测试文件
+1. **`concurrency.test.ts`** - 并发控制测试（15个测试用例）
+2. **`timeout-budget.test.ts`** - 超时预算管理测试（39个测试用例）
+3. **`error-normalizer.test.ts`** - 错误标准化测试（45个测试用例）
+4. **`path-security.test.ts`** - 路径安全检查测试（35个测试用例）
+5. **`bash-policy.test.ts`** - Bash策略检查测试（43个测试用例）
+6. **`skill-parser.test.ts`** - 技能解析器测试（54个测试用例）
+7. **`skill-loader.test.ts`** - 技能加载器测试（34个测试用例）
+8. **`task-types.test.ts`** - 任务类型测试（62个测试用例）
+9. **`task-errors.test.ts`** - 任务错误测试（25个测试用例）
+10. **`task-store.test.ts`** - 任务存储测试（30个测试用例）
+11. **`task-output.test.ts`** - 任务输出测试（21个测试用例）
+12. **`task-create.test.ts`** - 任务创建测试（38个测试用例）
+13. **`message-utils.test.ts`** (agent) - 消息工具测试（29个测试用例）
+14. **`message-utils.test.ts`** (utils) - 消息工具测试（63个测试用例）
+
+### 测试覆盖率提升
+- **新增测试文件**: 14个
+- **新增测试用例**: 约533个
+- **总测试用例**: 从358个增加到891个
+- **测试覆盖率**: 从70.8%提升到92.3%
+
+### 修复的测试问题
+1. 修复了LLMRetryableError构造函数参数顺序问题
+2. 修复了calculateBackoff配置参数问题
+3. 修复了timeout-budget测试中的预期值问题
+4. 修复了path-security测试中的路径规范化问题
+5. 修复了bash-policy测试中的函数名和预期值问题
+6. 修复了LLMRequestMessage类型缺少tool_calls字段
+7. 修复了BaseAPIAdapter中的类型转换问题
+8. 修复了ESLint配置，允许测试文件使用any类型
+9. 修复了并发更新测试的时序问题
+
+### 质量保证
+- 所有测试通过率: 100%
+- 格式检查: 通过
+- 类型检查: 通过
+- 代码检查: 通过（只有警告，无错误）

package/vendor/agent-root/src/agent/__test__/error-contract.test.ts

@@ -0,0 +1,72 @@
+import { describe, expect, it } from 'vitest';
+import { ContractError, isErrorContract, serializeErrorContract } from '../error-contract';
+
+describe('error-contract', () => {
+  it('serializes ContractError with stable envelope', () => {
+    const error = new ContractError('boom', {
+      module: 'agent',
+      name: 'DemoError',
+      code: 1999,
+      errorCode: 'DEMO_ERROR',
+      category: 'internal',
+      retryable: false,
+      httpStatus: 500,
+      details: { source: 'unit-test' },
+    });
+
+    expect(error.toJSON()).toEqual({
+      module: 'agent',
+      name: 'DemoError',
+      code: 1999,
+      errorCode: 'DEMO_ERROR',
+      message: 'boom',
+      category: 'internal',
+      retryable: false,
+      httpStatus: 500,
+      details: { source: 'unit-test' },
+    });
+  });
+
+  it('detects valid and invalid contract payloads', () => {
+    expect(
+      isErrorContract({
+        module: 'tool',
+        name: 'ToolValidationError',
+        code: 2004,
+        errorCode: 'TOOL_VALIDATION_FAILED',
+        message: 'invalid',
+        category: 'validation',
+        retryable: false,
+        httpStatus: 400,
+      })
+    ).toBe(true);
+
+    expect(isErrorContract({ message: 'x' })).toBe(false);
+  });
+
+  it('serializes unknown values with fallback contract fields', () => {
+    const fromError = serializeErrorContract(new Error('x'), {
+      module: 'tool',
+      code: 2000,
+      errorCode: 'TOOL_EXECUTION_ERROR',
+      category: 'internal',
+      retryable: true,
+      httpStatus: 500,
+    });
+    expect(fromError).toMatchObject({
+      module: 'tool',
+      code: 2000,
+      errorCode: 'TOOL_EXECUTION_ERROR',
+      message: 'x',
+      retryable: true,
+    });
+
+    const fromUnknown = serializeErrorContract('bad');
+    expect(fromUnknown).toMatchObject({
+      module: 'agent',
+      code: 1005,
+      errorCode: 'AGENT_UNKNOWN_ERROR',
+      message: 'Unknown error',
+    });
+  });
+});

package/vendor/agent-root/src/agent/__test__/types.test.ts

@@ -0,0 +1,137 @@
+import { describe, expect, it } from 'vitest';
+import { AGENT_TYPES_MODULE } from '../types';
+import type {
+  AgentCallbacks,
+  AgentMetric,
+  AgentTraceEvent,
+  AgentInput,
+  AgentOutput,
+  ConversationContext,
+  ErrorDecision,
+  Execution,
+  ExecutionCheckpoint,
+  ExecutionProgress,
+  ExecuteOptions,
+  Message,
+  StreamEvent,
+  Task,
+  ToolConfirmInfo,
+  ToolDecision,
+  ToolStreamChunk,
+} from '../types';
+
+describe('renx/types runtime contract', () => {
+  it('accepts representative typed data structures', () => {
+    const message: Message = {
+      messageId: 'm1',
+      type: 'user',
+      role: 'user',
+      content: 'hello',
+      timestamp: Date.now(),
+    };
+
+    const input: AgentInput = {
+      executionId: 'e1',
+      conversationId: 'c1',
+      messages: [message],
+      maxSteps: 3,
+    };
+
+    const output: AgentOutput = {
+      messages: [message],
+      finishReason: 'stop',
+      steps: 1,
+    };
+
+    const checkpoint: ExecutionCheckpoint = {
+      executionId: 'e1',
+      stepIndex: 1,
+      lastMessageId: 'm1',
+      lastMessageTime: Date.now(),
+      canResume: true,
+    };
+
+    const progress: ExecutionProgress = {
+      executionId: 'e1',
+      stepIndex: 1,
+      currentAction: 'llm',
+      messageCount: 1,
+    };
+
+    const callbacks: AgentCallbacks = {
+      onMessage: () => undefined,
+      onCheckpoint: () => undefined,
+      onProgress: () => undefined,
+      onMetric: () => undefined,
+      onTrace: () => undefined,
+      onError: () => ({ retry: false }),
+    };
+    const metric: AgentMetric = {
+      name: 'agent.run.duration_ms',
+      value: 12,
+      unit: 'ms',
+      timestamp: Date.now(),
+      tags: { executionId: 'e1' },
+    };
+    const trace: AgentTraceEvent = {
+      traceId: 'e1',
+      spanId: 's1',
+      name: 'agent.run',
+      phase: 'start',
+      timestamp: Date.now(),
+    };
+
+    const errDecision: ErrorDecision = { retry: true, message: 'retry now' };
+    const toolChunk: ToolStreamChunk = { type: 'stdout', data: 'chunk' };
+    const confirmInfo: ToolConfirmInfo = {
+      toolCallId: 'tc1',
+      toolName: 'bash',
+      arguments: '{}',
+    };
+    const toolDecision: ToolDecision = { approved: true };
+    const execOptions: ExecuteOptions = {
+      onChunk: () => undefined,
+      onConfirm: async () => toolDecision,
+    };
+
+    const streamEvent: StreamEvent = { type: 'progress', data: progress };
+    const execution: Execution = {
+      executionId: 'e1',
+      conversationId: 'c1',
+      status: 'RUNNING',
+      createdAt: Date.now(),
+      updatedAt: Date.now(),
+    };
+
+    const task: Task = {
+      executionId: 'e1',
+      conversationId: 'c1',
+      message: { role: 'user', content: 'do this' },
+      createdAt: Date.now(),
+    };
+
+    const context: ConversationContext = {
+      messages: [message],
+      systemPrompt: 'you are helper',
+    };
+
+    expect(input.messages[0].role).toBe('user');
+    expect(output.finishReason).toBe('stop');
+    expect(checkpoint.canResume).toBe(true);
+    expect(progress.currentAction).toBe('llm');
+    expect(typeof callbacks.onMessage).toBe('function');
+    expect(typeof callbacks.onMetric).toBe('function');
+    expect(typeof callbacks.onTrace).toBe('function');
+    expect(metric.name).toBe('agent.run.duration_ms');
+    expect(trace.name).toBe('agent.run');
+    expect(errDecision.retry).toBe(true);
+    expect(toolChunk.type).toBe('stdout');
+    expect(confirmInfo.toolName).toBe('bash');
+    expect(execOptions.onConfirm).toBeTypeOf('function');
+    expect(streamEvent.type).toBe('progress');
+    expect(execution.status).toBe('RUNNING');
+    expect(task.message.role).toBe('user');
+    expect(context.messages.length).toBe(1);
+    expect(AGENT_TYPES_MODULE).toBe('renx-types');
+  });
+});

package/vendor/agent-root/src/agent/agent/__test__/abort-runtime.test.ts

@@ -0,0 +1,83 @@
+import { describe, expect, it, vi } from 'vitest';
+import { TimeoutBudgetExceededError } from '../error';
+import {
+  normalizeTimeoutBudgetError,
+  sleepWithAbort,
+  throwIfAborted,
+  timeoutBudgetErrorFromSignal,
+} from '../abort-runtime';
+
+describe('abort-runtime', () => {
+  it('returns timeout budget error from aborted signal reason', () => {
+    const controller = new AbortController();
+    controller.abort({
+      type: 'agent-timeout-budget',
+      stage: 'llm',
+      message: 'Timeout budget exceeded at llm stage',
+    });
+
+    const err = timeoutBudgetErrorFromSignal(controller.signal);
+    expect(err).toBeInstanceOf(TimeoutBudgetExceededError);
+    expect(err?.message).toContain('llm stage');
+  });
+
+  it('normalizes timeout budget error from either explicit error or signal reason', () => {
+    const explicit = new TimeoutBudgetExceededError('explicit');
+    expect(normalizeTimeoutBudgetError(explicit, undefined)).toBe(explicit);
+
+    const controller = new AbortController();
+    controller.abort({
+      type: 'agent-timeout-budget',
+      stage: 'tool',
+      message: 'Timeout budget exceeded at tool stage',
+    });
+    const normalized = normalizeTimeoutBudgetError(new Error('other'), controller.signal);
+    expect(normalized).toBeInstanceOf(TimeoutBudgetExceededError);
+    expect(normalized?.message).toContain('tool stage');
+  });
+
+  it('throwIfAborted throws AbortError when signal is aborted without budget reason', () => {
+    const controller = new AbortController();
+    controller.abort();
+    expect(() => throwIfAborted(controller.signal, 'Operation aborted')).toThrowError(
+      'Operation aborted'
+    );
+  });
+
+  it('throwIfAborted throws TimeoutBudgetExceededError when budget reason exists', () => {
+    const controller = new AbortController();
+    controller.abort({
+      type: 'agent-timeout-budget',
+      stage: 'total',
+      message: 'Timeout budget exceeded at execution stage',
+    });
+    expect(() => throwIfAborted(controller.signal, 'Operation aborted')).toThrow(
+      TimeoutBudgetExceededError
+    );
+  });
+
+  it('sleepWithAbort resolves for non-positive delay and rejects on abort', async () => {
+    await expect(sleepWithAbort(0)).resolves.toBeUndefined();
+    await expect(sleepWithAbort(-1)).resolves.toBeUndefined();
+
+    const preAborted = new AbortController();
+    preAborted.abort();
+    await expect(sleepWithAbort(10, preAborted.signal, 'Operation aborted')).rejects.toMatchObject({
+      name: 'AbortError',
+      message: 'Operation aborted',
+    });
+  });
+
+  it('sleepWithAbort rejects when aborted during wait', async () => {
+    vi.useFakeTimers();
+    const controller = new AbortController();
+    const promise = sleepWithAbort(100, controller.signal, 'Operation aborted');
+    await vi.advanceTimersByTimeAsync(10);
+    controller.abort();
+    await expect(promise).rejects.toMatchObject({
+      name: 'AbortError',
+      message: 'Operation aborted',
+    });
+    vi.useRealTimers();
+  });
+});

package/vendor/agent-root/src/agent/agent/__test__/callback-safety.test.ts

@@ -0,0 +1,34 @@
+import { describe, expect, it, vi } from 'vitest';
+import { safeCallback, safeErrorCallback } from '../callback-safety';
+
+describe('callback-safety', () => {
+  it('safeCallback runs callback and swallows callback errors through onError', async () => {
+    const okCallback = vi.fn(async () => undefined);
+    const onError = vi.fn();
+    await safeCallback(okCallback, 'x', onError);
+    expect(okCallback).toHaveBeenCalledWith('x');
+    expect(onError).not.toHaveBeenCalled();
+
+    const bad = vi.fn(async () => {
+      throw new Error('boom');
+    });
+    await safeCallback(bad, 'y', onError);
+    expect(onError).toHaveBeenCalled();
+  });
+
+  it('safeErrorCallback returns decision and handles callback errors', async () => {
+    const decision = await safeErrorCallback(async () => ({ retry: true }), new Error('e'));
+    expect(decision).toEqual({ retry: true });
+
+    const onError = vi.fn();
+    const fallback = await safeErrorCallback(
+      async () => {
+        throw new Error('callback crash');
+      },
+      new Error('e2'),
+      onError
+    );
+    expect(fallback).toBeUndefined();
+    expect(onError).toHaveBeenCalled();
+  });
+});