Jesse Wright

Fair enough - in general I would recommend selecting those methods defined in the DatasetCore definition https://github.com/rdfjs/types/blob/183bda795f57a9464ddf95deac45a0c4a48879cf/dataset.d.ts#L7 wherever possible as this means your app can stay compatible with other dataset...

> Mainly just to leave the option of changing its behaviour in the future In light of the fact that it is mentioned in the documentation, this is a moot...

I wonder if it is possible to write an encryption/decryption component that can be used both here and in the [Comunica Engine](https://github.com/comunica/comunica) so both options are implemented at once. That...

> some work might already exist here This seems to be the latest work in the space http://semantic-web-journal.net/system/files/swj1895.pdf followed by https://link.springer.com/chapter/10.1007/11574620_24

I just wanted to jot a few top-of-mind notes around a couple of modifications that should be made down the line to improve the features & performance of reasoning. I'm...

> I would love to see this code in N3. As would I, however, I am currently undecided on whether the API should be as it is at present, or,...

> @jeswr thanks this works. I see that the `reason` method inline updates the N3Store. Would it be possible to also support that the reason method returns a new N3Store...

More importantly - to enable much faster `SELECT DISTINCT` queries and reasoning over data sources it would be useful to either have as part of `matchPattern`; or even `match` itself,...

Note that this case is covered by https://w3c.github.io/rdf-star/tests/turtle/syntax/#turtle-star-inside-2 in the RDF-star spec; so this will be resolved once all RDF-star spec tests are passing.

Ahh that distinction had slipped by me. Thanks for the clarification & I'll try to be more consistent from here on 😀