Skip to content

Conversation

@briananders
Copy link
Owner

Fix critical build bugs, improve build performance, and enhance CI/CD reliability by updating dependencies, optimizing JS bundling, and refining workflow configurations.


Open in Cursor Open in Web

This commit updates various dependencies, including replacing `node-sass` with `sass`, and refactors the CI workflows for CodeQL and deployment. It also improves the S3 uploader to handle large numbers of objects and invalidates CloudFront cache more efficiently.

Co-authored-by: brian.steven.anders <brian.steven.anders@gmail.com>
@cursor
Copy link

cursor bot commented Oct 18, 2025

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@briananders briananders marked this pull request as ready for review October 18, 2025 16:54
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

This PR is being reviewed by Cursor Bugbot

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

for (let i = 0; i < videoGlob.length; i++) {
const videoPath = videoGlob[i];
moveOneImage(videoPath, configs, () => {
moveOneVideo(videoPath, configs, () => {
Copy link

Choose a reason for hiding this comment

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

Bug: Incorrect Function Call Causes Video Processing Error

When processing videos, the code incorrectly calls moveOneImage instead of moveOneVideo. This results in a ReferenceError at runtime.

Fix in Cursor Fix in Web

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants