Implement JSON serialization interface

[BambOoxX]

Fixes #89

No addition for Julia VS Code extension.
Has to be non-breaking, otherwise this is a bug.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (main@5b4c4d5). Learn more about missing BASE report.

Additional details and impacted files

@@           Coverage Diff           @@
##             main      #92   +/-   ##
=======================================
  Coverage        ?   90.59%           
=======================================
  Files           ?        5           
  Lines           ?      287           
  Branches        ?        0           
=======================================
  Hits            ?      260           
  Misses          ?       27           
  Partials        ?        0           

Flag	Coverage Δ
unittests	90.59% <100.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[davidanthoff]

I really like this! Could you maybe add some tests? But I will certainly merge this.

[BambOoxX]

I thought about additional tests but I faced 2 problems

• I am not familiar with how test are implemented in JSONRPC and I could not really perform the tests locally.
• In a sense, I only implemented the JSON interface, with a default setting that should behave just like before. Consequently I figured either JSON tests or JSONRPC tests as they currently are should find any problem that may arise. I can add more tests, I just do not know what they might be.

[BambOoxX]

@davidanthoff Any ideas as to what tests could be added on your side ? I would gladly close this ASAP.

[davidanthoff]

I just added tests to this PR here.

I also changed the implementation slightly, I think the original version essentially had some type piracy issues. Can you take a look over this version, whether that works for you? If tests pass, I'll merge.

[BambOoxX]

@davidanthoff In a sense there was a bit of type piracy, in the sense that I defined

Base.print(io::IO, serialization_obj::Tuple{JSON.Serialization,Any})

instead of a

Base.print(io::IO, serialization_obj::Tuple{JSONRPCEndpoint,Any})

i'm fine with your implementation if you are.

Do you plan to move to JSON3 at some point ?

Also, do you plan to release a version with this ?