Conversation
This change finishes the planned feedback pass: recruiter-facing project facts now flow from one canonical source, flagship case-study copy is stronger, mobile navigation and contact UX are production-safe, and the app now emits route-level SEO/share metadata with automation to keep it from regressing. Constraint: Must avoid committing local agent/tooling artifacts from this workstation Constraint: Keep the contact hardening lightweight and self-contained without new services or dependencies Rejected: External CMS for project content | too much operational overhead for a portfolio-sized system Rejected: Third-party anti-spam service | unnecessary complexity for current inquiry volume Confidence: medium Scope-risk: broad Directive: Keep recruiter-visible facts anchored to shared project detail data; do not reintroduce duplicate summaries without regression checks Tested: pnpm lint; pnpm exec vitest run; pnpm build; pnpm exec playwright test tests/main.spec.ts tests/contact.mobile.spec.ts --project=Desktop Chrome --project=Mobile Chrome Not-tested: GitHub-hosted workflow execution; full cross-browser Playwright matrix; scheduled performance workflow in Actions
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
The root-level opengraph-image.tsx and twitter-image.tsx were unreachable because the i18n middleware intercepted /opengraph-image before the route handler could serve the image. Moving them under [locale] ensures the images are served at locale-prefixed paths and og:image / twitter:image meta tags are now emitted on all pages including the homepage. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR 유형
어떤 변경 사항이 있나요?
변경 요약
채용 관점 피드백을 반영해 포트폴리오의 신뢰성, 모바일 UX, SEO/공유 품질, 자동화 신호를 한 번에 정리했습니다.
1) 콘텐츠 / 신뢰성
projectDetailList기준 단일 소스로 정리2) 모바일 UX / Contact
3) SEO / 공유 / 구조화 데이터
sitemap.xml,robots.txt추가4) 품질 / 자동화
주요 파일
src/shared/constant/project.tsxsrc/shared/constant/project-detail.tsxsrc/shared/constant/site.tssrc/shared/ui/header.tsxsrc/feature/mail/action/send-mail.tssrc/feature/mail/ui/email-form.tsxsrc/app/sitemap.tssrc/app/robots.ts.github/workflows/ci.ymldocs/architecture-tradeoffs.md검증
pnpm lintpnpm exec vitest runpnpm buildpnpm exec playwright test tests/main.spec.ts tests/contact.mobile.spec.ts --project="Desktop Chrome" --project="Mobile Chrome"리스크 / 후속 메모
기타