Fixes two bugs introduced by recent code quality changes:
JSON parse errors: Replaced JSON.parse() with repairJSON() in desktop.js and mobile.js for infoBox/userStats parsing. Handles malformed JSON with trailing content.
Empty equipment tab: Added defensive initialization in renderEquipment() — ensures showEquipment defaults to true and equipment data structure exists. Added explicit render call during init.
- Replace JSON.parse() with repairJSON() in desktop.js and mobile.js
to handle malformed JSON from infoBox/userStats data
- Add defensive initialization in renderEquipment(): ensure showEquipment
defaults to true and equipment data structure exists
- Add explicit renderEquipment() call during initialization
The equipment section has display:none in CSS and updateSectionVisibility()
is the only function that calls .show() on it. It was never called during
initialization, only when settings toggles changed. Added the call to the
main init flow so the equipment tab becomes visible on page load.
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Fixes two bugs introduced by recent code quality changes:
JSON parse errors: Replaced
JSON.parse()withrepairJSON()in desktop.js and mobile.js for infoBox/userStats parsing. Handles malformed JSON with trailing content.Empty equipment tab: Added defensive initialization in
renderEquipment()— ensuresshowEquipmentdefaults totrueand equipment data structure exists. Added explicit render call during init.Tests: 78/78 passed. Closes #20
✅ No issues found — changes look consistent with the stated intent. Ready to be merged.
@Zephyre_Review please review again — added updateSectionVisibility() call during init to fix equipment tab visibility.
✅ No issues found — changes look consistent with the stated intent. Ready to be merged.
@Zephyre_Review please review — 3 commits fixing JSON parse errors, equipment tab visibility, and a broken import that caused 404.
✅ No issues found — changes look consistent with the stated intent. Ready to be merged.