Skip to content

Fix console output synchronization - #2615

Merged
Widthdom merged 1 commit into
mainfrom
fix-issue1734
May 24, 2026
Merged

Fix console output synchronization#2615
Widthdom merged 1 commit into
mainfrom
fix-issue1734

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

  • Synchronize Console.Out and Console.Error at CLI startup with TextWriter.Synchronized.
  • Ensure spinner startup also synchronizes captured/redirected writers before fallback output.
  • Add console regression coverage for concurrent whole-string writes and redirected spinner output.

Validation

  • dotnet build CodeIndex.sln
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter FullyQualifiedName~ConsoleUiTests
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter ConsoleUiTests --no-restore (adversarial review)
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll index . --commits fd8f40485bfd82f920e925799cc6ef53e0a9fbb9 --json

Full dotnet test CodeIndex.sln was attempted but failed in existing Rust associated-type-default coverage (SymbolExtractorTests.Extract_RustTraitAssociatedTypeDefaults_RecordsPropertySymbols) and then did not terminate promptly; open related issues already exist: #2603, #2609, #2611.

Documentation and Changelog

  • Added changelog fragment: changelog.d/unreleased/1734.fixed.md
  • No AGENTS.md, CLAUDE.md, or AGENT_GUIDE.md changes.

Follow-up Candidates

Fixes #1734

@Widthdom
Widthdom merged commit 482464a into main May 24, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue1734 branch May 24, 2026 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Spinner Task and main thread Console.Write interleave producing mangled output

1 participant