@pracht/vite-plugin 0.7.2 → 0.7.3

package/dist/index.mjs

@@ -5,8 +5,8 @@ import preact from "@preact/preset-vite";
 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,32 @@ function createEnvSafetyPlugin(envSafety) {
 	};
 }
 //#endregion
+//#region src/hot-update-reload.ts
+/**
+* True when `file` participates in server rendering but has no counterpart in
+* the client module graph, meaning client HMR can never deliver its change.
+*/
+function isServerOnlyModuleFile(server, file) {
+	const environments = server.environments;
+	const client = environments?.client;
+	if (!client) return false;
+	if (hasModules(client, file)) return false;
+	for (const [name, environment] of Object.entries(environments ?? {})) {
+		if (name === "client" || !environment) continue;
+		if (hasModules(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 hasModules(environment, file) {
+	return (environment.moduleGraph.getModulesByFile(file)?.size ?? 0) > 0;
+}
+//#endregion
 //#region src/plugin-assets.ts
 const PRACHT_CLIENT_MODULE_ID = "virtual:pracht/client";
 const PRACHT_SERVER_MODULE_ID = "virtual:pracht/server";
@@ -1415,57 +1441,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 +1649,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 +2487,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 +2523,7 @@ function pracht(options = {}) {
 			root = config.root;
 			isBuild = config.command === "build";
 			routeFileDirs = computeRouteFileDirs(root, resolved);
+			capabilityModulePaths = new Set(resolveCapabilityModulePaths(resolved, root).map((path) => toPosixPath(path)));
 		},
 		resolveId(id, importer, resolveIdOptions) {
 			if (isIslandsClientModule(id)) return PRACHT_ISLANDS_CLIENT_MODULE_ID;
@@ -2580,6 +2585,7 @@ function pracht(options = {}) {
 			if (isPagesMode && relative.startsWith(resolved.pagesDir)) {
 				clearPagesAppSourceCache();
 				invalidateVirtualModules(server);
+				sendServerOnlyFullReload(server, file);
 				return;
 			}
 			if (!isPagesMode && relative === resolved.appFile) {
@@ -2617,12 +2623,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;
@@ -2754,6 +2762,20 @@ const ROUTE_FILE_EXTENSIONS = new Set([
 function computeRouteFileDirs(root, resolved) {
 	return (resolved.pagesDir ? [resolved.pagesDir] : [resolved.routesDir, resolved.shellsDir]).map((dir) => 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(toPosixPath(path));
+}
 function isRouteOrShellFile(id, dirs) {
 	if (dirs.length === 0) return false;
 	const queryStart = id.indexOf("?");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pracht/vite-plugin",
-  "version": "0.7.2",
+  "version": "0.7.3",
   "description": "Vite plugin for Pracht apps with virtual modules, dev SSR, prerendering, route inspection, and multi-adapter builds.",
   "keywords": [
     "pracht",
@@ -44,9 +44,9 @@
   "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/adapter-node": "0.3.6",
+    "@pracht/capabilities": "0.1.1",
+    "@pracht/core": "0.11.3",
     "@pracht/preact-ssr-precompile": "0.1.2"
   },
   "peerDependencies": {