UI for edit and run tests

.noindex

@@ -0,0 +1,11 @@
+ui/node_modules/**
+ui/dist/**
+ui/workdir/**
+ui/pkg/**
+ui/public/**
+wasm/target/**
+rec/target/**
+cmp/target/**
+parser/target/**
+mcp/target/**
+wasm/pkg/**

README.md

@@ -61,6 +61,18 @@ echo $?
 
 The exit code will be 0 if the test passed (outputs match) or 1 if there are differences.
 
+## Web-based UI
+
+CLT includes a web-based user interface for editing and running tests. The UI provides an intuitive interface for managing test files, with real-time pattern matching and diff comparison. To learn more about the UI, its features, and how to set it up, see the [UI documentation](ui/README.md).
+
+Key UI features include:
+- File tree explorer for navigating test files
+- Real-time comparison with pattern matching
+- Docker image configuration for test validation
+- GitHub authentication for access control
+
+We utilize bash to initiate an interactive environment when you record a test. It's important to note that we reset the environment to ensure maximum compatibility with various operating systems. As of now, there is no option to pass environment variables from outside into the test environment.
+
 ### Refining Tests
 
 After recording, you can refine your test to handle dynamic outputs:

bin/cross-build

@@ -1,17 +1,17 @@
 #!/usr/bin/env bash
 set -e
-
-for folder in rec cmp; do
-  cd $folder
+declare -A bin_map
+bin_map=([rec]="rec" [cmp]="cmp" [mcp]="clt-mcp")
+for folder in rec cmp mcp; do
   # https://github.com/joseluisq/rust-linux-darwin-builder
-  parser_src="$(pwd)/../parser"
-  docker run --network host --rm -v "${parser_src}:/root/parser" -v "$(pwd):/root/src" -w /root/src -it joseluisq/rust-linux-darwin-builder:1.84.1 bash -c "\
+  docker run --network host --rm -v "$(pwd):/root/src" -w /root/src -it joseluisq/rust-linux-darwin-builder:1.84.1 bash -c "\
+		cd $folder; \
     cargo clean; \
     cargo build --target=x86_64-unknown-linux-musl --release; \
     cargo build --target=aarch64-unknown-linux-musl --release; \
   "
-  cd ..
+	binary=${bin_map[$folder]}
   # Copy binaries
-  cp "$folder/target/x86_64-unknown-linux-musl/release/$folder" bin/x86_64/
-  cp "$folder/target/aarch64-unknown-linux-musl/release/$folder" bin/aarch64/
+  cp "$folder/target/x86_64-unknown-linux-musl/release/$binary" bin/x86_64/
+  cp "$folder/target/aarch64-unknown-linux-musl/release/$binary" bin/aarch64/
 done

cmp/src/lib.rs

@@ -102,7 +102,7 @@ impl PatternMatcher {
         let mut config: HashMap<String, String> = HashMap::new();
 
         let file_path = std::path::Path::new(&file_name);
-        let file = File::open(&file_path)?;
+        let file = File::open(file_path)?;
         let reader = BufReader::new(file);
 
         for line in reader.lines() {

cmp/src/main.rs

@@ -20,7 +20,6 @@ use std::env;
 use std::path::Path;
 use termcolor::{Color, ColorChoice, ColorSpec, StandardStream, WriteColor};
 use std::io::Write;
-use tempfile;
 
 // Import from lib
 pub use cmp::{PatternMatcher, MatchingPart};
@@ -118,7 +117,7 @@ fn main() {
 			},
 			parser::Statement::Output => {
 
-				let pos = file1_reader.seek(SeekFrom::Current(0)).unwrap();
+				let pos = file1_reader.stream_position().unwrap();
 
 				let mut line = String::new();
 				file1_reader.read_line(&mut line).unwrap();
@@ -203,24 +202,22 @@ fn main() {
 											print_diff(&mut stdout, l1, Diff::Minus);
 											print_diff(&mut stdout, l2, Diff::Plus);
 										}
-									} else if debug_mode {
+									} else {
 										writeln!(stdout, "{}", l1).unwrap();
 									}
 								},
 								_ => {},
 							}
 						}
-					} else {
-						if debug_mode {
-							// Show all lines when in debug mode
-							for line in &lines1 {
-								writeln!(stdout, "{}", line).unwrap();
-							}
-						} else {
-							// Just show OK when no differences and not in debug mode
-							writeln!(stdout, "OK").unwrap();
-						}
-					}				}
+					} else if debug_mode {
+     							// Show all lines when in debug mode
+     							for line in &lines1 {
+     								writeln!(stdout, "{}", line).unwrap();
+     							}
+     						} else {
+     							// Just show OK when no differences and not in debug mode
+     							writeln!(stdout, "OK").unwrap();
+     						}				}
 			}
 			_ => {
 				// For any other section we simply print the next line from either file.
@@ -241,7 +238,7 @@ fn move_cursor_to_line<R: BufRead + Seek>(reader: &mut R, command_prefix: &str)
 	let mut line = String::new();
 
 	loop {
-		let pos = reader.seek(SeekFrom::Current(0))?;
+		let pos = reader.stream_position()?;
 		let len = reader.read_line(&mut line)?;
 
 		if len == 0 {
@@ -261,7 +258,7 @@ fn move_cursor_to_line<R: BufRead + Seek>(reader: &mut R, command_prefix: &str)
 
 /// Peek the statement and return it from the given reader
 fn peek_statement<R: BufRead + Seek>(reader: &mut R) -> io::Result<Option<parser::Statement>> {
-	let pos = reader.seek(SeekFrom::Current(0))?;
+	let pos = reader.stream_position()?;
 	let mut line = String::new();
 	let len = reader.read_line(&mut line)?;
 	reader.seek(SeekFrom::Start(pos))?;
@@ -281,7 +278,7 @@ fn buffer_block<R: BufRead + Seek>(reader: &mut R) -> io::Result<Vec<String>> {
 	let mut parsed = false;
 
 	loop {
-		let pos = reader.seek(SeekFrom::Current(0))?;
+		let pos = reader.stream_position()?;
 		line.clear();
 		let len = reader.read_line(&mut line)?;
 		if len == 0 {
@@ -315,7 +312,7 @@ fn buffer_block<R: BufRead + Seek>(reader: &mut R) -> io::Result<Vec<String>> {
 ///
 fn reader_at_end<R: BufRead + Seek>(reader: &mut R) -> bool {
 	match reader.fill_buf() {
-		Ok(buf) if buf.is_empty() => true,
+		Ok([]) => true,
 		_ => false,
 	}
 }
@@ -428,7 +425,7 @@ fn block_has_differences(lines1: &[String], lines2: &[String], pattern_matcher:
 // A helper to skip non-Input/Output blocks in the reader.
 fn skip_non_command_blocks<R: BufRead + Seek>(reader: &mut R) -> io::Result<()> {
     loop {
-        let pos = reader.seek(SeekFrom::Current(0))?;
+        let pos = reader.stream_position()?;
         if let Some(stmt) = peek_statement(reader)? {
             if stmt == parser::Statement::Input || stmt == parser::Statement::Output {
                 // Found a valid command block, so rewind and exit.

lib/container.sh

@@ -15,7 +15,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-set -e
+# ! We are handling exit codes so we cannot use set -e here
 
 # Detect proper path to the binary to run
 ARCH=$(arch)
@@ -27,11 +27,13 @@ container_exec() {
 	directory=${3:-tests}
 	interactive=${4:-}
 	if [ ! -d "$directory" ]; then
-		>&2 echo "Directory with tests does not exist: $directory" && exit 1
+		>&2 echo "Directory with tests does not exist: $directory"
+		return 1
 	fi
 
 	if [ -z "$image" ] || [ -z "$command" ]; then
-		>&2 echo 'Usage: container_exec "image" "command"' && exit 1
+		>&2 echo 'Usage: container_exec "image" "command"'
+		return 1
 	fi
 
 	# Merge base of patterns
@@ -63,14 +65,21 @@ container_exec() {
 
 	if [ -n "$interactive" ]; then
 		eval "$process"
+		exit_code=$?
 	else
 		eval "$process" & pid=$!
 
 		trap "kill -s INT '$pid'; exit 130" SIGINT
 		trap "kill -s TERM '$pid'; exit 143" SIGTERM
 		wait "$pid"
+		exit_code=$?
 
 		trap - SIGINT SIGTERM
-		wait "$pid"
 	fi
+
+	# Clean up temp file
+	rm -f "$temp_file"
+
+	return $exit_code
 }
+