simple-module-cli 0.0.34__tar.gz → 0.0.36__tar.gz

{simple_module_cli-0.0.34 → simple_module_cli-0.0.36}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: simple_module_cli
-Version: 0.0.34
+Version: 0.0.36
 Summary: Standalone scaffolder for the SimpleModule framework — `smpy new`, `smpy 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.34 → simple_module_cli-0.0.36}/pyproject.toml

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

{simple_module_cli-0.0.34 → simple_module_cli-0.0.36}/simple_module_cli/app_project.py

@@ -61,13 +61,24 @@ _APP_NPM_DEPS = {
     "@simple-module-py/i18n": _FRAMEWORK_VERSION,
     "react": "^19.0.0",
     "react-dom": "^19.0.0",
-    "@inertiajs/react": "^2.0.0",
+    "@inertiajs/react": "^3.7.0",
 }
+# Flat mode has no npm workspaces, so `client_app/package.json` is never
+# installed — the root package.json seeded from these two dicts is the only
+# manifest npm reads. It must therefore cover everything client_app needs to
+# build, not just the framework pins: `vite.config.ts` imports
+# `@tailwindcss/vite`, and `tsc` needs the @types packages. Keep this in step
+# with templates/host/client_app/package.json.tpl.
 _APP_NPM_DEV_DEPS = {
     "@simple-module-py/tsconfig": _FRAMEWORK_VERSION,
-    "@vitejs/plugin-react": "^5.0.0",
-    "typescript": "^5.6.0",
-    "vite": "^8.0.0",
+    "@tailwindcss/vite": "^4.0.0",
+    "@types/node": "^26.5.0",
+    "@types/react": "^19.0.0",
+    "@types/react-dom": "^19.0.0",
+    "@vitejs/plugin-react": "^6.1.1",
+    "tailwindcss": "^4.0.0",
+    "typescript": "^7.0.2",
+    "vite": "^8.2.2",
 }
 
 # Files the host template ships that the workspace template re-emits at

{simple_module_cli-0.0.34 → simple_module_cli-0.0.36}/simple_module_cli/skills/simple-module-migrations/SKILL.md

@@ -64,7 +64,7 @@ make migration msg="add orders module"
 make migrate
 ```
 
-`make migrate` runs `alembic -c host/alembic.ini upgrade heads` — note **`heads`** (plural). Branch labels create multiple independent heads, so `upgrade head` (singular) would error or under-apply.
+`make migrate` runs `alembic -c host/alembic.ini upgrade heads` — note **`heads`** (plural). Histories with a branch label can end up with multiple heads, so `upgrade head` (singular) may error or under-apply.
 
 ### Branch labels — set on the FIRST revision per module
 
