Skip to content

Conversation

@hanna-skryl
Copy link
Collaborator

@hanna-skryl hanna-skryl commented Jan 24, 2026

Partially addresses #677 by extending the model for issue location with sourceUrlLocationSchema.

Models:

  • Add sourceUrlLocationSchema
  • Create issueSourceSchema union type combining file and URL sources

Utils:

  • Add type guards
  • Rename issue table columns: "Source file" -> "Source", "Line(s)" -> "Location"
  • Add formatSelectorLocation to display only the target element in the location column to avoid verbosity

Axe plugin:

  • Populate the source field with a URL, an HTML snippet, and a CSS selector

Core/CI:

  • Handle the union type in report processing

Out of scope:

  • CLI upload: URL source data is not uploaded to the backend. The issueToGQL function includes source details only for file-based issues. Uploading URL sources requires changes to the Portal API first.
  • Portal API: Needs new fields to store URL-source data and display the full selector in the issues table (e.g., via a tooltip)

Note

The selector is temporarily duplicated in the Axe issue message (e.g., [`img`] Fix this...) for Portal backward compatibility until it supports URL source display.

@hanna-skryl hanna-skryl self-assigned this Jan 24, 2026
@nx-cloud
Copy link

nx-cloud bot commented Jan 24, 2026

View your CI Pipeline Execution ↗ for commit 2b58226

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 15s View ↗
nx run-many --targets=code-pushup --parallel=fa... ✅ Succeeded 10m 31s View ↗
nx run-many -t unit-test,int-test ✅ Succeeded 10s View ↗

☁️ Nx Cloud last updated this comment at 2026-01-26 17:04:55 UTC

@pkg-pr-new
Copy link

pkg-pr-new bot commented Jan 24, 2026

Open in StackBlitz

@code-pushup/ci

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

@code-pushup/cli

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

@code-pushup/core

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

@code-pushup/create-cli

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

@code-pushup/models

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

@code-pushup/nx-plugin

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

@code-pushup/axe-plugin

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

@code-pushup/coverage-plugin

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

@code-pushup/eslint-plugin

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

@code-pushup/js-packages-plugin

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

@code-pushup/jsdocs-plugin

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

@code-pushup/lighthouse-plugin

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

@code-pushup/typescript-plugin

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

@code-pushup/utils

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

commit: 2b58226

@github-actions
Copy link
Contributor

github-actions bot commented Jan 24, 2026

Code PushUp

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

🕵️ See full comparison in Code PushUp portal 🔍

🏷️ Categories

🏷️ Category ⭐ Previous score ⭐ Current score 🔄 Score change
Performance 🔴 32 🔴 29 ↓ −2.4
Code coverage 🟢 92 🟢 92 ↓ −0.1
Documentation 🔴 42 🔴 42 ↑ +0.1
Bug prevention 🟡 75 🟡 75 ↓ −0.1
Axe Accessibility 🟡 88 🟡 88 ↓ −0.1
Code style 🟢 100 🟢 100
Security 🔴 47 🔴 47
Updates 🟡 78 🟡 78
Type Safety 🟡 67 🟡 67
Miscellaneous 🟡 67 🟡 67
Accessibility 🟢 92 🟢 92
Best Practices 🟢 100 🟢 100
SEO 🟢 92 🟢 92
👍 1 group improved, 👎 2 groups regressed, 👍 3 audits improved, 👎 8 audits regressed, 11 audits changed without impacting score

🗃️ Groups

