audit: phase 2 verification U11-U15 (round 01)

[samtrion]

Summary

Independent Phase 2 verification of usability assumptions U11 through U15 from audit/assumptions/round-01-usability.md. Each assumption was re-evaluated against the worktree's current source, and a failing test or repro was produced where applicable.

This PR contains NO production code changes — Phase 3 builders will fix the issues. Failing tests + Notes only.

ID	Status	Title	Failing artifact
U11	CONFIRMED	Transport extensions silently overwrite previous registration	tests/NetEvolve.Pulse.Tests.Unit/Outbox/TransportOverwriteDiagnosticTests.cs
U12	CONFIRMED	IMediator Scoped — BackgroundService XML-doc example violates DI lifetime	tests/NetEvolve.Pulse.Tests.Unit/IMediatorSendOnlyBackgroundServiceLifetimeTests.cs
U13	NEEDS-NUANCE	NuGet metadata missing icon + embedded README	audit/repros/u13/verify.ps1 (+ committed nuspec evidence). README auto-detected on .NET 10 SDK; icon still missing.
U14	CONFIRMED	C# 14 extension(...) blocks lock source build to .NET 10 SDK	audit/repros/u14/verify.ps1 (+ committed build.log). Actual error: NETSDK1045, not CS1003/CS8400.
U15	CONFIRMED	Missing handler gives generic DI exception, no scanning helper	tests/NetEvolve.Pulse.Tests.Unit/MissingHandlerDiagnosticTests.cs

Notable findings beyond the original assumptions

• U13 is only PARTIALLY confirmed. The .NET 10 SDK (10.0.300) auto-detects an adjacent README.md and emits <readme>README.md</readme> in the nuspec without any explicit PackageReadmeFile property. The icon half (<icon> + logo.png packing) is still fully missing. Phase 3 should still set both properties explicitly to avoid behavior drifting with SDK version.
• U14 confirms the symptom (build fails on .NET 8 SDK) but the actual error is NETSDK1045 (cannot target net9.0), surfaced before the compiler runs. The original assumption predicted CS1003/CS8400 parser errors on the extension(...) syntax — those would only appear if the multi-target was reduced to net8.0 only.
• The existing test UseAzureServiceBusTransport_replaces_existing_transport (line 116-126 of AzureServiceBusExtensionsTests.cs) codifies the U11 footgun as if it were correct behavior. Phase 3 should reconsider.

Test plan

• dotnet build tests/NetEvolve.Pulse.Tests.Unit/NetEvolve.Pulse.Tests.Unit.csproj — 0 errors (12 style warnings only).
• Phase 3 builders run the three new TUnit tests; each should fail before the fix and pass after.
• Phase 3 runs pwsh audit/repros/u13/verify.ps1 — must exit 0 after fix.
• Phase 3 runs pwsh audit/repros/u14/verify.ps1 — must exit 0 after fix (i.e., build succeeds with .NET 8 SDK pinned).

🤖 Generated with Claude Code

[coderabbitai]

Important

Review skipped

Auto reviews are limited based on label configuration.

🏷️ Required labels (at least one) (1)

• state:ready for merge

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f6e19125-8f45-4d94-8a9a-d605d0a21fd1

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}