Rotate PHP runtime on crash so it can continue to satisfy subsequent requests

[brandonpayton]

We would like Playground to be more resilient in the face of crashes. We've encountered a variety of memory-related crashes, but this issue was inspired most recently by #1332.

We need to update the rotatePHPRuntime() function to rotate when it detects a crash.

[brandonpayton]

I'm assigning this issue to myself, but feel free to grab it if I haven't started on it yet.

[bgrgicak]

@brandonpayton would something like this work?
I'm working on a error modal task and wanted to check if rotating the runtime would help me get PHP logs (it didn't).

diff --git a/packages/php-wasm/universal/src/lib/base-php.ts b/packages/php-wasm/universal/src/lib/base-php.ts
index 9f5ff746..b9832903 100644
--- a/packages/php-wasm/universal/src/lib/base-php.ts
+++ b/packages/php-wasm/universal/src/lib/base-php.ts
@@ -31,6 +31,7 @@ import {
 } from '@php-wasm/util';
 import { PHPRequestHandler } from './php-request-handler';
 import { logger } from '@php-wasm/logger';
+import { rotatePHPRuntime } from './rotate-php-runtime';
 
 const STRING = 'string';
 const NUMBER = 'number';
@@ -361,6 +362,14 @@ export abstract class BasePHP implements IsomorphicLocalPHP, Disposable {
                                // Distinguish between PHP request and PHP-wasm errors
                                source: (e as any).source ?? 'php-wasm',
                        });
+                       if (this.requestHandler) {
+                               rotatePHPRuntime({
+                                       php: this,
+                                       cwd: this.requestHandler.documentRoot,
+                                       recreateRuntime: this[__private__dont__use],
+                                       maxRequests: 400,
+                               });
+                       }
                        throw e;
                } finally {
                        try {

[adamziel]

This one seems stale so I'll clear the assignee