Add Swarmakit Vue presets package#4670
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting
| <body> | ||
| {shell} | ||
| <script type="application/json" id="swarmakit-manifest">{manifest_json}</script> | ||
| {script} | ||
| </body> |
There was a problem hiding this comment.
Escape manifest JSON before embedding in script tag
The rendered page writes the serialized manifest straight into a <script type="application/json"> tag. Because manifest_json is produced with json.dumps and inserted verbatim, any tile props containing the substring </script> (for example, user-provided grid cell text) will close the script tag early and let the rest of the payload execute as HTML/JS, creating an XSS vector whenever untrusted data flows into the manifest. HTML‑escape or replace </ sequences before embedding, or serve the manifest via a separate endpoint instead of inline.
Useful? React with 👍 / 👎.
…mponents/2025-10-08
…mponents/2025-10-08
93c3606 to
99fdb63
Compare
99c5e35 to
0f89d2e
Compare
…onents/2025-10-08
Summary
layout-engine-swarmakit-vuepackage with Swarmakit Vue preset mappings, widgets, and runtime helpersTesting
https://chatgpt.com/codex/tasks/task_e_68e5d31c2d888326beb596f648135fbf