I definitely try to keep my sprocs simple and generally logic free. The only time I would have a sproc get over just a couple lines of code is if I had some logic that needed to be ran that would take substantially longer to run outside of a database. I really don't even like having to do that, but in the end performance beats design.