chore: Node 24 LTS / pnpm 10 / dependency latest 정렬#65
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the project to Node.js 24 LTS and upgrades several core dependencies, including Vite, TypeScript, and React. It improves type safety by implementing type guards for Chrome storage data and user profiles, and includes widespread code reformatting. Feedback was provided regarding the use of hardcoded error messages, suggesting they be centralized in a constants file for better maintainability.
| error: { | ||
| code: String(TOKEN_EXPIRED_CODE), | ||
| message: '세션이 만료되었습니다. 다시 로그인해주세요.', | ||
| message: "세션이 만료되었습니다. 다시 로그인해주세요.", |
There was a problem hiding this comment.
하드코딩된 오류 메시지는 향후 유지보수 및 다국어 지원을 어렵게 만들 수 있습니다. src/constants/messages.ts와 같은 중앙화된 파일에 모든 사용자 대면 문자열을 정의하고, 이를 가져와 사용하는 것을 고려해 보세요. 이렇게 하면 일관성을 유지하고 변경 사항을 쉽게 관리할 수 있습니다.
| console.error('Failed to load selected template:', err); | ||
| setError('템플릿을 불러오는데 실패했습니다.'); | ||
| console.error("Failed to load selected template:", err); | ||
| setError("템플릿을 불러오는데 실패했습니다."); |
There was a problem hiding this comment.
이곳에서도 하드코딩된 오류 메시지가 사용되었습니다. 일관된 사용자 경험과 쉬운 관리를 위해 모든 사용자 대면 문자열을 중앙의 상수 파일에서 관리하는 것을 권장합니다.
| console.error('Failed to load template:', result.error); | ||
| setError(result.error?.message || '템플릿을 불러올 수 없습니다.'); | ||
| console.error("Failed to load template:", result.error); | ||
| setError(result.error?.message || "템플릿을 불러올 수 없습니다."); |
There was a problem hiding this comment.
하드코딩된 오류 메시지가 여러 곳에서 발견됩니다. 유지보수성을 높이기 위해 src/constants/messages.ts와 같은 파일에서 모든 사용자 대면 문자열을 관리하는 것이 좋습니다.
| console.error('Error loading template:', err); | ||
| setError('템플릿 로딩 중 오류가 발생했습니다.'); | ||
| console.error("Error loading template:", err); | ||
| setError("템플릿 로딩 중 오류가 발생했습니다."); |
There was a problem hiding this comment.
이 오류 메시지 또한 중앙에서 관리하는 것이 좋습니다. 일관된 오류 메시지 관리는 코드의 유지보수성을 크게 향상시킵니다.
| console.error('Failed to save template selection:', err); | ||
| setError('템플릿 선택을 저장하는데 실패했습니다.'); | ||
| console.error("Failed to save template selection:", err); | ||
| setError("템플릿 선택을 저장하는데 실패했습니다."); |
There was a problem hiding this comment.
하드코딩된 오류 메시지입니다. 다른 오류 메시지들과 함께 중앙에서 관리하도록 리팩터링하는 것을 고려해보세요.
1 participant
변경 사항
검증
포함 커밋