lint, restore ci jobs

adamziel · adamziel · commit a44534d8ff93 · 2025-07-01T11:01:34.000+02:00
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -25,7 +25,7 @@ jobs:
             - run: npx nx affected --target=typecheck
     test-unit-asyncify:
         runs-on: ubuntu-latest
-        # needs: [lint-and-typecheck]
+        needs: [lint-and-typecheck]
         if: false # TODO: Re-enable this once the tests are fixed
         strategy:
             fail-fast: false
@@ -106,7 +106,7 @@ jobs:
     #         - run: node --experimental-wasm-jspi --experimental-wasm-stack-switching --expose-gc node_modules/nx/bin/nx affected --target=test --configuration=ci
     test-e2e:
         runs-on: ubuntu-latest
-        # needs: [lint-and-typecheck]
+        needs: [lint-and-typecheck]
         if: false # TODO: Re-enable this once the tests are fixed
         # Run as root to allow node to bind to port 80
         steps:
@@ -125,7 +125,7 @@ jobs:
 
     test-e2e-playwright-prepare:
         runs-on: ubuntu-latest
-        # needs: [lint-and-typecheck]
+        needs: [lint-and-typecheck]
         steps:
             - uses: actions/checkout@v4
               with:
@@ -187,7 +187,7 @@ jobs:
                   name: playwright-dist
     test-built-npm-packages:
         runs-on: ubuntu-latest
-        # needs: [lint-and-typecheck]
+        needs: [lint-and-typecheck]
         steps:
             - uses: actions/checkout@v4
               with:
@@ -274,7 +274,7 @@ jobs:
 
     test-running-unbuilt-playground-cli:
         runs-on: ubuntu-latest
-        # needs: [lint-and-typecheck]
+        needs: [lint-and-typecheck]
         steps:
             - uses: actions/checkout@v4
               with:
@@ -284,7 +284,7 @@ jobs:
 
     build:
         runs-on: ubuntu-latest
-        # needs: [lint-and-typecheck]
+        needs: [lint-and-typecheck]
         steps:
             - uses: actions/checkout@v4
               with:
