Branch Code Review

Review code changes on a feature branch, checking for:

• Pattern consistency with existing codebase
• Code duplication that should be factored out
• Unnecessary temporary variables
• Proper use of existing functions and utilities
• Potential N+1 query patterns or looped API/DB calls (frontend AND backend)
• Type safety issues (avoiding as any casts)
• Error handling consistency
• API vs model layer separation (backend)
• Raw PyMongo usage instead of Girder Model methods (backend)
• Missing exc=True on model loads (backend)
• Broad exception handling (backend)
• Access control and permission escalation (backend)
• Redundant validation that duplicates framework behavior
• API calls placed directly in Vue components instead of API files (frontend)
• Frontend code compensating for backend issues