Skip to content

ci(release): remove npm cache from release workflow#965

Merged
overbalance merged 1 commit into
mainfrom
freeze/alert-fix-50
Nov 11, 2025
Merged

ci(release): remove npm cache from release workflow#965
overbalance merged 1 commit into
mainfrom
freeze/alert-fix-50

Conversation

@overbalance
Copy link
Copy Markdown
Member

@overbalance overbalance commented Nov 11, 2025

What problem is this solving?

Removes npm caching from the release workflow to avoid potential cache-related issues during deployments alerted here: https://github.com/embrace-io/embrace-web-sdk/security/code-scanning/50

Short description of changes

  • Removed cache: npm configuration from setup-node action in release.yaml

@github-actions
Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@overbalance overbalance self-assigned this Nov 11, 2025
@github-actions
Copy link
Copy Markdown
Contributor

Performance results

CDP Performance Tests

Number of Requests Size of Requests Script Duration Task Duration Heap Used Size
Requests +3 requests +29.72 KB
Page Loaded +14.80 ms +6.41 ms +0.91 MB
Generate 100 fetch requests +11.89 ms +48.97 ms +1.48 MB
Generate 100 XHR requests +33.08 ms +86.88 ms +2.30 MB
Click 100 buttons and generate 100 logs +30.87 ms +38.17 ms +2.85 MB
Throw a 100 exceptions +1.29 ms +25.48 ms +2.12 MB
End Session +8.89 ms +14.82 ms +2.96 MB
Total +3 requests +29.72 KB +100.81 ms +220.74 ms +12.62 MB

Lighthouse Startup Performance Tests

Difference Description
Total Blocking Time 0 ms Difference in Total Blocking Time: Sum of all time periods between FCP and Time to Interactive, when task length exceeded 50ms, expressed in milliseconds. Learn more about the Total Blocking Time metric.
Main Thread Time +62.56 ms Difference in Main Thread Time: Consider reducing the time spent parsing, compiling and executing JS. You may find delivering smaller JS payloads helps with this. Learn how to minimize main-thread work
Script Evaluation Time +45.21 ms Difference in Script Evaluation Time: Consider reducing the time spent parsing, compiling, and executing JS. You may find delivering smaller JS payloads helps with this. Learn how to reduce Javascript execution time.

@github-actions
Copy link
Copy Markdown
Contributor

build results

vite-7 Platform Tests

Total Uncompressed Size Total Gzip Size
vite-7 - esnext +165.17 KB +47.82 KB
vite-7 - es2015 +172.04 KB +49.48 KB

vite-otel-latest Platform Tests

Total Uncompressed Size Total Gzip Size
vite-otel-latest - esnext +164.24 KB +49.49 KB
vite-otel-latest - es2015 +171.03 KB +51.11 KB

webpack-5 Platform Tests

Total Uncompressed Size Total Gzip Size
webpack-5 - esnext +122.81 KB +43.23 KB
webpack-5 - es2015 +123.49 KB +44.69 KB

@overbalance overbalance merged commit 9dee03c into main Nov 11, 2025
19 checks passed
@overbalance overbalance deleted the freeze/alert-fix-50 branch November 11, 2025 13:38
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