📝 chore: 제출 전 코드 정리 작업 목록

[za0012]

이력서 제출 전에 저장소를 "채용 담당자가 열어봐도 되는 상태"로 만드는 작업을 이 PR에 모읍니다.

현재 기본 브랜치는 bun run build, tsc --noEmit, bun test가 모두 실패합니다. 이력서에 GitHub URL을 적는 이상 이 세 가지 통과가 최소 조건입니다.

상세 내용은 docs/pre-submit-checklist.md 참고.

순서

• 1. PR 🔒 fix: 화면·리포트에 노출된 테스트 계정 문자열 제거 #6 머지 — 공개 노출된 테스트 계정 제거 + 해당 계정 비밀번호 변경
• 2. PR 🐛 fix: 타입 오류·타임존 버그 정리 + 테스트 추가 + 리뷰 워크플로 시크릿 분리 #7 파일별 처리 — 아래 표 기준으로 선별
• 3. 검증 — bun run build / ./node_modules/.bin/tsc --noEmit / bun test 3종 통과
• 4. 브랜치 보호 규칙 설정 — 3번 통과 후에 켤 것
• 5. 일간 리포트 재실행 — 결론이 바뀌는지 확인
• 6. 여유 시 추가 작업

PR #7 파일별 처리

AI가 생성한 PR입니다. 전부 받아들이지 말고 아래 기준으로 나눕니다.

판단 기준: 면접에서 "이걸 왜 이렇게 고쳤냐"고 물으면 답할 수 있는가?

파일	처리	이유
Heatmap/hook.test.ts	그대로	타임존 고정 + 적용 확인 테스트까지 잘 작성됨
LineChart/hook.test.ts	그대로	동일
Heatmap/hook.ts	그대로	format() 적용. 검증 완료
LineChart/hook.ts	그대로	format() + 범위 -5~0. 검증 완료
MakeStudy.tsx	그대로	주석 해제. 판단할 것 없음
Sidebar.tsx	그대로	study?.id guard 한 줄
calendar.tsx	그대로	react-day-picker 버전 타입 문제
TagSection.tsx	직접 다시 작성	오류만 지운 우회 수정
studyStore.ts	설계 결정	두 안 중 선택
tsconfig.json	판단	부작용 인지하고 유지 여부 결정
.github/workflows/automated-code-review.yml	읽고 검토	57줄 추가. 내용 확인 필요
problems/page.tsx	확인	13줄 변경분 확인

직접 작성해야 하는 것

TagSection.tsx

// PR 상태 — 오류만 사라지고 타입 안정성은 그대로
const selectedTags: string[] = useWatch({ control, name: "tags" }) ?? [];
//                  ^^^^^^^^ any 에 이름표만 붙인 것

ProblemFormValues를 공용 타입으로 분리하고 useFormContext<ProblemFormValues>()로 제네릭을 연결하면 타입이 자동으로 string[]이 됩니다.

studyStore.ts

• A안(현재 PR): StoredStudy — 필드마다 nullable. 사용하는 쪽에서 필드마다 null 체크
• B안: atomWithStorage<Study | null> — 한 번만 체크하면 이후 필드는 타입 보장

B안이 낫지만 참조하는 곳을 함께 손봐야 합니다. A안 유지 시 왜 골랐는지 설명할 수 있어야 합니다.

tsconfig.json

"exclude": ["node_modules", "**/*.test.ts"] — 빌드 타입 검사에서 테스트를 빼는 건 합리적이지만, 테스트 파일이 타입 검사를 전혀 안 받게 되는 부작용이 있습니다.

왜 4번이 필요한가

현재 브랜치 보호 규칙이 없어 체크가 실패해도 머지됩니다. PR #5가 fail 상태로 머지된 사례가 있습니다. 이 설정을 마쳐야 이력서의 "P1 발견 시 병합을 차단하는" 문구가 사실이 됩니다.

6번 — 여유 시

• api/extension/_shared.ts CORS * → allowlist
• problems/page.tsx queryKey에서 모달 상태 제거 + invalidateQueries
• History.tsx, MonthlySolve.tsx 삭제 (중복)
• biome 70 errors / 28 warnings 정리
• getAuthedMember discriminated union 반환으로 변경

Related: #6, #7

[github-actions]

자동 PR 코드 리뷰

• 실행 유형: PR 리뷰
• 생성 시각: 2026-07-31T02:12:50.629Z
• 기준 커밋: 8d4c24d6bb349045140caf41872afae0d8c24e0a
• 대상 커밋: 5eb2073fae1644fbc1f3a03dfbad8054b600bdcd
• 변경 파일 수: 1
• 발견 항목 수: 4
• 이번 PR 영향 가능 항목: 0
• 기존 실패 추정 항목: 4

검증 결과

• Lint: 실패 (1) - bun biome check .
• TypeScript 타입 검사: 실패 (2) - "/home/runner/.bun/bin/bun" "/home/runner/work/codeTest/codeTest/node_modules/typescript/bin/tsc" --noEmit --pretty false
• 테스트: 실패 (1) - bun run test
• 빌드: 실패 (1) - bun run build

이번 PR 영향 가능 항목

이번 PR 변경 파일에서 직접 발생한 것으로 보이는 항목은 없습니다.

기존 실패로 추정되는 항목

P1 빌드 실패

• 위치: package.json
• 출처: harness
• 분류: 기존 실패 추정
• 영향: 빌드 검증이 실패했지만, 출력에 나타난 파일이 이번 PR 변경 파일 밖에 있어 기존 실패일 가능성이 큽니다.
• 제안 수정: 로컬에서 bun run build를 실행한 뒤 아래 첫 출력부터 수정하세요.
  $ bun next build
  Failed to type check.

