Skip to content

Bound lifecycle log collection work - #3331

Merged
Widthdom merged 3 commits into
mainfrom
fix-issue3026-3027-3028
Jun 6, 2026
Merged

Bound lifecycle log collection work#3331
Widthdom merged 3 commits into
mainfrom
fix-issue3026-3027-3028

Conversation

@Widthdom

@Widthdom Widthdom commented Jun 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • Keep cdidx report lifecycle log-tail collection to a bounded recent-file candidate set instead of sorting/materializing every stderr-*.log file.
  • Cap best-effort private log permission hardening during startup.
  • Prune private logs with a bounded retained-file set, then stream-delete older files without sorting/materializing all matches.

Validation

  • dotnet build CodeIndex.sln -p:UseSharedCompilation=false
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~ReportCommandRunnerTests" -p:UseSharedCompilation=false
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~GlobalToolLogTests" -p:UseSharedCompilation=false
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~McpServerTests.ProcessLineAsync_CapsTelemetryArgumentKeyCount_Issue3237" -p:UseSharedCompilation=false
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet format CodeIndex.sln --verify-no-changes
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll status --check --json

Full-suite note: dotnet test CodeIndex.sln -p:UseSharedCompilation=false produced one unrelated MCP telemetry test failure during this work; the targeted rerun passed and the follow-up is tracked in #3325.

Documentation / Changelog

  • Added changelog.d/unreleased/3026.fixed.md.
  • Added changelog.d/unreleased/3027.fixed.md.
  • Added changelog.d/unreleased/3028.fixed.md.
  • Updated USER_GUIDE.md and generated report README text to document the 32-file report log-tail candidate cap.
  • No CHANGELOG.md edit; this uses the unreleased fragment workflow.

Review

Adversarial review found that the new report log-tail candidate cap was not documented in generated/user-facing report text. That was fixed in the #3026 commit; no remaining blocking/actionable issues were found.

codex exec review --base origin/main --ephemeral was attempted, but the nested review could not complete because its internal repository-read/tool calls failed repeatedly. A read-only Codex adversarial review followed .codex/workflows/adversarial-review.md against origin/main..HEAD.

Follow-up

Fixes #3026
Fixes #3027
Fixes #3028

@Widthdom
Widthdom force-pushed the fix-issue3026-3027-3028 branch from 73ea875 to 40947ca Compare June 6, 2026 12:54
@Widthdom
Widthdom force-pushed the fix-issue3026-3027-3028 branch from 40947ca to e5dca89 Compare June 6, 2026 13:23
@Widthdom
Widthdom marked this pull request as ready for review June 6, 2026 14:07
@Widthdom
Widthdom merged commit 267a1b0 into main Jun 6, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue3026-3027-3028 branch June 6, 2026 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant