com.wallstop-studios.dxmessaging 2.0.0-rc27.2 → 2.0.0-rc27.3

package/.github/workflows/csharpier-autofix.yml

@@ -100,7 +100,7 @@ jobs:
 
       - name: Open or update formatting PR in base repo
         if: steps.changes.outputs.has_changes == 'true'
-        uses: actions/github-script@v7
+        uses: actions/github-script@v8
         with:
           script: |
             const prNumber = context.payload.pull_request.number;
@@ -126,7 +126,7 @@ jobs:
 
       - name: Comment link on original PR
         if: steps.changes.outputs.has_changes == 'true'
-        uses: actions/github-script@v7
+        uses: actions/github-script@v8
         with:
           script: |
             const prNumber = context.payload.pull_request.number;

package/.github/workflows/format-on-demand.yml

@@ -25,7 +25,7 @@ jobs:
     steps:
       - name: Resolve PR metadata and authorize request
         id: meta
-        uses: actions/github-script@v7
+        uses: actions/github-script@v8
         with:
           script: |
             const prNumber = context.payload.issue.number;
@@ -138,7 +138,7 @@ jobs:
 
       - name: Open/Update formatting PR (fork)
         if: ${{ steps.meta.outputs.same_repo != 'true' && steps.changes.outputs.has_changes == 'true' }}
-        uses: actions/github-script@v7
+        uses: actions/github-script@v8
         with:
           script: |
             const prNumber = Number(core.getInput('pr')) || Number('${{ steps.meta.outputs.pr }}');
@@ -159,7 +159,7 @@ jobs:
 
       - name: Comment result on PR
         if: ${{ steps.changes.outputs.has_changes == 'true' }}
-        uses: actions/github-script@v7
+        uses: actions/github-script@v8
         with:
           script: |
             const prNumber = Number('${{ steps.meta.outputs.pr }}');
@@ -171,7 +171,7 @@ jobs:
 
       - name: No-op comment (nothing to change)
         if: ${{ steps.changes.outputs.has_changes != 'true' }}
-        uses: actions/github-script@v7
+        uses: actions/github-script@v8
         with:
           script: |
             const prNumber = Number('${{ steps.meta.outputs.pr }}');
@@ -185,7 +185,7 @@ jobs:
     steps:
       - name: Resolve PR metadata
         id: meta
-        uses: actions/github-script@v7
+        uses: actions/github-script@v8
         with:
           script: |
             const prNumber = Number(core.getInput('pr_number'));
@@ -284,7 +284,7 @@ jobs:
 
       - name: Open/Update formatting PR (fork)
         if: ${{ steps.meta.outputs.same_repo != 'true' && steps.changes.outputs.has_changes == 'true' }}
-        uses: actions/github-script@v7
+        uses: actions/github-script@v8
         with:
           script: |
             const prNumber = Number('${{ steps.meta.outputs.pr }}');

package/.github/workflows/markdown-link-text-check.yml

@@ -12,7 +12,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
-        uses: actions/checkout@v4
+        uses: actions/checkout@v5
 
       - name: Validate link text for markdown-to-markdown links
         run: python .github/scripts/check_markdown_links.py .

package/.github/workflows/markdown-link-validity.yml

@@ -12,7 +12,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
-        uses: actions/checkout@v4
+        uses: actions/checkout@v5
 
       - name: Run lychee link checker (repo-wide)
         uses: lycheeverse/lychee-action@v2

package/.github/workflows/prettier-autofix.yml

@@ -44,7 +44,11 @@ jobs:
 
       - name: Markdownlint (auto-fix)
         if: ${{ github.event.pull_request.user.login == 'dependabot[bot]' }}
-        run: npx markdownlint "**/*.md" "**/*.markdown" --config .markdownlint.json --ignore-path .markdownlintignore --fix
+        run: npx --yes markdownlint-cli@0.40.0 "**/*.md" "**/*.markdown" --config .markdownlint.json --ignore-path .markdownlintignore --fix
+
+      - name: Fix EOL/BOM (Dependabot only)
+        if: ${{ github.event.pull_request.user.login == 'dependabot[bot]' }}
+        run: node scripts/fix-eol.js -v
 
       - name: Commit formatting changes to PR branch (Dependabot only)
         if: ${{ github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name == github.repository && github.event.pull_request.user.login == 'dependabot[bot]' }}
@@ -112,7 +116,10 @@ jobs:
           npm run format:yaml
 
       - name: Markdownlint (auto-fix)
-        run: npx markdownlint "**/*.md" "**/*.markdown" --config .markdownlint.json --ignore-path .markdownlintignore --fix
+        run: npx --yes markdownlint-cli@0.40.0 "**/*.md" "**/*.markdown" --config .markdownlint.json --ignore-path .markdownlintignore --fix
+
+      - name: Fix EOL/BOM
+        run: node scripts/fix-eol.js -v
 
       - name: Detect changes
         id: changes
@@ -146,7 +153,7 @@ jobs:
 
       - name: Open or update formatting PR in base repo
         if: steps.changes.outputs.has_changes == 'true'
-        uses: actions/github-script@v7
+        uses: actions/github-script@v8
         with:
           script: |
             const prNumber = context.payload.pull_request.number;
@@ -171,7 +178,7 @@ jobs:
 
       - name: Comment link on original PR
         if: steps.changes.outputs.has_changes == 'true'
-        uses: actions/github-script@v7
+        uses: actions/github-script@v8
         with:
           script: |
             const prNumber = context.payload.pull_request.number;

package/.markdownlint-cli2.jsonc

@@ -1,4 +1,7 @@
 {
+  // Apply automatic fixes when available
+  "fix": true,
+
   // Limit linting impact until large, single-line docs are reformatted
   "ignores": [
     // Exclude third-party and generated content

package/.pre-commit-config.yaml

@@ -25,11 +25,20 @@ repos:
     hooks:
       - id: prettier
         name: Prettier (Markdown, JSON, asmdef, asmref, YAML)
-        entry: npx --yes prettier@3.3.3 --write
+        entry: npx --yes prettier@3.6.2 --write
         language: system
         files: '(?i)\.(md|markdown|json|asmdef|asmref|ya?ml)$'
         description: Use pinned Prettier version to match CI.
 
+  - repo: local
+    hooks:
+      - id: markdown-structure-fix
+        name: Markdown structure auto-fix (headings, fences)
+        entry: node scripts/fix-md036-headings.js
+        language: system
+        files: '(?i)\.(md|markdown)$'
+        description: Auto-fix MD036 (no-emphasis-as-heading) by converting bold-only lines to headings.
+
   - repo: local
     hooks:
       - id: markdownlint
@@ -48,6 +57,14 @@ repos:
 
   - repo: local
     hooks:
+      - id: eol-bom-fix
+        name: Auto-fix CRLF and remove BOM
+        entry: node scripts/fix-eol.js -v
+        language: system
+        pass_filenames: false
+        stages:
+          - pre-commit
+        description: Convert line endings to CRLF and strip UTF-8 BOM on text files.
       - id: eol-bom-check
         name: Enforce CRLF and no BOM
         entry: node scripts/check-eol.js
@@ -56,3 +73,13 @@ repos:
         stages:
           - pre-commit
           - pre-push
+
+  - repo: local
+    hooks:
+      - id: conflict-markers
+        name: Fail on merge conflict markers
+        entry: bash -lc 'if rg -n -S -g "!**/.git/**" -e "^(<<<<<<<|=======|>>>>>>>)" .; then echo "Conflict markers found. Resolve merges before committing." >&2; exit 1; fi'
+        language: system
+        pass_filenames: false
+        stages:
+          - pre-commit

package/AGENTS.md

@@ -23,6 +23,7 @@
 - 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.
 
 ## Testing Guidelines
 
@@ -36,6 +37,7 @@
 - Do not use Description annotations for tests.
 - Do not create `async Task` test methods - the Unity test runner does not support this. Make do with `IEnumerator` based UnityTestMethods.
 - Do not use `Assert.ThrowsAsync`, it does not exist.
+- When asserting that UnityEngine.Objects are null or not null, please check for null directly (thing != null, thing == null), to properly adhere to Unity Object existence checks.
 
 ## Commit & Pull Request Guidelines
 

package/CONTRIBUTING.md

@@ -25,8 +25,8 @@ 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-cli@0.41.0 "**/*.md" -c .markdownlint.jsonc --fix`
-- Format JSON/.asmdef (all files): `pre-commit run prettier-json --all-files`
-- Format JSON/.asmdef (manual): `npx prettier@3.3.3 --write "**/*.{json,asmdef}"`
+- Format JSON/.asmdef (all files): `pre-commit run prettier --all-files`
+- Format JSON/.asmdef (manual): `npx prettier@3.6.2 --write "**/*.{json,asmdef}"`
 - Format YAML (all files): `pre-commit run prettier-yaml --all-files`
 - Check YAML formatting + lint: `npm run check:yaml`
 - Fix EOL/BOM issues quickly: `node scripts/fix-eol.js -v`

package/Docs/Comparisons.md

@@ -358,7 +358,7 @@ Add/remove systems without affecting others
 - ✅ Observability - Count messages, inspect payloads easily
 - ✅ Determinism - Priority-based ordering eliminates flakiness
 
-**Example:**
+##### Example
 
 ```csharp
 // Test with isolated bus

package/Docs/DesignAndArchitecture.md

@@ -18,31 +18,26 @@ This document explains DxMessaging’s internal design, performance optimization
 DxMessaging was built with these principles:
 
 1. Zero‑Allocation Communication
-
    - Messages are `readonly struct` types passed by `ref`.
    - No boxing, no temporary objects, minimal GC pressure.
    - Handlers receive `ref` parameters for struct messages.
 
 1. Type‑Safe by Default
-
    - Compile‑time guarantees via generic constraints.
    - No string‑based dispatch (unlike Unity’s `SendMessage`).
    - Source generators provide boilerplate‑free message definitions.
 
 1. Predictable Execution
-
    - Priority‑based handler ordering (lower priority runs first).
    - Three‑stage pipeline: Interceptors → Handlers → Post‑Processors.
    - Deterministic behavior within each priority level.
 
 1. Observable & Debuggable
-
    - Built‑in diagnostics via `CyclicBuffer`.
    - Registration logging with `RegistrationLog`.
    - Inspector integration for runtime visibility.
 
 1. Lifecycle Safety
-
    - `MessageRegistrationToken` manages enable/disable.
    - Automatic cleanup prevents memory leaks.
    - Unity lifecycle integration via `MessageAwareComponent`.

package/Docs/EmitShorthands.md

@@ -60,7 +60,7 @@ var msg = new GamePaused();
 msg.Emit();  // Every listener receives this
 ```
 
-**Equivalent to:**
+#### Equivalent to
 
 ```csharp
 msg.EmitUntargeted();
@@ -68,7 +68,7 @@ msg.EmitUntargeted();
 MessageHandler.MessageBus.UntargetedBroadcast(ref msg);
 ```
 
-**Examples:**
+##### Examples
 
 - Scene loaded/unloaded
 - Game state changes (paused, resumed)
@@ -90,7 +90,7 @@ heal.EmitAt(player); // Only the player receives this
 
 **Caution:** `InstanceId` has implicit conversion from both `GameObject` and `Component`. Make sure your target type matches how listeners registered!
 
-**Examples:**
+#### Examples
 
 - Heal/damage commands to specific entities
 - UI updates for specific panels
@@ -109,7 +109,7 @@ var damage = new TookDamage(5);
 damage.EmitFrom(enemy);  // Anyone interested in this enemy receives it
 ```
 
-**Use cases:**
+#### Use cases
 
 - Combat logging ("Enemy X took damage")
 - Analytics ("Track damage from specific boss")
@@ -160,7 +160,7 @@ void OnAnyDamage(ref InstanceId source, ref TookDamage msg)
 }
 ```
 
-**Why this is revolutionary:**
+#### Why this is revolutionary
 
 - **Classic event bus:** Subscribe to PlayerDamaged, EnemyDamaged, NPCDamaged... (tight coupling)
 - **DxMessaging:** Subscribe to ALL damage once (zero coupling, infinite scalability)
@@ -169,7 +169,7 @@ void OnAnyDamage(ref InstanceId source, ref TookDamage msg)
 
 ---
 
-**📖 For the full deep dive on GameObject vs Component targeting and global observers, see [Targeting & Context](TargetingAndContext.md)**
+##### 📖 For the full deep dive on GameObject vs Component targeting and global observers, see [Targeting & Context](TargetingAndContext.md)
 
 ---
 
@@ -244,14 +244,14 @@ DxMessaging includes three built-in string message types for rapid prototyping:
 
 ### When to Use String Messages
 
-**Good for:**
+#### Good for
 
 - Rapid prototyping
 - Debug logging
 - Test utilities
 - Tool scripts
 
-**Not good for:**
+##### Not good for
 
 - Production gameplay code (use typed messages instead)
 - Performance-critical paths
@@ -271,7 +271,7 @@ msg.EmitAt(target, localBus);    // Target on specific bus
 msg.EmitFrom(source, localBus);  // Broadcast on specific bus
 ```
 
-**When to use:**
+### When to use
 
 - Testing with isolated buses
 - Subsystems with their own message domains
@@ -295,20 +295,17 @@ If a handler isn’t firing, first suspect a GameObject vs Component mismatch. S
 
 ### "My handler isn't firing!"
 
-**Check these in order:**
+#### Check these in order
 
 1. **GameObject vs Component mismatch?**
-
    - Did you register for a GameObject but emit to a Component (or vice versa)?
    - Use explicit helpers (`EmitGameObjectTargeted` vs `EmitComponentTargeted`) to eliminate this issue
 
 1. **Is the handler enabled?**
-
    - Check that `token.Enable()` was called (usually in `OnEnable`)
    - Verify the component is active
 
 1. **Correct message type?**
-
    - Untargeted messages use `Emit()`
    - Targeted messages use `EmitAt(target)`
    - Broadcast messages use `EmitFrom(source)`
@@ -344,7 +341,7 @@ See [Diagnostics](Diagnostics.md) for more debugging tools.
 | **Broadcast from Component**  | `msg.EmitFrom(component)`  | `msg.EmitComponentBroadcast(component)`   |
 | **Non-Unity InstanceId**      | `msg.EmitAt(id)`           | `msg.EmitTargeted(id)`                    |
 
-**Our recommendation:**
+### Our recommendation
 
 - In Unity gameplay code: Use **explicit helpers** for clarity and safety
 - In tests and non-Unity code: Use **shorthands** for brevity

package/Docs/FAQ.md

@@ -45,18 +45,18 @@ Do I need a global bus?
 
 ## Related Documentation
 
-**New to DxMessaging?**
+### New to DxMessaging?
 
 - → [Visual Guide](VisualGuide.md) — Beginner-friendly introduction
 - → [Getting Started](GettingStarted.md) — Complete guide
 - → [Glossary](Glossary.md) — All terms explained
 
-**Common Issues:**
+#### Common Issues
 
 - → [Troubleshooting](Troubleshooting.md) — Solutions to common problems
 - → [Common Patterns](Patterns.md) — See how to use it correctly
 
-**Reference:**
+##### Reference
 
 - → [Quick Reference](QuickReference.md) — API cheat sheet
 - → [Message Types](MessageTypes.md) — Which type to use when

package/Docs/Glossary.md

@@ -205,7 +205,7 @@ DxMessaging handles this automatically via `MessageAwareComponent`:
 
 Making systems **independent** so they don't need references to each other.
 
-**Before DxMessaging:**
+#### Before DxMessaging
 
 ```csharp
 public class UI : MonoBehaviour {
@@ -215,7 +215,7 @@ public class UI : MonoBehaviour {
 }
 ```
 
-**With DxMessaging:**
+##### With DxMessaging
 
 ```csharp
 public class UI : MessageAwareComponent {
@@ -245,18 +245,18 @@ public class UI : MessageAwareComponent {
 
 ## Related Documentation
 
-**Learn More:**
+### Learn More
 
 - → [Visual Guide](VisualGuide.md) — See these concepts visualized
 - → [Getting Started](GettingStarted.md) — Full introduction with examples
 - → [Message Types](MessageTypes.md) — When to use each type
 
-**Reference:**
+#### Reference
 
 - → [Quick Reference](QuickReference.md) — API cheat sheet
 - → [API Reference](Reference.md) — Complete API documentation
 
-**Examples:**
+##### Examples
 
 - → [Mini Combat sample](../Samples~/Mini%20Combat/README.md) — See concepts in action
 - → [Patterns](Patterns.md) — Real-world usage patterns