I agree.Artisan? It's more about knowing what to do than anything else. Thus, it should be based on intelligence.
Perhaps replacing STR with 'Fighting'? Needs a catchy name, though. Brawn?Give how the skills are broken down into combat / non-combat, I wonder if having a 'Fighting' and 'Shooting' stat in their own right, linked to the combat skills, would resolve this problem.