@plaudit/webpack-extensions 3.1.0 → 3.1.1

package/CHANGELOG.md

@@ -5,6 +5,11 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [3.1.1] - 2026-03-06
+### Fixed
+- Inlined scripts without any other path query parameters not being placed in the footer by default
+  - This comes from version `2.85.2`
+
 ## [3.1.0] - 2026-03-05
 ### Added
 - Support for inlining (and otherwise controlling the enqueuing of) assets
@@ -31,6 +36,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Legacy PostCSS features that have been integrated into modern CSS
 - `@extends` support
 
+## [2.85.2] - 2026-03-06
+### Fixed
+- Issues caused by trying to save a couple of cycles by using `hasAnyKeys` in place of checking the actual keys
+
+## [2.85.1] - 2026-03-06
+### Fixed
+- Inlined scripts without any other path query parameters not being placed in the footer by default
+
 ## [2.85.0] - 2026-03-05
 ### Added
 - Support for inlining (and otherwise controlling the enqueuing of) assets

package/USER-GUIDE.md

@@ -29,6 +29,7 @@
 * [Reference](#reference)
   * [Root Options](#root-options)
   * [Per-Entrypoint Options](#per-entrypoint-options)
+    * [Path Queries](#path-queries)
     * [Boolean-form](#boolean-form)
     * [String-form](#string-form)
     * [Object-form](#object-form)
@@ -269,7 +270,7 @@ After building, each asset gets a handle you can reference in PHP.
 **Best way:** Check the generated loader files after building:
 - `dist/blocks/blocks-loader.php`
 - `dist/extensions/extensions-loader.php`
-- The first argument of the `wp_register_script` and `wp_register_style` calls in `dist/plain-entrypoints-loader.php` 
+- The first argument of the `wp_register_script` and `wp_register_style` calls in `dist/plain-entrypoints-loader.php`
 
 ### Example: Manual Enqueue with Localization
 
@@ -468,7 +469,8 @@ src/blocks/
 #### Notes
 - Blocks **should** be created with `pnpm @plaudit/scaffold create block`
 - template.php and setup.php files are automatically loaded; however, they can be manually specified if desired
-- Script and Style files **must** be specified using the `file:./<path here>` syntax in the appropriate keys. See [WordPress' `block.json` reference](https://developer.wordpress.org/block-editor/reference-guides/block-api/block-metadata/) for further details
+- Script and Style files **must** be specified using the `file:./<path here>[?path-query-here]` syntax in the appropriate keys. See [WordPress' `block.json` reference](https://developer.wordpress.org/block-editor/reference-guides/block-api/block-metadata/) for further details
+  - The `[?path-query-here]` component is something that we added. You can see more details in its section [here](#path-queries)
 
 #### Per-Entrypoint Options
 The `blocks` entrypoint type doesn't have any.

package/build/utils/path-query-and-related-helpers.d.ts

@@ -1,4 +1,4 @@
-import { Compilation } from "webpack";
+import type { Compilation } from "webpack";
 import { InlinedAsset, PathQueryParameters, ScriptArgsObject, UsageLocations } from "../shared";
 export declare function getAssetFileContents(compilation: Compilation, name: string): string;
 export declare function unpackPotentiallyPrefixedFilePath(filePath: string): [string, PathQueryParameters | undefined];
@@ -40,8 +40,8 @@ export type NormalizedScriptEnqueuingControlFlags = {
 /**
  * This function does a few things:
  * <ol>
- *     <li>It extracts strategy, in_footer, and fetchpriority from pathQueryParameters</li>
- *     <li>If the strategy is, 'inline', it records the contents of the asset for later injection inline and removes the asset from the compilation output</li>
+ *     <li>It extracts inline, position, strategy, in_footer, and fetchpriority from pathQueryParameters</li>
+ *     <li>If inline is true or the strategy is, 'inline', it records the contents of the asset for later injection inline and removes the asset from the compilation output</li>
  * </ol>
  * @param compilation
  * @param file

package/build/utils/path-query-and-related-helpers.js

@@ -13,8 +13,8 @@ exports.isValidPositionForInlineStrategy = isValidPositionForInlineStrategy;
 exports.newInvalidPositionForInlineStrategyError = newInvalidPositionForInlineStrategyError;
 exports.mergeInPathQueryParameters = mergeInPathQueryParameters;
 exports.parseScriptArgsObjectFromPathQueryParameters = parseScriptArgsObjectFromPathQueryParameters;
-const shared_1 = require("../shared");
 const node_path_1 = require("node:path");
+const shared_1 = require("../shared");
 function getAssetFileContents(compilation, name) {
     const asset = compilation.getAsset(name);
     if (!asset) {
@@ -126,25 +126,20 @@ function unpackRegisterScriptArgsFromPathQueryParameters(pathQueryParameters, fi
     if (pathQueryParameters === undefined) {
         return undefined;
     }
-    let hasAnyKeys = false;
     let baseArgs;
     const strategy = pathQueryParameters['strategy'];
     switch (strategy) {
         case 'eager':
-            hasAnyKeys = true;
             baseArgs = { strategy: 'eager' };
             break;
         case 'lazy':
-            hasAnyKeys = true;
             baseArgs = { strategy: 'defer', in_footer: true };
             break;
         case 'inline':
-            hasAnyKeys = true;
             baseArgs = { inline: true };
             break;
         case true:
         case false:
-            hasAnyKeys = true;
             baseArgs = { in_footer: strategy };
             break;
         case null:
@@ -162,7 +157,6 @@ function unpackRegisterScriptArgsFromPathQueryParameters(pathQueryParameters, fi
         if (!isValidInFooter(in_footer)) {
             throw newInvalidInFooterError(in_footer, file);
         }
-        hasAnyKeys = true;
         baseArgs.in_footer = in_footer;
     }
     if (pathQueryParameters['fetchpriority'] !== undefined) {
@@ -172,18 +166,15 @@ function unpackRegisterScriptArgsFromPathQueryParameters(pathQueryParameters, fi
         if (baseArgs.fetchpriority !== undefined && baseArgs.fetchpriority !== pathQueryParameters['fetchpriority']) {
             throw (0, shared_1.newWebpackErrorForFile)(`The strategy and fetchpriority values in the ${sourceType} for the file conflict. Got ${strategy} and ${pathQueryParameters['fetchpriority']}`, file);
         }
-        hasAnyKeys = true;
         baseArgs.fetchpriority = pathQueryParameters['fetchpriority'];
     }
-    if (pathQueryParameters['inline'] !== undefined) {
-    }
-    return hasAnyKeys ? baseArgs : undefined;
+    return Object.keys(baseArgs).length > 0 ? baseArgs : undefined;
 }
 /**
  * This function does a few things:
  * <ol>
- *     <li>It extracts strategy, in_footer, and fetchpriority from pathQueryParameters</li>
- *     <li>If the strategy is, 'inline', it records the contents of the asset for later injection inline and removes the asset from the compilation output</li>
+ *     <li>It extracts inline, position, strategy, in_footer, and fetchpriority from pathQueryParameters</li>
+ *     <li>If inline is true or the strategy is, 'inline', it records the contents of the asset for later injection inline and removes the asset from the compilation output</li>
  * </ol>
  * @param compilation
  * @param file
@@ -195,20 +186,16 @@ function parseScriptArgsObjectFromPathQueryParameters(compilation, file, pathQue
     }
     const scriptArgsObject = {};
     let inlinedAsset = undefined;
-    let hasAnyKeys = false;
     const strategy = pathQueryParameters['strategy'];
     switch (strategy) {
         case 'async':
         case 'defer':
-            hasAnyKeys = true;
             scriptArgsObject.strategy = strategy;
             break;
         case 'eager':
-            hasAnyKeys = true;
             scriptArgsObject.in_footer = false;
             break;
         case 'lazy':
-            hasAnyKeys = true;
             scriptArgsObject.strategy = 'defer';
             scriptArgsObject.in_footer = true;
             break;
@@ -221,7 +208,6 @@ function parseScriptArgsObjectFromPathQueryParameters(compilation, file, pathQue
             break;
         case true:
         case false:
-            hasAnyKeys = true;
             scriptArgsObject.in_footer = strategy;
             break;
         case null:
@@ -236,7 +222,6 @@ function parseScriptArgsObjectFromPathQueryParameters(compilation, file, pathQue
         if (!isValidInFooter(in_footer)) {
             throw newInvalidInFooterError(in_footer, file);
         }
-        hasAnyKeys = true;
         scriptArgsObject.in_footer = in_footer;
     }
     if (inlinedAsset === undefined && pathQueryParameters['inline']) {
@@ -265,10 +250,9 @@ function parseScriptArgsObjectFromPathQueryParameters(compilation, file, pathQue
         if (inlinedAsset !== undefined) {
             compilation.warnings.push((0, shared_1.newWebpackErrorForFile)("Fetchpriority has no effect on inlined assets", file));
         }
-        hasAnyKeys = true;
         scriptArgsObject.fetchpriority = fetchpriority;
     }
-    return hasAnyKeys ? { scriptArgsObject, inlinedAsset } : { inlinedAsset };
+    return Object.keys(scriptArgsObject).length > 0 ? { scriptArgsObject, inlinedAsset } : { inlinedAsset };
 }
 function removeFileAndAssetPHP(compilation, file) {
     const pathParts = (0, node_path_1.parse)(file);

package/build/wordpress-scripts-wrapper.js

@@ -232,7 +232,7 @@ function buildVerifiedConfig(config) {
         }
         if (destination !== undefined) {
             const effectiveDestination = toEffectiveWebpackDestination(destination);
-            allocatedDestinations[effectiveDestination] = rawSource[0]; // We need to pre-populate the allocatedDestinations map with statically-declared destinations
+            allocatedDestinations[effectiveDestination] = rawSource[0]; // We need to pre-populate the allocatedDestinations map with statically declared destinations
             return [rawSource[0], { ...normalizedParts, locations, destination, effectiveDestination, staticallyDeclaredDestination: true, pathQueryParameters }];
         }
         else {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@plaudit/webpack-extensions",
-  "version": "3.1.0",
+  "version": "3.1.1",
   "license": "SEE LICENSE IN LICENSE.md",
   "files": [
     "/build",
@@ -45,7 +45,7 @@
     "browser-sync": "^3.0.4",
     "copy-webpack-plugin": "10.2.4",
     "css-minimizer-webpack-plugin": "^7.0.4",
-    "cssnano": "^7.1.2",
+    "cssnano": "^7.1.3",
     "fork-ts-checker-webpack-plugin": "^9.1.0",
     "http-proxy-middleware": "^3.0.5",
     "json2php": "^0.0.12",