[wasm] Templates don't pass the command line arguments to the app

[radical]

If we print the command line args:

using System;
using System.Runtime.InteropServices.JavaScript;

Console.WriteLine("Hello, Console!");
Console.WriteLine ($"command line args: {string.Join(',', args)}");

return 0;

public partial class MyClass
{
    [JSExport]
    internal static string Greeting()
    {
        var text = $"Hello, World! Greetings from node version: {GetNodeVersion()}";
        return text;
    }

    [JSImport("node.process.version", "main.mjs")]
    internal static partial string GetNodeVersion();
}

.. and run:

$ dotnet run x y z
WasmAppHost --runtime-config /tmp/cc/bin/Debug/net7.0/browser-wasm/AppBundle/cc.runtimeconfig.json x y z
Running: node main.mjs x y z
Using working directory: /tmp/cc/bin/Debug/net7.0/browser-wasm/AppBundle
mono_wasm_runtime_ready fe00e07a-5519-4dfe-b35a-f867dbaf2e28
Hello, World! Greetings from node version: v14.18.2
Hello, Console!
command line args: dotnet,is,great!

The command line arguments x y z were ignored, and instead we got dotnet,is,great. And that's because the main.mjs has:

await runMainAndExit(config.mainAssemblyName, ["dotnet", "is", "great!"]);

This doesn't get caught by Wasm.Build.Tests because the tests are explicitly patching the generated code

runtime/src/tests/BuildWasmApps/Wasm.Build.Tests/WasmTemplateTests.cs

Lines 48 to 58 in 4bd3ee5

	private void UpdateConsoleMainJs()
	{
	string mainJsPath = Path.Combine(_projectDir!, "main.mjs");
	string mainJsContent = File.ReadAllText(mainJsPath);
	mainJsContent = mainJsContent
	.Replace(".create()", ".withConsoleForwarding().create()")
	.Replace("[\"dotnet\", \"is\", \"great!\"]", "(await import(/* webpackIgnore: true */\"process\")).argv.slice(2)");
	File.WriteAllText(mainJsPath, mainJsContent);
	}

And the same is done for the browser too.

We should instead:

• Pass the command line args for console, and browser to the app
• Automatically set console forwarding if runArgs has forward console set
• The tests should be updated to run the template exactly as it is generated. And a follow up test can patch it to print the arguments, and check that

cc @pavelsavara

[ghost]

Tagging subscribers to 'arch-wasm': @lewing
See info in area-owners.md if you want to be subscribed.

Issue Details

---

If we print the command line args:

using System;
using System.Runtime.InteropServices.JavaScript;

Console.WriteLine("Hello, Console!");
Console.WriteLine ($"command line args: {string.Join(',', args)}");

return 0;

public partial class MyClass
{
    [JSExport]
    internal static string Greeting()
    {
        var text = $"Hello, World! Greetings from node version: {GetNodeVersion()}";
        return text;
    }

    [JSImport("node.process.version", "main.mjs")]
    internal static partial string GetNodeVersion();
}

.. and run:

$ dotnet run x y z
WasmAppHost --runtime-config /tmp/cc/bin/Debug/net7.0/browser-wasm/AppBundle/cc.runtimeconfig.json x y z
Running: node main.mjs x y z
Using working directory: /tmp/cc/bin/Debug/net7.0/browser-wasm/AppBundle
mono_wasm_runtime_ready fe00e07a-5519-4dfe-b35a-f867dbaf2e28
Hello, World! Greetings from node version: v14.18.2
Hello, Console!
command line args: dotnet,is,great!

The command line arguments x y z were ignored, and instead we got dotnet,is,great. And that's because the main.mjs has:

await runMainAndExit(config.mainAssemblyName, ["dotnet", "is", "great!"]);

This doesn't get caught by Wasm.Build.Tests because the tests are explicitly patching the generated code

runtime/src/tests/BuildWasmApps/Wasm.Build.Tests/WasmTemplateTests.cs

Lines 48 to 58 in 4bd3ee5

	private void UpdateConsoleMainJs()
	{
	string mainJsPath = Path.Combine(_projectDir!, "main.mjs");
	string mainJsContent = File.ReadAllText(mainJsPath);
	mainJsContent = mainJsContent
	.Replace(".create()", ".withConsoleForwarding().create()")
	.Replace("[\"dotnet\", \"is\", \"great!\"]", "(await import(/* webpackIgnore: true */\"process\")).argv.slice(2)");
	File.WriteAllText(mainJsPath, mainJsContent);
	}

And the same is done for the browser too.

We should instead:

• Pass the command line args for console, and browser to the app
• Automatically set console forwarding if runArgs has forward console set
• The tests should be updated to run the template exactly as it is generated. And a follow up test can patch it to print the arguments, and check that

cc @pavelsavara

Author:	radical
Assignees:	-
Labels:	arch-wasm, area-Build-mono
Milestone:	7.0.0

[pavelsavara]

we could have simpler await dotnet.run() instead of await runMainAndExit

[maraf]

we could have simpler await dotnet.run() instead of await runMainAndExit

Why do we have dotnet.run what implicitly reads main assembly and arguments, but runMainAndExit requires to pass them explicitly?

Using dotnet.run() after const runtime = dotnet.create() seems like "a break in code flow".

[pavelsavara]

Why do we have dotnet.run what implicitly reads main assembly and arguments, but runMainAndExit requires to pass them explicitly?

Because it's a lower level API which is not part of the builder pattern.

Using dotnet.run() after const runtime = dotnet.create() seems like "a break in code flow".

That's OK :)

[pavelsavara]

I believe this was fixed by #76182 and #76373
Is that right @radical @maraf ?

[maraf]

Partially. It passes command line args for the console template.

[pavelsavara]

I updated it for Net8 then

[lewing]

@radical is this resolved?

[pavelsavara]

I would like to see it resolved together with #88760

[lewing]

cc @richlander

[ghost]

Tagging subscribers to this area: @vitek-karas, @agocke
See info in area-owners.md if you want to be subscribed.

Issue Details

---

If we print the command line args:

using System;
using System.Runtime.InteropServices.JavaScript;

Console.WriteLine("Hello, Console!");
Console.WriteLine ($"command line args: {string.Join(',', args)}");

return 0;

public partial class MyClass
{
    [JSExport]
    internal static string Greeting()
    {
        var text = $"Hello, World! Greetings from node version: {GetNodeVersion()}";
        return text;
    }

    [JSImport("node.process.version", "main.mjs")]
    internal static partial string GetNodeVersion();
}

.. and run:

$ dotnet run x y z
WasmAppHost --runtime-config /tmp/cc/bin/Debug/net7.0/browser-wasm/AppBundle/cc.runtimeconfig.json x y z
Running: node main.mjs x y z
Using working directory: /tmp/cc/bin/Debug/net7.0/browser-wasm/AppBundle
mono_wasm_runtime_ready fe00e07a-5519-4dfe-b35a-f867dbaf2e28
Hello, World! Greetings from node version: v14.18.2
Hello, Console!
command line args: dotnet,is,great!

The command line arguments x y z were ignored, and instead we got dotnet,is,great. And that's because the main.mjs has:

await runMainAndExit(config.mainAssemblyName, ["dotnet", "is", "great!"]);

This doesn't get caught by Wasm.Build.Tests because the tests are explicitly patching the generated code

runtime/src/tests/BuildWasmApps/Wasm.Build.Tests/WasmTemplateTests.cs

Lines 48 to 58 in 4bd3ee5

	private void UpdateConsoleMainJs()
	{
	string mainJsPath = Path.Combine(_projectDir!, "main.mjs");
	string mainJsContent = File.ReadAllText(mainJsPath);
	mainJsContent = mainJsContent
	.Replace(".create()", ".withConsoleForwarding().create()")
	.Replace("[\"dotnet\", \"is\", \"great!\"]", "(await import(/* webpackIgnore: true */\"process\")).argv.slice(2)");
	File.WriteAllText(mainJsPath, mainJsContent);
	}

And the same is done for the browser too.

We should instead:

• Pass the command line args for console, and browser to the app
• Automatically set console forwarding if runArgs has forward console set
• The tests should be updated to run the template exactly as it is generated. And a follow up test can patch it to print the arguments, and check that

cc @pavelsavara

Author:	radical
Assignees:	-
Labels:	arch-wasm, area-HostModel, area-Build-mono
Milestone:	9.0.0

[lewing]

What area owns template argument parsing and where should this be redirected?

[maraf]

I think this is all WasmAppHost related

[lewing]

I think this is all WasmAppHost related

I think the app host doesn't get all the information it needs, but I hope I'm wrong