Skip to content

Conversation

@hbjORbj
Copy link
Contributor

@hbjORbj hbjORbj commented Dec 13, 2025

What does this PR do?

Moves OrganizationOnboardingRepository from packages/lib/server/repository/ to packages/features/organizations/repositories/ and updates all import references.

Why this change is needed

There's no reason for feature-specific repositories to live in lib - they belong in features where the related feature code resides. This refactor is part of a larger effort to move all repositories into repositories folders within their specific feature directories in features/, improving module boundaries and keeping feature code self-contained.

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • N/A - I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • N/A - I confirm automated tests are in place that prove my fix is effective or that my feature works.

@graphite-app graphite-app bot requested a review from a team December 13, 2025 14:11
@vercel
Copy link

vercel bot commented Dec 13, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
cal-companion Ready Ready Preview, Comment Dec 13, 2025 2:11pm

@hbjORbj hbjORbj changed the title refactor: move repositories to features 8 refactor: Move OrganizationOnboardingRepository from lib to /features/organizations Dec 13, 2025
@hbjORbj hbjORbj requested a review from sean-brydon December 13, 2025 14:12
@hbjORbj hbjORbj marked this pull request as draft December 13, 2025 14:12
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 11 files

@hbjORbj hbjORbj marked this pull request as ready for review December 13, 2025 14:28
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 11 files

@keithwillcode
Copy link
Contributor

Vercel dev deploy failed due to OOM issue. Non-blocking to merge since the production build passed here.

@keithwillcode keithwillcode merged commit 9ef650c into main Dec 14, 2025
86 of 93 checks passed
@keithwillcode keithwillcode deleted the refactor/move-repositories-to-features-8 branch December 14, 2025 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants