A couple of things with your code that I noticed:
- function OnEvent_Initialize... should be function OnEvent_SigninReady... so as to not get an error with Messenger.MyEmail for example. Also it is better to use SigninReady rather than Signin as to not slow down your computer further when it is trying to load the contact list
- instead of writing the likes of "item = e.item()" just use e.item() instead of item as there is not more than 2 times after that it is used. Also when defining variables in a function and only going to be using it in that function then prefix them with "var " eg. "var item = e.item()" (without the quotes) and only use the var prefix on the first instance of using the variable.
- you might also want to get rid of the Debug.Trace's in future to increase performance as it won't require as much processing time to run a command (only marginal but it all helps)
Just a couple suggestions for future reference. I hope you don't feel that I'm nit picking, I just want to see people have the best possible coding with the most useful features possible.
Btw, I'm from Brisbane