Skip to content

feat: support plain text reporter and use seconds#714

Merged
XGHeaven merged 1 commit intonextfrom
feat/support-normal-reporter
Apr 7, 2026
Merged

feat: support plain text reporter and use seconds#714
XGHeaven merged 1 commit intonextfrom
feat/support-normal-reporter

Conversation

@XGHeaven
Copy link
Copy Markdown
Collaborator

@XGHeaven XGHeaven commented Apr 7, 2026

No description provided.

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new bot commented Apr 7, 2026

More templates

@ice/create-pkg

npm i https://pkg.pr.new/ice-lab/icepkg/@ice/create-pkg@714

ice-npm-utils

npm i https://pkg.pr.new/ice-lab/icepkg/ice-npm-utils@714

@ice/pkg

npm i https://pkg.pr.new/ice-lab/icepkg/@ice/pkg@714

@ice/pkg-plugin-jsx-plus

npm i https://pkg.pr.new/ice-lab/icepkg/@ice/pkg-plugin-jsx-plus@714

@ice/pkg-plugin-mf

npm i https://pkg.pr.new/ice-lab/icepkg/@ice/pkg-plugin-mf@714

commit: f892d36

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates @ice/pkg runner reporting to better suit non-interactive environments by adding a plain-text reporter, and standardizes time cost display to seconds across reporter output.

Changes:

  • Switch time cost formatting from milliseconds to seconds and introduce an option to disable coloring.
  • Add a RunnerPlainTextReporter and auto-select between terminal vs plain-text reporters based on TTY/CI.
  • Update build and start commands to use the new reporter factory.

Reviewed changes

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

Show a summary per file
File Description
packages/pkg/src/utils.ts Changes time formatting utilities to output seconds and adds a non-colored mode.
packages/pkg/src/helpers/runnerReporter.ts Adds plain-text reporter and reporter factory; refactors options typing.
packages/pkg/src/commands/start.ts Uses createRunnerReporter() instead of directly instantiating the terminal reporter.
packages/pkg/src/commands/build.ts Uses createRunnerReporter() instead of directly instantiating the terminal reporter.
.changeset/calm-clocks-hang.md Declares a minor bump for the new reporter/time formatting behavior.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@XGHeaven XGHeaven force-pushed the feat/support-normal-reporter branch from 3e830d9 to 886fdc0 Compare April 7, 2026 07:21
@XGHeaven XGHeaven requested a review from Copilot April 7, 2026 07:21
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

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


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@XGHeaven XGHeaven force-pushed the feat/support-normal-reporter branch from 886fdc0 to 595a3a5 Compare April 7, 2026 07:45
@XGHeaven XGHeaven requested a review from Copilot April 7, 2026 07:56
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@XGHeaven XGHeaven force-pushed the feat/support-normal-reporter branch from 595a3a5 to f892d36 Compare April 7, 2026 08:36
@XGHeaven XGHeaven requested a review from Copilot April 7, 2026 08:36
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

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


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@XGHeaven XGHeaven merged commit 6224edc into next Apr 7, 2026
7 checks passed
@XGHeaven XGHeaven deleted the feat/support-normal-reporter branch April 7, 2026 09:04
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