@samva/vite 0.3.0 → 0.4.0

package/AGENTS.md

@@ -13,7 +13,13 @@ the fetched `origin` default branch. Publish creates an immutable receipt; it do
 rewrite history.
 
 Template discovery is `.tsx` only. The local email editor writes the canonical TSX entry with a
-revision check and a fresh comparison against disk; SMS and WhatsApp stay read-only. Every discovered entry is imported once through the dev
+revision check and a fresh comparison against disk; SMS and WhatsApp stay read-only. A change
+published without a client is what the editor reads as a foreign edit, so nothing the editor itself
+did may reach it that way. A save and a fixture switch both run in the store's mutation queue, so no
+rebuild can straddle one and publish the difference anonymously; the editor's write comes back
+through the file watcher like any other edit, and a changed file that still holds the bytes its
+entry was built from re-evaluates nothing. Watcher events arrive in bursts and one rebuild re-reads
+the whole project, so a rebuild that has not started yet covers every event queued before it. Every discovered entry is imported once through the dev
 server's own SSR graph, and a default export that is a template definition takes the email lane —
 compiled and rendered through `compileEmailProject`, so a local render cannot drift from a published
 one. Email requires an executable definition. Structured SMS/WhatsApp entries emit SML, and

package/dist/index.js

@@ -248,12 +248,39 @@ var EditorFileStore = class {
 	async catalog() {
 		return this.ensureLoaded();
 	}
-	/** Re-evaluate discovery and every TSX entry after template/dependency/theme HMR. */
+	/**
+	* Re-evaluate discovery and every TSX entry after template/dependency/theme HMR.
+	* `changed` names the file whose change asked for this refresh.
+	*
+	* This store writes template files itself, so an editor save comes back as a
+	* change event indistinguishable from an author's own edit in another program.
+	* Re-running the project for it costs a second full evaluation — the price of
+	* one save is the whole catalog, twice — and republishes the document with no
+	* client attached, which is exactly how a foreign edit reaches the editor. A
+	* changed file that still holds the bytes its current entry was built from is
+	* already evaluated, so there is nothing to publish and nothing to rebuild.
+	* Every other path, and every file whose bytes have moved, is a real change.
+	*/
 	async refreshAll(options = {}) {
 		const loading = this.initialLoad;
 		if (loading !== void 0) await loading;
 		else if (!this.loaded) return this.ensureLoaded();
-		return this.serialize(() => this.rebuild(options.documents ?? true));
+		return this.serialize(async () => await this.alreadyBuilt(options.changed) ? this.catalogSnapshot() : this.rebuild(options.documents ?? true));
+	}
+	/**
+	* True when `path` is a discovered entry whose file still holds the exact bytes
+	* that entry was evaluated from. Read inside the mutation queue, so a save's own
+	* rebuild has already landed by the time its change event asks.
+	*/
+	async alreadyBuilt(path) {
+		if (path === void 0) return false;
+		const id = this.idFor(path);
+		if (id === void 0) return false;
+		const entry = this.entries.get(id);
+		if (entry === void 0 || entry.template.stale || entry.authoredSource === void 0) return false;
+		const resolved = await this.resolvePath(id);
+		if (resolved === void 0) return false;
+		return await readFile(resolved, "utf8").catch(() => void 0) === entry.authoredSource;
 	}
 	async list() {
 		return (await this.catalog()).templates;
@@ -355,43 +382,51 @@ var EditorFileStore = class {
 	* Show another declared fixture of an email document. It re-renders the same
 	* build, so the revision does not move; the resulting document reaches every
 	* watcher on the change stream, attributed to the client that asked.
+	*
+	* It runs in the mutation queue for the same reason a save does: the viewed
+	* fixture is part of the entry, and a rebuild that straddled the switch would
+	* take its own snapshot from before it and its evaluation from after it, then
+	* publish the difference with no client — the reader's own fixture switch,
+	* returned to them as somebody else's edit.
 	*/
 	async viewFixture(id, fixture, client) {
 		if (!this.loaded) await this.ensureLoaded();
-		const entry = this.entries.get(id);
-		if (entry === void 0) return {
-			ok: false,
-			kind: "missing",
-			error: "template not found"
-		};
-		if (entry.content.channel !== "email") return {
-			ok: false,
-			kind: "refused",
-			error: `A ${entry.content.channel} template renders no fixture.`
-		};
-		if (!entry.content.fixtures.includes(fixture)) return {
-			ok: false,
-			kind: "refused",
-			error: `Template declares no fixture named ${JSON.stringify(fixture)}.`
-		};
-		const meta = this.session.get(id);
-		this.session.set(id, {
-			...meta,
-			metadata: meta?.metadata ?? {},
-			fixture
-		});
-		const loaded = await this.load(id, entry);
-		if (loaded.entry === void 0) {
-			if (loaded.diagnostics.length > 0) this.diagnostics.set(id, loaded.diagnostics);
-			return {
+		return this.serialize(async () => {
+			const entry = this.entries.get(id);
+			if (entry === void 0) return {
+				ok: false,
+				kind: "missing",
+				error: "template not found"
+			};
+			if (entry.content.channel !== "email") return {
 				ok: false,
 				kind: "refused",
-				error: `Template ${id} could not be re-rendered.`
+				error: `A ${entry.content.channel} template renders no fixture.`
 			};
-		}
-		this.entries.set(id, loaded.entry);
-		this.publishEntry(id, loaded.entry, client);
-		return { ok: true };
+			if (!entry.content.fixtures.includes(fixture)) return {
+				ok: false,
+				kind: "refused",
+				error: `Template declares no fixture named ${JSON.stringify(fixture)}.`
+			};
+			const meta = this.session.get(id);
+			this.session.set(id, {
+				...meta,
+				metadata: meta?.metadata ?? {},
+				fixture
+			});
+			const loaded = await this.load(id, entry);
+			if (loaded.entry === void 0) {
+				if (loaded.diagnostics.length > 0) this.diagnostics.set(id, loaded.diagnostics);
+				return {
+					ok: false,
+					kind: "refused",
+					error: `Template ${id} could not be re-rendered.`
+				};
+			}
+			this.entries.set(id, loaded.entry);
+			this.publishEntry(id, loaded.entry, client);
+			return { ok: true };
+		});
 	}
 	publishEntry(id, entry, client) {
 		const change = {
@@ -740,6 +775,9 @@ const samvaEditor = (options = {}) => {
 			const refreshTheme = () => {
 				queueStoreRefresh(server, () => store.refreshAll({ documents: true }));
 			};
+			let burst = /* @__PURE__ */ new Set();
+			let burstShape = false;
+			let burstQueued = false;
 			const refreshTsxDependents = (path, shapeChange) => {
 				const ssr = server.environments.ssr;
 				if (ssr === void 0) return;
@@ -747,18 +785,31 @@ const samvaEditor = (options = {}) => {
 				const directTemplate = path.endsWith(".tsx") && store.idFor(path) !== void 0;
 				const potentialModuleShapeChange = shapeChange && changedModules.size === 0 && MODULE_EXTENSIONS.has(extname(path));
 				if (!directTemplate && changedModules.size === 0 && !potentialModuleShapeChange) return;
+				burst.add(path);
+				burstShape ||= shapeChange;
+				if (burstQueued) return;
+				burstQueued = true;
 				queueStoreRefresh(server, async () => {
+					const paths = burst;
+					const shape = burstShape;
+					burst = /* @__PURE__ */ new Set();
+					burstShape = false;
+					burstQueued = false;
 					const invalidated = /* @__PURE__ */ new Set();
 					const collectImporters = (module) => {
 						if (invalidated.has(module)) return;
 						invalidated.add(module);
 						for (const importer of module.importers) collectImporters(importer);
 					};
-					for (const module of changedModules) collectImporters(module);
-					if (shapeChange) ssr.moduleGraph.invalidateAll();
+					for (const changed of paths) for (const module of ssr.moduleGraph.getModulesByFile(changed) ?? []) collectImporters(module);
+					if (shape) ssr.moduleGraph.invalidateAll();
 					else for (const module of invalidated) ssr.moduleGraph.invalidateModule(module);
 					moduleRunner.evaluatedModules.clear();
-					return store.refreshAll({ documents: true });
+					const only = paths.size === 1 ? [...paths][0] : void 0;
+					return store.refreshAll({
+						documents: true,
+						changed: only
+					});
 				});
 			};
 			const onChange = (path) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@samva/vite",
-  "version": "0.3.0",
+  "version": "0.4.0",
   "description": "Unified Vite editor for Samva email, SMS, and WhatsApp templates",
   "keywords": [
     "email",
@@ -70,7 +70,7 @@
   },
   "dependencies": {
     "@babel/parser": "^8.0.4",
-    "@samva/markup": "^0.3.0",
+    "@samva/markup": "^0.4.0",
     "caniemail": "2.0.2",
     "css-tree": "3.2.1",
     "effect": "4.0.0-rc.112",