com.wallstop-studios.dxmessaging 2.1.7 → 2.1.8

package/AGENTS.md

@@ -1,3 +0,0 @@
-# Repository Guidelines
-
-See the [AI Agent Guidelines](./.llm/context.md) for all AI agent guidelines.

package/AGENTS.md.meta

@@ -1,7 +0,0 @@
-fileFormatVersion: 2
-guid: d2b5565e2c69b104e9d8587c870fa762
-TextScriptImporter:
-  externalObjects: {}
-  userData: 
-  assetBundleName: 
-  assetBundleVariant: 

package/CLAUDE.md

@@ -1,3 +0,0 @@
-# Claude Configuration
-
-See the [AI Agent Guidelines](./.llm/context.md) for all AI agent guidelines.

package/CLAUDE.md.meta

@@ -1,7 +0,0 @@
-fileFormatVersion: 2
-guid: 8a3f7c41e5b9d026f1c4a8e372d9b5f0
-TextScriptImporter:
-  externalObjects: {}
-  userData: 
-  assetBundleName: 
-  assetBundleVariant: 

package/CONTRIBUTING.md

@@ -1,43 +0,0 @@
-# Contributing
-
-Thanks for helping improve DxMessaging!
-
-Before committing, please enable our git hooks and local linters so you catch issues early:
-
-- Install pre-commit: `pip install pre-commit` or `pipx install pre-commit`
-- Install hooks: `pre-commit install`
-- Run on all files: `pre-commit run --all-files`
-
-Line endings: Git normalizes most text files to CRLF through `.gitattributes`. **Exception:** Shell scripts (`.sh`, `.bash`, `.zsh`, `.ksh`, `.fish`) use LF for Unix compatibility. Run this once after cloning (especially on Windows) to fix your working tree:
-
-```bash
-git config core.autocrlf false
-node scripts/fix-eol.js
-```
-
-This directly converts files in your working directory to the correct line endings. Add `-v` for verbose output showing each file fixed.
-
-> **Note:** You may see references to `git add --renormalize`, but that command only updates the git index (staging area)—it does **not** modify your working tree files. Use `fix-eol.js` to actually fix files on disk.
-
-What runs locally:
-
-- Markdown link text check: enforces human-readable link text (no raw file names/paths)
-- Internal markdown link validity: verifies relative links and anchors point to real files/sections
-- Markdown style and formatting: markdownlint (auto-fix common issues)
-- JSON/.asmdef formatting: Prettier (2-space indent)
-- YAML formatting: Prettier (2-space indent) + yamllint
-
-On pull requests, CI also checks all markdown links (including external URLs) with lychee.
-
-Handy commands:
-
-- Internal links (local): `python .github/scripts/validate_markdown_links.py .`
-- Lint markdown (all files): `pre-commit run markdownlint --all-files`
-- Lint markdown (manual): `npx markdownlint-cli2@0.20.0 "**/*.md" --fix`
-- Format JSON/.asmdef (all files): `pre-commit run prettier --all-files`
-- Format JSON/.asmdef (manual): `npx prettier@3.8.1 --write "**/*.{json,asmdef}"`
-- Format YAML (all files): `pre-commit run prettier-yaml --all-files`
-- Check YAML formatting + lint: `npm run check:yaml`
-- Format C#: `dotnet tool restore && dotnet tool run csharpier format`
-
-If you do still need to repair line endings manually (for example, after copying files from an external tool), run `node scripts/fix-eol.js -v` and then re-stage the affected files.

package/CONTRIBUTING.md.meta

@@ -1,7 +0,0 @@
-fileFormatVersion: 2
-guid: 41a84e4c0c6e64b4aa1b887ae4ad61c6
-TextScriptImporter:
-  externalObjects: {}
-  userData: 
-  assetBundleName: 
-  assetBundleVariant: 

package/SourceGenerators/WallstopStudios.DxMessaging.SourceGenerators.Tests/DocsSnippetCompilationTests.cs

