Skip to content

EXPERIMENT (do not merge): pre-rmcp control for the WebKitGTK E2E regression - #67

Closed
runyourempire wants to merge 1 commit into
mainfrom
exp/pre-rmcp-control
Closed

EXPERIMENT (do not merge): pre-rmcp control for the WebKitGTK E2E regression#67
runyourempire wants to merge 1 commit into
mainfrom
exp/pre-rmcp-control

Conversation

@runyourempire

Copy link
Copy Markdown
Collaborator

Control experiment — DO NOT MERGE.

Tree is the pre-rmcp-upgrade commit 9f9bd83 (PR #61) unchanged; only an empty commit on top to force divergence, so CI runs the OLD code on today's ubuntu runner image.

Why: the E2E host-crash test webview_reload_during_introspection_is_safe began failing (~44-57%) on ubuntu with the rmcp 3.1.2 train. But ubuntu-latest also rolled at the same time (image 20260714.240.1 -> 20260810.271.1) — a confound. webkit2gtk is identical (2.52.3) across passing and failing runs, but the rest of the image is not.

  • E2E ubuntu fails here -> the environment regressed, rmcp exonerated.
  • E2E ubuntu passes repeatedly here -> rmcp 3.1.2 is implicated, and it ships to consumers.

…mage

Control for the WebKitGTK E2E regression attribution. Tree is unchanged from
9f9bd83 (pre-rmcp-3.1.2). Empty commit only, so CI runs the OLD code on the
CURRENT ubuntu image.

Co-Authored-By: Claude Fable 5 <[email protected]>
@runyourempire

Copy link
Copy Markdown
Collaborator Author

Experiment concluded — result recorded in #68.

Outcome: rmcp 3.1.2 is exonerated. This branch runs the pre-rmcp commit (9f9bd83) unchanged on today's ubuntu runner, and it fails the same two E2E tests (webview_reload_during_introspection_is_safe + window_state_check) while macOS and Windows pass. So the host crash is a long-standing bug present in released 0.8.7, not a regression introduced by the rmcp upgrade.

This corroborates the local A/B on Ubuntu 24.04 + WebKitGTK 2.52, where the pre-rmcp build died 6/8 and the current build 5/8 — statistically identical.

Root cause and fix in #68: concurrent main-thread round trips corrupt the heap; serializing them fixes it (0/28).

@runyourempire
runyourempire deleted the exp/pre-rmcp-control branch August 12, 2026 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant