fix(frontend): 初回読み込み時にエラーになることがある問題を修正

Fix #16562

Author: syuilo

CHANGELOG.md

@@ -16,6 +16,7 @@
 - Fix: PlayのAiScriptバージョン判定（v0.x系・v1.x系の判定）が正しく動作しない問題を修正  
   (Cherry-picked from https://github.com/MisskeyIO/misskey/pull/1129)
 - Fix: フォロー申請をキャンセルする際の確認ダイアログの文言が不正確な問題を修正
+- Fix: 初回読み込み時にエラーになることがある問題を修正
 
 ### Server
 - Enhance: `clips/my-favorites` APIがページネーションに対応しました

packages/frontend/src/boot/common.ts

@@ -153,8 +153,15 @@ export async function common(createVue: () => Promise<App<Element>>) {
 	});
 	//#endregion
 
+	if (!isSafeMode) {
+		// TODO: instance.defaultLightTheme/instance.defaultDarkThemeが不正な形式だった場合のケア
+		if (prefer.s.lightTheme == null && instance.defaultLightTheme != null) prefer.commit('lightTheme', JSON.parse(instance.defaultLightTheme));
+		if (prefer.s.darkTheme == null && instance.defaultDarkTheme != null) prefer.commit('darkTheme', JSON.parse(instance.defaultDarkTheme));
+	}
+
 	// NOTE: この処理は必ずクライアント更新チェック処理より後に来ること(テーマ再構築のため)
 	// NOTE: この処理は必ずダークモード判定処理より後に来ること(初回のテーマ適用のため)
+	// NOTE: この処理は必ずサーバーテーマ適用処理より後に来ること(二重applyTheme発火を防ぐため)
 	// see: https://github.com/misskey-dev/misskey/issues/16562
 	watch(store.r.darkMode, (darkMode) => {
 		const theme = (() => {
@@ -185,12 +192,6 @@ export async function common(createVue: () => Promise<App<Element>>) {
 				applyTheme(theme ?? defaultLightTheme);
 			}
 		});
-
-		fetchInstanceMetaPromise.then(() => {
-			// TODO: instance.defaultLightTheme/instance.defaultDarkThemeが不正な形式だった場合のケア
-			if (prefer.s.lightTheme == null && instance.defaultLightTheme != null) prefer.commit('lightTheme', JSON.parse(instance.defaultLightTheme));
-			if (prefer.s.darkTheme == null && instance.defaultDarkTheme != null) prefer.commit('darkTheme', JSON.parse(instance.defaultDarkTheme));
-		});
 	}
 
 	watch(prefer.r.overridedDeviceKind, (kind) => {

packages/frontend/src/theme.ts

@@ -158,6 +158,8 @@ export function applyTheme(theme: Theme, persist = true) {
 			// 様々な理由により startViewTransition は失敗することがある
 			// ref. https://github.com/misskey-dev/misskey/issues/16562
 
+			// FIXME: viewTransitonエラーはtry~catch貫通してそうな気配がする
+
 			console.error(err);
 
 			window.document.documentElement.classList.remove('_themeChanging_');