Skip to content

feat: sync profile facts with LinkedIn - #68

Merged
jonathanperis merged 1 commit into
mainfrom
feature/linkedin-sync
Oct 7, 2026
Merged

jonathanperis merged 1 commit into
mainfrom
feature/linkedin-sync

Conversation

@jonathanperis

@jonathanperis jonathanperis commented Oct 7, 2026 •

Copy link
Copy Markdown
Owner

Summary

Aligns the website with the LinkedIn profile audit (website stays the source of truth; LinkedIn is being updated to match in parallel).

  • Sabesp: 2011 — 2013, São Paulo → Aug 2011 — Apr 2013, Diadema (LinkedIn had the exact dates)
  • Derivative Path: adds "which manage billions in notional value" (previously only on LinkedIn)
  • Career line kind: "every tech role since 2011". LinkedIn also lists two pre-tech roles (2008–2011) that intentionally stay off the site
  • MetroMap: the eyebrow and swipe hint used period.slice(0, 4), which would render Aug now; they read the year via stationPeriod(...)[0].slice(-4) instead
  • CV PDF regenerated from /resume/ (still 2 pages)

Verification

  • bun run lint: 0 errors
  • bun run build with live GitHub data: OK; output shows 2011 — now, Swipe back to 2011, Aug 2011 — Apr 2013 on / and /resume/

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Updates
    • The career timeline now identifies its coverage as tech roles since 2011, and its displayed start year is derived from the first station’s date.
    • The Sabesp role now shows its period as August 2011–April 2013 and its location as Diadema, Brazil.
    • The Derivative Path experience now notes that DerivativeEDGE manages billions in notional value.

- Sabesp: exact dates (Aug 2011 — Apr 2013) and location (Diadema)
- Derivative Path: note the modules manage billions in notional value
- Career line: "every tech role since 2011" (pre-tech roles stay LinkedIn-only)
- Map eyebrow/swipe hint read the origin year from the period end, so a month prefix no longer breaks it
- Regenerate cv_jonathan_peris.pdf

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: jonathanperis/jonathanperis.github.io/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 839173c8-83ef-4115-a2e3-3d36c21024a3
📥 Commits

Reviewing files that changed from the base of the PR and between 02f74ec and 8334b83.

⛔ Files ignored due to path filters (1)
  • public/cv_jonathan_peris.pdf is excluded by !**/*.pdf
📒 Files selected for processing (3)
  • src/components/sections/MetroMap.astro
  • src/lib/data.ts
  • src/lib/metro.ts

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The Career metro map updates its descriptions and Sabesp station data. The map’s displayed start year now comes from the first station’s period.

Changes

Career Metro Map

Layer / File(s) Summary
Career data and station matching
src/lib/data.ts, src/lib/metro.ts
The Career description now specifies tech roles. The DerivativeEDGE description adds that the platform manages billions in notional value. The Sabesp period and location change, and the station lookup matches the updated period.
Metro map start year
src/components/sections/MetroMap.astro
The map derives the start year from the first station’s period. The eyebrow and swipe prompt use that year.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 8334b

The profile date and Career map year remain aligned; no material merge risk is identified in the reviewed changes.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the primary profile updates made to align website facts with LinkedIn. It is concise and specific.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 2…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@jonathanperis
jonathanperis merged commit eac6566 into main Oct 7, 2026
5 checks passed
@jonathanperis
jonathanperis deleted the feature/linkedin-sync branch October 7, 2026 20:26
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