fix: align the site with the verified employment record - #31
Merged
Merged
Conversation
The hire page and the about timeline each carried a claim the record does not support. The timeline merged two CTO roles into one five-year ItemBox entry running to 2026; ItemBox closed in October 2025, and the 2021-2024 half was GEOMEC. Split them, and give ItemBox the numbers the repositories actually show: nine engineers across Korea and Vietnam, a 20-module NestJS/TypeORM backend with 300 REST endpoints, 58 controllers, 55 entities and ~72k lines of TypeScript. Two smaller corrections travel with it. "5+ years owning engineering as a CTO" was 4 years 9 months, so it now reads "nearly five years across two companies". The site spelled the name "Si Hyeong Lee" while the resume and LinkedIn use "Sihyeong Lee" - a recruiter comparing the three reads that as two people, so the public surfaces (messages, JSON-LD, llms.txt) now use one spelling. LICENSE and the historical planning docs keep theirs. The hire page also never said "US permanent resident, no sponsorship required", which is the quietest reason a US employer skips a candidate. It says it now, in both languages, and the working style is stated as fully remote rather than remote-first. employment-positioning.test.ts and seo.test.ts assert these strings verbatim, so both move with the copy.
The ItemBox timeline copy carried raw commit, endpoint, controller and entity counts (857 of 2,458 commits, 300 REST endpoints, 58 controllers, 55 entities, ~72k lines of TypeScript). Those were pulled from the resume for reading as filler rather than evidence, and the site was about to put them back. Rewrote both locales to match the resume: what was owned and built, not how many of it there were. Also replaced the em dashes across the timeline copy with periods, colons and middots, and switched the title separators to the middot the Korean titles already use. Adds a test so neither regression can come back through the site copy. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LY9haZyJhdQfT45Jamnzrq
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
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
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.
The live site still carries an employment record that does not match the verified
one, and the copy that was queued to fix it had a second problem of its own.
What was wrong on the live site
Two commits
b025465 corrects the record: real ItemBox dates, a GEOMEC entry, the name
spelling, and remote/residency wording that matches the resume.
2a363c1 fixes that commit's own copy. It had carried raw counts over from an
earlier resume draft: 857 of 2,458 commits, 335 backend commits, 11 production
deployments, 300 REST endpoints, 58 controllers, 55 entities, ~72k lines of
TypeScript. Those read as filler rather than evidence and had already been cut
from the resume, so the site was about to put them back. Both locales now say
what was owned and built. Em dashes across the timeline copy are gone too, and
the title separators use the middot the Korean titles already use.
A test pins both so neither can come back through the site copy.
Note for the reviewer
Only these two files are staged in 2a363c1. The working tree separately shows
~196 files as modified because they drifted to CRLF while the committed versions
are LF. That is pre-existing, untouched here, and worth a
.gitattributespasson its own.
🤖 Generated with Claude Code
https://claude.ai/code/session_01LY9haZyJhdQfT45Jamnzrq