[frontend] Deduplicate utility functions into shared module (#792) #841

Closed

sleepy wants to merge 0 commits from fix/dedup-js-utilities-792 into main

sleepy commented

2026-06-03 20:46:49 +02:00

Owner

Fixes #792

sleepy added 1 commit

2026-06-03 20:46:49 +02:00

[frontend] Deduplicate utility functions into shared module (#792 ) 27883766d0

Replace 15+ local copies of esc() and el() across JS files with
references to the canonical implementations in ui.js (uiModule.esc
and uiModule.el).

Files changed:
- admin.js, settings.js: replace local el()/esc() with uiModule aliases
- search-chat.js: replace local el() with uiModule.el
- skills.js, usage.js: replace local esc() with uiModule.esc
- gallery.js, tasks.js: replace DOM-based _esc() with uiModule.esc
- research/panel.js: add uiModule import, replace local _esc()
- emailInbox.js: add uiModule import, replace local _esc()
- emailLibrary/utils.js: add uiModule import, export _esc as alias
- emailLibrary/signatureFold.js: import _esc from local utils
- document.js: replace _esc() and _escHtml() with uiModule.esc
- documentLibrary.js: simplify local _esc to direct uiModule.esc ref
- notes.js: simplify _esc to use uiModule.esc
- assistant.js: replace local _esc() with uiModule.esc

Net: -37 lines of duplicated code, single source of truth for HTML escaping.

sleepy closed this pull request

2026-06-04 11:42:19 +02:00