Web SDK Refactor: Fix Session Tracking Logic

[fadi-george]

1 Line Summary

Continuation of the Web SDK Refactor project.

Details

• Fixes ServiceWorker logic for tracking session time. Note this was always breaking dev since Vite's define globals doesn't work for the ServiceWorker since its registered separately outside of Vite's scope. But production is fine since we have different entries for worker and page init.
• Thus I created a EnvVariables util to grab the env vars and use a typeof check to see if it exists otherwise return a default value
• Update spots where we use these env vars to use if statements for better DCE (dead code elimination)
• Removed mock for RebuildService in the tests
• Removed unneeded track start/end operations since we just have the logic in the session manager
• Created new Database method to setPushToken and call it wherever when we also set push id

Systems Affected

• WebSDK

Validation

Tests

Info

The changes include fixes to existing tests and aim to improve the reliability of identity and subscription management.

Checklist

• All the automated tests pass or I explained why that is not possible
• I have personally tested this on my machine or explained why that is not possible
• I have included test coverage for these changes or explained why they are not needed

Programming Checklist
Interfaces:

• Don't use default export
• New interfaces are in model files

Functions:

• Don't use default export
• All function signatures have return types
• Helpers should not access any data but rather be given the data to operate on.

Typescript:

• No Typescript warnings
• Avoid silencing null/undefined warnings with the exclamation point

Other:

• Iteration: refrain from using elem of array syntax. Prefer forEach or use map
• Avoid using global OneSignal accessor for context if possible. Instead, we can pass it to function/constructor so that we don't call OneSignal.context

Screenshots

Info

N/A

Checklist

• I have included screenshots/recordings of the intended results or explained why they are not needed

---

Related Tickets

---

---

This change is 

---

• To see the specific tasks where the Asana app for GitHub is being used, see below:
  • https://app.asana.com/0/0/1210277968011529

[fadi-george]

unused