simple-module-cli 0.0.9__tar.gz → 0.0.10__tar.gz

{simple_module_cli-0.0.9 → simple_module_cli-0.0.10}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: simple_module_cli
-Version: 0.0.9
+Version: 0.0.10
 Summary: Standalone scaffolder for the SimpleModule framework — `sm new`, `sm create-module`, plugin host.
 Project-URL: Homepage, https://github.com/antosubash/simple_module_python
 Project-URL: Repository, https://github.com/antosubash/simple_module_python

{simple_module_cli-0.0.9 → simple_module_cli-0.0.10}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "simple_module_cli"
-version = "0.0.9"
+version = "0.0.10"
 description = "Standalone scaffolder for the SimpleModule framework — `sm new`, `sm create-module`, plugin host."
 readme = "README.md"
 license = "MIT"

{simple_module_cli-0.0.9 → simple_module_cli-0.0.10}/simple_module_cli/templates/host/client_app/vite.config.ts

@@ -35,14 +35,25 @@ const fsRoot = findNodeModulesRoot(__dirname);
 // dev server can read files outside the host root, and in
 // optimizeDeps.entries so its dependency scanner discovers bare imports
 // from wheel-installed pages and pre-bundles them.
+//
+// We also collect each module's package.json (one level up from pages/,
+// where Hatch's force-include drops it) so the dependency walk in
+// `collectOptimizeIncludes` reaches packages a wheel-installed page imports
+// directly (`sonner`, `lucide-react`, ...). Without this seed, Vite's
+// pre-bundler never sees those bare specifiers and Node module resolution
+// walks up from inside .venv/site-packages — never reaching
+// host/client_app/node_modules.
 const manifestPath = path.resolve(__dirname, 'modules.manifest.json');
 const moduleFsAllow: string[] = [];
 const moduleOptimizeEntries: string[] = [];
+const modulePkgJsonPaths: string[] = [];
 if (fs.existsSync(manifestPath)) {
   const manifest = JSON.parse(fs.readFileSync(manifestPath, 'utf-8')) as Record<string, string>;
   for (const pagesDir of Object.values(manifest)) {
     moduleFsAllow.push(path.dirname(pagesDir));
     moduleOptimizeEntries.push(path.join(pagesDir, '**/*.tsx'));
+    const modulePkgJson = path.join(path.dirname(pagesDir), 'package.json');
+    if (fs.existsSync(modulePkgJson)) modulePkgJsonPaths.push(modulePkgJson);
   }
 }
 
@@ -99,7 +110,7 @@ function collectOptimizeIncludes(): string[] {
     'use-sync-external-store/shim/with-selector',
   ]);
   const visited = new Set<string>();
-  const queue: string[] = [path.join(__dirname, 'package.json')];
+  const queue: string[] = [path.join(__dirname, 'package.json'), ...modulePkgJsonPaths];
   while (queue.length > 0) {
     const pkgJsonPath = queue.shift();
     if (!pkgJsonPath || visited.has(pkgJsonPath)) continue;

simple_module_cli-0.0.10/tests/test_cli_new_regressions.py

@@ -0,0 +1,61 @@
+"""Regression tests for ``sm new`` scaffold bugs filed against released wheels.
+
+Each test pins a specific issue's repro so the bug can't sneak back in
+without a CI failure pointing at the fix.
+"""
+
+from __future__ import annotations
+
+import json
+from pathlib import Path
+
+from simple_module_cli.cli import app
+from typer.testing import CliRunner
+
+
+def test_sm_new_flat_pins_inertia_react_to_v2(tmp_path: Path) -> None:
+    """Issue #128: flat scaffold's root package.json must peer-match
+    @simple-module-py/ui's ``@inertiajs/react: ^2.0.0`` peer dep."""
+    runner = CliRunner()
+    target = tmp_path / "demo"
+    runner.invoke(
+        app,
+        ["new", "demo", "--yes", "--flat", "--no-install", "--dest", str(target)],
+    )
+    data = json.loads((target / "package.json").read_text())
+    inertia = data.get("dependencies", {}).get("@inertiajs/react")
+    assert inertia is not None
+    assert inertia.startswith("^2."), f"expected ^2.x, got {inertia!r}"
+
+
+def test_sm_new_sample_module_pins_match_framework_version(tmp_path: Path) -> None:
+    """Issue #126/#119: sample hello module must pin framework deps to the
+    actual published framework version, not the future ``>=1.0,<2.0`` range."""
+    from importlib.metadata import version
+
+    expected = version("simple_module_cli")
+    runner = CliRunner()
+    target = tmp_path / "demo"
+    runner.invoke(
+        app,
+        ["new", "demo", "--yes", "--db", "sqlite", "--no-install", "--dest", str(target)],
+    )
+    text = (target / "modules" / "hello" / "pyproject.toml").read_text()
+    for pkg in ("simple_module_core", "simple_module_db", "simple_module_hosting"):
+        assert f"{pkg}=={expected}" in text, f"{pkg} should be pinned to =={expected}"
+    # Dev pin: simple_module_test was the original >=0.1,<1.0 unsatisfiable case.
+    assert f"simple_module_test=={expected}" in text
+
+
+def test_sm_new_sample_module_seeds_static_dist_placeholder(tmp_path: Path) -> None:
+    """Issue #127: hatch's force-include resolves at uv-sync time. A fresh
+    scaffold must ship an empty ``static/dist/`` so the build doesn't fail
+    before vite has had a chance to run."""
+    runner = CliRunner()
+    target = tmp_path / "demo"
+    runner.invoke(
+        app,
+        ["new", "demo", "--yes", "--db", "sqlite", "--no-install", "--dest", str(target)],
+    )
+    static_dist = target / "modules" / "hello" / "hello" / "static" / "dist"
+    assert static_dist.is_dir(), "static/dist/ must exist for hatch force-include"