Merge remote-tracking branch 'origin/master' into ci/enable-managed-pool

Author: marcschier

.agents/skills/opcua-v20-migration/README.md

@@ -19,7 +19,7 @@ Trigger this skill when a user asks for any of:
 ## When NOT to use
 
 - The user is starting a new OPC UA project from scratch — point them at
-  `Docs/README.md` and the `Applications/ConsoleReferenceClient` /
+  `docs/README.md` and the `samples/ConsoleReferenceClient` /
   `ConsoleReferenceServer` samples instead.
 - The user is migrating **within** 1.5.x (point or service-release upgrades).
 - The user is debugging server-side OPC UA functional behaviour — try the
@@ -53,16 +53,16 @@ opcua-v20-migration/
 This skill **distils** the following authoritative repo files. When you update
 the skill, also update them (and vice versa) so the views stay in sync:
 
-- [`Docs/MigrationGuide.md`](../../../Docs/MigrationGuide.md) — the migration
+- [`docs/MigrationGuide.md`](../../../docs/MigrationGuide.md) — the migration
   guide landing page (small; just an index across versions).
-- [`Docs/migrate/2.0.x/README.md`](../../../Docs/migrate/2.0.x/README.md) —
+- [`docs/migrate/2.0.x/README.md`](../../../docs/migrate/2.0.x/README.md) —
   the 2.0 version landing page + the same symptom → sub-doc table this skill
   uses to load only what's needed.
-- [`Docs/migrate/2.0.x/`](../../../Docs/migrate/2.0.x/) — the 12 thematic
+- [`docs/migrate/2.0.x/`](../../../docs/migrate/2.0.x/) — the 12 thematic
   sub-docs (telemetry, packages, source-generation, types, encoders,
   node-states, identity, certificates, configuration, sessions-subscriptions,
   alarms-model-change, timeprovider).
-- [`Tools/Opc.Ua.MigrationAnalyzer/NugetREADME.md`](../../../Tools/Opc.Ua.MigrationAnalyzer/NugetREADME.md)
+- [`tools/Opc.Ua.MigrationAnalyzer/NugetREADME.md`](../../../tools/Opc.Ua.MigrationAnalyzer/NugetREADME.md)
   — the package's own NuGet README.
 
 ## License

.agents/skills/opcua-v20-migration/SKILL.md

@@ -26,10 +26,10 @@ metadata:
   version: "1.0.0"
   upstream: https://github.com/OPCFoundation/UA-.NETStandard
   canonical-docs:
-    - Docs/MigrationGuide.md                      # landing page (small)
-    - Docs/migrate/2.0.x/README.md                # 2.0 version landing + sub-doc index
-    - Docs/migrate/2.0.x/                         # 12 thematic sub-docs (load only what you need)
-    - Tools/Opc.Ua.MigrationAnalyzer/NugetREADME.md
+    - docs/MigrationGuide.md                      # landing page (small)
+    - docs/migrate/2.0.x/README.md                # 2.0 version landing + sub-doc index
+    - docs/migrate/2.0.x/                         # 12 thematic sub-docs (load only what you need)
+    - tools/Opc.Ua.MigrationAnalyzer/NugetREADME.md
 ---
 
 # OPC UA .NET Standard 1.5.378 → 2.0 Migration
@@ -42,28 +42,28 @@ Upgrade existing OPC UA .NET Standard consumer projects from 1.5.378
 
 **Context-efficiency rule.** The full migration content is no longer in
 a single ~150 KB document; it is split across 12 thematic sub-docs under
-[`Docs/migrate/2.0.x/`](../../../Docs/migrate/2.0.x/README.md).
-**Do not load `Docs/MigrationGuide.md` into context** — it is now a thin
+[`docs/migrate/2.0.x/`](../../../docs/migrate/2.0.x/README.md).
+**Do not load `docs/MigrationGuide.md` into context** — it is now a thin
 landing page. Instead, match the user's symptom to a row below and load
 the **single** sub-doc named in that row.
 
 | When the user hits… | Load only |
 | --- | --- |
-| `CS0029` / `CS1503` / `CS0266` on `NodeId`, `Variant`, `DataValue`, `ExtensionObject`, `QualifiedName`, `LocalizedText`, `ArrayOf<T>` / `MatrixOf<T>`, `ByteString`, `StatusCode`, `XmlElement`, `EnumValue`, or `[Obsolete]` on built-in type APIs (analyzer `UA0001`–`UA0020`) | [`Docs/migrate/2.0.x/types.md`](../../../Docs/migrate/2.0.x/types.md) |
-| `Utils.LogX`, `Utils.Trace`, static logger helpers (`Utils.SetLogger` / `Utils.SetLogLevel` removed), `ITelemetryContext` constructor parameter shape per type, OLD-vs-NEW logger snippets, fluent `AddOpcUa().AddLogging().AddMetrics()` registration, breaking-changes inventory across Core / Configuration / Client / Server / PubSub / Certificate / Transport, migration utilities (`DefaultTelemetry`, `Telemetry.NullLogger`, `Utils.Fallback.Logger`) | [`Docs/migrate/2.0.x/telemetry.md`](../../../Docs/migrate/2.0.x/telemetry.md) |
-| Package upgrades, TFM changes, `Newtonsoft.Json` removal from `Opc.Ua.Core`, new published packages | [`Docs/migrate/2.0.x/packages.md`](../../../Docs/migrate/2.0.x/packages.md) |
-| Source-generated `*Collection` shims, NodeManager generator, default of `bool` properties, project structure | [`Docs/migrate/2.0.x/source-generation.md`](../../../Docs/migrate/2.0.x/source-generation.md) |
-| `IEncodeableFactoryBuilder`, `IType`, JSON / XML / binary encoders, `EncodeableFactory.GlobalFactory`, `IJsonEncodeable`, `ComplexTypes` namespace move | [`Docs/migrate/2.0.x/encoders.md`](../../../Docs/migrate/2.0.x/encoders.md) |
-| Custom NodeManagers, `NodeState` clone / read / write helpers, `Clone` → `CreateCopy`, `OnAfterCreate(CancellationToken)`, `INodeManager3`, `INodeCache.InvalidateNode`, generics on `BaseVariableState` / `BaseVariableTypeState` | [`Docs/migrate/2.0.x/node-states.md`](../../../Docs/migrate/2.0.x/node-states.md) |
-| `IUserIdentityTokenHandler`, `IClientIdentityProvider`, `IUserTokenAuthenticator`, `IAccessTokenProvider`, `ITokenIssuer`, caller-supplied secrets, secret store | [`Docs/migrate/2.0.x/identity.md`](../../../Docs/migrate/2.0.x/identity.md) |
-| `CertificateValidator` rename (`UA0021`), ref-counted `Certificate` wrapper, `CertificateManager`, `ICertificateProvider`, obsoleted `X509Certificate2` direct-exposure APIs | [`Docs/migrate/2.0.x/certificates.md`](../../../Docs/migrate/2.0.x/certificates.md) |
-| `ApplicationConfiguration` changes, Data-Contract serializer removal, `ParseExtension` / `UpdateExtension` signature, session / browser state persistence | [`Docs/migrate/2.0.x/configuration.md`](../../../Docs/migrate/2.0.x/configuration.md) |
-| `Session` → `ManagedSession`, V2 subscription engine, GDS-client `Task` → `ValueTask` modernisation, removed obsolete GDS APIs, durable subscriptions, PubSub, reverse-connect | [`Docs/migrate/2.0.x/sessions-subscriptions.md`](../../../Docs/migrate/2.0.x/sessions-subscriptions.md) |
-| `AlarmConditionState` state-transition behaviour, auto-emitted `GeneralModelChangeEvent`, `ModelChangeAggregator` | [`Docs/migrate/2.0.x/alarms-model-change.md`](../../../Docs/migrate/2.0.x/alarms-model-change.md) |
-| `DateTime.UtcNow`, `Timer`, deterministic-time tests, `System.TimeProvider` adoption | [`Docs/migrate/2.0.x/timeprovider.md`](../../../Docs/migrate/2.0.x/timeprovider.md) |
+| `CS0029` / `CS1503` / `CS0266` on `NodeId`, `Variant`, `DataValue`, `ExtensionObject`, `QualifiedName`, `LocalizedText`, `ArrayOf<T>` / `MatrixOf<T>`, `ByteString`, `StatusCode`, `XmlElement`, `EnumValue`, or `[Obsolete]` on built-in type APIs (analyzer `UA0001`–`UA0020`) | [`docs/migrate/2.0.x/types.md`](../../../docs/migrate/2.0.x/types.md) |
+| `Utils.LogX`, `Utils.Trace`, static logger helpers (`Utils.SetLogger` / `Utils.SetLogLevel` removed), `ITelemetryContext` constructor parameter shape per type, OLD-vs-NEW logger snippets, fluent `AddOpcUa().AddLogging().AddMetrics()` registration, breaking-changes inventory across Core / Configuration / Client / Server / PubSub / Certificate / Transport, migration utilities (`DefaultTelemetry`, `Telemetry.NullLogger`, `Utils.Fallback.Logger`) | [`docs/migrate/2.0.x/telemetry.md`](../../../docs/migrate/2.0.x/telemetry.md) |
+| Package upgrades, TFM changes, `Newtonsoft.Json` removal from `Opc.Ua.Core`, new published packages | [`docs/migrate/2.0.x/packages.md`](../../../docs/migrate/2.0.x/packages.md) |
+| Source-generated `*Collection` shims, NodeManager generator, default of `bool` properties, project structure | [`docs/migrate/2.0.x/source-generation.md`](../../../docs/migrate/2.0.x/source-generation.md) |
+| `IEncodeableFactoryBuilder`, `IType`, JSON / XML / binary encoders, `EncodeableFactory.GlobalFactory`, `IJsonEncodeable`, `ComplexTypes` namespace move | [`docs/migrate/2.0.x/encoders.md`](../../../docs/migrate/2.0.x/encoders.md) |
+| Custom NodeManagers, `NodeState` clone / read / write helpers, `Clone` → `CreateCopy`, `OnAfterCreate(CancellationToken)`, `INodeManager3`, `INodeCache.InvalidateNode`, generics on `BaseVariableState` / `BaseVariableTypeState` | [`docs/migrate/2.0.x/node-states.md`](../../../docs/migrate/2.0.x/node-states.md) |
+| `IUserIdentityTokenHandler`, `IClientIdentityProvider`, `IUserTokenAuthenticator`, `IAccessTokenProvider`, `ITokenIssuer`, caller-supplied secrets, secret store | [`docs/migrate/2.0.x/identity.md`](../../../docs/migrate/2.0.x/identity.md) |
+| `CertificateValidator` rename (`UA0021`), ref-counted `Certificate` wrapper, `CertificateManager`, `ICertificateProvider`, obsoleted `X509Certificate2` direct-exposure APIs | [`docs/migrate/2.0.x/certificates.md`](../../../docs/migrate/2.0.x/certificates.md) |
+| `ApplicationConfiguration` changes, Data-Contract serializer removal, `ParseExtension` / `UpdateExtension` signature, session / browser state persistence | [`docs/migrate/2.0.x/configuration.md`](../../../docs/migrate/2.0.x/configuration.md) |
+| `Session` → `ManagedSession`, V2 subscription engine, GDS-client `Task` → `ValueTask` modernisation, removed obsolete GDS APIs, durable subscriptions, PubSub, reverse-connect | [`docs/migrate/2.0.x/sessions-subscriptions.md`](../../../docs/migrate/2.0.x/sessions-subscriptions.md) |
+| `AlarmConditionState` state-transition behaviour, auto-emitted `GeneralModelChangeEvent`, `ModelChangeAggregator` | [`docs/migrate/2.0.x/alarms-model-change.md`](../../../docs/migrate/2.0.x/alarms-model-change.md) |
+| `DateTime.UtcNow`, `Timer`, deterministic-time tests, `System.TimeProvider` adoption | [`docs/migrate/2.0.x/timeprovider.md`](../../../docs/migrate/2.0.x/timeprovider.md) |
 
 If the user's symptom does not obviously map to one row, read