@@ -1,544 +0,0 @@
-using System.Collections.Generic;
-using System.IO;
-using System.Linq;
-using NUnit.Framework;
-
-namespace WallstopStudios.DxMessaging.SourceGenerators.Tests;
-
-[TestFixture]
-public sealed class DocsSnippetCompilationTests
-{
-    private static readonly HashSet<string> IgnoredSnippetDiagnosticIds = new(
-        StringComparer.OrdinalIgnoreCase
-    )
-    {
-        "CS0106", // modifier not valid in script (partial snippets showing members only)
-        "CS1001", // identifier expected (intentionally elided samples)
-        "CS8803", // top-level statements mixed with declarations (visual guide style snippets)
-    };
-
-    // Compiled regex patterns for API signature detection
-    private static readonly System.Text.RegularExpressions.Regex MethodSignatureStartRegex = new(
-        @"^\w+(?:<[^>]+>)?\s+\w+(?:<[^>]+>)?\s*\($",
-        System.Text.RegularExpressions.RegexOptions.Compiled
-    );
-
-    private static readonly System.Text.RegularExpressions.Regex DocumentationStyleMethodCallRegex =
-        new(
-            @"\(\s*[A-Z]\w*\s+\w+\s*\)[\s;]*$",
-            System.Text.RegularExpressions.RegexOptions.Compiled
-        );
-
-    private static readonly System.Text.RegularExpressions.Regex GenericMethodCallRegex = new(
-        @"<T>\s*\(",
-        System.Text.RegularExpressions.RegexOptions.Compiled
-    );
-
-    private static readonly System.Text.RegularExpressions.Regex HandlerParameterRegex = new(
-        @",\s*handler\s*,",
-        System.Text.RegularExpressions.RegexOptions.Compiled
-    );
-
-    private static readonly System.Text.RegularExpressions.Regex ApiReferenceUnityTypeRegex = new(
-        @"\(\s*(GameObject|Component|InstanceId)\s+\w+\s*,",
-        System.Text.RegularExpressions.RegexOptions.Compiled
-    );
-
-    private static readonly System.Text.RegularExpressions.Regex ApiReferenceHandlerPriorityRegex =
-        new(
-            @",\s*handler\s*,\s*(int|bool|string)\s+\w+\s*=",
-            System.Text.RegularExpressions.RegexOptions.Compiled
-        );
-
-    // Constants for signature detection thresholds
-    private const double ApiSignatureRatioThreshold = 0.30;
-    private const int MinimumApiSignatureLinesForDetection = 2;
-
-    [Test]
-    public void QuickStartStep1Compiles()
-    {
-        string docsRoot = ResolveDocsRoot();
-        string quickStartPath = Path.Combine(docsRoot, "getting-started", "quick-start.md");
-        Assert.That(File.Exists(quickStartPath), Is.True, $"Unable to locate {quickStartPath}.");
-
-        string snippet = ExtractFirstCodeBlock(quickStartPath, "csharp");
-        Assert.That(!string.IsNullOrWhiteSpace(snippet), Is.True, "QuickStart snippet not found.");
-
-        string source = $"""
-using DxMessaging.Core.Messages;
-using DxMessaging.Core.Attributes;
-using UnityEngine;
-
-{snippet}
-""";
-
-        var diagnostics = GeneratorTestUtilities
-            .CompileSnippet(source)
-            .Where(d => d.Severity == Microsoft.CodeAnalysis.DiagnosticSeverity.Error)
-            .ToArray();
-
-        if (diagnostics.Length > 0)
-        {
-            string message = string.Join(
-                System.Environment.NewLine,
-                diagnostics.Select(d => d.ToString())
-            );
-            Assert.Fail(
-                $"QuickStart snippet failed to compile:{System.Environment.NewLine}{message}"
-            );
-        }
-    }
-
-    [TestCaseSource(nameof(GetDocumentationSnippets))]
-    public void DocumentationSnippetsCompile(string markdownPath, string snippet)
-    {
-        Assert.That(
-            snippet,
-            Is.Not.Empty,
-            $"Snippet extracted from {markdownPath} should not be empty."
-        );
-
-        var diagnostics = GeneratorTestUtilities
-            .ParseSnippet(snippet)
-            .Where(d => d.Severity == Microsoft.CodeAnalysis.DiagnosticSeverity.Error)
-            .ToArray();
-
-        var actionableDiagnostics = diagnostics
-            .Where(d => !IgnoredSnippetDiagnosticIds.Contains(d.Id))
-            .ToArray();
-
-        if (actionableDiagnostics.Length > 0)
-        {
-            string message = string.Join(
-                System.Environment.NewLine,
-                actionableDiagnostics.Select(d => d.ToString())
-            );
-            Assert.Fail(
-                $"Documentation snippet in {markdownPath} failed to compile:{System.Environment.NewLine}{message}"
-            );
-        }
-    }
-
-    private static IEnumerable<TestCaseData> GetDocumentationSnippets()
-    {
-        string docsRoot = ResolveDocsRoot();
-        foreach (
-            string markdownPath in Directory.GetFiles(docsRoot, "*.md", SearchOption.AllDirectories)
-        )
-        {
-            foreach (string snippet in ExtractCodeBlocks(markdownPath, "csharp"))
-            {
-                if (ShouldSkipSnippet(snippet))
-                {
-                    continue;
-                }
-
-                yield return new TestCaseData(markdownPath, snippet).SetName(
-                    $"{Path.GetFileName(markdownPath)} compiles"
-                );
-            }
-        }
-    }
-
-    private static string ExtractFirstCodeBlock(string markdownPath, string infoString)
-    {
-        return ExtractCodeBlocks(markdownPath, infoString).FirstOrDefault() ?? string.Empty;
-    }
-
-    private static IEnumerable<string> ExtractCodeBlocks(string markdownPath, string infoString)
-    {
-        string[] lines = File.ReadAllLines(markdownPath);
-        bool inBlock = false;
-        System.Text.StringBuilder builder = new();
-        foreach (string rawLine in lines)
-        {
-            string line = rawLine.TrimEnd();
-            if (!inBlock)
-            {
-                if (line.StartsWith("```") && line.Length > 3 && line[3..].StartsWith(infoString))
-                {
-                    inBlock = true;
-                    builder.Clear();
-                }
-                continue;
-            }
-
-            if (line.StartsWith("```"))
-            {
-                inBlock = false;
-                string snippet = builder.ToString();
-                if (!string.IsNullOrWhiteSpace(snippet))
-                {
-                    yield return snippet;
-                }
-                continue;
-            }
-
-            builder.AppendLine(rawLine);
-        }
-    }
-
-    private static string ResolveDocsRoot()
-    {
-        string currentDirectoryPath = TestContext.CurrentContext.TestDirectory;
-        while (!string.IsNullOrEmpty(currentDirectoryPath))
-        {
-            string docsDirectory = Path.Combine(currentDirectoryPath, "docs");
-            string candidate = Path.Combine(docsDirectory, "getting-started", "quick-start.md");
-            if (File.Exists(candidate))
-            {
-                return docsDirectory;
-            }
-
-            string parentDirectoryPath =
-                Path.GetDirectoryName(currentDirectoryPath) ?? string.Empty;
-            if (string.IsNullOrEmpty(parentDirectoryPath))
-            {
-                break;
-            }
-
-            currentDirectoryPath = parentDirectoryPath;
-        }
-
-        throw new FileNotFoundException(
-            "Unable to locate docs/getting-started/quick-start.md from the current test directory."
-        );
-    }
-
-    [TestCase("", true, TestName = "Empty snippet should be skipped")]
-    [TestCase("   ", true, TestName = "Whitespace-only snippet should be skipped")]
-    [TestCase(
-        "var x = 1;\nConsole.WriteLine(x);",
-        false,
-        TestName = "Regular compilable code should not be skipped"
-    )]
-    [TestCase(
-        "public class MyClass { }",
-        false,
-        TestName = "Simple class declaration should not be skipped"
-    )]
-    [TestCase(
-        "// Comment only\nvar x = 1;",
-        false,
-        TestName = "Code with comments should not be skipped"
-    )]
-    [TestCase(
-        "MessageRegistrationHandle RegisterUntargeted<T>(Action<T> handler, int priority = 0)",
-        true,
-        TestName = "Single line method signature with default parameter should be skipped"
-    )]
-    [TestCase(
-        "void Process(string name = null)",
-        true,
-        TestName = "Method with null default should be skipped"
-    )]
-    [TestCase(
-        "bool IsEnabled(bool flag = false)",
-        true,
-        TestName = "Method with false default should be skipped"
-    )]
-    [TestCase(
-        "void Toggle(bool active = true)",
-        true,
-        TestName = "Method with true default should be skipped"
-    )]
-    [TestCase(
-        "MessageRegistrationHandle RegisterUntargeted<T>(\n    Action<T> handler,\n    int priority = 0)",
-        true,
-        TestName = "Multi-line signature with default params should be skipped"
-    )]
-    [TestCase(
-        "Do something...\nthen continue",
-        true,
-        TestName = "Snippet with ellipsis should be skipped"
-    )]
-    [TestCase(
-        "public void Method() { ... }",
-        true,
-        TestName = "Method with ellipsis body should be skipped"
-    )]
-    [TestCase(
-        "MessageRegistrationHandle RegisterUntargeted<T>(\n    Action<T> handler,\n    int priority = 0\n)",
-        true,
-        TestName = "Multi-line signature with closing paren on own line should be skipped"
-    )]
-    [TestCase(
-        "    // GameObject target\n    MessageRegistrationHandle RegisterGameObjectTargeted<T>(\n        GameObject target,\n        Action<T> handler,\n        int priority = 0\n    )",
-        true,
-        TestName = "Indented multi-line signature with comments should be skipped"
-    )]
-    [TestCase(
-        "// Emit to specific target (by InstanceId)\nmessage.EmitTargeted(InstanceId target);",
-        true,
-        TestName = "Method call with parameter type documentation should be skipped"
-    )]
-    [TestCase(
-        "Action RegisterUntargetedInterceptor<T>(\n    UntargetedInterceptor<T> interceptor,\n    int priority = 0\n)",
-        true,
-        TestName = "Interceptor signature should be skipped"
-    )]
-    [TestCase(
-        "token.RegisterUntargeted<T>(Action<T> handler, int priority = 0)",
-        true,
-        TestName = "Method call with generic type parameter and Action handler should be skipped"
-    )]
-    [TestCase(
-        "token.RegisterUntargeted<T>(FastHandler<T> handler, int priority = 0)",
-        true,
-        TestName = "Method call with FastHandler parameter should be skipped"
-    )]
-    [TestCase(
-        "token.RegisterGameObjectTargeted<T>(GameObject go, handler, int priority = 0)",
-        true,
-        TestName = "Method call with handler parameter name should be skipped"
-    )]
-    [TestCase(
-        "token.RegisterTargetedWithoutTargeting<T>(FastHandlerWithContext<T> handler, int priority = 0)",
-        true,
-        TestName = "Method call with FastHandlerWithContext should be skipped"
-    )]
-    [TestCase(
-        "bus.RegisterUntargetedInterceptor<T>(UntargetedInterceptor<T> interceptor, int priority = 0)",
-        true,
-        TestName = "Bus interceptor registration with generic type should be skipped"
-    )]
-    [TestCase(
-        "token.RegisterTargeted<T>(InstanceId id, handler, int priority = 0)",
-        true,
-        TestName = "Method with InstanceId parameter and handler should be skipped"
-    )]
-    [TestCase(
-        "token.RegisterBroadcast<T>(InstanceId id, handler, int priority = 0)",
-        true,
-        TestName = "Broadcast registration with InstanceId and handler should be skipped"
-    )]
-    [TestCase("int x = 0;", false, TestName = "Assignment with zero should not be skipped")]
-    [TestCase("var priority = 0;", false, TestName = "Variable assignment should not be skipped")]
-    [TestCase(
-        "int priority = 0;\nConsole.WriteLine(priority);",
-        false,
-        TestName = "Variable assignment with usage should not be skipped"
-    )]
-    [TestCase(
-        "bool isEnabled = false;\nif (isEnabled) { DoSomething(); }",
-        false,
-        TestName = "Boolean assignment with conditional should not be skipped"
-    )]
-    [TestCase(
-        "string name = null;\nname = GetName();",
-        false,
-        TestName = "Null assignment with reassignment should not be skipped"
-    )]
-    [TestCase(
-        "public void Process()\n{\n    int count = 0;\n}",
-        false,
-        TestName = "Method with local variable initialization should not be skipped"
-    )]
-    [TestCase(
-        "Action<int> handler = x => Console.WriteLine(x);",
-        false,
-        TestName = "Lambda assignment should not be skipped"
-    )]
-    [TestCase(
-        "var result = Calculate(value, 0);",
-        false,
-        TestName = "Method call with zero argument should not be skipped"
-    )]
-    public void ShouldSkipSnippetDetectsApiSignatures(string snippet, bool expectedSkip)
-    {
-        bool actualSkip = ShouldSkipSnippet(snippet);
-        Assert.That(
-            actualSkip,
-            Is.EqualTo(expectedSkip),
-            $"Expected ShouldSkipSnippet to return {expectedSkip} for snippet: '{snippet.Replace("\n", "\\n")}'"
-        );
-    }
-
-    private static bool ShouldSkipSnippet(string snippet)
-    {
-        if (string.IsNullOrWhiteSpace(snippet))
-        {
-            return true;
-        }
-
-        if (snippet.Contains("..."))
-        {
-            return true;
-        }
-
-        foreach (char c in snippet)
-        {
-            if (c > 127 && !char.IsWhiteSpace(c))
-            {
-                return true;
-            }
-        }
-
-        if (IsApiSignatureDocumentation(snippet))
-        {
-            return true;
-        }
-
-        return false;
-    }
-
-    /// <summary>
-    /// Detects whether a code snippet represents API signature documentation rather than
-    /// compilable sample code. API signatures in documentation (e.g., method signatures
-    /// with default parameters shown for reference) are not meant to be compiled.
-    /// </summary>
-    /// <remarks>
-    /// <para>
-    /// API signature snippets need special handling because documentation often shows
-    /// method signatures like <c>RegisterUntargeted&lt;T&gt;(Action&lt;T&gt; handler, int priority = 0)</c>
-    /// for reference. These are informational and not valid standalone C# code.
-    /// </para>
-    /// <para>
-    /// Detection heuristics:
-    /// <list type="bullet">
-    /// <item><description>Default parameter patterns on same line: <c>= 0)</c>, <c>= null)</c>, <c>= false)</c>, <c>= true)</c> and their comma variants</description></item>
-    /// <item><description>Default parameter patterns at end of line: <c>= 0</c>, <c>= null</c>, <c>= false</c>, <c>= true</c> for multi-line signatures</description></item>
-    /// <item><description>Method signature start patterns: return type followed by method name and opening paren</description></item>
-    /// <item><description>Partial signature patterns: lines ending in comma with parameter type keywords (Action&lt;&gt;, Func&lt;&gt;, int, bool, string, GameObject, Component, InstanceId, FastHandler)</description></item>
-    /// <item><description>Isolated closing parens: lines that are just <c>)</c> or <c>);</c></description></item>
-    /// <item><description>Documentation-style method calls: <c>method(TypeName param);</c> showing parameter types as placeholders</description></item>
-    /// <item><description>Generic method calls with type parameters: <c>method&lt;T&gt;(Action&lt;T&gt; handler, ...)</c> with generic handler types</description></item>
-    /// <item><description>API reference parameter lines: <c>(GameObject go, handler, int priority = 0)</c> with type-name pairs and handler placeholders</description></item>
-    /// </list>
-    /// </para>
-    /// <para>
-    /// Ratio threshold logic: A snippet is considered an API signature if:
-    /// <list type="bullet">
-    /// <item><description>At least 30% of non-empty lines match signature patterns, OR</description></item>
-    /// <item><description>Two or more lines match signature patterns (handles multi-line signatures)</description></item>
-    /// </list>
-    /// </para>
-    /// </remarks>
-    /// <param name="snippet">The code snippet to analyze.</param>
-    /// <returns>True if the snippet appears to be API signature documentation; otherwise false.</returns>
-    private static bool IsApiSignatureDocumentation(string snippet)
-    {
-        string[] lines = snippet.Split('\n');
-        int signatureLineCount = 0;
-        int totalNonEmptyLines = 0;
-
-        foreach (string rawLine in lines)
-        {
-            string line = rawLine.Trim();
-            if (string.IsNullOrEmpty(line))
-            {
-                continue;
-            }
-
-            totalNonEmptyLines++;
-
-            // Default parameters on same line as closing paren or comma
-            bool hasDefaultParameterSameLine =
-                line.Contains("= 0)")
-                || line.Contains("= null)")
-                || line.Contains("= false)")
-                || line.Contains("= true)")
-                || line.Contains("= 0,")
-                || line.Contains("= null,")
-                || line.Contains("= false,")
-                || line.Contains("= true,");
-
-            // Default parameters on their own line (multi-line signatures)
-            bool hasDefaultParameterEndOfLine =
-                line.EndsWith("= 0")
-                || line.EndsWith("= null")
-                || line.EndsWith("= false")
-                || line.EndsWith("= true");
-
-            // Method signature that starts with return type + method name pattern
-            // e.g., "MessageRegistrationHandle RegisterUntargeted<T>("
-            bool isMethodSignatureStart =
-                !line.StartsWith("//")
-                && !line.StartsWith("/*")
-                && !line.StartsWith("var ")
-                && !line.Contains(" = ")
-                && line.EndsWith("(")
-                && MethodSignatureStartRegex.IsMatch(line);
-
-            bool isPartialSignature =
-                line.EndsWith(",")
-                && !line.StartsWith("//")
-                && (
-                    line.Contains("Action<")
-                    || line.Contains("Func<")
-                    || line.Contains("int ")
-                    || line.Contains("bool ")
-                    || line.Contains("string ")
-                    || line.Contains("GameObject ")
-                    || line.Contains("Component ")
-                    || line.Contains("InstanceId ")
-                    || line.Contains("FastHandler")
-                );
-
-            // Isolated closing paren (multi-line signature endings)
-            bool isIsolatedClosingParen = line == ")" || line == ");";
-
-            // Method calls with type+parameter documentation style
-            // e.g., "message.EmitTargeted(InstanceId target);"
-            bool isDocumentationStyleMethodCall =
-                !line.StartsWith("//")
-                && !line.StartsWith("/*")
-                && (line.EndsWith(");") || line.EndsWith(")"))
-                && line.Contains("(")
-                && DocumentationStyleMethodCallRegex.IsMatch(line);
-
-            // Generic method calls with <T>( pattern that are API reference style
-            // e.g., "token.RegisterUntargeted<T>(Action<T> handler, int priority = 0)"
-            bool isGenericMethodCallWithTypeParams =
-                !line.StartsWith("//")
-                && !line.StartsWith("/*")
-                && GenericMethodCallRegex.IsMatch(line)
-                && (
-                    line.Contains("Action<T>")
-                    || line.Contains("FastHandler")
-                    || line.Contains("Interceptor<T>")
-                    || line.Contains("FastHandlerWithContext")
-                    || HandlerParameterRegex.IsMatch(line)
-                );
-
-            // API reference lines with parameter declarations showing type and name
-            // e.g., "(GameObject go, handler, int priority = 0)" or "(InstanceId id, handler,"
-            bool isApiReferenceParameterLine =
-                !line.StartsWith("//")
-                && !line.StartsWith("/*")
-                && line.Contains("(")
-                && (
-                    ApiReferenceUnityTypeRegex.IsMatch(line)
-                    || ApiReferenceHandlerPriorityRegex.IsMatch(line)
-                );
-
-            if (
-                hasDefaultParameterSameLine
-                || hasDefaultParameterEndOfLine
-                || isMethodSignatureStart
-                || isPartialSignature
-                || isIsolatedClosingParen
-                || isDocumentationStyleMethodCall
-                || isGenericMethodCallWithTypeParams
-                || isApiReferenceParameterLine
-            )
-            {
-                signatureLineCount++;
-            }
-        }
-
-        if (signatureLineCount > 0 && totalNonEmptyLines > 0)
-        {
-            double ratio = (double)signatureLineCount / totalNonEmptyLines;
-            if (
-                ratio >= ApiSignatureRatioThreshold
-                || signatureLineCount >= MinimumApiSignatureLinesForDetection
-            )
-            {
-                return true;
-            }
-        }
-
-        return false;
-    }
-}

package/SourceGenerators/WallstopStudios.DxMessaging.SourceGenerators.Tests/DocsSnippetCompilationTests.cs.meta

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