fix: add targetRuntime meta information to monitor

[dotkas]

The entire foundation for utilizing the targetRuntime exists in the Snyk backend, but is not utilized by snyk monitor.

Adding the targetRuntime will decorate the snyk monitor results with the Target Runtime information, used in multiple language ecosystems.

Here an example from .NET, before:

After:

This will also pave the way for support of snyk monitor of a single project with multiple target runtimes attached, without overwriting the same project multiple times.

[github-actions]

	Warnings
⚠️	Since the CLI is unifying on a standard and improved tooling, we're starting to migrate old-style imports and exports to ES6 ones. A file you've modified is using either module.exports or require(). If you can, please update them to ES6 import syntax and export syntax. Files found: test/tap/cli-monitor.acceptance.test.ts

Generated by 🚫 dangerJS against 6a92147

[darscan]

Blocking. We need to verify that this will not break existing pipelines (i.e. that tests map to the same projects that they used to, and that this won't create a flood of new projects in the system)

[dotkas]

@darscan I already did that.

[darscan]

So, you've manually tested every combination of cli-plugin and input argument, for every ecosystem against existing projects in the system?

[dotkas]

@darscan I checked which CLI plugin would populate meta.targetRuntime, could not find anything other than snyk-nuget-plugin that is used by the CLI currently. What have I missed?

[dotkas]

@darscan it only applies to runs done with the beta-flag --dotnet-runtime-resolution, meaning existing .NET projects are not affected.

[darscan]

Thanks, that clears things up a lot 👍

Addressed

[dotkas]

Disabling auto-merge and double-checking everything. @darscan made me paranoid.

[darscan]

it only applies to runs done with the beta-flag --dotnet-runtime-resolution, meaning existing .NET projects are not affected.

But what happens when that feature goes GA?

[dotkas]

Continuing the discussion outside this PR.