clay / claycli / 31741525467

87%

master: 87%

Pull #256

github

🍕 Drop the mountInEditMode escape hatch

Nothing should ever want component client.js running in edit mode. It was
never possible under clay compile, so there is no prior behavior to preserve,
and the option had no caller — it only existed because the fix was framed as
opt-in before it became the default.

A global flag is also the wrong granularity for the one need that sounds
plausible. A component that wants client-rendered output visible while editing
should provide it through kiln.js, not by running every ad call, analytics
beacon and comment embed on the page inside the editing surface.

The stale-manifest fallback stays. When public/js predates the no-mount entry,
serving no initializers at all would break Kiln outright, so that path
degrades rather than fails.

Co-authored-by: Cursor <cursoragent@cursor.com>

760 of 945 branches covered (80.42%)

Branch coverage included in aggregate %.

22 of 24 new or added lines in 2 files covered. (91.67%)

1633 of 1805 relevant lines covered (90.47%)

10.83 hits per line