feat(jsruntime): closures (#236)

* refactor(jsruntime): refactoring

* docs(jsruntime): update the link

* style(jsparser): format comments

* docs(jsruntime): documentation

* refactor(jsruntime): remove unnecessary operations

* refactor(jsruntime): remove unused field

* refactor(jsruntime): FunctionId

* feat(jstb): add the `parse` subcommand

* refactor(jsruntime): move `references` from `Analyzer` to `FunctionContext`

* style(jsruntime): format

* refactor(jsruntime): refactoring

* refactor(jsruntime): compute locations of variables at compile time

* feat(jsruntime): closures

* fix(jsruntime): the argument order was wrong

* fix(jsruntime): capture arguments

* style(jsruntime): format

* fix(jsruntime): add missing case clauses

* test(jsruntime): add test cases

* feat(jsruntime): implement std::fmt::Debug for Capture

* refactor(jsruntime): refactoring

* refactor(jsruntime): rename

* fix(jsruntime): workaround for #234

the test scripts should be replaced with original ones after #234 is fixed.

* feat(jstb): add options

* docs(jsruntime): add implementation notes

* refactor(jsruntime): remove `Locator::offset`

* chore(jsruntime): add TODO marks

* feat(jsruntime): always create a closure for a function

even if the function has no free variable.

* refactor(jsruntime): Remove ValueKind::Function

* refactor(jsruntime): rename

* refactor(jsruntime): refactoring

* refactor(jsruntime): remove argc and argv from FunctionScope

* refactor(jsruntime): refactoring

* refactor(jsruntime): rename

* chore(jsruntime): add TODO

* refactor(jsruntime): rename

* fix(jsruntime): eliminate warnings in the release build

* chore(tools): update the example

* refactor(jsruntime): reuse the Variable type as the Value type

* refactor(jsruntime): remove FunctionScope

* refactor(jsruntime): add num_formal_parameters and num_locals

Author: masnagam

Cargo.lock

@@ -127,7 +127,7 @@ update-devcontainer:
 
 .PHONY: doc
 doc:
-	cargo doc --workspace --all-features
+	cargo doc --workspace --all-features --document-private-items
 
 .PHONY: format
 format: format-rust format-cxx format-js

Makefile

@@ -16,9 +16,12 @@ struct CommandLine {
 /// A testbed for the jsruntime module.
 #[derive(clap::Subcommand)]
 enum Command {
-    /// Compile a JavaScript program and print the compiled module.
+    /// Parses a JavaScript program and print the result.
+    Parse(Parse),
+
+    /// Compiles a JavaScript program and print the compiled module.
     ///
-    /// lli cannot interpret the module directly.  Because it includes unresolved symbols for the
+    /// `lli` cannot interpret the module directly.  Because it includes unresolved symbols for the
     /// runtime function calls.  At this point, there is no command-line option to output anything
     /// containing the runtime functions which can link to the module.
     Compile(Compile),
@@ -28,8 +31,16 @@ enum Command {
 }
 
 #[derive(clap::Args)]
-struct Run {
-    /// The source file of the JavaScript program to run.
+struct Parse {
+    /// Prints the functions.
+    #[arg(long)]
+    print_functions: bool,
+
+    /// Prints the scope tree.
+    #[arg(long)]
+    print_scope_tree: bool,
+
+    /// The source file of the JavaScript program to parse.
     ///
     /// Reads the source text from STDIN if this argument is not specified.
     #[arg()]
@@ -42,6 +53,19 @@ struct Compile {
     #[arg(long)]
     no_optimize: bool,
 
+    /// The source file of the JavaScript program to compile.
+    ///
+    /// Reads the source text from STDIN if this argument is not specified.
+    #[arg()]
+    source: Option<PathBuf>,
+}
+
+#[derive(clap::Args)]
+struct Run {
+    /// Disable optimization.
+    #[arg(long)]
+    no_optimize: bool,
+
     /// The source file of the JavaScript program to run.
     ///
     /// Reads the source text from STDIN if this argument is not specified.
@@ -55,23 +79,31 @@ fn main() -> Result<()> {
     let cl = CommandLine::parse();
     let mut runtime = Runtime::new().with_host_function("print", print);
     match cl.command {
+        Command::Parse(args) => {
+            let source = read_source(args.source.as_ref())?;
+            let program = runtime.parse_script(&source)?;
+            if args.print_functions {
+                println!("### functions");
+                program.print_functions("");
+            }
+            if args.print_scope_tree {
+                println!("### scope tree");
+                program.print_scope_tree("");
+            }
+            Ok(())
+        }
         Command::Compile(args) => {
             let source = read_source(args.source.as_ref())?;
-            let module = match runtime.compile_script(&source, !args.no_optimize) {
-                Some(module) => module,
-                None => anyhow::bail!("Failed to parse"),
-            };
+            let program = runtime.parse_script(&source)?;
+            let module = runtime.compile(&program, !args.no_optimize)?;
             module.print(false); // to STDOUT
             Ok(())
         }
         Command::Run(args) => {
             let source = read_source(args.source.as_ref())?;
-            // Always perform optimization.
-            let module = match runtime.compile_script(&source, true) {
-                Some(module) => module,
-                None => anyhow::bail!("Failed to parse"),
-            };
-            match runtime.eval(module) {
+            let program = runtime.parse_script(&source)?;
+            let module = runtime.compile(&program, !args.no_optimize)?;
+            match runtime.evaluate(module) {
                 Ok(_) => Ok(()),
                 Err(v) => anyhow::bail!("Uncaught {v:?}"),
             }