metaobjects 0.19.1__tar.gz → 0.19.2__tar.gz

{metaobjects-0.19.1 → metaobjects-0.19.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: metaobjects
-Version: 0.19.1
+Version: 0.19.2
 Summary: Cross-language metadata standard: declare typed entities once, generate idiomatic drift-checked code across languages — Python port.
 Project-URL: Homepage, https://metaobjects.dev
 Project-URL: Repository, https://github.com/metaobjectsdev/metaobjects

{metaobjects-0.19.1 → metaobjects-0.19.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "metaobjects"
-version = "0.19.1"
+version = "0.19.2"
 description = "Cross-language metadata standard: declare typed entities once, generate idiomatic drift-checked code across languages — Python port."
 readme = "README.md"
 requires-python = ">=3.11"

{metaobjects-0.19.1 → metaobjects-0.19.2}/src/metaobjects/codegen/generators/entity_model.py

@@ -99,7 +99,7 @@ def _first_attr(field: MetaField, sub_type: str, attr_name: str) -> object | Non
     return None
 
 
-def _validator_constraints(field: MetaField, wire: bool = False) -> dict[str, object]:
+def _validator_constraints(field: MetaField, *, wire: bool) -> dict[str, object]:
     """Map the field's ``validator.*`` children + field attrs to Pydantic ``Field``
     kwargs. Cross-port-canonical semantics (TS is the reference):
 
@@ -114,8 +114,13 @@ def _validator_constraints(field: MetaField, wire: bool = False) -> dict[str, ob
     every OTHER constraint above unconditionally but must not self-validate Pin 1
     at construction (that broke a downstream adopter's in-process prompt-payload
     ``object.value`` models on legitimately-empty slots). Callers building a read
-    model (``_field_line``) pass the default ``False``; callers building a wire
-    model (``_create_field_line``/``_patch_field_line``) pass ``True``.
+    model (``_field_line``) pass ``wire=False``; callers building a wire model
+    (``_create_field_line``/``_patch_field_line``) pass ``wire=True``.
+
+    *wire* is keyword-only and REQUIRED (no default, item 5 of the #224
+    cleanup): a forgotten flag at a future wire call site must fail loudly
+    (``TypeError``) at generation time rather than silently falling back to a
+    permissive read-model tier.
     """
     kwargs: dict[str, object] = {}
 
@@ -171,17 +176,26 @@ def _validator_constraints(field: MetaField, wire: bool = False) -> dict[str, ob
         kwargs["pattern"] = _HOSTNAME_REGEX
 
     # FR-036 Pin 1 (#224 — WIRE TIER ONLY) — a @required non-array string is
-    # non-empty: reject null / "" but ACCEPT whitespace-only. Emit an implicit
-    # min_length of 1, keeping the stricter floor if a validator.length @min
-    # already set one. Never applied to the in-process read model (wire=False).
+    # non-empty by default: reject null / "" but ACCEPT whitespace-only. Emit an
+    # implicit min_length of 1 ONLY when no validator.length @min was authored.
+    # Never applied to the in-process read model (wire=False).
+    #
+    # An explicitly authored @min is always AUTHORITATIVE over the implicit floor
+    # (ADR-0044 ruling on #224) — @min: 0 restores "must be provided, may be
+    # empty" by opting out of the non-empty floor entirely. Per the cross-port
+    # emission pin, an authored @min: 0 emits NO minimum constraint at all
+    # (cleanest, deterministic output across ports), so it is popped rather than
+    # kept as min_length=0.
     if (
         wire
         and field.sub_type == fc.FIELD_SUBTYPE_STRING
         and not field_is_array(field)
         and field.attrs().get(fc.FIELD_ATTR_REQUIRED) is True
     ):
-        existing_min = kwargs.get("min_length")
-        kwargs["min_length"] = max(1, existing_min) if isinstance(existing_min, int) else 1
+        if min_len is None:
+            kwargs["min_length"] = 1
+        elif min_len == 0:
+            kwargs.pop("min_length", None)
 
     return kwargs
 
@@ -196,7 +210,7 @@ def _constraint_parts(constraints: dict[str, object]) -> list[str]:
 
 
 def _type_expr_for_field(
-    field: MetaField, imports: set[str], config: GenConfig, wire: bool = False
+    field: MetaField, imports: set[str], config: GenConfig, *, wire: bool
 ) -> tuple[str, str | None]:
     """The Python annotation text for *field* (arrays wrapped in ``list[...]``) plus the
     shared-enum class name when the field resolves to one (used to render an enum
@@ -211,7 +225,10 @@ def _type_expr_for_field(
     ``@objectRef`` annotates with the referenced VO's own WIRE model (``<Ref>Create``)
     rather than its read model, so a required-string Pin 1 violation nested inside the
     VO still validates on the wire (api-contract jsonb r10). A VO's own Create lines
-    are emitted through this same wire path, so nested-VO-in-VO recurses naturally."""
+    are emitted through this same wire path, so nested-VO-in-VO recurses naturally.
+
+    *wire* is keyword-only and REQUIRED (no default, item 5 of the #224 cleanup)
+    — see ``_validator_constraints`` for the rationale."""
     shared = (
         fr019.shared_enum_for_field(field)
         if field.sub_type == fc.FIELD_SUBTYPE_ENUM
@@ -275,7 +292,7 @@ def origin_guaranteed_non_null(field: MetaField) -> bool:
 
 def _field_line(field: MetaField, imports: set[str], config: GenConfig) -> tuple[str, bool]:
     """Return (source line, uses_field). Collects required imports into *imports*."""
-    type_expr, enum_type_name = _type_expr_for_field(field, imports, config)
+    type_expr, enum_type_name = _type_expr_for_field(field, imports, config, wire=False)
     required = field.attrs().get(fc.FIELD_ATTR_REQUIRED) is True
     default_raw = field.attrs().get(fc.FIELD_ATTR_DEFAULT)
     # A server-side expression default (now(), gen_random_uuid(), CURRENT_TIMESTAMP)
@@ -283,7 +300,7 @@ def _field_line(field: MetaField, imports: set[str], config: GenConfig) -> tuple
     # its required/optional shape). Only literal defaults become Pydantic defaults.
     has_default = default_raw is not None and not _is_sql_expr_default(default_raw)
 
-    parts = _constraint_parts(_validator_constraints(field))
+    parts = _constraint_parts(_validator_constraints(field, wire=False))
     uses_field = bool(parts)
 
     # @default — render an enum member (Type.MEMBER, UPPER_CASE) or a Python literal.

{metaobjects-0.19.1 → metaobjects-0.19.2}/src/metaobjects/spec_metamodel/field.json

@@ -10,7 +10,7 @@
         { "type": "validator", "subType": "*", "name": "*", "min": 0, "max": null },
         { "type": "view", "subType": "*", "name": "*", "min": 0, "max": null },
         { "type": "origin", "subType": "*", "name": "*", "min": 0, "max": null },
-        { "type": "attr", "subType": "boolean", "name": "required", "min": 0, "max": 1, "description": "When true, the field is NOT NULL. Equivalent to attaching a validator.required child." },
+        { "type": "attr", "subType": "boolean", "name": "required", "min": 0, "max": 1, "description": "When true, the field is NOT NULL. Equivalent to attaching a validator.required child. On a non-array string, generated wire-tier input validation (create/patch) additionally rejects the empty string by default — whitespace is accepted — unless an explicit validator.length @min: 0 opts back to presence-only. In-process read models never enforce this at construction." },
         { "type": "attr", "subType": "boolean", "name": "readOnly", "min": 0, "max": 1, "description": "FR-013: when true, the field is read-only — codegen emits no setter / writable property, the persistence layer skips the column on INSERT/UPDATE, and Zod/Pydantic/class-validator schemas mark it read-only on input variants. The value is populated by the database (computed column, default expression, trigger), by replication, or by another external owner." },
         { "type": "attr", "name": "default", "min": 0, "max": 1, "description": "Default value applied to the column when no value is supplied. Its type follows the field's own subtype (string / boolean / number / ...). Converted at consumption time via MetaField.defaultValue()." },
         { "type": "attr", "subType": "boolean", "name": "unique", "min": 0, "max": 1, "description": "When true, the field gets a column-level UNIQUE constraint." }

{metaobjects-0.19.1 → metaobjects-0.19.2}/src/metaobjects/spec_metamodel/validator.json

@@ -13,8 +13,8 @@
     {
       "type": "validator",
       "subType": "required",
-      "description": "Fails when the value is null/empty (NOT NULL). Equivalent to @required on the owning field.",
-      "whenToUse": "A field must be present (NOT NULL). Declare it instead of hand-writing a null/empty check."
+      "description": "Fails when the value is null (NOT NULL). Equivalent to @required on the owning field: on a non-array string, generated wire-tier input validation also rejects the empty string by default unless an explicit validator.length @min: 0 opts back to presence-only.",
+      "whenToUse": "A field must be present (NOT NULL). Declare it instead of hand-writing a presence check."
     },
     {
       "type": "validator",

{metaobjects-0.19.1 → metaobjects-0.19.2}/tests/codegen/test_cli_staleness_nudge.py

@@ -47,7 +47,7 @@ def test_gen_nudges_on_stale_manifest(tmp_path, capsys, monkeypatch) -> None:
     assert rc == 0  # advisory: never changes the exit code
     err = capsys.readouterr().err
     assert "0.0.1-old" in err
-    assert "metaobjects agent-docs" in err
+    assert "npx meta agent-docs" in err
 
 
 def test_verify_nudges_on_stale_manifest(tmp_path, capsys, monkeypatch) -> None:
@@ -65,7 +65,7 @@ def test_verify_nudges_on_stale_manifest(tmp_path, capsys, monkeypatch) -> None:
     assert rc == 0
     err = capsys.readouterr().err
     assert "0.0.1-old" in err
-    assert "metaobjects agent-docs" in err
+    assert "npx meta agent-docs" in err
 
 
 def test_gen_silent_when_no_manifest(tmp_path, capsys, monkeypatch) -> None:
@@ -76,7 +76,7 @@ def test_gen_silent_when_no_manifest(tmp_path, capsys, monkeypatch) -> None:
     rc = main(["gen", meta_dir, "--out", str(out)])
     assert rc == 0
     err = capsys.readouterr().err
-    assert "metaobjects agent-docs" not in err
+    assert "npx meta agent-docs" not in err
 
 
 def test_gen_silent_on_corrupt_manifest(tmp_path, capsys, monkeypatch) -> None:
@@ -90,4 +90,4 @@ def test_gen_silent_on_corrupt_manifest(tmp_path, capsys, monkeypatch) -> None:
     rc = main(["gen", meta_dir, "--out", str(out)])
     assert rc == 0
     err = capsys.readouterr().err
-    assert "metaobjects agent-docs" not in err
+    assert "npx meta agent-docs" not in err