It is a part module and it is indeed attached, I managed to get it to work in the end, all I did was move the
vessel.OnFlyByWire += new FlightInputCallback(Fly);
to my OnUpdate loop and replace the += with just an = so i dont have it run loads per frame, seems to do the trick hovever I gues this means it will mess up if i attach multiple modules, I havt tested yet but i guess i can just iteriate though all the parts on spawn and see if there are other parts already running the module and only enable self if no others?