revert: share assets across the three web targets (#3908)#3937
Conversation
This reverts commit 6677402.
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Our agent can fix these. Install it.
Gates Passed
3 Quality Gates Passed
Quality Gate Profile: The Bare Minimum
Install CodeScene MCP: safeguard and uplift AI-generated code. Catch issues early with our IDE extension and CLI tool.
|
it's needed because of the flagship app not respecting the manifest.wepabb with an harcoded path to ./index.html |
BundleMonFiles added (21)
Total files change +6.53MB Groups added (3)
Final result: ✅ View report in BundleMon website ➡️ |
Context
PR #3908 merged a single-compilation build that shares assets across the main, public and intents targets. We need to ship 1.102.1 without that change, so it is reverted here on the release branch.
Solution
Reverts #3908 in full, restoring the three separate build environments, the per-target Excalidraw font copies, the
/static-less manifest, the eager PDF view, and the public build without the PDF editor. The later version bump to 1.102.1 is preserved.