@office-kit/pptx-dev 0.5.0 → 0.6.0

package/README.md

@@ -75,6 +75,20 @@ CLI. Navigation while Claude is replying does not change that turn's context.
 Stale input is rejected with a retry message. If your Claude settings or managed
 policy disable hooks, automatic context attachment is unavailable; enable the
 preview hook in Claude Code to use it. No project settings file is rewritten.
+
+Use **Shift+Enter** to insert a newline in the embedded Claude prompt. On completion,
+the preview checks the actual deck build. Build errors are returned to Claude's
+Stop hook or to the Codex conversation for up to three repair attempts per user
+request. If repair fails, the error stays visible and the last successful preview
+is retained. Stopping the agent cancels further repair attempts.
+
+Both agents receive guidance to check text against its actual background, including
+inherited theme colors, and to use native `Text` paragraphs or mapped rows for lists
+and agendas. `Bullets` and `TableOfContents` are not built-in DSL components; define
+or import project helpers if using those names. See the
+[TSX authoring reference](../../skill/references/tsx.md#lists-and-tables-of-contents)
+for a complete agenda example. Contrast guidance still requires visual review;
+a successful build is not a visual accessibility check.
 Dependency paths are source candidates, not an exact slide-to-file mapping.
 “This slide” requests a focused patch; requests about another slide or the
 whole deck can edit other files or shared styling.

package/dist/cli.mjs

@@ -308,10 +308,16 @@ byId('chat-provider').onchange();
 void refreshChat();
 <\/script><script type="module" src="/terminal.js"><\/script></html>`;
 //#endregion
+//#region src/authoring-guidance.ts
+/** Shared guidance for agents editing a deck through the preview. */
+const authoringGuidance = `For lists and tables of contents, use Text paragraphs/bullets or mapped Text rows. Bullets and TableOfContents are not built-in DSL components: inspect installed exports, import existing local helpers or define them. Do not use Raw for these layouts.
+Choose foreground and actual background colors together. Explicitly set readable text colors on slides, cards, tables, chart labels and footnotes; check inherited theme styles. Aim for 4.5:1 contrast for normal text and 3:1 for large text (18pt or 14pt bold). Use a solid backing over images or gradients if needed. Inspect rendered slides; compilation alone cannot verify contrast.`;
+//#endregion
 //#region src/terminal.ts
 const REQUEST_LIMIT = 128e3;
 /** A real interactive Claude session, shared across reloads of its owning tab. */
-function createTerminal(entry, busy = () => false, base = "") {
+function createTerminal(entry, busy = () => false, base = "", verify = async () => null) {
+	let repairs = 0;
 	let process;
 	let owner = "";
 	let output = "";
@@ -352,17 +358,48 @@ function createTerminal(entry, busy = () => false, base = "") {
 			response.writeHead(code, { "Content-Type": "application/json" });
 			response.end(JSON.stringify(value));
 		};
-		if (request.url === "/terminal/context") {
+		if (request.url === "/terminal/context" || request.url === "/terminal/verify") {
 			if (request.method !== "POST" || request.headers.authorization !== `Bearer ${token}`) {
 				json(403, { error: "Invalid hook credentials" });
 				return;
 			}
 			request.resume();
+			if (request.url === "/terminal/verify") {
+				if (closing || stopping || !process) {
+					json(200, {});
+					return;
+				}
+				let error;
+				try {
+					error = await verify();
+				} catch (cause) {
+					error = cause instanceof Error ? cause.message : String(cause);
+				}
+				if (closing || stopping || !process) {
+					json(200, {});
+					return;
+				}
+				if (error && repairs < 3) {
+					repairs++;
+					status = `Repairing preview (${repairs}/3)…`;
+					json(200, {
+						decision: "block",
+						reason: `The actual preview build failed. Repair the source before finishing (attempt ${repairs}/3). Preserve unrelated edits, inspect the installed DSL exports and local components before changing imports. Do not hide the error or remove requested content. No Raw is needed for lists or an agenda: use Text paragraphs/bullets or mapped Text rows. Build error:\n${error.slice(0, 16e3)}`
+					});
+				} else {
+					status = error ? "Preview still has errors after 3 repair attempts." : "Preview build verified";
+					json(200, {});
+				}
+				emit("state", snapshot());
+				return;
+			}
+			repairs = 0;
 			json(200, { hookSpecificOutput: {
 				hookEventName: "UserPromptSubmit",
 				additionalContext: `You are editing the local PowerPoint TSX project at ${resolve(entry)}.
 Read the project's authoring instructions. Make focused source patches and preserve unrelated work.
-The preview rebuilds on save; do not start another dev server.
+The preview rebuilds on save; do not start another dev server. A Stop hook checks the actual build and returns errors for repair.
+${authoringGuidance}
 The focused slide is context, NOT a restriction. For "this slide", edit only its relevant source. For other slides or deck-wide requests, locate the relevant source or shared theme.
 Slide numbers are 1-based. Verify source before editing; dependency paths are candidates, not an exact slide-to-file mapping.
 Preview context captured with the latest terminal input: ${JSON.stringify(context)}`
@@ -423,12 +460,20 @@ Preview context captured with the latest terminal input: ${JSON.stringify(contex
 						}
 						const pty = await import("node-pty");
 						if (process || closing) throw new Error("Terminal state changed. Try again.");
-						const settings = { hooks: { UserPromptSubmit: [{ hooks: [{
-							type: "http",
-							url: `${origin}${base}/terminal/context`,
-							headers: { Authorization: `Bearer ${token}` },
-							timeout: 5
-						}] }] } };
+						const settings = { hooks: {
+							Stop: [{ hooks: [{
+								type: "http",
+								url: `${origin}${base}/terminal/verify`,
+								headers: { Authorization: `Bearer ${token}` },
+								timeout: 40
+							}] }],
+							UserPromptSubmit: [{ hooks: [{
+								type: "http",
+								url: `${origin}${base}/terminal/context`,
+								headers: { Authorization: `Bearer ${token}` },
+								timeout: 5
+							}] }]
+						} };
 						const env = { ...globalThis.process.env };
 						delete env.CLAUDECODE;
 						process = pty.spawn("claude", ["--settings", JSON.stringify(settings)], {
@@ -495,7 +540,7 @@ const markdown = new MarkdownIt({
 });
 markdown.renderer.rules.image = (tokens, index) => markdown.utils.escapeHtml(tokens[index]?.content ?? "");
 /** One shared conversation and one editing process per dev server. */
-function createChat(entry, notify, busy = () => false) {
+function createChat(entry, notify, busy = () => false, verify = async () => null) {
 	let messages = [];
 	let provider = "codex";
 	let child;
@@ -517,6 +562,10 @@ function createChat(entry, notify, busy = () => false) {
 		stopping = true;
 		status = "Stopping…";
 		const processToStop = child;
+		if (processToStop.exitCode !== null || processToStop.signalCode !== null) {
+			notify();
+			return;
+		}
 		const kill = (signal) => {
 			try {
 				if (process.platform !== "win32" && processToStop.pid) process.kill(-processToStop.pid, signal);
@@ -527,7 +576,7 @@ function createChat(entry, notify, busy = () => false) {
 		killTimer = setTimeout(() => kill("SIGKILL"), 2e3);
 		notify();
 	}
-	function start(text, selected, context) {
+	function start(text, selected, context, repairs = 0) {
 		provider = selected;
 		const user = {
 			role: "user",
@@ -541,6 +590,7 @@ function createChat(entry, notify, busy = () => false) {
 		const prompt = `You are editing a local PowerPoint TSX project in ${dirname(resolve(entry))}.
 Read the project's authoring instructions. Make focused patches to the TSX source, preserving unrelated work.
 The preview rebuilds automatically on save; do not start another dev server.
+${authoringGuidance}
 The attached focus is context, NOT a restriction: use it for references such as "this slide". For other slides or deck-wide requests, find the relevant source or shared theme instead. Do not rewrite the whole deck for a local edit.
 Slide numbers are 1-based and refer to the last successful preview at send time. Source may have changed since then; verify it before editing. Files are dependency candidates, not an exact slide-to-source mapping.
 Respond in the user's language, briefly describing edits or answering their question. If permissions prevent an action, explain that instead of claiming success.
@@ -616,10 +666,31 @@ Current request and preview context (JSON): ${JSON.stringify(user)}`;
 			running.on("error", (error) => {
 				failure = `Could not run ${selected}. Install it and log in from your terminal first. ${error.message}`;
 			});
-			running.on("close", (code) => {
+			running.on("close", async (code) => {
 				if (buffer.trim()) consume(buffer);
 				clearTimeout(killTimer);
 				status = stopping ? "Stopped. Saved edits remain in the preview." : failure || (code !== 0 ? stderr || `${selected} exited with code ${code}` : !sawResult ? "CLI exited without a completed result." : "Done");
+				if (status === "Done") {
+					status = "Verifying preview…";
+					notify();
+					let buildError;
+					try {
+						buildError = await verify();
+					} catch (cause) {
+						buildError = cause instanceof Error ? cause.message : String(cause);
+					}
+					if (!stopping && buildError && repairs < 3) {
+						child = void 0;
+						start(`The actual preview build failed. Repair the source and preserve unrelated work (attempt ${repairs + 1}/3). Do not remove requested content or hide the error.\n${buildError.slice(0, 16e3)}`, selected, {
+							...context,
+							buildError
+						}, repairs + 1);
+						await completion;
+						done();
+						return;
+					}
+					status = stopping ? "Stopped. Saved edits remain in the preview." : buildError ? "Preview still has errors after 3 repair attempts.\n" + buildError.slice(0, 16e3) : "Done";
+				}
 				if (status !== "Done") append(status);
 				child = void 0;
 				notify();
@@ -718,8 +789,8 @@ async function serveDeck(entry, port = 4173) {
 		if (!result) {
 			const chat = createChat(entry, () => {
 				for (const client of clients) client.write("data: chat\n\n");
-			}, () => terminal.isRunning());
-			const terminal = createTerminal(entry, () => chat.isRunning(), id ? "/agents/" + id : "");
+			}, () => terminal.isRunning(), verify);
+			const terminal = createTerminal(entry, () => chat.isRunning(), id ? "/agents/" + id : "", verify);
 			result = {
 				chat,
 				terminal,
@@ -736,7 +807,7 @@ async function serveDeck(entry, port = 4173) {
 			return;
 		}
 		response.setHeader("Cache-Control", "no-store");
-		const match = request.url?.match(/^\/agents\/([\w-]{1,64})(\/(?:chat(?:\/(?:stop|reset))?|terminal\/(?:start|input|resize|stop|events|context)|close))?$/);
+		const match = request.url?.match(/^\/agents\/([\w-]{1,64})(\/(?:chat(?:\/(?:stop|reset))?|terminal\/(?:start|input|resize|stop|events|context|verify)|close))?$/);
 		if (request.url?.startsWith("/agents/") && !match) {
 			response.writeHead(404).end();
 			return;
@@ -840,6 +911,15 @@ async function serveDeck(entry, port = 4173) {
 			response.end(page);
 		} else response.writeHead(404).end();
 	});
+	async function verify() {
+		rebuild();
+		const deadline = Date.now() + 3e4;
+		while (!closed && (building || timer || pending)) {
+			if (Date.now() >= deadline) return "Preview verification timed out. Inspect the build before claiming success.";
+			await new Promise((done) => setTimeout(done, 50));
+		}
+		return closed ? "Preview server closed." : error;
+	}
 	async function rebuild() {
 		if (closed) return;
 		if (building) {