Skip to content

fix(database): added pg support - #1

Merged
hemanth5544 merged 2 commits into
mainfrom
dev
Mar 20, 2026
Merged

hemanth5544 merged 2 commits into
mainfrom
dev

Conversation

@hemanth5544

@hemanth5544 hemanth5544 commented Mar 20, 2026 •

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Chores
    • Added ESLint as a development dependency to enforce consistent code quality standards and best practices across the entire codebase.
    • Refactored internal database infrastructure by consolidating database client management and updating core connectivity components to improve system consistency, reliability, and overall maintainability throughout the application while preserving all existing functionality and user experiences.

@coderabbitai

coderabbitai Bot commented Mar 20, 2026 •

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 0e66cdc8-a3b0-4222-a370-d9c2003a7494

📥 Commits

Reviewing files that changed from the base of the PR and between ee1d147 and b24560f.

⛔ Files ignored due to path filters (1)
  • bun.lock is excluded by !**/*.lock
📒 Files selected for processing (4)
  • apps/client/app/api/auth/github/callback/route.ts
  • package.json
  • packages/store/index.ts
  • packages/store/package.json

📝 Walkthrough

Walkthrough

The pull request migrates the database connectivity layer from Neon serverless adapter to the standard PostgreSQL adapter (@prisma/adapter-pg) in the shared @repo/store module. The GitHub OAuth callback route is refactored to use the centralized exported prismaClient instead of instantiating its own local Prisma instance. Additionally, eslint is added as a development dependency.

Changes

Cohort / File(s) Summary
Database Adapter Migration
packages/store/index.ts, packages/store/package.json
Switched Prisma adapter from PrismaNeon (@prisma/adapter-neon) to PrismaPg (@prisma/adapter-pg); added pg and dotenv as runtime dependencies while removing Neon-specific dependencies.
Shared Prisma Client Adoption
apps/client/app/api/auth/github/callback/route.ts
Removed local Prisma client instantiation and migrated all database operations (findUnique, update, create) to use the shared prismaClient exported from @repo/store.
Development Tooling
package.json
Added eslint version 9 as a new devDependency.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 A database hop from Neon's glow,
To PrismaPg's steady flow—
Shared clients now unite the way,
One source of truth, hip hooray!
Linting tools join in the dance,
Our codebase takes a fresh advance! ✨

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch dev
📝 Coding Plan
  • Generate coding plan for human review comments

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 and usage tips.

Tip

You can customize the high-level summary generated by CodeRabbit.

Configure the reviews.high_level_summary_instructions setting to provide custom instructions for generating the high-level summary.

@hemanth5544
hemanth5544 merged commit b01c979 into main Mar 20, 2026
1 of 2 checks passed
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