@adaptabletools/adaptable 22.0.0-canary.6 → 22.0.0-canary.8

package/src/agGrid/AdaptableAgGrid.js

@@ -6,6 +6,7 @@ import { PrimaryKeyDocsLink } from '../Utilities/Constants/DocumentationLinkCons
 import StringExtensions from '../Utilities/Extensions/StringExtensions';
 import Emitter from '../Utilities/Emitter';
 import { applyDefaultAdaptableOptions } from '../AdaptableOptions/DefaultAdaptableOptions';
+import { resolveContainerElement } from '../Utilities/resolveContainerElement';
 import { AgGridAdapter } from './AgGridAdapter';
 import * as GeneralConstants from '../Utilities/Constants/GeneralConstants';
 import { AG_GRID_GRAND_TOTAL_ROW_ID, AG_GRID_GROUPED_COLUMN, AUTOGENERATED_PK_COLUMN, ERROR_LAYOUT, GROUP_PATH_SEPARATOR, HALF_SECOND, QUARTER_SECOND, } from '../Utilities/Constants/GeneralConstants';
@@ -221,6 +222,8 @@ export class AdaptableAgGrid {
      * Temporary, these are MIGRATION technical debts, and should be removed as soon as possible
      */
     adaptableStatusPanelKeys = [];
+    adaptableStatusPanelDefs = [];
+    hasAdaptableStatusBar;
     // see #no_additional_react_root
     _PRIVATE_adaptableJSXElement;
     _prevLayout;
@@ -359,7 +362,7 @@ export class AdaptableAgGrid {
         this.adaptableOptions = this.normalizeAdaptableOptions(this.adaptableOptions);
         const { showLoadingScreen, loadingScreenDelay, loadingScreenText, loadingScreenTitle } = this.adaptableOptions.userInterfaceOptions.loadingScreenOptions;
         if (showLoadingScreen) {
-            this.logger.info(`Show Loading Screen`);
+            this.logger.info('Showing loading screen');
             // it's important to use ensureLoadingScreenPortalElement
             // and not ensurePortalElement, because multiple adaptable instances share the same portal element
             // so when displaying the second one, the react root associated to the portal element
@@ -375,7 +378,7 @@ export class AdaptableAgGrid {
                 }), portalElement);
             }
             else {
-                this.logger.consoleError(`Adaptable failed to show the loading screen!`);
+                this.logger.consoleError('Failed to create loading screen portal element.');
             }
         }
         this.forPlugins((plugin) => plugin.afterInitOptions(this, this.adaptableOptions));
