[7.2.0] `bazel build //X:all` records target completion events in the BEP stream with `file://` URIs, but sometimes doesn't write files to disk

[bazel-io]

Forked from #20737

[meteorcloudy]

/cc @coeuvre adding this one as a soft release blocker, let me know if this should actually block the release!

[bazel-io]

Cherry-pick was attempted but there were merge conflicts in the following file(s). Please resolve manually.

src/main/java/com/google/devtools/build/lib/skyframe/AspectCompletor.java src/main/java/com/google/devtools/build/lib/skyframe/BUILD src/main/java/com/google/devtools/build/lib/skyframe/CompletionFunction.java src/main/java/com/google/devtools/build/lib/skyframe/SkyframeExecutor.java src/main/java/com/google/devtools/build/lib/skyframe/TargetCompletor.java

cc: @bazelbuild/triage

[iancha1992]

Cherry-picked in https://github.com/bazelbuild/bazel/pull/22318 by @coeuvre

[coeuvre]

Unfortunately, the change was reverted in HEAD because it broke one of our internal test (not sure why the test was not triggered by the presubmit though). I will work on fix and patch it on the 7.2.0.

[iancha1992]

Will close this once https://github.com/bazelbuild/bazel/pull/22348 is merged