@simmalugnt-se/payload-editor-assistant 0.11.0 → 0.11.2

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # Changelog
 
+## 0.11.2
+
+- Translating a page whose text is all inside blocks or rows now works. The assistant left the
+  source language out of its context when it compared it with nothing, so it answered that there
+  was no English text to translate from. It now compares with the open form.
+- A batch link from the work list starts again after Cancel. It was remembered as taken for the
+  whole session, so the second click did nothing until the page was reloaded.
+
+## 0.11.1
+
+- The batch panel shows Start and Cancel right under the text that explains the batch, instead of
+  at the bottom of the panel, where they were easy to miss on a tall screen. "New chat" is hidden
+  while a batch is open.
+- Saved batch cards keep the assistant's accent blue, like proposals and selections, instead of
+  Payload's success colour.
+
 ## 0.11.0
 
 - Batches: fix many saved documents at once from content health's work list. The new

package/dist/components/BatchPanel.js

@@ -135,10 +135,10 @@ export function BatchPanel({ batch, onClose }) {
     const selected = cards.filter((card) => card.status === "proposed" && card.checked).length;
     const waiting = waitingKeys(cards).length;
     const savedCount = cards.filter((card) => card.status === "saved" || card.status === "draft").length;
-    return (_jsxs("div", { className: "ea-batch", children: [_jsxs("div", { className: "ea-batch__head", children: [_jsx("h3", { children: copy.title(batch.targets.length) }), _jsxs("p", { className: "ea-batch__task", children: [_jsxs("span", { children: [copy.task, ":"] }), " ", batch.prompt] }), phase === "ready" ? _jsx("p", { children: copy.intro(waiting) }) : null, phase === "running" || phase === "saving" ? (_jsx("p", { "aria-live": "polite", children: phase === "saving" ? copy.saving : copy.progress(finished, total) })) : null, phase === "review" && selected > 0 ? _jsx("p", { children: copy.review }) : null, phase === "done" ? (_jsx("p", { "aria-live": "polite", children: copy.summary(savedCount, total - savedCount) })) : null] }), _jsx("ul", { className: "ea-batch__cards", children: cards.map((card) => (_jsx(BatchCardItem, { adminRoute: config.routes.admin, card: card, copy: copy, editable: phase === "review" || phase === "done", onCheck: (checked) => update(card.key, (current) => ({ ...current, checked })), onEdit: (path, value) => update(card.key, (current) => ({
+    return (_jsxs("div", { className: "ea-batch", children: [_jsxs("div", { className: "ea-batch__head", children: [_jsx("h3", { children: copy.title(batch.targets.length) }), _jsxs("p", { className: "ea-batch__task", children: [_jsxs("span", { children: [copy.task, ":"] }), " ", batch.prompt] }), phase === "ready" ? (_jsxs(_Fragment, { children: [_jsx("p", { children: copy.intro(waiting) }), _jsxs("div", { className: "ea-actions", children: [_jsx(Button, { buttonStyle: "primary", disabled: waiting === 0, margin: false, onClick: () => void run(), size: "medium", children: copy.start }), _jsx(Button, { buttonStyle: "secondary", margin: false, onClick: onClose, size: "medium", children: copy.cancel })] })] })) : null, phase === "running" || phase === "saving" ? (_jsx("p", { "aria-live": "polite", children: phase === "saving" ? copy.saving : copy.progress(finished, total) })) : null, phase === "review" && selected > 0 ? _jsx("p", { children: copy.review }) : null, phase === "done" ? (_jsx("p", { "aria-live": "polite", children: copy.summary(savedCount, total - savedCount) })) : null] }), _jsx("ul", { className: "ea-batch__cards", children: cards.map((card) => (_jsx(BatchCardItem, { adminRoute: config.routes.admin, card: card, copy: copy, editable: phase === "review" || phase === "done", onCheck: (checked) => update(card.key, (current) => ({ ...current, checked })), onEdit: (path, value) => update(card.key, (current) => ({
                         ...current,
                         edits: { ...current.edits, [path]: value },
-                    })), onRetry: (instruction) => void retry(card, instruction) }, card.key))) }), _jsxs("div", { className: "ea-actions ea-batch__actions", children: [phase === "ready" ? (_jsxs(_Fragment, { children: [_jsx(Button, { buttonStyle: "primary", disabled: waiting === 0, margin: false, onClick: () => void run(), size: "medium", children: copy.start }), _jsx(Button, { buttonStyle: "secondary", margin: false, onClick: onClose, size: "medium", children: copy.cancel })] })) : null, phase === "running" ? (_jsx(Button, { buttonStyle: "secondary", margin: false, onClick: stop, size: "medium", children: copy.stop })) : null, phase === "review" ? (_jsxs(_Fragment, { children: [_jsx(Button, { buttonStyle: "primary", disabled: selected === 0, margin: false, onClick: () => void save(), size: "medium", children: copy.save(selected) }), waiting > 0 ? (_jsx(Button, { buttonStyle: "secondary", margin: false, onClick: () => void run(), size: "medium", children: copy.continue })) : null, _jsx(Button, { buttonStyle: "secondary", margin: false, onClick: () => void close(), size: "medium", children: copy.cancel })] })) : null, phase === "done" ? (_jsx(Button, { buttonStyle: "primary", margin: false, onClick: onClose, size: "medium", children: copy.close })) : null] })] }));
+                    })), onRetry: (instruction) => void retry(card, instruction) }, card.key))) }), phase === "ready" ? null : (_jsxs("div", { className: "ea-actions ea-batch__actions", children: [phase === "running" ? (_jsx(Button, { buttonStyle: "secondary", margin: false, onClick: stop, size: "medium", children: copy.stop })) : null, phase === "review" ? (_jsxs(_Fragment, { children: [_jsx(Button, { buttonStyle: "primary", disabled: selected === 0, margin: false, onClick: () => void save(), size: "medium", children: copy.save(selected) }), waiting > 0 ? (_jsx(Button, { buttonStyle: "secondary", margin: false, onClick: () => void run(), size: "medium", children: copy.continue })) : null, _jsx(Button, { buttonStyle: "secondary", margin: false, onClick: () => void close(), size: "medium", children: copy.cancel })] })) : null, phase === "done" ? (_jsx(Button, { buttonStyle: "primary", margin: false, onClick: onClose, size: "medium", children: copy.close })) : null] }))] }));
 }
 function BatchCardItem({ adminRoute, card, copy, editable, onCheck, onEdit, onRetry, }) {
     const [retrying, setRetrying] = useState(false);

package/dist/components/EditorAssistantAside.js

@@ -8,6 +8,7 @@ import { parseAdminView } from "../view.js";
 import { assistantRequest, assistantStream } from "./assistant-api.js";
 import { useAssistantState, } from "./assistant-context.js";
 import { BatchPanel } from "./BatchPanel.js";
+import { releaseTakenBatches } from "./batch-link.js";
 import { COPY, translateAction, uiLanguage } from "./copy.js";
 import { isFormProposal, proposalSteps, refineProposal, replaceLatestStep, } from "./proposal-chain.js";
 import { ReplyText } from "./ReplyText.js";
@@ -443,7 +444,10 @@ export function EditorAssistantAside() {
     if (!open) {
         return null;
     }
-    return (_jsxs("aside", { className: "ea-aside", "aria-label": copy.assistant, [KEEP_SELECTION_ATTR]: "", children: [_jsxs("div", { className: "ea-aside__head", children: [_jsxs("div", { className: "ea-aside__title", children: [_jsx("h2", { children: copy.assistant }), bridge ? _jsx("p", { className: "ea-aside__context", children: contextLine(bridge, config) }) : null] }), (messages.length > 0 || pendingApproval) && !pendingApproval?.applied ? (_jsx(Button, { buttonStyle: "subtle", margin: false, onClick: () => resetConversation(), size: "medium", children: copy.newChat })) : null, _jsx(Button, { "aria-label": copy.collapseLabel, buttonStyle: "icon-label", className: "ea-aside__collapse", icon: _jsx(ChevronIcon, { direction: "right" }), margin: false, onClick: () => setOpen(false), tooltip: copy.collapse })] }), batch ? (_jsx("div", { className: "ea-aside__body", children: _jsx(BatchPanel, { batch: batch, onClose: () => setBatch(null) }, batch.id) })) : (_jsxs("div", { className: "ea-aside__body", children: [_jsxs("div", { className: "ea-messages", ref: threadRef, onScroll: onThreadScroll, children: [messages.length === 0 ? (_jsx("div", { className: "ea-msg ea-msg--assistant", children: introText(view, bridge, collections, config, copy) })) : null, messages.map((message) => (_jsxs("div", { className: `ea-msg ea-msg--${message.role}`, children: [message.role === "assistant" ? _jsx(ReplyText, { text: message.text }) : message.text, message.candidates?.length ? (_jsx("div", { className: "ea-actions", children: message.candidates.map((candidate) => (_jsx(Button, { buttonStyle: "pill", margin: false, onClick: () => {
+    return (_jsxs("aside", { className: "ea-aside", "aria-label": copy.assistant, [KEEP_SELECTION_ATTR]: "", children: [_jsxs("div", { className: "ea-aside__head", children: [_jsxs("div", { className: "ea-aside__title", children: [_jsx("h2", { children: copy.assistant }), bridge ? _jsx("p", { className: "ea-aside__context", children: contextLine(bridge, config) }) : null] }), !batch && (messages.length > 0 || pendingApproval) && !pendingApproval?.applied ? (_jsx(Button, { buttonStyle: "subtle", margin: false, onClick: () => resetConversation(), size: "medium", children: copy.newChat })) : null, _jsx(Button, { "aria-label": copy.collapseLabel, buttonStyle: "icon-label", className: "ea-aside__collapse", icon: _jsx(ChevronIcon, { direction: "right" }), margin: false, onClick: () => setOpen(false), tooltip: copy.collapse })] }), batch ? (_jsx("div", { className: "ea-aside__body", children: _jsx(BatchPanel, { batch: batch, onClose: () => {
+                        releaseTakenBatches();
+                        setBatch(null);
+                    } }, batch.id) })) : (_jsxs("div", { className: "ea-aside__body", children: [_jsxs("div", { className: "ea-messages", ref: threadRef, onScroll: onThreadScroll, children: [messages.length === 0 ? (_jsx("div", { className: "ea-msg ea-msg--assistant", children: introText(view, bridge, collections, config, copy) })) : null, messages.map((message) => (_jsxs("div", { className: `ea-msg ea-msg--${message.role}`, children: [message.role === "assistant" ? _jsx(ReplyText, { text: message.text }) : message.text, message.candidates?.length ? (_jsx("div", { className: "ea-actions", children: message.candidates.map((candidate) => (_jsx(Button, { buttonStyle: "pill", margin: false, onClick: () => {
                                                 router.push(candidate.href);
                                             }, size: "small", children: candidate.label }, candidate.href))) })) : null] }, message.id))), status === "streaming" ? (_jsxs("div", { className: "ea-msg ea-msg--working", children: [_jsxs("span", { className: "ea-dots", "aria-hidden": "true", children: [_jsx("span", {}), _jsx("span", {}), _jsx("span", {})] }), _jsx("span", { children: step ? talk.steps[step] : talk.working })] })) : null] }), pendingApproval ? (_jsxs("div", { className: "ea-approval", children: [_jsx("h3", { children: pendingApproval.title }), pendingApproval.applied ? (_jsxs("p", { className: "ea-approval__hint", children: [autosave ? copy.previewHintAutosave : copy.previewHint, pendingApproval.earlier?.length ? ` ${copy.refinedHint}` : ""] })) : null, _jsx("ul", { className: "ea-approval__diff", children: pendingApproval.summary.split("\n").map((line, index) => line.startsWith("  ") ? (
                                 // biome-ignore lint/suspicious/noArrayIndexKey: lines are static for one approval

package/dist/components/batch-link.d.ts

@@ -1,3 +1,5 @@
+/** Once the editor leaves a batch, the same link may start it again. */
+export declare function releaseTakenBatches(): void;
 /**
  * Opens the panel in batch mode when the address carries a batch from content health's work list,
  * and takes the parameters out of it, so a reload does not start the batch again.

package/dist/components/batch-link.js

@@ -6,6 +6,10 @@ import { useAssistantState } from "./assistant-context.js";
 import { ASK_PARAM } from "./selection-dom.js";
 /** Batches already taken this session: a view that writes its old address back must not repeat one. */
 const taken = new Set();
+/** Once the editor leaves a batch, the same link may start it again. */
+export function releaseTakenBatches() {
+    taken.clear();
+}
 /**
  * Opens the panel in batch mode when the address carries a batch from content health's work list,
  * and takes the parameters out of it, so a reload does not start the batch again.

package/dist/runtime/context.js

@@ -28,7 +28,9 @@ export async function loadTurnContext(req, options, bridge, formData) {
         query: "",
         locale: bridge.locale,
     });
-    const otherLanguages = open ? await readOtherLanguages(req, options, bridge, fields) : undefined;
+    const otherLanguages = open
+        ? await readOtherLanguages(req, options, bridge, fields, form)
+        : undefined;
     const translation = form ? translationFor(req, bridge, fields, form, otherLanguages) : undefined;
     return {
         allowlist: schema.map(compactEntity),
@@ -62,7 +64,7 @@ async function readSavedDocument(req, options, bridge) {
     }
     return result.data;
 }
-async function readOtherLanguages(req, options, bridge, fields) {
+async function readOtherLanguages(req, options, bridge, fields, form) {
     const locales = configuredLocales(req).filter((code) => code !== bridge.locale);
     if (locales.length === 0) {
         return undefined;
@@ -75,9 +77,13 @@ async function readOtherLanguages(req, options, bridge, fields) {
             documentId: bridge.documentId,
             locale,
         });
-        // A language nobody wrote in yet is left out: it has nothing to translate from.
-        if (result.ok && translationPaths(fields, {}, result.data).empty.length > 0) {
-            otherLanguages[locale] = result.data;
+        // A language nobody wrote in yet is left out: it has nothing to translate from. Compared with
+        // the open form, since blocks and rows are matched one by one and never against nothing.
+        if (result.ok) {
+            const paths = translationPaths(fields, form ?? {}, result.data);
+            if (paths.empty.length + paths.translated.length > 0) {
+                otherLanguages[locale] = result.data;
+            }
         }
     }
     return Object.keys(otherLanguages).length > 0 ? otherLanguages : undefined;

package/dist/styles/assistant.css

@@ -484,13 +484,11 @@ html[data-theme="dark"] .ea-trigger--open .ea-trigger__icon {
   line-height: 1.45;
 }
 
-.ea-batch__card--proposed {
-  border-inline-start: 3px solid var(--ea-accent);
-}
-
+/* The accent, as for selections and changed fields; saved cards keep it. */
+.ea-batch__card--proposed,
 .ea-batch__card--saved,
 .ea-batch__card--draft {
-  border-inline-start: 3px solid var(--theme-success-500, #16a34a);
+  border-inline-start: 3px solid var(--ea-accent);
 }
 
 .ea-batch__card--failed,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@simmalugnt-se/payload-editor-assistant",
-  "version": "0.11.0",
+  "version": "0.11.2",
   "description": "Context-aware editor assistant plugin for Payload Admin",
   "keywords": [
     "payload",