Skip to content

Update npm dependencies#207

Merged
peterwilsoncc merged 12 commits into
developfrom
update-npm-dependencies
Jan 19, 2026
Merged

Update npm dependencies#207
peterwilsoncc merged 12 commits into
developfrom
update-npm-dependencies

Conversation

@peterwilsoncc
Copy link
Copy Markdown
Contributor

@peterwilsoncc peterwilsoncc commented Jan 19, 2026

Description of the Change

Update NPM Dependencies, see changelog below.

How to test the Change

Changelog Entry

Changed - Update NPM dependencies via npm audit fix
Changed - Add unused dependencies to react hooks.
Developer - Update cypress to 15.9.0
Developer - Update @wordpress/scripts to 31.3.0
Developer - Update @wordpress/env to 10.38.0

Credits

Props @peterwilsoncc

Checklist:

@github-actions github-actions Bot added this to the 1.3.0 milestone Jan 19, 2026
@peterwilsoncc peterwilsoncc force-pushed the update-npm-dependencies branch from 6167478 to 2a3e2c9 Compare January 19, 2026 04:02
@peterwilsoncc
Copy link
Copy Markdown
Contributor Author

@dkotter Before I mark this ready for review, are you able to help me figure out why the E2E tests are failing?

Testing locally, it looks like id is an object here rather than a string but I can't figure out if that's the result of something I've changed or if I am missing something. I suspect the latter.

cy.insertBlock("core/paragraph").then((id) => {
cy.getBlockEditor().find(`#${id}`).click().type(randomName());
});

@dkotter
Copy link
Copy Markdown
Collaborator

dkotter commented Jan 19, 2026

@dkotter Before I mark this ready for review, are you able to help me figure out why the E2E tests are failing?

Testing locally, it looks like id is an object here rather than a string but I can't figure out if that's the result of something I've changed or if I am missing something. I suspect the latter.

I wasn't able to fully figure this one out but I was able to reproduce on develop so not related to changes here. Also seems like it only fails on newer versions of WP so something that was changed there between 6.6 and 6.9. I imagine we'll need an update to our Cypress utility package to fix this.

In the meantime, I changed it to insert an HTML block instead of a paragraph block which works. Looks like on trunk (coming in 7.0) the HTML block has changed so needed to account for that as well but things are passing now.

@peterwilsoncc peterwilsoncc marked this pull request as ready for review January 19, 2026 23:15
@github-actions github-actions Bot added the needs:code-review This requires code review. label Jan 19, 2026
@peterwilsoncc
Copy link
Copy Markdown
Contributor Author

Thanks for figuring out the tests, Darin.

@peterwilsoncc peterwilsoncc merged commit d71d92f into develop Jan 19, 2026
14 checks passed
@peterwilsoncc peterwilsoncc deleted the update-npm-dependencies branch January 19, 2026 23:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs:code-review This requires code review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants