@ethisyscore/plugin-ui 1.100.5 → 1.101.0

package/dist/connector-mappings/index.d.cts

@@ -320,9 +320,14 @@ interface IConnectorMappingService {
      * member. Callers must pass the COMPLETE translation; `useConnectorMappingMutations` is where
      * that is guaranteed.
      *
-     * @returns the replacement row version, or `null` when the host returned none.
+     * @returns the replacement row version (or `null` when the host returned none), together with
+     * whatever the save-time gate established about the mapping that was stored.
+     *
+     * The findings are the point of the return type changing: they ride on a SUCCESS response, and
+     * returning the row version alone discarded them at this layer — so warnings the host composed
+     * were never seen by anyone.
      */
-    updateMapping(connectorId: string, moduleKey: string, operationKey: string, toolName: string, rowVersion: string, requestFieldMap: Record<string, string> | null, responseFieldPaths: Record<string, string> | null, staticRequestFields: Record<string, string> | null): Promise<string | null>;
+    updateMapping(connectorId: string, moduleKey: string, operationKey: string, toolName: string, rowVersion: string, requestFieldMap: Record<string, string> | null, responseFieldPaths: Record<string, string> | null, staticRequestFields: Record<string, string> | null): Promise<MappingSaveOutcome>;
     /**
      * `connectors:mappings:delete`
      *
@@ -405,6 +410,28 @@ interface IConnectorEnumerationService {
     listConnectorTools(connectorId: string): Promise<ConnectorToolSummary[]>;
 }
 
+/**
+ * What the save-time verification gate could establish about the mapping that was stored.
+ *
+ * Rides on a SUCCESS response: the gate refuses only what the invocation path refuses on every
+ * call, and everything else it establishes is true and worth an operator seeing without standing
+ * in the way of a save.
+ */
+interface MappingVerificationFinding {
+    /** Stable code, e.g. `CON_MAP_020`. */
+    code: string;
+    /** `Warning`, or `Refusal` for one the caller acknowledged. */
+    severity: string;
+    /** Short operator-facing summary. */
+    message: string;
+    /** The longer explanation, when there is one. */
+    detail?: string | null;
+}
+/** What a mapping save returns: the refreshed token, and anything the gate had to say. */
+interface MappingSaveOutcome {
+    rowVersion: string | null;
+    findings: MappingVerificationFinding[];
+}
 /**
  * The connector tool-mapping surface, served entirely by the host's `connectors:mappings:*` tools.
  *
@@ -455,7 +482,7 @@ declare class ConnectorMappingService implements IConnectorMappingService, IConn
     /** Single seam every method goes through, so a subclass can wrap logging or retries in one place. */
     protected tool<TRes>(name: string, args?: unknown): Promise<TRes>;
     getMappings(connectorId: string, moduleKey: string): Promise<ConnectorMappingItem[]>;
-    updateMapping(connectorId: string, moduleKey: string, operationKey: string, toolName: string, rowVersion: string, requestFieldMap: Record<string, string> | null, responseFieldPaths: Record<string, string> | null, staticRequestFields: Record<string, string> | null): Promise<string | null>;
+    updateMapping(connectorId: string, moduleKey: string, operationKey: string, toolName: string, rowVersion: string, requestFieldMap: Record<string, string> | null, responseFieldPaths: Record<string, string> | null, staticRequestFields: Record<string, string> | null): Promise<MappingSaveOutcome>;
     deleteMapping(connectorId: string, moduleKey: string, operationKey: string, rowVersion: string): Promise<void>;
     autoMatch(connectorId: string, moduleKey: string): Promise<AutoMatchResponse>;
     getMappingCompleteness(connectorId: string, moduleKey: string): Promise<MappingCompletenessResult>;
@@ -600,6 +627,12 @@ interface UseConnectorMappingMutationsOptions extends ConnectorMappingScope {
      * ultimately reached the same host event.
      */
     showSuccessToast?: (title: string, description?: string) => void;
+    /**
+     * Warning-toast sink, for the save-time gate's findings. Same defaulting and same reason as
+     * {@link showSuccessToast}; separate because a finding reported through the SUCCESS sink reads as
+     * part of "saved successfully", which is a warning nobody acts on.
+     */
+    showWarningToast?: (title: string, description?: string) => void;
 }
 /** Return shape of {@link useConnectorMappingMutations}. */
 interface UseConnectorMappingMutationsResult {
@@ -637,7 +670,7 @@ interface UseConnectorMappingMutationsResult {
  *    See {@link resolveTranslationMember}: it was present in all three repos and was found by this
  *    module's tests rather than in production.
  */
-declare function useConnectorMappingMutations({ service, connectorId, moduleKey, onMappingSaved, showSuccessToast, }: UseConnectorMappingMutationsOptions): UseConnectorMappingMutationsResult;
+declare function useConnectorMappingMutations({ service, connectorId, moduleKey, onMappingSaved, showSuccessToast, showWarningToast, }: UseConnectorMappingMutationsOptions): UseConnectorMappingMutationsResult;
 
 /** Variables for one suggest-translation call. */
 interface SuggestMappingTranslationVars {

package/dist/connector-mappings/index.d.ts

@@ -320,9 +320,14 @@ interface IConnectorMappingService {
      * member. Callers must pass the COMPLETE translation; `useConnectorMappingMutations` is where
      * that is guaranteed.
      *
-     * @returns the replacement row version, or `null` when the host returned none.
+     * @returns the replacement row version (or `null` when the host returned none), together with
+     * whatever the save-time gate established about the mapping that was stored.
+     *
+     * The findings are the point of the return type changing: they ride on a SUCCESS response, and
+     * returning the row version alone discarded them at this layer — so warnings the host composed
+     * were never seen by anyone.
      */
-    updateMapping(connectorId: string, moduleKey: string, operationKey: string, toolName: string, rowVersion: string, requestFieldMap: Record<string, string> | null, responseFieldPaths: Record<string, string> | null, staticRequestFields: Record<string, string> | null): Promise<string | null>;
+    updateMapping(connectorId: string, moduleKey: string, operationKey: string, toolName: string, rowVersion: string, requestFieldMap: Record<string, string> | null, responseFieldPaths: Record<string, string> | null, staticRequestFields: Record<string, string> | null): Promise<MappingSaveOutcome>;
     /**
      * `connectors:mappings:delete`
      *
@@ -405,6 +410,28 @@ interface IConnectorEnumerationService {
     listConnectorTools(connectorId: string): Promise<ConnectorToolSummary[]>;
 }
 
+/**
+ * What the save-time verification gate could establish about the mapping that was stored.
+ *
+ * Rides on a SUCCESS response: the gate refuses only what the invocation path refuses on every
+ * call, and everything else it establishes is true and worth an operator seeing without standing
+ * in the way of a save.
+ */
+interface MappingVerificationFinding {
+    /** Stable code, e.g. `CON_MAP_020`. */
+    code: string;
+    /** `Warning`, or `Refusal` for one the caller acknowledged. */
+    severity: string;
+    /** Short operator-facing summary. */
+    message: string;
+    /** The longer explanation, when there is one. */
+    detail?: string | null;
+}
+/** What a mapping save returns: the refreshed token, and anything the gate had to say. */
+interface MappingSaveOutcome {
+    rowVersion: string | null;
+    findings: MappingVerificationFinding[];
+}
 /**
  * The connector tool-mapping surface, served entirely by the host's `connectors:mappings:*` tools.
  *
@@ -455,7 +482,7 @@ declare class ConnectorMappingService implements IConnectorMappingService, IConn
     /** Single seam every method goes through, so a subclass can wrap logging or retries in one place. */
     protected tool<TRes>(name: string, args?: unknown): Promise<TRes>;
     getMappings(connectorId: string, moduleKey: string): Promise<ConnectorMappingItem[]>;
-    updateMapping(connectorId: string, moduleKey: string, operationKey: string, toolName: string, rowVersion: string, requestFieldMap: Record<string, string> | null, responseFieldPaths: Record<string, string> | null, staticRequestFields: Record<string, string> | null): Promise<string | null>;
+    updateMapping(connectorId: string, moduleKey: string, operationKey: string, toolName: string, rowVersion: string, requestFieldMap: Record<string, string> | null, responseFieldPaths: Record<string, string> | null, staticRequestFields: Record<string, string> | null): Promise<MappingSaveOutcome>;
     deleteMapping(connectorId: string, moduleKey: string, operationKey: string, rowVersion: string): Promise<void>;
     autoMatch(connectorId: string, moduleKey: string): Promise<AutoMatchResponse>;
     getMappingCompleteness(connectorId: string, moduleKey: string): Promise<MappingCompletenessResult>;
@@ -600,6 +627,12 @@ interface UseConnectorMappingMutationsOptions extends ConnectorMappingScope {
      * ultimately reached the same host event.
      */
     showSuccessToast?: (title: string, description?: string) => void;
+    /**
+     * Warning-toast sink, for the save-time gate's findings. Same defaulting and same reason as
+     * {@link showSuccessToast}; separate because a finding reported through the SUCCESS sink reads as
+     * part of "saved successfully", which is a warning nobody acts on.
+     */
+    showWarningToast?: (title: string, description?: string) => void;
 }
 /** Return shape of {@link useConnectorMappingMutations}. */
 interface UseConnectorMappingMutationsResult {
@@ -637,7 +670,7 @@ interface UseConnectorMappingMutationsResult {
  *    See {@link resolveTranslationMember}: it was present in all three repos and was found by this
  *    module's tests rather than in production.
  */
-declare function useConnectorMappingMutations({ service, connectorId, moduleKey, onMappingSaved, showSuccessToast, }: UseConnectorMappingMutationsOptions): UseConnectorMappingMutationsResult;
+declare function useConnectorMappingMutations({ service, connectorId, moduleKey, onMappingSaved, showSuccessToast, showWarningToast, }: UseConnectorMappingMutationsOptions): UseConnectorMappingMutationsResult;
 
 /** Variables for one suggest-translation call. */
 interface SuggestMappingTranslationVars {

package/dist/connector-mappings/index.js

@@ -37,7 +37,10 @@ var ConnectorMappingService = class {
       responseFieldPaths: responseFieldPaths ?? null,
       staticRequestFields: staticRequestFields ?? null
     });
-    return response?.rowVersion ?? null;
+    return {
+      rowVersion: response?.rowVersion ?? null,
+      findings: response?.findings ?? []
+    };
   }
   async deleteMapping(connectorId, moduleKey, operationKey, rowVersion) {
     await this.tool("connectors:mappings:delete", {
@@ -319,6 +322,9 @@ function dispatchToast(title, description, variant) {
 function showSuccessToast(title, description) {
   dispatchToast(title, description, "success");
 }
+function showWarningToast(title, description) {
+  dispatchToast(title, description, "warning");
+}
 
 // src/connector-mappings/types.ts
 var MAPPING_SOURCE = {
@@ -370,7 +376,8 @@ function useConnectorMappingMutations({
   connectorId,
   moduleKey,
   onMappingSaved,
-  showSuccessToast: showSuccessToast2 = showSuccessToast
+  showSuccessToast: showSuccessToast2 = showSuccessToast,
+  showWarningToast: showWarningToast2 = showWarningToast
 }) {
   const queryClient = useQueryClient();
   const [autoMatchSummary, setAutoMatchSummary] = useState(null);
@@ -413,12 +420,20 @@ function useConnectorMappingMutations({
       args.staticRequestFields
     ),
     meta: { errorTitle: "Update failed" },
-    onSuccess: () => {
+    onSuccess: (outcome) => {
       invalidateMappingsAndCompleteness();
       if (connectorId) {
         onMappingSaved?.(connectorId, moduleKey);
       }
-      showSuccessToast2("Mapping updated", "The tool mapping was saved.");
+      const findings = outcome?.findings ?? [];
+      if (findings.length === 0) {
+        showSuccessToast2("Mapping updated", "The tool mapping was saved.");
+        return;
+      }
+      showWarningToast2(
+        findings.length === 1 ? "Mapping saved, with a warning" : `Mapping saved, with ${findings.length} warnings`,
+        findings.map((f) => `${f.message}${f.detail ? ` ${f.detail}` : ""} (${f.code})`).join(" ")
+      );
     }
   });
   const deleteMapping = useMutation({