Skip to content

moltenhub-preserve follow-up task logs after local cleanup#450

Merged
jefking merged 1 commit intomainfrom
moltenhub-review-the-previous-local-task-logs-firs
Apr 11, 2026
Merged

moltenhub-preserve follow-up task logs after local cleanup#450
jefking merged 1 commit intomainfrom
moltenhub-review-the-previous-local-task-logs-firs

Conversation

@jefking
Copy link
Copy Markdown
Collaborator

@jefking jefking commented Apr 11, 2026

Summary

  • archive local task log files to .log/followup/... when building failure/no-changes follow-up prompts
  • use archived paths in follow-up prompt generation so cleanup of .log/local/... does not break follow-up log review
  • add tests that verify archived paths are used and remain readable after source cleanup

Validation

  • go test ./cmd/harness
  • go test ./internal/harness
  • go test ./internal/hub
  • go test ./...
  • go test ./internal/harness -run TestRunMultiRepoClonesConcurrently -count=1
  • go test ./internal/hub -run 'TestDispatchResultPayloadIncludesFailureMessageAndDetails|TestHandleDispatchQueuesFailureFollowUpWithTaskLogPaths' -count=1
  • go test ./cmd/harness -run 'TestFollowUpTaskLogPathsArchivesLocalTaskLogs|TestUnexpectedNoChangesFollowUpRunConfigPreservesTaskTargetingAndAddsContext|TestFailureFollowUpRunConfigUsesRequiredPayloadShapeAndLogContext' -count=1

@jefking jefking merged commit 25a8b81 into main Apr 11, 2026
2 checks passed
@jefking jefking deleted the moltenhub-review-the-previous-local-task-logs-firs branch April 11, 2026 05:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant