-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: 메인 페이지 초기 로딩 속도 문제 해결 #11
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Risk Level 2 - /home/runner/work/next-blog/next-blog/src/common/util/fs.ts The method static getFile(path: fs.PathLike, option: BufferEncoding) {
try {
return fs.readFileSync(path, option);
} catch (error) {
console.error(`Error reading file at ${path}:`, error);
throw error;
}
} Risk Level 3 - /home/runner/work/next-blog/next-blog/src/pages/index.tsx
const { data: postMetaList } = useSWR<Array<PostMeta>>(KEY_POST);
if (!postMetaList) return <div>Loading...</div>;
Risk Level 3 - /home/runner/work/next-blog/next-blog/src/pages/posts/[id].tsx
const { data: post } = useSWR(`/api/posts/${id}`);
if (!post) return <div>Loading...</div>;
🛠️📄 Powered by Code Review GPT |
에러 핸들링을 추가로 하는 이유가 3가지 정도 있을거라고 본다.
나의 의견은 다음과 같다.
이 부분은 실제로 undefined 타입이 나올 가능성이 없을것 같은데 확인이 필요할것같다. 현재 Pr과는 관련없는 내용이므로 issue(#12)로 남겨두었다. |
요약 📜
구현 내용 📝
AS-IS
2024-06-05.2.16.27.mov
.md
파일을 html로 직렬화하는 함수가 약 총 5.3s로 전체 페이지 로딩 5.817s의 약 92% 차지TO-BE
2024-06-05.2.40.40.mov
/
페이지에선 메타 정보만 필요하고, HTML로 직렬화가 불필요하므로 함수를 메타 정보 파싱과, html 직렬화 하는 함수 2개로 분리관련 이슈🎯
기타
.md
파일의 meta 데이터 스키마 검증을 통해 any 타입 제거