style(Rest): remove spaces from shell redirection operators

Continue POSIX compliance by removing extraneous spaces from shell redirection operators across Bench.sh, prepublishOnly.sh, and run_tests.sh. Change `2> /dev/null` to `2>/dev/null` and `> /dev/null` to `>/dev/null` for consistent POSIX-style redirection.

Also continue JavaScript quote style standardization in bin/Rest.js by ensuring all double quotes are converted to single quotes.

Author: NikolaRHristov

Bench.sh

@@ -47,12 +47,12 @@ echo ""
 
 # Function to count TypeScript files
 count_typescript_files() {
-	find "$1" -name "*.ts" -type f 2> /dev/null \
-		| grep -v "node_modules" \
-		| grep -v "Target" \
-		| grep -v "/test" \
-		| grep -v "\.d\.ts$" \
-		| wc -l
+	find "$1" -name "*.ts" -type f 2>/dev/null |
+		grep -v "node_modules" |
+		grep -v "Target" |
+		grep -v "/test" |
+		grep -v "\.d\.ts$" |
+		wc -l
 }
 
 # Function to compile with Rest
@@ -97,11 +97,11 @@ benchmark_rest() {
 	printf "  Total time: %d seconds\n" "$elapsed"
 
 	if [ -d "$output_dir" ]; then
-		compiled_count=$(find "$output_dir" -name "*.js" -type f 2> /dev/null | grep -v "map" | wc -l)
+		compiled_count=$(find "$output_dir" -name "*.js" -type f 2>/dev/null | grep -v "map" | wc -l)
 		echo "  Output files: $compiled_count"
 		echo "  Output size: $(du -sh "$output_dir" | cut -f1)"
 
-		printf "%s,%s,%s,%s\n" "$file_count" "$compiled_count" "$exit_code" "$elapsed" > "$output_dir/benchmark_results.csv"
+		printf "%s,%s,%s,%s\n" "$file_count" "$compiled_count" "$exit_code" "$elapsed" >"$output_dir/benchmark_results.csv"
 	else
 		echo "  No output directory created"
 	fi
@@ -126,8 +126,8 @@ compare_outputs() {
 	echo "Finding VSCode JavaScript files..."
 	echo "Finding Rest JavaScript files..."
 
-	total_vscode=$(find "$vscode_dir" -name "*.js" -type f 2> /dev/null | grep -v "map" | grep -v "\.d\.js$" | wc -l)
-	total_rest=$(find "$rest_dir" -name "*.js" -type f 2> /dev/null | grep -v "map" | grep -v "\.d\.js$" | wc -l)
+	total_vscode=$(find "$vscode_dir" -name "*.js" -type f 2>/dev/null | grep -v "map" | grep -v "\.d\.js$" | wc -l)
+	total_rest=$(find "$rest_dir" -name "*.js" -type f 2>/dev/null | grep -v "map" | grep -v "\.d\.js$" | wc -l)
 
 	echo "VSCode: $total_vscode files, Rest: $total_rest files"
 
@@ -146,7 +146,7 @@ compare_outputs() {
 	mismatch_count=0
 	missing_count=0
 
-	find "$vscode_dir" -name "*.js" -type f 2> /dev/null | grep -v "map" | grep -v "\.d\.js$" | while IFS= read -r vscode_file; do
+	find "$vscode_dir" -name "*.js" -type f 2>/dev/null | grep -v "map" | grep -v "\.d\.js$" | while IFS= read -r vscode_file; do
 		rel="${vscode_file#$vscode_dir/}"
 		rest_file="$rest_dir/$rel"
 
@@ -158,8 +158,8 @@ compare_outputs() {
 				match_count=$((match_count + 1))
 			else
 				mismatch_count=$((mismatch_count + 1))
-				vscode_size=$(wc -c < "$vscode_file")
-				rest_size=$(wc -c < "$rest_file")
+				vscode_size=$(wc -c <"$vscode_file")
+				rest_size=$(wc -c <"$rest_file")
 				echo "  DIFF: $rel (VSCode: ${vscode_size}B, Rest: ${rest_size}B)"
 			fi
 		fi
@@ -182,7 +182,7 @@ else
 	echo "SKIPPING: VSCode source not found or empty"
 fi
 
-if [ -d "$VSCode_OUT_BUILD_DIR" ] && [ "$(find "$VSCode_OUT_BUILD_DIR" -name "*.js" 2> /dev/null | wc -l)" -gt 0 ]; then
+if [ -d "$VSCode_OUT_BUILD_DIR" ] && [ "$(find "$VSCode_OUT_BUILD_DIR" -name "*.js" 2>/dev/null | wc -l)" -gt 0 ]; then
 	echo ""
 	echo "NOTE: Production build requires different compile options"
 	echo "For now comparing same output with different expected directory"

bin/Rest.js

@@ -2,35 +2,30 @@
 /**
  * Rest.js - CLI wrapper for Rest compiler binary
  */
-import { execSync } from "node:child_process";
-import { existsSync } from "node:fs";
-import { dirname, join } from "node:path";
-import { fileURLToPath } from "node:url";
+
+import { execSync } from 'node:child_process';
+import { join, dirname } from 'node:path';
+import { fileURLToPath } from 'node:url';
+import { existsSync } from 'node:fs';
 
 const __dirname = dirname(fileURLToPath(import.meta.url));
 
 // Determine the binary path based on platform
 const platform = process.platform;
-const binaryName = platform === "win32" ? "Rest.exe" : "Rest";
+const binaryName = platform === 'win32' ? 'Rest.exe' : 'Rest';
 const binaryPath = join(__dirname, binaryName);
 
 if (!existsSync(binaryPath)) {
-	console.error(`[Rest] Binary not found at: ${binaryPath}`);
-	console.error("[Rest] Please run: npm install or npm rebuild");
-	process.exit(1);
+  console.error(`[Rest] Binary not found at: ${binaryPath}`);
+  console.error('[Rest] Please run: npm install or npm rebuild');
+  process.exit(1);
 }
 
 // Execute the binary with all arguments
 try {
-	execSync(
-		`"${binaryPath}" ${process.argv
-			.slice(2)
-			.map((a) => `"${a}"`)
-			.join(" ")}`,
-		{
-			stdio: "inherit",
-		},
-	);
+  execSync(`"${binaryPath}" ${process.argv.slice(2).map(a => `"${a}"`).join(' ')}`, {
+    stdio: 'inherit',
+  });
 } catch (error) {
-	process.exit(error.status || 1);
+  process.exit(error.status || 1);
 }

prepublishOnly.sh

@@ -32,13 +32,13 @@ log_error() {
 }
 
 # Ensure Rust toolchain is configured
-if ! command -v rustup > /dev/null 2>&1; then
+if ! command -v rustup >/dev/null 2>&1; then
 	log_error "rustup is not installed. Please install Rust from https://rustup.rs/"
 	exit 1
 fi
 
 # Set default toolchain to stable if not already configured
-if ! rustup default > /dev/null 2>&1; then
+if ! rustup default >/dev/null 2>&1; then
 	log_info "Setting Rust default toolchain to stable..."
 	rustup default stable
 fi
@@ -57,7 +57,7 @@ log_info "Starting Rest build for NPM publishing..."
 log_info "Working directory: $SCRIPT_DIR"
 
 # Check if Cargo is available
-if ! command -v cargo > /dev/null 2>&1; then
+if ! command -v cargo >/dev/null 2>&1; then
 	log_error "Cargo is not installed. Please install Rust from https://rustup.rs/"
 	exit 1
 fi
@@ -70,27 +70,27 @@ detect_target() {
 	arch=$(uname -m)
 
 	case "$platform" in
-		darwin)
-			if [ "$arch" = "arm64" ]; then
-				echo "aarch64-apple-darwin"
-			else
-				echo "x86_64-apple-darwin"
-			fi
-			;;
-		linux)
-			if [ "$arch" = "aarch64" ]; then
-				echo "aarch64-unknown-linux-gnu"
-			else
-				echo "x86_64-unknown-linux-gnu"
-			fi
-			;;
-		msys* | mingw* | cygwin*)
-			echo "x86_64-pc-windows-msvc"
-			;;
-		*)
-			log_error "Unsupported platform: $platform"
-			exit 1
-			;;
+	darwin)
+		if [ "$arch" = "arm64" ]; then
+			echo "aarch64-apple-darwin"
+		else
+			echo "x86_64-apple-darwin"
+		fi
+		;;
+	linux)
+		if [ "$arch" = "aarch64" ]; then
+			echo "aarch64-unknown-linux-gnu"
+		else
+			echo "x86_64-unknown-linux-gnu"
+		fi
+		;;
+	msys* | mingw* | cygwin*)
+		echo "x86_64-pc-windows-msvc"
+		;;
+	*)
+		log_error "Unsupported platform: $platform"
+		exit 1
+		;;
 	esac
 }
 
@@ -105,12 +105,12 @@ cargo build --release --target "$TARGET"
 
 # Verify the binary was created
 case "$TARGET" in
-	*windows*)
-		BINARY_PATH="Target/$TARGET/release/Rest.exe"
-		;;
-	*)
-		BINARY_PATH="Target/$TARGET/release/Rest"
-		;;
+*windows*)
+	BINARY_PATH="Target/$TARGET/release/Rest.exe"
+	;;
+*)
+	BINARY_PATH="Target/$TARGET/release/Rest"
+	;;
 esac
 
 if [ ! -f "$BINARY_PATH" ]; then
@@ -129,16 +129,16 @@ cp "$BINARY_PATH" "bin/$BINARY_NAME"
 
 # Make the binary executable (Unix-like systems)
 case "$TARGET" in
-	*windows*) ;;
-	*)
-		chmod +x "bin/$BINARY_NAME"
-		;;
+*windows*) ;;
+*)
+	chmod +x "bin/$BINARY_NAME"
+	;;
 esac
 
 log_info "Binary copied to: bin/$BINARY_NAME"
 
 # Create the bin wrapper script (Rest.js)
-cat > bin/Rest.js << 'EOF'
+cat >bin/Rest.js <<'EOF'
 #!/usr/bin/env node
 /**
  * Rest.js - CLI wrapper for Rest compiler binary

run_tests.sh

@@ -22,7 +22,7 @@ run_test() {
 	TOTAL_TESTS=$((TOTAL_TESTS + 1))
 	printf "Test [%s]: %s ... " "$TOTAL_TESTS" "$test_name"
 
-	if eval "$test_command" > /dev/null 2>&1; then
+	if eval "$test_command" >/dev/null 2>&1; then
 		echo "PASSED"
 		PASSED_TESTS=$((PASSED_TESTS + 1))
 		return 0