com.wallstop-studios.dxmessaging 2.1.1 → 2.1.3

package/.github/workflows/dotnet-tests.yml

@@ -0,0 +1,72 @@
+name: dotnet-tests
+
+on:
+  push:
+    branches:
+      - master
+      - main
+      - develop
+  pull_request:
+    branches:
+      - "*"
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Setup .NET
+        uses: actions/setup-dotnet@v5
+        with:
+          dotnet-version: "9.0.x"
+
+      - name: Install reportgenerator tool
+        run: |
+          dotnet tool install --global dotnet-reportgenerator-globaltool
+          echo "$HOME/.dotnet/tools" >> "$GITHUB_PATH"
+
+      - name: Restore
+        run: dotnet restore SourceGenerators/WallstopStudios.DxMessaging.SourceGenerators.Tests/WallstopStudios.DxMessaging.SourceGenerators.Tests.csproj
+
+      - name: Run generator diagnostics tests
+        run: >
+          dotnet test SourceGenerators/WallstopStudios.DxMessaging.SourceGenerators.Tests/WallstopStudios.DxMessaging.SourceGenerators.Tests.csproj
+          --configuration Release
+          --collect:"XPlat Code Coverage"
+
+      - name: Generate coverage summary
+        run: |
+          mkdir -p artifacts/coverage
+          reportgenerator \
+            -reports:".artifacts/SourceGenerators.Tests/TestResults/**/coverage.cobertura.xml" \
+            -targetdir:"artifacts/coverage" \
+            -reporttypes:"TextSummary;HtmlSummary;Cobertura"
+          cat artifacts/coverage/Summary.txt
+
+      - name: Upload coverage report
+        uses: actions/upload-artifact@v4
+        with:
+          name: generator-coverage
+          path: artifacts/coverage
+
+      - name: Check for TODO/FIXME markers
+        shell: bash
+        run: |
+          set -uo pipefail
+          if rg --files-with-matches "(TODO|FIXME)" \
+                --hidden \
+                --no-ignore \
+                -g '!.git/**' \
+                -g '!.artifacts/**' \
+                -g '!node_modules/**' \
+                -g '!Tests/**/TestResults/**' \
+                -g '!PLAN.md' \
+                -g '!.vs/**'; then
+            echo "::error::Found TODO/FIXME markers in the repository. Please resolve or suppress them."
+            exit 1
+          else
+            echo "No TODO/FIXME markers detected."
+          fi

package/.lychee.toml

@@ -17,10 +17,12 @@ accept = ["200..=299", 429]
 # Only check web links
 scheme = ["https", "http"]
 
-# Ignore common local/test URLs
+# Ignore endpoints that block automated clients (local hosts, CF challenges)
 exclude = [
   "^https?://localhost",
   "^http://127\\.0\\.0\\.1",
   "^https?://0\\.0\\.0\\.0",
-  "^file://"
+  "^file://",
+  # NPM package page serves a Cloudflare JS challenge to non-browser clients; registry endpoint remains covered.
+  "^https://www\\.npmjs\\.com/package/com\\.wallstop-studios\\.dxmessaging$"
 ]

package/AGENTS.md

@@ -2,12 +2,12 @@
 
 ## Project Structure & Module Organization
 
-- `Runtime/` — core library (`DxMessaging.Core`) and Unity components (`DxMessaging.Unity`).
-- `Editor/` — editor utilities, analyzers, and setup (`DxMessaging.Editor`).
-- `SourceGenerators/` — Roslyn source generators (`netstandard2.0`).
-- `Tests/Runtime/` — NUnit/Unity Test Framework tests (e.g., `Core/NominalTests.cs`).
-- `Docs/` — usage patterns and examples.
-- Package manifest: `package.json` (published to NPM/UPM).
+- [Runtime](Runtime/) — core library (`DxMessaging.Core`) and Unity components (`DxMessaging.Unity`).
+- [Editor](Editor/) — editor utilities, analyzers, and setup (`DxMessaging.Editor`).
+- [SourceGenerators](SourceGenerators/) — Roslyn source generators (`netstandard2.0`).
+- [Tests/Runtime](Tests/Runtime/) — NUnit/Unity Test Framework tests (e.g., [Core Nominal tests](Tests/Runtime/Core/NominalTests.cs)).
+- [Docs](Docs/) — usage patterns and examples.
+- Package manifest: [package.json](package.json) (published to NPM/UPM).
 
 ## Build, Test, and Development Commands
 
@@ -17,18 +17,19 @@
 
 ## Coding Style & Naming Conventions
 
-- Indent with 4 spaces for `.cs` (JSON/YAML: 2). CRLF, UTF‑8 BOM (see `.editorconfig`).
+- Indent with 4 spaces for `.cs` (JSON/YAML: 2). CRLF, UTF‑8 BOM (see [.editorconfig](.editorconfig)).
 - Prefer explicit types over `var`. Braces required. `using` directives inside the namespace.
-- Naming: `PascalCase` for types/methods/properties; interfaces `I*`; type parameters `T*`; events prefixed `On*`; public fields lowerCamelCase (matches examples in `README.md`).
+- Naming: `PascalCase` for types/methods/properties; interfaces `I*`; type parameters `T*`; events prefixed `On*`; public fields lowerCamelCase (matches examples in the [README](README.md)).
 - Place code under `DxMessaging.Core`, `DxMessaging.Unity`, or `DxMessaging.Editor` as appropriate.
 - Do not use underscores in function names, especially test function names.
 - Do not use regions, anywhere, ever.
 - Avoid `var` wherever possible, use expressive types.
+- Do not use nullable reference types.
 
 ## Testing Guidelines
 
 - Frameworks: NUnit + Unity Test Framework. Use `[Test]`/`[UnityTest]` as needed.
-- Location: add files under `Tests/Runtime/<Area>/` named `*Tests.cs` with classes `*Tests`.
+- Location: add files under [Tests/Runtime](Tests/Runtime/) `<Area>/` named `*Tests.cs` with classes `*Tests`.
 - Keep tests independent: prefer a local `MessageBus` and explicit `MessageRegistrationToken` lifecycles.
 - Do not use underscores in test function names.
 - Prefer expressive assertions and failure messages so it is clear what exactly is failing when a test fails.
@@ -42,10 +43,10 @@
 ## Commit & Pull Request Guidelines
 
 - Commits: short, imperative subject; group related changes; reference issues/PRs (e.g., “Fix registration dedupe (#123)”).
-- PRs: include a clear description, linked issues, before/after notes for performance changes (see `Tests/Runtime/Benchmarks`), and tests for bug fixes/features.
-- Releasing: changes to `package.json` on `master` may trigger the NPM publish workflow.
+- PRs: include a clear description, linked issues, before/after notes for performance changes (see [Tests/Runtime/Benchmarks](Tests/Runtime/Benchmarks/)), and tests for bug fixes/features.
+- Releasing: changes to [package.json](package.json) on `master` may trigger the NPM publish workflow.
 
 ## Security & Configuration Tips
 
-- Editor analyzer DLLs are copied into the Unity project by `Editor/SetupCscRsp.cs`; do not commit generated DLLs into this repo.
+- Editor analyzer DLLs are copied into the Unity project by [SetupCscRsp.cs](Editor/SetupCscRsp.cs); do not commit generated DLLs into this repo.
 - Keep public APIs minimal and consistent; avoid breaking changes without a major version bump.

package/Docs/Comparisons.md

@@ -28,16 +28,16 @@
 
 ## Performance Benchmarks
 
-These sections are auto-updated by the PlayMode comparison benchmarks in `Tests/Runtime/Benchmarks/ComparisonPerformanceTests.cs`. Run the suite locally to refresh the tables.
+These sections are auto-updated by the PlayMode comparison benchmarks in the [Comparison Performance PlayMode tests](../Tests/Runtime/Benchmarks/ComparisonPerformanceTests.cs). Run the suite locally to refresh the tables.
 
 ### Comparisons (Windows)
 
 | Message Tech                       | Operations / Second | Allocations? |
 | ---------------------------------- | ------------------- | ------------ |
-| DxMessaging (Untargeted) - No-Copy | 14,740,000          | No           |
-| UniRx MessageBroker                | 17,858,000          | No           |
-| MessagePipe (Global)               | 96,052,000          | No           |
-| Zenject SignalBus                  | 2,632,000           | Yes          |
+| DxMessaging (Untargeted) - No-Copy | 14,640,000          | No           |
+| UniRx MessageBroker                | 18,074,000          | No           |
+| MessagePipe (Global)               | 97,824,000          | No           |
+| Zenject SignalBus                  | 2,354,000           | Yes          |
 
 ### Comparisons (macOS)
 

package/Docs/Install.md

@@ -9,7 +9,8 @@ This page helps you install DxMessaging into a Unity 2021.3+ project using the U
 - Unity Package Manager > Add package from git URL...
 - Paste:
 
-```text
+```bash
+# Unity UPM: Add package from git URL...
 https://github.com/wallstop/DxMessaging.git
 ```
 

package/Docs/InterceptorsAndOrdering.md

@@ -36,7 +36,7 @@ new GameEvent().Emit();  // Both DoWork() and ProcessLater() execute
 - Ensures all listeners see a consistent view of the registration state
 - Makes debugging and reasoning about message flow easier
 
-This snapshot behavior is extensively tested in `MutationDuringEmissionTests.cs`.
+This snapshot behavior is extensively tested in the [Mutation During Emission tests](../Tests/Runtime/Core/MutationDuringEmissionTests.cs).
 
 ---
 

package/Docs/Performance.md

@@ -1,6 +1,6 @@
 # Performance Benchmarks
 
-This page is auto-updated by the Unity PlayMode benchmark tests in `Tests/Runtime/Benchmarks/PerformanceTests.cs`.
+This page is auto-updated by the Unity PlayMode benchmark tests in the [Performance PlayMode benchmark suite](../Tests/Runtime/Benchmarks/PerformanceTests.cs).
 
 How it works:
 
@@ -8,21 +8,23 @@ How it works:
 - The benchmark test writes an OS-specific section below with a markdown table.
 - CI runs skip writing to avoid noisy diffs.
 
-See also: `Docs/DesignAndArchitecture.md#performance-optimizations` for design details.
+See also: [Performance optimizations](./DesignAndArchitecture.md#performance-optimizations) for design details.
 
 ## Windows
 
-| Message Tech                       | Operations / Second | Allocations? |
-| ---------------------------------- | ------------------- | ------------ |
-| Unity                              | 2,628,000           | Yes          |
-| DxMessaging (GameObject) - Normal  | 8,208,000           | No           |
-| DxMessaging (Component) - Normal   | 8,210,000           | No           |
-| DxMessaging (GameObject) - No-Copy | 9,412,000           | No           |
-| DxMessaging (Component) - No-Copy  | 9,354,000           | No           |
-| DxMessaging (Untargeted) - No-Copy | 14,812,000          | No           |
-| Reflexive (One Argument)           | 2,856,000           | No           |
-| Reflexive (Two Arguments)          | 2,372,000           | No           |
-| Reflexive (Three Arguments)        | 2,382,000           | No           |
+| Message Tech                               | Operations / Second | Allocations? |
+| ------------------------------------------ | ------------------- | ------------ |
+| Unity                                      | 2,504,000           | Yes          |
+| DxMessaging (GameObject) - Normal          | 8,522,000           | No           |
+| DxMessaging (Component) - Normal           | 8,524,000           | No           |
+| DxMessaging (GameObject) - No-Copy         | 9,398,000           | No           |
+| DxMessaging (Component) - No-Copy          | 9,508,000           | No           |
+| DxMessaging (Untargeted) - No-Copy         | 14,742,000          | No           |
+| DxMessaging (Untargeted) - Interceptors    | 6,620,000           | No           |
+| DxMessaging (Untargeted) - Post-Processors | 5,234,000           | No           |
+| Reflexive (One Argument)                   | 2,840,000           | No           |
+| Reflexive (Two Arguments)                  | 2,352,000           | No           |
+| Reflexive (Three Arguments)                | 2,318,000           | No           |
 
 ## macOS
 

package/Docs/QuickReference.md

@@ -106,7 +106,7 @@ public sealed class DamageSystem : IStartable, IDisposable
 }
 ```
 
-Tip: Define `ZENJECT_PRESENT`, `VCONTAINER_PRESENT`, or `REFLEX_PRESENT` to enable the optional shims under `Runtime/Unity/Integrations/` that bind the builder automatically for those containers.
+Tip: Define `ZENJECT_PRESENT`, `VCONTAINER_PRESENT`, or `REFLEX_PRESENT` to enable the optional shims under [Runtime/Unity/Integrations](../Runtime/Unity/Integrations/) that bind the builder automatically for those containers.
 
 Interceptors and post‑processors
 

package/Docs/Reference.md

@@ -64,11 +64,11 @@ Unity bridge types
 
 Source files (for exploration)
 
-- Message bus interface: Runtime/Core/MessageBus/IMessageBus.cs
-- Message bus: Runtime/Core/MessageBus/MessageBus.cs
-- Message handler/token: Runtime/Core/MessageHandler.cs, Runtime/Core/MessageRegistrationToken.cs
-- Emit helpers: Runtime/Core/Extensions/MessageExtensions.cs
-- Attributes: Runtime/Core/Attributes/\*.cs
+- Message bus interface: [IMessageBus](../Runtime/Core/MessageBus/IMessageBus.cs)
+- Message bus: [MessageBus](../Runtime/Core/MessageBus/MessageBus.cs)
+- Message handler/token: [MessageHandler](../Runtime/Core/MessageHandler.cs), [MessageRegistrationToken](../Runtime/Core/MessageRegistrationToken.cs)
+- Emit helpers: [MessageExtensions](../Runtime/Core/Extensions/MessageExtensions.cs)
+- Attributes: [Attributes directory](../Runtime/Core/Attributes/)
 
 API tables (quick view)
 

package/Editor/Analyzers/Microsoft.CodeAnalysis.CSharp.dll.meta

@@ -11,15 +11,26 @@ PluginImporter:
   isExplicitlyReferenced: 0
   validateReferences: 1
   platformData:
+  - first:
+      : Any
+    second:
+      enabled: 0
+      settings:
+        Exclude Editor: 0
+        Exclude Linux64: 1
+        Exclude OSXUniversal: 1
+        Exclude WebGL: 1
+        Exclude Win: 1
+        Exclude Win64: 1
   - first:
       Any: 
     second:
-      enabled: 1
+      enabled: 0
       settings: {}
   - first:
       Editor: Editor
     second:
-      enabled: 0
+      enabled: 1
       settings:
         DefaultValueInitialized: true
   - first:

package/Editor/Analyzers/Microsoft.CodeAnalysis.dll.meta

@@ -11,6 +11,17 @@ PluginImporter:
   isExplicitlyReferenced: 0
   validateReferences: 1
   platformData:
+  - first:
+      : Any
+    second:
+      enabled: 0
+      settings:
+        Exclude Editor: 0
+        Exclude Linux64: 1
+        Exclude OSXUniversal: 1
+        Exclude WebGL: 1
+        Exclude Win: 1
+        Exclude Win64: 1
   - first:
       Any: 
     second:

package/Editor/Analyzers/System.Collections.Immutable.dll.meta

@@ -11,6 +11,17 @@ PluginImporter:
   isExplicitlyReferenced: 0
   validateReferences: 1
   platformData:
+  - first:
+      : Any
+    second:
+      enabled: 0
+      settings:
+        Exclude Editor: 0
+        Exclude Linux64: 1
+        Exclude OSXUniversal: 1
+        Exclude WebGL: 1
+        Exclude Win: 1
+        Exclude Win64: 1
   - first:
       Any: 
     second:

package/Editor/Analyzers/System.Reflection.Metadata.dll.meta

@@ -11,15 +11,26 @@ PluginImporter:
   isExplicitlyReferenced: 0
   validateReferences: 1
   platformData:
+  - first:
+      : Any
+    second:
+      enabled: 0
+      settings:
+        Exclude Editor: 0
+        Exclude Linux64: 1
+        Exclude OSXUniversal: 1
+        Exclude WebGL: 1
+        Exclude Win: 1
+        Exclude Win64: 1
   - first:
       Any: 
     second:
-      enabled: 1
+      enabled: 0
       settings: {}
   - first:
       Editor: Editor
     second:
-      enabled: 0
+      enabled: 1
       settings:
         DefaultValueInitialized: true
   - first:

package/Editor/Analyzers/System.Runtime.CompilerServices.Unsafe.dll.meta

@@ -11,6 +11,17 @@ PluginImporter:
   isExplicitlyReferenced: 0
   validateReferences: 1
   platformData:
+  - first:
+      : Any
+    second:
+      enabled: 0
+      settings:
+        Exclude Editor: 0
+        Exclude Linux64: 1
+        Exclude OSXUniversal: 1
+        Exclude WebGL: 1
+        Exclude Win: 1
+        Exclude Win64: 1
   - first:
       Any: 
     second:

package/Editor/Analyzers/WallstopStudios.DxMessaging.SourceGenerators.dll.meta

@@ -18,9 +18,10 @@ PluginImporter:
     second:
       enabled: 0
       settings:
-        Exclude Editor: 1
+        Exclude Editor: 0
         Exclude Linux64: 1
         Exclude OSXUniversal: 1
+        Exclude WebGL: 1
         Exclude Win: 1
         Exclude Win64: 1
   - first:
@@ -31,7 +32,7 @@ PluginImporter:
   - first:
       Editor: Editor
     second:
-      enabled: 0
+      enabled: 1
       settings:
         CPU: AnyCPU
         DefaultValueInitialized: true

package/Editor/AssemblyInfo.cs

@@ -0,0 +1,3 @@
+using System.Runtime.CompilerServices;
+
+[assembly: InternalsVisibleTo("WallstopStudios.DxMessaging.Tests.Editor")]

package/Editor/AssemblyInfo.cs.meta

@@ -0,0 +1,3 @@
+fileFormatVersion: 2
+guid: 55bb3ae7ef064633b5fe4b17ce29f8cc
+timeCreated: 1761880591

package/Editor/CustomEditors/MessagingComponentEditor.cs

@@ -7,6 +7,7 @@ namespace DxMessaging.Editor.CustomEditors
     using Core.Diagnostics;
     using Core.MessageBus;
     using Core.Messages;
+    using DxMessaging.Editor.Testing;
     using Unity;
     using UnityEditor;
     using UnityEngine;
@@ -36,6 +37,9 @@ namespace DxMessaging.Editor.CustomEditors
             _listenerFoldouts.Clear();
         }
 
+        /// <summary>
+        /// Draws the custom inspector, including diagnostics summaries and listener management controls.
+        /// </summary>
         public override void OnInspectorGUI()
         {
             base.OnInspectorGUI();
@@ -66,6 +70,26 @@ namespace DxMessaging.Editor.CustomEditors
                 return;
             }
 
+            MessagingComponentInspectorState inspectorState =
+                MessagingComponentEditorHarness.Capture(component);
+            ProviderDiagnosticsView providerDiagnostics = inspectorState.ProviderDiagnostics;
+
+            if (providerDiagnostics.SerializedProviderMissingWarning)
+            {
+                EditorGUILayout.HelpBox(
+                    "Auto-configure serialized provider is enabled, but no provider asset is assigned.",
+                    MessageType.Warning
+                );
+            }
+
+            if (providerDiagnostics.SerializedProviderNullBusWarning)
+            {
+                EditorGUILayout.HelpBox(
+                    "The serialized provider resolved without returning a message bus. Verify the provider implementation.",
+                    MessageType.Warning
+                );
+            }
+
             if (component._registeredListeners.Count == 0)
             {
                 EditorGUILayout.LabelField("No listeners registered.");

package/Editor/DxMessagingEditorInitializer.cs

@@ -1,9 +1,11 @@
 namespace DxMessaging.Editor
 {
 #if UNITY_EDITOR
+    using Core;
     using Core.MessageBus;
     using Settings;
     using UnityEditor;
+    using UnityEngine;
 
     /// <summary>
     /// Applies DxMessaging Editor settings to global runtime defaults on domain load.
@@ -11,12 +13,67 @@ namespace DxMessaging.Editor
     [InitializeOnLoad]
     public static class DxMessagingEditorInitializer
     {
+        private static bool s_playModeWarningIssued;
+
         static DxMessagingEditorInitializer()
         {
+            EditorApplication.playModeStateChanged += OnPlayModeStateChanged;
+            ApplyEditorSettings();
+            WarnIfDomainReloadDisabled();
+        }
+
+        private static void OnPlayModeStateChanged(PlayModeStateChange stateChange)
+        {
+            if (
+                stateChange == PlayModeStateChange.EnteredEditMode
+                || stateChange == PlayModeStateChange.EnteredPlayMode
+            )
+            {
+                ApplyEditorSettings();
+                WarnIfDomainReloadDisabled();
+            }
+        }
+
+        [RuntimeInitializeOnLoadMethod(RuntimeInitializeLoadType.BeforeSceneLoad)]
+        private static void ApplySettingsBeforeSceneLoad()
+        {
+            if (!Application.isPlaying)
+            {
+                return;
+            }
+
+            ApplyEditorSettings();
+        }
+
+        private static void ApplyEditorSettings()
+        {
+            DxMessagingStaticState.Reset();
             DxMessagingSettings settings = DxMessagingSettings.GetOrCreateSettings();
-            IMessageBus.GlobalDiagnosticsMode = settings.EnableDiagnosticsInEditor;
+            IMessageBus.GlobalDiagnosticsTargets = settings.DiagnosticsTargets;
             IMessageBus.GlobalMessageBufferSize = settings.MessageBufferSize;
         }
+
+        private static void WarnIfDomainReloadDisabled()
+        {
+            DxMessagingSettings settings = DxMessagingSettings.GetOrCreateSettings();
+            if (
+                s_playModeWarningIssued
+                || settings.SuppressDomainReloadWarning
+                || !EditorSettings.enterPlayModeOptionsEnabled
+                || (EditorSettings.enterPlayModeOptions & EnterPlayModeOptions.DisableDomainReload)
+                    == 0
+            )
+            {
+                return;
+            }
+
+            s_playModeWarningIssued = true;
+            Debug.LogWarning(
+                "[DxMessaging] Enter Play Mode Options are disabling domain reload. "
+                    + "DxMessaging resets its internal statics, but third-party static state will persist. "
+                    + "Audit integration code or re-enable domain reload if inconsistent behaviour occurs."
+            );
+        }
     }
 #endif
 }

package/Editor/DxMessagingMenu.cs

@@ -0,0 +1,38 @@
+namespace DxMessaging.Editor
+{
+#if UNITY_EDITOR
+    using Core;
+    using Core.MessageBus;
+    using UnityEditor;
+    using UnityEngine;
+
+    internal static class DxMessagingMenu
+    {
+        private const string Root = "Tools/Wallstop Studios/DxMessaging/";
+
+        [MenuItem(Root + "Reset Static State")]
+        private static void ResetStatics()
+        {
+            DxMessagingStaticState.Reset();
+            Debug.Log("[DxMessaging] Static state reset.");
+        }
+
+        [MenuItem(Root + "Toggle Global Diagnostics")]
+        private static void ToggleGlobalDiagnostics()
+        {
+            DiagnosticsTarget current = IMessageBus.GlobalDiagnosticsTargets;
+            DiagnosticsTarget next = current switch
+            {
+                DiagnosticsTarget.Off => DiagnosticsTarget.Editor,
+                DiagnosticsTarget.Editor => DiagnosticsTarget.Runtime,
+                DiagnosticsTarget.Runtime => DiagnosticsTarget.All,
+                DiagnosticsTarget.All => DiagnosticsTarget.Off,
+                _ => DiagnosticsTarget.Off,
+            };
+
+            IMessageBus.GlobalDiagnosticsTargets = next;
+            Debug.Log($"[DxMessaging] Global diagnostics targets set to {next}.");
+        }
+    }
+#endif
+}

package/Editor/DxMessagingMenu.cs.meta

@@ -0,0 +1,11 @@
+fileFormatVersion: 2
+guid: 517da34a8f694371b843ba34f89077d3
+MonoImporter:
+  externalObjects: {}
+  serializedVersion: 2
+  defaultReferences: []
+  executionOrder: 0
+  icon: {instanceID: 0}
+  userData: 
+  assetBundleName: 
+  assetBundleVariant: