chore: add performance logger

This commit is contained in:
李华桥
2023-11-10 00:25:15 +08:00
committed by LongYinan
parent 927a6489f9
commit 09c3a8828f
10 changed files with 79 additions and 3 deletions

View File

@@ -14,8 +14,10 @@ import { RouterProvider } from 'react-router-dom';
import { CloudSessionProvider } from './providers/session-provider';
import { router } from './router';
import { performanceLogger, performanceRenderLogger } from './shared';
import createEmotionCache from './utils/create-emotion-cache';
const performanceI18nLogger = performanceLogger.namespace('i18n');
const cache = createEmotionCache();
const DevTools = lazy(() =>
@@ -37,16 +39,23 @@ const future = {
async function loadLanguage() {
if (environment.isBrowser) {
performanceI18nLogger.info('start');
const { createI18n, setUpLanguage } = await import('@affine/i18n');
const i18n = createI18n();
document.documentElement.lang = i18n.language;
performanceI18nLogger.info('set up');
await setUpLanguage(i18n);
performanceI18nLogger.info('done');
}
}
const languageLoadingPromise = loadLanguage().catch(console.error);
export const App = memo(function App() {
performanceRenderLogger.info('App');
use(languageLoadingPromise);
return (
<CacheProvider value={cache}>

View File

@@ -23,6 +23,9 @@ import { nanoid } from 'nanoid';
import { applyUpdate, Doc as YDoc, encodeStateAsUpdate } from 'yjs';
import { WorkspaceAdapters } from '../adapters/workspace';
import { performanceLogger } from '../shared';
const performanceSetupLogger = performanceLogger.namespace('setup');
async function tryMigration() {
const value = localStorage.getItem('jotai-workspaces');
@@ -148,6 +151,7 @@ export function createFirstAppData(store: ReturnType<typeof createStore>) {
}
export async function setup(store: ReturnType<typeof createStore>) {
performanceSetupLogger.info('start');
store.set(
workspaceAdaptersAtom,
WorkspaceAdapters as Record<
@@ -156,11 +160,15 @@ export async function setup(store: ReturnType<typeof createStore>) {
>
);
console.log('setup global');
performanceSetupLogger.info('setup global');
setupGlobal();
performanceSetupLogger.info('try migration');
await tryMigration();
performanceSetupLogger.info('get root workspace meta');
// do not read `rootWorkspacesMetadataAtom` before migration
await store.get(rootWorkspacesMetadataAtom);
console.log('setup done');
performanceSetupLogger.info('done');
}

View File

@@ -5,18 +5,28 @@ import { StrictMode, Suspense } from 'react';
import { createRoot } from 'react-dom/client';
import { bootstrapPluginSystem } from './bootstrap/register-plugins';
import { performanceLogger } from './shared';
const performanceMainLogger = performanceLogger.namespace('main');
async function main() {
performanceMainLogger.info('start');
const { setup } = await import('./bootstrap/setup');
const rootStore = getCurrentStore();
performanceMainLogger.info('setup start');
await setup(rootStore);
performanceMainLogger.info('setup done');
bootstrapPluginSystem(rootStore).catch(err => {
console.error('Failed to bootstrap plugin system', err);
});
performanceMainLogger.info('import app');
const { App } = await import('./app');
const root = document.getElementById('app');
assertExists(root);
performanceMainLogger.info('render app');
createRoot(root).render(
<StrictMode>
<Suspense fallback={<WorkspaceFallback key="AppLoading" />}>

View File

@@ -40,6 +40,7 @@ import { WorkspaceHeader } from '../../components/workspace-header';
import { useBlockSuiteMetaHelper } from '../../hooks/affine/use-block-suite-meta-helper';
import { useTrashModalHelper } from '../../hooks/affine/use-trash-modal-helper';
import { useCurrentWorkspace } from '../../hooks/current/use-current-workspace';
import { performanceRenderLogger } from '../../shared';
import * as styles from './all-page.css';
import { EmptyPageList } from './page-list-empty';
import { useFilteredPageMetas } from './pages';
@@ -302,5 +303,7 @@ export const AllPage = () => {
};
export const Component = () => {
performanceRenderLogger.info('AllPage');
return <AllPage />;
};

View File

@@ -29,6 +29,7 @@ import { WorkspaceHeader } from '../../components/workspace-header';
import { useRegisterBlocksuiteEditorCommands } from '../../hooks/affine/use-register-blocksuite-editor-commands';
import { useCurrentWorkspace } from '../../hooks/current/use-current-workspace';
import { useNavigateHelper } from '../../hooks/use-navigate-helper';
import { performanceRenderLogger } from '../../shared';
const DetailPageImpl = (): ReactElement => {
const { openPage, jumpToSubPath } = useNavigateHelper();
@@ -145,5 +146,7 @@ export const loader: LoaderFunction = async args => {
};
export const Component = () => {
performanceRenderLogger.info('DetailPage');
return <DetailPage />;
};

View File

@@ -15,10 +15,17 @@ import {
} from 'react-router-dom';
import { WorkspaceLayout } from '../../layouts/workspace-layout';
import { performanceLogger, performanceRenderLogger } from '../../shared';
const workspaceLoaderLogger = performanceLogger.namespace('workspace_loader');
export const loader: LoaderFunction = async args => {
workspaceLoaderLogger.info('start');
const rootStore = getCurrentStore();
const meta = await rootStore.get(rootWorkspacesMetadataAtom);
workspaceLoaderLogger.info('meta loaded');
const currentMetadata = meta.find(({ id }) => id === args.params.workspaceId);
if (!currentMetadata) {
return redirect('/404');
@@ -32,6 +39,8 @@ export const loader: LoaderFunction = async args => {
}
if (currentMetadata.flavour === WorkspaceFlavour.AFFINE_CLOUD) {
const [workspaceAtom] = getBlockSuiteWorkspaceAtom(currentMetadata.id);
workspaceLoaderLogger.info('get cloud workspace atom');
const workspace = await rootStore.get(workspaceAtom);
return (() => {
const blockVersions = workspace.meta.blockVersions;
@@ -46,10 +55,14 @@ export const loader: LoaderFunction = async args => {
return false;
})();
}
workspaceLoaderLogger.info('done');
return null;
};
export const Component = (): ReactElement => {
performanceRenderLogger.info('WorkspaceLayout');
const incompatible = useLoaderData();
return (
<WorkspaceLayout incompatible={!!incompatible}>

View File

@@ -1,3 +1,4 @@
import { DebugLogger } from '@affine/debug';
import type { WorkspaceRegistry } from '@affine/env/workspace';
import { Workspace as BlockSuiteWorkspace } from '@blocksuite/store';
@@ -26,3 +27,6 @@ export const pathGenerator = {
} satisfies {
[Path in WorkspaceSubPath]: (workspaceId: string) => string;
};
export const performanceLogger = new DebugLogger('performance');
export const performanceRenderLogger = performanceLogger.namespace('render');