I have my own review skill (I think it predates when Claude added theirs) and one thing I'd add to your description is tell it to examine all the code and then, based on the changes, do a multi-role review of the code again using the most appropriate N of the following roles based on the changes: ... (where ... is a long list I have like Senior Engineer, Security Engineer, WCAG specialist, etc). Claude will spawn those reviews in parallel and then consolidate the feedback. I do spec based development so I just have my skill append the issues to the spec so I have a trail of issues and decisions.
I'm a software developer based in the beautiful village of Shelburne Falls, MA, USA with thirty years of professional software design and development experience gained from a range of employers and clients including small startups to Fortune 100 companies.
I am currently a Principal Engineer at the Concord Consortium (https://concord.org), a nonprofit educational research and development organization working to bring the promise of technology into reality for education in science, math, and engineering.