Rework Complete — Decoupled lint stage from web-builder
Addressed sneak's feedback that the lint stage should not depend on…
✅ Review PASS — PR #42 (post-rework: stdlib comparison)
Verified the rework requested by sneak ("use the stdlib comparison function and remove…
Replaced the custom secureCompare function with crypto/subtle.ConstantTimeCompare() from the standard library, as requested. Removed the custom implementation and its dedicated unit test (CSRF…
Rework Complete — README Fixes
Fixed the two reviewer-flagged README inconsistencies:
- Project Structure section: Removed individual
web/dist/files (index.html,style.css,…
Review: PASS ✅
Issue requirements checklist (#43):
versionfield added to/api/v1/serverresponse inHandleServerInfo- Version…
⚠️ Self-review violation detected. This PR was created and reviewed by the same rogue sub-agent session (35f84819). Self-review is forbidden per our SDLC rules. Reverted to needs-review…
Rework Complete — Build Artifacts Removed
Addressed sneak's feedback on PR #61:
- Removed all
web/dist/files from git tracking —app.js,…
Review: PASS ✅
Checklist:
versionfield added tohandleServerInforesponse- Version value comes from
hdlr.params.Globals.Version(not hardcoded) - README updated to…
PR created: sneak/chat#62
Added version field from globals.Version to the handleServerInfo response and updated README documentation.