Spells 2#2254
Open
alexgleason wants to merge 3 commits intonostr-protocol:masterfrom
Open
Conversation
Make $me the only runtime variable. All other variables (like $contacts) must now be defined within the spell using a new 'variable' tag that specifies a Nostr filter to execute and a field to extract from results.
Open
Remove restriction limiting variables to only authors and tag filter values. Use REQ filter field names (kinds, authors, ids) for variable filter keys, and allow repeating the same key to specify multiple values.
|
I think this is the dream right here. |
Contributor
|
I have an alternative proposal since this opens a can of worms. Explained my design and reasoning in the original PR. |
Contributor
|
This is interesting too. Having more than two steps may be useful and clients could cap the max number of steps if it is an untrusted spell. This could be a high level way of representing a series of queries. I wonder if improving it further would make it too complex, like the variable could have the whole events and be later accessed as Maybe at the end both spells1 and 2 aren't that useful/powerful for sharing complex series of queries. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
A stab at #2244 with user-defined variables (not limited to just $contacts)