comphy-lab / comphy-lab/comphy-lab.github.io

Deduplicate layout script tags and enforce JS dependency order

オープン
#65 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
codex
主要言語
CSS
スター
0
フォーク
0
平均マージ
1時間 16分
マージ済み PR(30日)
3

説明

## Why
Multiple PRs repeatedly surfaced duplicate script includes and dependency-order regressions across layouts.

## Evidence
- PR #58 reviews: duplicate includes (`main.js`, `command-palette.js`) and missing `utils.js`/`search-manager.js` in some layouts.
- PR #62 review: duplicate `command-palette.js` include noted in teaching layout.

## Scope
- Audit `_layouts/*.html` for duplicated script tags.
- Ensure dependency order: `utils.js` and `search-manager.js` load before scripts that consume them.
- Remove redundant includes in affected layouts (`teaching`, `teaching-course`, `team`, `research`, `history` as needed).
- Add a small CI guard script/check to catch duplicate includes and order violations.

## Acceptance Criteria
- No layout has duplicate includes for `main.js` or `command-palette.js`.
- Pages depending on `window.Utils` and `window.SearchManager` always load those first.
- CI fails when duplicate includes or invalid ordering is introduced.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。