🔌 Plugin 🗃️ Group ⭐ Previous score ⭐ Current score 🔄 Score change
Lighthouse Performance 🔴 32 🔴 29 ↓ −2.4
Code coverage Code coverage metrics 🟢 92 🟢 92 ↓ −0.1
JSDocs coverage Documentation coverage 🔴 42 🔴 42 ↑ +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 510 ms 🟥 Root document took 700 ms ↑ +37.8 %
Lighthouse First Contentful Paint 🟥 3.4 s 🟥 3.9 s ↑ +14.9 %
Lighthouse Total Blocking Time 🟥 2,610 ms 🟥 4,520 ms ↑ +73 %
Lighthouse Time to Interactive 🟥 14.4 s 🟥 15.8 s ↑ +9.6 %
Lighthouse Speed Index 🟥 8.6 s 🟥 8.5 s ↓ −1.3 %
JSDocs coverage Functions coverage 🟥 577 undocumented functions 🟥 579 undocumented functions ↑ +0.3 %
JSDocs coverage Types coverage 🟥 268 undocumented types 🟥 272 undocumented types ↑ +1.5 %
Code coverage Function coverage 🟩 94.3 % 🟩 94.1 % ↓ −0.2 %
JSDocs coverage Variables coverage 🟥 268 undocumented variables 🟥 272 undocumented variables ↑ +1.5 %
Code coverage Line coverage 🟩 92.3 % 🟩 92.3 % ↑ +0.1 %
Code coverage Branch coverage 🟨 89 % 🟨 89 % ↓ −0.1 %
Lighthouse Minimizes main-thread work 🟥 12.7 s 🟥 17.7 s ↑ +39.2 %
Lighthouse JavaScript execution time 🟥 5.0 s 🟥 8.3 s ↑ +66 %
Lighthouse Metrics 🟩 100% 🟩 100% ↑ +9.6 %
Lighthouse Largest Contentful Paint 🟥 12.6 s 🟥 13.9 s ↑ +9.9 %
Lighthouse Max Potential First Input Delay 🟥 1,740 ms 🟥 2,980 ms ↑ +71.8 %
Lighthouse Uses efficient cache policy on static assets 🟨 31 resources found 🟨 31 resources found ↑ +0.1 %
Lighthouse Server Backend Latencies 🟩 820 ms 🟩 1,070 ms ↑ +29.4 %
Lighthouse Avoids enormous network payloads 🟩 Total size was 2,096 KiB 🟩 Total size was 2,096 KiB ↑ +0.1 %
Lighthouse Reduce unused CSS 🟥 Potential savings of 104 KiB 🟥 Potential savings of 125 KiB ↑ +10 %
Lighthouse Reduce unused JavaScript 🟥 Potential savings of 226 KiB 🟥 Potential savings of 226 KiB ↑ +1 %
Lighthouse Network Round Trip Times 🟩 70 ms 🟩 70 ms ↓ −3.3 %

657 other audits are unchanged.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 24, 2026

Code PushUp

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

💼 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 🟢 95 🟢 95 ↓ −0.3
Documentation 🔴 45 🔴 46 ↑ +0.3

4 other categories are unchanged.

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

🗃️ Groups

🔌 Plugin 🗃️ Group ⭐ Previous score ⭐ Current score 🔄 Score change
Code coverage Code coverage metrics 🟢 95 🟢 95 ↓ −0.3
JSDocs coverage Documentation coverage 🔴 45 🔴 46 ↑ +0.3

13 other groups are unchanged.

🛡️ Audits

🔌 Plugin 🛡️ Audit 📏 Previous value 📏 Current value 🔄 Value change
JSDocs coverage Functions coverage 🟥 239 undocumented functions 🟥 241 undocumented functions ↑ +0.8 %
Code coverage Function coverage 🟩 96.2 % 🟩 95.7 % ↓ −0.5 %
Code coverage Branch coverage 🟩 91.8 % 🟩 91.7 % ↓ −0.1 %
Code coverage Line coverage 🟩 97.8 % 🟩 97.7 % ↓ −0.1 %

440 other audits are unchanged.

💼 Project plugin-axe

🤨 Code PushUp report has both improvements and regressions.

🕵️ See full comparison in Code PushUp portal 🔍

🏷️ Category ⭐ Previous score ⭐ Current score 🔄 Score change
Documentation 🟡 70 🟡 70 ↓ −0.2
Code coverage 🟢 94 🟢 94 ↑ +0.1

4 other categories are unchanged.

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

🗃️ Groups

🔌 Plugin 🗃️ Group ⭐ Previous score ⭐ Current score 🔄 Score change
JSDocs coverage Documentation coverage 🟡 70 🟡 70 ↓ −0.2
Code coverage Code coverage metrics 🟢 94 🟢 94 ↑ +0.1

13 other groups are unchanged.

🛡️ Audits

🔌 Plugin 🛡️ Audit 📏 Previous value 📏 Current value 🔄 Value change
JSDocs coverage Functions coverage 🟨 12 undocumented functions 🟨 12 undocumented functions  +0 %
Code coverage Line coverage 🟨 72.7 % 🟨 73 % ↑ +0.3 %
Code coverage Branch coverage 🟨 89.9 % 🟩 90 % ↑ +0.1 %

440 other audits are unchanged.

💼 Project create-cli

🥳 Code PushUp report has improved.

