Skip to content

refactor: migrate nativeSdkHelpers module to TS - #1262

Open
jaissica12 wants to merge 2 commits into
developmentfrom
refactor/SDKE-1110-update-helpers-to-TS
Open

refactor: migrate nativeSdkHelpers module to TS#1262
jaissica12 wants to merge 2 commits into
developmentfrom
refactor/SDKE-1110-update-helpers-to-TS

replace self alias with arrow functions in nativeSdkHelpers

40cc85e
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded May 26, 2026 in 26s

Annotations

Check warning on line 45 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioEu&open=AZ4eNB2DxSI5yJcCioEu&pullRequest=1262

Check warning on line 147 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioE4&open=AZ4eNB2DxSI5yJcCioE4&pullRequest=1262

Check warning on line 53 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioEw&open=AZ4eNB2DxSI5yJcCioEw&pullRequest=1262

Check warning on line 191 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioFB&open=AZ4eNB2DxSI5yJcCioFB&pullRequest=1262

Check warning on line 54 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioEx&open=AZ4eNB2DxSI5yJcCioEx&pullRequest=1262

Check warning on line 82 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioE1&open=AZ4eNB2DxSI5yJcCioE1&pullRequest=1262

Check warning on line 45 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioEr&open=AZ4eNB2DxSI5yJcCioEr&pullRequest=1262

Check warning on line 44 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioEt&open=AZ4eNB2DxSI5yJcCioEt&pullRequest=1262

Check warning on line 190 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioFA&open=AZ4eNB2DxSI5yJcCioFA&pullRequest=1262

Check warning on line 103 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioE2&open=AZ4eNB2DxSI5yJcCioE2&pullRequest=1262

Check warning on line 168 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `childNode.remove()` over `parentNode.removeChild(childNode)`.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioE8&open=AZ4eNB2DxSI5yJcCioE8&pullRequest=1262

Check warning on line 189 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioE_&open=AZ4eNB2DxSI5yJcCioE_&pullRequest=1262

Check warning on line 152 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioE7&open=AZ4eNB2DxSI5yJcCioE7&pullRequest=1262

Check warning on line 201 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioFD&open=AZ4eNB2DxSI5yJcCioFD&pullRequest=1262

Check warning on line 191 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioE-&open=AZ4eNB2DxSI5yJcCioE-&pullRequest=1262

Check warning on line 147 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioE6&open=AZ4eNB2DxSI5yJcCioE6&pullRequest=1262

Check warning on line 194 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioFC&open=AZ4eNB2DxSI5yJcCioFC&pullRequest=1262

Check warning on line 59 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioEy&open=AZ4eNB2DxSI5yJcCioEy&pullRequest=1262

Check warning on line 43 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioEs&open=AZ4eNB2DxSI5yJcCioEs&pullRequest=1262

Check warning on line 52 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioEv&open=AZ4eNB2DxSI5yJcCioEv&pullRequest=1262

Check warning on line 79 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioEz&open=AZ4eNB2DxSI5yJcCioEz&pullRequest=1262

Check warning on line 206 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this redundant jump.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioFE&open=AZ4eNB2DxSI5yJcCioFE&pullRequest=1262

Check warning on line 50 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Complete the task associated to this "TODO" comment.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ5kDdzOpT_z9FcZ-OtQ&open=AZ5kDdzOpT_z9FcZ-OtQ&pullRequest=1262

Check warning on line 140 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this redundant jump.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioE3&open=AZ4eNB2DxSI5yJcCioE3&pullRequest=1262

Check warning on line 182 in src/nativeSdkHelpers.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer `globalThis` over `window`.

See more on https://sonarcloud.io/project/issues?id=mParticle_mparticle-web-sdk&issues=AZ4eNB2DxSI5yJcCioE9&open=AZ4eNB2DxSI5yJcCioE9&pullRequest=1262