./src/app/(with-sidebar)/problems/components/problem-add/TagSection.tsx:83:31
Type error: Parameter 'tag' implicitly has an 'any' type.

81 |

82 | {selectedTags.length > 0 ? (

83 | selectedTags.map((tag) => (
| ^
84 | <span
85 | key={tag}
86 | className="text-[12px] text-[#8b95a1] font-medium"
Next.js build worker exited with code: 1 and signal: null
error: "next" exited with code 1
error: script "build" exited with code 1
⚠ No build cache found. Please configure build caching for faster rebuilds. Read more: https://nextjs.org/docs/messages/no-cache
▲ Next.js 16.2.4 (Turbopack)

Creating an optimized production build ...
✓ Compiled successfully in 11.0s
Running TypeScript ...

• 확인 방법: bun run build가 exit code 0으로 종료되면 해결된 것입니다.

P1 테스트 실패

• 위치: src/**/*.test.ts
• 출처: harness
• 분류: 기존 실패 추정
• 영향: 테스트 검증이 실패했지만, 출력에 나타난 파일이 이번 PR 변경 파일 밖에 있어 기존 실패일 가능성이 큽니다.
• 제안 수정: 로컬에서 bun run test를 실행한 뒤 아래 첫 출력부터 수정하세요.
  $ bun test ./src//*.test.ts
  Test filter "./src//*.test.ts" had no matches
  error: script "test" exited with code 1
  bun test v1.3.14 (0d9b296a)
• 확인 방법: bun run test가 exit code 0으로 종료되면 해결된 것입니다.

P1 TypeScript 타입 검사 실패

• 위치: src/components/Sidebar.tsx
• 출처: harness
• 분류: 기존 실패 추정
• 영향: TypeScript 타입 검사 검증이 실패했지만, 출력에 나타난 파일이 이번 PR 변경 파일 밖에 있어 기존 실패일 가능성이 큽니다.
• 제안 수정: 로컬에서 "/home/runner/.bun/bin/bun" "/home/runner/work/codeTest/codeTest/node_modules/typescript/bin/tsc" --noEmit --pretty false를 실행한 뒤 아래 첫 출력부터 수정하세요.
  src/app/(with-sidebar)/problems/components/problem-add/TagSection.tsx(83,31): error TS7006: Parameter 'tag' implicitly has an 'any' type.
  src/app/(with-sidebar)/problems/page.tsx(135,15): error TS2322: Type 'any[] | undefined' is not assignable to type 'string[]'.
  Type 'undefined' is not assignable to type 'string[]'.
  src/app/(without-sidebar)/find/page.tsx(7,23): error TS2306: File '/home/runner/work/codeTest/codeTest/src/app/(without-sidebar)/find/component/MakeStudy.tsx' is not a module.
  src/components/Sidebar.tsx(49,36): error TS18048: 'study' is possibly 'undefined'.
  src/components/ui/Logo.tsx(2,21): error TS2307: Cannot find module '../../../public/Logo.png' or its corresponding type declarations.
  src/components/ui/calendar.tsx(94,9): error TS2353: Object literal may only specify known properties, and 'table' does not exist in type 'Partial'.
  src/lib/store/studyStore.ts(6,3): error TS2322: Type 'null' is not assignable to type 'string'.
  src/lib/store/studyStore.ts(7
  [output truncated]
• 확인 방법: "/home/runner/.bun/bin/bun" "/home/runner/work/codeTest/codeTest/node_modules/typescript/bin/tsc" --noEmit --pretty false가 exit code 0으로 종료되면 해결된 것입니다.

P2 Lint 실패

• 위치: package.json

• 출처: harness

• 분류: 기존 실패 추정

• 영향: Lint 검증이 실패했지만, 출력에 나타난 파일이 이번 PR 변경 파일 밖에 있어 기존 실패일 가능성이 큽니다.

• 제안 수정: 로컬에서 bun biome check .를 실행한 뒤 아래 첫 출력부터 수정하세요.
  src/app/(with-sidebar)/mypage/components/BarChart/view.tsx:8:3 lint/correctness/noUnusedImports FIXABLE ━━━━━━━━━━

  ! Several of these imports are unused.

  6 │ CartesianGrid,
  7 │ Label,

  8 │ LabelList,
  │ ^^^^^^^^^
  9 │ type LabelProps,
  10 │ ResponsiveContainer,

  i Unused imports might be the result of an incomplete refactoring.

  i Unsafe fix: Remove the unused imports.

  6   6 │     CartesianGrid,
  7   7 │     Label,
  8     │ -   LabelList,
  9   8 │     type LabelProps,
  

  10 9 │ ResponsiveContainer,

src/app/(with-sidebar)/mypage/components/BarChart/view.tsx:34:7 lint/correctness/noUnusedVariables FIXABLE ━━━━━━━━━━

! This variable CustomColorLabel is unused.

32 │ };
33 │ 

34 │ const CustomColorLabel = (props: LabelProps) => {
│ ^^^^^^^^^^^^^^^^
35 │ const fill = COLORS[(props.index ?? 0) % COLORS.length];
36 │ return <Label {...props} fill={fill} />;

[output truncated]

• 확인 방법: bun biome check .가 exit code 0으로 종료되면 해결된 것입니다.

경고

• Lint 실패: exit code 1.
• TypeScript 타입 검사 실패: exit code 2.
• 테스트 실패: exit code 1.
• 빌드 실패: exit code 1.
• OpenAI review skipped because OPENAI_API_KEY is missing.