Changeset 62604

Timestamp:

06/30/2026 09:19:46 PM (14 hours ago)

Author:

jorbin

Message:

Build/Test: Fix typescript error in Gutenberg Copy.

Backporting [62600] leads to the following error: Parameter 'dep' implicitly has an 'any' type. so this gives an explicit type and also does explicit type checking.

This isn't an issue in trunk due to [62428] removing this code and re-introducing client side media so this commit is only happening in the 7.0 branch.

Developed in: ​https://github.com/WordPress/wordpress-develop/pull/12370
Discussed in slack thread: ​https://wordpress.slack.com/archives/C02RQBWTW/p1782850291418919

Follow up-to [62600].

Props jorbin, joedolson.
See: #65452.

File:

• branches/7.0/tools/gutenberg/copy.js (modified) (1 diff)

branches/7.0/tools/gutenberg/copy.js

@@ -500 +500 @@
                 assetData.module_dependencies =
                     assetData.module_dependencies.filter(
+                        /**
+                         * @param {string|{ id: string }} dep
+                         * @return {boolean}
+                         */
                         ( dep ) =>
-                            ! ( dep.id || dep ).startsWith(
+                            ! ( typeof dep === 'string' ? dep : dep.id ).startsWith(
                                 '@wordpress/vips'
                             )