Update Packages

[fadi-george]

Description

1 Line Summary

Update vite packages and use recommend tsconfig settings.

Details

• Updates vite packages and some testing packages (msw, jsdom, etc.) to latest version
• Updates our tsconfig settings with erasableSyntaxOnly to discourage use of enums and other things in the app
• Updates the tsconfig settings to have verbatimModuleSyntax to force import type when importing interfaces e.g.

// E.g. this would give you a warning
import axios, { AxiosInstance, AxiosResponse } from "axios"

// but this is fine:
import axios, { type AxiosInstance, type AxiosResponse } from "axios"

// or if its all just types then you can
import type { TypeA, TypeB, ... } from 'types/...'

Systems Affected

• WebSDK
• Backend
• Dashboard

Validation

Tests

Info

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

Checklist

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

---

Related Tickets

---

---

This change is 

[sherwinski]

It would be good to include notes (in the PR description) on the motivation to use this flag vs isoladedModules e.g. how does it specific benefit developer experience, bundle size, etc. There are some comparisons in this reference.

[fadi-george]

Saves a little of space by not using enums. And import type helps to different real classes from typescript types/interfaces.

[sherwinski]

Why the choice to remove WebWorker here?

[fadi-george]

I still have it.

[sherwinski]

Are these two functionally the same?

return (await OneSignal.User.addTags(
        finalTagsObject, 
      )) as TagsObjectForApi;

[fadi-george]

AddTags doesnt return anything but the outer function (sendTags) has this Promise<TagsObjectForApi>.