MWPW-198237: Fix OST and checkout links in compare-chart feature columns

[yesil]

Fixes the Offer Selector Tool (OST) and checkout links in mas-compare-chart feature columns.

• Editor: the RTE cell no longer deactivates while a modal (offer selector, link/icon/mnemonic editor) is open — #rteFieldHasOpenModal guards #deactivateRteCell, so opening OST inside a feature column keeps the cell alive. Removes the now-redundant @rte-focusout handler.
• Render (hydrate.js): data-wcs-osi checkout links in the features slot are now transformed into buttons, matching description-slot behavior, via a new shared processLinks helper reused by both processDescriptionLinks and processFeaturesLinks.
• Unit test added for features-slot checkout-link transformation (hydrate.test.js).
• dist/mas.js + dist/merch-card.js rebuilt and committed.

Resolves https://jira.corp.adobe.com/browse/MWPW-198237
QA Checklist: https://wiki.corp.adobe.com/display/adobedotcom/M@S+Engineering+QA+Use+Cases

Please do the steps below before submitting your PR for a code review or QA

• C1. Cover code with Unit Tests
• C2. Add a Nala test (double check with #fishbags if nala test is needed)
• C3. Verify all Checks are green (unit tests, nala tests)
• C4. PR description contains working Test Page link where the feature can be tested
• C5: you are ready to do a demo from Test Page in PR
• C.6 read your Jira one more time to validate that you've addressed all AC's and nothing is missing

Test URLs:

• Before: https://main--mas--adobecom.aem.live/

• After: https://mwpw-198237--mas--adobecom.aem.live/

• Express demo page:

• Before: https://main--da-express-milo--adobecom.aem.page/drafts/ilyas/pricing?milolibs=stage

• After: https://main--da-express-milo--adobecom.aem.page/drafts/ilyas/pricing?milolibs=stage&maslibs=mwpw-198237--mas--adobecom

[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

• 56c48cc ✅ (latest)
• fce3fd3 ✅
• afeb080 ✅
• 00fecbf ✅
• 082036e ✅
• feb63b8 ✅
• 6422b6b ✅
• 6ea2262 ✅
• 98468b1 ✅
• 3d12656 ✅
• 6abcc66 ✅
• 76245d3 ✅
• 99f0b64 ✅
• 5ffee3b ✅
• 0826ef1 ✅

[github-actions]

SNOW Change Request Transaction ID [STUDIO]: 0000019c1eec85a3-12e0fd06

[codecov]

Codecov Report

❌ Patch coverage is 88.00000% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.45%. Comparing base (06d7c05) to head (56c48cc).

Files with missing lines	Patch %	Lines
studio/src/editors/mas-compare-chart-editor.js	84.21%	6 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #962      +/-   ##
==========================================
+ Coverage   89.40%   89.45%   +0.04%     
==========================================
  Files         269      269              
  Lines       84202    84245      +43     
==========================================
+ Hits        75284    75362      +78     
+ Misses       8918     8883      -35     

Files with missing lines	Coverage Δ
web-components/src/hydrate.js	92.54% <100.00%> (+0.25%)	⬆️
web-components/src/merch-card.js	89.36% <100.00%> (+0.09%)	⬆️
studio/src/editors/mas-compare-chart-editor.js	73.12% <84.21%> (+2.27%)	⬆️

... and 7 files with indirect coverage changes

---

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 06d7c05...56c48cc. 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.

[Axelcureno]

Hydrate refactor looks good, and the removed @rte-focusout was genuinely dead.

One blocker: the guard's flags can get stuck. showOfferSelector only clears on select (rte-field.js:1558) — closing OST via Escape/backdrop/Cancel leaves it true, so the cell never deactivates and Escape gets swallowed. Link/icon editors' Cancel dispatches close, but rte-field only binds @save. Suggest clearing the flag on every OST close and binding @close.

Smaller: Escape-revert leaves referencedFragmentStoresHaveChanges stuck true; spectrum="swc" CTAs lose their listener through the innerHTML→unsafeHTML round-trip.

@Axelcureno dismissing your review, I could not reproduce the potential issues you reported.

[seanchoi0]

The showOfferSelector sticky-flag concern is still there — ost.js onCancel doesn't reset the flag, and #handleEscKey in rte-field.js skips it, so dismissing OST via Escape/backdrop could leave the cell locked. That said, Axelcureno already reviewed and made the call to ship, so deferring to that judgment.

[yesil]

@seanchoi0 similar comment from @Axelcureno as well, I tried and could not reproduce so left it as is.

[Roycethan]

@yesil
1)Plz resolve conflicts.
2) Applied button style is not rendered - Tracking here https://jira.corp.adobe.com/browse/MWPW-199105 (In case if it need to be handled separately as followup)

Applied 'Accent' style at https://mwpw-198237--mas--adobecom.aem.live/studio.html#fragmentId=c481dd2a-a079-47d7-9383-3dbeda5b0575&page=fragment-
editor&path=sandbox&tags=mas%3Astudio%2Fcontent-type%2Fcompare-chart

Observed that this is not rendered here:
https://main--da-express-milo--adobecom.aem.page/drafts/ilyas/pricing?milolibs=stage&maslibs=mwpw-198237--mas--adobecom&tab=1