Bug: Remove operation has value field in it

[peterdk]

I noticed that when generating a jsonpatch the remove operation in this library does include the value, while that should not be the case.

Expected

[
    { "op": "remove", "path": "/from" }
]

Actual:

[
    { "op": "remove", "path": "/from", "value": "a value" }
]

I am willing to create a PR to fix this, but not sure if this project is still active and would merge it. Let me know.

[peterdk]

https://datatracker.ietf.org/doc/html/rfc6902#section-4.2

[123abceng]

On my side, all the tests are passing, but the error is not being thrown:

src:

{ "name": "John" }
patch:
[ { "op": "remove", "path": "/nonexistent" } ]

[peterdk]

Oh hmm, weird. I checked the output in our http requests. I'll see if I can create a reproducer.

[ReidSync]

Thank you for your efforts 👍