docs(nunit): record Rider MTP discovery traps and package 0.0.4

Discover is PE-only on the CLI too; Rider net48 still double-lists native NUnit with no suppression hook.

Author: trgiangv

docs/agents/nunit-host-testing.md

@@ -30,8 +30,8 @@ Host DLL changes: `scripts/build-host.ps1 -Year <year>`. Runner: `dotnet publish
 
 - Do not use `NUnit.Engine` in the host.
 - Do not add `NUnit3TestAdapter` or `NUnit.Microsoft.Testing.Platform` to a host-test project.
-- Rider always shows **two** `HostSmokeTests` trees (native NUnit + DevTools). Only the second/adapter node runs in-host. Proven settings: **Testing Platform** → enable, **uncheck** “Ignore projects discovered by other providers” (else MTP is invisible because `[Test]` is already claimed). **VSTest** → enable adapters, keep project mask `*Tests*` (ProjectReference to our adapter is not enough for Rider to show the VSTest suite). Do not add `executor://DevTools.NUnit.V1/` to the adapter ignore list. Do not add `NUnit3TestAdapter`. Visual Studio does not double-discover this way.
-- Test Explorer refresh must not start a host. Discovery is local PE metadata. `HostLaunch=false` still starts a matching-version host on **run** if none is open.
+- Rider **MTP + net48**: two `HostSmokeTests` trees (native NUnit PSI + DevTools MTP). Only the MTP/DevTools node runs in-host. **MTP + net8**: one tree (native NUnit does not claim the MTP exe). Visual Studio does not double-discover. There is **no** csproj/attribute/ExecutorUri hook to suppress native NUnit; ricaun’s 1-suite explorer is VSTest + NUnit 3, not a discovery API we are missing. Current Rider Testing Platform UI: enable MTP. The old “Ignore projects discovered by other providers” checkbox may be absent — do not block on it. **VSTest** sample: enable adapters, keep project mask `*Tests*` (ProjectReference alone is not enough). Do not add `executor://DevTools.NUnit.V1/` to the adapter ignore list. Do not add `NUnit3TestAdapter`. Sample `Intentional_failure_for_demo` is an expected `Assert.Fail`. A suite-level `ArgumentException` “same key already added” after running all MTP cases is IDE result merge, not a host failure.
+- Test Explorer refresh / `dotnet test --list-tests` / `Runner discover` must not start a host. Discovery is local PE metadata. `HostLaunch=false` still starts a matching-version host on **run** if none is open.
 - Autodesk configs flatten host obj/bin. MTP overrides `AppendTargetFrameworkToOutputPath=true` so its three TFMs never share a folder (CS2012 / MSB3713). Do not collapse `TargetFrameworks` on packable projects.
 - net48 Test Explorer "could not be discovered": `CreateTestSession` failed to load `Unsafe` 6.0. Package props generate binding redirects and pin Unsafe 6.1.2. Adapter does not hit this because it ILRepacks.
 - MTP samples are `OutputType=Exe`. Generation snapshot must treat `.exe` as a managed test assembly and skip `Log/` / `TestResults/` / `*.diag`.

docs/product/nunit-host-testing.md

@@ -10,8 +10,8 @@ controller ships in that installer. The live CAD host executes the tests.
 
 **Experimental.** Discover / run / progress work end-to-end. `RevitDevTool.NUnit`
 (MTP) is published independently of the RevitDevTool installer. Version is
-`<Version>` in `source/DevTools.NUnit.Mtp/DevTools.NUnit.Mtp.csproj` (starts at
-`0.0.1`). Run `.github/workflows/PublishNUnit.yml` from GitHub Actions to pack
+`<Version>` in `source/DevTools.NUnit.Mtp/DevTools.NUnit.Mtp.csproj` (currently
+`0.0.4`). Run `.github/workflows/PublishNUnit.yml` from GitHub Actions to pack
 and push nuget.org (OIDC Trusted Publishing). VSTest stays in-tree and is not
 published.
 
@@ -31,11 +31,11 @@ VSTest (`DevTools.NUnit.TestAdapter`). Keep them on **separate** test projects.
 
 | Project | Role |
 |---------|------|
-| `DevTools.NUnit.Core` | `nunit/*` wire contracts, timing, protocol version |
+| `DevTools.NUnit.Core` | `nunit/*` wire contracts, timing, protocol version. Out-of-process Runner client (`Client/`, linked into MTP/VSTest, not in the host DLL) |
 | `DevTools.NUnit.Host` | Native NUnit runtime in the CAD host |
-| `DevTools.NUnit.Runner` | CLI controller: find/launch host pipe, discover/run |
-| `DevTools.NUnit.Mtp` | MTP framework (`PackageId=RevitDevTool.NUnit`); proxies to Runner |
-| `DevTools.NUnit.TestAdapter` | VSTest adapter; same Runner/Host contract |
+| `DevTools.NUnit.Runner` | CLI: PE discover locally; find/launch host pipe only on **run** |
+| `DevTools.NUnit.Mtp` | MTP framework (`PackageId=RevitDevTool.NUnit`); PE discover in-process; Runner only on **run** |
+| `DevTools.NUnit.TestAdapter` | VSTest adapter; same PE discover + Runner **run** contract |
 
 `RevitDevTool.NUnit` is an independent test-platform package. Consumers set
 `HostName` / `HostVersion` / launch timeouts; the package does not read this
@@ -82,8 +82,8 @@ IDE selected-run uses FullName. Runner composes NUnit `TestFilter` XML for the h
 
 ## Behavior
 
-- IDE / `dotnet test --list-tests` discovery reads PE metadata locally. It does
-  **not** start a host process. Runner contacts the host only when tests execute.
+- IDE / `dotnet test --list-tests` / `Runner discover` read PE metadata locally.
+  They do **not** start a host process. Runner contacts the host only on `run`.
 - `HostLaunch=false` reuses a running host with the same `HostName` +
   `HostVersion` (oldest matching PID; no session picker). If none exists, it
   starts one. `HostLaunch=true` starts a new host for that Runner invocation