pulse-framework 0.1.38a1__tar.gz → 0.1.38a2__tar.gz

{pulse_framework-0.1.38a1 → pulse_framework-0.1.38a2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: pulse-framework
-Version: 0.1.38a1
+Version: 0.1.38a2
 Summary: Pulse - Full-stack framework for building real-time React applications in Python
 Requires-Dist: websockets>=12.0
 Requires-Dist: fastapi>=0.104.0

{pulse_framework-0.1.38a1 → pulse_framework-0.1.38a2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "pulse-framework"
-version = "0.1.38a1"
+version = "0.1.38a2"
 description = "Pulse - Full-stack framework for building real-time React applications in Python"
 readme = "README.md"
 requires-python = ">=3.11"

{pulse_framework-0.1.38a1 → pulse_framework-0.1.38a2}/src/pulse/app.py

@@ -165,7 +165,6 @@ class App:
 		internal_server_address: str | None = None,
 		not_found: str = "/not-found",
 		# Deployment and integration options
-		env: PulseEnv | None = None,
 		mode: PulseMode = "single-server",
 		api_prefix: str = "/_pulse",
 		cors: CORSOptions | None = None,
@@ -179,7 +178,7 @@ class App:
 		    codegen: Optional codegen configuration.
 		"""
 		# Resolve mode from environment and expose on the app instance
-		self.env = env or envvars.pulse_env
+		self.env = envvars.pulse_env
 		self.mode = mode
 		self.status = AppStatus.created
 		# Persist the server address for use by sessions (API calls, etc.)
@@ -326,9 +325,9 @@ class App:
 				)
 			# Production: use Node to serve the built bundle (Bun lacks renderToPipeableStream)
 			cmd = [
-				"node",
-				str(cli_path),
-				"./build/server/index.js",
+				"bun",
+				"run",
+				"start",
 				"--port",
 				str(port),
 			]
@@ -464,11 +463,35 @@ class App:
 			self.fastapi.add_middleware(CORSMiddleware, **self.cors)
 		else:
 			# Use deployment-specific CORS settings
+			cors_config = cors_options(self.mode, self.server_address)
+			print(f"CORS config: {cors_config}")
 			self.fastapi.add_middleware(
 				CORSMiddleware,
-				**cors_options(self.mode, self.server_address),
+				**cors_config,
 			)
 
+		# Debug middleware to log CORS-related request details
+		@self.fastapi.middleware("http")
+		async def cors_debug_middleware(  # pyright: ignore[reportUnusedFunction]
+			request: Request, call_next: Callable[[Request], Awaitable[Response]]
+		):
+			origin = request.headers.get("origin")
+			method = request.method
+			path = request.url.path
+			print(
+				f"[CORS Debug] {method} {path} | Origin: {origin} | "
+				+ f"Mode: {self.mode} | Server: {self.server_address}"
+			)
+			response = await call_next(request)
+			allow_origin = response.headers.get("access-control-allow-origin")
+			if allow_origin:
+				print(f"[CORS Debug] Response allows origin: {allow_origin}")
+			elif origin:
+				logger.warning(
+					f"[CORS Debug] Origin {origin} present but no Access-Control-Allow-Origin header set"
+				)
+			return response
+
 		# Mount PulseContext for all FastAPI routes (no route info). Other API
 		# routes / middleware should be added at the module-level, which means
 		# this middleware will wrap all of them.

{pulse_framework-0.1.38a1 → pulse_framework-0.1.38a2}/src/pulse/cli/cmd.py

@@ -21,6 +21,7 @@ from pulse.cli.dependencies import (
 	DependencyError,
 	DependencyPlan,
 	DependencyResolutionError,
+	check_web_dependencies,
 	prepare_web_dependencies,
 )
 from pulse.cli.folder_lock import FolderLock
@@ -68,7 +69,7 @@ def run(
 	prod: bool = typer.Option(False, "--prod", help="Run in production env"),
 	server_only: bool = typer.Option(False, "--server-only", "--backend-only"),
 	web_only: bool = typer.Option(False, "--web-only"),
-	reload: bool = typer.Option(True, "--reload/--no-reload"),
+	reload: bool | None = typer.Option(None, "--reload/--no-reload"),
 	find_port: bool = typer.Option(True, "--find-port/--no-find-port"),
 ):
 	"""Run the Pulse server and web development server together."""
@@ -88,6 +89,10 @@ def run(
 	if len(env_flags) == 1:
 		env.pulse_env = cast(PulseEnv, env_flags[0])
 
+	# Turn on reload in dev only
+	if reload is None:
+		reload = env.pulse_env == "dev"
+
 	if server_only and web_only:
 		typer.echo("❌ Cannot use --server-only and --web-only at the same time.")
 		raise typer.Exit(1)
@@ -121,10 +126,10 @@ def run(
 		)
 
 	if not server_only:
-		# Skip dependency installation in production mode (assumes pre-built)
-		if app_instance.env != "prod":
+		# Skip dependency checking and installation in production/CI mode (assumes pre-built)
+		if env.pulse_env == "dev":
 			try:
-				dep_plan = prepare_web_dependencies(
+				to_add = check_web_dependencies(
 					web_root,
 					pulse_version=PULSE_PY_VERSION,
 				)
@@ -135,9 +140,14 @@ def run(
 				console.log(f"❌ {exc}")
 				raise typer.Exit(1) from None
 
-			if dep_plan:
+			if to_add:
 				try:
-					_run_dependency_plan(console, web_root, dep_plan)
+					dep_plan = prepare_web_dependencies(
+						web_root,
+						pulse_version=PULSE_PY_VERSION,
+					)
+					if dep_plan:
+						_run_dependency_plan(console, web_root, dep_plan)
 				except subprocess.CalledProcessError:
 					console.log("❌ Failed to install web dependencies with Bun.")
 					raise typer.Exit(1) from None
@@ -219,6 +229,66 @@ def generate(
 		console.log("⚠️  No routes found to generate")
 
 
+@cli.command("check")
+def check(
+	app_file: str = typer.Argument(
+		..., help="App target: 'path.py[:var]' (default :app) or 'module:var'"
+	),
+	fix: bool = typer.Option(
+		False, "--fix", help="Install missing or outdated dependencies"
+	),
+):
+	"""Check if web project dependencies are in sync with Pulse app requirements."""
+	console = Console()
+
+	console.log(f"📁 Loading app from: {app_file}")
+	app_ctx = load_app_from_target(app_file)
+	_apply_app_context_to_env(app_ctx)
+	app_instance = app_ctx.app
+
+	web_root = app_instance.codegen.cfg.web_root
+	if not web_root.exists():
+		console.log(f"❌ Directory not found: {web_root.absolute()}")
+		raise typer.Exit(1)
+
+	try:
+		to_add = check_web_dependencies(
+			web_root,
+			pulse_version=PULSE_PY_VERSION,
+		)
+	except DependencyResolutionError as exc:
+		console.log(f"❌ {exc}")
+		raise typer.Exit(1) from None
+	except DependencyError as exc:
+		console.log(f"❌ {exc}")
+		raise typer.Exit(1) from None
+
+	if not to_add:
+		console.log("✅ Web dependencies are in sync")
+		return
+
+	console.log("📦 Web dependencies are out of sync:")
+	for pkg in to_add:
+		console.log(f"  - {pkg}")
+
+	if not fix:
+		console.log("💡 Run 'pulse check --fix' to install missing dependencies")
+		return
+
+	# Apply fix
+	try:
+		dep_plan = prepare_web_dependencies(
+			web_root,
+			pulse_version=PULSE_PY_VERSION,
+		)
+		if dep_plan:
+			_run_dependency_plan(console, web_root, dep_plan)
+		console.log("✅ Web dependencies synced successfully")
+	except subprocess.CalledProcessError:
+		console.log("❌ Failed to install web dependencies with Bun.")
+		raise typer.Exit(1) from None
+
+
 def build_uvicorn_command(
 	*,
 	app_ctx: AppLoadResult,
@@ -245,7 +315,7 @@ def build_uvicorn_command(
 		"--factory",
 	]
 
-	if reload_enabled and app_ctx.app.env != "prod":
+	if reload_enabled:
 		args.append("--reload")
 		args.extend(["--reload-include", "*.css"])
 		app_dir = app_ctx.app_dir or Path.cwd()

{pulse_framework-0.1.38a1 → pulse_framework-0.1.38a2}/src/pulse/cli/dependencies.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+import re
 from collections.abc import Callable, Iterable, Sequence
 from dataclasses import dataclass
 from pathlib import Path
@@ -18,6 +19,53 @@ from pulse.cli.packages import (
 from pulse.react_component import ReactComponent, registered_react_components
 
 
+def convert_pep440_to_semver(python_version: str) -> str:
+	"""Convert PEP 440 version format to NPM semver format.
+
+	PEP 440 formats:
+	- 0.1.37a1 -> 0.1.37-alpha.1
+	- 0.1.37b1 -> 0.1.37-beta.1
+	- 0.1.37rc1 -> 0.1.37-rc.1
+	- 0.1.37.dev1 -> 0.1.37-dev.1
+
+	Non-pre-release versions are returned unchanged.
+	"""
+	# Match pre-release patterns: version followed by a/b/rc/dev + number
+	# PEP 440: a1, b1, rc1, dev1, alpha1, beta1, etc.
+	pattern = r"^(\d+\.\d+\.\d+)([a-z]+)(\d+)$"
+	match = re.match(pattern, python_version)
+
+	if match:
+		base_version = match.group(1)
+		prerelease_type = match.group(2)
+		prerelease_num = match.group(3)
+
+		# Map PEP 440 prerelease types to NPM semver
+		type_map = {
+			"a": "alpha",
+			"alpha": "alpha",
+			"b": "beta",
+			"beta": "beta",
+			"rc": "rc",
+			"c": "rc",  # PEP 440 also allows 'c' for release candidate
+			"dev": "dev",
+		}
+
+		npm_type = type_map.get(prerelease_type.lower(), prerelease_type)
+		return f"{base_version}-{npm_type}.{prerelease_num}"
+
+	# Also handle .dev format (e.g., 0.1.37.dev1)
+	pattern2 = r"^(\d+\.\d+\.\d+)\.dev(\d+)$"
+	match2 = re.match(pattern2, python_version)
+	if match2:
+		base_version = match2.group(1)
+		dev_num = match2.group(2)
+		return f"{base_version}-dev.{dev_num}"
+
+	# No pre-release, return as-is
+	return python_version
+
+
 class DependencyError(RuntimeError):
 	"""Base error for dependency preparation failures."""
 
@@ -38,15 +86,15 @@ class DependencyPlan:
 	to_add: Sequence[str]
 
 
-def prepare_web_dependencies(
+def get_required_dependencies(
 	web_root: Path,
 	*,
 	pulse_version: str,
 	component_provider: Callable[
 		[], Iterable[ReactComponent[Any]]
 	] = registered_react_components,
-) -> DependencyPlan | None:
-	"""Inspect registered components and return the Bun command needed to sync dependencies."""
+) -> dict[str, str | None]:
+	"""Get the required dependencies for a Pulse app."""
 	if not web_root.exists():
 		raise DependencyError(f"Directory not found: {web_root}")
 
@@ -102,13 +150,30 @@ def prepare_web_dependencies(
 	]:
 		desired.setdefault(pkg, "^7")
 
+	return desired
+
+
+def check_web_dependencies(
+	web_root: Path,
+	*,
+	pulse_version: str,
+	component_provider: Callable[
+		[], Iterable[ReactComponent[Any]]
+	] = registered_react_components,
+) -> list[str]:
+	"""Check if web dependencies are in sync and return list of packages that need to be added/updated."""
+	desired = get_required_dependencies(
+		web_root=web_root,
+		pulse_version=pulse_version,
+		component_provider=component_provider,
+	)
 	pkg_json = load_package_json(web_root)
 
 	to_add: list[str] = []
 	for name, req_ver in sorted(desired.items()):
 		effective = req_ver
 		if name == "pulse-ui-client":
-			effective = pulse_version
+			effective = convert_pep440_to_semver(pulse_version)
 
 		existing = get_pkg_spec(pkg_json, name)
 		if existing is None:
@@ -123,6 +188,24 @@ def prepare_web_dependencies(
 
 		to_add.append(f"{name}@{effective}" if effective else name)
 
+	return to_add
+
+
+def prepare_web_dependencies(
+	web_root: Path,
+	*,
+	pulse_version: str,
+	component_provider: Callable[
+		[], Iterable[ReactComponent[Any]]
+	] = registered_react_components,
+) -> DependencyPlan | None:
+	"""Inspect registered components and return the Bun command needed to sync dependencies."""
+	to_add = check_web_dependencies(
+		web_root=web_root,
+		pulse_version=pulse_version,
+		component_provider=component_provider,
+	)
+
 	if to_add:
 		return DependencyPlan(command=["bun", "add", *to_add], to_add=to_add)
 

{pulse_framework-0.1.38a1 → pulse_framework-0.1.38a2}/src/pulse/cookies.py

@@ -155,10 +155,15 @@ def cors_options(mode: "PulseMode", server_address: str) -> CORSOptions:
 	}
 	if mode == "subdomains":
 		base = _base_domain(host)
+		# Escape dots in base domain for regex (doesn't affect localhost since it has no dots)
+		base = base.replace(".", r"\.")
+		# Allow any subdomain and any port for the base domain
 		opts["allow_origin_regex"] = rf"^https?://([a-z0-9-]+\\.)?{base}(:\\d+)?$"
 		return opts
 	elif mode == "single-server":
 		# For single-server mode, allow same origin
+		# Escape dots in host for regex (doesn't affect localhost since it has no dots)
+		host = host.replace(".", r"\.")
 		opts["allow_origin_regex"] = rf"^https?://{host}(:\\d+)?$"
 		return opts
 	else:

{pulse_framework-0.1.38a1 → pulse_framework-0.1.38a2}/src/pulse/env.py

@@ -39,13 +39,12 @@ class EnvVars:
 		else:
 			os.environ[key] = value
 
-	# Pulse mode
 	@property
 	def pulse_env(self) -> PulseEnv:
 		value = (self._get(ENV_PULSE_MODE) or "dev").lower()
 		if value not in ("dev", "ci", "prod"):
 			value = "dev"
-		return value  # type: ignore[return-value]
+		return value
 
 	@pulse_env.setter
 	def pulse_env(self, value: PulseEnv) -> None: