Skip to content

Handle missing groups in subgraph client#1085

Open
samsamtrum wants to merge 1 commit into
semaphore-protocol:mainfrom
samsamtrum:fix-subgraph-missing-group
Open

Handle missing groups in subgraph client#1085
samsamtrum wants to merge 1 commit into
semaphore-protocol:mainfrom
samsamtrum:fix-subgraph-missing-group

Conversation

@samsamtrum
Copy link
Copy Markdown

Summary

  • throw a clear Group '<id>' not found error when the subgraph returns no matching groups
  • avoid indexing groups[0] before confirming the query returned a group
  • add a regression test for the missing-group path

Testing

  • corepack yarn workspace @semaphore-protocol/utils build
  • corepack yarn test:library data --runInBand

Fixes a runtime TypeError in SemaphoreSubgraph.getGroup when the requested group does not exist.

@samsamtrum samsamtrum requested a review from a team as a code owner May 22, 2026 12:36
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.

1 participant