Skip to content

chore: remove promoted WORK24 implementation - #62

Merged
seoJing merged 1 commit into
mainfrom
cleanup/work24-standalone-extraction
Oct 22, 2026
Merged

seoJing merged 1 commit into
mainfrom
cleanup/work24-standalone-extraction

Conversation

@seoJing

@seoJing seoJing commented Oct 22, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • remove the WORK24 README Lab routes, UI/runtime, assets, proxy endpoints, and integration-only tests from SEOJing
  • remove navigation and sitemap exposure after promotion to the standalone private repository
  • preserve blog, article, post-management, MDX retirement, RSS, sitemap, and llms behavior unrelated to WORK24

Preservation

The latest WORK24 behavior is preserved in the private seoJing/work24-readme repository at 330463ce99c59801096ad915b2152de9e3588818. Independent parity review verified all 79 source-map rows and accepted this removal with no findings.

Verification

  • frozen install, formatting, lint, 211 tests, and production build passed
  • generated route inventory and bundles contain no /readme route
  • secret/auth scan found no Jev, Typesafe, API-key, or Codex-auth runtime artifact
  • git diff --check passed

Summary by CodeRabbit

  • 변경 사항
    • README 시연 페이지와 공모전 전용 페이지가 제거되었습니다.
    • README 분석·미리보기 및 Lab 기능과 관련 API 경로가 제거되었습니다.
    • 블로그·포트폴리오 내 README 시연 링크와 사이트맵 항목이 제거되었습니다.
    • README 관심도 분석 기능이 제거되었습니다.

@coderabbitai

coderabbitai Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: seoJing/SEOJing/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 051725cf-58c6-4080-af35-fe2da7ff24a7
📥 Commits

Reviewing files that changed from the base of the PR and between 88e2bd8 and b7f12af.

📒 Files selected for processing: 36
  • .prettierignore
  • apps/web/src/app/api/readme/lab/[...path]/route.test.ts
  • apps/web/src/app/api/readme/lab/[...path]/route.ts
  • apps/web/src/app/readme/api/analyze/route.test.ts
  • apps/web/src/app/readme/api/analyze/route.ts
  • apps/web/src/app/readme/api/preview/route.test.ts
  • apps/web/src/app/readme/api/preview/route.ts
  • apps/web/src/app/readme/contest/page.tsx
  • apps/web/src/app/readme/lab/__fixtures__/focus-correction.json
  • apps/web/src/app/readme/lab/__fixtures__/focus-late-answer.json
  • apps/web/src/app/readme/lab/__fixtures__/focus-no-question-long.json
  • apps/web/src/app/readme/lab/__fixtures__/focus-note-retraction.json
  • apps/web/src/app/readme/lab/__fixtures__/focus-parked-active.json
  • apps/web/src/app/readme/lab/__fixtures__/focus-partial-failure.json
  • apps/web/src/app/readme/lab/__fixtures__/focus-reconnect.json
  • apps/web/src/app/readme/lab/__fixtures__/focus-service-completed.json
  • apps/web/src/app/readme/lab/__fixtures__/situation-contract.json
  • apps/web/src/app/readme/lab/essay-input.test.ts
  • apps/web/src/app/readme/lab/focus-fixtures.test.ts
  • apps/web/src/app/readme/lab/focus-reader.test.ts
  • apps/web/src/app/readme/lab/html-raw.d.ts
  • apps/web/src/app/readme/lab/playback-pace.test.ts
  • apps/web/src/app/readme/lab/reader-state.test.ts
  • apps/web/src/app/readme/lab/readme-lab.html
  • apps/web/src/app/readme/lab/route.test.ts
  • apps/web/src/app/readme/lab/route.ts
  • apps/web/src/app/readme/lab/situation-reader.test.ts
  • apps/web/src/app/readme/page.tsx
  • apps/web/src/app/sitemap.ts
  • apps/web/src/shared/readme/analytics.test.ts
  • apps/web/src/shared/readme/analytics.ts
  • apps/web/src/shared/readme/backend-origin.ts
  • apps/web/src/shared/readme/preview.ts
  • apps/web/src/widgets/header/header.constants.ts
  • apps/web/src/widgets/readme/ReadmeDemo.test.tsx
  • apps/web/src/widgets/readme/ReadmeDemo.tsx
