Closed
Conversation
* Introduced a new GraphQL API endpoint for flexible data querying * Enabled the GraphiQL playground interface in non-production environments
Added ClinVar schema and corresponding resolver to the GraphQL API, enabling ClinVar data querying through the GraphQL endpoint.
Added Gene schema and resolver to the GraphQL API, enabling gene-level queries.
* Added DIOPT-related GraphQL support: * DioptOrtholog * DioptDomain * DioptAlignment * Implemented resolvers to enable querying these DIOPT data structures.
Add comprehensive Pharos target, drug, and ligand GraphQL schemas and resolver
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR adds comprehensive GraphQL API support to the MARRVEL server, providing a modern query interface for genomic data alongside the existing REST API. The changes include GraphQL schemas, resolvers, and dependencies for multiple data sources including Clinvar, genes, DIOPT, phenotype ontology, Pharos, STRING, and dbNSFP.
Key changes:
- Added GraphQL dependencies:
graphql,graphql-http, andrurufor GraphiQL interface - Implemented GraphQL schemas and resolvers for 7 data sources
- Configured GraphQL endpoint at
/graphqland development GraphiQL interface at/graphiql
Reviewed Changes
Copilot reviewed 20 out of 21 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| server/package.json | Added graphql, graphql-http, ruru dependencies; upgraded express to ^4.21.2 |
| server/package-lock.json | Lockfile updates for new dependencies and transitive packages |
| server/graphql/index.js | Entry point and module documentation for GraphQL API |
| server/graphql/graphqlHandler.js | Main GraphQL handler combining schemas and resolvers |
| server/graphql/schemas/*.graphql | Type definitions for 7 data sources |
| server/graphql/resolvers/*.js | Resolver implementations for all queries |
| server/models/string-interactions.model.js | Mongoose model for STRING protein interactions |
| server/graphql/README.md | Comprehensive API documentation with examples |
| server/app.js | Registered GraphQL routes and GraphiQL interface |
Files not reviewed (1)
- server/package-lock.json: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.