Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat!: JSON RPC based typegraph #877

Open
wants to merge 38 commits into
base: main
Choose a base branch
from
Open

Conversation

luckasRanarison
Copy link
Contributor

@luckasRanarison luckasRanarison commented Oct 16, 2024

Tasks

Replace WASM in typegraph with a client/server architecture using JSON RPC through stdin/stdout.

  • Remove wit from typgraph-core
  • Codegen system for the SDK types
  • Rewrite the client SDKs
  • Write the JSON RPC server in the CLI
  • Update the build infrastructure
  • Fix broken tests

Migration notes

...

  • The change comes with new or modified tests
  • Hard-to-understand functions have explanatory comments
  • End-user documentation is updated to reflect the change

@luckasRanarison luckasRanarison marked this pull request as draft October 16, 2024 05:48
Natoandro
Natoandro previously approved these changes Oct 16, 2024
@luckasRanarison luckasRanarison marked this pull request as ready for review November 17, 2024 13:35
Copy link

codecov bot commented Nov 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.40%. Comparing base (bfffb07) to head (af25ea4).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #877      +/-   ##
==========================================
+ Coverage   77.69%   81.40%   +3.70%     
==========================================
  Files         149      127      -22     
  Lines       18509    16458    -2051     
  Branches     1794     1741      -53     
==========================================
- Hits        14381    13397     -984     
+ Misses       4105     3038    -1067     
  Partials       23       23              

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

@luckasRanarison luckasRanarison requested a review from a team November 17, 2024 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants