perf(dotnet): align build/test MSBuild property sets to avoid duplicate compile

[dnyw4l3n13]

Summary

The "Dotnet: Build (Non-Release)" step and the "Dotnet: Test" step in .github/actions/dotnet/action.yml pass different MSBuild global-property sets. Different global properties invalidate MSBuild's CoreCompile incremental-build fingerprint, so dotnet test ends up recompiling the solution instead of reusing the prior build's output.

This PR aligns the two property sets so the solution compiles once per CI run instead of twice.

See the Implementation Plan on the linked issue for the root-cause analysis and approach.

This is currently a placeholder commit (.deleteme.now) to open the draft PR; the actual property-set alignment lands in a follow-up commit.

Closes #870

[dnyw4l3n13]

Simplify clean - advancing to code review. Reviewed the diff across reuse/simplification/efficiency/altitude; no changes applied. Two non-blocking observations noted for the next phase to re-cover if relevant: the -p:Optimize=true explicit flag is technically redundant under --configuration Release (pre-existing convention, out of scope for this diff), and there is no shared-props mechanism enforcing parity between the build/test property lists going forward (architectural, out of scope for this fix).

[dnyw4l3n13]

Code review clean - advancing to security review

[dnyw4l3n13]

Security review clean, advancing to coverage check

[dnyw4l3n13]

Coverage ratchet passed - advancing to Human Review

[dnyw4l3n13]

PR marked ready for review and auto-merge (merge commit) enabled. All AI review phases (Simplify, Code Review, Security Review, Coverage) passed clean; no outstanding comments or CI failures found. Ready for human review.

[credfeto]

Super-linter summary

Language	Validation result
YAML	Pass ✅

All files and directories linted successfully

For more information, see the GitHub Actions workflow run

Powered by Super-linter