Skip to content

Backend: Remove unused CDE related code - #84

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

Backend: Remove unused CDE related code#84
AustinSMueller wants to merge 1 commit into
prisma-to-mongoose-migrationfrom
remove-cde

Conversation

@AustinSMueller

Copy link
Copy Markdown
Contributor

Backend: Remove unused CDE related code

@relativeci

relativeci Bot commented Jul 23, 2026

Copy link
Copy Markdown

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

⚠️ Baseline job is missing

Warning

Bundle contains 54 duplicate packages – View duplicate packages

Bundle metrics  no changes
                 Current
#274
     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
#274
     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 remove-cdeProject 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 removes unused backend support for CDE (Common Data Elements), including the GraphQL query surface area, service/DAO implementation, Prisma model, and their associated unit tests.

Changes:

  • Removed retrieveCDEs GraphQL query and its CDE/CDEInput schema types.
  • Deleted the CDE service (CDEService) and DAO (dao/cde.js) plus corresponding Jest test suites.
  • Removed the CDE Prisma model and the MODEL_NAME.CDE constant entry.

Reviewed changes

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

Show a summary per file
File Description
apps/backend/test/services/CDEService.getCDEs.test.js Removed unit tests for the deleted CDE service method.
apps/backend/test/dao/dao.cde.test.js Removed unit tests for the deleted CDE DAO method.
apps/backend/services/CDEService.js Deleted unused CDE service implementation.
apps/backend/routers/graphql-router.js Removed CDE service wiring and resolver (retrieveCDEs) from GraphQL root.
apps/backend/resources/graphql/crdc-datahub.graphql Removed CDE/CDEInput types and retrieveCDEs query from the schema.
apps/backend/prisma/schema.prisma Removed the CDE Prisma model definition.
apps/backend/dao/cde.js Deleted unused CDE DAO implementation.
apps/backend/constants/db-constants.js Removed MODEL_NAME.CDE constant entry.

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

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