MWPW-191059: Switch Studio preview from Odin to Freyja v2 by Axelcureno · Pull Request #741 · adobecom/mas

Axelcureno · 2026-04-03T18:21:11Z

Resolves https://jira.corp.adobe.com/browse/MWPW-191059

Migrate MAS Studio preview from Odin to Freyja v2 API. Benchmarking confirmed 41.4× speedup on worst-case (catalog all-hydrated: 15.7s → 380ms).

Changes:

Switch preview endpoint from Odin to Freyja v2
Move authToken under preview namespace (preview.authToken)
Remove Odin fallback logic from internalFetch — clean switch, no dual-path
Remove compareBackends from fragment-client.js, move standalone bench to studio/bench/
Cache IMS token as masFreyjaToken in sessionStorage for both Studio and aem.page contexts
Conditional freyjaToken injection in aem-fragment.js

Test URLs:

Before (main): https://main--mas--adobecom.aem.live/studio.html
After (branch): https://mwpw-191059--mas--adobecom.aem.live/studio.html
Benchmark: https://mwpw-191059--mas--adobecom.aem.live/studio/bench/freyja-bench.html
aem.page preview: https://mwpw-191059--mas--adobecom.aem.page/studio.html?mas.preview=on

Checklist:

Code follows project conventions
Tests pass locally (230 io/www + 1226 web-components)
Linter runs without errors
Tested on Before/After URLs
Verified aem.page preview works with IMS token

aem-code-sync · 2026-04-03T18:21:15Z

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

d2d3fe5 ✅ (latest)
cae09d2 ✅
1c0bd44 ✅
68aee50 ✅
8eb6848 ✅
e060cb1 ✅
c72cddf ✅
03ae91f ✅
44cdd31 ✅
2196ec6 ✅
d2976dd ✅
89b0a66 ✅
60957df ✅
67bd494 ✅
4a8b27b ✅
fecc895 ✅
e0a787d ✅
6d05b53 ✅
ba6ff8f ✅
41ba251 ✅
a7dd497 ✅
152523b ✅

codecov · 2026-04-03T18:25:03Z

Codecov Report

❌ Patch coverage is 62.50000% with 12 lines in your changes missing coverage. Please review.
✅ Project coverage is 87.23%. Comparing base (66ef1ca) to head (1c0bd44).

Files with missing lines	Patch %	Lines
web-components/src/aem-fragment.js	0.00%	7 Missing ⚠️
web-components/src/ims.js	80.95%	4 Missing ⚠️
studio/src/mas-repository.js	75.00%	1 Missing ⚠️

