Guard native CLI packaging PHP selection

Author: codex

packages/playground/cli-native/src/bin/package-native-cli.rs

@@ -325,6 +325,7 @@ fn print_help() {
            --out-dir <path>             Output directory, defaults to target/package\n\
            --name <name>                Package directory/archive name\n\
           --php-version <version>      Include only this PHP version; repeatable\n\
+                                       Omit to package all supported PHP versions\n\
           --precompile-wasmtime        Generate target-specific Wasmtime .cwasm assets\n\
           --no-precompile-wasmtime     Copy wasm assets without target-specific precompile\n\
           --skip-wordpress-assets      Do not copy bundled WordPress/SQLite ZIPs\n\

packages/playground/cli-native/src/packaging.rs

@@ -1380,10 +1380,11 @@ mod tests {
 
     use super::{
         extract_package_archive, package_native_cli, precompile_packaged_wasm_for_target,
-        PackageOptions, PACKAGE_SHARE_DIR,
+        selected_php_manifest, PackageOptions, PACKAGE_SHARE_DIR,
     };
     use crate::{
-        assets::{load_php_assets_manifest, select_php_asset},
+        args::SUPPORTED_PHP_VERSIONS,
+        assets::{load_php_assets_manifest, select_php_asset, AssetManifest, FileAsset, PhpAsset},
         sha256::sha256_hex,
     };
     use zip::{write::SimpleFileOptions, CompressionMethod, ZipWriter};
@@ -1571,6 +1572,48 @@ mod tests {
         );
     }
 
+    fn fake_php_asset(version: &str) -> PhpAsset {
+        let normalized = version.replace('.', "_");
+        PhpAsset {
+            version: version.to_string(),
+            runtime: None,
+            js: FileAsset {
+                path: PathBuf::from(format!("php_{normalized}.js")),
+                sha256: "0".repeat(64),
+            },
+            wasm: FileAsset {
+                path: PathBuf::from(format!("php_{normalized}.wasm")),
+                sha256: "0".repeat(64),
+            },
+            wasmtime: None,
+        }
+    }
+
+    #[test]
+    fn default_selection_includes_every_supported_php_version() {
+        let mut php = vec![fake_php_asset("5.2")];
+        php.extend(
+            SUPPORTED_PHP_VERSIONS
+                .iter()
+                .map(|version| fake_php_asset(version)),
+        );
+        php.push(fake_php_asset("9.9"));
+        let manifest = AssetManifest {
+            schema_version: 1,
+            runtime: "node-builds/asyncify".to_string(),
+            php,
+        };
+
+        let selected = selected_php_manifest(&manifest, &[]).unwrap();
+        let versions = selected
+            .php
+            .iter()
+            .map(|asset| asset.version.as_str())
+            .collect::<Vec<_>>();
+
+        assert_eq!(versions, SUPPORTED_PHP_VERSIONS.to_vec());
+    }
+
     #[test]
     fn packages_supported_assets_by_default() {
         let root = temp_dir("supported-asset-root");