Skip to content

Conversation

@hbjORbj
Copy link
Contributor

@hbjORbj hbjORbj commented Dec 12, 2025

What does this PR do?

Moves PrismaAgentRepository and PrismaPhoneNumberRepository from packages/lib/server/repository/ to packages/features/calAIPhone/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 12, 2025 19:18
@vercel
Copy link

vercel bot commented Dec 12, 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 14, 2025 8:05am
2 Skipped Deployments
Project Deployment Review Updated (UTC)
cal Ignored Ignored Dec 14, 2025 8:05am
cal-eu Ignored Ignored Dec 14, 2025 8:05am

@hbjORbj hbjORbj marked this pull request as draft December 12, 2025 19:19
@hbjORbj hbjORbj changed the title refactor: move repositories to features refactor: move calAIPhone-specific repositories to features from lib Dec 12, 2025
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 12 files

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 17 files

volnei
volnei previously requested changes Dec 12, 2025
Copy link
Contributor

@volnei volnei left a comment

Choose a reason for hiding this comment

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

Let's fix all import prisma to import { prisma } 🙏

@github-actions github-actions bot marked this pull request as draft December 12, 2025 20:05
auto-merge was automatically disabled December 12, 2025 20:05

Pull request was converted to draft

Co-authored-by: Volnei Munhoz <volnei@cal.com>
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 17 files

@hbjORbj hbjORbj merged commit deca7ce into main Dec 15, 2025
60 of 63 checks passed
@hbjORbj hbjORbj deleted the refactor/move-repositories-to-features branch December 15, 2025 08:13
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.

4 participants