Improvements to querying documents for sync filters about earthstar HOT 4 OPEN

Yeah, this is awkward. I think the IStorage query functions should accept an array of queries which would work in the same way as sync queries:

workspace.documents([
    {pathPrefix: "/about/"},
    {pathPrefix: "/wiki/"},
])

It would also still accept a single query object, or no query at all:

documents(q?: Query | Query[]) => Document[]

Matching against one query object, a document has to match EVERY property in the query. E.g. every property in a query object narrows down the search.

Overall, it would return documents that match ANY of the queries in the array. E.g. each additional query in the array broadens the search.

Would you be open to changing the GraphQL query to work this way instead of having arrays within one query object? I think this combination of narrowing and broadening will allow us to express complex queries.

from earthstar.

See also #47 (More comprehensive query options) for ideas about revamping query objects

from earthstar.

@cinnamon-bun I like this, and I'm definitely opening to changing how filters work in earthstar-graphql. Sure it's a little more complex, but I'm not too worried as GraphQL will be able to help people understand how to make valid filters.

from earthstar.

I've gotten stuck: what to do with the limit field? It makes sense to limit each query inside its own object, but what if you want an overall limit?

Maybe you just can't do that.

from earthstar.