@@ -434,7 +437,7 @@ export class AdaptableAgGrid {
         // see #map_dateString_to_date
         this.agGridAdapter.patchColDefs(columnDefs, (colDef) => {
             if (colDef.cellDataType === 'dateString') {
-                this.logger.consoleError(`AG Grid: Column '${colDef.colId}' specifies cellDataType='dateString' which is no longer supported. It has been automatically replaced with cellDataType='date'. Please update your column definition.`);
+                this.logger.consoleError(`Column "${colDef.colId}" uses cellDataType="dateString" which is no longer supported. It has been replaced with cellDataType="date". Please update your column definition.`);
                 colDef.cellDataType = 'date';
             }
         });
@@ -465,8 +468,8 @@ export class AdaptableAgGrid {
         const agGridApi = await this.initializeAgGrid(gridOptions, config.modules, config.renderAgGridFrameworkComponent);
         if (agGridApi === false) {
             this.midwayDestroy();
-            this.logger.consoleError(`Adaptable failed to initialize AG Grid!`);
-            return Promise.reject('Adaptable failed to initialize AG Grid!');
+            this.logger.consoleError('Failed to initialize AG Grid.');
+            return Promise.reject('Failed to initialize AG Grid.');
         }
         this.layoutManager = new LayoutManager({
             gridApi: agGridApi,
@@ -491,7 +494,7 @@ export class AdaptableAgGrid {
         this.layoutManager.onColumnDefsChanged(() => {
             this.updateColumnModelAndRefreshGrid();
         });
-        this.logger.info(`Hide Loading Screen`);
+        this.logger.info('Hiding loading screen');
         this.unmountLoadingScreen?.();
         perfInitAgGrid.end();
         initAgGridMarker.end();
@@ -502,7 +505,7 @@ export class AdaptableAgGrid {
         this.lifecycleState = 'agGridReady';
         this.api.entitlementApi.internalApi.setModulesAgGridDepsInfos();
         this.ModuleService.logMissingAgGridDepsInfos();
-        this.logger.info(`Registered AG Grid modules (incl. dependent modules): `, this.agGridModulesAdapter.getAgGridRegisteredModuleNames().sort());
+        this.logger.info('Registered AG Grid modules (including dependents):', this.agGridModulesAdapter.getAgGridRegisteredModuleNames().sort());
         /**
          * At this point AG Grid is initialized!
          */
@@ -592,10 +595,7 @@ export class AdaptableAgGrid {
         const layoutState = state.Layout;
         // ensure that at least one Layout has been provided
         if (!layoutState || !layoutState.Layouts?.length) {
-            this.logger
-                .consoleError(`You have not defined any Layout in your InitialState.Layout.Layouts[] state!
-
-You need to define at least one Layout!`);
+            this.logger.consoleError('No Layouts defined in InitialState.Layout.Layouts[]. At least one Layout is required.');
         }
         // ensure CurrentLayout is valid
         if (!layoutState.CurrentLayout ||
@@ -758,21 +758,90 @@ You need to define at least one Layout!`);
     hideQuickFilter() {
         this.agGridAdapter.getAgGridApi().setGridOption('floatingFiltersHeight', 0);
     }
+    showAdaptableToolPanel() {
+        if (!this.hasAdaptableToolPanel) {
+            // don't re-add the ToolPanel if it's hidden by entitlement
+            if (this.api.entitlementApi.isModuleHiddenEntitlement('ToolPanel')) {
+                return;
+            }
+            const currentSideBar = this.agGridAdapter.getGridOption('sideBar');
+            if (currentSideBar?.toolPanels) {
+                const updatedSideBar = {
+                    ...currentSideBar,
+                    toolPanels: [...currentSideBar.toolPanels, this.buildAdaptableToolPanelDef()],
+                };
+                this.agGridAdapter.setGridOption('sideBar', updatedSideBar);
+                this.hasAdaptableToolPanel = true;
+            }
+        }
+    }
+    hideAdaptableToolPanel() {
+        if (this.hasAdaptableToolPanel) {
+            // close the tool panel if it's currently open
+            this.api.toolPanelApi.closeAdapTableToolPanel();
+            const currentSideBar = this.agGridAdapter.getGridOption('sideBar');
+            if (currentSideBar?.toolPanels) {
+                const updatedSideBar = {
+                    ...currentSideBar,
+                    toolPanels: currentSideBar.toolPanels.filter((toolPanelDef) => {
+                        if (typeof toolPanelDef === 'string') {
+                            return toolPanelDef !== GeneralConstants.ADAPTABLE_TOOLPANEL_ID;
+                        }
+                        return toolPanelDef.id !== GeneralConstants.ADAPTABLE_TOOLPANEL_ID;
+                    }),
+                };
+                this.agGridAdapter.setGridOption('sideBar', updatedSideBar);
+            }
+            this.hasAdaptableToolPanel = false;
+        }
+    }
+    buildAdaptableToolPanelDef() {
+        return {
+            id: GeneralConstants.ADAPTABLE_TOOLPANEL_ID,
+            toolPanel: GeneralConstants.ADAPTABLE_TOOLPANEL_COMPONENT,
+            labelDefault: GeneralConstants.ADAPTABLE,
+            labelKey: 'adaptable',
+            iconKey: 'menu',
+            width: UIHelper.getAdaptableToolPanelWidth(),
+            minWidth: UIHelper.getAdaptableToolPanelWidth(),
+        };
+    }
+    showAdaptableStatusBar() {
+        if (!this.hasAdaptableStatusBar && this.adaptableStatusPanelDefs.length > 0) {
+            const currentStatusBar = this.agGridAdapter.getGridOption('statusBar');
+            if (currentStatusBar) {
+                // filter out any Adaptable panels that may already exist to avoid duplicates
+                const existingNonAdaptablePanels = (currentStatusBar.statusPanels ?? []).filter((panel) => !this.adaptableStatusPanelKeys.includes(panel.key));
+                const updatedStatusBar = {
+                    ...currentStatusBar,
+                    statusPanels: [...existingNonAdaptablePanels, ...this.adaptableStatusPanelDefs],
+                };
+                this.agGridAdapter.setGridOption('statusBar', updatedStatusBar);
+                this.hasAdaptableStatusBar = true;
+            }
+        }
+    }
+    hideAdaptableStatusBar() {
+        if (this.hasAdaptableStatusBar) {
+            const currentStatusBar = this.agGridAdapter.getGridOption('statusBar');
+            if (currentStatusBar?.statusPanels) {
+                const updatedStatusBar = {
+                    ...currentStatusBar,
+                    statusPanels: currentStatusBar.statusPanels.filter((panel) => !this.adaptableStatusPanelKeys.includes(panel.key)),
+                };
+                this.agGridAdapter.setGridOption('statusBar', updatedStatusBar);
+            }
+            this.hasAdaptableStatusBar = false;
+        }
+    }
     normalizeAdaptableOptions(adaptableOptions) {
         if (this.hasAutogeneratedPrimaryKey) {
-            this.logger
-                .warn(`Autogenerated primary key (adaptableOptions.autogeneratedPrimaryKey = TRUE) should be used only as a last resort,
-      when no unique column is available, as it limits some Adaptable functionalities!
-
-      For more details see: ${PrimaryKeyDocsLink}`);
+            this.logger.warn(`Autogenerated primary key is enabled (adaptableOptions.autogeneratedPrimaryKey = true). This should be a last resort as it limits some Adaptable features. See: ${PrimaryKeyDocsLink}`);
             this.adaptableOptions.primaryKey = AUTOGENERATED_PK_COLUMN;
             return this.adaptableOptions;
         }
         if (StringExtensions.IsNullOrEmpty(adaptableOptions.primaryKey)) {
-            this.logger.consoleError(`AdaptableOptions.primaryKey is required and cannot be empty or null!
-      As a fallback, you can set adaptableOptions.autogeneratedPrimaryKey = TRUE
-
-      For more details see: ${PrimaryKeyDocsLink}`);
+            this.logger.consoleError(`adaptableOptions.primaryKey is required and must not be empty. As a fallback, set adaptableOptions.autogeneratedPrimaryKey = true. See: ${PrimaryKeyDocsLink}`);
         }
         return adaptableOptions;
     }
@@ -804,7 +873,7 @@ You need to define at least one Layout!`);
          */
         this.agGridOptionsService.setGridOptionsProperty(gridOptions, 'defaultColDef', (original_defaultColDef) => {
             if (original_defaultColDef?.headerValueGetter) {
-                this.logger.warn(`defaultColDef.headerValueGetter and overrides the Adaptable custom header mechanism! We recommend using a ColumnOptions.columnHeader instead!`);
+                this.logger.warn('defaultColDef.headerValueGetter overrides the Adaptable header mechanism. Consider using ColumnOptions.columnHeader instead.');
                 return original_defaultColDef;
             }
             // #customize_header
@@ -821,7 +890,7 @@ You need to define at least one Layout!`);
          */
         this.agGridOptionsService.setGridOptionsProperty(gridOptions, 'defaultColGroupDef', (original_defaultColGroupDef) => {
             if (original_defaultColGroupDef?.headerValueGetter) {
-                this.logger.warn(`defaultColGroupDef.headerValueGetter and overrides the Adaptable custom header mechanism! We recommend using a ColumnOptions.tableColumnHeader instead!`);
+                this.logger.warn('defaultColGroupDef.headerValueGetter overrides the Adaptable header mechanism. Consider using ColumnOptions.tableColumnHeader instead.');
                 return original_defaultColGroupDef;
             }
             const defaultColGroupDef = { ...original_defaultColGroupDef };
@@ -841,12 +910,11 @@ You need to define at least one Layout!`);
             }
             if (original_autoGroupColumnDef?.filter === false) {
                 // if user disables filter on autoGroupColumnDef, we respect that
-                this.logger.info(`autoGroupColumnDef.filter is disabled as per user configuration.`);
+                this.logger.info('autoGroupColumnDef.filter is disabled via user configuration.');
                 return original_autoGroupColumnDef;
             }
             if (typeof original_autoGroupColumnDef?.filter === 'object') {
-                this.logger.consoleWarn(`autoGroupColumnDef.filter is set and overrides the Adaptable custom filter mechanism, so Adaptable will not be able to manage filtering for the Group Column!
-                      Please contact the Adaptable Support Team for further assistance and investigation.`);
+                this.logger.consoleWarn('autoGroupColumnDef.filter overrides the Adaptable filtering mechanism for the Group Column. Adaptable-managed filters will not apply to this column. Contact support for alternatives.');
                 return original_autoGroupColumnDef;
             }
             const autoGroupColumnDef = { ...original_autoGroupColumnDef };
@@ -854,8 +922,7 @@ You need to define at least one Layout!`);
             // !! DO NOT REMOVE THIS !!
             // see https://github.com/ag-grid/ag-grid/blob/6f43ff257c2e285068eb425b655e58d6eeb89816/packages/ag-grid-enterprise/src/rowHierarchy/autoColService.ts#L225
             autoGroupColumnDef.filterValueGetter = (params) => {
-                this.logger.consoleWarn(`An unexpected invocation of autoGroupColumnDef.filterValueGetter was detected. This execution path is not expected under normal operation.
-                      Please contact the Adaptable Support Team for further assistance and investigation.`, params);
+                this.logger.consoleWarn('Unexpected invocation of autoGroupColumnDef.filterValueGetter. This is not expected under normal operation. Please contact support.', params);
                 return '';
             };
             // !! it is important that we always return the same component / handler instances
@@ -1169,16 +1236,7 @@ You need to define at least one Layout!`);
                     return original_sideBar;
                 }
                 const isAdaptableToolPanelHidden = this.api.entitlementApi.isModuleHiddenEntitlement('ToolPanel');
-                const adaptableToolPanelDef = {
-                    id: GeneralConstants.ADAPTABLE_TOOLPANEL_ID,
-                    toolPanel: GeneralConstants.ADAPTABLE_TOOLPANEL_COMPONENT,
-                    labelDefault: GeneralConstants.ADAPTABLE,
-                    labelKey: 'adaptable',
-                    iconKey: 'menu',
-                    width: UIHelper.getAdaptableToolPanelWidth(),
-                    minWidth: UIHelper.getAdaptableToolPanelWidth(),
-                    // maxWidth = undefined,
-                };
+                const adaptableToolPanelDef = this.buildAdaptableToolPanelDef();
                 const mapToolPanelDefs = (toolPanelDefs = []) => {
                     // if it's an alias for the adaptable tool panel, map it to a ToolPanelDef, otherwise return it as it is
                     return toolPanelDefs.map((toolPanelDef) => toolPanelDef === GeneralConstants.ADAPTABLE_TOOLPANEL_ID
@@ -1253,19 +1311,29 @@ You need to define at least one Layout!`);
             this.agGridOptionsService.setGridOptionsProperty(gridOptions, 'statusBar', (original_statusBar) => {
                 const statusPanels = (original_statusBar?.statusPanels ?? [])?.map((statusPanel) => {
                     if (statusPanel.statusPanel === ADAPTABLE_STATUS_PANEL) {
+                        if (!statusPanel.key) {
+                            this.logger.consoleWarn('AdaptableStatusPanel is missing a "key" property. This panel will not be managed by HideAdaptableUI. Please provide a unique key.');
+                            return statusPanel;
+                        }
                         this.adaptableStatusPanelKeys.push(statusPanel.key);
                         const context = {
                             Key: statusPanel.key,
                         };
-                        return {
+                        const adaptablePanelDef = {
                             ...statusPanel,
                             statusPanel: createAgStatusPanelComponent(AdaptableStatusBar, this, context),
                         };
+                        this.adaptableStatusPanelDefs.push(adaptablePanelDef);
+                        return adaptablePanelDef;
                     }
                     return statusPanel;
                 });
                 return { ...original_statusBar, statusPanels };
             });
+            this.hasAdaptableStatusBar = this.adaptableStatusPanelKeys.length > 0;
+        }
+        else {
+            this.hasAdaptableStatusBar = false;
         }
         /**
          * `getRowStyle`
@@ -1425,15 +1493,14 @@ You need to define at least one Layout!`);
             const rowData = agGridApi.getGridOption('rowData');
             this.initWithLazyData = rowData == undefined || rowData.length === 0;
             if (this.initWithLazyData) {
-                this.logger.info('initWithLazyData = TRUE');
+                this.logger.info('Initializing with lazy data (no initial rowData).');
             }
             if (!this.getAgGridContainerElement()) {
                 // initialize the agGridContainerElement from the AgGrid instance
                 const gridRoot = this.agGridAdapter.getAgGridRootElement();
                 const gridContainer = gridRoot?.closest('[class*="ag-theme"]');
                 if (!gridContainer) {
-                    this.logger.consoleError(`No AG Grid container could be derived from the Adaptable framework wrapper.
-            Please contact AdapTable Support and in the meantime provide a valid container element in 'ContainerOptions.agGridContainer'!`);
+                    this.logger.consoleError('Failed to derive AG Grid container from the framework wrapper. Please provide a valid container element via ContainerOptions.agGridContainer.');
                 }
                 this.DANGER_USE_GETTER_agGridContainerElement = gridContainer;
             }
@@ -1441,7 +1508,7 @@ You need to define at least one Layout!`);
         }
         const agGridContainer = this.getAgGridContainerElement();
         if (!agGridContainer) {
-            this.logger.consoleError('No AG Grid container element found in the DOM. Please provide a valid container element in `ContainerOptions.agGridContainer`');
+            this.logger.consoleError('AG Grid container element not found in the DOM. Please provide a valid element via ContainerOptions.agGridContainer.');
             return Promise.resolve(false);
         }
         let gridParams;
@@ -1454,21 +1521,22 @@ You need to define at least one Layout!`);
     useRowNodeLookUp() {
         return this.agGridAdapter.getAgGridApi().getGridOption('getRowId') != undefined;
     }
+    buildInitContainerContext() {
+        return {
+            adaptableId: this.adaptableOptions?.adaptableId ?? '',
+            adaptableStateKey: this.adaptableOptions?.adaptableStateKey ?? '',
+            adaptableContext: this.adaptableOptions?.adaptableContext,
+        };
+    }
     getAgGridContainerElement() {
         if (!this.DANGER_USE_GETTER_agGridContainerElement) {
-            this.DANGER_USE_GETTER_agGridContainerElement =
-                typeof this.adaptableOptions?.containerOptions?.agGridContainer === 'string'
-                    ? document.getElementById(this.adaptableOptions?.containerOptions?.agGridContainer)
-                    : this.adaptableOptions?.containerOptions?.agGridContainer;
+            this.DANGER_USE_GETTER_agGridContainerElement = resolveContainerElement(this.adaptableOptions?.containerOptions?.agGridContainer, this.buildInitContainerContext());
         }
         return this.DANGER_USE_GETTER_agGridContainerElement;
     }
     getAdaptableContainerElement() {
         if (!this.DANGER_USE_GETTER_adaptableContainerElement) {
-            this.DANGER_USE_GETTER_adaptableContainerElement =
-                typeof this.adaptableOptions?.containerOptions?.adaptableContainer === 'string'
-                    ? document.getElementById(this.adaptableOptions?.containerOptions?.adaptableContainer)
-                    : this.adaptableOptions?.containerOptions?.adaptableContainer;
+            this.DANGER_USE_GETTER_adaptableContainerElement = resolveContainerElement(this.adaptableOptions?.containerOptions?.adaptableContainer, this.buildInitContainerContext());
         }
         return this.DANGER_USE_GETTER_adaptableContainerElement;
     }
@@ -1856,7 +1924,7 @@ You need to define at least one Layout!`);
             // if no primary key column then lets check the first row to see if its a data item
             const primaryKeyDataItem = this.getFirstRowNode()?.data[primaryKey];
             if (!primaryKeyDataItem) {
-                errorMessage = `The Primary Key '${this.adaptableOptions.primaryKey}' does not exist.  This will affect many functions in AdapTable.`;
+                errorMessage = `Primary key column "${this.adaptableOptions.primaryKey}" not found. This will affect many Adaptable features.`;
                 if (this.adaptableOptions.alertOptions.showMissingPrimaryKeyAlert) {
                     // show an alert if that is the option
                     this.api.alertApi.showAlertError('No Primary Key', errorMessage);
@@ -1913,12 +1981,12 @@ You need to define at least one Layout!`);
     checkShouldClearExistingFiltersOrSearches() {
         // if they have selected to clear Filters on startup then do it
         if (this.adaptableOptions.filterOptions.clearFiltersOnStartUp) {
-            this.logger.warn('Clearing any saved Filters as "clearFiltersOnStartUp" is true');
+            this.logger.warn('Clearing saved filters (clearFiltersOnStartUp is enabled).');
             this.api.filterApi.clearAllFilters();
         }
         // if they have selected to clear searches on startup then do it
         if (this.adaptableOptions.quickSearchOptions.clearQuickSearchOnStartUp) {
-            this.logger.warn('Clearing any saved Searches as "clearQuickSearchOnStartUp" is true');
+            this.logger.warn('Clearing saved Quick Search (clearQuickSearchOnStartUp is enabled).');
             this.api.quickSearchApi.clearQuickSearch();
         }
     }
@@ -2147,7 +2215,7 @@ You need to define at least one Layout!`);
         }
     }
     _updateColumnModelAndRefreshGridNow() {
-        this.logger.info(`Updating Column Model and Refreshing Grid.`);
+        this.logger.info('Updating column model and refreshing grid.');
         this.deriveAdaptableColumnStateFromAgGrid();
         this.agGridColumnAdapter.setupColumns();
         this.redrawBody();
@@ -2173,7 +2241,7 @@ You need to define at least one Layout!`);
             this.agGridAdapter.getAgGridApi().redrawRows(redrawRowsParams);
         }
         catch (ex) {
-            this.logger.consoleError('AG Grid redrawRows was unable to find some row nodes. Tried to redraw row nodes: ', rowNodes, ex);
+            this.logger.consoleError('AG Grid redrawRows failed to locate some row nodes.', rowNodes, ex);
         }
     }
     redrawRow(rowNode) {
@@ -2467,31 +2535,31 @@ You need to define at least one Layout!`);
             });
         }
     }
-    async manageGridRows(dataRows, config = {}) {
+    async manageGridRows(dataRowConfig, dataUpdateConfig = {}) {
         const result = {
             addedRows: [],
             updatedRows: [],
             removedRows: [],
         };
-        if (!dataRows) {
+        if (!dataRowConfig || !dataRowConfig.dataRows) {
             return result;
         }
-        const addDataRows = dataRows.add;
-        const updateDataRows = dataRows.update;
-        const removeDataRows = dataRows.delete;
+        const addDataRows = dataRowConfig.dataRows.add;
+        const updateDataRows = dataRowConfig.dataRows.update;
+        const removeDataRows = dataRowConfig.dataRows.delete;
         if (this.hasAutogeneratedPrimaryKey) {
             this.addSyntheticPrimaryKeyIfMissing(addDataRows);
         }
-        if (config.runAsync) {
+        if (dataUpdateConfig.runAsync) {
             return new Promise((resolve) => {
                 this.applyAgGridTransactionAsync({
                     update: updateDataRows,
                     add: addDataRows,
                     remove: removeDataRows,
-                    addIndex: config.addIndex,
+                    addIndex: dataUpdateConfig.addIndex,
                 }, (transaction) => {
-                    if (typeof config.callback === 'function') {
-                        config.callback(transaction);
+                    if (typeof dataUpdateConfig.callback === 'function') {
+                        dataUpdateConfig.callback(transaction);
                     }
                     if (transaction?.add) {
                         this.updateRowGroupsAndColumnGroupsExpandedState();
@@ -2502,7 +2570,7 @@ You need to define at least one Layout!`);
                         removedRows: transaction?.remove,
                     });
                 });
-                if (config.flushAsync) {
+                if (dataUpdateConfig.flushAsync) {
                     this.agGridAdapter.getAgGridApi().flushAsyncTransactions();
                 }
             });
@@ -2512,12 +2580,12 @@ You need to define at least one Layout!`);
                 update: updateDataRows,
                 add: addDataRows,
                 remove: removeDataRows,
-                addIndex: config.addIndex,
+                addIndex: dataUpdateConfig.addIndex,
             });
             if (transaction?.add) {
                 this.updateRowGroupsAndColumnGroupsExpandedState();
             }
-            if (config.flushAsync) {
+            if (dataUpdateConfig.flushAsync) {
                 this.agGridAdapter.getAgGridApi().flushAsyncTransactions();
             }
             return Promise.resolve({
@@ -2583,14 +2651,14 @@ You need to define at least one Layout!`);
     }
     autoSizeColumns(columnIds) {
         if (!this.agGridModulesAdapter.isAgGridModuleRegistered('ColumnAutoSizeModule')) {
-            this.logger.consoleError(`Could not auto-size columns [${columnIds}] - missing required AG Grid module 'ColumnAutoSizeModule'`);
+            this.logger.consoleError(`Auto-sizing columns [${columnIds}] requires the AG Grid "ColumnAutoSizeModule". Please register this module.`);
             return;
         }
         this.layoutManager.autoSizeColumns(columnIds);
     }
     autoSizeAllColumns() {
         if (!this.agGridModulesAdapter.isAgGridModuleRegistered('ColumnAutoSizeModule')) {
-            this.logger.consoleError(`Could not auto-size all columns - missing required AG Grid module 'ColumnAutoSizeModule'`);
+            this.logger.consoleError('Auto-sizing all columns requires the AG Grid "ColumnAutoSizeModule". Please register this module.');
             return;
         }
         const agGridApi = this.agGridAdapter.getAgGridApi();
@@ -2748,7 +2816,7 @@ You need to define at least one Layout!`);
                 customInFilterValuesResult = await customInFilterValues(customInFilterValuesContext);
             }
             catch (error) {
-                this.logger.consoleError(`Failed to fetch custom filter values`, column, error);
+                this.logger.consoleError('Failed to retrieve custom filter values.', column, error);
             }
             return customInFilterValuesResult;
         }
@@ -2788,7 +2856,7 @@ You need to define at least one Layout!`);
                     customDistinctValues = await customEditColumnValues(customEditValuesContext);
                 }
                 catch (error) {
-                    this.logger.consoleError(`Failed to fetch custom edit values`, column, error);
+                    this.logger.consoleError('Failed to retrieve custom edit values.', column, error);
                 }
                 return customDistinctValues ?? [];
             }
@@ -3243,7 +3311,7 @@ You need to define at least one Layout!`);
         if (this.getAgGridRowModelType() !== 'clientSide') {
             // only in client-side row model can we loop through filtered&sorted rows
             // see https://www.ag-grid.com/javascript-data-grid/accessing-data/#iterating-rows
-            this.logger.warn('`forAllVisibleRowNodesDo()` is only supported in client-side row model. `forAllRowNodesDo` will be used instead.');
+            this.logger.warn('forAllVisibleRowNodesDo() is only supported with client-side row model. Falling back to forAllRowNodesDo().');
             return this.forAllRowNodesDo(func, config);
         }
         this.agGridAdapter.getAgGridApi().forEachNodeAfterFilterAndSort((rowNode, rowIndex) => {
@@ -3295,14 +3363,14 @@ You need to define at least one Layout!`);
     }
     selectNode(rowNode, clearSelection) {
         if (!rowNode) {
-            this.logger.error('No node to select');
+            this.logger.error('Cannot select: row node is null.');
             return;
         }
         rowNode.setSelected(true, clearSelection);
     }
     deSelectNode(rowNode, clearSelection) {
         if (!rowNode) {
-            this.logger.error('No node to deselect');
+            this.logger.error('Cannot deselect: row node is null.');
             return;
         }
         rowNode.setSelected(false, clearSelection);
@@ -3390,7 +3458,7 @@ You need to define at least one Layout!`);
     }
     showChart(chartDefinition, container) {
         if (!this.isReady) {
-            this.logger.consoleError('Adaptable must be instantiated before calling showChart');
+            this.logger.consoleError('Cannot call showChart before Adaptable is fully initialized.');
             return null;
         }
         /**
@@ -3411,7 +3479,7 @@ You need to define at least one Layout!`);
     }
     updateChart(chart) {
         if (!chart.Model) {
-            this.logger.consoleError(`Chart definition must have a Model property to be updated: ${chart.Name}`);
+            this.logger.consoleError(`Cannot update chart "${chart.Name}": the chart definition is missing the required Model property.`);
             return;
         }
         const upgradableProperties = {
@@ -3437,7 +3505,7 @@ You need to define at least one Layout!`);
     }
     getChartModels() {
         if (!this.isReady) {
-            this.logger.consoleError('Adaptable must be instantiated before calling getChartModels');
+            this.logger.consoleError('Cannot call getChartModels before Adaptable is fully initialized.');
             return [];
         }
         return this.agGridAdapter.getAgGridApi().getChartModels();
@@ -3471,7 +3539,7 @@ You need to define at least one Layout!`);
     }
     setAgGridQuickSearch(searchText) {
         if (!this.agGridModulesAdapter.isAgGridModuleRegistered('QuickFilterModule')) {
-            this.logger.consoleError(`Can NOT set Quick Search text - missing required AG Grid module 'QuickFilterModule'`);
+            this.logger.consoleError('Quick Search requires the AG Grid "QuickFilterModule". Please register this module to enable Quick Search.');
             return;
         }
         this.agGridAdapter.setGridOption('quickFilterText', searchText);
@@ -3481,7 +3549,7 @@ You need to define at least one Layout!`);
     }
     setAgGridFindSearchValue(searchText) {
         if (!this.agGridModulesAdapter.isAgGridModuleRegistered('FindModule')) {
-            this.logger.consoleError(`Can NOT set Find Search text - missing required AG Grid module 'FindModule'`);
+            this.logger.consoleError('Find Search requires the AG Grid "FindModule". Please register this module to enable Find Search.');
             return;
         }
         this.agGridAdapter.setGridOption('findSearchValue', searchText);
@@ -4010,7 +4078,7 @@ You need to define at least one Layout!`);
         cellDataChangedInfos.forEach((cellDataChangedInfo) => {
             // if a Cell Data Change is undone, log to the Console
             if (cellDataChangedInfo.trigger === 'undo') {
-                this.logger.info(`Undo data change: PK(${cellDataChangedInfo.primaryKeyValue}) Col(${cellDataChangedInfo.column}) RevertedValue(${cellDataChangedInfo.oldValue}) OriginalValue(${cellDataChangedInfo.newValue})`);
+                this.logger.info(`Data change reverted: column="${cellDataChangedInfo.column}", primaryKey="${cellDataChangedInfo.primaryKeyValue}", from=${cellDataChangedInfo.newValue} to=${cellDataChangedInfo.oldValue}`);
             }
             // For Changes except Aggregations, reset cell selection if required
             if (cellDataChangedInfo.trigger === 'edit' ||
@@ -4215,7 +4283,7 @@ You need to define at least one Layout!`);
         this.api.layoutApi.createOrUpdateLayout(layout);
     }
     refreshAdaptableAfterLayoutChange(layout) {
-        this.logger.info('refreshAdaptableAfterLayoutChange()');
+        this.logger.info('Refreshing Adaptable after layout change.');
         const prevLayoutForRefresh = this.__prevLayoutForRefresh || this.api.layoutApi.getCurrentLayout();
         // see #on-regroup-expect-group-column-to-be-recomputed-and-setup-properly
         const rowGroupsChanged = this.isRowGroupDifferentInLayout(prevLayoutForRefresh, layout);
@@ -4261,7 +4329,7 @@ You need to define at least one Layout!`);
                 .forEach((colType) => {
                 const newValidCellDataType = mapOldTypeToDataType(colType);
                 if (newValidCellDataType) {
-                    this.logger.consoleWarn(`Column ${columnDefinition.colId ?? columnDefinition.field} has a type of ${colType} which is no longer in use${newValidCellDataType !== 'unknown' ? `. Please use 'colDef.cellDataType' instead: ${newValidCellDataType}` : ''}`);
+                    this.logger.consoleWarn(`Column "${columnDefinition.colId ?? columnDefinition.field}" uses deprecated type "${colType}"${newValidCellDataType !== 'unknown' ? `. Use colDef.cellDataType="${newValidCellDataType}" instead.` : '. This type is no longer supported.'}`);
                 }
             });
         });

