In your nested proc; to be sure, you can check whether table exist or not. If not exist then RAISERROR
with some error message. Since, you are using SS2005, #temptable
would be the option you have. Commenting your code for ease of readability is never a bad practice.
Talking about naming convention; follow any convention that fits better in your organization (I just gave a proper name to the SP that will reflect the purpose of the SP). If code changes is happening then changing the comment accordingly is the developer responsibility. Other than this, whatever you are doing looks correct to me.