Skip to content

Build: Fix Gutenberg copy script missing constants.php and incorrect base URL#10965

Closed
youknowriad wants to merge 1 commit intoWordPress:trunkfrom
youknowriad:fix-font-library-require
Closed

Build: Fix Gutenberg copy script missing constants.php and incorrect base URL#10965
youknowriad wants to merge 1 commit intoWordPress:trunkfrom
youknowriad:fix-font-library-require

Conversation

@youknowriad
Copy link
Contributor

@youknowriad youknowriad commented Feb 18, 2026

Summary

The previous Gutenberg hash upgrade brought a new @wordpress/build package to Core but the build scripts in Core were not updated in consequence. It resulted in a broken font library page. This PR fixes it.

  • Add constants.php to the list of PHP infrastructure files copied from Gutenberg's build output to Core in copy-gutenberg-build.js
  • Fix the --base-url argument in build-gutenberg.js to include the trailing slash inside includes_url('build/') instead of outside the function call
  • Rename fast to skip-types to get back to sane performance for CI jobs and gutenberg builds.

Fixes https://core.trac.wordpress.org/ticket/64656.

Test plan

  • Visit the Font Library admin page and confirm it loads without errors

🤖 Generated with Claude Code

@github-actions
Copy link

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

Core Committers: Use this line as a base for the props when committing in SVN:

Props youknowriad.

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

…base URL.

The copy script was not including `constants.php` in the list of PHP infrastructure
files to sync from Gutenberg's build output to Core. Additionally, the base URL
passed to the Gutenberg build had the trailing slash outside the `includes_url()`
call, resulting in URLs like `build/pages` being generated as `buildpages`.

Props youknowriad.
Fixes #64656.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@youknowriad youknowriad force-pushed the fix-font-library-require branch from e0f8d7e to 3241835 Compare February 18, 2026 09:02
@github-actions
Copy link

Test using WordPress Playground

The changes in this pull request can previewed and tested using a WordPress Playground instance.

WordPress Playground is an experimental project that creates a full WordPress instance entirely within the browser.

Some things to be aware of

  • All changes will be lost when closing a tab with a Playground instance.
  • All changes will be lost when refreshing the page.
  • A fresh instance is created each time the link below is clicked.
  • Every time this pull request is updated, a new ZIP file containing all changes is created. If changes are not reflected in the Playground instance,
    it's possible that the most recent build failed, or has not completed. Check the list of workflow runs to be sure.

For more details about these limitations and more, check out the Limitations page in the WordPress Playground documentation.

Test this pull request with WordPress Playground.

Copy link
Member

@ellatrix ellatrix left a comment

Choose a reason for hiding this comment

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

Thanks, Font Library works now.

@github-actions
Copy link

A commit was made that fixes the Trac ticket referenced in the description of this pull request.

SVN changeset: 61673
GitHub commit: 5439957

This PR will be closed, but please confirm the accuracy of this and reopen if there is more work to be done.

@github-actions github-actions bot closed this Feb 18, 2026
@youknowriad youknowriad deleted the fix-font-library-require branch February 18, 2026 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants