Added debugging to form show

Author: shai-almog

Ports/JavaScriptPort/STATUS.md

@@ -16,6 +16,7 @@ Current State
   - `TOP_BLOCKER=none|none|none`
 - The screenshot pipeline now decodes/report-generates reliably from logs, but screenshot content is still mostly wrong (white-frame capture path is still being used in CI artifacts).
 - New patch (not yet CI-validated in this document revision): worker-side fallback screenshot path now waits for a host-side UI-settle barrier before ready-callback dispatch and before canvas capture, to avoid pre-paint frame capture.
+- New patch (not yet CI-validated in this document revision): `BaseTest.registerReadyCallback` fallback now attempts to force-show the target `Form` before UI-settle/callback dispatch to avoid running screenshot setup against a stale previous form.
 
 What Was Fixed In This Pass
 ---------------------------
@@ -86,6 +87,18 @@ What Was Fixed In This Pass
    - Motivation:
      - CI showed near-identical screenshot payloads (same hash/size) indicating repeated capture of a stale frame rather than per-test painted UI.
 
+8. Added force-show step in ready-callback fallback for per-test form activation.
+   - File:
+     - `Ports/JavaScriptPort/src/main/webapp/port.js`
+   - Changes:
+     - In `BaseTest.registerReadyCallbackImmediate`, attempt `Form.show()` before UI-settle and callback execution.
+     - Added diagnostics:
+       - `PARPAR:DIAG:FALLBACK:baseTestRegisterReady:forceShow=1:...`
+       - `PARPAR:DIAG:FALLBACK:baseTestRegisterReady:forceShow=0:...`
+       - `PARPAR:DIAG:FALLBACK:baseTestRegisterReady:forceShow=formMissing:...`
+   - Motivation:
+     - CI diagnostics showed canvas signature locking to a single frame after initial tests, consistent with callbacks running while stale form remains active.
+
 Known Failing Symptoms (Latest CI Logs/Artifacts)
 -------------------------------------------------
 
@@ -110,11 +123,14 @@ Priority Next Steps
 1. Validate CI output after UI-settle barrier:
    - Expect non-identical PNG hashes for distinct tests.
    - Confirm `settleChanged` and `canvasSig` diagnostics vary across tests.
-2. If white-frame reuse persists, capture and compare per-test `settleSig`/`canvasSig` to identify whether paint is not happening or capture target is wrong.
-3. Fix per-test null receiver/init path (`__classDef` null) at first failing stack, not via broad fallbacks.
-4. Fix missing `Button.initLaf(UIManager)` symbol resolution in worker runtime path.
-5. Fix worker-mode orientation lock path so DOM access is host-bridge mediated (no direct `document` access in worker).
-6. Confirm VM completeness stability in CI with parser/runtime patches (`expected 7` consistently).
+2. Validate force-show diagnostics in CI:
+   - Expect `forceShow=1` for screenshot tests with forms.
+   - Expect per-test `canvasSig` to vary for graphics tests if activation works.
+3. If white-frame reuse persists, capture and compare per-test `settleSig`/`canvasSig` to identify whether paint is not happening or capture target is wrong.
+4. Fix per-test null receiver/init path (`__classDef` null) at first failing stack, not via broad fallbacks.
+5. Fix missing `Button.initLaf(UIManager)` symbol resolution in worker runtime path.
+6. Fix worker-mode orientation lock path so DOM access is host-bridge mediated (no direct `document` access in worker).
+7. Confirm VM completeness stability in CI with parser/runtime patches (`expected 7` consistently).
 
 Files Touched In This Pass
 --------------------------

Ports/JavaScriptPort/src/main/webapp/port.js

@@ -3236,12 +3236,37 @@ bindCiFallback("BaseTest.registerReadyCallbackImmediate", [
   baseTestRegisterReadyCallbackMethodId + "__impl"
 ], function*(_baseTest, _form, callback) {
   const activeTest = normalizeCn1ssTestName(cn1ssActiveTestName || "default");
+  if (_form && _form.__class) {
+    let showInvoked = false;
+    const showMethodIds = [
+      "cn1_com_codename1_ui_Form_show",
+      "cn1_com_codename1_ui_Form_show__"
+    ];
+    for (let i = 0; i < showMethodIds.length && !showInvoked; i++) {
+      const methodId = showMethodIds[i];
+      try {
+        const showMethod = jvm.resolveVirtual(_form.__class, methodId);
+        if (typeof showMethod === "function") {
+          yield* showMethod(_form);
+          showInvoked = true;
+          emitDiagLine("PARPAR:DIAG:FALLBACK:baseTestRegisterReady:forceShow=1:test=" + activeTest + ":method=" + methodId);
+        }
+      } catch (_showErr) {
+        // Try next candidate method id.
+      }
+    }
+    if (!showInvoked) {
+      emitDiagLine("PARPAR:DIAG:FALLBACK:baseTestRegisterReady:forceShow=0:test=" + activeTest);
+    }
+  } else {
+    emitDiagLine("PARPAR:DIAG:FALLBACK:baseTestRegisterReady:forceShow=formMissing:test=" + activeTest);
+  }
   let settleChanged = "na";
   if (jvm && typeof jvm.invokeHostNative === "function") {
     try {
       const settleResult = yield jvm.invokeHostNative("__cn1_wait_for_ui_settle__", [{
         reason: "ready:" + activeTest,
-        maxFrames: 24,
+        maxFrames: 36,
         stableFrames: 2
       }]);
       if (settleResult && settleResult.changedFromPrevious != null) {