@ohos-ports/slides-grab 1.5.0-beta.0

package/src/editor/edit-subprocess.js

@@ -0,0 +1,170 @@
+import { spawn } from 'node:child_process';
+
+export const DEFAULT_EDIT_TIMEOUT_MS = 10 * 60 * 1000;
+export const EDIT_TIMEOUT_EXIT_CODE = 124;
+export const EDIT_TIMEOUT_ENV_VAR = 'PPT_AGENT_EDIT_TIMEOUT_MS';
+export const EDIT_TIMEOUT_KILL_SIGNAL = 'SIGTERM';
+export const EDIT_TIMEOUT_FORCE_KILL_AFTER_MS = 5_000;
+
+export function parseEditTimeoutMs(rawValue = process.env[EDIT_TIMEOUT_ENV_VAR]) {
+  if (rawValue == null || rawValue === '') {
+    return DEFAULT_EDIT_TIMEOUT_MS;
+  }
+
+  const timeoutMs = Number(rawValue);
+  if (!Number.isFinite(timeoutMs) || timeoutMs <= 0) {
+    return DEFAULT_EDIT_TIMEOUT_MS;
+  }
+
+  return Math.floor(timeoutMs);
+}
+
+export function buildEditTimeoutMessage({ engineLabel = 'Editor process', timeoutMs }) {
+  return `${engineLabel} edit timed out after ${timeoutMs}ms and was terminated.`;
+}
+
+export const EDIT_ABORT_EXIT_CODE = 130;
+export const EDIT_ABORT_KILL_SIGNAL = 'SIGTERM';
+export const EDIT_ABORT_FORCE_KILL_AFTER_MS = 5_000;
+
+export function buildEditAbortMessage({ engineLabel = 'Editor process' } = {}) {
+  return `${engineLabel} edit was aborted and the child process was terminated.`;
+}
+
+export function runEditSubprocess({
+  bin,
+  args,
+  cwd,
+  env,
+  stdio = ['ignore', 'pipe', 'pipe'],
+  timeoutMs = DEFAULT_EDIT_TIMEOUT_MS,
+  engineLabel,
+  onLog = () => {},
+  onChild = () => {},
+  signal,
+  spawnImpl = spawn,
+}) {
+  return new Promise((resolvePromise, rejectPromise) => {
+    const child = spawnImpl(bin, args, { cwd, env, stdio });
+
+    try {
+      onChild(child);
+    } catch {
+      // Never let a faulty observer crash the spawn lifecycle.
+    }
+
+    let stdout = '';
+    let stderr = '';
+    let timedOut = false;
+    let aborted = false;
+    let settled = false;
+    let forceKillTimer = null;
+    let abortForceKillTimer = null;
+
+    const timeoutMessage = buildEditTimeoutMessage({ engineLabel, timeoutMs });
+    const abortMessage = buildEditAbortMessage({ engineLabel });
+
+    const timeoutTimer = setTimeout(() => {
+      if (settled || aborted) return;
+      timedOut = true;
+      const messageLine = `${timeoutMessage}\n`;
+      stderr += messageLine;
+      onLog('stderr', messageLine);
+      try {
+        child.kill(EDIT_TIMEOUT_KILL_SIGNAL);
+      } catch {}
+      forceKillTimer = setTimeout(() => {
+        try {
+          child.kill('SIGKILL');
+        } catch {}
+      }, EDIT_TIMEOUT_FORCE_KILL_AFTER_MS);
+      forceKillTimer.unref?.();
+    }, timeoutMs);
+    timeoutTimer.unref?.();
+
+    function abortChild() {
+      if (settled || aborted) return;
+      aborted = true;
+      const messageLine = `${abortMessage}\n`;
+      stderr += messageLine;
+      try {
+        onLog('stderr', messageLine);
+      } catch {}
+      try {
+        child.kill(EDIT_ABORT_KILL_SIGNAL);
+      } catch {}
+      abortForceKillTimer = setTimeout(() => {
+        try {
+          child.kill('SIGKILL');
+        } catch {}
+      }, EDIT_ABORT_FORCE_KILL_AFTER_MS);
+      abortForceKillTimer.unref?.();
+    }
+
+    if (signal) {
+      if (signal.aborted) {
+        setImmediate(abortChild);
+      } else {
+        const onAbort = () => abortChild();
+        signal.addEventListener?.('abort', onAbort, { once: true });
+        child.once('close', () => {
+          signal.removeEventListener?.('abort', onAbort);
+        });
+      }
+    }
+
+    child.stdout?.on('data', (chunk) => {
+      const text = chunk.toString();
+      stdout += text;
+      onLog('stdout', text);
+    });
+
+    child.stderr?.on('data', (chunk) => {
+      const text = chunk.toString();
+      stderr += text;
+      onLog('stderr', text);
+    });
+
+    child.on('close', (code, exitSignal) => {
+      if (settled) return;
+      settled = true;
+      clearTimeout(timeoutTimer);
+      clearTimeout(forceKillTimer);
+      clearTimeout(abortForceKillTimer);
+
+      let resolvedCode;
+      if (timedOut) {
+        resolvedCode = EDIT_TIMEOUT_EXIT_CODE;
+      } else if (aborted) {
+        resolvedCode = EDIT_ABORT_EXIT_CODE;
+      } else {
+        resolvedCode = code ?? 1;
+      }
+
+      let resolvedSignal = exitSignal;
+      if (timedOut && !exitSignal) resolvedSignal = EDIT_TIMEOUT_KILL_SIGNAL;
+      else if (aborted && !exitSignal) resolvedSignal = EDIT_ABORT_KILL_SIGNAL;
+
+      resolvePromise({
+        code: resolvedCode,
+        stdout,
+        stderr,
+        signal: resolvedSignal,
+        timedOut,
+        aborted,
+        timeoutMs: timedOut ? timeoutMs : null,
+        timeoutMessage: timedOut ? timeoutMessage : null,
+        abortMessage: aborted ? abortMessage : null,
+      });
+    });
+
+    child.on('error', (error) => {
+      if (settled) return;
+      settled = true;
+      clearTimeout(timeoutTimer);
+      clearTimeout(forceKillTimer);
+      clearTimeout(abortForceKillTimer);
+      rejectPromise(error);
+    });
+  });
+}

