I don't think it does. As he explicitly provides the instance to give to the constructor, any type should work.
But typically an extension should never need a project instance. So it depends on what you do with that project instance, but most probably it is bad practice, yes.