Skip to content

Conversation

@BioPhoton
Copy link
Collaborator

@BioPhoton BioPhoton commented Dec 2, 2025

Followup of #1163
Precondition of #1188

This PR includes:

  • create nx plugin under zod2md-jsdocs
  • logic to add target config if file matches zod2md.config.ts
  • plugin docs
  • remove old config
  • remove file reference from production inputs as changes in the config may impact build output

A potential followup PR could implement a nx plugin for our build target and add dependsOn settings automatically. This is also noted in a clean up issue.

Note

This is tooling code and it is not planned to get released

Related:
#1168

@github-actions github-actions bot added 📖 Project documentation improvements or additions to the project documentation 🛠️ tooling labels Dec 2, 2025
@nx-cloud
Copy link

nx-cloud bot commented Dec 2, 2025

View your CI Pipeline Execution ↗ for commit e88d62d

Command Status Duration Result
nx run-many --target=build --parallel=3 ✅ Succeeded 33s View ↗

☁️ Nx Cloud last updated this comment at 2025-12-16 01:09:28 UTC

@pkg-pr-new
Copy link

pkg-pr-new bot commented Dec 2, 2025

Open in StackBlitz

@code-pushup/ci

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

@code-pushup/cli

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

@code-pushup/core

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

@code-pushup/create-cli

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

@code-pushup/nx-plugin

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

@code-pushup/models

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

@code-pushup/axe-plugin

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

@code-pushup/coverage-plugin

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

@code-pushup/js-packages-plugin

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

@code-pushup/eslint-plugin

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

@code-pushup/jsdocs-plugin

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

@code-pushup/lighthouse-plugin

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

@code-pushup/typescript-plugin

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

@code-pushup/utils

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

commit: e88d62d

@github-actions
Copy link
Contributor

github-actions bot commented Dec 2, 2025

Code PushUp

🤨 Code PushUp report has both improvements and regressions – compared current commit 3f6c4a2 with previous commit d1e8192.

🕵️ See full comparison in Code PushUp portal 🔍

🏷️ Categories

🏷️ Category ⭐ Previous score ⭐ Current score 🔄 Score change
Security 🔴 43 🟡 67 ↑ +24.4
Performance 🔴 35 🔴 41 ↑ +6
Code style 🟢 99 🟢 100 ↑ +0.4
Updates 🟡 71 🟡 71 ↑ +0.2
Code coverage 🟡 89 🟡 89 ↓ −0.1
Axe Accessibility 🟡 88 🟡 88 ↓ −0.1
Bug prevention 🟡 67 🟡 67
Type Safety 🟡 56 🟡 56
Miscellaneous 🟡 67 🟡 67
Documentation 🔴 35 🔴 35
Accessibility 🟢 92 🟢 92
Best Practices 🟢 100 🟢 100
SEO 🟢 92 🟢 92
👍 5 groups improved, 👎 1 group regressed, 👍 8 audits improved, 👎 3 audits regressed, 12 audits changed without impacting score

🗃️ Groups

🔌 Plugin 🗃️ Group ⭐ Previous score ⭐ Current score 🔄 Score change
JS Packages NPM audit 🔴 43 🟡 67 ↑ +24.4
ESLint Style guide (import) 🟡 80 🟢 100 ↑ +20
Lighthouse Performance 🔴 35 🔴 41 ↑ +6
ESLint Suggestions 🟢 99 🟢 100 ↑ +0.4
JS Packages NPM outdated dependencies 🟡 71 🟡 71 ↑ +0.2
Code coverage Code coverage metrics 🟡 89 🟡 89 ↓ −0.1

28 other groups are unchanged.

🛡️ Audits

🔌 Plugin 🛡️ Audit 📏 Previous value 📏 Current value 🔄 Value change
Lighthouse Initial server response time was short 🟥 Root document took 710 ms 🟩 Root document took 520 ms ↓ −26.2 %
ESLint Forbid unassigned imports 🟥 1 warning 🟩 passed ↓ −100 %
JS Packages Vulnerabilities for NPM prod dependencies. 🟨 11 vulnerabilities (3 high, 1 moderate, 7 low) 🟨 6 vulnerabilities (1 high, 5 low) ↓ −45.5 %
Lighthouse Speed Index 🟥 7.4 s 🟥 6.3 s ↓ −15.5 %
Lighthouse Total Blocking Time 🟥 1,830 ms 🟥 1,140 ms ↓ −37.9 %
Lighthouse First Contentful Paint 🟥 3.3 s 🟥 3.1 s ↓ −6.3 %
Lighthouse Time to Interactive 🟥 13.1 s 🟥 12.3 s ↓ −5.9 %
JS Packages Outdated NPM dev dependencies. 🟨 60 outdated package versions (35 major, 19 minor, 6 patch) 🟨 59 outdated package versions (34 major, 19 minor, 6 patch) ↓ −1.7 %
Code coverage Line coverage 🟨 84.9 % 🟨 84.6 % ↓ −0.4 %
Code coverage Function coverage 🟩 90.6 % 🟩 90.6 % ↓ −0.1 %
Code coverage Branch coverage 🟨 87.3 % 🟨 87.2 % ↓ −0.1 %
Lighthouse Avoids enormous network payloads 🟩 Total size was 2,059 KiB 🟩 Total size was 2,099 KiB ↑ +2 %
Lighthouse Minimizes main-thread work 🟥 10.3 s 🟥 8.1 s ↓ −21.8 %
Lighthouse Largest Contentful Paint 🟥 11.9 s 🟥 10.8 s ↓ −8.8 %
Lighthouse Metrics 🟩 100% 🟩 100% ↓ −5.9 %
Lighthouse Server Backend Latencies 🟩 1,780 ms 🟩 1,010 ms ↓ −43.3 %
Lighthouse JavaScript execution time 🟥 3.9 s 🟥 3.4 s ↓ −11.5 %
Lighthouse Max Potential First Input Delay 🟥 1,260 ms 🟥 940 ms ↓ −25.1 %
Lighthouse Reduce unused JavaScript 🟥 Potential savings of 187 KiB 🟥 Potential savings of 189 KiB ↓ −15.1 %
Lighthouse Uses efficient cache policy on static assets 🟨 30 resources found 🟨 30 resources found ↓ −0.1 %
Lighthouse Network Round Trip Times 🟩 70 ms 🟩 20 ms ↓ −76.1 %
JS Packages Outdated NPM prod dependencies. 🟨 18 outdated package versions (7 major, 8 minor, 3 patch) 🟨 15 outdated package versions (7 major, 5 minor, 3 patch) ↓ −16.7 %
JS Packages Vulnerabilities for NPM dev dependencies. 🟥 30 vulnerabilities (1 critical, 6 high, 18 moderate, 5 low) 🟥 32 vulnerabilities (1 critical, 6 high, 19 moderate, 6 low) ↑ +6.7 %

656 other audits are unchanged.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 2, 2025

Code PushUp

😟 Code PushUp report has regressed – compared current commit 3f6c4a2 with previous commit d1e8192.

💼 Project models

😟 Code PushUp report has regressed.

🕵️ See full comparison in Code PushUp portal 🔍

🏷️ Category ⭐ Previous score ⭐ Current score 🔄 Score change
Code coverage 🟢 93 🟢 93 ↓ −0.1

3 other categories are unchanged.

👎 1 group regressed, 👎 1 audit regressed, 1 audit changed without impacting score

🗃️ Groups

🔌 Plugin 🗃️ Group ⭐ Previous score ⭐ Current score 🔄 Score change
Code coverage Code coverage metrics 🟢 93 🟢 93 ↓ −0.1

11 other groups are unchanged.

🛡️ Audits

🔌 Plugin 🛡️ Audit 📏 Previous value 📏 Current value 🔄 Value change
Code coverage Branch coverage 🟩 91.5 % 🟩 91.4 % ↓ −0.1 %
Code coverage Line coverage 🟩 97.2 % 🟩 97.2 % ↑ +0.1 %

435 other audits are unchanged.


13 other projects are unchanged.

@BioPhoton BioPhoton marked this pull request as ready for review December 2, 2025 23:41
@BioPhoton BioPhoton requested a review from matejchalk December 2, 2025 23:41
@BioPhoton
Copy link
Collaborator Author

BioPhoton commented Dec 16, 2025

After switching to .ts format for the Nx plugin, the graph stopped computing because of NX invalid type: unit value, expected struct Options.
I researched and found out package updates help so I updated nx packages.

After the update errors popped up that i could not fully relate to the changes:

@matejchalk could you check this changes pls.

@BioPhoton BioPhoton requested a review from matejchalk December 16, 2025 01:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants