Skip to content

feat(cli,core): improve logs #1174

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
matejchalk merged 7 commits into from
Dec 9, 2025
Merged

feat(cli,core): improve logs #1174

matejchalk merged 7 commits into from
Dec 9, 2025

Conversation

@matejchalk
Copy link
Collaborator

@matejchalk matejchalk commented Dec 8, 2025 •
edited
Loading

Part of #888

Improves various logs from the core CLI execution:

  • consistent header with our logo and version number
  • log group configuration resolution
  • log spinners for slow async tasks - importing config, uploading to portal
  • logs each plugin's cache usage

Examples

CI logs

No upload, plugin cache

screen-capture.-.2025-12-08T175500.746.webm

Upload

screen-capture.-.2025-12-08T180419.929.webm

@nx-cloud
Copy link

nx-cloud bot commented Dec 8, 2025 •
edited
Loading

View your CI Pipeline Execution ↗ for commit 1976457

Command Status Duration Result
nx run ci:code-pushup -- merge-diffs --files=/h... ✅ Succeeded 4s View ↗
nx run-many --targets=code-pushup --parallel=fa... ✅ Succeeded 1m 17s View ↗
nx run-many --targets=code-pushup --parallel=fa... ✅ Succeeded 11m 21s View ↗
nx run-many -t unit-test,int-test ✅ Succeeded 8s View ↗

☁️ Nx Cloud last updated this comment at 2025-12-08 18:15:08 UTC

@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 8, 2025 •
edited
Loading

Open in StackBlitz

@code-pushup/ci

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/ci@1174

@code-pushup/cli

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/cli@1174

@code-pushup/core

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/core@1174

@code-pushup/create-cli

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/create-cli@1174

@code-pushup/models

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/models@1174

@code-pushup/nx-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/nx-plugin@1174

@code-pushup/axe-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/axe-plugin@1174

@code-pushup/coverage-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/coverage-plugin@1174

@code-pushup/eslint-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/eslint-plugin@1174

@code-pushup/js-packages-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/js-packages-plugin@1174

@code-pushup/jsdocs-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/jsdocs-plugin@1174

@code-pushup/lighthouse-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/lighthouse-plugin@1174

@code-pushup/typescript-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/typescript-plugin@1174

@code-pushup/utils

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/utils@1174

commit: 1976457

@github-actions
Copy link
Contributor

github-actions bot commented Dec 8, 2025 •
edited
Loading

Code PushUp

🤨 Code PushUp report has both improvements and regressions – compared current commit b5ec3b1 with previous commit eb23ed4.

🕵️ See full comparison in Code PushUp portal 🔍

🏷️ Categories

🏷️ Category ⭐ Previous score ⭐ Current score 🔄 Score change
Code coverage 🟡 89 🟡 89 ↑ +0.1
Performance 🔴 45 🔴 45 ↓ −0.1
Documentation 🔴 35 🔴 35 ↑ +0.1
Bug prevention 🟡 67 🟡 67 –
Code style 🟢 100 🟢 100 –
Security 🔴 43 🔴 43 –
Updates 🟡 71 🟡 71 –
Type Safety 🟡 56 🟡 56 –
Miscellaneous 🟡 67 🟡 67 –
Accessibility 🟢 92 🟢 92 –
Best Practices 🟢 100 🟢 100 –
SEO 🟢 92 🟢 92 –
👍 2 groups improved, 👎 1 group regressed, 👍 9 audits improved, 👎 1 audit regressed, 14 audits changed without impacting score

🗃️ Groups

🔌 Plugin 🗃️ Group ⭐ Previous score ⭐ Current score 🔄 Score change
Code coverage Code coverage metrics 🟡 89 🟡 89 ↑ +0.1
Lighthouse Performance 🔴 45 🔴 45 ↓ −0.1
JSDoc coverage Documentation coverage 🔴 35 🔴 35 ↑ +0.1

31 other groups are unchanged.

🛡️ Audits

🔌 Plugin 🛡️ Audit 📏 Previous value 📏 Current value 🔄 Value change
Lighthouse Initial server response time was short 🟥 Root document took 690 ms 🟩 Root document took 430 ms ↓ −38 %
Lighthouse Speed Index 🟥 6.2 s 🟥 5.9 s ↓ −3.8 %
Lighthouse First Contentful Paint 🟥 3.1 s 🟥 3.0 s ↓ −1.9 %
Lighthouse Total Blocking Time 🟥 820 ms 🟥 850 ms ↑ +4.4 %
Lighthouse Time to Interactive 🟥 12.6 s 🟥 12.2 s ↓ −3.2 %
Code coverage Function coverage 🟩 90.3 % 🟩 90.4 % ↑ +0.2 %
Code coverage Line coverage 🟨 84.7 % 🟨 84.7 % ↑ +0.1 %
JSDoc coverage Functions coverage 🟥 560 undocumented functions 🟥 556 undocumented functions ↓ −0.7 %
JSDoc coverage Types coverage 🟥 245 undocumented types 🟥 243 undocumented types ↓ −0.8 %
Code coverage Branch coverage 🟨 87.1 % 🟨 87.2 % ↑ +0.1 %
Lighthouse Avoids enormous network payloads 🟩 Total size was 2,013 KiB 🟩 Total size was 2,003 KiB ↓ −0.5 %
Lighthouse Metrics 🟩 100% 🟩 100% ↓ −3.2 %
Lighthouse Largest Contentful Paint 🟥 10.7 s 🟥 10.4 s ↓ −2.6 %
Lighthouse Uses efficient cache policy on static assets 🟨 30 resources found 🟨 30 resources found ↓ −0.1 %
Lighthouse Reduce unused JavaScript 🟥 Potential savings of 187 KiB 🟥 Potential savings of 186 KiB ↑ +34.2 %
Lighthouse Minimizes main-thread work 🟥 7.5 s 🟥 7.6 s ↑ +0.9 %
Lighthouse Server Backend Latencies 🟩 240 ms 🟩 190 ms ↓ −19.6 %
Lighthouse JavaScript execution time 🟥 3.1 s 🟥 3.1 s ↓ −1.5 %
Lighthouse Reduce unused CSS 🟥 Potential savings of 105 KiB 🟥 Potential savings of 105 KiB ↓ −6.2 %
Lighthouse Remove duplicate modules in JavaScript bundles 🟥 Potential savings of 104 KiB 🟥 Potential savings of 102 KiB ↓ −6.2 %
Lighthouse Max Potential First Input Delay 🟥 840 ms 🟥 820 ms ↓ −2.1 %
Lighthouse Network Round Trip Times 🟩 20 ms 🟩 10 ms ↓ −62.8 %
TypeScript Semantic errors 🟥 2623 errors 🟥 2628 errors ↑ +0.2 %
TypeScript No implicit any errors 🟥 96 errors 🟥 97 errors ↑ +1 %

654 other audits are unchanged.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 8, 2025 •
edited
Loading

Code PushUp

🤨 Code PushUp report has both improvements and regressions – compared current commit b5ec3b1 with previous commit eb23ed4.

💼 Project cli

🥳 Code PushUp report has improved.

🕵️ See full comparison in Code PushUp portal 🔍

🏷️ Category ⭐ Previous score ⭐ Current score 🔄 Score change
Code coverage 🟢 92 🟢 93 ↑ +1
Documentation 🟡 65 🟡 65 ↑ +0.1

4 other categories are unchanged.

👍 2 groups improved, 👍 4 audits improved

🗃️ Groups

🔌 Plugin 🗃️ Group ⭐ Previous score ⭐ Current score 🔄 Score change
Code coverage Code coverage metrics 🟢 92 🟢 93 ↑ +1
JSDoc coverage Documentation coverage 🟡 65 🟡 65 ↑ +0.1

13 other groups are unchanged.

🛡️ Audits

🔌 Plugin 🛡️ Audit 📏 Previous value 📏 Current value 🔄 Value change
Code coverage Function coverage 🟩 95.9 % 🟩 97.3 % ↑ +1.5 %
Code coverage Line coverage 🟨 88 % 🟨 89.1 % ↑ +1.2 %
JSDoc coverage Functions coverage 🟥 17 undocumented functions 🟥 16 undocumented functions ↓ −5.9 %
Code coverage Branch coverage 🟨 84.5 % 🟨 84.6 % ↑ +0.1 %

