Skip to content

Conversation

@supervoidcoder
Copy link
Member

we'll finally be free from this uhh i forgot the lyrics

dependabot bot and others added 2 commits October 10, 2025 02:26
Bumps [scratch-vm](https://github.com/OmniBlocks/scratch-vm) from `882d43d` to `1c10641`.
- [Commits](OmniBlocks/scratch-vm@882d43d...1c10641)

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

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

Bump scratch-vm from `882d43d` to `1c10641`
@supervoidcoder supervoidcoder merged commit 534b401 into extensions Oct 10, 2025
6 checks passed
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 10, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch main

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 10, 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/193/


🔦 Performance Audit (Lighthouse)

Core Scores:

  • 🔴 Performance: 38/100
  • 🟠 Accessibility: 78/100
  • 🟢 Best Practices: 96/100
  • 🟢 SEO: 100/100

📊 Detailed Performance Metrics:

  • First Contentful Paint (FCP): 43.2 s
  • Largest Contentful Paint (LCP): 44.6 s
  • Total Blocking Time (TBT): 700 ms
  • Cumulative Layout Shift (CLS): 0
  • Speed Index: 43.2 s
  • Time to Interactive (TTI): 44.3 s
  • Max Potential FID: 980 ms
  • Server Response Time: Root document took 0 ms
  • DOM Size: 17 elements
  • Main Thread Work: 4.4 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

⚠️ 3 issue(s) found

  • ✅ Passed checks: 12
  • ⚠️ Violations: 3
  • Full report available in artifacts

📊 Build Details


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

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.

This PR updates the dependency scratch-vm in the package-lock.json file to a newer commit and modifies the associated resolved URL and integrity hash. While this is a small change, it potentially impacts a core dependency of the project, so it is crucial to ensure compatibility.

Review Notes:

  1. Syntax/Runtime Errors:
    The provided patch only updates the dependency reference and integrity hash, neither of which cause syntax or runtime errors directly. However, as scratch-vm appears to be a critical dependency, it's important to verify that the updated commit on scratch-vm doesn't introduce issues downstream such as breaking API changes or unexpected behaviors.

  2. Security or Safety Concerns:

    • Ensure that the new scratch-vm commit (referencing hash 1c1064128dddd035b8e8f60b7cbc4c4ca4043158) is from a trusted

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