❌ Your patch check has failed because the patch coverage (62.50%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #741      +/-   ##
==========================================
+ Coverage   87.22%   87.23%   +0.01%     
==========================================
  Files         206      206              
  Lines       61740    61759      +19     
==========================================
+ Hits        53851    53876      +25     
+ Misses       7889     7883       -6

Files with missing lines	Coverage Δ
studio/src/mas-repository.js	`61.94% <75.00%> (+0.02%)`	⬆️
web-components/src/ims.js	`94.28% <80.95%> (-5.72%)`	⬇️
web-components/src/aem-fragment.js	`85.28% <0.00%> (-0.87%)`	⬇️

... and 4 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 66ef1ca...1c0bd44. Read the comment docs.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

…on plan

…k in internalFetch

…URLs

…din fallback

…nchmark tool for Freyja vs Odin comparison

…ble proxy port freyja-bench.html: browser-based latency tool comparing Freyja v2 vs Odin, token injected via sessionStorage (never URL params) proxy-server.mjs: PROXY_PORT env var support so proxy can run on non-8080 ports

Automatically extract and persist the IMS access token to sessionStorage.masFreyjaToken when IMS completes initialization. This ensures Freyja requests have the required authentication token immediately, preventing 403 Forbidden responses and unnecessary fallback to Odin Preview. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

Move token caching from imsReady() to imsSignedIn() to ensure the token is cached only after the user is confirmed to be signed in, not just when IMS initializes. This ensures we have a valid, authorized token before attempting to use it. Previously: Token cached during IMS initialization, may be invalid Now: Token cached only after isSignedInUser() returns true Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

Update fragment-client.js to use Freyja preview endpoint (localhost:9006) when a Freyja token is provided, with fallback to Odin. - Pass Freyja token from sessionStorage through aem-fragment.js generatePreview() - In fragment-client.js previewFragment(), check for freyjaToken in context - If token available: use Freyja endpoint and add Authorization header - If no token: fall back to Odin (existing behavior) Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

The DEFAULT_HEADERS containing the Authorization token was being set on the main context but lost during structuredClone when creating initContext for transformers. This caused fetch requests to Freyja to fail with 403 Forbidden. The fix preserves DEFAULT_HEADERS when creating initContext so the Authorization header is passed through the entire pipeline. Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

Replace hardcoded localhost:9006 with FREYJA_PREVIEW_URL constant pointing to the production Freyja AEM Cloud instance: https://preview-p22655-e59433.adobeaemcloud.com/adobe/contentFragments Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

npeltier

results look promising but:

i would make authToken part of the preview configuration, as it's only for preview,
io common fetch should not be odin specific (it's used with other hosts), nor preview specific, i would remove the fallback all together, and just make switch from one to the other easy enough,
please add tests URLs so we can test your change (not sure how easy it is)
don't forget this should be working on aem.page URLs in milo as well! right now it's only considering studio,
please move benchmarks to a dedicated place, leaving fragment-client.js to what it does

…lidate auth under preview - Move authToken under preview namespace (preview.authToken) - Remove Odin fallback logic from internalFetch - Remove compareBackends benchmark from fragment-client.js - Move freyja-bench.html to studio/bench/ - Fix prettier formatting in ims.js

honstar · 2026-04-10T08:40:51Z

Please address formatting issues and improve code coverage.

honstar · 2026-04-10T09:53:32Z

@@ -0,0 +1,703 @@
+# Freyja v2 Preview Migration — Implementation Plan


Just curious: what AI agent/workflow did you use for these markdown files?

https://github.com/obra/superpowers
Sorry, I will remove this and add a rule to not commit these files.

honstar · 2026-04-10T09:55:46Z


 const FRAGMENT_URL_PREFIX = 'https://odin.adobe.com/adobe/sites/fragments';

+const FREYJA_PREVIEW_URL = 'https://preview-p22655-e59433.adobeaemcloud.com/adobe/contentFragments';


This is hardcoded to prod, what about supporting non-prod environments via aem.env?

it's not really prod, but "preview" freyja
prod is odin.adobe.com

Well, with prod I meant the prod environment in AEMaaCS speak. e59433 is prod, regardless if it's odin or freyja.

honstar · 2026-04-10T09:58:54Z

@@ -0,0 +1,157 @@
+<!doctype html>


When trying to run this via https://mwpw-191059--mas--adobecom.aem.live/studio/bench/freyja-bench.html, all requests from the feature branch to freyja preview fail with CORS errors.

npeltier

what i meant by aem.page test link @Axelcureno is to have
https://main--cc--adobecom.aem.page/sa_ar/products/catalog?maslibs=mwpw-191059--mas--adobecom
working (which isn't atm)

npeltier · 2026-04-13T12:24:34Z

also @Axelcureno, as discussed, performance is one part of the fix, we need to know how much time it takes from when content is changed on author to when content is previewed on that freyja server.
Once we do:

either it's acceptabe time (let's say < 10s),
either we need to mitigate with authors,
either we need to auto-publish to preview freyja when we save

Axelcureno · 2026-04-13T20:33:32Z

We are blocked by product team who needs to make more changes to enable Freyja for us, so the PR is not yet ready for review. I will in the meantime address the comments. Once we are unblocked I will request reviews. @honstar @npeltier

Freyja preview URL was hardcoded to the prod AEM environment (e59433). Adding freyjaUrl(env) helper in paths.js that maps qa/stage/prod to their respective AEM Cloud env IDs, and wiring it to the existing aem.env URL param in fragment-client.js so previews hit the correct Freyja instance.

…citly passed

yesil

@Axelcureno please see ODIN-949, let's connect if you need more details about my change request.

yesil · 2026-04-17T11:39:45Z

        /* c8 ignore next 10 */
        function poll() {
            if (window.adobeIMS?.initialized) {
+                const token = window.adobeIMS?.getAccessToken?.()?.token;


ims.js is meant to be used for loggedin users and is not freyja token.

preview on acom stage should work for both logged-out and logged-in users, so we cannot use an access token.

yesil · 2026-04-17T11:41:46Z

        const fragmentClientUrl = this.getFragmentClientUrl();
        const { previewFragment } = await import(fragmentClientUrl);
-        const data = await previewFragment(this.#fragmentId, {
+        const freyjaToken = sessionStorage.getItem('masFreyjaToken');


Please see ODIN-949, aem-fragment should work with the second url in the ticket and we will likely remove transformAuthorData from aem-fragment and just use transformPublishData for preview as well, happy to discuss in more detail.

yesil · 2026-04-17T11:44:42Z

in this file, we need to switch the value of FRAGMENT_URL_PREFIX from https://odin.adobe.com/adobe/sites/fragments to https://odinpreview.corp.adobe.com/adobe/sites/fragments

In the case of preview.

aem-code-sync bot temporarily deployed to MWPW-191059 April 3, 2026 19:02 Inactive

aem-code-sync bot temporarily deployed to MWPW-191059 April 3, 2026 19:12 Inactive

aem-code-sync bot temporarily deployed to MWPW-191059 April 3, 2026 19:16 Inactive

Axelcureno force-pushed the MWPW-191059 branch from 45945f8 to afaca3a Compare April 3, 2026 19:17

aem-code-sync bot temporarily deployed to MWPW-191059 April 3, 2026 19:17 Inactive

aem-code-sync bot temporarily deployed to MWPW-191059 April 3, 2026 19:36 Inactive

Axelcureno and others added 14 commits April 3, 2026 12:40

docs(MWPW-191059): add Freyja preview migration spec and implementati…

152523b

…on plan

feat(MWPW-191059): add Freyja and Odin preview URL constants to paths.js

a7dd497

feat(MWPW-191059): add auth header injection and Freyja-first fallbac…

41ba251

…k in internalFetch

test(MWPW-191059): update pipeline and replace test mocks for Freyja …

ba6ff8f

…URLs

test(MWPW-191059): update Nala variation spec for Freyja preview URLs

6d05b53

feat(MWPW-191059): switch fragment-client preview to Freyja v2 with O…

e0a787d

…din fallback

feat(MWPW-191059): switch mas-repository preview URLs to Freyja v2

fecc895

feat(MWPW-191059): add backend toggle, manual token injection, and be…

4a8b27b

…nchmark tool for Freyja vs Odin comparison

Axelcureno force-pushed the MWPW-191059 branch from b8d7f94 to 44cdd31 Compare April 3, 2026 19:40

aem-code-sync bot temporarily deployed to MWPW-191059 April 3, 2026 19:40 Inactive

Merge branch 'main' into MWPW-191059

03ae91f

aem-code-sync bot temporarily deployed to MWPW-191059 April 7, 2026 12:50 Inactive

npeltier requested changes Apr 7, 2026

View reviewed changes

Axelcureno self-assigned this Apr 7, 2026

Merge branch 'main' into MWPW-191059

c72cddf

aem-code-sync bot temporarily deployed to MWPW-191059 April 9, 2026 17:09 Inactive

aem-code-sync bot temporarily deployed to MWPW-191059 April 9, 2026 18:01 Inactive

honstar reviewed Apr 10, 2026

View reviewed changes

npeltier requested changes Apr 10, 2026

View reviewed changes

Merge branch 'main' into MWPW-191059

8eb6848

aem-code-sync bot temporarily deployed to MWPW-191059 April 10, 2026 13:58 Inactive

Axelcureno marked this pull request as draft April 13, 2026 20:31

Axelcureno added 2 commits April 13, 2026 13:33

Merge branch 'main' into MWPW-191059

68aee50

remove docs/superpowers from tracking, add to .gitignore

1c0bd44

aem-code-sync bot temporarily deployed to MWPW-191059 April 13, 2026 20:41 Inactive

aem-code-sync bot temporarily deployed to MWPW-191059 April 15, 2026 15:18 Inactive

honstar approved these changes Apr 15, 2026

View reviewed changes

MWPW-191059 - also use env-aware Freyja URL when freyjaToken is expli…

d2d3fe5

…citly passed

aem-code-sync bot deployed to MWPW-191059 April 15, 2026 15:46 View deployment

yesil requested changes Apr 17, 2026

View reviewed changes

		@@ -0,0 +1,703 @@
		# Freyja v2 Preview Migration — Implementation Plan


		const FRAGMENT_URL_PREFIX = 'https://odin.adobe.com/adobe/sites/fragments';

		const FREYJA_PREVIEW_URL = 'https://preview-p22655-e59433.adobeaemcloud.com/adobe/contentFragments';

Conversation

Axelcureno commented Apr 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Test URLs:

Checklist:

Uh oh!

aem-code-sync bot commented Apr 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

codecov bot commented Apr 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

npeltier left a comment

Choose a reason for hiding this comment

Uh oh!

honstar commented Apr 10, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

npeltier left a comment

Choose a reason for hiding this comment

Uh oh!

npeltier commented Apr 13, 2026

Uh oh!

Axelcureno commented Apr 13, 2026

Uh oh!

yesil left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Axelcureno commented Apr 3, 2026 •

edited

Loading

aem-code-sync bot commented Apr 3, 2026 •

edited

Loading

codecov bot commented Apr 3, 2026 •

edited

Loading