jac-client 0.2.11__tar.gz → 0.2.12__tar.gz

{jac_client-0.2.11 → jac_client-0.2.12}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: jac-client
-Version: 0.2.11
+Version: 0.2.12
 Summary: Build full-stack web applications with Jac - one language for frontend and backend.
 Author-email: Jason Mars <jason@mars.ninja>
 Maintainer-email: Jason Mars <jason@mars.ninja>
@@ -11,7 +11,7 @@ Project-URL: Documentation, https://jac-lang.org
 Keywords: jac,jaclang,jaseci,frontend,full-stack,web-development
 Requires-Python: >=3.12
 Description-Content-Type: text/markdown
-Requires-Dist: jaclang>=0.9.11
+Requires-Dist: jaclang>=0.9.12
 Provides-Extra: dev
 Requires-Dist: python-dotenv==1.0.1; extra == "dev"
 Requires-Dist: pytest==8.3.5; extra == "dev"

{jac_client-0.2.11 → jac_client-0.2.12}/jac_client/plugin/src/impl/vite_bundler.impl.jac

@@ -426,22 +426,29 @@ impl ViteBundler.build(self: ViteBundler, entry_file: Optional[Path] = None) ->
     import sys;
     import time;
     import shutil;
-    import from jac_client.plugin.utils { ensure_bun_available }
+    import from jac_client.plugin.utils { ensure_bun_available, ensure_client_deps }
     # Ensure bun is available before proceeding
     if not ensure_bun_available() {
         raise ClientBundleError('Bun is required. Install manually: https://bun.sh') from None ;
     }
-    self.output_dir.mkdir(parents=True, exist_ok=True);
-    generated_package_json = self._get_client_dir() / 'configs' / 'package.json';
-    if not generated_package_json.exists() {
-        self.create_package_json();
-    } else {
-        # Ensure tsconfig.json exists even if package.json already exists
-        self.create_tsconfig();
+    # Ensure client npm deps are configured; prompt to install defaults if missing
+    if not ensure_client_deps(self.config_loader) {
+        raise ClientBundleError(
+            'Client dependencies not configured. Add [dependencies.npm] to jac.toml '
+            'or create a project with: jac create --use client'
+        ) from None ;
     }
+    self.output_dir.mkdir(parents=True, exist_ok=True);
+    # Always regenerate package.json to pick up any dependency changes
+    self.create_package_json();
     try {
         build_dir = self._get_client_dir();
         node_modules = build_dir / 'node_modules';
+        # Reinstall if node_modules is missing or stale (e.g. empty from a prior failed install)
+        vite_bin = node_modules / '.bin' / 'vite';
+        if node_modules.exists() and not vite_bin.exists() {
+            shutil.rmtree(node_modules);
+        }
         if not node_modules.exists() {
             # Temporarily copy package.json to client build dir for bun install
             build_package_json = build_dir / 'package.json';
@@ -455,15 +462,17 @@ impl ViteBundler.build(self: ViteBundler, entry_file: Optional[Path] = None) ->
                 print("\n  ⏳ Installing dependencies...\n", flush=True);
                 start_time = time.time();
                 result = subprocess.run(
-                    ['bun', 'install'], cwd=build_dir, check=False, text=True
+                    ['bun', 'install'],
+                    cwd=build_dir,
+                    check=False,
+                    text=True,
+                    capture_output=True
                 );
                 elapsed = time.time() - start_time;
                 if result.returncode != 0 {
-                    print(
-                        f"\n  ✖ bun install failed after {elapsed:.1f}s",
-                        file=sys.stderr
-                    );
-                    raise ClientBundleError("Failed to install dependencies") ;
+                    error_output = result.stderr or result.stdout;
+                    error_msg = f"Dependency installation failed after {elapsed:.1f}s\n\n{error_output}\nCommand: bun install";
+                    raise ClientBundleError(error_msg) from None ;
                 }
                 print(f"\n  ✔ Dependencies installed ({elapsed:.1f}s)", flush=True);
             } except FileNotFoundError {
@@ -500,13 +509,16 @@ impl ViteBundler.build(self: ViteBundler, entry_file: Optional[Path] = None) ->
         # Run vite from client build directory so it can find node_modules
         print("\n  ⏳ Building client bundle...\n", flush=True);
         start_time = time.time();
-        result = subprocess.run(command, cwd=build_dir, check=False, text=True);
+        result = subprocess.run(
+            command, cwd=build_dir, check=False, text=True, capture_output=True
+        );
         elapsed = time.time() - start_time;
         if result.returncode != 0 {
-            print(f"\n  ✖ Vite build failed after {elapsed:.1f}s", file=sys.stderr);
-            raise ClientBundleError(
-                f"Vite build failed (see output above)\nCommand: {' '.join(command)}"
-            ) from None ;
+            error_output = result.stderr or result.stdout;
+            error_msg = f"Vite build failed after {elapsed:.1f}s\n\n{error_output}\nCommand: {' '.join(
+                command
+            )}";
+            raise ClientBundleError(error_msg) from None ;
         }
         print(f"\n  ✔ Client bundle built ({elapsed:.1f}s)", flush=True);
     } finally {
@@ -615,6 +627,10 @@ export default defineConfig({{
         target: "http://localhost:{api_port}",
         changeOrigin: true,
       }},
+      "/static": {{
+        target: "http://localhost:{api_port}",
+        changeOrigin: true,
+      }},
     }},
   }},
   resolve: {{

{jac_client-0.2.11 → jac_client-0.2.12}/jac_client/plugin/utils/__init__.jac

@@ -1,3 +1,4 @@
 """Utility modules for jac-client plugin."""
 
 import from .bun_installer { ensure_bun_available, prompt_install_bun }
+import from .client_deps { ensure_client_deps }

jac_client-0.2.12/jac_client/plugin/utils/client_deps.jac

@@ -0,0 +1,14 @@
+"""Client dependency checker utility for jac-client.
+
+Provides a function to check if required npm dependencies are configured
+and prompt the user to install defaults if missing.
+"""
+
+"""Check if client npm dependencies are configured, prompt to install if missing.
+
+Takes a JacClientConfig instance, checks if dependencies and devDependencies
+are both empty, and if so prompts the user to install default jac-client deps.
+
+Returns True if deps are configured (or were just installed), False if user declined.
+"""
+def ensure_client_deps(config_loader: object) -> bool;

jac_client-0.2.12/jac_client/plugin/utils/impl/client_deps.impl.jac

@@ -0,0 +1,73 @@
+"""Implementation of client dependency checker utility."""
+
+"""Check if client npm dependencies are configured, prompt to install if missing."""
+impl ensure_client_deps(config_loader: object) -> bool {
+    package_config = config_loader.get_package_config();
+    dependencies = package_config.get('dependencies', {});
+    dev_dependencies = package_config.get('devDependencies', {});
+    # If either deps or devDeps has entries, assume configured
+    if dependencies or dev_dependencies {
+        return True;
+    }
+    # No deps configured — prompt the user
+    return prompt_install_client_deps(config_loader);
+}
+
+"""Prompt user to install default jac-client npm dependencies."""
+def prompt_install_client_deps(config_loader: object) -> bool {
+    import sys;
+    print("\n  ⚠ Client dependencies are not configured.", file=sys.stderr);
+    print(
+        "  jac-client requires npm packages (react, vite, etc.) to build client pages.",
+        file=sys.stderr
+    );
+    print("  These will be added to your jac.toml file.\n", file=sys.stderr);
+    try {
+        response = input("  Install default client dependencies now? [Y/n]: ").strip().lower();
+    } except (EOFError, KeyboardInterrupt) {
+        print("", file=sys.stderr);
+        return False;
+    }
+    if response and response not in ('y', 'yes', '') {
+        print(
+            "\n  To configure manually, add [dependencies.npm] to your jac.toml.",
+            file=sys.stderr
+        );
+        print(
+            "  Or create a new project with: jac create --use client\n",
+            file=sys.stderr
+        );
+        return False;
+    }
+    # Default runtime dependencies
+    default_deps: dict[str, str] = {
+        'react': '^18.2.0',
+        'react-dom': '^18.2.0',
+        'react-router-dom': '^6.22.0',
+        'react-error-boundary': '^5.0.0'
+    };
+    # Default build infrastructure dependencies
+    default_dev_deps: dict[str, str] = {
+        'vite': '^6.4.1',
+        '@vitejs/plugin-react': '^4.2.1',
+        'typescript': '^5.3.3',
+        '@types/react': '^18.2.0',
+        '@types/react-dom': '^18.2.0'
+    };
+    print("\n  ⏳ Adding default client dependencies to jac.toml...", flush=True);
+    try {
+        for (name, version) in default_deps.items() {
+            config_loader.add_dependency(name, version, is_dev=False);
+        }
+        for (name, version) in default_dev_deps.items() {
+            config_loader.add_dependency(name, version, is_dev=True);
+        }
+        config_loader.save();
+        config_loader.invalidate();
+        print("  ✔ Default client dependencies added to jac.toml\n", flush=True);
+        return True;
+    } except Exception as e {
+        print(f"  ✖ Failed to add dependencies: {e}", file=sys.stderr);
+        return False;
+    }
+}

{jac_client-0.2.11 → jac_client-0.2.12}/jac_client/templates/client.jacpack

@@ -33,10 +33,6 @@
     "assets",
     ".jac/client"
   ],
-  "gitignore_entries": [
-    "# Ignore all build artifacts in .jac directory",
-    "*"
-  ],
   "root_gitignore_entries": [
     "# Jac project",
     "packages/",

{jac_client-0.2.11 → jac_client-0.2.12}/jac_client/templates/fullstack.jacpack

@@ -36,10 +36,6 @@
     ".jac",
     "assets"
   ],
-  "gitignore_entries": [
-    "# Ignore all build artifacts in .jac directory",
-    "*"
-  ],
   "root_gitignore_entries": [
     "# Jac build artifacts",
     ".jac/",

{jac_client-0.2.11 → jac_client-0.2.12}/jac_client/tests/test_cli.py

@@ -1,9 +1,11 @@
 """Test create-jac-app command."""
 
+import contextlib
 import os
 import tempfile
 import tomllib
 from subprocess import PIPE, Popen, run
+from unittest.mock import patch
 
 
 def test_create_jac_app() -> None:
@@ -920,3 +922,69 @@ def test_create_cl_and_run_no_root_files() -> None:
 
         finally:
             os.chdir(original_cwd)
+
+
+def test_vite_build_prompts_for_missing_client_deps() -> None:
+    """Test that ViteBundler.build() prompts to install deps when jac.toml is missing.
+
+    Exercises the same code path as `jac start` → server.start() → ensure_bundle()
+    → ViteBundler.build(), which checks for npm deps before building.
+    """
+    import json
+    from pathlib import Path
+
+    from jac_client.plugin.src.vite_bundler import ViteBundler
+
+    with tempfile.TemporaryDirectory() as temp_dir:
+        project_dir = Path(temp_dir)
+        config_file = project_dir / "jac.toml"
+
+        # No jac.toml — ViteBundler.build() should prompt via ensure_client_deps
+        bundler = ViteBundler(project_dir)
+
+        # Mock input to accept, and mock bun/vite so we don't need real installs.
+        # We only care that ensure_client_deps wrote jac.toml before reaching bun.
+        with (
+            patch("builtins.input", return_value="Y"),
+            patch(
+                "jac_client.plugin.utils.bun_installer.ensure_bun_available",
+                return_value=True,
+            ),
+            patch("subprocess.run") as mock_subprocess,
+        ):
+            # bun install returns success, vite build returns failure
+            # (we don't have real vite — that's fine, we're testing the dep prompt)
+            mock_subprocess.side_effect = [
+                type("Result", (), {"returncode": 0})(),  # bun install
+                type("Result", (), {"returncode": 1})(),  # vite build
+            ]
+            with contextlib.suppress(Exception):
+                bundler.build()  # vite build failure is expected
+
+        # The key assertion: jac.toml was created with default client deps
+        assert config_file.exists(), (
+            "jac.toml should have been created after accepting the prompt"
+        )
+
+        with open(config_file, "rb") as f:
+            data = tomllib.load(f)
+
+        npm_deps = data.get("dependencies", {}).get("npm", {})
+        assert "react" in npm_deps, "react should be in dependencies.npm"
+        assert "react-dom" in npm_deps, "react-dom should be in dependencies.npm"
+
+        npm_dev = npm_deps.get("dev", {})
+        assert "vite" in npm_dev, "vite should be in dev dependencies"
+        assert "@vitejs/plugin-react" in npm_dev, (
+            "@vitejs/plugin-react should be in dev deps"
+        )
+
+        # Verify the generated package.json also picked up the deps
+        package_json = project_dir / ".jac" / "client" / "configs" / "package.json"
+        assert package_json.exists(), "package.json should have been generated"
+
+        with open(package_json) as f:
+            pkg = json.load(f)
+
+        assert pkg["dependencies"].get("react"), "package.json should have react"
+        assert pkg["devDependencies"].get("vite"), "package.json should have vite"

{jac_client-0.2.11 → jac_client-0.2.12}/jac_client.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: jac-client
-Version: 0.2.11
+Version: 0.2.12
 Summary: Build full-stack web applications with Jac - one language for frontend and backend.
 Author-email: Jason Mars <jason@mars.ninja>
 Maintainer-email: Jason Mars <jason@mars.ninja>
@@ -11,7 +11,7 @@ Project-URL: Documentation, https://jac-lang.org
 Keywords: jac,jaclang,jaseci,frontend,full-stack,web-development
 Requires-Python: >=3.12
 Description-Content-Type: text/markdown
-Requires-Dist: jaclang>=0.9.11
+Requires-Dist: jaclang>=0.9.12
 Provides-Extra: dev
 Requires-Dist: python-dotenv==1.0.1; extra == "dev"
 Requires-Dist: pytest==8.3.5; extra == "dev"

{jac_client-0.2.11 → jac_client-0.2.12}/jac_client.egg-info/SOURCES.txt

@@ -94,7 +94,9 @@ jac_client/plugin/src/targets/impl/registry.impl.jac
 jac_client/plugin/src/targets/impl/web_target.impl.jac
 jac_client/plugin/utils/__init__.jac
 jac_client/plugin/utils/bun_installer.jac
+jac_client/plugin/utils/client_deps.jac
 jac_client/plugin/utils/impl/bun_installer.impl.jac
+jac_client/plugin/utils/impl/client_deps.impl.jac
 jac_client/templates/client.jacpack
 jac_client/templates/fullstack.jacpack
 jac_client/tests/__init__.py

{jac_client-0.2.11 → jac_client-0.2.12}/jac_client.egg-info/requires.txt

@@ -1,4 +1,4 @@
-jaclang>=0.9.11
+jaclang>=0.9.12
 
 [dev]
 python-dotenv==1.0.1

{jac_client-0.2.11 → jac_client-0.2.12}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "jac-client"
-version = "0.2.11"
+version = "0.2.12"
 description = "Build full-stack web applications with Jac - one language for frontend and backend."
 authors = [{name = "Jason Mars", email = "jason@mars.ninja"}]
 maintainers = [{name = "Jason Mars", email = "jason@mars.ninja"}]
@@ -16,7 +16,7 @@ keywords = [
 ]
 requires-python = ">=3.12"
 dependencies = [
-    "jaclang>=0.9.11",
+    "jaclang>=0.9.12",
 ]
 
 [project.urls]