Skip to content

Conversation

@nikgraf
Copy link
Collaborator

@nikgraf nikgraf commented Jan 19, 2026

No description provided.

Copy link
Contributor

Copilot AI left a comment

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 adds support for distinguishing between PERSONAL and DAO space types in the hypergraph package. It introduces a type field to the PublicSpace schema and adds filtering capability by space type.

Changes:

  • Added type field (either 'PERSONAL' or 'DAO') to PublicSpace schema and query results
  • Refactored query construction from three static parameterized queries to dynamic query building based on filters
  • Added spaceType filter option to FindManyPublicFilter allowing filtering by space type, optionally combined with existing member/editor filters

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

File Description
.changeset/add-space-type.md Documents the new type field and spaceType filter in the changeset
.claude/settings.local.json Adds additional allowed bash commands for development tooling
packages/hypergraph/src/space/find-many-public.ts Adds type field to schema, implements dynamic query building with spaceType filter support
packages/hypergraph/test/space/find-many-public.test.ts Updates tests to include type field and adds test case for DAO type parsing

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

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

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


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

@nikgraf nikgraf merged commit 5b22a3f into main Jan 20, 2026
12 checks passed
@nikgraf nikgraf deleted the nik/spaces branch January 20, 2026 08:29
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