package/src/editor/editor-codex-prompt.md

@@ -0,0 +1,50 @@
+# Editor Codex Prompt — Slide Edit Rules
+
+This prompt is sent to Codex when the editor requests a single-slide edit.
+It is intentionally separate from the full design skill (SKILL.md) because
+the editor context assumes the deck design is already established.
+
+## Primary Objective
+The user's edit request is the primary objective. All rules below exist to support it, not override it. When a rule conflicts with the user's intent, follow the user.
+
+## Edit Workflow
+1. Read the target slide HTML file.
+2. Apply the user's edit request to the selected region.
+3. Run `slides-grab validate --slides-dir <path>` after editing.
+4. If validation fails, fix the HTML/CSS and re-run until it passes.
+5. Return after applying the change.
+
+## Slide Rules
+- Keep slide size appropriate for the current mode (`720pt x 405pt` for presentation, `720pt x 720pt` for card-news).
+- Keep semantic text tags (`p`, `h1-h6`, `ul`, `ol`, `li`).
+- Never place text directly in `<div>` or `<span>`.
+- Always include `#` prefix in CSS colors.
+- Always include the Pretendard webfont CDN link.
+
+## Asset Rules
+- Put local images and videos under `<slides-dir>/assets/` and reference as `./assets/<file>`.
+- Always include `alt` on `<img>` tags.
+- Allow `data:` URLs only when the slide must be fully self-contained.
+- Do not leave remote `http(s)://` image URLs in saved slide HTML; download into `./assets/`.
+- Do not use absolute filesystem paths in slide HTML.
+- Do not use non-body `background-image` for content imagery; use `<img>` instead.
+- Use `data-image-placeholder` to reserve space when no image is available yet.
+- When the request needs bespoke imagery, prefer `slides-grab image --prompt "<prompt>" --slides-dir <path>` so the default codex-imagen provider saves the asset under `<slides-dir>/assets/` via your local Codex ChatGPT login (run `codex login` once if needed).
+- Default provider codex-imagen reuses `~/.codex/auth.json` — no OpenAI/Google API key required; requires a Codex/ChatGPT account entitled to image generation. ⚠️ codex calls an unsupported private Codex backend that may break without notice. Optional fallbacks: `--provider openai` (OpenAI gpt-image-2 via `OPENAI_API_KEY`; maps `--aspect-ratio` to the nearest supported OpenAI image size; `--image-size 2K|4K` is Nano Banana-only) or `--provider nano-banana` (Google `gemini-3-pro-image-preview` via `GOOGLE_API_KEY` / `GEMINI_API_KEY`; supports `--image-size 2K|4K`). If credentials are unavailable, fall back to web search + download into `./assets/`.
+- For local videos, use `<video src="./assets/<file>">` with `poster="./assets/<file>"`.
+- If a video starts on YouTube or a supported page, use `slides-grab fetch-video --url <url> --slides-dir <path>` to download first.
+
+## Art Direction Defaults
+- Give each slide one job, one dominant visual anchor, one primary takeaway.
+- Keep copy short enough to scan in seconds.
+- Use whitespace, alignment, scale, cropping, and contrast before adding decorative chrome.
+- Prefer Lucide as the default icon library for slide UI elements, callouts, and supporting visuals.
+- Do not default to emoji for iconography unless the brief explicitly asks for a playful or native-emoji tone.
+- Default to cardless layouts unless a card improves structure.
+- Limit to two typefaces max and one accent color.
+
+## Do NOT
+- Re-open style selection or run `slides-grab preview-styles`.
+- Modify other slide HTML files unless explicitly requested.
+- Persist runtime-only editor/viewer injections (`<base>`, debug scripts, viewer wrappers).
+- Start PPTX/PDF conversion.