Skip to content

Move PendingPVs DAO to Mongoose - #85

Open
AustinSMueller wants to merge 1 commit into
prisma-to-mongoose-migrationfrom
mongoose-pendingpvs
Open

Move PendingPVs DAO to Mongoose#85
AustinSMueller wants to merge 1 commit into
prisma-to-mongoose-migrationfrom
mongoose-pendingpvs

Conversation

@AustinSMueller

Copy link
Copy Markdown
Contributor

No description provided.

@relativeci

relativeci Bot commented Jul 23, 2026

Copy link
Copy Markdown

#276 Bundle Size — 12.87MiB (+100%).

⚠️ Baseline job is missing

Warning

Bundle contains 54 duplicate packages – View duplicate packages

Bundle metrics  no changes
                 Current
#276
     Baseline
No change  Initial JS 877.16KiB -
No change  Initial CSS 0B -
No change  Cache Invalidation 0% -
No change  Chunks 74 -
No change  Assets 117 -
No change  Modules 9159 -
No change  Duplicate Modules 0 -
No change  Duplicate Code 0% -
No change  Packages 320 -
No change  Duplicate Packages 53 -
Bundle size by type  no changes
                 Current
#276
     Baseline
No change  JS 10.35MiB (+100%) -
No change  IMG 1.72MiB (+100%) -
No change  Fonts 805.07KiB (+100%) -
No change  CSS 18KiB (+100%) -

Bundle analysis reportBranch mongoose-pendingpvsProject dashboard


Generated by RelativeCIDocumentationReport issue

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR migrates the PendingPVs data-access layer from Prisma to the existing Mongoose-based DAO infrastructure, aligning pending PV operations with the ongoing Prisma→DocumentDB/Mongoose transition in the backend.

Changes:

  • Added a new Mongoose model for the pendingPvs collection.
  • Refactored PendingPVDAO to extend MongooseGenericDAO and use the new Mongoose model for findBySubmissionID and insertOne.
  • Added Jest unit tests covering DAO behavior and ID mapping, and removed an unused DB constant import from the GraphQL router.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
apps/backend/test/dao/dao.pendingPV.test.js Adds unit tests validating the new Mongoose-backed PendingPV DAO behavior (querying, inserting, and id/_id mapping).
apps/backend/routers/graphql-router.js Removes unused PENDING_PVS_COLLECTION import now that pending PVs are not wired via a MongoDBCollection here.
apps/backend/mongoose/models/pending-pv.js Introduces the Mongoose schema/model for pending PV documents (string _id, submissionID index, createdAt/updatedAt fields).
apps/backend/dao/pendingPV.js Migrates PendingPV DAO implementation from Prisma to MongooseGenericDAO using the new PendingPV Mongoose model.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@AustinSMueller
AustinSMueller marked this pull request as ready for review July 23, 2026 21:22
@AustinSMueller
AustinSMueller requested a review from n2iw July 23, 2026 21:22
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.

2 participants