Make the generated code compatible with nodejs natives modules

Cargo.lock

@@ -561,7 +561,7 @@ checksum = "49f1f14873335454500d59611f1cf4a4b0f786f9ac11f4312a78e4cf2566695b"
 
 [[package]]
 name = "jrpc-cli"
-version = "0.1.2"
+version = "0.1.5"
 dependencies = [
  "anyhow",
  "clap",
@@ -607,7 +607,7 @@ checksum = "561d97a539a36e26a9a5fad1ea11a3039a67714694aaa379433e580854bc3dc5"
 
 [[package]]
 name = "libjrpc"
-version = "0.1.0"
+version = "0.1.5"
 dependencies = [
  "anyhow",
  "lazy_static",

Cargo.toml

@@ -1,7 +1,7 @@
 [package]
 edition = "2021"
 name = "jrpc-cli"
-version = "0.1.2"
+version = "0.1.5"
 
 [workspace]
 resolver = "2"

libjrpc/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "libjrpc"
-version = "0.1.0"
+version = "0.1.5"
 edition = "2021"
 
 [features]

libjrpc/src/compile.rs

@@ -88,6 +88,7 @@ impl CompileContext {
     pub fn write_file(&self, filename: &str, content: &str) -> Result<()> {
         let res_path = self.output_folder.clone().join(filename);
         let res_dir = res_path.parent().context("Path has no parent!")?;
+        log::debug!("Writing to file {:?}", res_path);
         std::fs::create_dir_all(res_dir)?;
         std::fs::write(res_path, content)?;
         Ok(())

libjrpc/src/targets/mod.rs

@@ -4,6 +4,7 @@ use std::{
 };
 
 use anyhow::Result;
+use log::debug;
 
 use crate::{
     compile::{Compile, CompileContext},
@@ -51,6 +52,7 @@ pub fn compile<T: Compile>(ir: IR, output: &str) -> Result<()> {
     for step in ir.steps.iter() {
         match step {
             crate::ir::Step::Type(definition) => {
+                debug!("Generating type {}", definition.name);
                 match compiler.generate_type(&mut ctx, &definition) {
                     Ok(_) => (),
                     Err(err) => {
@@ -59,6 +61,7 @@ pub fn compile<T: Compile>(ir: IR, output: &str) -> Result<()> {
                 }
             }
             crate::ir::Step::Enum(definition) => {
+                debug!("Generating enum {}", definition.name);
                 match compiler.generate_enum(&mut ctx, &definition) {
                     Ok(_) => (),
                     Err(err) => {
@@ -67,6 +70,7 @@ pub fn compile<T: Compile>(ir: IR, output: &str) -> Result<()> {
                 }
             }
             crate::ir::Step::Service(definition) => {
+                debug!("Generating service {}", definition.name);
                 match compiler.generate_service(&mut ctx, &definition) {
                     Ok(_) => (),
                     Err(err) => {

libjrpc/src/targets/rust.rs

@@ -463,7 +463,17 @@ impl Compile for RustCompiler {
 
         self.add_dependencies(&mut f, &definition.depends)?;
 
-        f.a0("#[derive(Clone, Debug, Serialize, Deserialize)]");
+        let only_optional = definition
+            .fields
+            .iter()
+            .find(|f| !f.typ.is_optional())
+            .is_none();
+
+        let derive_default_none = if only_optional { ", Default" } else { "" };
+        f.a0(format!(
+            "#[derive(Clone, Debug, Serialize, Deserialize{})]",
+            derive_default_none
+        ));
         f.a0(format!("pub struct {} {{", definition.name));
         for field in definition.fields.iter() {
             f.a(1, "#[allow(non_snake_case)]");

libjrpc/src/targets/typescript.rs

@@ -435,12 +435,8 @@ import {{ VerificationError }} from \"./ts_base{esm}\";
 }
 
 impl<F: Flavour> Compile for TypeScriptCompiler<F> {
-    fn new(options: &BTreeMap<String, String>) -> Result<Self> {
-        let flavour = options
-            .get("flavour")
-            .cloned()
-            .unwrap_or_else(|| "node".to_string());
-        info!("TypeScript target initialized with flavour: {}", flavour);
+    fn new(_options: &BTreeMap<String, String>) -> Result<Self> {
+        info!("TypeScript target initialized with flavour: {}", F::name());
         Ok(TypeScriptCompiler {
             flavour: std::marker::PhantomData,
         })

libjrpc/templates/TypeScript/ts_service_client.ts

@@ -63,7 +63,10 @@ export class ServiceProvider {
             let resListener = this.requests.get(msg.id);
             if (!resListener) return; // Ignore wrong responses
             if (msg.error) {
-        if (msg.error.data && msg.error.data.$ == "verification_error") {
+                if (
+                    msg.error.data &&
+                    msg.error.data.$ == "verification_error"
+                ) {
                     resListener.err(
                         new VerificationError(
                             msg.error.data.type,
@@ -112,14 +115,26 @@ export const getRandomBytes = (
               return function (n: number) {
                   var a = new Uint8Array(n);
                   for (var i = 0; i < n; i += QUOTA) {
-            crypto.getRandomValues(a.subarray(i, i + Math.min(n - i, QUOTA)));
+                      crypto.getRandomValues(
+                          a.subarray(i, i + Math.min(n - i, QUOTA)),
+                      );
                   }
                   return a;
               };
           }
         : function () {
               // Node
+              if (typeof require !== "undefined") {
                   return require("crypto").randomBytes;
+              } else {
+                  return (n: number) => {
+                      let a = new Uint8Array(n);
+                      for (let i = 0; i < n; i++) {
+                          a[i] = Math.floor(Math.random() * 256);
+                      }
+                      return a;
+                  };
+              }
           }
 )() as (cnt: number) => Uint8Array;
 

src/main.rs

@@ -4,7 +4,7 @@ use libjrpc::{
     targets::{
         csharp::CSharpCompiler,
         rust::RustCompiler,
-        typescript::{Node, TypeScriptCompiler},
+        typescript::{Node, TypeScriptCompiler, ESM},
     },
     FileProcessor,
 };
@@ -66,7 +66,7 @@ pub fn main() -> Result<()> {
                         libjrpc::targets::compile::<TypeScriptCompiler<Node>>(ir, output_dir)?
                     }
                     "ts-esm" => {
-                        libjrpc::targets::compile::<TypeScriptCompiler<Node>>(ir, output_dir)?
+                        libjrpc::targets::compile::<TypeScriptCompiler<ESM>>(ir, output_dir)?
                     }
                     "csharp" => libjrpc::targets::compile::<CSharpCompiler>(ir, output_dir)?,
                     _ => {

src/test.rs

@@ -6,12 +6,12 @@ use std::{
 
 #[test]
 fn compare_tools() {
-    let targets = vec!["rust"];
+    let targets = vec!["js-esm"];
     for target in targets {
         std::fs::remove_dir_all("./tests").unwrap();
         std::fs::create_dir_all("./tests").unwrap();
 
-        Command::new("cargo")
+        let result1 = Command::new("cargo")
             .arg("run")
             .arg("--")
             .arg("compile")
@@ -26,7 +26,11 @@ fn compare_tools() {
             .wait()
             .unwrap();
 
-        Command::new("node")
+        if !result1.success() {
+            panic!("Failed to generate Rust code");
+        }
+
+        let result2 = Command::new("node")
             .arg("JsonRPC/lib/jrpc.js")
             .arg("compile")
             .arg("--verbose")
@@ -40,6 +44,10 @@ fn compare_tools() {
             .wait()
             .unwrap();
 
+        if !result2.success() {
+            panic!("Failed to generate JavaScript code");
+        }
+
         let rust_files = walkdir::WalkDir::new("tests/rust")
             .into_iter()
             .map(|e| e.unwrap())