Skip to content

Commit 2382137

Browse files
committed
Added debugging to form show
1 parent 12144ce commit 2382137

1 file changed

Lines changed: 10 additions & 0 deletions

File tree

Ports/JavaScriptPort/STATUS.md

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ Current State
1616
- `TOP_BLOCKER=none|none|none`
1717
- 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).
1818
- 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.
19+
- Note: a short-lived `forceShow()` experiment in `BaseTest.registerReadyCallback` was reverted because it caused re-entrant callback loops and prevented `CN1SS:SUITE:FINISHED`.
1920

2021
What Was Fixed In This Pass
2122
---------------------------
@@ -86,6 +87,15 @@ What Was Fixed In This Pass
8687
- Motivation:
8788
- CI showed near-identical screenshot payloads (same hash/size) indicating repeated capture of a stale frame rather than per-test painted UI.
8889

90+
8. Reverted force-show experiment due re-entrancy regression.
91+
- File:
92+
- `Ports/JavaScriptPort/src/main/webapp/port.js`
93+
- Observation:
94+
- Calling `Form.show()` inside `BaseTest.registerReadyCallbackImmediate` caused repeated `registerReadyCallback`/settle cycles on the same test and CI timeout before suite completion.
95+
- Action:
96+
- Removed `forceShow()` from this fallback path.
97+
- Kept the host-side settle barrier and diagnostics (non-recursive).
98+
8999
Known Failing Symptoms (Latest CI Logs/Artifacts)
90100
-------------------------------------------------
91101

0 commit comments

Comments
 (0)