fix(seo): make the bare DoctorDerek.com domain canonical - #226
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
🧹 ESLintQuality check 1 of 3 ✅ Zero warnings. |
Contributor
🧪 VitestQuality check 2 of 3 ✅ Vitest completed successfully. |
Contributor
🎭 PlaywrightQuality check 3 of 3 ✅ Playwright completed successfully. |
Contributor
🗺️ XState v5 State Machine Diff VisualizationArchitecture visualization Base No XState state-machine topology changes detected. Download canonical graphs, structured diff, Mermaid, and diagnostics |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #226 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 54 54
Lines 828 828
Branches 162 162
=========================================
Hits 828 828
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Owner
Author
|
Code reviewed and approved. QA deferred. Merging now. |
This branch was successfully deployed
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
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.
Summary
Make
https://doctorderek.com/the canonical production origin using the existing URL owners. This is a six-file hostname-only change, split into three atomic commits.metadataBase; the existing relative canonical and Open Graph URLs resolve through it.Closes #225
Vercel routing and release boundary
Read-only Vercel CLI 60.0.1 inspection under
doctordereks-projects/doctor-derekconfirmed the existing configuration already matches the requested behavior:doctorderek.com: verified, no redirect, no branch/custom-environment binding.www.doctorderek.com: verified, HTTP 308 redirect todoctorderek.com.doctor-derek.vercel.appalias unchanged.Live HTTPS checks confirmed the bare homepage returns 200; WWW returns one 308 followed by 200. Manifest and favicon paths and their query parameters survive unchanged, including an encoded space. No redirect loop. No Vercel settings, DNS, subscriptions, or secrets were changed.
The current published metadata still uses WWW. The new built metadata is bare-domain, but publishing it requires the owner's merge and normal production deployment. Recheck the live canonical/Open Graph values after that release before calling the migration complete.
Verification
og:urlboth exactlyhttps://doctorderek.com, with no stale WWW identity.e2e/startup.spec.ts:72, waiting for the first Rive canvas. It failed identically in an isolated rerun and with both application URL owners temporarily restored to matchorigin/main; this failure is present without the hostname change. Approved bare-domain values were then restored. No animation or E2E code changed..pnpm-store/); all six changed files pass Prettier. Formatting was scoped to avoid unrelated churn.pnpm audit --prodis not clean: the unchanged dependency tree reports 3 high and 2 critical findings (browserslist, sharp, Next.js). Dependency remediation is outside this hostname PR; no packages or lockfile changed.Scope preserved
Preview and localhost URLs, production-only analytics gating, third-party URLs, sibling portfolio subdomains, relative manifest paths, copy, styles, and application behavior remain unchanged. No new origin abstraction, redirect middleware, test framework, storage migration, or deployment pipeline. No conflicting canonical GDD hostname was found, so no authority documents were edited.
After merge