I thought protos had the $ in the argument names. Can you check that and update the patch accordingly?