@arpproject/recrate 0.1.38 → 0.1.39

package/dist/crate-builder/RenderEntity/EntityProperty.spec.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/crate-builder/RenderEntity/RenderLinkedItem.spec.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/crate-builder/RenderEntity/profile-tabs.lib.d.ts

@@ -8,6 +8,7 @@ export declare const findAddedProfileTab: (tabs: any[], previousTabKeys: Set<str
 export declare const isAboutTab: (tab: any) => boolean;
 export declare const tabContainsProperty: (tab: any, property: string) => boolean;
 export declare const findTabForProperty: (tabs: any[], property: string) => any;
+export declare const addMissingReviewPropertiesToTabs: (tabs: any[], properties: string[]) => any[];
 export declare const clampLabelTabPaneWidth: (width: number) => number;
 export declare const readPendingAddedProfileTabKeys: (entityId: string) => string[] | null;
 export declare const writePendingAddedProfileTabKeys: (entityId: string, tabKeys: string[]) => void;

package/dist/recrate.es.js

@@ -59412,6 +59412,27 @@ const findTabForProperty = (tabs, property) => {
   const matchingTabs = tabs.filter((tab) => tabContainsProperty(tab, property));
   return matchingTabs.find((tab) => !isAboutTab(tab)) ?? matchingTabs[0] ?? tabs.find((tab) => ["overflow", "other"].includes(String((tab == null ? void 0 : tab.name) || "").toLowerCase()));
 };
+const addMissingReviewPropertiesToTabs = (tabs, properties) => {
+  const uniqueProperties = Array.from(new Set(
+    properties.filter((property) => typeof property === "string" && property.length > 0)
+  ));
+  if (!tabs.length || !uniqueProperties.length) return tabs;
+  const assignedProperties = new Set(
+    tabs.flatMap((tab) => Array.isArray(tab == null ? void 0 : tab.inputs) ? tab.inputs.flatMap((input) => [input == null ? void 0 : input.name, input == null ? void 0 : input.id]) : []).filter((property) => typeof property === "string" && property.length > 0)
+  );
+  const missingProperties = uniqueProperties.filter((property) => !assignedProperties.has(property));
+  if (!missingProperties.length) return tabs;
+  const fallbackTabIndex = tabs.findIndex((tab) => ["overflow", "other"].includes(String((tab == null ? void 0 : tab.name) || "").toLowerCase()));
+  const aboutTabIndex = tabs.findIndex(isAboutTab);
+  const targetTabIndex = fallbackTabIndex >= 0 ? fallbackTabIndex : aboutTabIndex >= 0 ? aboutTabIndex : 0;
+  return tabs.map((tab, index2) => index2 === targetTabIndex ? {
+    ...tab,
+    inputs: [
+      ...Array.isArray(tab == null ? void 0 : tab.inputs) ? tab.inputs : [],
+      ...missingProperties.map((property) => ({ name: property }))
+    ]
+  } : tab);
+};
 const clampLabelTabPaneWidth = (width) => Math.max(MIN_LABEL_TAB_PANE_WIDTH, Math.min(MAX_LABEL_TAB_PANE_WIDTH, Math.round(width)));
 const getPendingAddedProfileStorageKey = (entityId) => `recrate.pending-added-profile-tab.${entityId}`;
 const readPendingAddedProfileTabKeys = (entityId) => {
@@ -85864,7 +85885,7 @@ const RenderLinkedItem = ({
           {
             entityId: entity["@id"],
             rows: aiEntityDiffRows,
-            readonly: Boolean(((_b = state.configuration) == null ? void 0 : _b.readonly) || readonly),
+            readonly: Boolean((_b = state.configuration) == null ? void 0 : _b.readonly),
             onClose: () => setShowAiEntityDiff(false),
             onAccept: () => onAcceptAiEntityEdit == null ? void 0 : onAcceptAiEntityEdit(effectiveEntity),
             onReject: () => onRejectAiEntityEdit == null ? void 0 : onRejectAiEntityEdit(effectiveEntity)
@@ -85888,7 +85909,7 @@ const RenderLinkedItem = ({
             displayEntity: linkDisplayEntity,
             property,
             sourceEntityId: aiUnlinkSourceEntityId,
-            readonly: Boolean(((_d = state.configuration) == null ? void 0 : _d.readonly) || readonly),
+            readonly: Boolean((_d = state.configuration) == null ? void 0 : _d.readonly),
             onLoadEntity: handleLoadEntity,
             onAcceptUnlink: onAcceptAiEntityUnlink,
             onRejectUnlink: onRejectAiEntityUnlink
@@ -87829,6 +87850,67 @@ const EntityProperty = ({
       idx
     });
   }, [approvalContext, entity, hasCurrentPropertyValue, isAiDelete, onCreateProperty, onSaveProperty, previousValueDiffers, property, unapprovedApprovalRecord]);
+  const renderAiActions = (idx) => {
+    var _a3, _b2;
+    return showAiReview && idx === ((_a3 = renderedSimpleInstances[0]) == null ? void 0 : _a3.idx) && !((_b2 = state.configuration) == null ? void 0 : _b2.readonly) ? /* @__PURE__ */ jsxRuntimeExports.jsxs("div", { className: "recrate-ai-actions", children: [
+      showAiDiffToggle && /* @__PURE__ */ jsxRuntimeExports.jsx(Tooltip2, { title: showAiDiff ? t2("ai_hide_diff") : t2("ai_show_diff"), children: /* @__PURE__ */ jsxRuntimeExports.jsx(
+        Button$1,
+        {
+          size: "middle",
+          className: "recrate-ai-action-diff",
+          icon: /* @__PURE__ */ jsxRuntimeExports.jsx(RefIcon$7, {}),
+          "aria-label": showAiDiff ? String(t2("ai_hide_ai_diff_aria")) : String(t2("ai_show_ai_diff_aria")),
+          "aria-pressed": showAiDiff,
+          onClick: () => setShowAiDiff((visible) => !visible)
+        }
+      ) }),
+      /* @__PURE__ */ jsxRuntimeExports.jsx(Tooltip2, { title: isAiDelete ? t2("ai_approve_deletion_tooltip") : t2("ai_approve_suggested_value_tooltip"), children: /* @__PURE__ */ jsxRuntimeExports.jsx(
+        Button$1,
+        {
+          size: "middle",
+          className: "recrate-ai-action-approve",
+          onClick: handleApproveAiSuggestion,
+          children: isAiDelete ? t2("ai_delete") : t2("ai_accept")
+        }
+      ) }),
+      previousValueDiffers && /* @__PURE__ */ jsxRuntimeExports.jsx(Tooltip2, { title: t2("ai_restore_previous_value_tooltip"), children: /* @__PURE__ */ jsxRuntimeExports.jsx(
+        Button$1,
+        {
+          size: "middle",
+          className: "recrate-ai-action-reject",
+          onClick: () => handleRejectAiSuggestion(idx),
+          children: isAiDelete ? t2("ai_restore") : t2("ai_reject")
+        }
+      ) })
+    ] }) : null;
+  };
+  const renderPropertyValue = (instance2) => {
+    if (showAiDiff && showAiDiffToggle) {
+      return /* @__PURE__ */ jsxRuntimeExports.jsx(
+        AiFieldDiffView,
+        {
+          previousValue: unapprovedApprovalRecord == null ? void 0 : unapprovedApprovalRecord.previousValue,
+          currentValue: instance2.value,
+          operation: unapprovedApprovalRecord == null ? void 0 : unapprovedApprovalRecord.operation
+        }
+      );
+    }
+    if (propertyDefinition == null ? void 0 : propertyDefinition.readonly) {
+      return /* @__PURE__ */ jsxRuntimeExports.jsx("div", { className: showAiReview ? "recrate-property-value-readonly recrate-ai-field-control" : "recrate-property-value-readonly", children: instance2.value });
+    }
+    return /* @__PURE__ */ jsxRuntimeExports.jsx(
+      EntityPropertyInstance,
+      {
+        className: "flex-grow",
+        property,
+        value: instance2.value,
+        idx: instance2.idx,
+        definition: propertyDefinition,
+        onSaveProperty: handleSaveProperty,
+        onCreateEntity
+      }
+    );
+  };
   return /* @__PURE__ */ jsxRuntimeExports.jsxs(
     "div",
     {
@@ -87862,59 +87944,11 @@ const EntityProperty = ({
         ] }),
         /* @__PURE__ */ jsxRuntimeExports.jsxs("div", { className: "w-2/3 xl:w-4/5 flex flex-col flex-grow recrate-property-value", children: [
           /* @__PURE__ */ jsxRuntimeExports.jsx("div", { className: "flex flex-col space-y-1", children: renderedSimpleInstances.map((instance2) => {
-            var _a3, _b2, _c, _d;
-            return /* @__PURE__ */ jsxRuntimeExports.jsx("div", { children: (propertyDefinition == null ? void 0 : propertyDefinition.readonly) ? /* @__PURE__ */ jsxRuntimeExports.jsx("div", { className: showAiReview ? "recrate-property-value-readonly recrate-ai-field-control" : "recrate-property-value-readonly", children: instance2.value }) : /* @__PURE__ */ jsxRuntimeExports.jsxs("div", { className: `flex flex-row gap-2 ${showAiReview ? "recrate-ai-field-row" : ""}`, children: [
-              /* @__PURE__ */ jsxRuntimeExports.jsx("div", { className: `flex-grow min-w-0 ${showAiReview ? "recrate-ai-field-control" : ""}`, children: showAiDiff && showAiDiffToggle ? /* @__PURE__ */ jsxRuntimeExports.jsx(
-                AiFieldDiffView,
-                {
-                  previousValue: unapprovedApprovalRecord == null ? void 0 : unapprovedApprovalRecord.previousValue,
-                  currentValue: instance2.value,
-                  operation: unapprovedApprovalRecord == null ? void 0 : unapprovedApprovalRecord.operation
-                }
-              ) : /* @__PURE__ */ jsxRuntimeExports.jsx(
-                EntityPropertyInstance,
-                {
-                  className: "flex-grow",
-                  property,
-                  value: instance2.value,
-                  idx: instance2.idx,
-                  definition: propertyDefinition,
-                  onSaveProperty: handleSaveProperty,
-                  onCreateEntity
-                }
-              ) }),
-              showAiReview && instance2.idx === ((_a3 = renderedSimpleInstances[0]) == null ? void 0 : _a3.idx) && !((_b2 = state.configuration) == null ? void 0 : _b2.readonly) && /* @__PURE__ */ jsxRuntimeExports.jsxs("div", { className: "recrate-ai-actions", children: [
-                showAiDiffToggle && /* @__PURE__ */ jsxRuntimeExports.jsx(Tooltip2, { title: showAiDiff ? t2("ai_hide_diff") : t2("ai_show_diff"), children: /* @__PURE__ */ jsxRuntimeExports.jsx(
-                  Button$1,
-                  {
-                    size: "middle",
-                    className: "recrate-ai-action-diff",
-                    icon: /* @__PURE__ */ jsxRuntimeExports.jsx(RefIcon$7, {}),
-                    "aria-label": showAiDiff ? String(t2("ai_hide_ai_diff_aria")) : String(t2("ai_show_ai_diff_aria")),
-                    "aria-pressed": showAiDiff,
-                    onClick: () => setShowAiDiff((visible) => !visible)
-                  }
-                ) }),
-                /* @__PURE__ */ jsxRuntimeExports.jsx(Tooltip2, { title: isAiDelete ? t2("ai_approve_deletion_tooltip") : t2("ai_approve_suggested_value_tooltip"), children: /* @__PURE__ */ jsxRuntimeExports.jsx(
-                  Button$1,
-                  {
-                    size: "middle",
-                    className: "recrate-ai-action-approve",
-                    onClick: handleApproveAiSuggestion,
-                    children: isAiDelete ? t2("ai_delete") : t2("ai_accept")
-                  }
-                ) }),
-                previousValueDiffers && /* @__PURE__ */ jsxRuntimeExports.jsx(Tooltip2, { title: t2("ai_restore_previous_value_tooltip"), children: /* @__PURE__ */ jsxRuntimeExports.jsx(
-                  Button$1,
-                  {
-                    size: "middle",
-                    className: "recrate-ai-action-reject",
-                    onClick: () => handleRejectAiSuggestion(instance2.idx),
-                    children: isAiDelete ? t2("ai_restore") : t2("ai_reject")
-                  }
-                ) })
-              ] }),
-              isNotValue && !showAiReview && !((_c = state.configuration) == null ? void 0 : _c.readonly) && (instance2.value || instance2.value === false) && !((_d = instance2.value) == null ? void 0 : _d["@id"]) && /* @__PURE__ */ jsxRuntimeExports.jsx(
+            var _a3, _b2;
+            return /* @__PURE__ */ jsxRuntimeExports.jsx("div", { children: /* @__PURE__ */ jsxRuntimeExports.jsxs("div", { className: `flex flex-row gap-2 ${showAiReview ? "recrate-ai-field-row" : ""}`, children: [
+              /* @__PURE__ */ jsxRuntimeExports.jsx("div", { className: `flex-grow min-w-0 ${showAiReview ? "recrate-ai-field-control" : ""}`, children: renderPropertyValue(instance2) }),
+              renderAiActions(instance2.idx),
+              isNotValue && !(propertyDefinition == null ? void 0 : propertyDefinition.readonly) && !showAiReview && !((_a3 = state.configuration) == null ? void 0 : _a3.readonly) && (instance2.value || instance2.value === false) && !((_b2 = instance2.value) == null ? void 0 : _b2["@id"]) && /* @__PURE__ */ jsxRuntimeExports.jsx(
                 DeleteProperty,
                 {
                   type: "delete",
@@ -116646,6 +116680,11 @@ const RenderEntity = forwardRef((props, ref) => {
   )), [approvedAiApprovals]);
   const selectedAiApproval = pendingAiApprovals[selectedAiApprovalIndex] ?? pendingAiApprovals[0];
   const selectedAiApprovalProperty = (selectedAiApproval == null ? void 0 : selectedAiApproval.propertyName) ?? pendingDeletedEntityReviewProperties[0];
+  const aiReviewPropertyNames = useMemo$1(() => Array.from(/* @__PURE__ */ new Set([
+    ...pendingAiApprovalProperties,
+    ...pendingDeletedEntityReviewProperties
+  ])), [pendingAiApprovalProperties, pendingDeletedEntityReviewProperties]);
+  const tabsWithAiReviewProperties = useMemo$1(() => addMissingReviewPropertiesToTabs(tabs, aiReviewPropertyNames), [aiReviewPropertyNames, tabs]);
   const pendingContextApprovalItems = useMemo$1(() => getPendingContextApprovalRecords(approvalContext == null ? void 0 : approvalContext.roCrateApproval), [approvalContext == null ? void 0 : approvalContext.roCrateApproval]);
   const currentContextApprovalItem = useMemo$1(() => {
     if (contextReviewEntityId) {
@@ -117179,7 +117218,7 @@ const RenderEntity = forwardRef((props, ref) => {
     }
     suppressAiScrollRestore();
     pendingAiScrollPropertyRef.current = property;
-    const propertyTab = renderTabs ? findTabForProperty(tabs, property) : null;
+    const propertyTab = renderTabs ? findTabForProperty(tabsWithAiReviewProperties, property) : null;
     if ((propertyTab == null ? void 0 : propertyTab.name) && propertyTab.name !== activeTab) {
       captureViewContext(activeTab);
       setActiveTab(propertyTab.name);
@@ -117194,7 +117233,7 @@ const RenderEntity = forwardRef((props, ref) => {
     pendingAiApprovals,
     renderTabs,
     scrollToAiApprovalProperty,
-    tabs
+    tabsWithAiReviewProperties
   ]);
   const navigateToAiApproval = useCallback((index2) => {
     if (!pendingAiApprovals.length) return;
@@ -117239,7 +117278,7 @@ const RenderEntity = forwardRef((props, ref) => {
   useEffect(() => {
     const pendingProperty = pendingAiScrollPropertyRef.current;
     if (!pendingProperty) return;
-    const propertyTab = renderTabs ? findTabForProperty(tabs, pendingProperty) : null;
+    const propertyTab = renderTabs ? findTabForProperty(tabsWithAiReviewProperties, pendingProperty) : null;
     if ((propertyTab == null ? void 0 : propertyTab.name) && propertyTab.name !== activeTab) {
       setActiveTab(propertyTab.name);
       saveTabToState(propertyTab.name);
@@ -117256,7 +117295,7 @@ const RenderEntity = forwardRef((props, ref) => {
       cancelScroll();
       window.clearTimeout(captureTimer);
     };
-  }, [activeTab, contextEntity == null ? void 0 : contextEntity["@id"], renderTabs, selectedAiApprovalIndex, tabs, scrollToAiApprovalProperty]);
+  }, [activeTab, contextEntity == null ? void 0 : contextEntity["@id"], renderTabs, selectedAiApprovalIndex, tabsWithAiReviewProperties, scrollToAiApprovalProperty]);
   const isEntityReference2 = (value) => Boolean(value && typeof value === "object" && typeof value["@id"] === "string");
   const restoreNonCoreApprovalValue = (approvalRecord) => {
     const property = approvalRecord == null ? void 0 : approvalRecord.propertyName;
@@ -117844,9 +117883,9 @@ const RenderEntity = forwardRef((props, ref) => {
           ),
           tabbedAboutContent: activeTab === "about" && /* @__PURE__ */ jsxRuntimeExports.jsxs("div", { className: showFieldHelp ? "space-y-3" : "space-y-1", children: [
             renderCoreEntityFields("tabbed"),
-            renderEntityPropertyFields(((_k = tabs.find((tab) => tab.name === "about")) == null ? void 0 : _k.inputs) ?? [], "tabbed")
+            renderEntityPropertyFields(((_k = tabsWithAiReviewProperties.find((tab) => tab.name === "about")) == null ? void 0 : _k.inputs) ?? [], "tabbed")
           ] }),
-          tabbedOtherTabsContent: tabs.map((tab) => {
+          tabbedOtherTabsContent: tabsWithAiReviewProperties.map((tab) => {
             if (tab.name === activeTab && tab.name !== "about") {
               return /* @__PURE__ */ jsxRuntimeExports.jsx("div", { className: showFieldHelp ? "space-y-3" : "space-y-1", children: renderEntityPropertyFields(tab.inputs, "tabbed") }, tab.name);
             }
@@ -126465,7 +126504,7 @@ var EmotionCacheContext = /* @__PURE__ */ React.createContext(
   }) : null
 );
 var CacheProvider = EmotionCacheContext.Provider;
-const version = "0.1.38";
+const version = "0.1.39";
 const pkg = {
   version
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@arpproject/recrate",
-  "version": "0.1.38",
+  "version": "0.1.39",
   "type": "module",
   "main": "./dist/recrate.es.js",
   "module": "./dist/recrate.es.js",