Down to personal preference I think.
I personally dont annotate every line as I feel the line is self explanatory enough, I usually write a comment before the block of code indicating I am loading the executor via reflection and grabbing that properly. I dislike breaking apart every line as it can make longer methods harder to read.
As for changing the type declarations to var, again I dislike doing this in certain contexts as I find the code alot more readable if I can see the datatype directly. I had also noticed I left the types variable in, which I was only using during debugging to get see all the types I could mess around with.
Important thing is your code is to your standards. I’d reverse most of your changes, but its not my project to manage.
For example in my custom activity I have just done one that outputs alot of information compared to Konrad_Mierzwa’s approach of making each activity output a single value, again this is partly personal preference in my opinion. The code snippet I gave was edited to just so a single data point grabbed from the executor the keep it simple.