feat: Add configureScope transformer

[mydea]

Rewrites usages of configureScope() to use getCurrentScope() instead. Note that this will rewrite this to code
blocks, which may not be the preferred syntax in all cases, but it's the only way to make this work somewhat reliably
with avoiding variable clashes etc.

This will rewrite:

Sentry.configureScope(scope => {
  scope.setTag('ccc', 'ccc');
  scope.setExtra('ddd', { ddd: 'ddd' });
});

to

{
  const scope = Sentry.getCurrentScope();
  scope.setTag('ccc', 'ccc');
  scope.setExtra('ddd', { ddd: 'ddd' });
}

[Lms24]

Oh interesting - It's been a while! 😅

Good call with still wrapping the statements in a block to avoid variable colissions! Not sure but didn't we add some way for transformers to write something to the CLI if they actually changed stuff? Maybe we should add a message that folks should clean this up?

[mydea]

So I looked some more into this, and added a special handling for when there is only a single statement - in this case, we can rewrite it nicer:

configureScope((scope) => {
  scope.setTag('xx', 'yy');
});

becomes

getCurrentScope().setTag('xx', 'yy');

Which is much nicer!

I found another edge case though I also need to handle, which is getCurrentHub().configureScope() - there, it has to be getScope() instead of getCurrentScope() 😬

[mydea]

@Lms24 I added this somewhat hacky check, but I think it should work well enough...

[Lms24]

there, it has to be getScope() instead of getCurrentScope() 😬

Are we adding another transformer to replace getCurrentHub().getScope() with getCurrentScope()?

[mydea]

Yeah, figured that too - we should also do that then! But I guess that will be a general getCurrentHub() transformer that rewrites that to all the new/global APIs instead!

[Lms24]

what about configureScope(scope => scope.setTag(...))?

[Lms24]

there, it has to be getScope() instead of getCurrentScope() 😬

Are we adding another transformer to replace getCurrentHub().getScope() with getCurrentScope()?

[mydea]

OK, I think I fix all/most edge cases - ran it on the monorepo, and it seemed good to me - no false positives anymore, at least!

[Lms24]

Nice! 🚀