💤 Files with no reviewable changes: 34
  • apps/web/src/app/readme/lab/focus-reader.test.ts
  • apps/web/src/app/readme/lab/route.ts
  • apps/web/src/app/readme/lab/fixtures/focus-service-completed.json
  • apps/web/src/app/sitemap.ts
  • apps/web/src/app/readme/lab/html-raw.d.ts
  • .prettierignore
  • apps/web/src/app/readme/contest/page.tsx
  • apps/web/src/app/readme/page.tsx
  • apps/web/src/app/readme/lab/fixtures/focus-partial-failure.json
  • apps/web/src/shared/readme/analytics.test.ts
  • apps/web/src/app/readme/lab/focus-fixtures.test.ts
  • apps/web/src/app/readme/lab/fixtures/focus-late-answer.json
  • apps/web/src/app/readme/lab/fixtures/focus-note-retraction.json
  • apps/web/src/shared/readme/backend-origin.ts
  • apps/web/src/app/readme/api/analyze/route.ts
  • apps/web/src/app/readme/lab/route.test.ts
  • apps/web/src/app/readme/api/preview/route.test.ts
  • apps/web/src/app/readme/lab/fixtures/focus-parked-active.json
  • apps/web/src/app/readme/lab/essay-input.test.ts
  • apps/web/src/app/api/readme/lab/[...path]/route.test.ts
  • apps/web/src/app/readme/lab/fixtures/situation-contract.json
  • apps/web/src/widgets/readme/ReadmeDemo.test.tsx
  • apps/web/src/app/readme/api/analyze/route.test.ts
  • apps/web/src/shared/readme/analytics.ts
  • apps/web/src/app/readme/lab/situation-reader.test.ts
  • apps/web/src/app/readme/lab/fixtures/focus-correction.json
  • apps/web/src/app/readme/lab/fixtures/focus-reconnect.json
  • apps/web/src/app/readme/api/preview/route.ts
  • apps/web/src/app/readme/lab/reader-state.test.ts
  • apps/web/src/widgets/readme/ReadmeDemo.tsx
  • apps/web/src/widgets/header/header.constants.ts
  • apps/web/src/app/readme/lab/playback-pace.test.ts
  • apps/web/src/app/api/readme/lab/[...path]/route.ts
  • apps/web/src/shared/readme/preview.ts

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Walkthrough

README 데모와 공모전 페이지, Lab 페이지 및 관련 API 경로를 제거했습니다. 공유 분석·미리보기 코드, Lab 픽스처와 테스트, 내비게이션 항목 및 사이트맵 항목도 삭제했습니다.

Changes

README 기능 제거

Layer / File(s) Summary
README 데모와 공개 경로 제거
apps/web/src/widgets/readme/*, apps/web/src/shared/readme/*, apps/web/src/app/readme/api/{analyze,preview}/*, apps/web/src/app/readme/{page.tsx,contest/page.tsx}, apps/web/src/widgets/header/header.constants.ts, apps/web/src/app/sitemap.ts
README 데모 컴포넌트와 분석·미리보기 API, 공유 분석 및 미리보기 코드를 제거했습니다. README 페이지와 공모전 페이지, 두 내비게이션 항목 및 사이트맵 항목도 제거했습니다.
Lab 페이지와 프록시 경로 제거
apps/web/src/app/readme/lab/route.ts, apps/web/src/app/readme/lab/route.test.ts, apps/web/src/app/api/readme/lab/[...path]/*, apps/web/src/app/readme/lab/html-raw.d.ts, .prettierignore
Lab HTML 제공 라우트와 API 프록시, 관련 라우트 테스트 및 HTML 모듈 타입 선언을 제거했습니다. .prettierignore에서 Lab HTML 페이지와 픽스처 디렉터리 제외 항목도 삭제했습니다.
Lab 판독 데이터와 테스트 제거
apps/web/src/app/readme/lab/__fixtures__/*, apps/web/src/app/readme/lab/*.{test.ts}
Lab 판독 시나리오 픽스처와 입력, 판독 상태, 이벤트 재생 및 렌더링을 검증하던 테스트를 제거했습니다.

Priority: ⬇️ Low

Estimated code review effort: 3 (Moderate) | ~25 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to b7f12

The removed README pages are no longer exposed through the inspected site links. No actionable merge risk remains beyond normal checks.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed 제목은 PR의 주요 변경 사항인 승격된 WORK24 구현 제거를 정확하고 간결하게 설명합니다.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files.

✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@seoJing
seoJing merged commit aaecab1 into main Oct 22, 2026
5 checks passed
@seoJing
seoJing deleted the cleanup/work24-standalone-extraction branch October 22, 2026 01:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant