Skip to content

Commit 1d436bd

Browse files
committed
fix: capture all files in attempts
Right now subsequent attempts only display the repaired files. This is good, but it makes it super hard to see the "final product" and is a bit confusing. We should go back to the original behavior where we display the full files on every displayed attempt.
1 parent e5d1f1b commit 1d436bd

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

runner/orchestration/build-repair.ts

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -120,10 +120,12 @@ async function handleRepairResponse(
120120
progress
121121
);
122122

123+
// Capture attempt's full files. Copy because `finalOutputFiles` can be
124+
// mutated in subsequent repair attempts.
125+
const attemptFullFiles = finalOutputFiles.map((f) => ({ ...f }));
126+
123127
return {
124-
// Log the `outputFiles` from the repair response specifically, because
125-
// we want a snapshot after the current API call, not the full file set.
126-
outputFiles: repairResponse.outputFiles,
128+
outputFiles: attemptFullFiles,
127129
usage: repairResponse.usage,
128130
reasoning: repairResponse.reasoning,
129131
buildResult,

0 commit comments

Comments
 (0)