diff --git a/packages/docs/site/package.json b/packages/docs/site/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "docusaurus-classic-typescript",
-	"version": "0.0.0",
+	"version": "1751036065",
 	"private": true,
 	"scripts": {
 		"docusaurus": "docusaurus",
diff --git a/packages/php-wasm/cli/package.json b/packages/php-wasm/cli/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@php-wasm/cli",
-	"version": "1.1.4",
+	"version": "1751036065",
 	"description": "PHP.wasm CLI for node.js",
 	"repository": {
 		"type": "git",
diff --git a/packages/php-wasm/compile/package.json b/packages/php-wasm/compile/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@php-wasm/compile",
-	"version": "0.1.5",
+	"version": "1751036065",
 	"description": "PHP.wasm – compilation pipeline",
 	"repository": {
 		"type": "git",
diff --git a/packages/php-wasm/fs-journal/package.json b/packages/php-wasm/fs-journal/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@php-wasm/fs-journal",
-	"version": "1.1.4",
+	"version": "1751036065",
 	"description": "Bindings to journal the PHP filesystem",
 	"repository": {
 		"type": "git",
diff --git a/packages/php-wasm/logger/package.json b/packages/php-wasm/logger/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@php-wasm/logger",
-	"version": "1.1.4",
+	"version": "1751036065",
 	"description": "A logger for PHP-wasm clients like Playground and WP-now.",
 	"repository": {
 		"type": "git",
diff --git a/packages/php-wasm/node-polyfills/package.json b/packages/php-wasm/node-polyfills/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@php-wasm/node-polyfills",
-	"version": "1.1.4",
+	"version": "1751036065",
 	"description": "PHP.wasm – polyfills for Node.js",
 	"repository": {
 		"type": "git",
diff --git a/packages/php-wasm/node/package.json b/packages/php-wasm/node/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@php-wasm/node",
-	"version": "1.1.4",
+	"version": "1751036065",
 	"description": "PHP.wasm for Node.js",
 	"repository": {
 		"type": "git",
diff --git a/packages/php-wasm/progress/package.json b/packages/php-wasm/progress/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@php-wasm/progress",
-	"version": "1.1.4",
+	"version": "1751036065",
 	"description": "PHP.wasm – loading progress monitoring",
 	"repository": {
 		"type": "git",
diff --git a/packages/php-wasm/scopes/package.json b/packages/php-wasm/scopes/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@php-wasm/scopes",
-	"version": "1.1.4",
+	"version": "1751036065",
 	"description": "PHP.wasm – scoped URLs utils",
 	"repository": {
 		"type": "git",
diff --git a/packages/php-wasm/stream-compression/package.json b/packages/php-wasm/stream-compression/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@php-wasm/stream-compression",
-	"version": "1.1.4",
+	"version": "1751036065",
 	"description": "Stream-based compression bindings.",
 	"repository": {
 		"type": "git",
diff --git a/packages/php-wasm/universal/package.json b/packages/php-wasm/universal/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@php-wasm/universal",
-	"version": "1.1.4",
+	"version": "1751036065",
 	"description": "PHP.wasm – emscripten bindings for PHP",
 	"repository": {
 		"type": "git",
diff --git a/packages/php-wasm/universal/src/lib/serialize-error.ts b/packages/php-wasm/universal/src/lib/serialize-error.ts
@@ -23,7 +23,8 @@ const list = [
 	(globalThis as any).AssertionError,
 	(globalThis as any).SystemError,
 ]
-	// Non-native Errors are used with `globalThis` because they might be missing. This filter drops them when undefined.
+	// Non-native Errors are used with `globalThis` because they might be missing. This filter drops
+	// them when undefined.
 	.filter(Boolean)
 	.map((constructor) => [constructor.name, constructor]);
 
@@ -197,7 +198,9 @@ const destroyCircular = ({
 			// Gracefully handle non-configurable errors like `DOMException`.
 			try {
 				to[key] = value;
-			} catch {}
+			} catch {
+				// ignore
+			}
 
 			continue;
 		}
@@ -288,7 +291,8 @@ export function isErrorLike(value: any) {
 	);
 }
 
-// Used as a weak check for immediately-passed objects, whereas `isErrorLike` is used for nested values to avoid bad detection
+// Used as a weak check for immediately-passed objects, whereas `isErrorLike` is used for nested
+// values to avoid bad detection
 function isMinimumViableSerializedError(value: any) {
 	// @ts-ignore
 	return (
diff --git a/packages/php-wasm/util/package.json b/packages/php-wasm/util/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@php-wasm/util",
-	"version": "1.1.4",
+	"version": "1751036065",
 	"type": "module",
 	"types": "index.d.ts",
 	"typedoc": {
diff --git a/packages/php-wasm/web-service-worker/package.json b/packages/php-wasm/web-service-worker/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@php-wasm/web-service-worker",
-	"version": "1.1.4",
+	"version": "1751036065",
 	"description": "PHP.wasm – service worker utils",
 	"repository": {
 		"type": "git",
diff --git a/packages/php-wasm/web/package.json b/packages/php-wasm/web/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@php-wasm/web",
-	"version": "1.1.4",
+	"version": "1751036065",
 	"description": "PHP.wasm for the web",
 	"repository": {
 		"type": "git",
diff --git a/packages/playground/blueprints/package.json b/packages/playground/blueprints/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@wp-playground/blueprints",
-	"version": "1.1.4",
+	"version": "1751036065",
 	"exports": {
 		".": {
 			"import": "./index.js",
diff --git a/packages/playground/cli/package.json b/packages/playground/cli/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@wp-playground/cli",
-	"version": "1.1.4",
+	"version": "1751036065",
 	"description": "WordPress Playground CLI",
 	"repository": {
 		"type": "git",
diff --git a/packages/playground/cli/src/v2.ts b/packages/playground/cli/src/v2.ts
@@ -90,36 +90,31 @@ export async function getV2Runner(): Promise<File> {
 	 * This breaks the offline mode as the static assets list is not yet updated to accommodate
 	 * for the new .phar file.
 	 */
-	try {
-		// @ts-ignore
-		const v2_runner_url = (await import('../public/blueprints.phar?url'))
-			.default;
+	// @ts-ignore
+	const v2_runner_url = (await import('../public/blueprints.phar?url'))
+		.default;
 
-		/**
-		 * Only load the v2 runner via node:fs when running in Node.js.
-		 */
-		if (typeof process !== 'undefined' && process.versions?.node) {
-			let path = v2_runner_url;
-			if (path.startsWith('/@fs/')) {
-				path = path.slice('/@fs'.length);
-			}
-			if (path.startsWith('file://')) {
-				path = path.slice('file://'.length);
-			}
-
-			const { readFile } = await import('node:fs/promises');
-			data = await readFile(path);
-		} else {
-			const response = await fetch(v2_runner_url);
-			data = await response.blob();
+	/**
+	 * Only load the v2 runner via node:fs when running in Node.js.
+	 */
+	if (typeof process !== 'undefined' && process.versions?.node) {
+		let path = v2_runner_url;
+		if (path.startsWith('/@fs/')) {
+			path = path.slice('/@fs'.length);
+		}
+		if (path.startsWith('file://')) {
+			path = path.slice('file://'.length);
 		}
-		return new File([data], `blueprints.phar`, {
-			type: 'application/zip',
-		});
-	} catch (e) {
-		console.log('e', e);
-		throw e;
+
+		const { readFile } = await import('node:fs/promises');
+		data = await readFile(path);
+	} else {
+		const response = await fetch(v2_runner_url);
+		data = await response.blob();
 	}
+	return new File([data], `blueprints.phar`, {
+		type: 'application/zip',
+	});
 }
 
 export async function runBlueprintV2(
diff --git a/packages/playground/cli/vite.config.ts b/packages/playground/cli/vite.config.ts
@@ -1,6 +1,6 @@
 /// <reference types="vitest" />
 import { join } from 'path';
-import { PluginOption, defineConfig } from 'vite';
+import { type PluginOption, defineConfig } from 'vite';
 import dts from 'vite-plugin-dts';
 
 // eslint-disable-next-line @nx/enforce-module-boundaries
@@ -61,12 +61,10 @@ const plugins = [
 	{
 		name: 'build-phars-as-URL-modules-not-data-imports',
 
-		transform(code, id, options) {
+		transform(code, id) {
 			if (id?.includes('.phar')) {
-				console.log({ id, options });
 				// @TODO don't hardcode it
 				// @TODO use URL on the web and path on Node.js
-				// @TODO use import.url in ESM and __dirname in CJS
 				return {
 					code: `
 						import { fileURLToPath } from 'url';
diff --git a/packages/playground/client/package.json b/packages/playground/client/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@wp-playground/client",
-	"version": "1.1.4",
+	"version": "1751036065",
 	"description": "WordPress Playground client",
 	"repository": {
 		"type": "git",
diff --git a/packages/playground/common/package.json b/packages/playground/common/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@wp-playground/common",
-	"version": "1.1.4",
+	"version": "1751036065",
 	"description": "Common exports and utilities for WordPress Playground",
 	"repository": {
 		"type": "git",
diff --git a/packages/playground/components/package.json b/packages/playground/components/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@wp-playground/components",
-	"version": "1.1.4",
+	"version": "1751036065",
 	"exports": {
 		".": {
 			"import": "./index.js",
diff --git a/packages/playground/php-cors-proxy/package.json b/packages/playground/php-cors-proxy/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@wp-playground/php-cors-proxy",
-	"version": "0.0.1",
+	"version": "1751036065",
 	"description": "",
 	"private": true
 }
diff --git a/packages/playground/remote/package.json b/packages/playground/remote/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@wp-playground/remote",
-	"version": "1.1.4",
+	"version": "1751036065",
 	"description": "WordPress Playground remote host",
 	"repository": {
 		"type": "git",
diff --git a/packages/playground/storage/package.json b/packages/playground/storage/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@wp-playground/storage",
-	"version": "1.1.4",
+	"version": "1751036065",
 	"description": "Bindings for storing WordPress Playground on different backends.",
 	"repository": {
 		"type": "git",
diff --git a/packages/playground/sync/package.json b/packages/playground/sync/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@wp-playground/sync",
-	"version": "0.0.1",
+	"version": "1751036065",
 	"description": "Sync changes across multiple WordPress Playground clients.",
 	"repository": {
 		"type": "git",
diff --git a/packages/playground/test-built-npm-packages/es-modules-and-vitest/package.json b/packages/playground/test-built-npm-packages/es-modules-and-vitest/package.json
@@ -19,6 +19,6 @@
 	},
 	"devDependencies": {
 		"@types/node": "^22.14.1",
-		"@wp-playground/cli": "https://wordpress-playground-packages.atomicsites.blog/packages/v1.0.36/@wp-playground-cli-1.0.36.tar.gz"
+		"@wp-playground/cli": "http://127.0.0.1:9934/1751036065/v1751036065/@wp-playground-cli-1751036065.tar.gz"
 	}
 }
diff --git a/packages/playground/website/package.json b/packages/playground/website/package.json
@@ -1,5 +1,5 @@
 {
-	"version": "0.0.1",
+	"version": "1751036065",
 	"type": "commonjs",
 	"private": true
 }
diff --git a/packages/playground/wordpress-builds/package.json b/packages/playground/wordpress-builds/package.json
@@ -1,7 +1,7 @@
 {
 	"private": true,
 	"name": "@wp-playground/wordpress-builds",
-	"version": "0.9.23",
+	"version": "1751036065",
 	"description": "WordPress Compilation Pipeline for the WordPress Playground",
 	"repository": {
 		"type": "git",
diff --git a/packages/playground/wordpress/package.json b/packages/playground/wordpress/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "@wp-playground/wordpress",
-	"version": "1.1.4",
+	"version": "1751036065",
 	"description": "WordPress-related plumbing for WordPress Playground",
 	"repository": {
 		"type": "git",

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "docusaurus-classic-typescript",`
`3`		`- "version": "0.0.0",`
	`3`	`+ "version": "1751036065",`
`4`	`4`	`"private": true,`
`5`	`5`	`"scripts": {`
`6`	`6`	`"docusaurus": "docusaurus",`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@php-wasm/cli",`
`3`		`- "version": "1.1.4",`
	`3`	`+ "version": "1751036065",`
`4`	`4`	`"description": "PHP.wasm CLI for node.js",`
`5`	`5`	`"repository": {`
`6`	`6`	`"type": "git",`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@php-wasm/compile",`
`3`		`- "version": "0.1.5",`
	`3`	`+ "version": "1751036065",`
`4`	`4`	`"description": "PHP.wasm – compilation pipeline",`
`5`	`5`	`"repository": {`
`6`	`6`	`"type": "git",`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@php-wasm/fs-journal",`
`3`		`- "version": "1.1.4",`
	`3`	`+ "version": "1751036065",`
`4`	`4`	`"description": "Bindings to journal the PHP filesystem",`
`5`	`5`	`"repository": {`
`6`	`6`	`"type": "git",`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@php-wasm/logger",`
`3`		`- "version": "1.1.4",`
	`3`	`+ "version": "1751036065",`
`4`	`4`	`"description": "A logger for PHP-wasm clients like Playground and WP-now.",`
`5`	`5`	`"repository": {`
`6`	`6`	`"type": "git",`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@php-wasm/node-polyfills",`
`3`		`- "version": "1.1.4",`
	`3`	`+ "version": "1751036065",`
`4`	`4`	`"description": "PHP.wasm – polyfills for Node.js",`
`5`	`5`	`"repository": {`
`6`	`6`	`"type": "git",`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@php-wasm/node",`
`3`		`- "version": "1.1.4",`
	`3`	`+ "version": "1751036065",`
`4`	`4`	`"description": "PHP.wasm for Node.js",`
`5`	`5`	`"repository": {`
`6`	`6`	`"type": "git",`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@php-wasm/progress",`
`3`		`- "version": "1.1.4",`
	`3`	`+ "version": "1751036065",`
`4`	`4`	`"description": "PHP.wasm – loading progress monitoring",`
`5`	`5`	`"repository": {`
`6`	`6`	`"type": "git",`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@php-wasm/scopes",`
`3`		`- "version": "1.1.4",`
	`3`	`+ "version": "1751036065",`
`4`	`4`	`"description": "PHP.wasm – scoped URLs utils",`
`5`	`5`	`"repository": {`
`6`	`6`	`"type": "git",`