Skip to content

Conversation

@supervoidcoder
Copy link
Member

Resolves

What Github issue does this resolve (if any, if not then please include link)?

  • Resolves #

Proposed Changes

Describe what this Pull Request does

Reason for Changes

Explain why these changes should be made. Why is this helpful or necessary? Why should this be added?

Test Coverage

Please show how you have added tests to cover your changes

Browser Coverage

Check the OS/browser combinations tested (At least 2)

Mac

  • Chrome
  • Firefox
  • Safari

Windows

  • Chrome
  • Firefox
  • Edge

Chromebook

  • Chrome

iPad

  • Safari

Android Tablet

  • Chrome

dependabot bot and others added 2 commits October 9, 2025 14:06
Bumps [scratch-vm](https://github.com/OmniBlocks/scratch-vm) from `6865374` to `a7ad12d`.
- [Commits](OmniBlocks/scratch-vm@6865374...a7ad12d)

---
updated-dependencies:
- dependency-name: scratch-vm
  dependency-version: a7ad12dcb29f4aea6253aac03a483af25940e116
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
…ch-vm-a7ad12d

Bump scratch-vm from `6865374` to `a7ad12d`
@supervoidcoder supervoidcoder merged commit 76f5124 into extensions Oct 9, 2025
5 checks passed
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Thank you for submitting this pull request.

Summary

This pull request updates the scratch-vm dependency in package-lock.json to a newer version, changing the Git commit reference and associated integrity hash. Likely, this is intended to pull in updates or bug fixes made to the scratch-vm repository.

Code Analysis

  • Syntax/Runtime Errors: No syntax or runtime errors are evident from the package-lock.json changes.
  • Security Concerns: It's important to confirm the trustworthiness of the new Git commit in scratch-vm. Ensure that the referenced commit does not introduce vulnerabilities or unsafe code.
  • Performance/Maintainability: While the update itself looks straightforward, changes in dependencies should be carefully tested in case of regressions or unexpected behavior due to the scratch-vm update.
  • Testing and Documentation: It's unclear from the pull request description whether testing has been done against specific browser or OS environments to confirm

@github-actions
Copy link
Contributor

github-actions bot commented Oct 9, 2025

🎉 MEGA PR Test & Analysis Complete!

Hi @supervoidcoder! Your PR has been fully tested and analyzed! 🚀✨


🌍 Test Deployment

Live Preview: https://OmniBlocks.github.io/scratch-gui/supervoidcoder/183/


🔦 Performance Audit (Lighthouse)

Core Scores:

  • 🔴 Performance: 33/100
  • 🟠 Accessibility: 70/100
  • 🟢 Best Practices: 96/100
  • 🟠 SEO: 88/100

📊 Detailed Performance Metrics:

  • First Contentful Paint (FCP): 44.0 s
  • Largest Contentful Paint (LCP): 46.8 s
  • Total Blocking Time (TBT): 1,000 ms
  • Cumulative Layout Shift (CLS): 0
  • Speed Index: 44.0 s
  • Time to Interactive (TTI): 49.3 s
  • Max Potential FID: 1,890 ms
  • Server Response Time: Root document took 0 ms
  • DOM Size: 350 elements
  • Main Thread Work: 9.7 s

📥 Download full Lighthouse report


📦 Bundle Size Analysis

Total Build Size: 136M

📦 Top 5 Largest JavaScript Files:

  • 20M - samples3.js
  • 20M - player/samples3.js
  • 13M - player/drumsamples.js
  • 13M - drumsamples.js
  • 9.5M - samples2.js

🎨 Top 5 Largest CSS Files:

  • No CSS files found (might be inlined in JS)

♿ Accessibility Testing

⚠️ 5 issue(s) found

  • ✅ Passed checks: 25
  • ⚠️ Violations: 5
  • Full report available in artifacts

📊 Build Details


💡 Pro Tip: Only one build was needed for all these checks - saving CI/CD minutes! 🎯

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

1 participant