clay / claycli / 31738592455

87%

master: 87%

Pull #256

github

🍕 Stop component client.js from running in Kiln edit mode

The Vite pipeline serves the view-mode bootstrap in edit mode, and that
bootstrap calls mountComponentModules() at module scope — so every on-page
component's client.js executes as soon as the module evaluates, while an
editor is in Kiln.

The legacy clay compile pipeline resolved client.js for view mode only (the
edit branch of getDependencies() ships model.js, kiln.js and kiln plugins,
but no client bundle and no _client-init.js), so this is a regression from
clay compile. It runs ads (GPT injects an iframe per slot), analytics and
comment embeds inside the editing surface, where they mutate the DOM Kiln
is trying to decorate.

Generate a second entry, .clay/vite-bootstrap-no-mount.js, from the same
initializer prelude — the window.modules stub, _env-init, _globals-init and
the sticky-events shim, all of which edit mode genuinely needs — but without
the mount runtime, and serve that in edit mode. resolveModuleScripts() takes
mountInEditMode to opt back in, and falls back to the mounting bootstrap when
public/js was built by a claycli predating the new entry.

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.91 hits per line