Repair current-main broker cleanup failure after PR #1426 #1434

Merged

coilyco-ops merged 1 commit from issue-1428 into main

2026-07-15 12:38:03 +00:00

coilyco-ops commented

2026-07-15 12:36:11 +00:00

Member

The change stays narrow: it removes the brittle cleanup path without changing broker launch behavior, so the failure remains focused on the actual launch/env regression rather than artifact teardown.

Removed the explicit RemoveAll calls for the dispatch artifact subtrees.
Kept the broker env-restoration assertions and temp-home cleanup coverage.
Validated with go test ./cmd/ward -run TestRunHostDispatchBrokerRequestClearsBrokerEnvWhileLaunchRuns -count=20 and ward exec test.

closes #1428
ward.workflow: pull-request-and-merge

This tightens the broker cleanup regression test that surfaced after PR #1425 landed on top of PR #1426. The test keeps the env-restoration assertions and relies on the temp-home .ward teardown instead of explicitly removing the raw and redacted dispatch subtrees, which could race finalization. The change stays narrow: it removes the brittle cleanup path without changing broker launch behavior, so the failure remains focused on the actual launch/env regression rather than artifact teardown. - Removed the explicit RemoveAll calls for the dispatch artifact subtrees. - Kept the broker env-restoration assertions and temp-home cleanup coverage. - Validated with go test ./cmd/ward -run TestRunHostDispatchBrokerRequestClearsBrokerEnvWhileLaunchRuns -count=20 and ward exec test. closes #1428 ward.workflow: pull-request-and-merge

coilyco-ops added 1 commit

2026-07-15 12:36:12 +00:00

Repair broker cleanup regression after PR #1426

test / test (pull_request) Successful in 1m7s

Details

13c3c5f79b

coilyco-ops merged commit 7c7a1ce172 into main

2026-07-15 12:38:03 +00:00

coilyco-ops referenced this pull request from a commit

2026-07-15 12:38:04 +00:00

Repair current-main broker cleanup failure after PR #1426 (#1434)

coilyco-ops referenced this pull request