Keep PrimitiveSchemaDefinition reachable on mcp.server.elicitation

Its module-level import moved into the validation function that uses it, so
the attribute would otherwise disappear from the module namespace. Resolve it
on first access via a module-level __getattr__ (and keep it in dir()) so
existing references to mcp.server.elicitation.PrimitiveSchemaDefinition keep
working without reintroducing the import at module load.

Author: maxisbey

src/mcp/server/elicitation.py

@@ -191,3 +191,21 @@ async def elicit_url(
     else:  # pragma: no cover
         # This should never happen, but handle it just in case
         raise ValueError(f"Unexpected elicitation action: {result.action}")
+
+
+def __getattr__(name: str) -> Any:
+    # `PrimitiveSchemaDefinition` used to be bound in this module by a
+    # module-level import of the internal wire package; that import now
+    # happens on first use (inside `_validate_rendered_properties`) so that
+    # importing this module does not build the wire schema. Keep the attribute
+    # resolving lazily for existing references to it.
+    if name == "PrimitiveSchemaDefinition":
+        from mcp_types._v2025_11_25 import PrimitiveSchemaDefinition
+
+        globals()[name] = PrimitiveSchemaDefinition
+        return PrimitiveSchemaDefinition
+    raise AttributeError(f"module {__name__!r} has no attribute {name!r}")
+
+
+def __dir__() -> list[str]:
+    return sorted({*globals(), "PrimitiveSchemaDefinition"})

tests/server/test_elicitation.py

@@ -0,0 +1,12 @@
+"""`mcp.server.elicitation` module-level behaviour."""
+
+import mcp_types._v2025_11_25 as wire
+
+from mcp.server import elicitation
+
+
+def test_wire_schema_gate_type_is_still_reachable_on_the_module():
+    """`PrimitiveSchemaDefinition` used to be bound here by a module-level import; it now
+    resolves lazily to the same class, and `dir()` still lists it."""
+    assert elicitation.PrimitiveSchemaDefinition is wire.PrimitiveSchemaDefinition
+    assert "PrimitiveSchemaDefinition" in dir(elicitation)