[AdrienClairembault]

Extracted from #26020 to keep its scope smaller and ease reviews.

This wrapper is outdated and does nothing here, I think it can be safely removed.

Deleting it allow us to use Html::scriptBlock to generate the import map (because the import map script must be valid JSON and CDATA breaks it), like this:

[AdrienClairembault]

@coderabbitai review

[coderabbitai]

Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[qodo-free-for-open-source-projects]

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Powered by Qodo

[coderabbitai]

No actionable comments were generated in the recent review.

Summary

Summary by CodeRabbit

• Bug Fixes
  • Script blocks are now output without extra newline or CDATA wrapper strings, resulting in cleaner generated HTML.

Html::scriptBlock() no longer wraps script content with newline and CDATA marker strings before generating the script element.

Merge Risk: Minimal · up to 3fc7b

This change drops an outdated CDATA wrapper from generated script blocks so JSON-bearing scripts such as import maps are valid. No concrete merge-blocking risk was identified.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

• X
• Mastodon
• Reddit
• LinkedIn