package/src/agGrid/AgGridAdapter.js

@@ -145,7 +145,7 @@ export class AgGridAdapter {
                     }
                 }
                 options['columnDefs'] = colDefsWithSpecialColumns;
-                self.logger.info(`Added SpecialColumns on GridOptions.columnDefs update (source=${source})`);
+                self.logger.info(`Added special columns on GridOptions.columnDefs update (source="${source}").`);
             }
             // `context`
             const passedContext = options.context;
@@ -165,7 +165,7 @@ export class AgGridAdapter {
         }
         const agGridColumnFilterService = this.DANGER_getPrivateAgGridBeans()?.colFilter;
         if (!agGridColumnFilterService) {
-            this.logger.consoleError('Could not get hold of ColumnFilterService! This is a critical error and will prevent Adaptable from working correctly.');
+            this.logger.consoleError('Failed to initialize ColumnFilterService. Filtering and related features will not function correctly.');
             return;
         }
         const self = this;
@@ -211,7 +211,7 @@ export class AgGridAdapter {
     DANGER_getPrivateAgGridBeans() {
         const beans = DANGER_AG_GRID_BEANS_MAP[this._agGridId];
         if (!beans) {
-            this.logger.consoleError('Could not get hold of AgGridBeans! This is a critical error and will prevent Adaptable from working correctly.');
+            this.logger.consoleError('Failed to access AG Grid internal beans. Adaptable will not function correctly.');
         }
         return beans;
     }
@@ -355,7 +355,7 @@ export class AgGridAdapter {
                     if (isEndRowPin) {
                         shouldIncludeRange = false;
                     }
-                    this.logger.consoleWarn('Cannot select pinned rows in AG Grid.');
+                    this.logger.consoleWarn('Pinned rows cannot be selected in AG Grid.');
                 }
                 if (shouldIncludeRange) {
                     const y1 = Math.min(rangeSelection.startRow.rowIndex, rangeSelection.endRow.rowIndex);
@@ -525,7 +525,7 @@ export class AgGridAdapter {
     deriveAdaptableColumnDataType(agColumn, logWarning = true) {
         // Some columns can have no ID or Title. we return string as a consequence but it needs testing
         if (!agColumn) {
-            this.logger.warn(`Column is undefined, returning 'text' for Type`);
+            this.logger.warn('Column is undefined. Defaulting data type to "text".');
             return 'text';
         }
         if (this.adaptableApi.columnApi.isAutoRowGroupColumnForSingle(agColumn.getId())) {
@@ -568,7 +568,7 @@ export class AgGridAdapter {
         let row = this.getAgGridApi().getDisplayedRowAtIndex(0);
         if (row == null) {
             // possible that there will be no data.
-            this.logger.consoleError(`No data in grid, returning type "unknown" for Column: "${agColumn.getColId()}". This will impact several Adaptable features, such as Filters and ColumnFormats.`);
+            this.logger.consoleError(`No data in grid. Returning type "unknown" for column "${agColumn.getColId()}". This will affect features such as Filters and Column Formats.`);
             return 'unknown';
         }
         // // if it's a group we need the content of the group
@@ -610,7 +610,7 @@ export class AgGridAdapter {
                 dataType = arrayDataType;
             }
         }
-        this.logger.consoleWarn(`No defined type for column '${agColumn.getColId()}'. Defaulting to type of first row value: ${dataType}`);
+        this.logger.consoleWarn(`No explicit type for column "${agColumn.getColId()}". Inferred type from first row value: "${dataType}".`);
         return dataType;
     }
     isColumnReadonly(colDef) {
@@ -796,7 +796,7 @@ export class AgGridAdapter {
                 colDef.colId = colDef.field;
             }
             if (!colDef.colId) {
-                this.logger.warn('A column is missing the colId - please check ', colDef, 'Either pass a "field" property or a "colId" property.');
+                this.logger.warn('Column definition is missing colId. Provide either a "field" or "colId" property.', colDef);
             }
         };
         this.patchColDefs(colDefs, assignColId);