refactor: remove meaningless comments
Remove 50+ noise comments across the codebase: - AI-generated verbose prose (💡 emoji, '超进化', '大闸', etc.) - Comments that restate what the code obviously does - Comments about deleted code - Import-level noise comments - Overly verbose Chinese section markers Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This commit is contained in:
@@ -1,6 +1,6 @@
|
||||
import React from 'react';
|
||||
import { ThemeModeProvider } from './ThemeModeProvider';
|
||||
import { RouterProvider } from './RouterProvider'; // 💡 1. 全局合龙:引入满血重构的路由中枢
|
||||
import { RouterProvider } from './RouterProvider';
|
||||
|
||||
interface AppRootProps {
|
||||
children: React.ReactNode;
|
||||
|
||||
@@ -7,7 +7,7 @@ import {
|
||||
useRef,
|
||||
useState,
|
||||
} from 'react';
|
||||
import { browser } from 'wxt/browser'; // 💡 1. 规范回归:引入 WXT 标准多端一致性代理空间
|
||||
import { browser } from 'wxt/browser';
|
||||
import type { ContextMenuPendingData, PageType, StorageSchema } from '@/types/storage';
|
||||
import { storageUtil } from '@/utils/chromeStorage';
|
||||
import {
|
||||
@@ -96,7 +96,7 @@ export function RouterProvider({
|
||||
visiblePagesKey = 'app/visiblePages',
|
||||
pageOrderKey = 'app/pageOrder',
|
||||
}: RouterProviderProps) {
|
||||
// 1. 初始化派生状态流(0 延迟快照同步)
|
||||
// 1. Initialize state with sync snapshot
|
||||
const [currentPage, setCurrentPage] = useState<PageType>(() =>
|
||||
getSyncSnapshot(syncKey as string, defaultRoute, isValidPage),
|
||||
);
|
||||
@@ -153,7 +153,7 @@ export function RouterProvider({
|
||||
}
|
||||
}, [defaultRoute, syncKey, syncRoute, visiblePagesKey, pageOrderKey]);
|
||||
|
||||
// 副作用 1:组件挂载时,激活初始路由校验与右键菜单传递载荷(Context Payload)的嗅探
|
||||
// Load initial data from async storage on mount
|
||||
useEffect(() => {
|
||||
let cancelled = false;
|
||||
|
||||
@@ -162,7 +162,7 @@ export function RouterProvider({
|
||||
.then(() => {
|
||||
if (cancelled) return;
|
||||
|
||||
// 检查 URL 参数中的右键菜单高阶中转数据
|
||||
// Check URL params for context menu data
|
||||
if (typeof window !== 'undefined') {
|
||||
const params = new URLSearchParams(window.location.search);
|
||||
const feature = params.get('feature') as PageType | null;
|
||||
@@ -182,7 +182,7 @@ export function RouterProvider({
|
||||
}
|
||||
}
|
||||
|
||||
// 检查 storage 中的右键菜单待处理数据(针对 openPopup 的闭环场景)
|
||||
// Check storage for pending context menu data
|
||||
storageUtil
|
||||
.get('contextMenu/pendingData', undefined)
|
||||
.then((pendingData) => {
|
||||
@@ -276,7 +276,6 @@ export function RouterProvider({
|
||||
}
|
||||
};
|
||||
|
||||
// 💡 修复点:全域绑定 WXT 跨浏览器代理监听器,彻底闭环多端多进程广播
|
||||
browser.storage.onChanged.addListener(handleStorageChange);
|
||||
return () => {
|
||||
browser.storage.onChanged.removeListener(handleStorageChange);
|
||||
|
||||
@@ -7,7 +7,7 @@ import {
|
||||
useMemo,
|
||||
useState,
|
||||
} from 'react';
|
||||
import { browser } from 'wxt/browser'; // 💡 1. 规范回归:引入 WXT 标准多端一致性代理,消灭裸奔的 chrome 空间
|
||||
import { browser } from 'wxt/browser';
|
||||
import { storageUtil } from '@/utils/chromeStorage';
|
||||
|
||||
export type ThemeMode = 'light' | 'dark' | 'system';
|
||||
@@ -59,18 +59,15 @@ export function ThemeModeProvider({ children }: ThemeModeProviderProps) {
|
||||
mode === 'system' ? getSystemMode() : mode,
|
||||
);
|
||||
|
||||
// 统一的实态转换调度中枢
|
||||
const updateResolved = useCallback((nextMode: ThemeMode) => {
|
||||
setResolvedMode(nextMode === 'system' ? getSystemMode() : nextMode);
|
||||
}, []);
|
||||
|
||||
// 主动切流触发大闸
|
||||
const setMode = useCallback(
|
||||
(next: ThemeMode) => {
|
||||
setModeState(next);
|
||||
updateResolved(next);
|
||||
|
||||
// 💡 修复点:对异步微任务链追加标准的显式 void 断链安全隔离,彻底净化 Linter 悬空 Promise 警告
|
||||
void storageUtil.set(THEME_MODE_KEY, next).catch((err) => {
|
||||
console.error('[Theme Storage Error] Failed to persistent theme state:', err);
|
||||
});
|
||||
@@ -84,7 +81,7 @@ export function ThemeModeProvider({ children }: ThemeModeProviderProps) {
|
||||
[updateResolved],
|
||||
);
|
||||
|
||||
// 副作用 1:冷启动时异步对齐来自长期存储的权威配置,自愈快照偏差
|
||||
// Sync theme from storage on mount
|
||||
useEffect(() => {
|
||||
let cancelled = false;
|
||||
storageUtil
|
||||
@@ -104,7 +101,7 @@ export function ThemeModeProvider({ children }: ThemeModeProviderProps) {
|
||||
};
|
||||
}, [updateResolved]);
|
||||
|
||||
// 副作用 2:当处于跟随系统模式下,动态监听操作系统级别的高级霓虹换色
|
||||
// Listen for system theme changes when in system mode
|
||||
useEffect(() => {
|
||||
if (mode !== 'system') return;
|
||||
const mq = window.matchMedia('(prefers-color-scheme: dark)');
|
||||
@@ -115,15 +112,12 @@ export function ThemeModeProvider({ children }: ThemeModeProviderProps) {
|
||||
return () => mq.removeEventListener('change', handler);
|
||||
}, [mode]);
|
||||
|
||||
// 副作用 3:跨上下文广播事件同步大闸(Popup <-> Sidepanel 联动核心)
|
||||
// Sync theme across extension contexts (popup <-> sidepanel)
|
||||
useEffect(() => {
|
||||
// 💡 2. 强类型对齐修复:对齐标准的 browser.storage.onChanged.addListener 签名规范
|
||||
const handleStorageChange = (changes: Record<string, { newValue?: unknown }>) => {
|
||||
if (changes[THEME_MODE_KEY]) {
|
||||
const next = changes[THEME_MODE_KEY].newValue;
|
||||
if (isValidMode(next)) {
|
||||
// 💡 3. 架构解耦:改用函数式状态更新,彻底将本 Effect 从当前的 [mode] 依赖中解脱出来!
|
||||
// 这能保证多视口监听器在初始化时【只注册一次】,彻底消灭高频切流时的事件撕裂与重复注销。
|
||||
setModeState((currentMode) => {
|
||||
if (next !== currentMode) {
|
||||
updateResolved(next);
|
||||
@@ -139,9 +133,8 @@ export function ThemeModeProvider({ children }: ThemeModeProviderProps) {
|
||||
return () => {
|
||||
browser.storage.onChanged.removeListener(handleStorageChange);
|
||||
};
|
||||
}, [updateResolved]); // 👈 完美剔除 mode 脏依赖!
|
||||
}, [updateResolved]);
|
||||
|
||||
// 副作用 4:宿主 DOM 的 class 样式原子映射注入(一帧直出)
|
||||
useEffect(() => {
|
||||
if (typeof document !== 'undefined') {
|
||||
document.documentElement.classList.toggle('dark', resolvedMode === 'dark');
|
||||
|
||||
Reference in New Issue
Block a user