fix(build-tools): surface real exceptions from done-file generation

[tylerbutler]

Description

Improves error reporting in fluid-build's incremental-cache machinery so that exceptions thrown while computing or comparing a task's "done file" content surface the real cause instead of being swallowed into a generic warning or silently marking the task non-incremental.

LeafWithDoneFileTask.getDoneFileContent() produces the cache key used to decide whether a task is up-to-date. Previously several implementations and their callers caught any exception and converted it to undefined or a generic "unable to read compare file" trace, discarding the underlying message and stack. A buggy tool or missing dependency would then only show up as unable to generate content for …, with no way to find the root cause without enabling verbose trace logging.

Changes:

• LeafWithDoneFileTask.markExecDone — log the error's message and stack, and rename the warning to "unable to generate or write done file" (it covered both, but only mentioned writing).
• LeafWithDoneFileTask.checkLeafIsUpToDate — separate errors from generating expected content from a missing done file. Bind the caught error into the trigger trace and special-case ENOENT so a fresh build stays quiet.
• LeafWithFileStatDoneFileTask.getDoneFileContent / getHashDoneFileContent — re-throw instead of returning undefined, letting markExecDone format and log the error.
• TscDependentTask.getDoneFileContent (api-extractor / eslint / generate-entrypoints / prettier) — re-throw instead of swallowing.
• TscTask.checkTsBuildInfoVersionSame — include the underlying error in the existing "Unable to get installed package version for typescript" trace.

Behavioral change is in the unhappy path only: successful builds emit no new output, while failed or non-incremental tasks now print the underlying error and stack on stderr.

[github-actions]

Hi! Thank you for opening this PR. Want me to review it?

Based on the diff (62 lines, 2 files), I've queued these reviewers:

• Correctness — logic errors, race conditions, lifecycle issues
• Security — vulnerabilities, secret exposure, injection
• API Compatibility — breaking changes, release tags, type design
• Performance — algorithmic regressions, memory leaks
• Testing — coverage gaps, hollow tests

How this works

• Adjust the reviewer set by ticking/unticking boxes above. Reviewer toggles alone don't trigger anything.

• Tick Start review below to dispatch the review fleet.

• After review finishes, tick Start review again to request another run — it auto-resets after each dispatch.

• This comment updates as new commits land; your reviewer selections are preserved.

• Start review

[Copilot]

Pull request overview

This PR improves fluid-build incremental-cache diagnostics by ensuring exceptions thrown during done-file generation/comparison surface meaningful details (including underlying error info), rather than being swallowed and only causing opaque “non-incremental” behavior.

Changes:

• Enhance LeafWithDoneFileTask.markExecDone() warning output to include both generation/write context and (when available) the error stack.
• Improve LeafWithDoneFileTask.checkLeafIsUpToDate() failure reporting by capturing the thrown error and handling expected ENOENT (first-run) cases separately.
• Stop silently converting failures into undefined in done-file content generators by rethrowing from LeafWithFileStatDoneFileTask and TscDependentTask, letting the outer handler report the real cause.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
build-tools/packages/build-tools/src/fluidBuild/tasks/leaf/leafTask.ts	Surfaces underlying exceptions for done-file read/compare/generation (incl. stack) and special-cases missing done files.
build-tools/packages/build-tools/src/fluidBuild/tasks/leaf/tscTask.ts	Improves error detail surfaced when TypeScript version lookup fails and rethrows done-file generation failures for dependent tasks.

[github-actions]

🔭 PR Review Fleet Report

Note

This report is generated by an experimental AI review fleet and is provided as a beta feature. Findings are a starting point for discussion, not a gate. Use your own judgement.

Verdict: ⚠️ Approve with Suggestions

0 Alert, 0 Stop, 1 Caution

Findings

Sev	#	Area	File	What	Fix
🚧 Caution	M1	Correctness	build-tools/packages/build-tools/src/fluidBuild/tasks/leaf/leafTask.ts:552-561	The checkLeafIsUpToDate catch block checks code === "ENOENT" to distinguish an expected "done file not found on first run" from other errors, and logs done file not found: ${doneFileFullPath}. Before this PR, getDoneFileContent() always returned undefined on error — meaning ENOENT in that catch could only come from readFile(doneFileFullPath, 'utf8') on line 537. After this PR, getDoneFileContent() now throws on error (e.g. when stat() or getFileHash() fails on a missing input or output file). If an input/output file is absent, stat() throws ENOENT, that error propagates through getDoneFileContent(), and is caught here. The code === "ENOENT" branch fires and logs done file not found: ${doneFileFullPath} — but doneFileFullPath names the done file, not the file that is actually missing. A user seeing this trace while the done file actually exists on disk would be misled about what is wrong. In the worst case (first build, output files haven't been created yet), this mis-attribution could mask a broken input-file configuration because the message looks like a routine "first run" trace rather than a real problem.	Split the try block so that getDoneFileContent() and readFile(doneFileFullPath) have separate error handling, or re-throw errors that originate from getDoneFileContent() before reaching the ENOENT check. For example: compute doneFileExpectedContent outside the outer try, catch only its errors separately, then wrap just the readFile call in the try that does the ENOENT check. This preserves the original intent of suppressing the noisy "first run" ENOENT while still surfacing real errors with accurate messages.

---

View workflow run