Skip to content

Fix spinner delay async wait - #2625

Merged
Widthdom merged 4 commits into
mainfrom
codex/fix-issue1421
May 24, 2026
Merged

Fix spinner delay async wait#2625
Widthdom merged 4 commits into
mainfrom
codex/fix-issue1421

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

  • Replaced the spinner loop's synchronous Task.Delay(...).Wait(...) with an awaited delay inside the background task.
  • Added regression coverage that checks StartSpinner-related generated methods do not call Task.Wait.
  • Added bilingual changelog fragment changelog.d/unreleased/1421.fixed.md.

Validation

  • dotnet build CodeIndex.sln -c Release -p:UseSharedCompilation=false
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj -c Release --filter ConsoleUiTests -p:UseSharedCompilation=false
  • dotnet run --project tools/CodeIndex.Changelog/CodeIndex.Changelog.csproj -- check
  • Adversarial review: No blocking/actionable issues found.

Full-suite note: dotnet test CodeIndex.sln -c Release -p:UseSharedCompilation=false was attempted before the latest origin/main merge and surfaced an unrelated existing failure in SymbolExtractorTests.Extract_RustTraitAssociatedTypeDefaults_RecordsPropertySymbols on both net8/net9; the run was not used as final validation for this scoped fix.

Documentation and Changelog

  • Changelog fragment: changelog.d/unreleased/1421.fixed.md
  • No README or guide updates were required because the CLI surface and documented command contract did not change.

Fixes #1421

@Widthdom
Widthdom merged commit 941313d into main May 24, 2026
10 checks passed
@Widthdom
Widthdom deleted the codex/fix-issue1421 branch May 24, 2026 11:33
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.

ConsoleUi.StartSpinner blocks a Task.Run worker with Task.Delay().Wait() (sync-over-async)

1 participant