Skip to content

main.jsの独立責務を専用モジュールへ分離 - #68

Merged
katakata0522 merged 1 commit into
mainfrom
refactor/main-responsibility-split-clean-20260806
Aug 6, 2026
Merged

main.jsの独立責務を専用モジュールへ分離#68
katakata0522 merged 1 commit into
mainfrom
refactor/main-responsibility-split-clean-20260806

Conversation

@katakata0522

@katakata0522 katakata0522 commented Aug 6, 2026

Copy link
Copy Markdown
Owner

概要

js/main.jsへ残っていた独立責務を専用モジュールへ分離します。

利用者向けの機能・文言・計算結果・URL・Analyticsイベント名・公開HTML構造は変更しません。

分離した責務

  • 地域切替と地域別ナビゲーション
  • ブラウザ言語に基づく言語版の提案
  • ウィークリーリワードのカレンダー登録
  • PWAインストール案内
  • ウィジェット流入の計測
  • Service Workerの遅延登録と更新確認

js/main.jsには初期化と画面全体の調停を残しています。

配信・ビルド対応

  • 新しい6モジュールを圧縮対象へ追加
  • 内容ハッシュ・Service Workerキャッシュ世代の算出対象へ追加
  • オフライン先読み対象へ追加
  • Service Worker登録処理の所在を新旧どちらでも検出できるよう生成処理を更新
  • 配置に依存していた既存テストを、責務と不変条件を確認するテストへ更新

確認結果

  • 公開HTML 200ファイルの可視内容・構造が不変
  • 生成物の再現性検証
  • JavaScript構文検証
  • 全回帰テスト 231件
  • 圧縮後JavaScript構文検証
  • 圧縮後の全回帰テスト 231件
  • Browser calculator smoke(JP / US / KR / TW)
  • 変更対象を18ファイルへ限定

補足

HTML 4ファイルとsw.jsの差分は、新モジュールを配信するための内容ハッシュ・キャッシュ登録更新です。診断用スクリプトや一時ワークフローはこのPRに含めていません。

@katakata0522
katakata0522 changed the base branch from refactor/internal-runtime-cleanup-20260806 to main August 6, 2026 13:59
@katakata0522
katakata0522 force-pushed the refactor/main-responsibility-split-clean-20260806 branch from a868611 to 44d89fa Compare August 6, 2026 14:00
@katakata0522
katakata0522 marked this pull request as ready for review August 6, 2026 14:02
@katakata0522
katakata0522 merged commit 3d1ef69 into main Aug 6, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant