@gjreda what do you think of removing the top level sidecar folder (well, keep it to the server cmd)? IMO That's now more of an implementation detail.

@cguedes I'd like to keep it (or rename it) in order to follow a fairly standard python project layout. sidecar still feels like a fairly apt name (as does backend or something similar).

@gjreda two notes before the review:

• we need to update the project.json script for the yarn web:api
• I would like (need) the API url to start with a /api. This is useful to create a (rewrite/proxy) rule when running the client in dev mode. Would also be useful if we serve the client app (html, css, ...) via the same backend server.

Codecov Report

Merging #495 (8f96715) into main (5ebffb3) will increase coverage by 0.16%.
The diff coverage is 86.77%.

@@            Coverage Diff             @@
##             main     #495      +/-   ##
==========================================
+ Coverage   81.52%   81.68%   +0.16%     
==========================================
  Files         202      213      +11     
  Lines       12089    12109      +20     
  Branches     1181     1181              
==========================================
+ Hits         9855     9891      +36     
+ Misses       2220     2204      -16     
  Partials       14       14              

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@cguedes I think this is good to go now. The codegen schemas look ok to me, but can you verify on your end as well? There aren't any tests for that code, so it was hard to verify that the changes were ok.

I'll write some tests for generate_schema.py in a follow-up PR.