fix: rewrite aem.page/aem.live image srcs in editor to prevent 401 errors

[andreituicu]

Description

Images stored with *.aem.page or *.aem.live URLs now display correctly in the editor without triggering 401 errors.

When DA documents contain images served from AEM preview/publish hosts, the editor lacks the auth those hosts require. This PR rewrites those image URLs to their *.preview.da.live equivalent at render time via ProseMirror nodeViews — display-only, never written back to the document or synced via collab.

How Has This Been Tested?

Manually with: https://fximg--da-live--adobe.aem.live/edit#/andreituicu/da-test/mediabus

Screenshots (if appropriate):

The image with the cat is behind helix auth: https://main--da-test--andreituicu.aem.page/media_1222da127066e69ab4693b1aa98ad4cd6fdfe1fe4.jpg?width=2000&format=webply&optimize=medium

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• I have signed the Adobe Open Source CLA.
• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[aem-code-sync]

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

• Re-sync branch

Commits

• dfe691e ✅ (latest)
• 91afe27 ✅
• 7e714ac ✅
• aee0536 ✅
• 5910474 ✅
• b08d1d9 ✅
• 836289a ✅
• c22aa1e ✅
• 6685bd9 ✅
• 176d3be ❌

[kptdobe]

Missing tests...

Also, .preview.da.live maps to da-ue worker. I do not see the relationship here. It feels like this worker is a place for a lot of things. If we want .preview to become a central place for our editor, I think we should make a clean worker that includes only what's needed.

[andreituicu]

Thank you for the review!

Also, .preview.da.live maps to da-ue worker. I do not see the relationship here. It feels like this worker is a place for a lot of things. If we want .preview to become a central place for our editor, I think we should make a clean worker that includes only what's needed.

This was a decision in the last Helix hackathon to have the preview proxy and the UE proxy in the same worker, because they have a lot in common. They actually have more in common the differences.

As with everything it is in a bit of transition phase, but essentially they both:

1. Provide access to content behind auth (both IMS and Helix), in the context of editors (DA and UE)
2. They both proxy context and merge DA with Helix
3. They can both be used for injecting later scripts for live preview, quick edit, UE instrumentation, on top of the content served.

Probably the only essential difference in the long-run is that the UE side of that worker, injects instrumentation for the HTML that is required for the UE editor.

The tests are a fair point, will add them.