sync back fixed weaver

Author: LoneWandererProductions

Weaver/CommandSignature.cs

@@ -1,4 +1,4 @@
-/*
+/*
  * COPYRIGHT:   See COPYING in the top level directory
  * PROJECT:     Weaver;
  * FILE:        CommandSignature.cs
@@ -15,4 +15,4 @@ namespace Weaver
     /// <param name="Name">The name of the command.</param>
     /// <param name="ParameterCount">The number of parameters the command expects.</param>
     public record CommandSignature(string Namespace, string Name, int ParameterCount);
-}
+}

Weaver/Core/CommandExtension.cs

@@ -46,4 +46,4 @@ public sealed class CommandExtension
         /// </value>
         public bool IsPreview { get; init; }
     }
-}
+}

Weaver/Core/DeleteValueCommand.cs

@@ -66,4 +66,4 @@ public CommandResult Execute(params string[] args)
         public CommandResult InvokeExtension(string extensionName, params string[] args)
             => CommandResult.Fail("'deleteValue' has no extensions.");
     }
-}
+}

Weaver/Core/EvaluateCommand.cs

@@ -29,7 +29,7 @@ internal class EvaluateCommand : ICommand
         public string Namespace => WeaverResources.GlobalNamespace;
 
         /// <inheritdoc />
-        public int ParameterCount => 1;
+        public int ParameterCount => -1;
 
         /// <inheritdoc />
         public CommandSignature Signature => new(Namespace, Name, ParameterCount);
@@ -58,13 +58,17 @@ public EvaluateCommand(IEvaluator evaluator, IVariableRegistry? registry = null)
         /// <inheritdoc />
         public CommandResult Execute(string[] args)
         {
-            if (args.Length < 1)
-                return CommandResult.Fail("evaluate() requires at least 1 argument: the expression to evaluate.");
+            string? expression = args.Length > 0 ? args[0] : null;
+            string? targetVar = args.Length > 1 ? args[1] : null;
 
-            var expression = args[0];
-            var targetVar = args.Length > 1 ? args[1] : null;
+            // If no expression, maybe store previous pipeline value or return null
+            if (string.IsNullOrWhiteSpace(expression))
+            {
+                // Allow empty input if .store() will handle it
+                return CommandResult.Ok(null);
+            }
 
-            // Resolve variables from registry
+            // Resolve variables from registry as before
             if (_registry != null)
             {
                 foreach (var variable in _registry.GetAll())
@@ -81,7 +85,6 @@ public CommandResult Execute(string[] args)
             object? result;
             EnumTypes type;
 
-            // Detect if expression looks like boolean
             var isBooleanExpr = _evaluator.IsBooleanExpression(expression);
 
             try
@@ -105,7 +108,7 @@ public CommandResult Execute(string[] args)
                 );
             }
 
-            // Optional target variable storage
+            // Store in registry if requested
             if (!string.IsNullOrWhiteSpace(targetVar))
             {
                 if (_registry == null)
@@ -116,15 +119,15 @@ public CommandResult Execute(string[] args)
                     );
                 }
 
-                _registry.Set(targetVar, result, type);
+                _registry.Set(targetVar, result!, type);
                 return CommandResult.Ok(
                     $"Stored '{result}' in '{targetVar}'.",
                     result,
                     type
                 );
             }
 
-            // Return result
+            // Return computed result
             var message = type switch
             {
                 EnumTypes.Wbool => result?.ToString() ?? "false",
@@ -141,4 +144,4 @@ public CommandResult InvokeExtension(string extensionName, string[] args)
             return CommandResult.Fail($"'{Name}' has no extensions.");
         }
     }
-}
+}

Weaver/Core/GetValueCommand.cs

@@ -82,4 +82,4 @@ public CommandResult InvokeExtension(string extensionName, params string[] args)
             return CommandResult.Fail($"'{Name}' has no extensions.");
         }
     }
-}
+}

Weaver/Core/HelpCommand.cs

@@ -82,4 +82,4 @@ public CommandResult InvokeExtension(string extensionName, params string[] args)
             return CommandResult.Fail($"'{Name}' has no extensions.");
         }
     }
-}
+}

Weaver/Core/HelpExtension.cs

@@ -29,10 +29,10 @@ public sealed class HelpExtension : ICommandExtension
         public int ExtensionParameterCount => 0;
 
         /// <inheritdoc />
-        public CommandResult Invoke(ICommand command, string[] args, Func<string[], CommandResult> executor)
+        public CommandResult Invoke(ICommand command, string[] extensionArgs, Func<string[], CommandResult> executor, string[] commandArgs)
         {
             // Return description; executor not needed
             return CommandResult.Ok(command.Description);
         }
     }
-}
+}

Weaver/Core/ListCommand.cs

@@ -70,4 +70,4 @@ public CommandResult InvokeExtension(string extensionName, params string[] args)
             return CommandResult.Fail($"'{Name}' has no extensions.");
         }
     }
-}
+}

Weaver/Core/MemoryCommand.cs

@@ -63,4 +63,4 @@ public CommandResult InvokeExtension(string extensionName, params string[] args)
             return CommandResult.Fail($"'{Name}' has no extensions.");
         }
     }
-}
+}

Weaver/Core/PrintCommand.cs

@@ -74,4 +74,4 @@ public CommandResult InvokeExtension(string extensionName, params string[] args)
             return CommandResult.Fail($"'{Name}' has no extensions.");
         }
     }
-}
+}