feat(personaliztion-utils): Add A/B Test Trimming Support to trimHtml Function

[teleaziz]

This PR extends the trimHtml function to support trimming A/B test variants in addition to personalization containers.

Changes:

• Updated trimHtml function to process A/B test variants before personalization containers
• Added abTests parameter to trimHtml options
• Implemented logic to select winning A/B test variants based on provided abTests object
• Updated README with information on A/B test trimming and usage examples

[changeset-bot]

⚠️ No Changeset found

Latest commit: d1c939a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit d1c939a. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution

---

✅ Successfully ran 36 targets

• nx test @e2e/qwik-city
• nx test @e2e/nuxt
• nx test @e2e/hydrogen
• nx test @e2e/angular
• nx test @e2e/react-sdk-next-app
• nx test @e2e/angular-ssr
• nx test @e2e/svelte
• nx test @e2e/vue
• nx test @e2e/sveltekit
• nx test @e2e/gen1-react
• nx test @e2e/remix
• nx test @e2e/solid
• nx test @e2e/react-sdk-next-pages
• nx test @e2e/solid-start
• nx test @e2e/react-native
• nx test @e2e/react
• nx test @e2e/nextjs-sdk-next-app
• nx test @e2e/gen1-remix
• nx test @e2e/gen1-next
• nx test @snippet/react-sdk-next-pages
• nx test @snippet/sveltekit
• nx test @snippet/angular
• nx test @snippet/solid
• nx test @snippet/react-sdk-next-app
• nx test @snippet/vue
• nx test @snippet/react-native
• nx test @snippet/svelte
• nx test @snippet/nuxt
• nx test @snippet/hydrogen
• nx test @snippet/qwik-city
• nx test @snippet/angular-ssr
• nx test @snippet/react
• nx test @snippet/nextjs-sdk-next-app
• nx test @builder.io/sdks
• nx typecheck @builder.io/sdks
• nx build @builder.io/sdk

---

Sent with 💌 from NxCloud.

[steve8708]

awesome! I wonder if we should also return an object so if we want to return more stuff in the future its easier. e.g.

const { html } = trimHtml({ userAttributes: { ... } })

e.g. if we want to setup something that auto dice rolls for a/b tests that you aren't assigned to yet, we ideally want to make sure you set that as a cookie, so we could always return like

const { html, newCookies } = ...

if we wanted. maybe this is already accounted for, but just an idea

[samijaber]

when I read trimHtml I think of a utility that trims excess whitespace.

Could we name it personalizeHTML or something like that, which more explicitly describes that it's going to process/handle personalization?

[steve8708]

Open to names, but this isn't specific to personalization. It handles scheduling, ab testing, and potentially more in future

…

On Wed, Sep 25, 2024 at 11:45 AM, Sami Jaber < ***@***.*** > wrote: ***@***.**** commented on this pull request. when I read trimHtml I think of a utility that trims excess whitespace. Could we name it personalizeHTML or something like that, which more explicitly describes that it's going to process/handle personalization? — Reply to this email directly, view it on GitHub ( #3575 (review) ) , or unsubscribe ( https://github.com/notifications/unsubscribe-auth/AAGOEA5EXOK72ORTC4KVYXTZYMAGDAVCNFSM6AAAAABO2GQJI6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGMRZGIYDQOJQGE ). You are receiving this because you commented. Message ID: <BuilderIO/builder/pull/3575/review/2329208901 @ github. com>