@simmalugnt-se/payload-editor-assistant 0.11.1 → 0.11.3

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog
 
+## 0.11.3
+
+- Clicking "Fix N with the assistant" again after closing the panel opens it on the batch that is
+  still there. The link was remembered as taken and did nothing.
+
+## 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

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] }), !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: () => 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.
@@ -13,11 +17,13 @@ const taken = new Set();
 export function useBatchLink() {
     const params = useSearchParams();
     const { setBatch, setOpen } = useAssistantState();
-    const value = params?.get(BATCH_PARAM);
+    // Depends on `params`, not the value: the same batch clicked twice leaves the value unchanged.
     useEffect(() => {
-        if (!value || taken.has(value)) {
+        const value = params?.get(BATCH_PARAM);
+        if (!value) {
             return;
         }
+        const repeated = taken.has(value);
         taken.add(value);
         const url = new URL(window.location.href);
         const targets = parseBatchTargets(url.searchParams.get(BATCH_PARAM));
@@ -27,9 +33,14 @@ export function useBatchLink() {
             url.searchParams.delete(name);
         }
         window.history.replaceState(window.history.state, "", url.toString());
+        if (repeated) {
+            // The batch is still in the panel, which the editor closed: show it again.
+            setOpen(true);
+            return;
+        }
         if (targets.length > 0 && prompt) {
             setBatch({ id: crypto.randomUUID(), targets, scope, prompt });
             setOpen(true);
         }
-    }, [value, setBatch, setOpen]);
+    }, [params, setBatch, setOpen]);
 }

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/package.json

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