aws / aws/graph-explorer

Clean up summary API usage

Open
#1,287 0 comments 0 reactions 0 assignees View on GitHub
reliability schema
Dominant language
TypeScript
Stars
481
Forks
108
Avg merge
8d 9h
Merged PRs (30d)
7

Description

While investigating a user's schema sync issue I noticed some code that could use some cleaning up.

* Calls with `mode=detailed` but doesn't use extra fields
* Does not validate response structure
* Performs unchecked property access on response

https://github.com/aws/graph-explorer/blob/b09962331c82802ea35593d6f381cc26f6e2cb01/packages/graph-explorer/src/connector/gremlin/gremlinExplorer.ts#L47-L69

Contributor guide

Open the contributing guide

Research direction

Open packages/graph-explorer/src/connector/gremlin/gremlinExplorer.ts at lines 47-69 and trace the summary API call and response handling. Review how the response is used, then make the cleanup address the detailed-mode request, response validation, and unchecked property access; done means these three concerns are resolved without changing the intended summary behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
api
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.