metaobjects 1.0.7__tar.gz → 1.0.8rc1__tar.gz

{metaobjects-1.0.7 → metaobjects-1.0.8rc1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: metaobjects
-Version: 1.0.7
+Version: 1.0.8rc1
 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-1.0.7 → metaobjects-1.0.8rc1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "metaobjects"
-version = "1.0.7"
+version = "1.0.8rc1"
 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-1.0.7 → metaobjects-1.0.8rc1}/src/metaobjects/cli.py

@@ -1069,11 +1069,6 @@ def _cmd_gen(args: argparse.Namespace) -> int:
             for msg in gen_errors:
                 print(f"  {msg}", file=sys.stderr)
             return 1
-    else:
-        # Codegen is opt-in: refuse here, at the door, rather than letting the run reach
-        # the generator loop and fail with an empty out dir half-created.
-        print(f"error: {NO_GENERATORS_SELECTED}", file=sys.stderr)
-        return 2
 
     # SP-1: declarative Mustache generators from a JSON template-spec. Their output
     # is format-agnostic (text/markdown/csv/json/xml/html), so they run as a SECOND,
@@ -1088,6 +1083,14 @@ def _cmd_gen(args: argparse.Namespace) -> int:
         print(spec_err, file=sys.stderr)
         return 1
 
+    # Codegen is opt-in: refuse at the door when nothing is selected, rather than letting
+    # the run reach the generator loop and fail with an empty out dir half-created. A
+    # template spec IS a selection (flag or the conventional template-spec.json), so a
+    # project that generates only from one is not refused; verify already agreed.
+    if not generators and not spec_gens:
+        print(f"error: {NO_GENERATORS_SELECTED}", file=sys.stderr)
+        return 2
+
     entities = _parse_entities(getattr(args, "entities", None))
     providers, providers_ok = _providers_from_args(args)
     if not providers_ok:
@@ -1112,7 +1115,7 @@ def _cmd_gen(args: argparse.Namespace) -> int:
             root, args.out, generators, entities,
             gen_state_dir=gen_state, column_naming=column_naming,
             project_root=gen_project_root, baseline=baseline, refused_out=refused,
-        )
+        ) if generators else []
     except ValueError as exc:
         # A generator refused the model — a run_gen output collision, or
         # `render-helper`'s build-time drift gate. The targets path below has caught

{metaobjects-1.0.7 → metaobjects-1.0.8rc1}/src/metaobjects/codegen/generator_registry.py

@@ -181,6 +181,10 @@ GENERATOR_REGISTRY: dict[str, GeneratorEntry] = {
         tier="native",
         layer="api",
         factory=lambda _ctx: router_generator(),
+        # The router imports <Entity>Create/Patch from the entity module and the
+        # FIELDS/OPS constants from <entity>_filter_allowlist; without either the
+        # generated package fails at import (ModuleNotFoundError).
+        requires=("entity", "filter-allowlist"),
         source=router_generator,
     ),
     "output-parser": GeneratorEntry(
@@ -279,8 +283,8 @@ def unsatisfied_requires(names: list[str]) -> list[str]:
         listed = ", ".join(f'"{m}"' for m in missing)
         warnings.append(
             f'"{name}" is selected but {listed} {"is" if len(missing) == 1 else "are"} not. '
-            f'The code "{name}" emits imports the value-object models {listed} would have '
-            f"emitted, so importing it will fail. Add {listed} to --generators, or keep your "
-            "own hand-written models at those module paths."
+            f'The code "{name}" emits imports the modules {listed} would have emitted, so '
+            f"importing it will fail. Add {listed} to --generators, or keep your own "
+            "hand-written modules at those paths."
         )
     return warnings

{metaobjects-1.0.7 → metaobjects-1.0.8rc1}/tests/codegen/test_cli.py

@@ -199,6 +199,24 @@ def test_gen_auto_discovers_template_spec(tmp_path: Path) -> None:
     assert (out / "_model.txt").exists()
 
 
+def test_gen_with_only_a_template_spec_is_not_refused(tmp_path: Path) -> None:
+    """A template spec is a selection. A project that generates only from one used to be
+    refused with "no generators selected", while verify already honoured it."""
+    root = _spec_project(tmp_path)
+    out = root / "out"
+    rc = main(["gen", str(root / "meta"), "--out", str(out), "--templates", str(root / "templates")])
+    assert rc == 0
+    assert (out / "Product.txt").exists()
+    assert not (out / "__init__.py").exists(), "no Python suite ran, so no package init"
+
+
+def test_gen_with_neither_generators_nor_spec_is_refused(tmp_path: Path, capsys) -> None:
+    root = _spec_project(tmp_path, spec_name=None)
+    rc = main(["gen", str(root / "meta"), "--out", str(root / "out")])
+    assert rc == 2
+    assert "no generators selected" in capsys.readouterr().err
+
+
 def test_gen_without_spec_file_emits_no_template_output(tmp_path: Path) -> None:
     """No spec file ⇒ today's behaviour exactly: the default suite only."""
     root = _spec_project(tmp_path, spec_name=None)

{metaobjects-1.0.7 → metaobjects-1.0.8rc1}/tests/codegen/test_cli_registry.py

@@ -101,6 +101,28 @@ def test_template_tier_without_entity_warns(capsys, tmp_path: Path) -> None:
     assert 'warning: "extractor" is selected but "entity" is not.' in err
 
 
+def test_routes_without_filter_allowlist_warns(capsys, tmp_path: Path) -> None:
+    """The router imports ``<entity>_filter_allowlist``; selecting ``routes`` without it
+    generated a package that failed at import with ModuleNotFoundError and no warning."""
+    meta_dir = _meta_dir(tmp_path)
+    rc = main(
+        ["gen", meta_dir, "--out", str(tmp_path / "out"), "--generators", "entity,names,routes"]
+    )
+    assert rc == 0
+    err = capsys.readouterr().err
+    assert 'warning: "routes" is selected but "filter-allowlist" is not.' in err
+
+
+def test_routes_with_its_requires_does_not_warn(capsys, tmp_path: Path) -> None:
+    meta_dir = _meta_dir(tmp_path)
+    rc = main(
+        ["gen", meta_dir, "--out", str(tmp_path / "out"),
+         "--generators", "entity,names,filter-allowlist,routes"]
+    )
+    assert rc == 0
+    assert "is selected but" not in capsys.readouterr().err
+
+
 def test_template_tier_with_entity_does_not_warn(capsys, tmp_path: Path) -> None:
     meta_dir = _meta_dir(tmp_path)
     rc = main(