Skip to content

Truncate discovery command log#84

Merged
anmarchenko merged 1 commit into
mainfrom
anmarchenko/truncate-discovery-command-log
Jun 15, 2026
Merged

Truncate discovery command log#84
anmarchenko merged 1 commit into
mainfrom
anmarchenko/truncate-discovery-command-log

Conversation

@anmarchenko

Copy link
Copy Markdown
Member

What

Truncate the test discovery command log value to 300 characters and log it as a single command field.

Why

When test discovery receives many explicit files, the logged argument list can become very long and noisy. Capping the logged command keeps the discovery log readable without changing the command that is actually executed.

E2E testing

Run ddtest plan in a project where the test discovery exclude pattern leaves many explicit test files, then verify the Discovering tests with command log line is capped at 300 characters and discovery still completes.

Validation run:

  • go test ./internal/discovery
  • make test
  • make lint

@anmarchenko anmarchenko marked this pull request as ready for review June 15, 2026 10:26
@anmarchenko anmarchenko requested a review from a team as a code owner June 15, 2026 10:26
@anmarchenko anmarchenko changed the title [codex] truncate discovery command log truncate discovery command log Jun 15, 2026
@anmarchenko anmarchenko changed the title truncate discovery command log Truncate discovery command log Jun 15, 2026
@anmarchenko anmarchenko merged commit 0862196 into main Jun 15, 2026
4 checks passed
@anmarchenko anmarchenko deleted the anmarchenko/truncate-discovery-command-log branch June 15, 2026 10:31
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.

2 participants