object OnPlayerAttack(BasePlayer attacker, HitInfo info)
{
    Puts("OnPlayerAttack works!");
    return null;
}

Not really a typo, and hooks do not have to return anything. It can be updated though.