fix(seo): use real LinkedIn URL in AUTHOR_CONFIG.sameAs#404
Merged
Conversation
The Tier C identity migration (#400) shipped with a placeholder LinkedIn URL because the actual profile didn't exist yet. Julian confirmed his real handle: julian-k-ba6b5897. Update both AUTHOR_CONFIG.sameAs and the docs/seo-strategy/drafts/julianken-profile-readme.md draft. A placeholder/404 URL in sameAs is an active-negative signal to Google's entity graph - same class of issue the original broken github.com/detached-node sameAs had. This closes the entity-graph gap. Closes #403
julianken
added
status:in-review
5 tasks
julianken-bot
approved these changes
May 17, 2026
Collaborator
julianken-bot
left a comment
julianken-bot
left a comment
There was a problem hiding this comment.
Verdict: APPROVE
Verification ledger
pnpm test:unitat PR HEADc3d50c0: 584/584 passing across 37 files (includingtests/unit/lib/schema/profile-page.test.ts)pnpm typecheckat PR HEAD: cleangrep -rn "julian-kennon" src/ tests/: zero matches — all in-scope source/test references are updated- LinkedIn URL
https://www.linkedin.com/in/julian-k-ba6b5897:curl -Ireturns 999 (LinkedIn's standard anti-bot response, expected; the PR body's "verified HTTP 200 from browser" claim is the right verification path here) - Bot collaborator permission on
julianken/detached-node:write— APPROVE event valid
Findings
None.
Notes
Second-pass thoughts (R8) on potential improvements:
- 8 strategy-archive files under
docs/seo-strategy/phase-*/,context-packets/, andissues/C-body.mdstill referencejulian-kennon. These are historical phase records (decisions captured at a point in time) and the closed source issue body; updating them would be revisionist and is explicitly out of the issue's stated scope. Correct call to leave alone. - Trailing-slash convention preserved: new URL has no trailing slash, matching the
https://github.com/juliankenentry — per the acceptance criterion in #403.
Bottom line
Three-line, well-scoped fix replacing a placeholder identifier with a verified real one. Tests + types green at HEAD. The companion test fixture update is the right amount of test maintenance for a config change — neither overshoots nor undershoots.
Owner
Author
|
@Mergifyio queue |
Contributor
Merge Queue Status
This pull request spent 16 seconds in the queue, including 2 seconds running CI. Required conditions to merge
|
mergify Bot
pushed a commit
that referenced
this pull request
May 18, 2026
* chore(docs): drop seo-strategy folder; align README to renamed post slug Removes docs/seo-strategy/ — research artifacts from the SEO + AI- discovery analysis funnel, no longer load-bearing now that the gate-1/2/3 work has shipped (#393 #394 #395 #396 #397 #400 #402 #404 #406 #408). History preserved in git. README: align "Recent essays" entry with the renamed post slug (where-agentic-patterns-actually-live → agentic-patterns-in-your-coding-workflow). The rename satisfies Bing Site Scan's 70-char title cap. No redirect deployed — article is two days old, no significant external link equity to preserve. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * fix: defer README slug update; gitignore docs/seo-strategy Address julianken-bot review of PR #409: BLOCKER (README:75) — New slug URL serves an SSR 404 fallback because the Payload post slug hasn't been renamed yet (intentionally deferred until the in-flight Bing Site Scan completes). Reverting the README link change here; it will land in a follow-up PR after the actual Payload slug rename, so the link is never broken in main. Plus: add /docs/seo-strategy/ to .gitignore so future analysis-funnel artifacts (phase-*, context-packets, STATUS.md, issues/) stay on disk without polluting the index. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.7 (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
2 participants
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
Replace the LinkedIn placeholder
/in/julian-kennonin two places with Julian's real URL/in/julian-k-ba6b5897.Changes
src/lib/schema/config.ts—AUTHOR_CONFIG.sameAssecond entry.docs/seo-strategy/drafts/julianken-profile-readme.md— LinkedIn link in the contact section.tests/unit/lib/schema/profile-page.test.ts— mockAUTHOR_CONFIG.sameAsupdated to match (test fixture follow-on from config change).URL verified HTTP 200 at time of writing.
Test plan
pnpm lint,pnpm test:unit(584 passing),pnpm typecheckall passCloses #403