@pracht/vite-plugin 0.7.2 → 0.7.4

package/dist/index.mjs

@@ -1,12 +1,12 @@
-import { i as scanPagesDirectory, n as generatePagesManifestSource, o as createRouteLoaderHints } from "./pages-router-BsVlzz-e.mjs";
+import { i as scanPagesDirectory, n as generatePagesManifestSource, o as createRouteLoaderHints } from "./pages-router-CStJY2Zp.mjs";
 import { createRequire } from "node:module";
 import { preactSsrPrecompile } from "@pracht/preact-ssr-precompile";
 import preact from "@preact/preset-vite";
+import { existsSync, readFileSync, readdirSync, realpathSync, statSync } from "node:fs";
 import { dirname, extname, join, resolve } from "node:path";
 import { loadEnv, parseAst } from "vite";
-import { existsSync, readFileSync, readdirSync, statSync } from "node:fs";
-import { CAPABILITY_SETTLED_EVENT, CAPABILITY_TRANSPORT_HEADER, CONFIRMATION_HEADER, capabilityHttpPath, isValidCapabilityHttpPath } from "@pracht/capabilities";
-import { evaluateLiteral, extractCapabilityRegistrations, extractDefineCapabilityArgs, scanTopLevelProperties } from "@pracht/capabilities/static";
+import { CAPABILITY_SETTLED_EVENT, CAPABILITY_TRANSPORT_HEADER, CONFIRMATION_HEADER } from "@pracht/capabilities";
+import { extractCapabilityProjection, extractCapabilityRegistrations } from "@pracht/capabilities/static";
 import { createNodeServerEntryModule } from "@pracht/adapter-node";
 import { resolveRegistryModule } from "@pracht/core";
 //#region src/client-module-query.ts
@@ -1219,6 +1219,35 @@ function createEnvSafetyPlugin(envSafety) {
 	};
 }
 //#endregion
+//#region src/hot-update-reload.ts
+/**
+* True when `file` participates in server rendering but has no runtime
+* counterpart in the client module graph, meaning client HMR can never deliver
+* its change. File-only asset entries created by content scanners are watchers,
+* not browser modules, so they do not make an update client-reachable.
+*/
+function isServerOnlyModuleFile(server, file) {
+	const environments = server.environments;
+	const client = environments?.client;
+	if (!client) return false;
+	if (hasRuntimeModules(client, file)) return false;
+	for (const [name, environment] of Object.entries(environments ?? {})) {
+		if (name === "client" || !environment) continue;
+		if (hasRuntimeModules(environment, file)) return true;
+	}
+	return false;
+}
+/** Reload open pages when `file` can only reach them through the server. */
+function sendServerOnlyFullReload(server, file) {
+	if (!isServerOnlyModuleFile(server, file)) return false;
+	server.environments?.client?.hot?.send({ type: "full-reload" });
+	return true;
+}
+function hasRuntimeModules(environment, file) {
+	for (const module of environment.moduleGraph.getModulesByFile(file) ?? []) if (module.type !== "asset") return true;
+	return false;
+}
+//#endregion
 //#region src/plugin-assets.ts
 const PRACHT_CLIENT_MODULE_ID = "virtual:pracht/client";
 const PRACHT_SERVER_MODULE_ID = "virtual:pracht/server";
@@ -1415,57 +1444,37 @@ function extractCapabilities(options = {}, root = process.cwd()) {
 		return extractCapabilityMetadata(name, file, source);
 	});
 }