-[`Docs/migrate/2.0.x/README.md`](../../../Docs/migrate/2.0.x/README.md) (small —
+[`docs/migrate/2.0.x/README.md`](../../../docs/migrate/2.0.x/README.md) (small —
 the same table plus a short intro) and pick from there. Avoid loading
 multiple sub-docs unless the symptom genuinely spans two areas (for
 example, `node-states.md` *and* `types.md` when a NodeManager runs into
@@ -153,7 +153,7 @@ dotnet format analyzers <YourSolution>.sln \
   migration window, then peel each entry back as you fix the rule.
 - **The legacy `Quickstarts.Servers` meta-package does not exist on 2.0.** If
   your project depends on it, switch to a `<ProjectReference>` to
-  `Applications/Quickstarts.Servers` or to an equivalent project of your own.
+  `samples/Quickstarts.Servers` or to an equivalent project of your own.
 
 ---
 
@@ -287,7 +287,7 @@ MigrationAnalyzer package is removed.
   added inline to each csproj.
 - The `OPCFoundation.NetStandard.Opc.Ua.Quickstarts.Servers` meta-package is
   not published on 2.0 — consumers must switch to a `<ProjectReference>` to
-  `Applications/Quickstarts.Servers` or an equivalent first-party project.
+  `samples/Quickstarts.Servers` or an equivalent first-party project.
 - Public APIs returning a `<Type>Collection` will hit `CS0050` because the
   generator's shim is `internal` by design — migrate the public surface to
   `List<T>` / `ArrayOf<T>` first.
@@ -336,7 +336,7 @@ material. Load them on demand via your agent runtime's
 
 ### Canonical upstream docs
 
-- [`Docs/MigrationGuide.md`](https://github.com/OPCFoundation/UA-.NETStandard/blob/master/Docs/MigrationGuide.md) — the human-facing migration guide landing page (~3 KB; intentionally small, just an index across versions).
-- [`Docs/migrate/2.0.x/README.md`](https://github.com/OPCFoundation/UA-.NETStandard/blob/master/Docs/migrate/2.0.x/README.md) — the 2.0 migration index + the same symptom → sub-doc table this skill uses.
-- [`Docs/migrate/2.0.x/`](https://github.com/OPCFoundation/UA-.NETStandard/tree/master/Docs/migrate/2.0.x) — the 12 thematic sub-docs (telemetry, packages, source-generation, types, encoders, node-states, identity, certificates, configuration, sessions-subscriptions, alarms-model-change, timeprovider).
-- [`Tools/Opc.Ua.MigrationAnalyzer/NugetREADME.md`](https://github.com/OPCFoundation/UA-.NETStandard/blob/master/Tools/Opc.Ua.MigrationAnalyzer/NugetREADME.md) — the package's own README, shipped inside the NuGet.
+- [`docs/MigrationGuide.md`](https://github.com/OPCFoundation/UA-.NETStandard/blob/master/docs/MigrationGuide.md) — the human-facing migration guide landing page (~3 KB; intentionally small, just an index across versions).
+- [`docs/migrate/2.0.x/README.md`](https://github.com/OPCFoundation/UA-.NETStandard/blob/master/docs/migrate/2.0.x/README.md) — the 2.0 migration index + the same symptom → sub-doc table this skill uses.
+- [`docs/migrate/2.0.x/`](https://github.com/OPCFoundation/UA-.NETStandard/tree/master/docs/migrate/2.0.x) — the 12 thematic sub-docs (telemetry, packages, source-generation, types, encoders, node-states, identity, certificates, configuration, sessions-subscriptions, alarms-model-change, timeprovider).
+- [`tools/Opc.Ua.MigrationAnalyzer/NugetREADME.md`](https://github.com/OPCFoundation/UA-.NETStandard/blob/master/tools/Opc.Ua.MigrationAnalyzer/NugetREADME.md) — the package's own README, shipped inside the NuGet.

.agents/skills/opcua-v20-migration/references/analyzer-rules.md

@@ -365,7 +365,7 @@ config.CertificateManager.Options.AcceptError = (cert, error) =>
 // or implement ICertificateValidatorEx.ValidateAsync for full control.
 ```
 
-See [`Docs/migrate/2.0.x/certificates.md`](../../../Docs/migrate/2.0.x/certificates.md)
+See [`docs/migrate/2.0.x/certificates.md`](../../../../docs/migrate/2.0.x/certificates.md)
 §"Certificate Manager and segregated interfaces" for the structural model in full.
 
 ---

.agents/skills/opcua-v20-migration/references/known-gaps.md

@@ -63,7 +63,7 @@ happen on plain 1.5.378 too.
 not published on 2.0.
 
 **Mitigation:** consumers must switch to a `<ProjectReference>` to
-`Applications/Quickstarts.Servers` in this repo, or to an equivalent
+`samples/Quickstarts.Servers` in this repo, or to an equivalent
 first-party project of their own. All 2.0 packages are published to the
 OPC Foundation Azure Artifacts preview feed
 (`https://opcfoundation.visualstudio.com/opcua-netstandard/_packaging/opcua-preview/nuget/v3/index.json`)

.agents/skills/opcua-v20-migration/references/migration-patterns.md

@@ -698,7 +698,7 @@ sc = sc.WithAggregateBits(agg);
 4. **Build incrementally.** Fix one priority layer at a time and rebuild to
    track progress.
 5. **Refer to canonical docs.** The
-   [`Docs/migrate/2.0.x/`](../../../Docs/migrate/2.0.x/) sub-docs are the
+   [`docs/migrate/2.0.x/`](../../../../docs/migrate/2.0.x/) sub-docs are the
    upstream source of truth — sync this skill against them on every PR.
    Use the symptom → sub-doc table in `SKILL.md` to load only the
    relevant one.

.agents/skills/opcua-v20-migration/references/package-install.md

@@ -75,7 +75,7 @@ Common packages and their 2.0 paths:
 | `OPCFoundation.NetStandard.Opc.Ua.Server` | Unchanged name. |
 | `OPCFoundation.NetStandard.Opc.Ua.Bindings.Https` | Unchanged name. |
 | `OPCFoundation.NetStandard.Opc.Ua.Gds.Common` | **New in 2.0** — intermediate project with shared GDS types. If you reference `Gds.Client.Common` or `Gds.Server.Common`, they now depend on it transitively (no action needed in most cases). |
-| `OPCFoundation.NetStandard.Opc.Ua.Quickstarts.Servers` | **Not published on 2.0** — switch to a `<ProjectReference>` to `Applications/Quickstarts.Servers` or equivalent. |
+| `OPCFoundation.NetStandard.Opc.Ua.Quickstarts.Servers` | **Not published on 2.0** — switch to a `<ProjectReference>` to `samples/Quickstarts.Servers` or equivalent. |
 
 If your solution already uses **Central Package Management** (`Directory.Packages.props`),
 update there instead:

.agents/skills/opcua-v20-migration/references/runtime-shim.md

@@ -44,7 +44,7 @@ so call sites bind:
 - Sync + APM wrappers for the `GlobalDiscoveryServerClient` /
   `LocalDiscoveryServerClient` / `ServerPushConfigurationClient` (UA0015)
 - `GlobalDiscoverySampleServer` 1.5.378-shape ctor (in-tree at
-  `Libraries/Opc.Ua.Gds.Server.Common/GlobalDiscoverySampleServer.cs`) — the 5-arg
+  `src/Opc.Ua.Gds.Server.Common/GlobalDiscoverySampleServer.cs`) — the 5-arg
   variant without `ITelemetryContext`
 
 ### Conversion helpers

.agents/skills/opcua-v20-migration/references/source-generator.md

@@ -92,7 +92,7 @@ reference (`global::Acme.WaterPump`, `global::Vendor.Devices.BoilerState`, etc).
 Zero matches or > 1 matches → the generator emits **no** shim and reports
 `MIG01` instead, with the help link pointing at this skill's
 [`analyzer-rules.md`](analyzer-rules.md#mig01--generator-cant-resolve-element-type-for-foocollection)
-and [`Docs/migrate/2.0.x/source-generation.md`](../../../Docs/migrate/2.0.x/source-generation.md).
+and [`docs/migrate/2.0.x/source-generation.md`](../../../../docs/migrate/2.0.x/source-generation.md).
 
 ## Generated file shape
 

.azurepipelines/generate-slnx.ps1

@@ -6,8 +6,8 @@
 .DESCRIPTION
     Replaces the static `UA Core Library.slnx`.  The single source of
     truth for the project graph is `UA.slnx`; this script reads it,
-    drops every folder whose name starts with `/Tests/` or
-    `/Fuzzing/` (and every project inside those folders), and writes
+    drops every folder whose name starts with `/tests/` or
+    `/fuzzing/` (and every project inside those folders), and writes
     the result.
 
     Whether each remaining project actually produces a NuGet on
@@ -30,7 +30,7 @@
 
 .PARAMETER ExcludeFolderPrefixes
     Folder-name prefixes (as they appear inside the source .slnx) to
-    drop entirely.  Default: `/Tests/`, `/Fuzzing/`, `/Solution Items/`.
+    drop entirely.  Default: `/tests/`, `/fuzzing/`, `/Solution Items/`.
     Folder matching is whole-name based on the `Name=` attribute, so
     `/Solution Items/` and any `/Solution Items/...` sub-folder
     matches the prefix.
@@ -43,7 +43,7 @@
 param(
     [string]$OutputPath,
     [string]$SourceSolution,
-    [string[]]$ExcludeFolderPrefixes = @('/Tests/', '/Fuzzing/', '/Solution Items/')
+    [string[]]$ExcludeFolderPrefixes = @('/tests/', '/fuzzing/', '/Solution Items/')
 )
 
 $ErrorActionPreference = 'Stop'

.azurepipelines/preview.yml

@@ -101,10 +101,10 @@ jobs:
     displayName: 'List of assemblies to sign'
     inputs:
       script: |
-        dir /b /s Stack\Opc.Ua*.dll > .\list.txt
-        dir /b /s Tools\Opc.Ua*.dll >> .\list.txt
-        dir /b /s Libraries\Opc.Ua*.dll >> .\list.txt
-        dir /b /s Applications\McpServer\bin\Opc.Ua*.dll >> .\list.txt
+        dir /b /s src\Opc.Ua*.dll > .\list.txt
+        dir /b /s tools\Opc.Ua*.dll >> .\list.txt
+        dir /b /s src\Opc.Ua*.dll >> .\list.txt
+        dir /b /s tools\Opc.Ua.Mcp\bin\Opc.Ua*.dll >> .\list.txt
         dir /b /s .azurepipelines\*.* >> .\list.txt
         type .\list.txt
   - task: CmdLine@2
@@ -127,7 +127,7 @@ jobs:
     continueOnError: true
     inputs:
       command: 'pack'
-      # Scoped to nuget/ — the MigrationAnalyzer nuspec at Tools/Opc.Ua.MigrationAnalyzer/
+      # Scoped to nuget/ — the MigrationAnalyzer nuspec at tools/Opc.Ua.MigrationAnalyzer/
       # uses MSBuild $(...) substitutions ($(shimBin), $(repoRoot), $(analyzerDll))
       # that only `dotnet pack`'s _SetMigrationAnalyzerNuspecProperties target can
       # resolve. Standalone `nuget.exe pack` leaves them literal and crashes with