ci: use test:ci to skip integration tests in CI #543

2026-05-26T15:08:36Z

xingyue commented

2026-05-26 15:08:36 +00:00

What

CI 里用 bun run test:ci 替代 bun run test

Why

@uncaged/workflow-agent-hermes 的 3 个 ACP 集成测试需要真实的 Hermes 进程，在 CI 容器里必定超时（每个 120s），导致每次 CI 都白等 6 分钟然后 fail。

Changes

.gitea/workflows/ci.yml: bun run test → bun run test:ci
test:ci 已在所有可测试的包里定义好了，唯一区别是 workflow-agent-hermes 的 test:ci 只跑 __tests__/*.test.ts，跳过 integration/ 目录

Ref

每次 CI failure 的日志都是这三个超时：

HermesAcpClient > connect() returns a UUID sessionId (120s timeout)
HermesAcpClient > prompt() returns a non-empty text response (120s timeout)
HermesAcpClient > prompt() can be called twice on the same session (resume) (120s timeout)

## What CI 里用 `bun run test:ci` 替代 `bun run test` ## Why `@uncaged/workflow-agent-hermes` 的 3 个 ACP 集成测试需要真实的 Hermes 进程，在 CI 容器里必定超时（每个 120s），导致每次 CI 都白等 6 分钟然后 fail。 ## Changes - `.gitea/workflows/ci.yml`: `bun run test` → `bun run test:ci` - `test:ci` 已在所有可测试的包里定义好了，唯一区别是 `workflow-agent-hermes` 的 `test:ci` 只跑 `__tests__/*.test.ts`，跳过 `integration/` 目录 ## Ref 每次 CI failure 的日志都是这三个超时： - `HermesAcpClient > connect() returns a UUID sessionId` (120s timeout) - `HermesAcpClient > prompt() returns a non-empty text response` (120s timeout) - `HermesAcpClient > prompt() can be called twice on the same session (resume)` (120s timeout)

xingyue added 1 commit 2026-05-26 15:08:36 +00:00

ci: use test:ci to skip integration tests in CI

CI / test (pull_request) Successful in 9m13s

Details

168e604602

The HermesAcpClient integration tests require a live Hermes agent
process and always timeout (3 × 120s) in CI containers, causing
every CI run to fail for ~6 minutes before reporting failure.

Switch from `bun run test` to `bun run test:ci` which was already
defined in all testable packages — workflow-agent-hermes's test:ci
runs only unit tests (__tests__/*.test.ts), skipping integration/.

xiaomo approved these changes 2026-05-26 15:26:01 +00:00

xiaomo left a comment

LGTM ✅ CI 跑集成测试白等 6 分钟确实没意义，test:ci 跳过需要真实 Hermes 的测试，合理。

LGTM ✅ CI 跑集成测试白等 6 分钟确实没意义，`test:ci` 跳过需要真实 Hermes 的测试，合理。

xiaomo merged commit aba5642908 into main

2026-05-26 15:26:02 +00:00

This repo is archived. You cannot comment on pull requests.

2 Participants

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: uncaged/workflow#543