Add RunDetail.taskDag fixtures to worker tests #3

Merged

florianbeisel merged 1 commit from codex/fix-ci-pipeline into main

2026-03-30 11:38:14 +02:00

florianbeisel commented

2026-03-30 02:55:56 +02:00

(Migrated from github.com)

Motivation

The RunDetail contract now requires a taskDag field, so worker test fixtures must include a matching DAG structure to satisfy type checks and runtime expectations.

Description

Added an empty taskDag graph to apps/worker/test/runtime.test.ts and a single-node taskDag (matching the seeded validation task) to apps/worker/test/validation-runner.test.ts so fixtures conform to packages/contracts runDetailSchema.

Testing

Ran corepack pnpm ci:typecheck (success) and corepack pnpm --filter @codex-swarm/worker test (success), while the full corepack pnpm ci:test run still fails due to unrelated API test issues (a timeout in test/app.test.ts and a defaultBranch assertion in test/repository-provider.test.ts).

Codex Task

### Motivation - The `RunDetail` contract now requires a `taskDag` field, so worker test fixtures must include a matching DAG structure to satisfy type checks and runtime expectations. ### Description - Added an empty `taskDag` graph to `apps/worker/test/runtime.test.ts` and a single-node `taskDag` (matching the seeded validation task) to `apps/worker/test/validation-runner.test.ts` so fixtures conform to `packages/contracts` `runDetailSchema`. ### Testing - Ran `corepack pnpm ci:typecheck` (success) and `corepack pnpm --filter @codex-swarm/worker test` (success), while the full `corepack pnpm ci:test` run still fails due to unrelated API test issues (a timeout in `test/app.test.ts` and a `defaultBranch` assertion in `test/repository-provider.test.ts`). ------ [Codex Task](https://chatgpt.com/codex/tasks/task_e_69c9bc674928832aa31348b06d7254e6)

chatgpt-codex-connector[bot] commented

2026-03-30 02:58:07 +02:00

(Migrated from github.com)

Summary

Added an explicit regression assertion in the worker runtime test to verify the mocked RunDetail.taskDag remains the expected empty graph shape (nodes, edges, rootTaskIds, blockedTaskIds, unblockPaths). apps/worker/test/runtime.test.tsL428-L442
Added a focused validation-runner test that ensures the seeded task fixture and taskDag stay aligned (single node, matching root task id, matching node task id). apps/worker/test/validation-runner.test.tsL85-L93

Testing

✅ corepack pnpm --filter @codex-swarm/worker test
✅ corepack pnpm ci:typecheck

Committed on branch work as:

8a75f79 — Add DAG alignment assertions to worker run fixtures.

Also created a follow-up PR via the make_pr tool with title:

Add regression assertions for worker RunDetail taskDag fixtures

View task →

### Summary * Added an explicit regression assertion in the worker runtime test to verify the mocked `RunDetail.taskDag` remains the expected empty graph shape (`nodes`, `edges`, `rootTaskIds`, `blockedTaskIds`, `unblockPaths`). [apps/worker/test/runtime.test.tsL428-L442](https://github.com/beisel-it/codex-swarm/blob/3ce9d05425d2eb3a1bab7ffa427f2346b59d31f0/apps/worker/test/runtime.test.ts#L428-L442) * Added a focused validation-runner test that ensures the seeded task fixture and `taskDag` stay aligned (single node, matching root task id, matching node task id). [apps/worker/test/validation-runner.test.tsL85-L93](https://github.com/beisel-it/codex-swarm/blob/3ce9d05425d2eb3a1bab7ffa427f2346b59d31f0/apps/worker/test/validation-runner.test.ts#L85-L93) **Testing** * ✅ `corepack pnpm --filter @codex-swarm/worker test` * ✅ `corepack pnpm ci:typecheck` Committed on branch `work` as: * `8a75f79` — **Add DAG alignment assertions to worker run fixtures**. Also created a follow-up PR via the `make_pr` tool with title: * **Add regression assertions for worker RunDetail taskDag fixtures** [View task →](https://chatgpt.com/s/cd_69c9ca28e5488191a3ab81ccfe0d48a2)