Report artifact post-processing input provenance

[Amaury Levé (Evangelink)]

Summary

• add additive FileArtifactMessage.InputArtifactPaths field ID 8 to report the exact manifest inputs represented by each processed artifact
• populate provenance per dispatcher output and define non-null ProcessedArtifact results as representing every supplied input
• preserve manifest path strings verbatim so the SDK can intersect them with the originating job without cross-process normalization drift
• cover serializer compatibility, old readers skipping field 8, multiple disjoint outputs, and the generic .xml collision from MTP artifact post-processing: a merged artifact can unlist unrelated artifacts sharing its extension dotnet/sdk#55555
• update the dotnet-test pipe protocol and artifact post-processing RFC

Compatibility

This remains an additive field on serializer ID 7. Older readers skip it, newer readers treat its absence as an old-host response, and no protocol-version bump or handshake capability is required.

A follow-up SDK change is still required to vendor field 8 and remove only provenance paths that intersect the dispatched job's manifest. Relates to dotnet/sdk#55555.

Testing

• ./build.cmd -pack -bl
• 129 protocol test executions across net462, net8.0, and net9.0
• 69 shared protocol-contract test executions across net462, net8.0, and net9.0
• 3 focused artifact post-processing acceptance tests on net11.0

[Copilot]

Pull request overview

Adds per-output artifact provenance to resolve incorrect artifact collapsing tracked by dotnet/sdk#55555.

Changes:

• Adds and serializes InputArtifactPaths as field ID 8.
• Reports exact manifest inputs for each processed artifact.
• Adds compatibility, routing, and acceptance coverage plus protocol documentation.

Show a summary per file

File	Description
ProtocolTests.cs	Tests provenance round-tripping.
ProtocolEdgeCaseTests.cs	Tests absent and unknown fields.
DotnetTestProtocolSerializerTests.cs	Pins field ID 8 and shared serialization.
DotnetTestPipeProtocol.cs	Decodes provenance in acceptance tests.
DotnetTestPipeArtifactPostProcessingTests.cs	Tests disjoint outputs and legacy readers.
FileArtifactMessagesSerializer.cs	Serializes provenance paths.
ObjectFieldIds.cs	Defines field ID 8.
FileArtifactMessages.cs	Adds the provenance model property.
InternalAPI.Unshipped.txt	Tracks the platform internal API changes.
IArtifactPostProcessor.cs	Documents all-input output semantics.
ArtifactPostProcessingDispatcherTool.cs	Attaches matched inputs to outputs.
018-Artifact-Post-Processing.md	Updates the post-processing RFC.
004-protocol-dotnet-test-pipe.md	Documents the additive wire field.

Review details

• Files reviewed: 13/13 changed files
• Comments generated: 2
• Review effort level: Balanced

[Copilot]

Review details

• Files reviewed: 17/17 changed files
• Comments generated: 0 new
• Review effort level: Balanced

[github-actions]

🧵 Parallel-safety audit — PR #10463

Parallelization — assemblies containing the changed test files:

Test assembly	Scope	Workers	Analyzer coverage
Microsoft.Testing.Platform.Acceptance.IntegrationTests	MethodLevel ([assembly: Parallelize] in Program.cs)	CPU count (Workers = 0)	coverable once MSTEST0074–0077 ship (attribute opt-in)
Microsoft.Testing.Platform.DotnetTestProtocolContract.UnitTests	MethodLevel	CPU count	coverable once MSTEST0074–0077 ship (attribute opt-in)
Microsoft.Testing.Platform.UnitTests	MethodLevel	CPU count	coverable once MSTEST0074–0077 ship (attribute opt-in)

No .runsettings/testconfig.json/MSBuild parallelization overrides found for these projects, and this PR does not touch any [assembly: Parallelize]/[assembly: DoNotParallelize] declaration, so the configured MethodLevel scope stands unmodified.

Findings: A (global-state) 0 · B (paths) 0 · C (declaration) 0 · D (over-serialization) 0 — by severity: Critical 0 · High 0 · Warning 0 · Info 0.

Analysis:

• DotnetTestPipeArtifactPostProcessingTests.cs (new methods Dispatcher_ReportsOnlyInputsRepresentedByEachOutput and Dispatcher_ReportsDisjointInputsForMultipleOutputsAndLegacyReaderSkipsProvenance) each build their working directory as Path.Combine(Path.GetTempPath(), $"artifact-dispatcher-...-{Guid.NewGuid():N}") — a per-invocation unique path, so no cross-test filesystem collision under MethodLevel concurrency (category B: none). Each method's finally deletes only its own GUID-scoped directory. No environment variables, CWD, console, or mutable static state are touched. No [ResourceLock]/[DoNotParallelize] declarations were added, removed, or would be warranted.
• DotnetTestProtocolSerializerTests.cs, ProtocolEdgeCaseTests.cs, ProtocolTests.cs — all new/changed assertions operate on local MemoryStream instances and local FileArtifactMessage/FileArtifactMessages values passed through a pure round-trip helper; no shared or process-global state is read or written.

Nothing in this diff falls into categories A–D. Scope is MethodLevel for all three assemblies (the highest-risk configuration), but the added tests are already written in an isolation-safe style (unique GUID temp dirs, no shared statics), so there is nothing to flag.

_{Advisory only — heuristic, non-blocking. Re-run with /parallel-audit. This audit answers "is it parallel-safe?"; for testability, smells, or flakiness see the detect-static-dependencies / test-smell-detection / test-anti-patterns analyses.}

🤖 Automated content by GitHub Copilot. Generated by the Parallel-safety audit on PR (on open / sync) workflow. · auto · 100.7 AIC · ⌖ 3.66 AIC · ⊞ 24.7K · [◷]( · ◷)

[github-actions]

🧪 Test quality grade — PR #10463

Graded 8 new/modified test methods across 3 files, all covering the new InputArtifactPaths provenance field on FileArtifactMessage (wire protocol, serializer round-trips, and the artifact-dispatcher acceptance tests). Every test uses a clear Arrange-Act-Assert shape, asserts concrete values (not just null-checks), and exercises the specific mutation points introduced by this change (wire field id, serialization round-trip, dispatcher provenance grouping, and legacy-reader opt-out). No anti-patterns, tautological assertions, or unawaited async issues were observed, so all 8 tests grade A.

Grade	Test	Mutation	Notes	How to improve
A (90–100)	mod DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ AdvertisesCapabilityAndReturnsMergedArtifact	2/2 killed	Asserts advertised capability string and exact input-path provenance via AreSequenceEqual.	—
A (90–100)	new DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ ReportsOnlyInputsRepresentedByEachOutput	2/2 killed	Verifies per-kind attribution and explicitly asserts the unrelated input is excluded.	—
A (90–100)	new DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ ReportsDisjointInputsForMultipleOutputsAndLegacyReaderSkipsProvenance	3/3 killed	Covers disjoint per-kind grouping and that the legacy reader path omits provenance.	—
A (90–100)	mod DotnetTestProtocolSerializerTests. FileArtifactMessageFieldIds_ AreStable	3/3 killed	Reflection-based pin of the SessionUid, Kind, and new InputArtifactPaths wire ids.	—
A (90–100)	mod DotnetTestProtocolSerializerTests. FileArtifactMessages_ RoundTrips	2/2 killed	Round-trips InputArtifactPaths for a populated and an empty-array artifact.	—
A (90–100)	mod ProtocolEdgeCaseTests. FileArtifactMessages_ WhenMultipleArtifactsWithMixedNulls_ RoundTrips	1/1 killed	Confirms InputArtifactPaths stays null when the source message omits it.	—
A (90–100)	new ProtocolEdgeCaseTests. FileArtifactMessages_ WhenUnknownFieldsPrecedeKnownFields_ DeserializesRemainingFields	1/1 killed	Verifies unknown-field skipping does not corrupt the InputArtifactPaths default.	—
A (90–100)	mod ProtocolTests. FileArtifactMessagesSerializeDeserialize	2/2 killed	Loop-based assertions verify InputArtifactPaths sequence equality for every artifact.	—

_{This advisory comment was generated automatically. Grades are heuristic
and informational — they do not block merging. Suggestions on the Files
changed tab can be applied with one click. Re-run with
/grade-tests.}

🤖 Automated content by GitHub Copilot. Generated by the Grade Tests on PR (on open / sync) workflow. · auto · 144.1 AIC · ⌖ 3.75 AIC · ⊞ 16.2K · [◷]( · ◷)

[Amaury Levé (Evangelink)]

Summary

• add additive FileArtifactMessage.InputArtifactPaths field ID 8 to report the exact manifest inputs represented by each processed artifact
• populate provenance per dispatcher output and define non-null ProcessedArtifact results as representing every supplied input
• preserve manifest path strings verbatim so the SDK can intersect them with the originating job without cross-process normalization drift
• cover serializer compatibility, old readers skipping field 8, multiple disjoint outputs, and the generic .xml collision from MTP artifact post-processing: a merged artifact can unlist unrelated artifacts sharing its extension dotnet/sdk#55555
• update the dotnet-test pipe protocol and artifact post-processing RFC

Compatibility

This remains an additive field on serializer ID 7. Older readers skip it, newer readers treat its absence as an old-host response, and no protocol-version bump or handshake capability is required.

A follow-up SDK change is still required to vendor field 8 and remove only provenance paths that intersect the dispatched job's manifest. Relates to dotnet/sdk#55555.

Testing

• ./build.cmd -pack -bl
• 129 protocol test executions across net462, net8.0, and net9.0
• 69 shared protocol-contract test executions across net462, net8.0, and net9.0
• 3 focused artifact post-processing acceptance tests on net11.0

[Copilot]

Pull request overview

Adds per-output artifact provenance to resolve incorrect artifact collapsing tracked by dotnet/sdk#55555.

Changes:

• Adds and serializes InputArtifactPaths as field ID 8.
• Reports exact manifest inputs for each processed artifact.
• Adds compatibility, routing, and acceptance coverage plus protocol documentation.

Show a summary per file

File	Description
ProtocolTests.cs	Tests provenance round-tripping.
ProtocolEdgeCaseTests.cs	Tests absent and unknown fields.
DotnetTestProtocolSerializerTests.cs	Pins field ID 8 and shared serialization.
DotnetTestPipeProtocol.cs	Decodes provenance in acceptance tests.
DotnetTestPipeArtifactPostProcessingTests.cs	Tests disjoint outputs and legacy readers.
FileArtifactMessagesSerializer.cs	Serializes provenance paths.
ObjectFieldIds.cs	Defines field ID 8.
FileArtifactMessages.cs	Adds the provenance model property.
InternalAPI.Unshipped.txt	Tracks the platform internal API changes.
IArtifactPostProcessor.cs	Documents all-input output semantics.
ArtifactPostProcessingDispatcherTool.cs	Attaches matched inputs to outputs.
018-Artifact-Post-Processing.md	Updates the post-processing RFC.
004-protocol-dotnet-test-pipe.md	Documents the additive wire field.

Review details

• Files reviewed: 13/13 changed files
• Comments generated: 2
• Review effort level: Balanced

[Copilot]

Review details

• Files reviewed: 17/17 changed files
• Comments generated: 0 new
• Review effort level: Balanced

[github-actions]

🧵 Parallel-safety audit — PR #10463

Parallelization — assemblies containing the changed test files:

Test assembly	Scope	Workers	Analyzer coverage
Microsoft.Testing.Platform.Acceptance.IntegrationTests	MethodLevel ([assembly: Parallelize] in Program.cs)	CPU count (Workers = 0)	coverable once MSTEST0074–0077 ship (attribute opt-in)
Microsoft.Testing.Platform.DotnetTestProtocolContract.UnitTests	MethodLevel	CPU count	coverable once MSTEST0074–0077 ship (attribute opt-in)
Microsoft.Testing.Platform.UnitTests	MethodLevel	CPU count	coverable once MSTEST0074–0077 ship (attribute opt-in)

No .runsettings/testconfig.json/MSBuild parallelization overrides found for these projects, and this PR does not touch any [assembly: Parallelize]/[assembly: DoNotParallelize] declaration, so the configured MethodLevel scope stands unmodified.

Findings: A (global-state) 0 · B (paths) 0 · C (declaration) 0 · D (over-serialization) 0 — by severity: Critical 0 · High 0 · Warning 0 · Info 0.

Analysis:

• DotnetTestPipeArtifactPostProcessingTests.cs (new methods Dispatcher_ReportsOnlyInputsRepresentedByEachOutput and Dispatcher_ReportsDisjointInputsForMultipleOutputsAndLegacyReaderSkipsProvenance) each build their working directory as Path.Combine(Path.GetTempPath(), $"artifact-dispatcher-...-{Guid.NewGuid():N}") — a per-invocation unique path, so no cross-test filesystem collision under MethodLevel concurrency (category B: none). Each method's finally deletes only its own GUID-scoped directory. No environment variables, CWD, console, or mutable static state are touched. No [ResourceLock]/[DoNotParallelize] declarations were added, removed, or would be warranted.
• DotnetTestProtocolSerializerTests.cs, ProtocolEdgeCaseTests.cs, ProtocolTests.cs — all new/changed assertions operate on local MemoryStream instances and local FileArtifactMessage/FileArtifactMessages values passed through a pure round-trip helper; no shared or process-global state is read or written.

Nothing in this diff falls into categories A–D. Scope is MethodLevel for all three assemblies (the highest-risk configuration), but the added tests are already written in an isolation-safe style (unique GUID temp dirs, no shared statics), so there is nothing to flag.

_{Advisory only — heuristic, non-blocking. Re-run with /parallel-audit. This audit answers "is it parallel-safe?"; for testability, smells, or flakiness see the detect-static-dependencies / test-smell-detection / test-anti-patterns analyses.}

🤖 Automated content by GitHub Copilot. Generated by the Parallel-safety audit on PR (on open / sync) workflow. · auto · 100.7 AIC · ⌖ 3.66 AIC · ⊞ 24.7K · [◷]( · ◷)

[github-actions]

🧪 Test quality grade — PR #10463

Graded 8 new/modified test methods across 3 files, all covering the new InputArtifactPaths provenance field on FileArtifactMessage (wire protocol, serializer round-trips, and the artifact-dispatcher acceptance tests). Every test uses a clear Arrange-Act-Assert shape, asserts concrete values (not just null-checks), and exercises the specific mutation points introduced by this change (wire field id, serialization round-trip, dispatcher provenance grouping, and legacy-reader opt-out). No anti-patterns, tautological assertions, or unawaited async issues were observed, so all 8 tests grade A.

Grade	Test	Mutation	Notes	How to improve
A (90–100)	mod DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ AdvertisesCapabilityAndReturnsMergedArtifact	2/2 killed	Asserts advertised capability string and exact input-path provenance via AreSequenceEqual.	—
A (90–100)	new DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ ReportsOnlyInputsRepresentedByEachOutput	2/2 killed	Verifies per-kind attribution and explicitly asserts the unrelated input is excluded.	—
A (90–100)	new DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ ReportsDisjointInputsForMultipleOutputsAndLegacyReaderSkipsProvenance	3/3 killed	Covers disjoint per-kind grouping and that the legacy reader path omits provenance.	—
A (90–100)	mod DotnetTestProtocolSerializerTests. FileArtifactMessageFieldIds_ AreStable	3/3 killed	Reflection-based pin of the SessionUid, Kind, and new InputArtifactPaths wire ids.	—
A (90–100)	mod DotnetTestProtocolSerializerTests. FileArtifactMessages_ RoundTrips	2/2 killed	Round-trips InputArtifactPaths for a populated and an empty-array artifact.	—
A (90–100)	mod ProtocolEdgeCaseTests. FileArtifactMessages_ WhenMultipleArtifactsWithMixedNulls_ RoundTrips	1/1 killed	Confirms InputArtifactPaths stays null when the source message omits it.	—
A (90–100)	new ProtocolEdgeCaseTests. FileArtifactMessages_ WhenUnknownFieldsPrecedeKnownFields_ DeserializesRemainingFields	1/1 killed	Verifies unknown-field skipping does not corrupt the InputArtifactPaths default.	—
A (90–100)	mod ProtocolTests. FileArtifactMessagesSerializeDeserialize	2/2 killed	Loop-based assertions verify InputArtifactPaths sequence equality for every artifact.	—

_{This advisory comment was generated automatically. Grades are heuristic
and informational — they do not block merging. Suggestions on the Files
changed tab can be applied with one click. Re-run with
/grade-tests.}

🤖 Automated content by GitHub Copilot. Generated by the Grade Tests on PR (on open / sync) workflow. · auto · 144.1 AIC · ⌖ 3.75 AIC · ⊞ 16.2K · [◷]( · ◷)

[Amaury Levé (Evangelink)]

Summary

• add additive FileArtifactMessage.InputArtifactPaths field ID 8 to report the exact manifest inputs represented by each processed artifact
• populate provenance per dispatcher output and define non-null ProcessedArtifact results as representing every supplied input
• preserve manifest path strings verbatim so the SDK can intersect them with the originating job without cross-process normalization drift
• cover serializer compatibility, old readers skipping field 8, multiple disjoint outputs, and the generic .xml collision from MTP artifact post-processing: a merged artifact can unlist unrelated artifacts sharing its extension dotnet/sdk#55555
• update the dotnet-test pipe protocol and artifact post-processing RFC

Compatibility

This remains an additive field on serializer ID 7. Older readers skip it, newer readers treat its absence as an old-host response, and no protocol-version bump or handshake capability is required.

A follow-up SDK change is still required to vendor field 8 and remove only provenance paths that intersect the dispatched job's manifest. Relates to dotnet/sdk#55555.

Testing

• ./build.cmd -pack -bl
• 129 protocol test executions across net462, net8.0, and net9.0
• 69 shared protocol-contract test executions across net462, net8.0, and net9.0
• 3 focused artifact post-processing acceptance tests on net11.0

[Copilot]

Pull request overview

Adds per-output artifact provenance to resolve incorrect artifact collapsing tracked by dotnet/sdk#55555.

Changes:

• Adds and serializes InputArtifactPaths as field ID 8.
• Reports exact manifest inputs for each processed artifact.
• Adds compatibility, routing, and acceptance coverage plus protocol documentation.

Show a summary per file

File	Description
ProtocolTests.cs	Tests provenance round-tripping.
ProtocolEdgeCaseTests.cs	Tests absent and unknown fields.
DotnetTestProtocolSerializerTests.cs	Pins field ID 8 and shared serialization.
DotnetTestPipeProtocol.cs	Decodes provenance in acceptance tests.
DotnetTestPipeArtifactPostProcessingTests.cs	Tests disjoint outputs and legacy readers.
FileArtifactMessagesSerializer.cs	Serializes provenance paths.
ObjectFieldIds.cs	Defines field ID 8.
FileArtifactMessages.cs	Adds the provenance model property.
InternalAPI.Unshipped.txt	Tracks the platform internal API changes.
IArtifactPostProcessor.cs	Documents all-input output semantics.
ArtifactPostProcessingDispatcherTool.cs	Attaches matched inputs to outputs.
018-Artifact-Post-Processing.md	Updates the post-processing RFC.
004-protocol-dotnet-test-pipe.md	Documents the additive wire field.

Review details

• Files reviewed: 13/13 changed files
• Comments generated: 2
• Review effort level: Balanced

[Copilot]

Review details

• Files reviewed: 17/17 changed files
• Comments generated: 0 new
• Review effort level: Balanced

[github-actions]

🧵 Parallel-safety audit — PR #10463

Parallelization — assemblies containing the changed test files:

Test assembly	Scope	Workers	Analyzer coverage
Microsoft.Testing.Platform.Acceptance.IntegrationTests	MethodLevel ([assembly: Parallelize] in Program.cs)	CPU count (Workers = 0)	coverable once MSTEST0074–0077 ship (attribute opt-in)
Microsoft.Testing.Platform.DotnetTestProtocolContract.UnitTests	MethodLevel	CPU count	coverable once MSTEST0074–0077 ship (attribute opt-in)
Microsoft.Testing.Platform.UnitTests	MethodLevel	CPU count	coverable once MSTEST0074–0077 ship (attribute opt-in)

No .runsettings/testconfig.json/MSBuild parallelization overrides found for these projects, and this PR does not touch any [assembly: Parallelize]/[assembly: DoNotParallelize] declaration, so the configured MethodLevel scope stands unmodified.

Findings: A (global-state) 0 · B (paths) 0 · C (declaration) 0 · D (over-serialization) 0 — by severity: Critical 0 · High 0 · Warning 0 · Info 0.

Analysis:

• DotnetTestPipeArtifactPostProcessingTests.cs (new methods Dispatcher_ReportsOnlyInputsRepresentedByEachOutput and Dispatcher_ReportsDisjointInputsForMultipleOutputsAndLegacyReaderSkipsProvenance) each build their working directory as Path.Combine(Path.GetTempPath(), $"artifact-dispatcher-...-{Guid.NewGuid():N}") — a per-invocation unique path, so no cross-test filesystem collision under MethodLevel concurrency (category B: none). Each method's finally deletes only its own GUID-scoped directory. No environment variables, CWD, console, or mutable static state are touched. No [ResourceLock]/[DoNotParallelize] declarations were added, removed, or would be warranted.
• DotnetTestProtocolSerializerTests.cs, ProtocolEdgeCaseTests.cs, ProtocolTests.cs — all new/changed assertions operate on local MemoryStream instances and local FileArtifactMessage/FileArtifactMessages values passed through a pure round-trip helper; no shared or process-global state is read or written.

Nothing in this diff falls into categories A–D. Scope is MethodLevel for all three assemblies (the highest-risk configuration), but the added tests are already written in an isolation-safe style (unique GUID temp dirs, no shared statics), so there is nothing to flag.

_{Advisory only — heuristic, non-blocking. Re-run with /parallel-audit. This audit answers "is it parallel-safe?"; for testability, smells, or flakiness see the detect-static-dependencies / test-smell-detection / test-anti-patterns analyses.}

🤖 Automated content by GitHub Copilot. Generated by the Parallel-safety audit on PR (on open / sync) workflow. · auto · 100.7 AIC · ⌖ 3.66 AIC · ⊞ 24.7K · [◷]( · ◷)

[github-actions]

🧪 Test quality grade — PR #10463

Graded 8 new/modified test methods across 3 files, all covering the new InputArtifactPaths provenance field on FileArtifactMessage (wire protocol, serializer round-trips, and the artifact-dispatcher acceptance tests). Every test uses a clear Arrange-Act-Assert shape, asserts concrete values (not just null-checks), and exercises the specific mutation points introduced by this change (wire field id, serialization round-trip, dispatcher provenance grouping, and legacy-reader opt-out). No anti-patterns, tautological assertions, or unawaited async issues were observed, so all 8 tests grade A.

Grade	Test	Mutation	Notes	How to improve
A (90–100)	mod DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ AdvertisesCapabilityAndReturnsMergedArtifact	2/2 killed	Asserts advertised capability string and exact input-path provenance via AreSequenceEqual.	—
A (90–100)	new DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ ReportsOnlyInputsRepresentedByEachOutput	2/2 killed	Verifies per-kind attribution and explicitly asserts the unrelated input is excluded.	—
A (90–100)	new DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ ReportsDisjointInputsForMultipleOutputsAndLegacyReaderSkipsProvenance	3/3 killed	Covers disjoint per-kind grouping and that the legacy reader path omits provenance.	—
A (90–100)	mod DotnetTestProtocolSerializerTests. FileArtifactMessageFieldIds_ AreStable	3/3 killed	Reflection-based pin of the SessionUid, Kind, and new InputArtifactPaths wire ids.	—
A (90–100)	mod DotnetTestProtocolSerializerTests. FileArtifactMessages_ RoundTrips	2/2 killed	Round-trips InputArtifactPaths for a populated and an empty-array artifact.	—
A (90–100)	mod ProtocolEdgeCaseTests. FileArtifactMessages_ WhenMultipleArtifactsWithMixedNulls_ RoundTrips	1/1 killed	Confirms InputArtifactPaths stays null when the source message omits it.	—
A (90–100)	new ProtocolEdgeCaseTests. FileArtifactMessages_ WhenUnknownFieldsPrecedeKnownFields_ DeserializesRemainingFields	1/1 killed	Verifies unknown-field skipping does not corrupt the InputArtifactPaths default.	—
A (90–100)	mod ProtocolTests. FileArtifactMessagesSerializeDeserialize	2/2 killed	Loop-based assertions verify InputArtifactPaths sequence equality for every artifact.	—

_{This advisory comment was generated automatically. Grades are heuristic
and informational — they do not block merging. Suggestions on the Files
changed tab can be applied with one click. Re-run with
/grade-tests.}

🤖 Automated content by GitHub Copilot. Generated by the Grade Tests on PR (on open / sync) workflow. · auto · 144.1 AIC · ⌖ 3.75 AIC · ⊞ 16.2K · [◷]( · ◷)

[Amaury Levé (Evangelink)]

Summary

• add additive FileArtifactMessage.InputArtifactPaths field ID 8 to report the exact manifest inputs represented by each processed artifact
• populate provenance per dispatcher output and define non-null ProcessedArtifact results as representing every supplied input
• preserve manifest path strings verbatim so the SDK can intersect them with the originating job without cross-process normalization drift
• cover serializer compatibility, old readers skipping field 8, multiple disjoint outputs, and the generic .xml collision from MTP artifact post-processing: a merged artifact can unlist unrelated artifacts sharing its extension dotnet/sdk#55555
• update the dotnet-test pipe protocol and artifact post-processing RFC

Compatibility

This remains an additive field on serializer ID 7. Older readers skip it, newer readers treat its absence as an old-host response, and no protocol-version bump or handshake capability is required.

A follow-up SDK change is still required to vendor field 8 and remove only provenance paths that intersect the dispatched job's manifest. Relates to dotnet/sdk#55555.

Testing

• ./build.cmd -pack -bl
• 129 protocol test executions across net462, net8.0, and net9.0
• 69 shared protocol-contract test executions across net462, net8.0, and net9.0
• 3 focused artifact post-processing acceptance tests on net11.0

[Copilot]

Pull request overview

Adds per-output artifact provenance to resolve incorrect artifact collapsing tracked by dotnet/sdk#55555.

Changes:

• Adds and serializes InputArtifactPaths as field ID 8.
• Reports exact manifest inputs for each processed artifact.
• Adds compatibility, routing, and acceptance coverage plus protocol documentation.

Show a summary per file

File	Description
ProtocolTests.cs	Tests provenance round-tripping.
ProtocolEdgeCaseTests.cs	Tests absent and unknown fields.
DotnetTestProtocolSerializerTests.cs	Pins field ID 8 and shared serialization.
DotnetTestPipeProtocol.cs	Decodes provenance in acceptance tests.
DotnetTestPipeArtifactPostProcessingTests.cs	Tests disjoint outputs and legacy readers.
FileArtifactMessagesSerializer.cs	Serializes provenance paths.
ObjectFieldIds.cs	Defines field ID 8.
FileArtifactMessages.cs	Adds the provenance model property.
InternalAPI.Unshipped.txt	Tracks the platform internal API changes.
IArtifactPostProcessor.cs	Documents all-input output semantics.
ArtifactPostProcessingDispatcherTool.cs	Attaches matched inputs to outputs.
018-Artifact-Post-Processing.md	Updates the post-processing RFC.
004-protocol-dotnet-test-pipe.md	Documents the additive wire field.

Review details

• Files reviewed: 13/13 changed files
• Comments generated: 2
• Review effort level: Balanced

[Copilot]

Review details

• Files reviewed: 17/17 changed files
• Comments generated: 0 new
• Review effort level: Balanced

[github-actions]

🧵 Parallel-safety audit — PR #10463

Parallelization — assemblies containing the changed test files:

Test assembly	Scope	Workers	Analyzer coverage
Microsoft.Testing.Platform.Acceptance.IntegrationTests	MethodLevel ([assembly: Parallelize] in Program.cs)	CPU count (Workers = 0)	coverable once MSTEST0074–0077 ship (attribute opt-in)
Microsoft.Testing.Platform.DotnetTestProtocolContract.UnitTests	MethodLevel	CPU count	coverable once MSTEST0074–0077 ship (attribute opt-in)
Microsoft.Testing.Platform.UnitTests	MethodLevel	CPU count	coverable once MSTEST0074–0077 ship (attribute opt-in)

No .runsettings/testconfig.json/MSBuild parallelization overrides found for these projects, and this PR does not touch any [assembly: Parallelize]/[assembly: DoNotParallelize] declaration, so the configured MethodLevel scope stands unmodified.

Findings: A (global-state) 0 · B (paths) 0 · C (declaration) 0 · D (over-serialization) 0 — by severity: Critical 0 · High 0 · Warning 0 · Info 0.

Analysis:

• DotnetTestPipeArtifactPostProcessingTests.cs (new methods Dispatcher_ReportsOnlyInputsRepresentedByEachOutput and Dispatcher_ReportsDisjointInputsForMultipleOutputsAndLegacyReaderSkipsProvenance) each build their working directory as Path.Combine(Path.GetTempPath(), $"artifact-dispatcher-...-{Guid.NewGuid():N}") — a per-invocation unique path, so no cross-test filesystem collision under MethodLevel concurrency (category B: none). Each method's finally deletes only its own GUID-scoped directory. No environment variables, CWD, console, or mutable static state are touched. No [ResourceLock]/[DoNotParallelize] declarations were added, removed, or would be warranted.
• DotnetTestProtocolSerializerTests.cs, ProtocolEdgeCaseTests.cs, ProtocolTests.cs — all new/changed assertions operate on local MemoryStream instances and local FileArtifactMessage/FileArtifactMessages values passed through a pure round-trip helper; no shared or process-global state is read or written.

Nothing in this diff falls into categories A–D. Scope is MethodLevel for all three assemblies (the highest-risk configuration), but the added tests are already written in an isolation-safe style (unique GUID temp dirs, no shared statics), so there is nothing to flag.

_{Advisory only — heuristic, non-blocking. Re-run with /parallel-audit. This audit answers "is it parallel-safe?"; for testability, smells, or flakiness see the detect-static-dependencies / test-smell-detection / test-anti-patterns analyses.}

🤖 Automated content by GitHub Copilot. Generated by the Parallel-safety audit on PR (on open / sync) workflow. · auto · 100.7 AIC · ⌖ 3.66 AIC · ⊞ 24.7K · [◷]( · ◷)

[github-actions]

🧪 Test quality grade — PR #10463

Graded 8 new/modified test methods across 3 files, all covering the new InputArtifactPaths provenance field on FileArtifactMessage (wire protocol, serializer round-trips, and the artifact-dispatcher acceptance tests). Every test uses a clear Arrange-Act-Assert shape, asserts concrete values (not just null-checks), and exercises the specific mutation points introduced by this change (wire field id, serialization round-trip, dispatcher provenance grouping, and legacy-reader opt-out). No anti-patterns, tautological assertions, or unawaited async issues were observed, so all 8 tests grade A.

Grade	Test	Mutation	Notes	How to improve
A (90–100)	mod DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ AdvertisesCapabilityAndReturnsMergedArtifact	2/2 killed	Asserts advertised capability string and exact input-path provenance via AreSequenceEqual.	—
A (90–100)	new DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ ReportsOnlyInputsRepresentedByEachOutput	2/2 killed	Verifies per-kind attribution and explicitly asserts the unrelated input is excluded.	—
A (90–100)	new DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ ReportsDisjointInputsForMultipleOutputsAndLegacyReaderSkipsProvenance	3/3 killed	Covers disjoint per-kind grouping and that the legacy reader path omits provenance.	—
A (90–100)	mod DotnetTestProtocolSerializerTests. FileArtifactMessageFieldIds_ AreStable	3/3 killed	Reflection-based pin of the SessionUid, Kind, and new InputArtifactPaths wire ids.	—
A (90–100)	mod DotnetTestProtocolSerializerTests. FileArtifactMessages_ RoundTrips	2/2 killed	Round-trips InputArtifactPaths for a populated and an empty-array artifact.	—
A (90–100)	mod ProtocolEdgeCaseTests. FileArtifactMessages_ WhenMultipleArtifactsWithMixedNulls_ RoundTrips	1/1 killed	Confirms InputArtifactPaths stays null when the source message omits it.	—
A (90–100)	new ProtocolEdgeCaseTests. FileArtifactMessages_ WhenUnknownFieldsPrecedeKnownFields_ DeserializesRemainingFields	1/1 killed	Verifies unknown-field skipping does not corrupt the InputArtifactPaths default.	—
A (90–100)	mod ProtocolTests. FileArtifactMessagesSerializeDeserialize	2/2 killed	Loop-based assertions verify InputArtifactPaths sequence equality for every artifact.	—

_{This advisory comment was generated automatically. Grades are heuristic
and informational — they do not block merging. Suggestions on the Files
changed tab can be applied with one click. Re-run with
/grade-tests.}

🤖 Automated content by GitHub Copilot. Generated by the Grade Tests on PR (on open / sync) workflow. · auto · 144.1 AIC · ⌖ 3.75 AIC · ⊞ 16.2K · [◷]( · ◷)

[Amaury Levé (Evangelink)]

Summary

• add additive FileArtifactMessage.InputArtifactPaths field ID 8 to report the exact manifest inputs represented by each processed artifact
• populate provenance per dispatcher output and define non-null ProcessedArtifact results as representing every supplied input
• preserve manifest path strings verbatim so the SDK can intersect them with the originating job without cross-process normalization drift
• cover serializer compatibility, old readers skipping field 8, multiple disjoint outputs, and the generic .xml collision from MTP artifact post-processing: a merged artifact can unlist unrelated artifacts sharing its extension dotnet/sdk#55555
• update the dotnet-test pipe protocol and artifact post-processing RFC

Compatibility

This remains an additive field on serializer ID 7. Older readers skip it, newer readers treat its absence as an old-host response, and no protocol-version bump or handshake capability is required.

A follow-up SDK change is still required to vendor field 8 and remove only provenance paths that intersect the dispatched job's manifest. Relates to dotnet/sdk#55555.

Testing

• ./build.cmd -pack -bl
• 129 protocol test executions across net462, net8.0, and net9.0
• 69 shared protocol-contract test executions across net462, net8.0, and net9.0
• 3 focused artifact post-processing acceptance tests on net11.0

[Copilot]

Pull request overview

Adds per-output artifact provenance to resolve incorrect artifact collapsing tracked by dotnet/sdk#55555.

Changes:

• Adds and serializes InputArtifactPaths as field ID 8.
• Reports exact manifest inputs for each processed artifact.
• Adds compatibility, routing, and acceptance coverage plus protocol documentation.

Show a summary per file

File	Description
ProtocolTests.cs	Tests provenance round-tripping.
ProtocolEdgeCaseTests.cs	Tests absent and unknown fields.
DotnetTestProtocolSerializerTests.cs	Pins field ID 8 and shared serialization.
DotnetTestPipeProtocol.cs	Decodes provenance in acceptance tests.
DotnetTestPipeArtifactPostProcessingTests.cs	Tests disjoint outputs and legacy readers.
FileArtifactMessagesSerializer.cs	Serializes provenance paths.
ObjectFieldIds.cs	Defines field ID 8.
FileArtifactMessages.cs	Adds the provenance model property.
InternalAPI.Unshipped.txt	Tracks the platform internal API changes.
IArtifactPostProcessor.cs	Documents all-input output semantics.
ArtifactPostProcessingDispatcherTool.cs	Attaches matched inputs to outputs.
018-Artifact-Post-Processing.md	Updates the post-processing RFC.
004-protocol-dotnet-test-pipe.md	Documents the additive wire field.

Review details

• Files reviewed: 13/13 changed files
• Comments generated: 2
• Review effort level: Balanced

[Copilot]

Review details

• Files reviewed: 17/17 changed files
• Comments generated: 0 new
• Review effort level: Balanced

[github-actions]

🧵 Parallel-safety audit — PR #10463

Parallelization — assemblies containing the changed test files:

Test assembly	Scope	Workers	Analyzer coverage
Microsoft.Testing.Platform.Acceptance.IntegrationTests	MethodLevel ([assembly: Parallelize] in Program.cs)	CPU count (Workers = 0)	coverable once MSTEST0074–0077 ship (attribute opt-in)
Microsoft.Testing.Platform.DotnetTestProtocolContract.UnitTests	MethodLevel	CPU count	coverable once MSTEST0074–0077 ship (attribute opt-in)
Microsoft.Testing.Platform.UnitTests	MethodLevel	CPU count	coverable once MSTEST0074–0077 ship (attribute opt-in)

No .runsettings/testconfig.json/MSBuild parallelization overrides found for these projects, and this PR does not touch any [assembly: Parallelize]/[assembly: DoNotParallelize] declaration, so the configured MethodLevel scope stands unmodified.

Findings: A (global-state) 0 · B (paths) 0 · C (declaration) 0 · D (over-serialization) 0 — by severity: Critical 0 · High 0 · Warning 0 · Info 0.

Analysis:

• DotnetTestPipeArtifactPostProcessingTests.cs (new methods Dispatcher_ReportsOnlyInputsRepresentedByEachOutput and Dispatcher_ReportsDisjointInputsForMultipleOutputsAndLegacyReaderSkipsProvenance) each build their working directory as Path.Combine(Path.GetTempPath(), $"artifact-dispatcher-...-{Guid.NewGuid():N}") — a per-invocation unique path, so no cross-test filesystem collision under MethodLevel concurrency (category B: none). Each method's finally deletes only its own GUID-scoped directory. No environment variables, CWD, console, or mutable static state are touched. No [ResourceLock]/[DoNotParallelize] declarations were added, removed, or would be warranted.
• DotnetTestProtocolSerializerTests.cs, ProtocolEdgeCaseTests.cs, ProtocolTests.cs — all new/changed assertions operate on local MemoryStream instances and local FileArtifactMessage/FileArtifactMessages values passed through a pure round-trip helper; no shared or process-global state is read or written.

Nothing in this diff falls into categories A–D. Scope is MethodLevel for all three assemblies (the highest-risk configuration), but the added tests are already written in an isolation-safe style (unique GUID temp dirs, no shared statics), so there is nothing to flag.

_{Advisory only — heuristic, non-blocking. Re-run with /parallel-audit. This audit answers "is it parallel-safe?"; for testability, smells, or flakiness see the detect-static-dependencies / test-smell-detection / test-anti-patterns analyses.}

🤖 Automated content by GitHub Copilot. Generated by the Parallel-safety audit on PR (on open / sync) workflow. · auto · 100.7 AIC · ⌖ 3.66 AIC · ⊞ 24.7K · [◷]( · ◷)

[github-actions]

🧪 Test quality grade — PR #10463

Graded 8 new/modified test methods across 3 files, all covering the new InputArtifactPaths provenance field on FileArtifactMessage (wire protocol, serializer round-trips, and the artifact-dispatcher acceptance tests). Every test uses a clear Arrange-Act-Assert shape, asserts concrete values (not just null-checks), and exercises the specific mutation points introduced by this change (wire field id, serialization round-trip, dispatcher provenance grouping, and legacy-reader opt-out). No anti-patterns, tautological assertions, or unawaited async issues were observed, so all 8 tests grade A.

Grade	Test	Mutation	Notes	How to improve
A (90–100)	mod DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ AdvertisesCapabilityAndReturnsMergedArtifact	2/2 killed	Asserts advertised capability string and exact input-path provenance via AreSequenceEqual.	—
A (90–100)	new DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ ReportsOnlyInputsRepresentedByEachOutput	2/2 killed	Verifies per-kind attribution and explicitly asserts the unrelated input is excluded.	—
A (90–100)	new DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ ReportsDisjointInputsForMultipleOutputsAndLegacyReaderSkipsProvenance	3/3 killed	Covers disjoint per-kind grouping and that the legacy reader path omits provenance.	—
A (90–100)	mod DotnetTestProtocolSerializerTests. FileArtifactMessageFieldIds_ AreStable	3/3 killed	Reflection-based pin of the SessionUid, Kind, and new InputArtifactPaths wire ids.	—
A (90–100)	mod DotnetTestProtocolSerializerTests. FileArtifactMessages_ RoundTrips	2/2 killed	Round-trips InputArtifactPaths for a populated and an empty-array artifact.	—
A (90–100)	mod ProtocolEdgeCaseTests. FileArtifactMessages_ WhenMultipleArtifactsWithMixedNulls_ RoundTrips	1/1 killed	Confirms InputArtifactPaths stays null when the source message omits it.	—
A (90–100)	new ProtocolEdgeCaseTests. FileArtifactMessages_ WhenUnknownFieldsPrecedeKnownFields_ DeserializesRemainingFields	1/1 killed	Verifies unknown-field skipping does not corrupt the InputArtifactPaths default.	—
A (90–100)	mod ProtocolTests. FileArtifactMessagesSerializeDeserialize	2/2 killed	Loop-based assertions verify InputArtifactPaths sequence equality for every artifact.	—

_{This advisory comment was generated automatically. Grades are heuristic
and informational — they do not block merging. Suggestions on the Files
changed tab can be applied with one click. Re-run with
/grade-tests.}

🤖 Automated content by GitHub Copilot. Generated by the Grade Tests on PR (on open / sync) workflow. · auto · 144.1 AIC · ⌖ 3.75 AIC · ⊞ 16.2K · [◷]( · ◷)

[Amaury Levé (Evangelink)]

Summary

• add additive FileArtifactMessage.InputArtifactPaths field ID 8 to report the exact manifest inputs represented by each processed artifact
• populate provenance per dispatcher output and define non-null ProcessedArtifact results as representing every supplied input
• preserve manifest path strings verbatim so the SDK can intersect them with the originating job without cross-process normalization drift
• cover serializer compatibility, old readers skipping field 8, multiple disjoint outputs, and the generic .xml collision from MTP artifact post-processing: a merged artifact can unlist unrelated artifacts sharing its extension dotnet/sdk#55555
• update the dotnet-test pipe protocol and artifact post-processing RFC

Compatibility

This remains an additive field on serializer ID 7. Older readers skip it, newer readers treat its absence as an old-host response, and no protocol-version bump or handshake capability is required.

A follow-up SDK change is still required to vendor field 8 and remove only provenance paths that intersect the dispatched job's manifest. Relates to dotnet/sdk#55555.

Testing

• ./build.cmd -pack -bl
• 129 protocol test executions across net462, net8.0, and net9.0
• 69 shared protocol-contract test executions across net462, net8.0, and net9.0
• 3 focused artifact post-processing acceptance tests on net11.0

[Copilot]

Pull request overview

Adds per-output artifact provenance to resolve incorrect artifact collapsing tracked by dotnet/sdk#55555.

Changes:

• Adds and serializes InputArtifactPaths as field ID 8.
• Reports exact manifest inputs for each processed artifact.
• Adds compatibility, routing, and acceptance coverage plus protocol documentation.

Show a summary per file

File	Description
ProtocolTests.cs	Tests provenance round-tripping.
ProtocolEdgeCaseTests.cs	Tests absent and unknown fields.
DotnetTestProtocolSerializerTests.cs	Pins field ID 8 and shared serialization.
DotnetTestPipeProtocol.cs	Decodes provenance in acceptance tests.
DotnetTestPipeArtifactPostProcessingTests.cs	Tests disjoint outputs and legacy readers.
FileArtifactMessagesSerializer.cs	Serializes provenance paths.
ObjectFieldIds.cs	Defines field ID 8.
FileArtifactMessages.cs	Adds the provenance model property.
InternalAPI.Unshipped.txt	Tracks the platform internal API changes.
IArtifactPostProcessor.cs	Documents all-input output semantics.
ArtifactPostProcessingDispatcherTool.cs	Attaches matched inputs to outputs.
018-Artifact-Post-Processing.md	Updates the post-processing RFC.
004-protocol-dotnet-test-pipe.md	Documents the additive wire field.

Review details

• Files reviewed: 13/13 changed files
• Comments generated: 2
• Review effort level: Balanced

[Copilot]

Review details

• Files reviewed: 17/17 changed files
• Comments generated: 0 new
• Review effort level: Balanced

[github-actions]

🧵 Parallel-safety audit — PR #10463

Parallelization — assemblies containing the changed test files:

Test assembly	Scope	Workers	Analyzer coverage
Microsoft.Testing.Platform.Acceptance.IntegrationTests	MethodLevel ([assembly: Parallelize] in Program.cs)	CPU count (Workers = 0)	coverable once MSTEST0074–0077 ship (attribute opt-in)
Microsoft.Testing.Platform.DotnetTestProtocolContract.UnitTests	MethodLevel	CPU count	coverable once MSTEST0074–0077 ship (attribute opt-in)
Microsoft.Testing.Platform.UnitTests	MethodLevel	CPU count	coverable once MSTEST0074–0077 ship (attribute opt-in)

No .runsettings/testconfig.json/MSBuild parallelization overrides found for these projects, and this PR does not touch any [assembly: Parallelize]/[assembly: DoNotParallelize] declaration, so the configured MethodLevel scope stands unmodified.

Findings: A (global-state) 0 · B (paths) 0 · C (declaration) 0 · D (over-serialization) 0 — by severity: Critical 0 · High 0 · Warning 0 · Info 0.

Analysis:

• DotnetTestPipeArtifactPostProcessingTests.cs (new methods Dispatcher_ReportsOnlyInputsRepresentedByEachOutput and Dispatcher_ReportsDisjointInputsForMultipleOutputsAndLegacyReaderSkipsProvenance) each build their working directory as Path.Combine(Path.GetTempPath(), $"artifact-dispatcher-...-{Guid.NewGuid():N}") — a per-invocation unique path, so no cross-test filesystem collision under MethodLevel concurrency (category B: none). Each method's finally deletes only its own GUID-scoped directory. No environment variables, CWD, console, or mutable static state are touched. No [ResourceLock]/[DoNotParallelize] declarations were added, removed, or would be warranted.
• DotnetTestProtocolSerializerTests.cs, ProtocolEdgeCaseTests.cs, ProtocolTests.cs — all new/changed assertions operate on local MemoryStream instances and local FileArtifactMessage/FileArtifactMessages values passed through a pure round-trip helper; no shared or process-global state is read or written.

Nothing in this diff falls into categories A–D. Scope is MethodLevel for all three assemblies (the highest-risk configuration), but the added tests are already written in an isolation-safe style (unique GUID temp dirs, no shared statics), so there is nothing to flag.

_{Advisory only — heuristic, non-blocking. Re-run with /parallel-audit. This audit answers "is it parallel-safe?"; for testability, smells, or flakiness see the detect-static-dependencies / test-smell-detection / test-anti-patterns analyses.}

🤖 Automated content by GitHub Copilot. Generated by the Parallel-safety audit on PR (on open / sync) workflow. · auto · 100.7 AIC · ⌖ 3.66 AIC · ⊞ 24.7K · [◷]( · ◷)

[github-actions]

🧪 Test quality grade — PR #10463

Graded 8 new/modified test methods across 3 files, all covering the new InputArtifactPaths provenance field on FileArtifactMessage (wire protocol, serializer round-trips, and the artifact-dispatcher acceptance tests). Every test uses a clear Arrange-Act-Assert shape, asserts concrete values (not just null-checks), and exercises the specific mutation points introduced by this change (wire field id, serialization round-trip, dispatcher provenance grouping, and legacy-reader opt-out). No anti-patterns, tautological assertions, or unawaited async issues were observed, so all 8 tests grade A.

Grade	Test	Mutation	Notes	How to improve
A (90–100)	mod DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ AdvertisesCapabilityAndReturnsMergedArtifact	2/2 killed	Asserts advertised capability string and exact input-path provenance via AreSequenceEqual.	—
A (90–100)	new DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ ReportsOnlyInputsRepresentedByEachOutput	2/2 killed	Verifies per-kind attribution and explicitly asserts the unrelated input is excluded.	—
A (90–100)	new DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ ReportsDisjointInputsForMultipleOutputsAndLegacyReaderSkipsProvenance	3/3 killed	Covers disjoint per-kind grouping and that the legacy reader path omits provenance.	—
A (90–100)	mod DotnetTestProtocolSerializerTests. FileArtifactMessageFieldIds_ AreStable	3/3 killed	Reflection-based pin of the SessionUid, Kind, and new InputArtifactPaths wire ids.	—
A (90–100)	mod DotnetTestProtocolSerializerTests. FileArtifactMessages_ RoundTrips	2/2 killed	Round-trips InputArtifactPaths for a populated and an empty-array artifact.	—
A (90–100)	mod ProtocolEdgeCaseTests. FileArtifactMessages_ WhenMultipleArtifactsWithMixedNulls_ RoundTrips	1/1 killed	Confirms InputArtifactPaths stays null when the source message omits it.	—
A (90–100)	new ProtocolEdgeCaseTests. FileArtifactMessages_ WhenUnknownFieldsPrecedeKnownFields_ DeserializesRemainingFields	1/1 killed	Verifies unknown-field skipping does not corrupt the InputArtifactPaths default.	—
A (90–100)	mod ProtocolTests. FileArtifactMessagesSerializeDeserialize	2/2 killed	Loop-based assertions verify InputArtifactPaths sequence equality for every artifact.	—

_{This advisory comment was generated automatically. Grades are heuristic
and informational — they do not block merging. Suggestions on the Files
changed tab can be applied with one click. Re-run with
/grade-tests.}

🤖 Automated content by GitHub Copilot. Generated by the Grade Tests on PR (on open / sync) workflow. · auto · 144.1 AIC · ⌖ 3.75 AIC · ⊞ 16.2K · [◷]( · ◷)

[Amaury Levé (Evangelink)]

Summary

• add additive FileArtifactMessage.InputArtifactPaths field ID 8 to report the exact manifest inputs represented by each processed artifact
• populate provenance per dispatcher output and define non-null ProcessedArtifact results as representing every supplied input
• preserve manifest path strings verbatim so the SDK can intersect them with the originating job without cross-process normalization drift
• cover serializer compatibility, old readers skipping field 8, multiple disjoint outputs, and the generic .xml collision from MTP artifact post-processing: a merged artifact can unlist unrelated artifacts sharing its extension dotnet/sdk#55555
• update the dotnet-test pipe protocol and artifact post-processing RFC

Compatibility

This remains an additive field on serializer ID 7. Older readers skip it, newer readers treat its absence as an old-host response, and no protocol-version bump or handshake capability is required.

A follow-up SDK change is still required to vendor field 8 and remove only provenance paths that intersect the dispatched job's manifest. Relates to dotnet/sdk#55555.

Testing

• ./build.cmd -pack -bl
• 129 protocol test executions across net462, net8.0, and net9.0
• 69 shared protocol-contract test executions across net462, net8.0, and net9.0
• 3 focused artifact post-processing acceptance tests on net11.0

[Copilot]

Pull request overview

Adds per-output artifact provenance to resolve incorrect artifact collapsing tracked by dotnet/sdk#55555.

Changes:

• Adds and serializes InputArtifactPaths as field ID 8.
• Reports exact manifest inputs for each processed artifact.
• Adds compatibility, routing, and acceptance coverage plus protocol documentation.

Show a summary per file

File	Description
ProtocolTests.cs	Tests provenance round-tripping.
ProtocolEdgeCaseTests.cs	Tests absent and unknown fields.
DotnetTestProtocolSerializerTests.cs	Pins field ID 8 and shared serialization.
DotnetTestPipeProtocol.cs	Decodes provenance in acceptance tests.
DotnetTestPipeArtifactPostProcessingTests.cs	Tests disjoint outputs and legacy readers.
FileArtifactMessagesSerializer.cs	Serializes provenance paths.
ObjectFieldIds.cs	Defines field ID 8.
FileArtifactMessages.cs	Adds the provenance model property.
InternalAPI.Unshipped.txt	Tracks the platform internal API changes.
IArtifactPostProcessor.cs	Documents all-input output semantics.
ArtifactPostProcessingDispatcherTool.cs	Attaches matched inputs to outputs.
018-Artifact-Post-Processing.md	Updates the post-processing RFC.
004-protocol-dotnet-test-pipe.md	Documents the additive wire field.

Review details

• Files reviewed: 13/13 changed files
• Comments generated: 2
• Review effort level: Balanced

[Copilot]

Review details

• Files reviewed: 17/17 changed files
• Comments generated: 0 new
• Review effort level: Balanced

[github-actions]

🧵 Parallel-safety audit — PR #10463

Parallelization — assemblies containing the changed test files:

Test assembly	Scope	Workers	Analyzer coverage
Microsoft.Testing.Platform.Acceptance.IntegrationTests	MethodLevel ([assembly: Parallelize] in Program.cs)	CPU count (Workers = 0)	coverable once MSTEST0074–0077 ship (attribute opt-in)
Microsoft.Testing.Platform.DotnetTestProtocolContract.UnitTests	MethodLevel	CPU count	coverable once MSTEST0074–0077 ship (attribute opt-in)
Microsoft.Testing.Platform.UnitTests	MethodLevel	CPU count	coverable once MSTEST0074–0077 ship (attribute opt-in)

No .runsettings/testconfig.json/MSBuild parallelization overrides found for these projects, and this PR does not touch any [assembly: Parallelize]/[assembly: DoNotParallelize] declaration, so the configured MethodLevel scope stands unmodified.

Findings: A (global-state) 0 · B (paths) 0 · C (declaration) 0 · D (over-serialization) 0 — by severity: Critical 0 · High 0 · Warning 0 · Info 0.

Analysis:

• DotnetTestPipeArtifactPostProcessingTests.cs (new methods Dispatcher_ReportsOnlyInputsRepresentedByEachOutput and Dispatcher_ReportsDisjointInputsForMultipleOutputsAndLegacyReaderSkipsProvenance) each build their working directory as Path.Combine(Path.GetTempPath(), $"artifact-dispatcher-...-{Guid.NewGuid():N}") — a per-invocation unique path, so no cross-test filesystem collision under MethodLevel concurrency (category B: none). Each method's finally deletes only its own GUID-scoped directory. No environment variables, CWD, console, or mutable static state are touched. No [ResourceLock]/[DoNotParallelize] declarations were added, removed, or would be warranted.
• DotnetTestProtocolSerializerTests.cs, ProtocolEdgeCaseTests.cs, ProtocolTests.cs — all new/changed assertions operate on local MemoryStream instances and local FileArtifactMessage/FileArtifactMessages values passed through a pure round-trip helper; no shared or process-global state is read or written.

Nothing in this diff falls into categories A–D. Scope is MethodLevel for all three assemblies (the highest-risk configuration), but the added tests are already written in an isolation-safe style (unique GUID temp dirs, no shared statics), so there is nothing to flag.

_{Advisory only — heuristic, non-blocking. Re-run with /parallel-audit. This audit answers "is it parallel-safe?"; for testability, smells, or flakiness see the detect-static-dependencies / test-smell-detection / test-anti-patterns analyses.}

🤖 Automated content by GitHub Copilot. Generated by the Parallel-safety audit on PR (on open / sync) workflow. · auto · 100.7 AIC · ⌖ 3.66 AIC · ⊞ 24.7K · [◷]( · ◷)

[github-actions]

🧪 Test quality grade — PR #10463

Graded 8 new/modified test methods across 3 files, all covering the new InputArtifactPaths provenance field on FileArtifactMessage (wire protocol, serializer round-trips, and the artifact-dispatcher acceptance tests). Every test uses a clear Arrange-Act-Assert shape, asserts concrete values (not just null-checks), and exercises the specific mutation points introduced by this change (wire field id, serialization round-trip, dispatcher provenance grouping, and legacy-reader opt-out). No anti-patterns, tautological assertions, or unawaited async issues were observed, so all 8 tests grade A.

Grade	Test	Mutation	Notes	How to improve
A (90–100)	mod DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ AdvertisesCapabilityAndReturnsMergedArtifact	2/2 killed	Asserts advertised capability string and exact input-path provenance via AreSequenceEqual.	—
A (90–100)	new DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ ReportsOnlyInputsRepresentedByEachOutput	2/2 killed	Verifies per-kind attribution and explicitly asserts the unrelated input is excluded.	—
A (90–100)	new DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ ReportsDisjointInputsForMultipleOutputsAndLegacyReaderSkipsProvenance	3/3 killed	Covers disjoint per-kind grouping and that the legacy reader path omits provenance.	—
A (90–100)	mod DotnetTestProtocolSerializerTests. FileArtifactMessageFieldIds_ AreStable	3/3 killed	Reflection-based pin of the SessionUid, Kind, and new InputArtifactPaths wire ids.	—
A (90–100)	mod DotnetTestProtocolSerializerTests. FileArtifactMessages_ RoundTrips	2/2 killed	Round-trips InputArtifactPaths for a populated and an empty-array artifact.	—
A (90–100)	mod ProtocolEdgeCaseTests. FileArtifactMessages_ WhenMultipleArtifactsWithMixedNulls_ RoundTrips	1/1 killed	Confirms InputArtifactPaths stays null when the source message omits it.	—
A (90–100)	new ProtocolEdgeCaseTests. FileArtifactMessages_ WhenUnknownFieldsPrecedeKnownFields_ DeserializesRemainingFields	1/1 killed	Verifies unknown-field skipping does not corrupt the InputArtifactPaths default.	—
A (90–100)	mod ProtocolTests. FileArtifactMessagesSerializeDeserialize	2/2 killed	Loop-based assertions verify InputArtifactPaths sequence equality for every artifact.	—

_{This advisory comment was generated automatically. Grades are heuristic
and informational — they do not block merging. Suggestions on the Files
changed tab can be applied with one click. Re-run with
/grade-tests.}

🤖 Automated content by GitHub Copilot. Generated by the Grade Tests on PR (on open / sync) workflow. · auto · 144.1 AIC · ⌖ 3.75 AIC · ⊞ 16.2K · [◷]( · ◷)

[Amaury Levé (Evangelink)]

Summary

• add additive FileArtifactMessage.InputArtifactPaths field ID 8 to report the exact manifest inputs represented by each processed artifact
• populate provenance per dispatcher output and define non-null ProcessedArtifact results as representing every supplied input
• preserve manifest path strings verbatim so the SDK can intersect them with the originating job without cross-process normalization drift
• cover serializer compatibility, old readers skipping field 8, multiple disjoint outputs, and the generic .xml collision from MTP artifact post-processing: a merged artifact can unlist unrelated artifacts sharing its extension dotnet/sdk#55555
• update the dotnet-test pipe protocol and artifact post-processing RFC

Compatibility

This remains an additive field on serializer ID 7. Older readers skip it, newer readers treat its absence as an old-host response, and no protocol-version bump or handshake capability is required.

A follow-up SDK change is still required to vendor field 8 and remove only provenance paths that intersect the dispatched job's manifest. Relates to dotnet/sdk#55555.

Testing

• ./build.cmd -pack -bl
• 129 protocol test executions across net462, net8.0, and net9.0
• 69 shared protocol-contract test executions across net462, net8.0, and net9.0
• 3 focused artifact post-processing acceptance tests on net11.0

[Copilot]

Pull request overview

Adds per-output artifact provenance to resolve incorrect artifact collapsing tracked by dotnet/sdk#55555.

Changes:

• Adds and serializes InputArtifactPaths as field ID 8.
• Reports exact manifest inputs for each processed artifact.
• Adds compatibility, routing, and acceptance coverage plus protocol documentation.

Show a summary per file

File	Description
ProtocolTests.cs	Tests provenance round-tripping.
ProtocolEdgeCaseTests.cs	Tests absent and unknown fields.
DotnetTestProtocolSerializerTests.cs	Pins field ID 8 and shared serialization.
DotnetTestPipeProtocol.cs	Decodes provenance in acceptance tests.
DotnetTestPipeArtifactPostProcessingTests.cs	Tests disjoint outputs and legacy readers.
FileArtifactMessagesSerializer.cs	Serializes provenance paths.
ObjectFieldIds.cs	Defines field ID 8.
FileArtifactMessages.cs	Adds the provenance model property.
InternalAPI.Unshipped.txt	Tracks the platform internal API changes.
IArtifactPostProcessor.cs	Documents all-input output semantics.
ArtifactPostProcessingDispatcherTool.cs	Attaches matched inputs to outputs.
018-Artifact-Post-Processing.md	Updates the post-processing RFC.
004-protocol-dotnet-test-pipe.md	Documents the additive wire field.

Review details

• Files reviewed: 13/13 changed files
• Comments generated: 2
• Review effort level: Balanced

[Copilot]

Review details

• Files reviewed: 17/17 changed files
• Comments generated: 0 new
• Review effort level: Balanced

[github-actions]

🧵 Parallel-safety audit — PR #10463

Parallelization — assemblies containing the changed test files:

Test assembly	Scope	Workers	Analyzer coverage
Microsoft.Testing.Platform.Acceptance.IntegrationTests	MethodLevel ([assembly: Parallelize] in Program.cs)	CPU count (Workers = 0)	coverable once MSTEST0074–0077 ship (attribute opt-in)
Microsoft.Testing.Platform.DotnetTestProtocolContract.UnitTests	MethodLevel	CPU count	coverable once MSTEST0074–0077 ship (attribute opt-in)
Microsoft.Testing.Platform.UnitTests	MethodLevel	CPU count	coverable once MSTEST0074–0077 ship (attribute opt-in)

No .runsettings/testconfig.json/MSBuild parallelization overrides found for these projects, and this PR does not touch any [assembly: Parallelize]/[assembly: DoNotParallelize] declaration, so the configured MethodLevel scope stands unmodified.

Findings: A (global-state) 0 · B (paths) 0 · C (declaration) 0 · D (over-serialization) 0 — by severity: Critical 0 · High 0 · Warning 0 · Info 0.

Analysis:

• DotnetTestPipeArtifactPostProcessingTests.cs (new methods Dispatcher_ReportsOnlyInputsRepresentedByEachOutput and Dispatcher_ReportsDisjointInputsForMultipleOutputsAndLegacyReaderSkipsProvenance) each build their working directory as Path.Combine(Path.GetTempPath(), $"artifact-dispatcher-...-{Guid.NewGuid():N}") — a per-invocation unique path, so no cross-test filesystem collision under MethodLevel concurrency (category B: none). Each method's finally deletes only its own GUID-scoped directory. No environment variables, CWD, console, or mutable static state are touched. No [ResourceLock]/[DoNotParallelize] declarations were added, removed, or would be warranted.
• DotnetTestProtocolSerializerTests.cs, ProtocolEdgeCaseTests.cs, ProtocolTests.cs — all new/changed assertions operate on local MemoryStream instances and local FileArtifactMessage/FileArtifactMessages values passed through a pure round-trip helper; no shared or process-global state is read or written.

Nothing in this diff falls into categories A–D. Scope is MethodLevel for all three assemblies (the highest-risk configuration), but the added tests are already written in an isolation-safe style (unique GUID temp dirs, no shared statics), so there is nothing to flag.

_{Advisory only — heuristic, non-blocking. Re-run with /parallel-audit. This audit answers "is it parallel-safe?"; for testability, smells, or flakiness see the detect-static-dependencies / test-smell-detection / test-anti-patterns analyses.}

🤖 Automated content by GitHub Copilot. Generated by the Parallel-safety audit on PR (on open / sync) workflow. · auto · 100.7 AIC · ⌖ 3.66 AIC · ⊞ 24.7K · [◷]( · ◷)

[github-actions]

🧪 Test quality grade — PR #10463

Graded 8 new/modified test methods across 3 files, all covering the new InputArtifactPaths provenance field on FileArtifactMessage (wire protocol, serializer round-trips, and the artifact-dispatcher acceptance tests). Every test uses a clear Arrange-Act-Assert shape, asserts concrete values (not just null-checks), and exercises the specific mutation points introduced by this change (wire field id, serialization round-trip, dispatcher provenance grouping, and legacy-reader opt-out). No anti-patterns, tautological assertions, or unawaited async issues were observed, so all 8 tests grade A.

Grade	Test	Mutation	Notes	How to improve
A (90–100)	mod DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ AdvertisesCapabilityAndReturnsMergedArtifact	2/2 killed	Asserts advertised capability string and exact input-path provenance via AreSequenceEqual.	—
A (90–100)	new DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ ReportsOnlyInputsRepresentedByEachOutput	2/2 killed	Verifies per-kind attribution and explicitly asserts the unrelated input is excluded.	—
A (90–100)	new DotnetTestPipeArtifactPostProcessingTests. Dispatcher_ ReportsDisjointInputsForMultipleOutputsAndLegacyReaderSkipsProvenance	3/3 killed	Covers disjoint per-kind grouping and that the legacy reader path omits provenance.	—
A (90–100)	mod DotnetTestProtocolSerializerTests. FileArtifactMessageFieldIds_ AreStable	3/3 killed	Reflection-based pin of the SessionUid, Kind, and new InputArtifactPaths wire ids.	—
A (90–100)	mod DotnetTestProtocolSerializerTests. FileArtifactMessages_ RoundTrips	2/2 killed	Round-trips InputArtifactPaths for a populated and an empty-array artifact.	—
A (90–100)	mod ProtocolEdgeCaseTests. FileArtifactMessages_ WhenMultipleArtifactsWithMixedNulls_ RoundTrips	1/1 killed	Confirms InputArtifactPaths stays null when the source message omits it.	—
A (90–100)	new ProtocolEdgeCaseTests. FileArtifactMessages_ WhenUnknownFieldsPrecedeKnownFields_ DeserializesRemainingFields	1/1 killed	Verifies unknown-field skipping does not corrupt the InputArtifactPaths default.	—
A (90–100)	mod ProtocolTests. FileArtifactMessagesSerializeDeserialize	2/2 killed	Loop-based assertions verify InputArtifactPaths sequence equality for every artifact.	—

_{This advisory comment was generated automatically. Grades are heuristic
and informational — they do not block merging. Suggestions on the Files
changed tab can be applied with one click. Re-run with
/grade-tests.}

🤖 Automated content by GitHub Copilot. Generated by the Grade Tests on PR (on open / sync) workflow. · auto · 144.1 AIC · ⌖ 3.75 AIC · ⊞ 16.2K · [◷]( · ◷)