🕵️ See full comparison in Code PushUp portal 🔍

🏷️ Category ⭐ Previous score ⭐ Current score 🔄 Score change
Code coverage 🟢 96 🟢 96 ↑ +0.2

5 other categories are unchanged.

👍 1 group improved, 👍 1 audit improved

🗃️ Groups

🔌 Plugin 🗃️ Group ⭐ Previous score ⭐ Current score 🔄 Score change
Code coverage Code coverage metrics 🟢 96 🟢 96 ↑ +0.2

14 other groups are unchanged.

🛡️ Audits

🔌 Plugin 🛡️ Audit 📏 Previous value 📏 Current value 🔄 Value change
Code coverage Branch coverage 🟨 87.5 % 🟨 88.2 % ↑ +0.8 %

443 other audits are unchanged.

💼 Project ci

😟 Code PushUp report has regressed.

🕵️ See full comparison in Code PushUp portal 🔍

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

5 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 🟢 92 🟢 92 ↓ −0.1

14 other groups are unchanged.

🛡️ Audits

🔌 Plugin 🛡️ Audit 📏 Previous value 📏 Current value 🔄 Value change
Code coverage Branch coverage 🟨 83.6 % 🟨 83.6 % ↓ −0.1 %
Code coverage Line coverage 🟩 94.8 % 🟩 94.8 % ↑ +0.1 %

443 other audits are unchanged.

💼 Project models

🤨 Code PushUp report has both improvements and regressions.

🕵️ See full comparison in Code PushUp portal 🔍

🏷️ Category ⭐ Previous score ⭐ Current score 🔄 Score change
Documentation 🟡 66 🟡 66 ↓ −0.1
Code coverage 🟢 94 🟢 94 ↑ +0.1

2 other categories are unchanged.

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

🗃️ Groups

🔌 Plugin 🗃️ Group ⭐ Previous score ⭐ Current score 🔄 Score change
JSDocs coverage Documentation coverage 🟡 66 🟡 66 ↓ −0.1
Code coverage Code coverage metrics 🟢 94 🟢 94 ↑ +0.1

10 other groups are unchanged.

🛡️ Audits

🔌 Plugin 🛡️ Audit 📏 Previous value 📏 Current value 🔄 Value change
JSDocs coverage Variables coverage 🟥 72 undocumented variables 🟥 76 undocumented variables ↑ +5.6 %
Code coverage Line coverage 🟩 99.4 % 🟩 99.4 % ↑ +0.1 %
JSDocs coverage Types coverage 🟥 53 undocumented types 🟥 57 undocumented types ↑ +7.5 %

434 other audits are unchanged.

💼 Project core

🥳 Code PushUp report has improved.

🕵️ See full comparison in Code PushUp portal 🔍

🏷️ Category ⭐ Previous score ⭐ Current score 🔄 Score change
Code coverage 🟢 90 🟢 90 ↑ +0.1

5 other categories are unchanged.

👍 1 group improved, 👍 1 audit improved

🗃️ Groups

🔌 Plugin 🗃️ Group ⭐ Previous score ⭐ Current score 🔄 Score change
Code coverage Code coverage metrics 🟢 90 🟢 90 ↑ +0.1

14 other groups are unchanged.

🛡️ Audits

🔌 Plugin 🛡️ Audit 📏 Previous value 📏 Current value 🔄 Value change
Code coverage Line coverage 🟩 94.8 % 🟩 94.8 % ↑ +0.1 %

443 other audits are unchanged.

💼 Project plugin-lighthouse

🤨 Code PushUp report has both improvements and regressions.

🕵️ See full comparison in Code PushUp portal 🔍

All of 6 categories are unchanged.

1 audit changed without impacting score

🗃️ Groups

All of 15 groups are unchanged.

🛡️ Audits

🔌 Plugin 🛡️ Audit 📏 Previous value 📏 Current value 🔄 Value change
Code coverage Branch coverage 🟩 98.8 % 🟩 98.8 % ↑ +0.1 %

443 other audits are unchanged.


7 other projects are unchanged.

@hanna-skryl hanna-skryl marked this pull request as ready for review January 26, 2026 17:07
Copy link
Collaborator

@matejchalk matejchalk left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@hanna-skryl hanna-skryl merged commit fe6f94e into main Jan 26, 2026
25 checks passed
@hanna-skryl hanna-skryl deleted the issue-source-url branch January 26, 2026 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants