fix: restore compact JS placeholders in session export template#49961
fix: restore compact JS placeholders in session export template#49961martenzi wants to merge 2 commits into
Conversation
Automated formatters like Prettier can expand the single-line placeholders into multi-line blocks with semicolons. This prevents the literal string replacement in the export code from matching.
Greptile SummaryThis PR is a targeted bug fix that restores the single-line The fix is correct: the injection logic in
Confidence Score: 5/5
Last reviewed commit: "fix(export): restore..." |
…r corruption
This prevents Prettier from reformatting {{PLACEHOLDER}} blocks into multi-line scripts, which breaks the string-replacement logic in the export command. Follow-up to PR openclaw#49961.
…r corruption
This prevents Prettier from reformatting {{PLACEHOLDER}} blocks into multi-line scripts, which breaks the string-replacement logic in the export command. Follow-up to PR openclaw#49961.
|
Thanks for the fix here. This path is now covered by #41861, which landed the canonical export HTML placeholder repair with regression coverage. Closing this PR as superseded so we keep one landed implementation and preserve contributor credit in the cluster record. |
Fixes #49957
The session export template at
src/auto-reply/reply/export-html/template.htmlhad its JS placeholders reformatted into multi-line blocks. This prevented the.replace()calls in the export logic from finding and injecting the vendored JS, resulting in empty exports in the browser.This PR restores the single-line format.