Skip to content

perf: split plugin loading to get editor rendered faster; GET the doc for permissions and metadata check#741

Open
chrischrischris wants to merge 12 commits intomainfrom
parrcon
Open

perf: split plugin loading to get editor rendered faster; GET the doc for permissions and metadata check#741
chrischrischris wants to merge 12 commits intomainfrom
parrcon

Conversation

@chrischrischris
Copy link
Contributor

@chrischrischris chrischrischris commented Jan 30, 2026

  1. Do async plugin loading
  2. Change getting permissions from HEAD to GET in order to prep for BYO Editor. Also preps for future perf enhancement of rendering doc in prosemirror before websocket connection finishes.
  3. Add ESM Module loading error detection, which then sets a da-cache=off cookie to force the browser to not use the cache (pending cloudflare worker update).

After merging this PR, this nexter PR can be merged: adobe/da-nx#160

Use `PluginKey` to ensure plugin state and view persistence during `state.reconfigure`. Without a key, ProseMirror treats the plugin as new, causing unnecessary teardown and re-initialization.
@aem-code-sync
Copy link

aem-code-sync bot commented Jan 30, 2026

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

  • Re-run PSI checks
  • Re-sync branch
Commits

chrischrischris added a commit to adobe/da-nx that referenced this pull request Jan 30, 2026
Re-throws `SyntaxError: The requested module ... does not provide an export named ...`

This is so the da-live code in adobe/da-live#741 will then attempt to reload the page without any local caching.
chrischrischris added a commit to adobe/da-nx that referenced this pull request Feb 18, 2026
Re-throws `SyntaxError: The requested module ... does not provide an export named ...`

This is so the da-live code in adobe/da-live#741 will then attempt to reload the page without any local caching.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant