breaking: Change default middleware priority from 0 to 100 as per PROTOCOL_SPEC §11.2, updating the base middleware class, tests, and changelog.

Author: tercel

CHANGELOG.md

@@ -11,6 +11,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - **Middleware priority** — `Middleware` base class now accepts `priority: int` (0-1000, default 0). Higher priority executes first; equal priority preserves registration order. `BeforeMiddleware` and `AfterMiddleware` adapters also accept `priority`.
 - **Priority range validation** — `ValueError` raised for priority values outside 0-1000
 
+### Breaking Changes
+- Middleware default priority changed from `0` to `100` per PROTOCOL_SPEC §11.2. Middleware without explicit priority will now execute before priority-0 middleware.
+
+
 ## [0.13.2] - 2026-03-22
 
 ### Changed

src/apcore/middleware/base.py

@@ -16,12 +16,12 @@ class Middleware:
     Attributes:
         priority: Execution priority (0-1000). Higher priority executes first.
             Middlewares with equal priority preserve registration order.
-            Defaults to 0 for backward compatibility.
+            Defaults to 100 per PROTOCOL_SPEC §11.2.
     """
 
-    priority: int = 0
+    priority: int = 100
 
-    def __init__(self, *, priority: int = 0) -> None:
+    def __init__(self, *, priority: int = 100) -> None:
         if not (0 <= priority <= 1000):
             raise ValueError(f"priority must be between 0 and 1000, got {priority}")
         self.priority = priority

tests/test_middleware.py

@@ -178,10 +178,10 @@ def test_callback_receives_correct_args(self) -> None:
 class TestMiddlewarePriority:
     """Tests for middleware priority ordering in MiddlewareManager."""
 
-    def test_default_priority_is_zero(self) -> None:
-        """Middleware instances default to priority 0."""
+    def test_default_priority_is_100(self) -> None:
+        """Middleware instances default to priority 100 per PROTOCOL_SPEC."""
         mw = Middleware()
-        assert mw.priority == 0
+        assert mw.priority == 100
 
     def test_custom_priority(self) -> None:
         """Middleware accepts a custom priority via constructor."""
@@ -235,7 +235,7 @@ def test_mixed_priorities_with_ties(self) -> None:
         assert snapshot == [d, a, c, b, e]
 
     def test_default_priority_backward_compatible(self) -> None:
-        """Middlewares without explicit priority still work (default 0)."""
+        """Middlewares without explicit priority still work (default 100)."""
         manager = MiddlewareManager()
         mw1 = Middleware()
         mw2 = Middleware()
@@ -248,15 +248,15 @@ def test_default_priority_backward_compatible(self) -> None:
         snapshot = manager.snapshot()
         assert snapshot == [mw1, mw2, mw3]
 
-    def test_subclass_without_super_init_defaults_to_zero(self) -> None:
-        """Subclasses that don't call super().__init__() still have priority 0."""
+    def test_subclass_without_super_init_defaults_to_100(self) -> None:
+        """Subclasses that don't call super().__init__() still have class default priority 100."""
 
         class CustomMiddleware(Middleware):
             def __init__(self) -> None:
                 self.custom_field = "hello"
 
         mw = CustomMiddleware()
-        assert mw.priority == 0
+        assert mw.priority == 100
 
     def test_remove_preserves_priority_order(self) -> None:
         """Removing a middleware preserves the priority-sorted order."""