@adaptabletools/adaptable-cjs 23.0.4-canary.0 → 23.0.5-canary.0

package/src/AdaptableState/Common/AdaptablePredicate.js

@@ -23,13 +23,12 @@ function getGroupValuesForNode(config) {
     const { node, value } = config;
     const groupValues = node.group
         ? []
-        : /*for the leaf node, also use its value*/ value != null
+        : value != null
             ? [value]
             : [];
     let currentNode = node.group ? node : node.parent;
     while (currentNode && currentNode.groupData && Object.keys(currentNode.groupData).length > 0) {
         if (currentNode.treeParent && !currentNode.data) {
-            // skip filler groups in tree mode
         }
         else {
             groupValues.unshift(currentNode.groupData[GeneralConstants_1.AG_GRID_GROUPED_COLUMN]);
@@ -38,9 +37,6 @@ function getGroupValuesForNode(config) {
     }
     return groupValues;
 }
-/**
- * Array of Predicate Defs which are shipped by AdapTable
- */
 exports.SystemPredicateDefs = [
     {
         id: 'In',
@@ -59,11 +55,8 @@ exports.SystemPredicateDefs = [
                 if (!predicateInput) {
                     return;
                 }
-                // make sure that the input Predicate  is included in the 'In' Predicate
                 if (adaptableApi.columnScopeApi.scopeIsAll(predicateInput.columnScope) ||
-                    adaptableApi.columnScopeApi.isScopeInScope(predicateInput.columnScope, 
-                    // 'In' ColumnScope
-                    {
+                    adaptableApi.columnScopeApi.isScopeInScope(predicateInput.columnScope, {
                         DataTypes: ['text', 'number', 'date', 'textArray', 'numberArray'],
                     })) {
                     return predicateInput;
@@ -79,7 +72,6 @@ exports.SystemPredicateDefs = [
                     }, nestedContext, false);
                 });
                 if (predicateResult) {
-                    // We want to use the only true values, to allow for multiple predicates to be used
                     return predicateResult;
                 }
             }
@@ -89,17 +81,7 @@ exports.SystemPredicateDefs = [
             if (column.columnId === GeneralConstants_1.AG_GRID_GROUPED_COLUMN) {
                 const { node, treeSelectionState } = context;
                 const groupValues = getGroupValuesForNode({ node, value });
-                // for non-trees, this line is reached for every leaf-row
-                // so for 2 levels-deep, groupValues is an array of ['level1-value','level2-value','leaf-value']
-                // but for trees, the groupValues is first an array with only
-                // one value, the current branch root - and if we go via the treeSelectionState
-                // that root might have a leaf selected, but if we return true here
-                // ag grid would include all rows of that branch - which is not what we want
-                // so we want to enter the `if` block and use the startsWith check
                 if (column.isTreeColumn || !treeSelectionState) {
-                    // in case we don't have treeSelectionState
-                    // this is another possible implementation
-                    // though it won't be as performant as using the treeSelectionState
                     const currentValues = (groupValues ?? []).join('#');
                     return inputs.some((values) => {
                         const valuesString = values.join('#');
@@ -107,32 +89,14 @@ exports.SystemPredicateDefs = [
                     });
                 }
                 const result = groupValues ? treeSelectionState.isNodeSelected(groupValues) : false;
-                // console.log('filter', groupValues, 'pass', result, treeSelectionState.getState());
                 return result;
             }
             if (column.dataType === 'date') {
-                // Inputs come from one of two flows:
-                //   1. The default-values tree's calendar leaves — short strings
-                //      like "2024", "2024-01" or "2024-01-15" (no time component).
-                //   2. A full timestamp — either the tree's optional time leaf or
-                //      the `customInFilterValues` + `orderedValues` path, where
-                //      `Date` values are normalised to ISO strings (always carrying
-                //      a `T` separator).
-                //
-                // `evaluateInPredicateUsingTime` is only meaningful for case (1):
-                // it switches the calendar leaf between day-level matching and
-                // exact-instant matching against the cell's `Date`. For case (2)
-                // the user has explicitly picked a specific instant, so we always
-                // compare with time regardless of the option.
                 return inputs.some((input) => {
                     const hasTimeComponent = typeof input === 'string' && input.includes('T');
                     const useTime = hasTimeComponent ||
                         adaptableApi.predicateApi.internalApi.shouldEvaluateInPredicateUsingTime(column.columnId);
                     if (useTime) {
-                        // `input` may be a string (e.g. an ISO timestamp serialised through
-                        // JSON when the listbox tree dropdown round-trips path keys), so
-                        // normalise both sides to `Date` before comparing — date-fns
-                        // `isEqual` only accepts `Date | number`.
                         return (0, date_fns_3.isEqual)((0, DateHelper_1.parseDateValue)(input), (0, DateHelper_1.parseDateValue)(value));
                     }
                     else {
@@ -172,80 +136,6 @@ exports.SystemPredicateDefs = [
         toString: ({ inputs }) => `IN (${inputs.join(', ')})`,
         shortcuts: ['#', '['],
     },
-    // {
-    //   id: 'Tree',
-    //   label: 'Tree',
-    //   icon: { text: 'Tree' },
-    //   columnScope: { DataTypes: ['date'] },
-    //   moduleScope: ['columnFilter', 'flashingcell', 'formatColumn', 'alert'],
-    //   handler: (context) => {
-    //     const { inputs = [], column, value, adaptableApi } = context;
-    //     const predicateInputs = (context.inputs || []).reduce<AdaptablePredicateDef[]>(
-    //       (acc, input) => {
-    //         const predicate = adaptableApi.predicateApi.getPredicateDefById(input);
-    //         if (predicate) {
-    //           acc.push(predicate);
-    //         }
-    //         return acc;
-    //       },
-    //       []
-    //     );
-    //     if (predicateInputs.length) {
-    //       const nestedContext = { ...context };
-    //       delete nestedContext.inputs;
-    //       const predicateResult = predicateInputs.some((predicate) => {
-    //         return adaptableApi.predicateApi.handlePredicate(
-    //           {
-    //             PredicateId: predicate.id,
-    //           },
-    //           nestedContext,
-    //           false
-    //         );
-    //       });
-    //       if (predicateResult) {
-    //         // We want to use the only true values, to allow for multiple predicates to be used
-    //         return predicateResult;
-    //       }
-    //     }
-    //     if (inputs.length === 0) {
-    //       return true;
-    //     }
-    //     if (column.dataType === 'date') {
-    //       const treeSelectionState = new TreeSelectionState(
-    //         {
-    //           defaultSelection: false,
-    //           // make sure something like
-    //           // Predicates: [{ PredicateId: 'Tree', Inputs: [['2021', '2'], ['2022']] }],
-    //           // is always converted to numbers
-    //           selectedPaths: inputs
-    //             .map((array) => {
-    //               if (!Array.isArray(array)) {
-    //                 return null;
-    //               }
-    //               return array.map(Number);
-    //             })
-    //             .filter(Boolean),
-    //         },
-    //         () => {
-    //           return {
-    //             treeDeepMap: new DeepMap<string, any>(),
-    //             treePaths: new DeepMap<string, any>(),
-    //           };
-    //         }
-    //       );
-    //       const currrentDate = parseDateValue(value);
-    //       const currentYear = currrentDate.getFullYear();
-    //       const currentMonth = currrentDate.getMonth() + 1;
-    //       const currentDay = currrentDate.getDate();
-    //       const result = treeSelectionState.isNodeSelected([currentYear, currentMonth, currentDay]);
-    //       // console.log({ currentYear, currentMonth, currentDay, result, inputs: inputs.join('-') });
-    //       return result;
-    //     }
-    //     return true;
-    //   },
-    //   toString: ({ inputs }) => `IN Tree (${inputs.join(', ')})`,
-    //   shortcuts: ['#', '['],
-    // },
     {
         id: 'NotIn',
         label: 'Not In',
@@ -260,11 +150,8 @@ exports.SystemPredicateDefs = [
                 if (!predicateInput) {
                     return;
                 }
-                // make sure that the input Predicate  is included in the 'NotIn' Predicate
                 if (adaptableApi.columnScopeApi.scopeIsAll(predicateInput.columnScope) ||
-                    adaptableApi.columnScopeApi.isScopeInScope(predicateInput.columnScope, 
-                    // 'NotIn' ColumnScope
-                    {
+                    adaptableApi.columnScopeApi.isScopeInScope(predicateInput.columnScope, {
                         DataTypes: ['text', 'number', 'date', 'textArray', 'numberArray'],
                     })) {
                     return predicateInput;
@@ -281,11 +168,9 @@ exports.SystemPredicateDefs = [
                     return res === false;
                 });
                 if (!predicateResult) {
-                    // We want to use the only true values, to allow for multiple predicates to be used
                     return false;
                 }
             }
-            // basically negation of IN
             if (inputs.length === 0) {
                 return true;
             }
@@ -361,7 +246,6 @@ exports.SystemPredicateDefs = [
             return Helper_1.default.isInputNotNullOrEmpty(value);
         },
     },
-    // Numeric System Filters
     {
         id: 'GreaterThan',
         label: 'Greater Than',
@@ -474,7 +358,6 @@ exports.SystemPredicateDefs = [
         toString: ({ inputs }) => `Not Between ${inputs[0] ?? ''}:${inputs[1]}`,
         shortcuts: ['!:'],
     },
-    // String System Filters
     {
         id: 'Is',
         label: 'Equals',
@@ -595,7 +478,6 @@ exports.SystemPredicateDefs = [
         handler: ({ value, inputs }) => new RegExp(inputs[0]).test(value),
         toString: ({ inputs }) => `Regex ${inputs[0] ?? ''}`,
     },
-    // Date System Filters
     {
         id: 'Today',
         label: 'Today',
@@ -749,7 +631,6 @@ exports.SystemPredicateDefs = [
         inputs: [{ type: 'date' }, { type: 'date' }],
         handler: ({ value, inputs }) => new Date(value) >= new Date(inputs[0]) && new Date(value) <= new Date(inputs[1]),
     },
-    // Boolean System Filters
     {
         id: 'True',
         label: 'True',
@@ -773,17 +654,14 @@ exports.SystemPredicateDefs = [
         columnScope: { DataTypes: ['boolean'] },
         moduleScope: ['columnFilter'],
         inputs: [{ type: 'boolean' }],
-        // working with string aliases instead of booleans because the ColumnFilterAPI cannot handle falsy filter values (it ignores them)
         handler: ({ value, inputs }) => {
             if (inputs[0] == 'all') {
-                // if it's indeterminate, all values (true&false) are matched
                 return true;
             }
             const booleanInputValue = inputs[0] === 'checked' ? true : false;
             return Boolean(value) === Boolean(booleanInputValue);
         },
     },
-    // Other System Filters
     {
         id: 'AnyChange',
         label: 'Any Change',

package/src/AdaptableState/Common/AggregationColumns.js

@@ -23,22 +23,12 @@ const getAggFuncName = (aggFunc) => {
     }
 };
 exports.getAggFuncName = getAggFuncName;
-/**
- * Whether the Weighted Average aggregation should be offered for this Column:
- * it must be a numeric, aggregatable Column that either has no agg-func
- * restriction or explicitly allows `weightedAvg`.
- */
 const isWeightedAverageAggFuncEligible = (column) => {
     const weightedAverageAllowed = !column.userAllowedAggFuncs?.length ||
         column.userAllowedAggFuncs.includes(exports.WEIGHTED_AVERAGE_AGG_FN_NAME);
     return column.dataType === 'number' && column.aggregatable && weightedAverageAllowed;
 };
 exports.isWeightedAverageAggFuncEligible = isWeightedAverageAggFuncEligible;
-/**
- * Returns the Column's available aggregation function names ordered for display,
- * ensuring `weightedAvg` is positioned right after `avg` (and only present when
- * the Column is eligible for Weighted Average).
- */
 const getDisplayAggFuncNames = (column) => {
     const names = [...new Set(column.availableAggregationFunctions || [])].filter((name) => name !== exports.WEIGHTED_AVERAGE_AGG_FN_NAME);
     if ((0, exports.isWeightedAverageAggFuncEligible)(column)) {

package/src/AdaptableState/Common/Enums.js

@@ -1,5 +1,4 @@
 "use strict";
-// General Enums
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.SummaryOperation = exports.StatusColour = exports.FontSize = exports.FontStyle = exports.FontWeight = exports.SortOrder = exports.MathOperation = void 0;
 var MathOperation;
@@ -9,7 +8,6 @@ var MathOperation;
     MathOperation["Multiply"] = "Multiply";
     MathOperation["Divide"] = "Divide";
 })(MathOperation || (exports.MathOperation = MathOperation = {}));
-// Enums used in Strategies
 var SortOrder;
 (function (SortOrder) {
     SortOrder["Asc"] = "Asc";
@@ -27,7 +25,6 @@ var FontStyle;
 })(FontStyle || (exports.FontStyle = FontStyle = {}));
 var FontSize;
 (function (FontSize) {
-    //  Default = 'Default',
     FontSize["XSmall"] = "XSmall";
     FontSize["Small"] = "Small";
     FontSize["Medium"] = "Medium";

package/src/AdaptableState/Common/Menu.js

@@ -1,9 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ADAPTABLE_CONTEXT_MENU_ITEMS = exports.ADAPTABLE_COLUMN_MENU_ITEMS = void 0;
-/**
- * List of Shipped Adaptable Column Menu Items
- */
 exports.ADAPTABLE_COLUMN_MENU_ITEMS = [
     'calculated-column-edit',
     'cell-summary-show',
@@ -64,12 +61,9 @@ exports.ADAPTABLE_COLUMN_MENU_ITEMS = [
     'styled-column-icon-add',
     'styled-column-icon-edit',
     'system-status-show',
-    '_navbar', // used in SettingsPanel navigation bar and in Dashboard Module buttons
+    '_navbar',
     'separator',
 ];
-/**
- * List of Shipped Adaptable Context Menu Items
- */
 exports.ADAPTABLE_CONTEXT_MENU_ITEMS = [
     'menu-group',
     'alert-clear',

package/src/AdaptableState/Common/MenuItem.js

@@ -3,7 +3,6 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.MenuItemShowPopup = exports.MenuItemDoClickFunction = exports.MenuItemDoReduxAction = void 0;
 const tslib_1 = require("tslib");
 const PopupRedux = tslib_1.__importStar(require("../../Redux/ActionsReducers/PopupRedux"));
-// A menu item which performs a Redux Action when it is clicke
 class MenuItemDoReduxAction {
     constructor(name, label, module, reduxAction, icon, isVisible) {
         this.name = name;
@@ -42,7 +41,6 @@ class MenuItemDoClickFunction {
     icon;
 }
 exports.MenuItemDoClickFunction = MenuItemDoClickFunction;
-// A menu item which shows a popup screen when it is clieked
 class MenuItemShowPopup {
     constructor(name, label, module, componentName, icon, isVisible, popupParams) {
         this.name = name;

package/src/AdaptableState/Common/Types.js

@@ -1,7 +1,4 @@
 "use strict";
-/*
-This page includes frequently used typed - particularly helpful for State and Api where we can strongly type stuff
-*/
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ALL_MODULES = exports.ALL_TOOL_PANELS = exports.ALL_STATUS_SUB_PANELS = exports.ALL_TOOLBARS = void 0;
 const tslib_1 = require("tslib");
@@ -16,8 +13,8 @@ exports.ALL_TOOLBARS = [
     'Export',
     'IPushPull',
     'Layout',
-    'ColumnFilter', // controlled by layout module
-    'GridFilter', // controlled by layout module
+    'ColumnFilter',
+    'GridFilter',
     'OpenFin',
     'QuickSearch',
     'SmartEdit',

package/src/AdaptableState/StatusBarState.js

@@ -1,7 +1,4 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ADAPTABLE_STATUS_PANEL = void 0;
-/**
- * Name of the component specified in AG Grid gridOptions.statusBar.statusPanels
- */
 exports.ADAPTABLE_STATUS_PANEL = 'AdaptableStatusPanel';

package/src/AdaptableState/StyledColumns/BadgeStyle.js

@@ -1,7 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.BADGE_PILL_STYLE_KEYS = void 0;
-/** Keys copied from legacy `BadgeStyleDefinition.Style` (`AdaptableStyle`) into `PillStyle`. */
 exports.BADGE_PILL_STYLE_KEYS = [
     'BackColor',
     'ForeColor',

package/src/AdaptableState/StyledColumns/Common/BarStyleProperties.js

@@ -1,6 +1,2 @@
 "use strict";
-/**
- * Marker primitives shared by bar-chart-style Styled Columns —
- * Bullet Chart target markers and Range Bar value / reference markers.
- */
 Object.defineProperty(exports, "__esModule", { value: true });

package/src/AdaptableState/StyledColumns/Common/CellTextOptions.js

@@ -1,7 +1,2 @@
 "use strict";
-/**
- * Cell-text token primitives shared by several Styled Column types
- * (Percent Bar, Gradient, Bullet Chart, Range Bar, Rating) when configuring
- * what to display as text overlay / tooltip content.
- */
 Object.defineProperty(exports, "__esModule", { value: true });

package/src/AdaptableState/StyledColumns/Common/NumericStyledColumn.js

@@ -1,10 +1,2 @@
 "use strict";
-/**
- * Numeric scale primitives shared by every numeric Styled Column type
- * (Gradient, Percent Bar, Bullet Chart, Range Bar).
- *
- * These types describe how a column's numeric scale is resolved (via
- * literal numbers, dynamic column-wide aggregates, or comparison against
- * another column) and how that scale is segmented into coloured ranges.
- */
 Object.defineProperty(exports, "__esModule", { value: true });

package/src/AdaptableState/Uuid.js

@@ -2,6 +2,5 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.createUuid = void 0;
 const uuid_1 = require("uuid");
-// TODO: replace this redundancy with 'the createUuid()' from 'utils/uuid.ts'
 const createUuid = () => (0, uuid_1.v4)();
 exports.createUuid = createUuid;

package/src/Api/Implementation/AdaptableApiImpl.js

@@ -154,15 +154,11 @@ class AdaptableApiImpl {
         this.dataImportApi = new DataImportApiImpl_1.DataImportApiImpl(adaptable);
         this.filterApi = new FilterApiImpl_1.FilterApiImpl(adaptable);
         this.commentApi = new CommentsApiImpl_1.CommentsApiImpl(adaptable);
-        // adaptable internal api
         this.internalApi = new AdaptableInternalApi_1.AdaptableInternalApi(adaptable);
     }
     get agGridApi() {
         return this.adaptable?.agGridAdapter?.getAgGridApi();
     }
-    /**
-     * This is only meant to be called by the Adaptable instance.
-     */
     _internalDestroySelf() {
         if (this.destroyed) {
             return;

package/src/Api/Implementation/AlertApiImpl.js

@@ -47,29 +47,21 @@ class AlertApiImpl extends ApiBase_1.ApiBase {
         return this.getAlertDefinitionById(alertDefinition.Uuid);
     }
     async displayAdaptableAlert(alertToShow) {
-        // There are 3 things we always do with an alert:
-        // 1. Dispatch the Alert (so it appears in the toolbar)
         this.addUidToAdaptableObject(alertToShow);
         this.dispatchAction(InternalRedux.AdaptableAlertAdd(alertToShow, this.getAlertOptions().maxAlertsInStore));
-        // 2. Publish the Alert Fired Event
         this.getEventApi().internalApi.fireAlertFiredEvent(alertToShow);
-        // 3. Manage the many different possible Alert behaviours
-        // Note:  alertProperties.PreventEdit is handled separately, in the Adaptable.setupColumnValueSetter() function
         if (alertToShow.alertDefinition?.AlertProperties != undefined) {
             const alertDefinition = alertToShow.alertDefinition;
             const presentationProperties = alertDefinition.AlertProperties;
-            // Show a Toast Popup
             if (presentationProperties.DisplayNotification) {
                 this.displayAdaptableAlertNotification(alertToShow);
             }
-            // Log to console
             if (presentationProperties.LogToConsole) {
                 this.getAdatableLogger().consoleLogByMessageType(alertToShow.header + ': ' + alertToShow.message, alertDefinition.MessageType);
             }
             if (presentationProperties.DisplaySystemStatusMessage) {
                 this.getSystemStatusApi().setSystemStatus(alertToShow.header, alertDefinition.MessageType, alertToShow.message);
             }
-            // Show Alert in a Div (if one has been set)
             if (presentationProperties.ShowInDiv) {
                 const alertDiv = (0, resolveContainerElement_1.resolveContainerElement)(this.getContainerOptions().alertContainer, this.getAdaptableInternalApi().buildBaseContext());
                 if (alertDiv) {
@@ -81,13 +73,11 @@ class AlertApiImpl extends ApiBase_1.ApiBase {
                 return;
             }
             const alertProperties = alertDefinition.AlertProperties ?? {};
-            // Jump to the Cell
             if (alertProperties.JumpToCell &&
                 (0, AdaptableAlert_1.isAdaptableCellChangedAlert)(alertToShow) &&
                 alertToShow.cellDataChangedInfo) {
                 this.getGridApi().jumpToCell(alertToShow.cellDataChangedInfo.primaryKeyValue, alertToShow.cellDataChangedInfo.column.columnId, alertToShow.cellDataChangedInfo.rowNode);
             }
-            // Jump to the Row
             else if (alertProperties.JumpToRow &&
                 (0, AdaptableAlert_1.isAdaptableRowChangedAlert)(alertToShow) &&
                 alertToShow.rowDataChangedInfo?.rowTrigger === 'Add') {
@@ -96,8 +86,6 @@ class AlertApiImpl extends ApiBase_1.ApiBase {
                     this.getGridApi().jumpToRow(firstRowNode);
                 }
             }
-            // For CellChanged Alerts either Highlight the cell or row
-            // for former - we just refresh the cells to trigger a re-evaluation of the Adaptable.setupColumn[Style/Class]
             if ((0, AdaptableAlert_1.isAdaptableCellChangedAlert)(alertToShow) &&
                 alertToShow.cellDataChangedInfo &&
                 (alertProperties.HighlightCell || alertProperties.HighlightRow)) {
@@ -120,7 +108,6 @@ class AlertApiImpl extends ApiBase_1.ApiBase {
                     }
                 }
             }
-            // For RowChange Alerts only Highlight the row - we just refresh the rows to trigger a re-evaluation of the Adaptable.setupColumn[Style/Class]
             else if (alertProperties.HighlightRow &&
                 (0, AdaptableAlert_1.isAdaptableRowChangedAlert)(alertToShow) &&
                 alertToShow.rowDataChangedInfo?.rowTrigger === 'Add') {
@@ -185,8 +172,6 @@ class AlertApiImpl extends ApiBase_1.ApiBase {
             }
             if (this.getColumnScopeApi().scopeIsAll(alertDefinition.Scope) &&
                 !this.getExpressionApi().getAdaptableQueryExpression(alertDefinition.Rule)) {
-                // we don't support scope ALL without an expression
-                // the predicates for scope ALL do NOT make any sense for a programmatic evaluation
                 return false;
             }
             return true;
@@ -204,7 +189,6 @@ class AlertApiImpl extends ApiBase_1.ApiBase {
             const alertDefinitions = this.internalApi
                 .getAlertDefinitionsForCellDataChange(cellDataChangedInfo)
                 .filter((alertDefinition) => filterScopeAllDefinitions(alertDefinition))
-                // some alert definitions might NOT be relevant for the given locator
                 .filter((alertDefinition) => relevantAlertDefinitions.some((relevantAlertDefinition) => relevantAlertDefinition.Uuid === alertDefinition.Uuid));
             this.getAlertApi().internalApi.showAlertForDefinitions(cellDataChangedInfo, alertDefinitions);
         });
@@ -219,19 +203,15 @@ class AlertApiImpl extends ApiBase_1.ApiBase {
         if (!(0, ScheduledAlertHelper_1.isRuleBasedAlertDefinition)(alertDefinition)) {
             return [];
         }
-        //  if there is an Expression defined, extract only the columns from there (in that case the Scope would be 'All' anyway)
-        // columns referenced in rule expression
         const ruleExpression = this.getExpressionApi().getAdaptableQueryExpression(alertDefinition.Rule);
         if (this.getColumnScopeApi().scopeIsAll(alertDefinition.Scope) &&
             StringExtensions_1.default.IsNotNullOrEmpty(ruleExpression)) {
-            // return only first referenced column (otherwise we would generate multiple alerts for the same row)
             const queryColumns = this.getAdaptableInternalApi()
                 .getQueryLanguageService()
                 .getColumnsFromExpression(ruleExpression);
             return queryColumns.length > 0 ? [queryColumns[0]] : [];
         }
         else {
-            // otherwise return all columns for given scope
             return this.getColumnScopeApi()
                 .getColumnsInScope(alertDefinition.Scope)
                 .map((adaptableColumn) => adaptableColumn.columnId);
@@ -239,7 +219,6 @@ class AlertApiImpl extends ApiBase_1.ApiBase {
     }
     createCellDataChangeInfoStubs(relevantColumnIds) {
         const cellDataChangeInfos = [];
-        //const adaptable: IAdaptable = this.getAdaptableInternalApi().getAdaptableInstance();
         const relevantColumns = this.getColumnApi()
             .getColumnsWithColumnIds(relevantColumnIds)
             .filter(Boolean);

package/src/Api/Implementation/ApiBase.js

@@ -4,13 +4,8 @@ exports.ApiBase = void 0;
 const tslib_1 = require("tslib");
 const Helper_1 = tslib_1.__importDefault(require("../../Utilities/Helpers/Helper"));
 const Uuid_1 = require("../../AdaptableState/Uuid");
-// Base class for the Api - provides checking dispatching methods
 class ApiBase {
     _adaptable;
-    /**
-     * Constructor for all the api classes which simply takes and assigns an instance of Adaptable
-     * @param _adaptable core IAdaptable object - for INTERNAL USE ONLY and not part of public API
-     */
     constructor(_adaptable) {
         this._adaptable = _adaptable;
     }
@@ -35,14 +30,6 @@ class ApiBase {
     isAdapTableReady() {
         return this._adaptable.isReady;
     }
-    /**
-     * Base api helper method which ensure that Adaptable Object being used in the function is not null or undefined
-     *
-     * If it does not exist then we log an error and the api method should stop
-     * @param item AdaptableObject being checked
-     * @param name the name of the object if it has one (e.g. if its a search)
-     * @param type the actual type of the object being checked
-     */
     checkItemExists(item, name, type) {
         if (Helper_1.default.objectNotExists(item)) {
             this.logError(`No ${type} found with the name: ${name}`);
@@ -66,29 +53,18 @@ class ApiBase {
             : this.getLayoutApi().getCurrentLayout();
         return objectList.filter((objectItem) => this.getLayoutApi().internalApi.isObjectExtendedInLayout(objectItem, module, extendedLayout));
     }
-    /**
-     * Base api Helper method that dispatches a *Redux Action* to the Store
-     * @param action the Redux Action to be dispatched
-     */
     dispatchAction(action) {
         if (this._adaptable.isDestroyed) {
             return;
         }
         this._adaptable.adaptableStore.TheStore.dispatch(action);
     }
-    /**
-     * Returns the entire State from the Store
-     *
-     * This is a simple *getState()* call
-     */
     getAdaptableState() {
         return this.getAdaptableStore().TheStore.getState();
     }
     getAdaptableStore() {
         return this._adaptable.adaptableStore;
     }
-    // Add the Uuid to the object if not present;
-    // It's generally done in Redux also but no harm checking in Api also in case we missed it.
     addUidToAdaptableObject(adaptableObject) {
         if (!adaptableObject.Uuid) {
             adaptableObject.Uuid = (0, Uuid_1.createUuid)();

package/src/Api/Implementation/CalculatedColumnApiImpl.js

@@ -52,7 +52,6 @@ class CalculatedColumnApiImpl extends ApiBase_1.ApiBase {
                     .getCalculatedColumnExpressionService()
                     .createAggregatedScalarLiveValue(calculatedColumn);
             });
-            // refresh calculated columns
             const aggCalculatedColumnIds = aggregatedCalculatedColumns.map((calculatedColumn) => calculatedColumn.ColumnId);
             if (this.getLayoutApi().internalApi.isLayoutGrouped()) {
                 this.getGridApi().refreshGroupRowNodes();

package/src/Api/Implementation/ChartingApiImpl.js

@@ -10,9 +10,6 @@ const ChartingInternalApi_1 = require("../Internal/ChartingInternalApi");
 const ObjectFactory_1 = tslib_1.__importDefault(require("../../Utilities/ObjectFactory"));
 const ChartHelper_1 = require("../../Utilities/Helpers/ChartHelper");
 class ChartingApiImpl extends ApiBase_1.ApiBase {
-    /**
-     * @internal
-     */
     internalApi;
     constructor(_adaptable) {
         super(_adaptable);