@renxqoo/renx-code 0.0.8 → 0.0.10

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

@@ -1,60 +0,0 @@
-# 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

@@ -1,278 +0,0 @@
-# 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

@@ -1,72 +0,0 @@
-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

@@ -1,137 +0,0 @@
-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

@@ -1,83 +0,0 @@
-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

@@ -1,34 +0,0 @@
-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();
-  });
-});