@aglyn/plugins-ai 1.0.0-beta.256 → 1.0.0-beta.258

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aglyn/plugins-ai",
-  "version": "1.0.0-beta.256",
+  "version": "1.0.0-beta.258",
   "license": "Apache-2.0",
   "homepage": "https://aglyn.com",
   "repository": {
@@ -25,22 +25,22 @@
     "./package.json": "./package.json"
   },
   "dependencies": {
-    "@aglyn/aglyn": "1.0.0-beta.256",
-    "@aglyn/shared-data-enums": "1.0.0-beta.256",
-    "@aglyn/shared-data-mdi": "1.0.0-beta.256",
-    "@aglyn/shared-data-types": "1.0.0-beta.256",
-    "@aglyn/shared-ui-email-campaigns": "1.0.0-beta.256",
-    "@aglyn/shared-ui-jsx": "1.0.0-beta.256",
-    "@aglyn/shared-ui-snackstack": "1.0.0-beta.256",
-    "@aglyn/shared-ui-theme": "1.0.0-beta.256",
-    "@aglyn/shared-util-email": "1.0.0-beta.256",
-    "@aglyn/shared-util-errors": "1.0.0-beta.256",
-    "@aglyn/shared-util-http": "1.0.0-beta.256",
-    "@aglyn/shared-util-timestamp": "1.0.0-beta.256",
-    "@aglyn/shared-util-tools": "1.0.0-beta.256",
-    "@aglyn/tenant-data-admin": "1.0.0-beta.256",
-    "@aglyn/tenant-feature-instance": "1.0.0-beta.256",
-    "@aglyn/tenant-runtime": "1.0.0-beta.256",
+    "@aglyn/aglyn": "1.0.0-beta.258",
+    "@aglyn/shared-data-enums": "1.0.0-beta.258",
+    "@aglyn/shared-data-mdi": "1.0.0-beta.258",
+    "@aglyn/shared-data-types": "1.0.0-beta.258",
+    "@aglyn/shared-ui-email-campaigns": "1.0.0-beta.258",
+    "@aglyn/shared-ui-jsx": "1.0.0-beta.258",
+    "@aglyn/shared-ui-snackstack": "1.0.0-beta.258",
+    "@aglyn/shared-ui-theme": "1.0.0-beta.258",
+    "@aglyn/shared-util-email": "1.0.0-beta.258",
+    "@aglyn/shared-util-errors": "1.0.0-beta.258",
+    "@aglyn/shared-util-http": "1.0.0-beta.258",
+    "@aglyn/shared-util-timestamp": "1.0.0-beta.258",
+    "@aglyn/shared-util-tools": "1.0.0-beta.258",
+    "@aglyn/tenant-data-admin": "1.0.0-beta.258",
+    "@aglyn/tenant-feature-instance": "1.0.0-beta.258",
+    "@aglyn/tenant-runtime": "1.0.0-beta.258",
     "@swc/helpers": "0.5.23"
   },
   "peerDependencies": {

package/src/lib/components/assist-panel.component.js

@@ -19,6 +19,7 @@ import { jsx as _jsx, jsxs as _jsxs, Fragment as _Fragment } from "react/jsx-run
 import { canvas, checkEntitlement, lockdownRefusalText, parseLockdownRefusal } from "@aglyn/aglyn";
 import { editorSessionsRevision, openEditorSession, subscribeEditorSessions } from "@aglyn/aglyn/plugin-manager/editor-sessions";
 import { trackEvent } from "@aglyn/aglyn/app-utils/analytics-events";
+import { CANVAS_ROOT_ELEMENT_ID } from "@aglyn/aglyn/foundation/constants/canvas";
 import { mdiChevronDown, mdiCreation, mdiChevronUp, mdiClose, mdiOpenInNew, mdiSend, mdiThumbDownOutline, mdiThumbUpOutline } from "@aglyn/shared-data-mdi";
 import { AppLink, MdiIcon } from "@aglyn/shared-ui-jsx";
 import { useSnackbar } from "@aglyn/shared-ui-snackstack";
@@ -32,7 +33,7 @@ import { applyAssistEdit, describeAssistEditCanvas } from "./assist-edit-canvas.
 import { AssistEditCard } from "./assist-edit-card.component.js";
 import { AssistBuildCard } from "./assist-build-card.component.js";
 import { isAssistBuildProposal } from "../model/assist-build.js";
-import { assistBesignerVersionOf, assistBuildDraftLinks, isLiveAssistFollowUp } from "../model/assist-follow-up.js";
+import { assistBuildDraftLinks, isAssistFollowUpDue, isLiveAssistFollowUp } from "../model/assist-follow-up.js";
 import { startAssistBuildRequest } from "./ai-job-requests.js";
 import { aiInsightSurfaceForPath } from "../model/ai-insight.js";
 import { AssistJobsDrawer } from "./assist-jobs-drawer.component.js";
@@ -536,7 +537,7 @@ export function AssistPanelComponent(props) {
     // The open editor, re-read whenever one opens, closes or reports a change —
     // so a card raised on a live version offers Apply once its new version is
     // the one open.
-    useSyncExternalStore(subscribeEditorSessions, editorSessionsRevision, editorSessionsRevision);
+    const editorRevision = useSyncExternalStore(subscribeEditorSessions, editorSessionsRevision, editorSessionsRevision);
     const editorSession = openEditorSession();
     // Thread is per-org, session-persisted; switching orgs swaps threads.
     // Keyed on the SCOPED org (AGL-1934): gating only the render would still
@@ -684,11 +685,13 @@ export function AssistPanelComponent(props) {
                 } : {}, canvasOutline ? {
                     canvas: canvasOutline
                 } : {}, draftLinks.length ? {
-                    drafts: draftLinks.map(({ ref, label, noun })=>({
+                    drafts: draftLinks.map(({ ref, label, noun, href })=>_extends({
                             ref,
                             label,
                             noun
-                        }))
+                        }, canvasOutline && href === pathname ? {
+                            open: true
+                        } : {}))
                 } : {}))
             });
             if (!response.ok || !response.body) {
@@ -763,9 +766,10 @@ export function AssistPanelComponent(props) {
                         // An open-draft proposal goes to that draft's own Besigner, from
                         // this panel's record of its build, and asks the request again
                         // there; one naming a draft this panel does not hold is dropped.
+                        // Never the draft already open here (AGL-3833).
                         const draftLink = (resolved == null ? void 0 : resolved.draft) ? draftLinks.find((link)=>{
                             var _resolved_draft;
-                            return link.ref === ((_resolved_draft = resolved.draft) == null ? void 0 : _resolved_draft.ref);
+                            return link.ref === ((_resolved_draft = resolved.draft) == null ? void 0 : _resolved_draft.ref) && link.href !== pathname;
                         }) : null;
                         const proposal = (resolved == null ? void 0 : resolved.draft) ? draftLink ? _extends({}, resolved, {
                             href: draftLink.href,
@@ -874,18 +878,27 @@ export function AssistPanelComponent(props) {
             saveFollowUp(scopedOrgId, null);
             return;
         }
-        // Asked once the draft's own canvas is open, so the edit rung reads it.
-        if (busy || !ai.use || pathname !== followUp.path) return;
-        if (!followUpVersion || followUpVersion !== assistBesignerVersionOf(followUp.path)) return;
+        // Asked once the draft's own canvas is open AND loaded, so the edit rung
+        // reads it (AGL-3833). The editor reports its version as it mounts, before
+        // the document reaches the canvas; it reports again once it has.
+        if (busy || !ai.use) return;
+        const stand = {
+            path: pathname != null ? pathname : '',
+            versionId: followUpVersion,
+            canvasLoaded: Boolean(canvas.getNode(CANVAS_ROOT_ELEMENT_ID))
+        };
+        if (!isAssistFollowUpDue(followUp, stand)) return;
         setFollowUp(null);
         saveFollowUp(scopedOrgId, null);
         setOpen(true);
         void send(followUp.question);
+    // The revision is how a canvas that finished loading reaches this effect.
     }, [
         ai.use,
         busy,
         followUp,
         followUpVersion,
+        editorRevision,
         pathname,
         scopedOrgId,
         send