Skip to content

refactor: centralize public profile data#39

Merged
corwinm merged 1 commit into
mainfrom
refactor/public-profile-source
Jul 13, 2026
Merged

refactor: centralize public profile data#39
corwinm merged 1 commit into
mainfrom
refactor/public-profile-source

Conversation

@corwinm

@corwinm corwinm commented Jul 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • add a shared publicProfile content module for durable identity, career, image, employer, and canonical profile-link data
  • use it across About, Connect, footer, header, social sharing, SEO defaults, profile imagery, and ProfilePage structured data
  • add regression tests for Corwin's public identity and canonical LinkedIn/GitHub/Bluesky/resume links

Why

These facts were repeated across several components and SEO helpers, making it easy for visible copy, contact URLs, and machine-readable metadata to drift apart.

The resume route is intentionally excluded because application-specific resume work is active in a separate worktree.

Test plan

  • TDD RED: profile tests initially failed because the shared module did not exist
  • focused profile tests: 2/2 passing
  • pnpm lint — 0 warnings/errors
  • pnpm typecheck
  • pnpm test — 36/36 passing
  • pnpm build
  • git diff --check
  • production browser verification: homepage identity/career copy, profile image, Connect links, header title, and JSON-LD values remained unchanged
  • independent pre-commit review: passed with no security concerns or logic errors

@vercel

vercel Bot commented Jul 13, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
corwinm-remix Ready Ready Preview, Comment Jul 13, 2026 8:40pm

@corwinm
corwinm merged commit de843f6 into main Jul 13, 2026
3 checks passed
@corwinm
corwinm deleted the refactor/public-profile-source branch July 13, 2026 20:47
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