Skip to content

Add MariaDB/PostgreSQL support and 100% test coverage - #1

Closed
seonghobae with Copilot wants to merge 7 commits into
developfrom
copilot/add-mariadb-postgresql-support
Closed

seonghobae with Copilot wants to merge 7 commits into
developfrom
copilot/add-mariadb-postgresql-support

Conversation

Copilot AI commented Jul 23, 2026

Copy link
Copy Markdown

Superseded by issue #2 for a migration-safe rebuild from current protected develop.

The branch contains valuable test coverage and multi-provider exploration, but its current exact head cannot be merged safely. Current-head Security Scan fails on fixable brace-expansion, postcss, undici, and frontend react-router findings. More importantly, this branch changes the production db:migrate:deploy path from prisma migrate deploy to prisma db push --skip-generate, and the PostgreSQL-specific deployment path also uses db push despite shipping an explicit migration. That converts a reviewed migration workflow into runtime schema drift.

The provider schemas also replicate physical single-word table names such as users, files, and folders; these violate the CWL database naming contract requiring descriptive two-or-more-word snake_case object names. Multi-valued JSON fields also need a deliberate 1NF/3NF review instead of being copied across four provider schemas without a normalization decision.

Issue #2 preserves the useful work and defines the canonical rebuild: port the tests first, regenerate patched locks, converge on one provider-neutral logical schema, use explicit reversible migrations, normalize relational data to 3NF, validate upgrade/rollback on MySQL/MariaDB/PostgreSQL/SQLite, and only then merge/release. Keep this branch available as source material until those tests/provider deltas have been absorbed.

Copilot AI self-assigned this Jul 23, 2026
Copilot AI review requested due to automatic review settings July 23, 2026 10:06

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

Copilot AI changed the title Add MariaDB/PostgreSQL support and achieve 100% TDD coverage Add MariaDB/PostgreSQL support and 100% test coverage Jul 23, 2026
Copilot AI requested a review from seonghobae July 23, 2026 10:07
@seonghobae
seonghobae marked this pull request as ready for review July 23, 2026 10:07
Comment thread backend/coverage/lcov-report/prettify.js Fixed
Comment thread backend/coverage/prettify.js Fixed
Comment thread backend/src/modules/uploads/upload.routes.test.ts Fixed
Copilot AI review requested due to automatic review settings July 23, 2026 10:35

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

Copilot AI review requested due to automatic review settings July 24, 2026 11:22

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

@coderabbitai

coderabbitai Bot commented Aug 4, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@opencode-agent[bot], you've reached your PR review limit, so we couldn't start this review.

Next review available in: 14 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 03f321f7-8e57-4de4-b9cf-cfd93b28162d

📥 Commits

Reviewing files that changed from the base of the PR and between fea4e44 and 8430d78.

⛔ Files ignored due to path filters (1)
  • backend/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (50)
  • README.md
  • backend/.gitignore
  • backend/Dockerfile
  • backend/package.json
  • backend/prisma/migrations-mysql/README.md
  • backend/prisma/migrations-postgresql/.gitkeep
  • backend/prisma/migrations-postgresql/20260604000000_init/migration.sql
  • backend/prisma/migrations-postgresql/README.md
  • backend/prisma/migrations-postgresql/migration_lock.toml
  • backend/prisma/migrations-sqlite/.gitkeep
  • backend/prisma/migrations-sqlite/README.md
  • backend/prisma/schema.mariadb.prisma
  • backend/prisma/schema.mysql.prisma
  • backend/prisma/schema.postgresql.prisma
  • backend/prisma/schema.prisma
  • backend/prisma/schema.sqlite.prisma
  • backend/src/app.test.ts
  • backend/src/config/env.test.ts
  • backend/src/config/prisma.test.ts
  • backend/src/middleware/api-key.middleware.test.ts
  • backend/src/middleware/auth.middleware.test.ts
  • backend/src/middleware/error.middleware.test.ts
  • backend/src/modules/api-keys/api-key.routes.test.ts
  • backend/src/modules/audit-logs/audit-log.routes.test.ts
  • backend/src/modules/auth/auth.routes.test.ts
  • backend/src/modules/connected-accounts/connected-account.routes.test.ts
  • backend/src/modules/files/file.routes.test.ts
  • backend/src/modules/files/stream-file.test.ts
  • backend/src/modules/files/stream-google-file.test.ts
  • backend/src/modules/folders/folder.routes.test.ts
  • backend/src/modules/google/google.service.test.ts
  • backend/src/modules/invites/invite.routes.test.ts
  • backend/src/modules/provider-configs/provider-config.routes.test.ts
  • backend/src/modules/public/public.routes.test.ts
  • backend/src/modules/s3/s3.service.test.ts
  • backend/src/modules/storage/storage.routes.test.ts
  • backend/src/modules/system/system.routes.test.ts
  • backend/src/modules/uploads/upload.routes.test.ts
  • backend/src/modules/uploads/upload.routes.ts
  • backend/src/test/setup.ts
  • backend/src/utils/audit.test.ts
  • backend/src/utils/audit.ts
  • backend/src/utils/crypto.test.ts
  • backend/src/utils/crypto.ts
  • backend/src/utils/jwt.test.ts
  • backend/src/utils/password.test.ts
  • backend/vitest.config.ts
  • docker-compose.yml
  • frontend/Dockerfile
  • frontend/nginx.conf

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

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.

4 participants