Architecture: net.connect() and connection callbacks ...

At some point in the future, connect can be changed such that it can return a i'm connecting value - see this

... but honestly right now it'll work fine if you do it as above. I don't think there's any need to do any massive changes - it's more important to just get something working.