BaekjoonHub

개요

BaekjoonHub은 백준, 프로그래머스, SWEA에서 맞은 풀이를 자동으로 GitHub 저장소에 커밋해 주는 브라우저 확장입니다. 제가 SWEA에서 직접 쓰다가 마주친 자동 업로드 중단 버그를 수정해 기여했습니다.

기여 내용

SWEA 결과 페이지의 DOM 구조가 바뀌면서 scripts/swexpertacademy/parsing.js의 셀렉터가 더 이상 맞지 않아, 첫 제출만 올라가고 이후 제출은 업로드가 끊기는 문제였습니다(이슈 #333).

PR #334에서 다음과 같이 수정했습니다.

• DOM 변경에 대비한 selector fallback 추가
• 닉네임, 문제 번호, 제출 정보 파싱을 null-safe 하게 보강
• 파싱이 성공한 뒤에만 업로드 UI를 노출
• problemId 조회 실패 시 contestProbId 로 대체하는 fallback
• GitHub hook/token/stats에 대한 명시적 null 체크

관련 이슈와 PR

SWEA push 끊김 · Issue #333 · BaekjoonHub/BaekjoonHub

버그를 설명해주세요 SW Expert Academy(SWEA)에서 정답(Pass) 제출 이후 BaekjoonHub가 GitHub에 자동 push를 수행하지 않습니다. 프로그래머스에서는 정상적으로 동작하지만, SWEA에서는 동작하지 않습니다. 재현 방법을 기술해주세요 OS: macO…

GitHub

fix: SWEA 자동 업로드 실패 수정 (#333) by bnbong · Pull Request #334 · BaekjoonHub/BaekjoonHub

Summary Fixes #333 SWEA(SW Expert Academy)에서 Pass 제출 후 GitHub 자동 업로드가 실패하던 문제를 수정했습니다. 주요 원인은 SWEA 결과 페이지 DOM 변경으로 인해 parseData()가 제출 정보를 제대로 파싱하지 못하는 상황에서도 …

GitHub

실수와 배운 점

PR이 머지된 직후 회귀(regression)가 생겼습니다. 새로 추가한 safeText() 헬퍼가 .innerText 대신 .textContent 를 사용한 것이 원인이었습니다.

• .textContent 는 숨겨진 자식 요소(.badge 등)의 텍스트와 공백, 개행까지 그대로 가져옵니다. 그 결과 제목에서 뽑아낸 값에 newline/tab이 섞여 파일 경로가 깨졌습니다.
• 메인테이너가 공백 정규화 + .badge 자식 제거로 핫픽스를 배포했습니다.

회고하며 두 가지를 정리했습니다.

• .textContent 와 .innerText 의 의미 차이를 정확히 몰랐습니다. 화면에 보이는 텍스트가 필요할 때와 원시 텍스트가 필요할 때를 구분해야 합니다.
• 테스트 격리가 부족했습니다. 두 개의 확장을 동시에 켠 채로 검증하는 바람에 버그가 가려졌습니다. 확장 프로그램은 격리된 단일 환경에서 검증해야 합니다.

기능을 고치는 것만큼이나 "왜 제 테스트에서는 안 잡혔는가"를 되짚는 과정이 남는 기여였습니다.