439 other audits are unchanged.

💼 Project core

🤨 Code PushUp report has both improvements and regressions.

🕵️ See full comparison in Code PushUp portal 🔍

🏷️ Category ⭐ Previous score ⭐ Current score 🔄 Score change
Code coverage 🟡 89 🟡 90 ↑ +0.3
Documentation 🟡 74 🟡 74 ↓ −0.1

4 other categories are unchanged.

👍 1 group improved, 👎 1 group regressed, 👍 3 audits improved, 👎 1 audit regressed

🗃️ Groups

🔌 Plugin 🗃️ Group ⭐ Previous score ⭐ Current score 🔄 Score change
Code coverage Code coverage metrics 🟡 89 🟡 90 ↑ +0.3
JSDoc coverage Documentation coverage 🟡 74 🟡 74 ↓ −0.1

13 other groups are unchanged.

🛡️ Audits

🔌 Plugin 🛡️ Audit 📏 Previous value 📏 Current value 🔄 Value change
Code coverage Line coverage 🟩 90.5 % 🟩 91.2 % ↑ +0.8 %
JSDoc coverage Functions coverage 🟥 11 undocumented functions 🟥 12 undocumented functions ↑ +9.1 %
Code coverage Branch coverage 🟨 81.1 % 🟨 81.7 % ↑ +0.8 %
Code coverage Function coverage 🟩 93.1 % 🟩 93.2 % ↑ +0.1 %

439 other audits are unchanged.

💼 Project utils

🤨 Code PushUp report has both improvements and regressions.

🕵️ See full comparison in Code PushUp portal 🔍

🏷️ Category ⭐ Previous score ⭐ Current score 🔄 Score change
Code coverage 🟢 94 🟢 95 ↑ +0.1
Documentation 🔴 38 🔴 38 ↑ +0.1

4 other categories are unchanged.

👍 2 groups improved, 👍 3 audits improved, 👎 2 audits regressed

🗃️ Groups

🔌 Plugin 🗃️ Group ⭐ Previous score ⭐ Current score 🔄 Score change
Code coverage Code coverage metrics 🟢 94 🟢 95 ↑ +0.1
JSDoc coverage Documentation coverage 🔴 38 🔴 38 ↑ +0.1

13 other groups are unchanged.

🛡️ Audits

🔌 Plugin 🛡️ Audit 📏 Previous value 📏 Current value 🔄 Value change
JSDoc coverage Types coverage 🟥 47 undocumented types 🟥 45 undocumented types ↓ −4.3 %
Code coverage Function coverage 🟩 95.3 % 🟩 95.5 % ↑ +0.2 %
JSDoc coverage Functions coverage 🟥 236 undocumented functions 🟥 232 undocumented functions ↓ −1.7 %
Code coverage Line coverage 🟩 97 % 🟩 96.9 % ↓ −0.1 %
Code coverage Branch coverage 🟩 91.9 % 🟩 91.9 % ↓ −0.1 %

438 other audits are unchanged.

11 other projects are unchanged.

@matejchalk matejchalk marked this pull request as ready for review December 8, 2025 18:17
@matejchalk matejchalk requested a review from BioPhoton as a code owner December 8, 2025 18:17
@matejchalk matejchalk merged commit 8809684 into main Dec 9, 2025
24 checks passed
@matejchalk matejchalk deleted the improve-cli-logs branch December 9, 2025 08:51
@matejchalk matejchalk mentioned this pull request Dec 9, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vmasek vmasek vmasek approved these changes

@BioPhoton BioPhoton Awaiting requested review from BioPhoton BioPhoton is a code owner

@hanna-skryl hanna-skryl Awaiting requested review from hanna-skryl

Assignees

@matejchalk matejchalk

Labels

🧩 cli 🧩 core 🧩 coverage-plugin 🧩 lighthouse-plugin 📖 Project documentation improvements or additions to the project documentation 🔬 testing writing tests 🧩 utils

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@matejchalk @vmasek