3 people 66e0a3ed93 [2.x] fix: Register every Def.declareOutput execution, not one per call site (#9492) ...

The cached-task macro allocated one mutable slot per syntactic
Def.declareOutput / Def.declareOutputDirectory call site and snapshotted the
slots into the task's outputs after the body ran. A call inside a loop or .map
over a runtime-determined list is a single syntactic site executed many times,
so each iteration overwrote the same slot and only the last file was cached and
restored on a cache hit. There was also no way for a conditional call site that
did not execute to stay out of the outputs: its slot remained null.

Declared outputs now accumulate in a per-task ListBuffer: the macro emits one
buffer at the top of the cached body and rewrites each call site to
ActionCache.registerOutput(vf, buffer), which appends and returns the value.
Every execution registers, an unexecuted site contributes nothing, and the
static multi-site shape is unchanged.

Refs #9462 (the declareOutput-in-a-loop half)

Co-authored-by: Claude Opus 4.8 (1M context) <[email protected]>
Co-authored-by: eugene yokota <[email protected]>

2026-07-25 15:57:06 -04:00

..

main

[2.x] fix: Register every Def.declareOutput execution, not one per call site (#9492)

2026-07-25 15:57:06 -04:00

test/scala/sbt/internal

[2.0.x] ci: Scalafmt 3.11.1 (#9280)

2026-05-31 16:30:32 -04:00