Skip to content

Revert to using fancier reporter except in slow terminals - #1212

Merged
Elizabeth Craig (ecraig12345) merged 3 commits into
microsoft:mainfrom
ecraig12345:default-reporter
Aug 17, 2026
Merged

Revert to using fancier reporter except in slow terminals#1212
Elizabeth Craig (ecraig12345) merged 3 commits into
microsoft:mainfrom
ecraig12345:default-reporter

Conversation

@ecraig12345

Copy link
Copy Markdown
Member

#967 added a basic progress reporter which is very fast but doesn't show the running task names, which can be a source of confusion if a task is slow or entirely hangs (there's no way to see what is hanging).

This PR reverts to using the fancier progress reporter by default, except in known slow terminals (codespaces, SSH). This should be okay in most cases after #1097 which updated @ms-cloudpack/task-reporter to pick up significant perf improvements.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Restores ProgressReporter as the default for interactive local runs while retaining BasicReporter for slower terminals.

Changes:

  • Adds reporter selection based on terminal environment.
  • Exposes basic as a reporter option and updates tests/docs.
  • Removes the unused trace-reporter concurrency option.

Reviewed changes

Copilot reviewed 15 out of 17 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
yarn.lock Updates transitive dependencies.
packages/reporters/src/ProgressReporter.ts Updates reporter documentation.
packages/reporters/src/ChromeTraceEventsReporter.ts Removes unused concurrency option.
packages/reporters/src/__tests__/ChromeTraceEventsReporter.test.ts Updates constructor usage.
packages/reporters/etc/reporters.api.md Updates the public API report.
packages/config/src/types/ConfigOptions.ts Clarifies reporter precedence.
packages/cli/src/types/ReporterInitOptions.ts Adds the basic reporter name.
packages/cli/src/commands/initializeReporters.ts Simplifies reporter-option normalization.
packages/cli/src/commands/createReporter.ts Implements environment-based defaults.
packages/cli/src/__tests__/initializeReporters.test.ts Updates the default-reporter test.
packages/cli/src/__tests__/customReporter.test.ts Updates supported-reporter output.
packages/cli/etc/cli.api.md Records the new reporter name.
package.json Removes verbose output from root tests.
lage.config.js Removes the local reporter override.
docs/yarn.lock Updates documentation dependencies.
docs/docs/reference/cli.md Documents reporter behavior.
change/change-ad6a24c6-619b-43b1-9369-b9cca461cac8.json Adds release metadata.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread packages/cli/src/commands/createReporter.ts
Comment thread packages/cli/src/__tests__/initializeReporters.test.ts
Comment thread docs/docs/reference/cli.md Outdated
@ecraig12345
Elizabeth Craig (ecraig12345) merged commit e69b146 into microsoft:main Aug 17, 2026
15 checks passed
@ecraig12345
Elizabeth Craig (ecraig12345) deleted the default-reporter branch August 18, 2026 01:58
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