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 13:12
Bumps [scratch-vm](https://github.com/OmniBlocks/scratch-vm) from `e8e4f8b` to `6865374`.
- [Commits](OmniBlocks/scratch-vm@e8e4f8b...6865374)

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

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

Bump scratch-vm from `e8e4f8b` to `6865374`
@supervoidcoder supervoidcoder merged commit 375039b 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.

High-Level Summary

This PR updates the package-lock.json file to reflect a new version of the scratch-vm dependency. It appears to replace the previously-resolved commit with a newer commit from the scratch-vm repository.

Observations and Recommendations

  1. Obvious Syntax/Runtime Errors:
    As this PR only updates the package-lock.json file, there are no observable syntax or runtime errors.

  2. Security or Safety Concerns:

    • This change alters the dependency version for scratch-vm. Ensure that the new commit being referenced (68653741649a181a4fda3c3a51e19cb540c4ac36) has been appropriately vetted for security issues and conforms to the behavior expected by the current application.
  3. Performance or Maintainability:

    • If these changes introduce any significant updates or modifications in `scratch-v

@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/181/


🔦 Performance Audit (Lighthouse)

Core Scores:

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

📊 Detailed Performance Metrics:

  • First Contentful Paint (FCP): 45.3 s
  • Largest Contentful Paint (LCP): 49.1 s
  • Total Blocking Time (TBT): 450 ms
  • Cumulative Layout Shift (CLS): 0
  • Speed Index: 45.3 s
  • Time to Interactive (TTI): 50.0 s
  • Max Potential FID: 2,550 ms
  • Server Response Time: Root document took 0 ms
  • DOM Size: 350 elements
  • Main Thread Work: 12.5 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: 26
  • ⚠️ 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