LogoSEO Jing
  • All Posts
  • SEO Jing
  • KD Team
  • CLab CoreTeam
  • Study

Contact Me

© 2026 SEOJing. All rights reserved.

DevLogSEO JingInsight

엄청난 피드백

2026년 3월 15일·5분 읽기

코드 리뷰 배경

개발을 어느 정도 마무리한 뒤, CLI에게 코드 리뷰를 부탁했다. 보안/버그/품질 이슈를 수정하고, 전체 주석 스타일을 통일하는 계획이 나왔다.


주석 규칙

  • 불필요한 자질구래한 주석 삭제 (자명한 코드에 대한 설명) - 단일 주석: //로 통일 - TSX 내부 주석: {/* */}로 통일 - export 되는 함수: JSDoc /** */ + @example + 간단 설명

보안/버그 수정

packages/utils/src/content.ts

  • Path traversal 방어: getContentBySlug, isContentFolder에서 resolved path가 contentDir 내부인지 검증 - 에러 핸들링: getContentBySlug, scanContentDir에 try-catch 추가

packages/ui/src/article/code-block.tsx

  • dangerouslySetInnerHTML 주의 JSDoc 추가 (신뢰된 HTML만 사용 명시)

apps/web/src/widgets/recently-read/RecentlyRead.tsx

  • console.log(commentedPosts) 삭제- N+1 성능 개선: findDescription을 Map 기반으로 전환 - localStorage try-catch: markAsRead, updateReadProgress에 추가

기타

  • .gitignore: #cluade → #claude 오타 수정 - comment-tracker.ts: localStorage try-catch 추가 - theme-provider.tsx: localStorage try-catch 추가

수정 대상 파일 전체 목록

packages/utils/src/

- content.ts — path traversal 방어 + 에러핸들링 + 주석

packages/ui/src/

  • article/code-block.tsx — JSDoc 추가 - theme/theme-provider.tsx — localStorage try-catch

apps/web/src/widgets/

  • blog-search/BlogSearch.tsx — 주석 정리 + JSDoc - header/Header.tsx — 주석 정리 - mdx-renderer/MdxRenderer.tsx — 주석 정리 + JSDoc - article-toolbar/ArticleToolbar.tsx — 주석 정리 + JSDoc - comment/GiscusComment.tsx — 주석 정리 + JSDoc - comment/CommentModal.tsx — 주석 정리 + JSDoc - comment/comment-tracker.ts — try-catch + 주석 + JSDoc - post-explorer/PostExplorer.tsx — 주석 정리 + JSDoc - recently-read/RecentlyRead.tsx — console.log 삭제 + N+1 수정 + try-catch - new-posts-carousel/NewPostsCarousel.tsx — 주석 정리 + JSDoc

검증

  1. pnpm build 성공 확인 2. pnpm test 통과 확인 3. TypeScript 컴파일 에러 없음 확인

느낀 점

SEO 블로그기에 필요한 설정, 주석 정리, 라우터 관리 뿐만 아니라, 순회 로직까지도 깐깐하게 체킹해줬다. 그리고 이번엔 커밋을 분리하지 않고 바로 작업 했더니 console.log 지우는거 까먹고, try-catch 빼먹고 난리도 아님.

포스트 목록

/SEOJing/devLog/insight
파일 10개, 폴더 0개
엄청난 피드백생각보다 어려웠던 댓글, 완독 로컬스토리지디자인 시스템을 구축할 때 주의할 점폰트는 왜 메인 페이지에서만 적용이 안되고 있었을까?MDX DOM 트리 파싱하기MDX 관련 이슈 노트결국 Node.js 까지 와버렸다전체적인 플로우Storybook으로 디자인 시스템 테스팅하기MDX가 뭘까?