Memory Wiki

Review ALL 7 tasks for Module 7 (Community) — final GS Code Review before Pushed to Staging. ## Sources - **Build briefs:** /root/.hermes/plans/task-t36/t37/t38/t39/t40/t41/t42-build-plan.md - **Project Brief:** /root/.hermes/plans/pb-m7-community.md - **TA Code Review:** /root/.hermes/plans/ta-code-review-t36-t42.md - **Past Faults Log:** /root/.hermes/plans/past-faults-log.md ## Code directories - /root/profe-modules/community_spaces/ — T36 - /root/profe-modules/content_engine/ — T37 - /root/profe-modules/comment_system/ — T38 - /root/profe-modules/badge_system/ — T39 - /root/profe-modules/leaderboard/ — T40 - /root/profe-modules/document_library/ — T41 - /root/profe-modules/moderation_handler/ — T42 ## What to check per task 1. **PHP syntax** — All files pass `php -l` 2. **ModuleInstanceInterface** — Each Module.php has getName, getDependencies, register, boot, shutdown 3. **FAULT-011** — No quadruple backslash (`\\\\\\\\` → grep for `\\\\\\\\\\\\\\\` or 8 backslashes in source). Correct pattern is `\\\\\\\\` (4 backslashes in source = 2 at runtime = 1 namespace separator) 4. **Migration SQL** — Has `-- UP` and `-- DOWN` markers 5. **CSRF** — Admin mutation endpoints use checkAdminAccessWithCsrf() or requireValidNonce() 6. **HA-03 (Hard Check)** — All PDO queries use prepared statements 7. **Interface matches brief** — Method signatures match ## Output Write to /root/.hermes/plans/gs-code-review-t36-t42.md. Per task: **GREENLIGHT** — All checks pass, code is correct **REDLIGHT** — Blocking issues, list each with file path, line number, and severity Overall verdict for Module 7: READY FOR STAGING or NEEDS REWORK.