-function extractCapabilityMetadata(name, file, source) {
-	const args = extractDefineCapabilityArgs(source);
-	if (!args) throw new Error(`[pracht] Capability "${name}" (${file}) does not contain a defineCapability({ ... }) call the build can analyze.`);
-	const properties = scanTopLevelProperties(args);
-	const exposeText = properties.get("expose");
-	if (!exposeText) return {
-		name,
-		file,
-		description: "",
-		effect: null,
-		httpPath: null,
-		webmcp: false,
-		inputSchema: null
-	};
-	const expose = evaluateLiteral(exposeText);
-	if (!isPlainObject(expose)) throw new Error(`[pracht] Capability "${name}" (${file}): "expose" must be an inline object literal so the client projection can be generated at build time.`);
-	const http = expose.http;
-	let httpPath = null;
-	if (http === true) httpPath = capabilityHttpPath(name);
-	else if (isPlainObject(http)) httpPath = typeof http.path === "string" ? http.path : capabilityHttpPath(name);
-	if (httpPath && !isValidCapabilityHttpPath(httpPath)) throw new Error(`[pracht] Capability "${name}" (${file}): HTTP exposure "path" must be an exact same-origin pathname starting with "/".`);
-	const webmcp = expose.webmcp === true;
-	if (webmcp && !httpPath) throw new Error(`[pracht] Capability "${name}" (${file}): expose.webmcp requires expose.http.`);
-	let description = "";
-	const descriptionText = properties.get("description");
-	if (descriptionText) {
-		const value = evaluateLiteral(descriptionText);
-		if (typeof value === "string") description = value;
-	}
-	let effect = null;
-	const effectText = properties.get("effect");
-	if (effectText) {
-		const value = evaluateLiteral(effectText);
-		if (typeof value === "string") effect = value;
-	}
-	if (httpPath && effect !== "read" && effect !== "write" && effect !== "destructive") throw new Error(`[pracht] Capability "${name}" (${file}) is exposed via HTTP, but its "effect" could not be extracted at build time. HTTP-exposed capabilities must declare "effect" as an inline "read", "write", or "destructive" string literal.`);
-	let inputSchema = null;
-	if (webmcp) {
-		const inputText = properties.get("input");
-		const value = inputText ? evaluateLiteral(inputText) : void 0;
-		if (!isPlainObject(value)) throw new Error(`[pracht] Capability "${name}" (${file}) is exposed via WebMCP, but its "input" schema could not be extracted at build time. WebMCP-exposed capabilities must declare their input schema as an inline object literal.`);
-		inputSchema = value;
+/**
+* Absolute paths of the capability modules the manifest registers.
+*
+* The client-import guard uses this rather than a `capabilitiesDir` prefix
+* test: registration is what makes a module server-only, and the manifest may
+* point anywhere. A directory test both misses capabilities registered from
+* elsewhere and wrongly rejects ordinary co-located files (shared constants,
+* types) that happen to sit in the capability folder.
+*
+* Returns an empty list when the manifest cannot be read or parsed — the
+* virtual-module generation raises its own precise error for those, and
+* guessing here would turn one clear failure into two confusing ones.
+*/
+function resolveCapabilityModulePaths(options = {}, root = process.cwd()) {
+	const resolved = resolveOptions(options);
+	if (resolved.pagesDir) return [];
+	const appFileAbs = resolve(root, resolved.appFile.replace(/^\//, ""));
+	let manifestSource;
+	try {
+		manifestSource = readFileSync(appFileAbs, "utf-8");
+	} catch {
+		return [];
 	}
+	const appDir = dirname(appFileAbs);
+	return extractCapabilityRegistrations(manifestSource).map(({ file }) => file.startsWith("/") ? resolve(root, file.replace(/^\//, "")) : resolve(appDir, file));
+}
+function extractCapabilityMetadata(name, file, source) {
 	return {
 		name,
 		file,
-		description,
-		effect,
-		httpPath,
-		webmcp,
-		inputSchema
+		...extractCapabilityProjection(name, source, (detail) => `[pracht] Capability ${JSON.stringify(name)} (${file}) ${detail}`)
 	};
 }
 /**
@@ -1643,9 +1652,6 @@ function hasWebmcpCapabilities(options = {}, root = process.cwd()) {
 		return true;
 	}
 }
-function isPlainObject(value) {
-	return typeof value === "object" && value !== null && !Array.isArray(value);
-}
 //#endregion
 //#region src/plugin-codegen.ts
 const ROUTE_MODULE_EXTENSIONS = new Set([
@@ -2484,6 +2490,7 @@ function pracht(options = {}) {
 	const isPagesMode = !!resolved.pagesDir;
 	let root = process.cwd();
 	let routeFileDirs = [];
+	let capabilityModulePaths = /* @__PURE__ */ new Set();
 	if (isPagesMode && options.appFile) console.warn("[pracht] Both `pagesDir` and `appFile` are set. `pagesDir` takes precedence — `appFile` will be ignored.");
 	let isBuild = false;
 	const prachtPlugin = {
@@ -2519,6 +2526,7 @@ function pracht(options = {}) {
 			root = config.root;
 			isBuild = config.command === "build";
 			routeFileDirs = computeRouteFileDirs(root, resolved);
+			capabilityModulePaths = new Set(resolveCapabilityModulePaths(resolved, root).map(canonicalFilePath));
 		},
 		resolveId(id, importer, resolveIdOptions) {
 			if (isIslandsClientModule(id)) return PRACHT_ISLANDS_CLIENT_MODULE_ID;
@@ -2543,8 +2551,8 @@ function pracht(options = {}) {
 			return null;
 		},
 		transform(code, id) {
-			const appFileAbs = resolveConfigPath(root, resolved.appFile);
-			if (toPosixPath(id.split("?")[0]) !== appFileAbs) return null;
+			const appFileAbs = canonicalFilePath(resolveConfigPath(root, resolved.appFile));
+			if (canonicalFilePath(id.split("?")[0]) !== appFileAbs) return null;
 			const transformed = rewriteManifestCoreImports(code.replace(/\(\)\s*=>\s*import\(\s*(['"])([^'"]+)\1\s*\)/g, "$1$2$1"));
 			if (transformed === code) return null;
 			return {
@@ -2580,6 +2588,7 @@ function pracht(options = {}) {
 			if (isPagesMode && relative.startsWith(resolved.pagesDir)) {
 				clearPagesAppSourceCache();
 				invalidateVirtualModules(server);
+				sendServerOnlyFullReload(server, file);
 				return;
 			}
 			if (!isPagesMode && relative === resolved.appFile) {
@@ -2617,12 +2626,14 @@ function pracht(options = {}) {
 					if (capabilityMod) server.moduleGraph.invalidateModule(capabilityMod);
 				}
 			}
+			sendServerOnlyFullReload(server, file);
 		}
 	};
 	const clientModuleTransformPlugin = {
 		name: "pracht:client-module-transform",
 		enforce: "post",
 		transform(code, id, transformOptions) {
+			if (!transformOptions?.ssr && isCapabilityModule(id, capabilityModulePaths)) throw new Error(`[pracht] Capability module ${JSON.stringify(toPosixPath(id))} was imported by client code. Capability modules are server-only — their run() implementation and its imports would be bundled for every visitor. Call the capability instead: \`callCapability\`/\`capabilities\` from "virtual:pracht/capabilities" in the browser, or \`invokeCapability\` from "@pracht/core/server" in loaders, middleware, and API routes.`);
 			if (!(isPrachtClientModuleId(id) || !transformOptions?.ssr && isRouteOrShellFile(id, routeFileDirs))) return null;
 			const transformed = stripServerOnlyExportsForClient(code, id);
 			if (transformed === code) return null;
@@ -2752,7 +2763,34 @@ const ROUTE_FILE_EXTENSIONS = new Set([
 	".tsrx"
 ]);
 function computeRouteFileDirs(root, resolved) {
-	return (resolved.pagesDir ? [resolved.pagesDir] : [resolved.routesDir, resolved.shellsDir]).map((dir) => resolveConfigPath(root, dir)).map(withTrailingSep);
+	return (resolved.pagesDir ? [resolved.pagesDir] : [resolved.routesDir, resolved.shellsDir]).map((dir) => canonicalFilePath(resolveConfigPath(root, dir))).map(withTrailingSep);
+}
+/**
+* Whether `id` is one of the capability modules the manifest registers.
+* Matching the registered set rather than a directory keeps ordinary files that
+* merely live beside capabilities importable, and still catches a capability
+* registered from anywhere else in the project. Extension-agnostic, because the
+* comparison is against paths the manifest already resolved.
+*/
+function isCapabilityModule(id, capabilityModulePaths) {
+	if (capabilityModulePaths.size === 0) return false;
+	const queryStart = id.indexOf("?");
+	const path = queryStart === -1 ? id : id.slice(0, queryStart);
+	if (path.startsWith("\0") || path.startsWith("virtual:")) return false;
+	return capabilityModulePaths.has(canonicalFilePath(path));
+}
+/**
+* Match Vite's canonical module ids even when the manifest path crosses a
+* symlink (including macOS' /var -> /private/var alias). Missing paths keep
+* their lexical identity so the projection code can raise its precise missing
+* capability error later.
+*/
+function canonicalFilePath(path) {
+	try {
+		return toPosixPath(realpathSync.native(path));
+	} catch {
+		return toPosixPath(path);
+	}
 }
 function isRouteOrShellFile(id, dirs) {
 	if (dirs.length === 0) return false;

package/dist/pages-router.mjs

@@ -1,2 +1,2 @@
-import { a as sortRoutes, i as scanPagesDirectory, n as generatePagesManifestSource, r as generateRoutesFile, t as filePathToRoutePath } from "./pages-router-BsVlzz-e.mjs";
+import { a as sortRoutes, i as scanPagesDirectory, n as generatePagesManifestSource, r as generateRoutesFile, t as filePathToRoutePath } from "./pages-router-CStJY2Zp.mjs";
 export { filePathToRoutePath, generatePagesManifestSource, generateRoutesFile, scanPagesDirectory, sortRoutes };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pracht/vite-plugin",
-  "version": "0.7.2",
+  "version": "0.7.4",
   "description": "Vite plugin for Pracht apps with virtual modules, dev SSR, prerendering, route inspection, and multi-adapter builds.",
   "keywords": [
     "pracht",
@@ -44,10 +44,10 @@
   "dependencies": {
     "@preact/preset-vite": "^2.10.5",
     "@prefresh/vite": "^2.0.0",
-    "@pracht/adapter-node": "0.3.5",
-    "@pracht/capabilities": "0.1.0",
-    "@pracht/core": "0.11.2",
-    "@pracht/preact-ssr-precompile": "0.1.2"
+    "@pracht/core": "0.11.3",
+    "@pracht/adapter-node": "0.3.6",
+    "@pracht/preact-ssr-precompile": "0.1.2",
+    "@pracht/capabilities": "0.1.1"
   },
   "peerDependencies": {
     "vite": "^8.0.0"

package/dist/{pages-router-BsVlzz-e.mjs → pages-router-CStJY2Zp.mjs}

@@ -1,5 +1,5 @@
-import { basename, extname, join, relative } from "node:path";
 import { readFileSync, readdirSync, statSync, writeFileSync } from "node:fs";
+import { basename, extname, join, relative } from "node:path";
 //#region src/route-loader-hints.ts
 const ROUTE_EXTENSIONS = new Set([
 	".tsx",