Style: Replace Grunt with @wordpress/scripts and modernize the build

[obenland]

Modernizes the build tooling and code style for wordpress.org/public_html/style/, and adds CI coverage for it.

Tooling

• Replaces Grunt/JSHint with @wordpress/scripts 33: npm run build / build:css / build:js / format / lint:js.
• js/navigation.min.js is built in place by webpack (webpack.config.js); wp4-rtl.css by a small bin/build-rtl.js, preserving the Dashicons arrow-swap and @import renaming that RTLCSS can't infer and that wp-scripts' built-in RTL support can't express.
• Browser targets now extend the official @wordpress/browserslist-config, matching the wporg-*-2024 themes, replacing the 2013-era list (IE 7+, Android 2.1+). Autoprefixer and the self-referential wp4.css.map are dropped: under current targets the stylesheet needs no generated prefixes (the few remaining -webkit-/-moz- occurrences are intentional non-standard properties).

Code style

• All JS in style/ (including trac/) now passes the stock @wordpress/scripts ESLint ruleset with zero errors, and is Prettier-formatted with a single local override (printWidth: 120).
• HTML-building string concatenation is converted to template literals; query-string building on the GitHub-PRs endpoint uses URLSearchParams (which also percent-encodes values that previously went onto the wire raw).
• Removes trac/trac-search.js, dead since its include was removed in r7275 (2018) — its API endpoint no longer exists.

CI

• New style-lint.yml workflow runs on changes to JS/CSS under style/: ESLint, prettier --check, and a build-drift check that fails when committed build output (wp4-rtl.css, js/navigation.min.js) is out of sync with its source.

Docs

• Adds README.md, trac/README.md, and AGENTS.md (with a CLAUDE.md wrapper): file inventory, development flow, testing Trac changes via DevTools local overrides, and the deploy flow (commit + sandbox deploy, then the follow-up scripts_version bump in both Trac templates).

Verification

• wp4.css builds byte-identical under the new pipeline before the browserslist change; the RTL diff beyond that consists of drift the old pipeline had accumulated plus the prefix removal.
• All converted string sites verified output-equivalent (modulo whitespace between block-level tags) across every conditional branch.
• npm run lint:js, npx prettier --check ., node --check on all sources, and a full npm run build idempotency check all pass.

Note for deployment: the changes to trac/*.js will need the usual scripts_version bump in site_head.html/site_footer.html as a follow-up commit once the assets are deployed.

🤖 Generated with Claude Code

[github-actions]

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 obenland.

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

[Copilot]

Pull request overview

Modernizes the build and maintenance workflow for wordpress.org/public_html/style/ by migrating from Grunt/JSHint to @wordpress/scripts, regenerating committed build artifacts accordingly, and adding CI checks to prevent drift and enforce lint/format rules for the Style assets.

Changes:

• Replace Grunt/JSHint tooling with @wordpress/scripts-based build/lint/format commands and updated browser targets.
• Introduce a custom webpack config and a Node-based RTL build script, committing regenerated outputs (wp4-rtl.css, js/navigation.min.js).
• Add documentation for the Style and Trac asset workflows plus a dedicated GitHub Actions workflow for lint/format/build-drift checks.

Reviewed changes

Copilot reviewed 17 out of 21 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
wordpress.org/public_html/style/wp4.css.map	Removes the committed source map output.
wordpress.org/public_html/style/wp4.css	Updates CSS output (formatting/normalization, vendor-prefix removal, mapping comment removal).
wordpress.org/public_html/style/wp4-rtl.css	Updates committed RTL build output to match the new pipeline.
wordpress.org/public_html/style/webpack.config.js	Customizes @wordpress/scripts webpack output to build js/navigation.min.js in-place.
wordpress.org/public_html/style/trac/README.md	Documents Trac-specific static assets and deployment/version-bump workflow.
wordpress.org/public_html/style/README.md	Documents Style directory inventory, local dev commands, and deployment workflow.
wordpress.org/public_html/style/package.json	Switches to @wordpress/scripts tooling, adds browserslist, and defines build/lint/format scripts.
wordpress.org/public_html/style/js/navigation.min.js	Updates committed minified navigation script output.
wordpress.org/public_html/style/js/navigation.js	Modernizes the navigation script source to satisfy @wordpress/scripts lint/style expectations.
wordpress.org/public_html/style/Gruntfile.js	Removes legacy Grunt build configuration.
wordpress.org/public_html/style/CLAUDE.md	Adds a wrapper pointing tooling/agent guidance to AGENTS.md.
wordpress.org/public_html/style/bin/build-rtl.js	Adds a Node script to generate wp4-rtl.css from wp4.css (RTL build step).
wordpress.org/public_html/style/AGENTS.md	Adds directory-specific contributor/agent workflow rules and commands.
wordpress.org/public_html/style/.stylelintrc.js	Adds Stylelint configuration extending WordPress defaults with local rule relaxations.
wordpress.org/public_html/style/.stylelintignore	Adds Stylelint ignore rules for vendored/frozen/generated assets.
wordpress.org/public_html/style/.prettierrc.js	Adds a Prettier config extending WordPress defaults with printWidth: 120.
wordpress.org/public_html/style/.prettierignore	Excludes generated/minified/CSS (and vendored Trac assets) from Prettier formatting.
wordpress.org/public_html/style/.jshintrc	Removes legacy JSHint configuration.
wordpress.org/public_html/style/.jshintignore	Removes legacy JSHint ignore configuration.
.github/workflows/style-lint.yml	Adds CI workflow to lint/format and verify generated files are up-to-date for Style changes.

Suppressed comments (2)

wordpress.org/public_html/style/js/navigation.js:22

• document.getElementById() returns null when an element is missing, not undefined. The current typeof checks won’t catch a missing #mobile-menu-button or #wporg-header-menu, and the code will then throw when accessing properties on null.

	const button = document.getElementById( 'mobile-menu-button' );
	if ( 'undefined' === typeof button ) {
		return;
	}

	const menu = document.getElementById( 'wporg-header-menu' );

	// Hide menu toggle button if menu is empty and return early.
	if ( 'undefined' === typeof menu ) {
		button.style.display = 'none';
		return;
	}

.github/workflows/style-lint.yml:16

• Same as the PR trigger: the push path filters should use **/*.js / **/*.css so nested files under style/ are included.

    - 'wordpress.org/public_html/style/**.js'
    - 'wordpress.org/public_html/style/**.css'

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.