i will leave style decisions to the bike shed manufacturer [those who do the work]. but i will opine anyway. i'm with the no indentation people. [except -- not implemented -- plain lists indented by 2. :]]
but my reason for posting is that as a default for org, i think (setq org-src-preserve-indentation t) would surprise users less. and to congratulate for dogfooding. :]