@@ -78,16 +78,14 @@ branch_labels = ("audit_log",)  # ← add by hand on this first revision
 depends_on = None
 ```
 
-Why: it lets operators roll back **just one module's** schema, which is how a module is uninstalled cleanly:
+Why: the label is a named target for that revision (`upgrade audit_log@head`, the module column in the Doctor). It does **not** isolate the module: autogenerate chains the revision off the current head, so history is one linear chain and `downgrade audit_log@base` walks the **entire chain beneath it, other modules included**. Never use it as a per-module rollback.
 
-```bash
-make downgrade   # back one revision; OR, to a target:
-uv run --project host alembic -c host/alembic.ini downgrade audit_log@base
-```
+To remove one module's schema:
 
-`audit_log@base` drops everything that module owns and leaves the others. Without the label, downgrading to a bare revision id walks linear history and rolls back unrelated modules' migrations sitting between.
+- If the module's revisions are the latest in the chain, downgrade to the `down_revision` of its first revision: `uv run --project host alembic -c host/alembic.ini downgrade <that down_revision id>`.
+- Otherwise write a dedicated migration that drops that module's tables.
 
-Subsequent revisions for the same module **don't** need a `branch_labels` — only the first; they inherit the branch.
+Subsequent revisions for the same module **don't** need a `branch_labels` — only the first; they chain off it as ordinary revisions.
 
 > Exception: the repo's bootstrap `initial_schema` revision bundles several core modules (users, permissions, settings, file_storage, feature_flags, background_tasks) into one unlabeled root migration. The per-module `branch_labels` convention applies to modules added **after** that bootstrap.
 
@@ -112,7 +110,7 @@ SM010/SM011 are **not** emitted at boot or by `make doctor`. They only surface w
 
 ## Pitfalls
 
-- **Skipped the branch label on the first revision.** `alembic downgrade <module>@base` then fails or silently rolls back unrelated revisions. Fix by editing the migration file before applying; if already deployed, write a no-op revision that adds the label retroactively.
+- **Treating `downgrade <module>@base` as a per-module rollback.** It rolls back every revision beneath the labelled one, other modules included. See the recipe under "Branch labels".
 - **Renamed a column.** Autogenerate emits `drop_column` + `add_column`, which loses data. Edit to `op.alter_column(..., new_column_name=...)` and write the matching `downgrade()`.
 - **Hand-edited operations with stubbed `downgrade()`.** Server defaults, CHECK constraints, expression indexes — Alembic can't always infer these. When you fill in `upgrade()`, fill in `downgrade()` too.
 - **Concurrent `make migrate` (`upgrade heads`) from multiple processes.** The `alembic_version` table isn't race-safe across all backends. Run upgrades from one place (a release pipeline step), not from the booting app.

{simple_module_cli-0.0.34 → simple_module_cli-0.0.36}/simple_module_cli/templates/docker/host-flat.Dockerfile

@@ -15,8 +15,8 @@ ENV UV_LINK_MODE=copy \
 
 WORKDIR /app
 
-# Node 22 for the Vite build.
-RUN curl -fsSL https://deb.nodesource.com/setup_22.x | bash - \
+# Node 24 for the Vite build.
+RUN curl -fsSL https://deb.nodesource.com/setup_24.x | bash - \
     && apt-get install -y --no-install-recommends nodejs \
     && rm -rf /var/lib/apt/lists/*
 

{simple_module_cli-0.0.34 → simple_module_cli-0.0.36}/simple_module_cli/templates/docker/host.Dockerfile

@@ -15,8 +15,8 @@ ENV UV_LINK_MODE=copy \
 
 WORKDIR /app
 
-# Node 22 for the Vite build.
-RUN curl -fsSL https://deb.nodesource.com/setup_22.x | bash - \
+# Node 24 for the Vite build.
+RUN curl -fsSL https://deb.nodesource.com/setup_24.x | bash - \
     && apt-get install -y --no-install-recommends nodejs \
     && rm -rf /var/lib/apt/lists/*
 

{simple_module_cli-0.0.34 → simple_module_cli-0.0.36}/simple_module_cli/templates/host/client_app/app.tsx

@@ -1,6 +1,8 @@
 import { createInertiaApp, router } from '@inertiajs/react';
 import { configureI18n, updateI18n } from '@simple-module-py/i18n';
+import { formatTitle, setTitleAppName } from '@simple-module-py/ui/lib/app-title';
 import { startSpaLinkInterception } from '@simple-module-py/ui/lib/spa-links';
+import { initTheme } from '@simple-module-py/ui/lib/theme';
 import { createRoot } from 'react-dom/client';
 import { resolvePage } from './pages';
 
@@ -11,22 +13,34 @@ type I18nBlock = {
 };
 
 createInertiaApp({
+  title: (title) => formatTitle(title),
   resolve: async (name) => {
     return await resolvePage(name);
   },
   setup({ el, App, props }) {
-    const initial = (props.initialPage.props as { i18n?: I18nBlock }).i18n;
+    const pageProps = props.initialPage.props as {
+      i18n?: I18nBlock;
+      branding?: { appName?: string | null };
+    };
+    setTitleAppName(pageProps.branding?.appName);
+    // Before the first render, so a dark-theme user never sees a light frame.
+    initTheme();
+    const initial = pageProps.i18n;
     configureI18n({
       locale: initial?.locale ?? 'en',
       messages: initial?.messages ?? {},
     });
-    let activeLocale = initial?.locale ?? null;
     router.on('success', (event) => {
       const block = (event.detail.page.props as { i18n?: I18nBlock }).i18n;
-      if (!block) return;
-      if (block.locale !== activeLocale && block.messages) {
+      // A non-null `messages` payload IS the server's signal that the client
+      // needs it — the backend sends `null` whenever the catalog the client
+      // already holds is still good. Gating on a locale change instead drops
+      // the catalog that arrives when the *audience* changes: signing in swaps
+      // the anonymous snapshot for one including admin-only modules, at the
+      // same locale, so every admin screen rendered raw keys
+      // ("dashboard.home.title") until a hard refresh.
+      if (block?.messages) {
         updateI18n({ locale: block.locale, messages: block.messages });
-        activeLocale = block.locale;
       }
     });
     // Authored content (pagebuilder widgets, markdown and rich-text fields)

{simple_module_cli-0.0.34 → simple_module_cli-0.0.36}/simple_module_cli/templates/host/client_app/package.json.tpl

@@ -8,7 +8,7 @@
     "preview": "vite preview"
   },
   "dependencies": {
-    "@inertiajs/react": "^2.0.0",
+    "@inertiajs/react": "^3.7.0",
     "@simple-module-py/i18n": "{{FRAMEWORK_VERSION}}",
     "@simple-module-py/ui": "{{FRAMEWORK_VERSION}}",
     "react": "^19.0.0",
@@ -17,12 +17,12 @@
   "devDependencies": {
     "@simple-module-py/tsconfig": "{{FRAMEWORK_VERSION}}",
     "@tailwindcss/vite": "^4.0.0",
-    "@types/node": "^22.0.0",
+    "@types/node": "^26.5.0",
     "@types/react": "^19.0.0",
     "@types/react-dom": "^19.0.0",
-    "@vitejs/plugin-react": "^5.0.0",
+    "@vitejs/plugin-react": "^6.1.1",
     "tailwindcss": "^4.0.0",
-    "typescript": "^5.7.0",
-    "vite": "^6.0.0"
+    "typescript": "^7.0.2",
+    "vite": "^8.2.2"
   }
 }

{simple_module_cli-0.0.34 → simple_module_cli-0.0.36}/simple_module_cli/templates/host/client_app/pages.ts

@@ -15,7 +15,14 @@ import { moduleGlobs } from './modules.generated';
 type PageModule = { default: React.ComponentType<Record<string, unknown>> };
 type PageLoader = () => Promise<PageModule>;
 
-const hostPages = import.meta.glob<PageModule>('./pages/**/*.tsx');
+// A `*.test.tsx` beside a page is a test, not a page — without these
+// negations it registers as a phantom page and Vite follows the import into
+// the production bundle.
+const hostPages = import.meta.glob<PageModule>([
+  './pages/**/*.tsx',
+  '!./pages/**/*.test.tsx',
+  '!./pages/**/*.spec.tsx',
+]);
 
 const pages: Record<string, PageLoader> = {};
 

{simple_module_cli-0.0.34 → simple_module_cli-0.0.36}/simple_module_cli/templates/host/migrations/env.py

@@ -27,12 +27,15 @@ logger = logging.getLogger("alembic.env")
 config = context.config
 
 if config.config_file_name is not None:
-    fileConfig(config.config_file_name)
+    # Not the default disable_existing_loggers=True: the setup wizard runs this
+    # in-process, and that default would silence every app logger until restart.
+    fileConfig(config.config_file_name, disable_existing_loggers=False)
 
 target_metadata = build_module_metadata()
 include_object = make_include_object(target_metadata)
-# Re-emit expression-based indexes (e.g. ``lower(email)``) that autogenerate
-# silently drops under SQLite. See ``make_process_revision_directives`` docstring.
+# Re-emit expression-based indexes that autogenerate drops under SQLite, rewrite
+# Boolean 0/1 server defaults to sa.false()/sa.true(), and warn that SQLite cannot
+# verify expression indexes (GH #342) — see make_process_revision_directives.
 process_revision_directives = make_process_revision_directives(target_metadata)
 
 

{simple_module_cli-0.0.34 → simple_module_cli-0.0.36}/simple_module_cli/templates/host/migrations/script.py.mako

@@ -8,6 +8,7 @@ Create Date: ${create_date}
 from collections.abc import Sequence
 
 import sqlalchemy as sa
+import sqlmodel  # noqa: F401  (autogenerate may emit sqlmodel types)
 from alembic import op
 ${imports if imports else ""}
 

{simple_module_cli-0.0.34 → simple_module_cli-0.0.36}/simple_module_cli/templates/host/routes.py

@@ -10,8 +10,8 @@ module is installed.
 from __future__ import annotations
 
 from fastapi import APIRouter, Request
-from inertia import InertiaResponse
 from simple_module_hosting.inertia_deps import InertiaDep
+from simple_module_inertia import InertiaResponse
 
 router = APIRouter()
 

{simple_module_cli-0.0.34 → simple_module_cli-0.0.36}/simple_module_cli/templates/host/templates/index.html

@@ -7,8 +7,9 @@
        crawlers and link previews see the configured site name before React
        hydrates; Inertia's <Head> takes over per-page titles after that. #}
     {% set brand = branding_head(request) %}
-    <title>{{ brand.app_name }}</title>
+    <title data-inertia="">{{ brand.app_name }}</title>
     {% if brand.theme_color %}<meta name="theme-color" content="{{ brand.theme_color }}" />{% endif %}
+    {% if brand.favicon_url %}<link rel="icon" href="{{ brand.favicon_url }}" />{% endif %}
     <link rel="preconnect" href="https://fonts.googleapis.com" />
     <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin />
     <link href="https://fonts.googleapis.com/css2?family=DM+Sans:ital,opsz,wght@0,9..40,400;0,9..40,500;0,9..40,600;0,9..40,700;1,9..40,400&family=Outfit:wght@400;500;600;700;800&display=swap" rel="stylesheet" />

{simple_module_cli-0.0.34 → simple_module_cli-0.0.36}/simple_module_cli/templates/module/.github/workflows/ci.yml

@@ -12,10 +12,10 @@ jobs:
   test:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v7
 
       - name: Install uv
-        uses: astral-sh/setup-uv@v3
+        uses: astral-sh/setup-uv@v9.0.0
         with:
           version: latest
 
@@ -34,14 +34,14 @@ jobs:
   frontend:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v7
 
-      - uses: actions/setup-node@v4
+      - uses: actions/setup-node@v7
         with:
-          node-version: '22'
+          node-version: '24'
 
       - name: Install uv
-        uses: astral-sh/setup-uv@v3
+        uses: astral-sh/setup-uv@v9.0.0
         with:
           version: latest
 

{simple_module_cli-0.0.34 → simple_module_cli-0.0.36}/simple_module_cli/templates/module/.github/workflows/publish.yml.tpl

@@ -29,10 +29,10 @@ jobs:
       contents: read
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v7
 
       - name: Install uv
-        uses: astral-sh/setup-uv@v3
+        uses: astral-sh/setup-uv@v9.0.0
         with:
           version: latest
 

{simple_module_cli-0.0.34 → simple_module_cli-0.0.36}/simple_module_cli/templates/module/__PACKAGE__/endpoints/views.py.tpl

@@ -3,8 +3,8 @@
 from __future__ import annotations
 
 from fastapi import APIRouter
-from inertia import InertiaResponse
 from simple_module_hosting.inertia_deps import InertiaDep
+from simple_module_inertia import InertiaResponse
 
 router = APIRouter()
 

{simple_module_cli-0.0.34 → simple_module_cli-0.0.36}/simple_module_cli/templates/module/_optional/standalone/package.json.tpl

@@ -9,19 +9,21 @@
   "peerDependencies": {
     "react": "^19.0.0",
     "react-dom": "^19.0.0",
-    "@inertiajs/react": "^2.0.0",
+    "@inertiajs/react": "^3.7.0",
     "@simple-module-py/ui": "*"
   },
   "devDependencies": {
-    "@inertiajs/react": "^2.0.0",
+    "@inertiajs/react": "^3.7.0",
     "@simple-module-py/i18n": "{{FRAMEWORK_VERSION}}",
     "@simple-module-py/tsconfig": "{{FRAMEWORK_VERSION}}",
     "@simple-module-py/ui": "{{FRAMEWORK_VERSION}}",
+    "@types/node": "^26.5.0",
     "@types/react": "^19.0.0",
     "@types/react-dom": "^19.0.0",
     "react": "^19.0.0",
     "react-dom": "^19.0.0",
-    "typescript": "^5.7.0"
+    "typescript": "^7.0.2",
+    "vite": "^8.2.2"
   },
   "dependencies": {}
 }

{simple_module_cli-0.0.34 → simple_module_cli-0.0.36}/simple_module_cli/templates/module/package.json.tpl

@@ -6,7 +6,7 @@
   "peerDependencies": {
     "react": "^19.0.0",
     "react-dom": "^19.0.0",
-    "@inertiajs/react": "^2.0.0",
+    "@inertiajs/react": "^3.7.0",
     "@simple-module-py/ui": "*"
   },
   "devDependencies": {

{simple_module_cli-0.0.34 → simple_module_cli-0.0.36}/simple_module_cli/templates/module/tests/test___PACKAGE__.py.tpl

@@ -8,6 +8,7 @@ that package is installed). No conftest.py is required.
 from __future__ import annotations
 
 from {{PACKAGE_NAME}}.module import {{MODULE_NAME}}Module
+from simple_module_test import effective_route_paths
 
 
 class TestMeta:
@@ -22,6 +23,8 @@ class TestRoutes:
     async def test_app_boots_with_module(self, build_test_app):
         """Module registers cleanly into a minimal FastAPI host."""
         app = build_test_app({{MODULE_NAME}}Module)
-        paths = {getattr(r, "path", None) for r in app.routes}
+        # effective_route_paths reads the OpenAPI schema: FastAPI >= 0.137
+        # includes routers lazily, so ``app.routes`` no longer lists them.
+        paths = effective_route_paths(app)
         # The placeholder GET / endpoint lives under the route_prefix.
-        assert any(p and p.startswith("/api/{{MODULE_SLUG}}") for p in paths)
+        assert any(p.startswith("/api/{{MODULE_SLUG}}") for p in paths)

{simple_module_cli-0.0.34 → simple_module_cli-0.0.36}/tests/test_cli_new_regressions.py

@@ -13,9 +13,9 @@ 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:
+def test_sm_new_flat_pins_inertia_react_to_v3(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."""
+    @simple-module-py/ui's ``@inertiajs/react: ^3.0.0`` peer dep."""
     runner = CliRunner()
     target = tmp_path / "demo"
     runner.invoke(
@@ -25,7 +25,7 @@ def test_sm_new_flat_pins_inertia_react_to_v2(tmp_path: Path) -> None:
     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}"
+    assert inertia.startswith("^3."), f"expected ^3.x, got {inertia!r}"
 
 
 def test_sm_new_sample_module_pins_match_framework_version(tmp_path: Path) -> None:
@@ -211,6 +211,38 @@ def test_sm_new_rejects_mixed_separators(tmp_path: Path) -> None:
     assert result.exit_code != 0
 
 
+def test_scaffold_adopts_every_locale_catalog_the_server_sends(tmp_path: Path) -> None:
+    """Issue #321: the scaffold gated ``updateI18n`` on a locale *change*, which
+    drops the catalog that arrives when the audience changes instead. Signing in
+    swaps the anonymous snapshot for one including admin-only modules at the same
+    locale, so every admin screen rendered raw keys ("dashboard.home.title")
+    until a hard refresh. A non-null ``messages`` payload is itself the server's
+    signal that the client needs it."""
+    from simple_module_cli.scaffolding import create_host
+
+    dest = tmp_path / "demo"
+    create_host(dest, name="demo-host", modules=["Dashboard", "Auth"])
+
+    app_tsx = (dest / "client_app" / "app.tsx").read_text()
+    assert "updateI18n" in app_tsx
+    assert "activeLocale" not in app_tsx, (
+        "the locale gate is back — a same-locale catalog would be dropped again"
+    )
+
+
+def test_sm_new_host_app_tsx_wires_theme_and_title(tmp_path: Path) -> None:
+    """Issue #429: the scaffolded app.tsx must init the theme and brand titles."""
+    from simple_module_cli.scaffolding import create_host
+
+    dest = tmp_path / "demo"
+    create_host(dest, name="demo-host", modules=["Dashboard", "Auth"])
+
+    app_tsx = (dest / "client_app" / "app.tsx").read_text()
+    assert "initTheme(" in app_tsx
+    assert "formatTitle" in app_tsx
+    assert "setTitleAppName" in app_tsx
+
+
 def test_sm_new_no_install_next_steps_include_initial_migration(tmp_path: Path) -> None:
     """Issue #135: ``host/migrations/versions/`` ships empty, so a fresh
     ``make migrate`` is a no-op. The printed next-steps must therefore
@@ -224,3 +256,25 @@ def test_sm_new_no_install_next_steps_include_initial_migration(tmp_path: Path)
     assert result.exit_code == 0, result.output
     assert 'make migration msg="initial schema"' in result.output
     assert "make migrate" in result.output
+
+
+def test_sm_new_flat_root_covers_client_app_build_deps(tmp_path: Path) -> None:
+    """Flat scaffolds have no npm workspaces, so ``client_app/package.json`` is
+    never installed and the root manifest is the only one npm reads. It must
+    therefore carry everything the client build imports — ``vite.config.ts``
+    pulls in ``@tailwindcss/vite``, so a root missing it failed ``vite build``
+    with ``Could not resolve '@tailwindcss/vite'``."""
+    runner = CliRunner()
+    target = tmp_path / "demo"
+    runner.invoke(
+        app,
+        ["new", "demo", "--yes", "--flat", "--no-install", "--dest", str(target)],
+    )
+    root = json.loads((target / "package.json").read_text())
+    client_app = json.loads((target / "client_app" / "package.json").read_text())
+    assert "workspaces" not in root, "flat mode must stay workspace-free"
+
+    installed = {**root.get("dependencies", {}), **root.get("devDependencies", {})}
+    required = {**client_app.get("dependencies", {}), **client_app.get("devDependencies", {})}
+    missing = sorted(set(required) - set(installed))
+    assert not missing, f"flat root package.json is missing client_app deps: {missing}"

simple_module_cli-0.0.36/tests/test_module_css_subdir_sources.py

@@ -0,0 +1,47 @@
+"""uv's .venv/.gitignore hides subdirectories from Tailwind, so each gets its own @source (#419)."""
+
+from __future__ import annotations
+
+from simple_module_hosting.assets import ModuleAssets, render_modules_css
+
+
+def _wheel(tmp_path, **overrides):
+    defaults = {
+        "name": "Pagebuilder",
+        "package_name": "pagebuilder",
+        "package_dir": tmp_path / "pagebuilder",
+        "pages_dir": None,
+        "theme_css": None,
+        "styles_css": None,
+    }
+    return ModuleAssets(**{**defaults, **overrides})
+
+
+def test_each_subdirectory_with_sources_gets_its_own_line(tmp_path):
+    components = tmp_path / "pagebuilder" / "components"
+    (components / "widgets" / "quote").mkdir(parents=True)
+    (components / "Root.tsx").write_text("")
+    (components / "widgets" / "Gallery.tsx").write_text("")
+    (components / "widgets" / "quote" / "Quote.tsx").write_text("")
+    (components / "empty").mkdir()
+    (components / "__pycache__").mkdir()
+    (components / "__pycache__" / "x.ts").write_text("")
+
+    css = render_modules_css(
+        [_wheel(tmp_path, components_dir=components)], in_repo=lambda _p: False
+    )
+
+    lines = [line for line in css.splitlines() if line.startswith("@source")]
+    assert lines == [
+        f'@source "{components.as_posix()}/**/*.{{ts,tsx}}";',
+        f'@source "{(components / "widgets").as_posix()}/**/*.{{ts,tsx}}";',
+        f'@source "{(components / "widgets" / "quote").as_posix()}/**/*.{{ts,tsx}}";',
+    ]
+
+
+def test_in_repo_module_subdirectories_get_nothing(tmp_path):
+    components = tmp_path / "local" / "components"
+    (components / "widgets").mkdir(parents=True)
+    (components / "widgets" / "A.tsx").write_text("")
+    css = render_modules_css([_wheel(tmp_path, components_dir=components)], in_repo=lambda _p: True)
+    assert "@source" not in css

simple_module_cli-0.0.36/tests/test_scaffolded_host_setup_wizard.py

@@ -0,0 +1,57 @@
+"""A freshly scaffolded host gets a working /setup with no files of its own.
+
+GH #351: ``SetupMiddleware`` redirected every request to ``/setup`` while the
+route and page lived only in the framework repo's unpublished host, so a host
+made by ``smpy create-host`` answered ``/`` → ``/setup`` → 404. The wizard now
+ships with ``simple_module_hosting``: ``create_app`` mounts the route and
+``gen-pages`` registers the page, so the scaffold must stay free of both.
+"""
+
+from __future__ import annotations
+
+import json
+import re
+
+import pytest
+from simple_module_hosting.manifest import write_module_pages_manifest
+from simple_module_hosting.setup_wizard import PAGES_NAME, pages_dir
+
+pytestmark = pytest.mark.anyio
+
+
+async def test_scaffold_carries_no_setup_code_and_resolves_the_wizard(tmp_path) -> None:
+    from simple_module_cli.scaffolding import create_host
+
+    dest = tmp_path / "demo"
+    create_host(dest, name="demo-host", modules=[])
+    client_app = dest / "client_app"
+
+    # Nothing host-side: no route module, no page.
+    assert not (dest / "routes_setup.py").exists()
+    assert not list((client_app / "pages").rglob("Setup*"))
+    assert "setup" not in (dest / "main.py").read_text(encoding="utf-8").replace(
+        "setup_logging", ""
+    )
+
+    # What `smpy gen-pages` writes for this host, with no modules installed.
+    write_module_pages_manifest([], client_app, repo_root=dest)
+
+    manifest = json.loads((client_app / "modules.manifest.json").read_text(encoding="utf-8"))
+    assert manifest[PAGES_NAME] == pages_dir().as_posix()
+    assert (pages_dir() / "Wizard.tsx").is_file()
+
+    generated = (client_app / "modules.generated.ts").read_text(encoding="utf-8")
+    assert f'"{PAGES_NAME}": import.meta.glob' in generated
+
+    # The scaffold's resolver keys a module glob entry as `<name>/<path under pages/>`,
+    # which is how `inertia.render("Setup/Wizard")` finds the page.
+    pages_ts = (client_app / "pages.ts").read_text(encoding="utf-8")
+    assert "pages[`${moduleName}/${match[1]}`]" in pages_ts
+    match = re.search(r"/pages/(.+)\.tsx$", (pages_dir() / "Wizard.tsx").as_posix())
+    assert match and f"{PAGES_NAME}/{match.group(1)}" == "Setup/Wizard"
+
+    # Tailwind must scan the wheel's wizard, and Vite must be allowed to serve it.
+    css = (client_app / "modules.generated.css").read_text(encoding="utf-8")
+    assert f'@source "{pages_dir().as_posix()}/**/*.{{ts,tsx}}";' in css
+    assets = json.loads((client_app / "modules.assets.json").read_text(encoding="utf-8"))
+    assert assets[PAGES_NAME]["pages"] == pages_dir().as_posix()

{simple_module_cli-0.0.34 → simple_module_cli-0.0.36}/tests/test_scaffolding_host.py

@@ -41,6 +41,17 @@ class TestCreateHost:
         ]:
             assert (dest / relpath).exists(), f"missing: {relpath}"
 
+    async def test_pages_ts_excludes_colocated_test_files(self, tmp_path):
+        """Guards template drift from host/client_app/pages.ts: without these
+        negations a colocated test file ships to the production bundle."""
+        from simple_module_cli.scaffolding import create_host
+
+        dest = tmp_path / "demo"
+        create_host(dest, name="demo-host", modules=[])
+        pages_ts = (dest / "client_app" / "pages.ts").read_text(encoding="utf-8")
+        assert "'!./pages/**/*.test.tsx'" in pages_ts
+        assert "'!./pages/**/*.spec.tsx'" in pages_ts
+
     async def test_package_json_carries_host_name(self, tmp_path):
         """client_app/package.json has its `name` prefixed with the host name."""
         from simple_module_cli.scaffolding import create_host

{simple_module_cli-0.0.34 → simple_module_cli-0.0.36}/tests/test_scaffolding_module.py

@@ -30,6 +30,16 @@ class TestCreateModule:
         # pytest try to register `tests.conftest` as a plugin twice.
         assert not (dest / "tests" / "__init__.py").exists()
 
+    async def test_generated_test_uses_effective_route_paths(self, tmp_path):
+        """#431: app.routes hides lazily-included routers on FastAPI >= 0.137."""
+        from simple_module_cli.scaffolding import create_module
+
+        dest = tmp_path / "simple-module-my-feature"
+        create_module(dest, name="MyFeature")
+        text = (dest / "tests" / "test_my_feature.py").read_text()
+        assert "effective_route_paths(app)" in text
+        assert "app.routes" not in text.replace("``app.routes``", "")
+
     async def test_test_file_basename_is_unique_per_module(self, tmp_path):
         """Two modules in one workspace must not ship same-named test files.
 

{simple_module_cli-0.0.34 → simple_module_cli-0.0.36}/tests/test_scaffolding_module_js.py

@@ -92,6 +92,37 @@ class TestStandaloneOverlay:
         assert '"@simple-module-py/tsconfig": "0.0.27"' in pkg
         assert '"typescript"' in pkg
 
+    async def test_standalone_ships_the_types_its_tsconfig_declares(self, tmp_path):
+        """The shared base.json sets ``types: ["vite/client", "node"]``, so a
+        standalone module that doesn't depend on `vite` and `@types/node` fails
+        its own CI `npm run typecheck` with TS2688 "Cannot find type definition
+        file for 'node'/'vite/client'" on the very first run."""
+        import json
+        from pathlib import Path
+
+        from simple_module_cli.scaffolding import create_module
+
+        dest = tmp_path / "simple-module-my-feature"
+        create_module(dest, name="MyFeature", standalone=True)
+
+        base = json.loads(
+            (Path(__file__).parents[3] / "packages" / "tsconfig" / "base.json").read_text(
+                encoding="utf-8"
+            )
+        )
+        declared = base.get("compilerOptions", {}).get("types", [])
+        # "vite/client" is shipped by `vite`; "node" by `@types/node`.
+        providers = {"vite/client": "vite", "node": "@types/node"}
+        pkg = json.loads((dest / "package.json").read_text(encoding="utf-8"))
+        dev_deps = pkg.get("devDependencies", {})
+        for entry in declared:
+            provider = providers.get(entry)
+            assert provider is not None, f"no known provider for tsconfig type {entry!r}"
+            assert provider in dev_deps, (
+                f"base.json declares types {entry!r} but the standalone module "
+                f"scaffold does not depend on {provider!r}"
+            )
+
     async def test_in_repo_keeps_workspace_configs(self, tmp_path):
         from simple_module_cli.scaffolding import create_module