McpAuditLog request-id truncation regression test writes no audit event

[Widthdom]

Summary

After merging current origin/main into fix-issue3045, the existing regression test McpAuditLogTests.ToolsCall_TruncatesAuditRequestId_Issue3237 fails deterministically because the audit log file contains no event records.

Reproduction

dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter FullyQualifiedName~McpAuditLogTests.ToolsCall_TruncatesAuditRequestId_Issue3237

Observed

Both net8.0 and net9.0 fail with:

Assert.Single() Failure: The collection was empty
at CodeIndex.Tests.McpAuditLogTests.ReadOnlyRecord() ... McpAuditLogTests.cs:line 626
at CodeIndex.Tests.McpAuditLogTests.ToolsCall_TruncatesAuditRequestId_Issue3237() ... McpAuditLogTests.cs:line 486

Expected

The test should write one audit event with a truncated request_id field, then assert the truncation metadata.

[Widthdom]

Work started on branch fix-issue3052 because this failure is currently blocking PR #3305's pipeline.

[Widthdom]

Resolved by the final merged state of #3305.

During conflict resolution, the direct #3304 changelog/body reference was removed because the latest origin/main already carried the request-id bound changes needed for McpAuditLogTests.ToolsCall_TruncatesAuditRequestId_Issue3237 to pass. The merged PR #3305 was validated with the targeted audit test and full test run before merge, so this issue is completed.

[Widthdom]

After merging the latest origin/main into fix-issue3045, dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~McpAuditLogTests.ToolsCall_TruncatesAuditRequestId_Issue3237" passes on both net8.0 and net9.0, and the full dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj run also passes. This is no longer reproducible on current main, so I’m closing it.