Skip to content

Split focused CLI command runner modules - #3647

Merged
Widthdom merged 2 commits into
mainfrom
fix-issue3380-3418
Jun 14, 2026
Merged

Split focused CLI command runner modules#3647
Widthdom merged 2 commits into
mainfrom
fix-issue3380-3418

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

  • Extracted query batch-mode handling from QueryCommandRunner into QueryCommandRunner.Batch.
  • Extracted ProgramRunner dispatch/helper command handling into ProgramRunner.Dispatch.
  • Added unreleased internal changelog fragments for both issues.

Validation

  • dotnet build CodeIndex.sln -c Debug -p:UseSharedCompilation=false
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj -c Debug --filter "FullyQualifiedName~QueryCommandRunnerTests.RunBatch" -p:UseSharedCompilation=false
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj -c Debug --filter "FullyQualifiedName~ProgramRunnerTests" -p:UseSharedCompilation=false
  • dotnet test CodeIndex.sln -c Release -p:UseSharedCompilation=false
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • cdidx index . --files src/CodeIndex/Cli/QueryCommandRunner.cs src/CodeIndex/Cli/QueryCommandRunner.Batch.cs changelog.d/unreleased/3380.internal.md --json
  • cdidx index . --files src/CodeIndex/Cli/ProgramRunner.cs src/CodeIndex/Cli/ProgramRunner.Dispatch.cs changelog.d/unreleased/3418.internal.md --json
  • cdidx index . --commits 9f96e71d7 --json
  • cdidx index . --commits a677391bb --json
  • Codex adversarial review: no blocking/actionable issues found.

Changelog

  • changelog.d/unreleased/3380.internal.md
  • changelog.d/unreleased/3418.internal.md

Fixes #3380
Fixes #3418

@Widthdom
Widthdom marked this pull request as ready for review June 14, 2026 22:38
@Widthdom
Widthdom merged commit 74485ea into main Jun 14, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue3380-3418 branch June 14, 2026 23:06
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.

ProgramRunner should be split into focused command modules QueryCommandRunner should be split into smaller focused modules

1 participant