refactor: Improve code quality and organization

[modeitsch]

Summary

• Move shared components (ScrollProgress, ThreeBackground, ScrollAnimations) to layout component, reducing code duplication across pages
• Add TypeScript types for data structures in src/lib/types/:
  • Skill, Project, Experience, ContactMethod, FAQ
• Extract hardcoded data to src/lib/data/ folder:
  • skills.ts: Home and about page skills
  • projects.ts: Featured and all projects lists
  • experiences.ts: Work experience history
  • technologies.ts: Tech stack lists
  • contact.ts: Contact methods and FAQs
• Create barrel exports for easy imports from $lib/data and $lib/types

Benefits

• Reduced code duplication - shared components are now in one place
• Better type safety with TypeScript interfaces
• Easier to update data without touching page components
• More maintainable codebase structure

Test plan

• Build succeeds
• All pages render correctly
• Data displays properly on all pages

🤖 Generated with Claude Code

Summary by CodeRabbit

Release Notes

• Refactor
  • Centralized application data management by consolidating skills, projects, experiences, contact methods, and FAQs into dedicated data modules.
  • Reorganized global components (scroll progress indicator, background effects, scroll animations) into the main layout for consistent behavior across all pages.
  • Simplified page components by removing redundant data definitions and importing from the centralized data layer.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

This PR centralizes application data and types into modular structures. New TypeScript interfaces define data shapes for skills, projects, experiences, contact methods, and FAQs. Data modules containing static arrays are created. A barrel export file aggregates data imports. Global components are moved to the root layout. Pages are refactored to import centralized data instead of maintaining inline definitions.

Changes

Cohort / File(s)	Summary
Type Definitions src/lib/types/index.ts	Adds five new TypeScript interfaces: Skill, Project, Experience, ContactMethod, and FAQ to define data shapes across the application.
Data Modules src/lib/data/contact.ts, src/lib/data/experiences.ts, src/lib/data/projects.ts, src/lib/data/skills.ts, src/lib/data/technologies.ts	Introduces five new data modules, each exporting typed constants: contactMethods and faqs (contact); experiences (career); featuredProjects and allProjects (projects); homeSkills and aboutSkills (skills); techStackPreview and allTechnologies (technologies).
Data Barrel Export src/lib/data/index.ts	Creates a centralized barrel file re-exporting all data collections from individual modules for simplified imports across the application.
Global Components Integration src/routes/+layout.svelte	Adds imports and renders ScrollProgress, ThreeBackground, and ScrollAnimations as global components in the root layout.
Page Refactoring src/routes/+page.svelte, src/routes/about/+page.svelte, src/routes/contact/+page.svelte, src/routes/projects/+page.svelte	Replaces inline data definitions and local component imports with centralized data imports from $lib/data; removes redundant component declarations now available globally.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~15 minutes

• Verify data accuracy in new data modules to ensure all hard-coded arrays transcribed correctly
• Confirm barrel export file includes all necessary re-exports
• Check that all page imports reference correct symbols from $lib/data
• Validate that removed component imports don't affect page functionality (now provided by layout)

Poem

🐰 Hop, skip, and a leap—data now organized and neat!
From scattered inline to modules so sweet,
Types at the ready, imports so clean,
This refactored codebase is pristine! 🎉

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch code-quality

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between eb82249 and b45420a.

📒 Files selected for processing (12)

• src/lib/data/contact.ts (1 hunks)
• src/lib/data/experiences.ts (1 hunks)
• src/lib/data/index.ts (1 hunks)
• src/lib/data/projects.ts (1 hunks)
• src/lib/data/skills.ts (1 hunks)
• src/lib/data/technologies.ts (1 hunks)
• src/lib/types/index.ts (1 hunks)
• src/routes/+layout.svelte (2 hunks)
• src/routes/+page.svelte (2 hunks)
• src/routes/about/+page.svelte (1 hunks)
• src/routes/contact/+page.svelte (1 hunks)
• src/routes/projects/+page.svelte (1 hunks)

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}