Skip to content

Commit f979689

Browse files
committed
test(task-orchestrator): wait for clarification comment materialization
1 parent e13bd56 commit f979689

2 files changed

Lines changed: 9 additions & 12 deletions

File tree

‎CHANGELOG.md‎

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -14,18 +14,6 @@ The format is based on Keep a Changelog, and this project follows SemVer where p
1414

1515
## [0.1.15] - 2026-03-16
1616

17-
### Fixed
18-
19-
- Treated resident `task-orchestrator serve` shutdown aborts as clean exits so release verification does not fail after the clarification-resume acceptance flow has already passed.
20-
21-
## [0.1.14] - 2026-03-16
22-
23-
### Fixed
24-
25-
- Removed unused TypeScript symbols that caused the release verify job (`bun x tsc --noEmit`) to fail before GitHub could publish `v0.1.13`.
26-
27-
## [0.1.13] - 2026-03-16
28-
2917
### Changed
3018

3119
- Completion checks now isolate their decision files per check invocation instead of sharing one mutable decision path across the run.
@@ -34,6 +22,9 @@ The format is based on Keep a Changelog, and this project follows SemVer where p
3422
### Fixed
3523

3624
- Prevented long-running agent workflows from exhausting 100 completion-check iterations with an empty task queue by detecting routing deadlocks and failing fast with explicit diagnostics.
25+
- Removed unused TypeScript symbols that caused the release verify job (`bun x tsc --noEmit`) to fail before GitHub could publish the release.
26+
- Treated resident `task-orchestrator serve` shutdown aborts as clean exits so release verification does not fail after the clarification-resume acceptance flow has already passed.
27+
- Hardened the clarification-resume acceptance test to wait for clarification-comment materialization before asserting the waiting-state handoff.
3728

3829
## [0.1.12] - 2026-03-15
3930

‎tests/at/cli-subcommands.test.ts‎

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2278,6 +2278,12 @@ exit 1
22782278
const waitingState = await readTaskState();
22792279
const waitingRunId = waitingState?.latest_run_id;
22802280
expect(waitingRunId).toBeTruthy();
2281+
2282+
await waitForCondition(async () => {
2283+
const text = await readFile(clarificationCommentPath, "utf-8").catch(() => "");
2284+
return text.includes("Need clarification before implementation can proceed");
2285+
}, 20_000, 100);
2286+
22812287
const clarificationComment = await readFile(clarificationCommentPath, "utf-8");
22822288
expect(clarificationComment).toContain("Need clarification before implementation can proceed");
22832289

0 commit comments

Comments
 (0)