fix: avoid syntax collision with self-invoking function

[aaa123eee]

What is it?

• [ ] Feature / enhancement
• [x] Bug
• [ ] Docs / tests

Description

Currently, the replacement of this symbol includes an expression wrapped with braces. It might cause an incorrect syntax if the previous function declaration has no semicolon at the end

Example: setupUid2Sdk is called immediately

I suggest an adjustment to avoid syntax collision in 3rd party scripts since we have no control over those scripts. Instead of an expression in braces, we can encapsulate this logic using a function

Checklist:

• [x] My code follows the developer guidelines of this project
• [x] I have performed a self-review of my own code
• [ ] I have made corresponding changes to the documentation
• [ ] Added new tests to cover the fix / functionality

[vercel[bot]]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
partytown	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Aug 8, 2024 4:09pm

[aaa123eee]

linting is failing on the file I didn't touch :-(

[aaa123eee]

I've just added a typescript version bump that should solve the issue with the lint job https://github.com/BuilderIO/partytown/pull/616

[aaa123eee]

unfortunately this fix breaks other usages, converted to draft

anyway it would be nice to solve the issue anyhow

[gioboa]

Thanks for the update, let's create an issue for that and we can link this PR too. I'll close this one because I would like to have no pending PRs in the list. Thanks for your help btw 👏