`grunt verify:old-files` incorrectly flagging certain files

[desrosj]

In [50441], a PHPUnit test that verified built files were not included in the $_old_files list was moved into a Grunt task, verify:old-files.

For some reason, this is incorrectly flagging a few files that were removed and should be included in this list for WordPress 5.8:

• wp-includes/css/dist/editor/editor-styles.min.css
• wp-includes/css/dist/editor/editor-styles-rtl.css
• wp-includes/css/dist/editor/editor-styles-rtl.min.css
• wp-includes/css/dist/editor/editor-styles.css

A build error is only encountered when running build, build:dev does not result in an error.

It seems that the -rtl and .min parts of the file are the culprit because wp-includes/css/dist/editor/editor-styles.css is not resulting in the task failing.

I don't necessarily think that this is a blocker for WordPress 5.8. The $_old_files list is run over for every update. So these files can be added in 5.8.1 and they will be removed appropriately.

[desrosj]

For more context, after applying 53687.diff​, running npm run build results in the following:

Running "verify:old-files" task
Warning: build/wp-includes/css/dist/editor/editor-styles.min.css should not be present in the $_old_files array. Use --force to continue.

[johnbillion]

Chatting with Jon about this, the script is behaving as expected as the failure only appears when these files already exist in src and are copied over with the build. If these files have been removed from version control and are now built, I think there should be a cleanup so they're removed from src.

[SergeyBiryukov]

Related: #53702

[johnbillion]

Closing this off as the problem is not with the verify:old-files task. The warning it raised when the files were added to $_old_files was correct because the files still existed in the local copy.

[azaozz]

Just ran into this too.

the problem is not with the verify:old-files task

Yes, but the results is that npm run build a.k.a. grunt build fails/is aborted.

failure only appears when these files already exist in src and are copied over with the build. If these files have been removed from version control and are now built, I think there should be a cleanup so they're removed from src.

This is somewhat similar to #47078. A cleanup of the old built files from both /src and /build sounds like the better solution. Also see #47749.

However cleaning of all built files will not work at the moment as the above four files (and many others) are not being "cleaned" from /src anyway. Running grunt clean --dev leaves quite a few "built" files behind. Then grunt copy:files copies a mess of "original" and "built" files. Then most of the copied built files are overwritten when building the js and css or by webpack.

For this ticket is seems it would be enough to exclude the 4 offending files from copying to /build, for now. A better way to fix this and similar cases in the future is to fix grunt clean --dev and run it together with grunt clean before every build.

Going to reopen this as it prevents building to /build after building to /src.

[desrosj]

@azaozz Would the solution proposed on #53719 solve this?

[azaozz]

Replying to desrosj:

Yes but that cleaning of /src will have to happen before building to /build. To fix this ticket it would need the solution proposed on #53719 plus running grunt clean on both /src and /build.

[chaion07]

Hi @desrosj! Thank you for reporting this. Leaving it on the report for 5.9 and also be happy to push it to future release. Since the build tools don't affect the release code, and also someone can pick it up even during the RC period, keeping my fingers crossed. Cheers!

[hellofromTonya]

With 5.9 Beta 1 release happening in a few hours, moving this ticket to 6.0.

[costdev]

Per the discussion in the bug scrub, I'm moving this ticket to _Future Release_. The patch also needs a refresh, so adding the needs-refresh keyword too.

[hellofromTonya]

Ran into this issue today during Dry Run when running npm run grunt prerelease. An older file, i.e. src/wp-includes/blocks/heading/editor.css, was created from an older npm run build:dev. That files was removed in WP 5.9. As a result, the verify:old-files grunt task failed as the file was added into the build folder.

As @azaozz suggests, cleaning src before running build would have resolved the issue and the time lost during Dry Run.

Moving this ticket into 6.1 for resolution before 6.1 Dry Run cycle.

[desrosj]

This still needs a patch. Punting with 6.1 coming to a close. If a patch is ready and tested before then and a committer is confident in merging, it can be moved back.

[danielbachhuber]

I ran into this too:

Running "verify:old-files" task
Warning: build/wp-includes/blocks/post-comments/editor.css should not be present in the $_old_files array. Use --force to continue.

I like to use npm run build when I don't need the watch process running.

[hellofromTonya]

This still needs a patch. With RC1 next week, punting to 6.3 (which alpha opens next week). However, it can be committed at any time when a patch is ready and a committer is confident in committing it.

[SergeyBiryukov]

Does [55484] / #47749 resolve this?

[oglekler]

Hi @desrosj and @azaozz can you please take a look if this issue is solved / can be solved by tickets mentioned in the previous comment and if not, possibly page can be refreshed. Let's solve it in 6.3 🙂 Thank you 🙏

[azaozz]

Replying to SergeyBiryukov:

Does [55484] / #47749 resolve this?

Yep, think it does, sorry for missing to close it.

Lets close as fixed with [55484]. Please feel free to reopen if somebody runs into this again.