-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Labels
Description
Why
Multiple PRs repeatedly surfaced duplicate script includes and dependency-order regressions across layouts.
Evidence
- PR JavaScript optimization: eliminate redundancy and improve performance #58 reviews: duplicate includes (
main.js,command-palette.js) and missingutils.js/search-manager.jsin some layouts. - PR Harden content and script maintenance safeguards #62 review: duplicate
command-palette.jsinclude noted in teaching layout.
Scope
- Audit
_layouts/*.htmlfor duplicated script tags. - Ensure dependency order:
utils.jsandsearch-manager.jsload before scripts that consume them. - Remove redundant includes in affected layouts (
teaching,teaching-course,team,research,historyas needed). - Add a small CI guard script/check to catch duplicate includes and order violations.
Acceptance Criteria
- No layout has duplicate includes for
main.jsorcommand-palette.js. - Pages depending on
window.Utilsandwindow.SearchManageralways load those first. - CI fails when duplicate includes or invalid ordering is introduced.
Reactions are currently unavailable