Fixed current window issue

Author: shai-almog

Ports/JavaScriptPort/STATUS.md

@@ -16,7 +16,13 @@ Current State
 - Worker host-call misses now emit explicit first-failure diagnostics:
   - `PARPAR:DIAG:FIRST_FAILURE:category=host_call_unhandled`
   - `PARPAR:DIAG:FIRST_FAILURE:symbol=<nativeSymbol>`
-- Latest CI artifacts still ran `main-thread-mode` (before this change) and timed out before `CN1SS:SUITE:FINISHED`.
+- Worker native-rebind bug identified and fixed:
+  - Cause: `translated_app.js` redefined native stubs after `port.js` bind phase in worker startup order.
+  - Fix: `worker.js` now calls `__parparInstallNativeBindings()` after imports, before handling `start`.
+- Latest CI artifacts now run worker mode but fail early before suite start with:
+  - `PARPAR:DIAG:FIRST_FAILURE:category=runtime_error`
+  - `TypeError: Cannot read properties of null (reading '__classDef')`
+  - stack rooted in `HTML5Implementation.__init` after `Window.current()` returned `null`.
 - Existing form-constructor recovery diagnostics remain active in `port.js` and are still relevant while migrating.
 
 Next Steps
@@ -25,23 +31,27 @@ Next Steps
 1. Validate worker-only boot in CI and local:
    - Required markers: `PARPAR:worker-mode`, `PARPAR:DIAG:BOOT:bridgeMode=worker`.
    - Any `main-thread-mode` marker now indicates stale artifact or wrong bundle.
-2. Separate VM/EDT execution from main-thread host services cleanly:
+2. Confirm worker native rebind fix is present in produced bundle:
+   - In generated `worker.js`, ensure `__parparInstallNativeBindings()` is invoked after imports and before `start`.
+   - This must eliminate `Window.current()` null stubs from startup execution.
+3. Separate VM/EDT execution from main-thread host services cleanly:
    - Keep VM/EDT scheduling in worker.
    - Ensure main-thread browser APIs are reached through explicit host-call handlers rather than direct worker DOM access.
-3. Re-triage screenshot correctness in worker mode only:
+4. Re-triage screenshot correctness in worker mode only:
    - Re-run screenshot suite and classify first blocker using the existing `TOP_BLOCKER` output.
    - Prioritize deterministic runtime failures before throughput tuning.
-4. Restore full screenshot count and correctness:
+5. Restore full screenshot count and correctness:
    - Exit gate remains `CN1SS:SUITE:FINISHED` with expected screenshot artifacts and no `BROWSER:PARPAR_ERROR`.
 
 Important Notes
 --------------
 
 - Current CI artifact (`~/Downloads/javascript-ui-tests/browser.log`) shows:
-  - `PARPAR:main-thread-mode`
-  - `PARPAR:DIAG:BOOT:bridgeMode=main-thread`
-  - timeout with `TOP_BLOCKER=unknown|none|none`
-- This is consistent with the new migration priority: enforce worker mode first, then debug screenshot behavior.
+  - `PARPAR:worker-mode`
+  - `PARPAR:DIAG:BOOT:bridgeMode=worker`
+  - `TOP_BLOCKER=runtime_error|none|none`
+  - first crash in `HTML5Implementation.__init` due null window wrapper.
+- This is consistent with native rebind order being a primary startup blocker in worker mode.
 
 Known Important Context
 -----------------------

vm/ByteCodeTranslator/src/javascript/worker.js

@@ -1,6 +1,9 @@
 self.window = self;
 self.global = self;
 /*__IMPORTS__*/
+if (typeof self.__parparInstallNativeBindings === 'function') {
+  self.__parparInstallNativeBindings();
+}
 self.onmessage = function(event) {
   if (!event || !event.data) {
     return;

vm/tests/src/test/java/com/codename1/tools/translator/JavascriptTargetIntegrationTest.java

@@ -82,6 +82,8 @@ void generatesBrowserBundleForJavascriptTarget(CompilerHelper.CompilerConfig con
                 "Unsupported filesystem natives should fail with an explicit JS-mode message when translated");
         assertTrue(worker.contains("importScripts('parparvm_runtime.js');"),
                 "Worker bootstrap should load the runtime first");
+        assertTrue(worker.contains("__parparInstallNativeBindings"),
+                "Worker bootstrap should reapply runtime native bindings after translated app load");
         assertTrue(index.contains("browser_bridge.js") && index.contains("js/fontmetrics.js") && index.contains("codenameone-canvas"),
                 "Generated host page should use the JavaScript port browser shell assets");
         assertTrue(browserBridge.contains("cn1HostBridge") && browserBridge.contains("host-callback") && browserBridge.contains("host-call"),