NIP-7A: comment-based patches - #2438
Conversation
|
Implementation skill https://github.com/gsovereignty/nip9a |
|
I don't think this is really human readable. Could you choose a different kind at least? Also, nip 9a is already used by #2194 (and several relays signal for it) |
|
Don't overload kind 1111. |
Sure, it isn't fully human readable. Do you think it is possible to make it more human-readable or do you just want to complain?
Using a different kinds defeats the entire purpose of it, as you should know as the author of the proposal linked above. So no, that can't be the solution.
Changed to 7A. |
|
This DIFF is complete shit for a regular user. Displaying as text in replies is absolutely horrendous. |
|
I still think annotations are better #1091. But I've not implemented it so I obviously don't have strong opinions. |
|
I'm not a fan of redundant information as it opens up for inconsistency. Why not just: |
You mean the length of the added text? It's because some characters are evil, like newlines. But I'm ok with this and then we just forbid adding evil characters ever if more people prefer this way. |
Got it. I do think newlines are desirable for edits, but can't they just be escaped |
|
What if we did something like this: Would it be more naturally readable? |
You guys have a sense of humor. |
This is inspired by #1091.
The core idea is that this is meant for small quality-of-life improvements, like fixing a typo or adding a missing word, it's not meant to fully replace the original content.
Because of that it is backwards-compatible (i.e. the original event still stands on its own) and these patches at best add some new things, which can be read anyway by any existing client as these patches will just show up as normal comments.
NIP text: https://github.com/nostr-protocol/nips/blob/8fec4a5f8c477a3454eeba411d956919c6a1cbfb/7A.md
The patch format is invented by me because I couldn't find anything that produced reasonably-human-readable output and was simple enough. It works pretty well in my tests, including some edge cases. Here's a stupid playground: https://claude.ai/public/artifacts/fb4718ff-ea13-45eb-8460-f55e9a7e225c