feat(MSDK-4514): Propagate isExempt to the ServiceConsent bridge model

[islameldesoky95]

Threads the isExempt field (added to the native UsercentricsServiceConsent in MSDK-4513) through the TS model, iOS Swift and Android Kotlin serialization extensions, and their mock/test fixtures.

[coderabbitai]

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 603912b6-7ba2-4e14-bd0d-0ea9c2c27570

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[pantoaibot]

PR Summary:

Propagate isExempt on ServiceConsent objects across native-to-JS bridge so the RN layer receives the new boolean field.

Changes:

• JS model: added isExempt:boolean to UsercentricsServiceConsent and extended constructor to accept isExempt.
• Android:
  • Include "isExempt" in UsercentricsServiceConsent.toWritableMap().
  • Updated android test mocks/expected maps to include isExempt (false).
• iOS:
  • Include "isExempt" in UsercentricsServiceConsent.toDictionary().
  • Updated sample tests and mocks to set isExempt (false).
• Tests: updated unit tests/mocks across src/tests, androidTest and sample iOS tests to include isExempt where service consents are created/serialized.
• Breaking change: UsercentricsServiceConsent constructor signature in the JS model now requires the additional isExempt argument — update any instantiations accordingly.
• No dependency, performance, or behavioral changes beyond exposing the new field.

Reviewed by Panto AI

[pantoaibot]

[CRITICAL_BUG] You made isExempt a required constructor parameter and a required class property. This is a breaking change for any existing callers that instantiate UsercentricsServiceConsent without the new argument. Make the new parameter optional with a sensible default (e.g. isExempt: boolean = false) or provide an overloaded/secondary constructor to preserve backward compatibility. Also update any factory/mapper code that constructs this class to pass the new value (or rely on the default).

export class UsercentricsServiceConsent {

    templateId: string
    status: boolean
    dataProcessor: string
    version: string
    type: UsercentricsConsentType
    isEssential: boolean
    history: UsercentricsConsentHistoryEntry[]
    category: string
    isExempt: boolean

    constructor(
        templateId: string,
        status: boolean,
        dataProcessor: string,
        version: string,
        type: UsercentricsConsentType,
        isEssential: boolean,
        history: UsercentricsConsentHistoryEntry[],
        category: string,
        isExempt: boolean = false,
    ) {
        this.templateId = templateId
        this.status = status
        this.dataProcessor = dataProcessor
        this.version = version
        this.type = type
        this.isEssential = isEssential
        this.history = history
        this.category = category
        this.isExempt = isExempt
    }
}

[pantoaibot]

[REFACTORING] Multiple tests were updated to include the new isExempt field in literal consent objects. To reduce duplication and the risk of missing future fields, introduce a small test helper/factory (e.g. makeConsentMock(overrides?)) that returns a consent object with sensible defaults (including isExempt: false). Replace inline literals with calls to that helper so future schema changes require modifications in a single place.

// src/__tests__/index.test.ts

type ConsentLiteral = {
  templateId: string
  status: boolean
  dataProcessor: string
  version: string
  type: number
  isEssential: boolean
  history: Array<{
    status: boolean
    timestampInMillis: number
    type: number
  }>
  category: string
  isExempt: boolean
}

const makeConsentMock = (overrides: Partial<ConsentLiteral> = {}): ConsentLiteral => ({
  templateId: "123",
  status: false,
  dataProcessor: "facebook",
  version: "123",
  type: 0,
  isEssential: false,
  history: [
    {
      status: false,
      timestampInMillis: 123.0,
      type: 0,
    },
  ],
  category: "essemtial",
  isExempt: false,
  ...overrides,
})

// Usage example inside tests:
const readyStatus = new UsercentricsReadyStatus(
  true,
  [makeConsentMock()],
  {
    countryCode: "PT",
    regionCode: "PT11",
    isInEU: true,
    isInUS: false,
    isInCalifornia: false,
  },
  {
    activeSettingsId: "",
    bannerRequiredAtLocation: false,
  },
)

[pantoaibot]

Reviewed up to commit:fd35f3cc94b4494978a3ec262ec485b46b4b2d04

Reviewed by Panto AI