Remove hardwired return type, push up to Program.cs and pass generic type

isaacvale · isaacvale · commit f09136c6a987 · 2023-05-25T13:21:50.000+02:00
diff --git a/src/Dotnet.Script.Core/Commands/ExecuteScriptCommand.cs b/src/Dotnet.Script.Core/Commands/ExecuteScriptCommand.cs
@@ -31,7 +31,7 @@ public async Task<TReturn> Run<TReturn, THost>(ExecuteScriptCommandOptions optio
                 return await DownloadAndRunCode<TReturn>(options);
             }
 
-            var pathToLibrary = GetLibrary(options);
+            var pathToLibrary = GetLibrary<TReturn>(options);
 
             var libraryOptions = new ExecuteLibraryCommandOptions(pathToLibrary, options.Arguments, options.NoCache)
             {
@@ -50,7 +50,7 @@ private async Task<TReturn> DownloadAndRunCode<TReturn>(ExecuteScriptCommandOpti
             return await new ExecuteCodeCommand(_scriptConsole, _logFactory).Execute<TReturn>(options);
         }
 
-        private string GetLibrary(ExecuteScriptCommandOptions executeOptions)
+        private string GetLibrary<TReturn>(ExecuteScriptCommandOptions executeOptions)
         {
             var projectFolder = FileUtils.GetPathToScriptTempFolder(executeOptions.File.Path);
             var executionCacheFolder = Path.Combine(projectFolder, "execution-cache");
@@ -70,7 +70,7 @@ private string GetLibrary(ExecuteScriptCommandOptions executeOptions)
                 AssemblyLoadContext = executeOptions.AssemblyLoadContext
 #endif
             };
-            new PublishCommand(_scriptConsole, _logFactory).Execute(options);
+            new PublishCommand(_scriptConsole, _logFactory).Execute<TReturn>(options);
             if (hash != null)
             {
                 File.WriteAllText(Path.Combine(executionCacheFolder, "script.sha256"), hash);
diff --git a/src/Dotnet.Script.Core/Commands/PublishCommand.cs b/src/Dotnet.Script.Core/Commands/PublishCommand.cs
@@ -17,7 +17,7 @@ public PublishCommand(ScriptConsole scriptConsole, LogFactory logFactory)
             _logFactory = logFactory;
         }
 
-        public void Execute(PublishCommandOptions options)
+        public void Execute<TReturn>(PublishCommandOptions options)
         {
             var absoluteFilePath = options.File.Path;
 
@@ -41,11 +41,11 @@ public void Execute(PublishCommandOptions options)
 
             if (options.PublishType == PublishType.Library)
             {
-                publisher.CreateAssembly<int, CommandLineScriptGlobals>(context, _logFactory, options.LibraryName);
+                publisher.CreateAssembly<TReturn, CommandLineScriptGlobals>(context, _logFactory, options.LibraryName);
             }
             else
             {
-                publisher.CreateExecutable<int, CommandLineScriptGlobals>(context, _logFactory, options.RuntimeIdentifier, options.LibraryName);
+                publisher.CreateExecutable<TReturn, CommandLineScriptGlobals>(context, _logFactory, options.RuntimeIdentifier, options.LibraryName);
             }
         }
     }
diff --git a/src/Dotnet.Script/Program.cs b/src/Dotnet.Script/Program.cs
@@ -188,7 +188,7 @@ private static int Wain(string[] args)
                     );
 
                     var logFactory = CreateLogFactory(verbosity.Value(), debugMode.HasValue());
-                    new PublishCommand(ScriptConsole.Default, logFactory).Execute(options);
+                    new PublishCommand(ScriptConsole.Default, logFactory).Execute<int>(options);
                     return 0;
                 });
             });

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ public async Task<TReturn> Run<TReturn, THost>(ExecuteScriptCommandOptions optio`
`31`	`31`	`return await DownloadAndRunCode<TReturn>(options);`
`32`	`32`	`}`
`33`	`33`
`34`		`- var pathToLibrary = GetLibrary(options);`
	`34`	`+ var pathToLibrary = GetLibrary<TReturn>(options);`
`35`	`35`
`36`	`36`	`var libraryOptions = new ExecuteLibraryCommandOptions(pathToLibrary, options.Arguments, options.NoCache)`
`37`	`37`	`{`
`@@ -50,7 +50,7 @@ private async Task<TReturn> DownloadAndRunCode<TReturn>(ExecuteScriptCommandOpti`
`50`	`50`	`return await new ExecuteCodeCommand(_scriptConsole, _logFactory).Execute<TReturn>(options);`
`51`	`51`	`}`
`52`	`52`
`53`		`- private string GetLibrary(ExecuteScriptCommandOptions executeOptions)`
	`53`	`+ private string GetLibrary<TReturn>(ExecuteScriptCommandOptions executeOptions)`
`54`	`54`	`{`
`55`	`55`	`var projectFolder = FileUtils.GetPathToScriptTempFolder(executeOptions.File.Path);`
`56`	`56`	`var executionCacheFolder = Path.Combine(projectFolder, "execution-cache");`
`@@ -70,7 +70,7 @@ private string GetLibrary(ExecuteScriptCommandOptions executeOptions)`
`70`	`70`	`AssemblyLoadContext = executeOptions.AssemblyLoadContext`
`71`	`71`	`#endif`
`72`	`72`	`};`
`73`		`- new PublishCommand(_scriptConsole, _logFactory).Execute(options);`
	`73`	`+ new PublishCommand(_scriptConsole, _logFactory).Execute<TReturn>(options);`
`74`	`74`	`if (hash != null)`
`75`	`75`	`{`
`76`	`76`	`File.WriteAllText(Path.Combine(executionCacheFolder, "script.sha256"), hash);`
Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ public PublishCommand(ScriptConsole scriptConsole, LogFactory logFactory)`
`17`	`17`	`_logFactory = logFactory;`
`18`	`18`	`}`
`19`	`19`
`20`		`- public void Execute(PublishCommandOptions options)`
	`20`	`+ public void Execute<TReturn>(PublishCommandOptions options)`
`21`	`21`	`{`
`22`	`22`	`var absoluteFilePath = options.File.Path;`
`23`	`23`
`@@ -41,11 +41,11 @@ public void Execute(PublishCommandOptions options)`
`41`	`41`
`42`	`42`	`if (options.PublishType == PublishType.Library)`
`43`	`43`	`{`
`44`		`- publisher.CreateAssembly<int, CommandLineScriptGlobals>(context, _logFactory, options.LibraryName);`
	`44`	`+ publisher.CreateAssembly<TReturn, CommandLineScriptGlobals>(context, _logFactory, options.LibraryName);`
`45`	`45`	`}`
`46`	`46`	`else`
`47`	`47`	`{`
`48`		`- publisher.CreateExecutable<int, CommandLineScriptGlobals>(context, _logFactory, options.RuntimeIdentifier, options.LibraryName);`
	`48`	`+ publisher.CreateExecutable<TReturn, CommandLineScriptGlobals>(context, _logFactory, options.RuntimeIdentifier, options.LibraryName);`
`49`	`49`	`}`
`50`	`50`	`}`
`51`	`51`	`}`