Skip to content

[REFACTOR] 미사용 서비스워커 클린업 로직 제거 - #284

Open
kimminna wants to merge 1 commit into
developfrom
refactor/web/283-remove-service-worker-cleanup
Open

[REFACTOR] 미사용 서비스워커 클린업 로직 제거#284
kimminna wants to merge 1 commit into
developfrom
refactor/web/283-remove-service-worker-cleanup

Conversation

@kimminna

Copy link
Copy Markdown
Member

ISSUE 🔗

close #283



What is this PR? 🔍

도메인 이전 이후 더 이상 필요 없는 구버전 서비스워커/캐시 정리 로직을 제거했습니다.

배경

  • 기존 구조: 과거 다른 도메인에서 PWA로 설치된 구버전 서비스워커가 사용자 브라우저에 남아 캐시된 구버전 페이지를 계속 서빙하는 문제를 막기 위해, 루트 레이아웃에서 ServiceWorkerCleanup 컴포넌트가 현재 origin의 서비스워커/캐시를 전부 unregister/delete하고 있었습니다.
  • 발생 문제: 도메인 이전이 완료되어 구버전 서비스워커가 남아있는 사용자가 사실상 없어졌음에도, 모든 페이지 로드마다 불필요하게 serviceWorker.getRegistrations()caches.keys()를 호출하는 코드가 계속 남아 있었습니다.
  • 해결 방향: 더 이상 필요 없는 ServiceWorkerCleanup 컴포넌트와 루트 레이아웃에서의 사용 코드를 제거했습니다.

서비스워커 클린업

  • 변경 요약: apps/timo-web/providers/service-worker/ServiceWorkerCleanup.tsx 파일을 삭제하고, app/[locale]/layout.tsx에서 해당 컴포넌트의 import와 렌더링 코드를 제거했습니다.
  • 이유: 목적이었던 구버전 서비스워커/캐시 정리가 이미 완료되어 더 이상 유지할 필요가 없는 코드였습니다.
  • 구현 방식: 별도 대체 로직 없이 컴포넌트와 사용처를 함께 삭제하는 방식으로 정리했습니다. 다른 파일에서의 참조 여부를 grep으로 확인했으며 남아있는 참조는 없었습니다.



To Reviewers

기능 동작에 영향을 주지 않는 순수 삭제(remove) 변경입니다. ServiceWorkerCleanup을 참조하는 다른 코드가 없는지 확인 부탁드립니다.



Screenshot 📷



Test Checklist ✔

  • pnpm check-types 통과
  • pnpm lint 통과
  • pnpm build 통과

- 도메인 이전 이후 불필요해진 구버전 서비스워커/캐시 정리 로직을 제거했습니다
- 루트 레이아웃에서 관련 import 및 사용 코드를 제거했습니다
@github-actions
github-actions Bot requested review from ehye1 and jjangminii August 24, 2026 10:03
@vercel

vercel Bot commented Aug 24, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
timo Ready Ready Preview Aug 24, 2026 10:04am

@github-actions github-actions Bot added the ⏰ Timo-web Timo 웹 서비스 label Aug 24, 2026
@coderabbitai

coderabbitai Bot commented Aug 24, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 33 minutes.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 11b65ce7-43a4-48e2-8203-965ebbeb353e

📥 Commits

Reviewing files that changed from the base of the PR and between 48370eb and bd64729.

📒 Files selected for processing (2)
  • apps/timo-web/app/[locale]/layout.tsx
  • apps/timo-web/providers/service-worker/ServiceWorkerCleanup.tsx

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added ♻ Refactor 기능 개선 및 리팩토링 작업 ♦️ 민아 민아상 labels Aug 24, 2026
@github-actions

Copy link
Copy Markdown

Timo Performance Report

Bundle Size — timo-web
라우트 크기 First Load JS
/[locale]/home 274.70 kB 🔴 480.51 kB
/[locale]/today 258.89 kB 🔴 464.70 kB
/[locale]/focus 221.51 kB 🔴 427.32 kB
/[locale]/settings 228.40 kB 🔴 434.21 kB
/[locale]/statistics 210.09 kB 🔴 415.91 kB
/[locale]/[...rest] 0 B 🟡 205.81 kB
/[locale]/login 281.73 kB 🔴 487.54 kB
/[locale]/oauth/calendar/callback 120.82 kB 🟡 326.63 kB
/[locale]/oauth/callback 120.48 kB 🟡 326.30 kB
/[locale]/onboarding 294.21 kB 🔴 500.02 kB
/[locale] 119.80 kB 🟡 325.61 kB
/[locale]/policy 120.77 kB 🟡 326.58 kB
/robots.txt/route 0 B 🟡 205.81 kB
/sitemap.xml/route 0 B 🟡 205.81 kB

공유 번들: 205.81 kB
🟢 < 200kB  |  🟡 < 350kB  |  🔴 ≥ 350kB (First Load JS · gzip)

Lighthouse — timo-web
URL Perf A11y LCP CLS TBT
/en/home 🔴 66 🟢 96 🔴 16.2s 🟢 0.000 🟡 367ms
/en/today 🔴 65 🟢 96 🔴 16.2s 🟢 0.000 🟡 386ms
/en/focus 🔴 67 🟢 96 🔴 16.0s 🟢 0.000 🟡 319ms
/en/statistics 🔴 67 🟢 96 🔴 16.0s 🟢 0.000 🟡 344ms

Perf ≥ 70 / A11y ≥ 85 목표
LCP 🟢 < 2.5s 🟡 < 4s 🔴 ≥ 4s  |  CLS 🟢 < 0.1 🟡 < 0.25 🔴 ≥ 0.25  |  TBT 🟢 < 200ms 🟡 < 600ms 🔴 ≥ 600ms

Image Optimization — timo-web
파일 크기 포맷 상태
favicon.png 27.84 kB PNG ⚠️ 🟢
images/google-calendar.png 36.20 kB PNG ⚠️ 🟢
images/google-logo.png 26.79 kB PNG ⚠️ 🟢
og.png 437.44 kB PNG ⚠️ 🟡

총 4개 · 528.28 kB  |  🟢 < 200KB  |  🟡 < 500KB  |  🔴 ≥ 500KB
⚠️ 4개 파일 WebP/AVIF 변환 권장

측정 커밋: a4af604

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⏰ Timo-web Timo 웹 서비스 ♦️ 민아 민아상 ♻ Refactor 기능 개선 및 리팩토링 작업

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[REFACTOR] 미사용 서비스워커 클린업 로직 제거

1 participant