Skip to content

implement Space.findManyPublic - #563

Merged
nikgraf merged 8 commits into
mainfrom
nik/find-spaces
Dec 4, 2025
Merged

implement Space.findManyPublic#563
nikgraf merged 8 commits into
mainfrom
nik/find-spaces

Conversation

@nikgraf

@nikgraf nikgraf commented Dec 3, 2025

Copy link
Copy Markdown
Collaborator

No description provided.

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 implements Space.findManyPublic functionality to fetch and filter public spaces from the Graph testnet. It adds both a low-level SDK function and a React hook for convenient integration.

  • Adds Space.findManyPublic() function with optional filtering by member or editor account address
  • Introduces usePublicSpaces() React hook with React Query integration
  • Updates documentation with usage examples for both the SDK function and React hook

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
packages/hypergraph/src/space/find-many-public.ts Core implementation with GraphQL queries, schema validation, and filtering logic
packages/hypergraph/src/space/index.ts Exports the new Space module
packages/hypergraph/test/space/find-many-public.test.ts Unit tests for the parsing logic with valid, missing, and invalid data scenarios
packages/hypergraph-react/src/hooks/usePublicSpaces.ts React Query hook wrapper for the SDK function
packages/hypergraph/src/index.ts Adds Space namespace export
packages/hypergraph-react/src/index.ts Exports the new usePublicSpaces hook
packages/hypergraph/package.json Adds package export path for the space module
docs/docs/query-public-data.md Documentation for the new functionality with code examples
apps/events/src/routes/podcasts.lazy.tsx Example migration from useEntities to usePublicSpaces
.changeset/loud-houses-design.md Changeset describing the new feature

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

Comment thread docs/docs/query-public-data.md Outdated
Comment on lines +6 to +7
Add Space.findManyPublic plus the usePublicSpaces hook so apps can fetch and render public spaces (with invalid entries surfaced) via one shared SDK call

No newline at end of file

Copilot AI Dec 3, 2025

Copy link

Choose a reason for hiding this comment

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

This line contains trailing whitespace. Consider removing it for consistency with standard formatting practices.

Suggested change
Add Space.findManyPublic plus the usePublicSpaces hook so apps can fetch and render public spaces (with invalid entries surfaced) via one shared SDK call
Add Space.findManyPublic plus the usePublicSpaces hook so apps can fetch and render public spaces (with invalid entries surfaced) via one shared SDK call

Copilot uses AI. Check for mistakes.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@nikgraf
nikgraf merged commit 4642397 into main Dec 4, 2025
6 checks passed
@nikgraf
nikgraf deleted the nik/find-spaces branch December 4, 2025 08: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.

2 participants