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

package/src/Utilities/Services/AnnotationsService.js

@@ -1,9 +1,5 @@
 import * as InternalRedux from '../../Redux/ActionsReducers/InternalRedux';
 import ArrayExtensions from '../Extensions/ArrayExtensions';
-/**
- * This service controls the interaction between Notes & Comments
- * If there are active Notes or Comments it listens to Cell Selection and Mouse Enter events
- */
 export class AnnotationsService {
     api;
     adaptable;
@@ -28,7 +24,6 @@ export class AnnotationsService {
         if (a?.ColumnId === b?.ColumnId && a?.PrimaryKeyValue === b?.PrimaryKeyValue) {
             return true;
         }
-        // Primary keys retrieved from the grid dom are always strings, so we must also consider them strings
         if ((typeof a.PrimaryKeyValue === 'number' && typeof b.PrimaryKeyValue === 'string') ||
             (typeof b.PrimaryKeyValue === 'string' && typeof a.PrimaryKeyValue === 'number')) {
             return (a.PrimaryKeyValue.toString() === b.PrimaryKeyValue.toString() && a.ColumnId === b.ColumnId);
@@ -42,7 +37,6 @@ export class AnnotationsService {
         }
     }
     shouldListenToEvents() {
-        // Listen to events if we either have Notes in state or if Comments have been set up
         return (ArrayExtensions.IsNotNullOrEmpty(this.api.noteApi.getAllNotes()) ||
             this.api.internalApi.getModuleService().isAdapTableModulePresent('Comment'));
     }
@@ -81,28 +75,19 @@ export class AnnotationsService {
     handleMouseEnter(event) {
         const editMode = this.getEditMode();
         if (editMode) {
-            // ignore
             return;
         }
         const cellPosition = this.getCellPositionFromEvent(event);
         const openCellAddress = this.getOpenCellAddress();
-        // Hovering over something that is not a cell and there is no open note
         if (!cellPosition && openCellAddress) {
-            // need to close if the new cell does not have a note
             this.hidePopup();
             return;
         }
-        // call only if cell address is different
         if (AnnotationsService.isSameAddress(openCellAddress, cellPosition)) {
             return;
         }
         const cellNote = this.api.noteApi.getNoteForCell(cellPosition);
         const cellComments = this.api.commentApi.getCommentThreadForCell(cellPosition);
-        // Notes / Comments only open on hover when explicitly opted in via
-        // `noteOptions.openNote = 'hover'` / `commentOptions.showComment =
-        // 'hover'` (the defaults). When either is set to 'menu' the popup is
-        // only opened via the Cell Menu, so we ignore that side of the
-        // annotation when deciding whether to show the popup here.
         const openNoteMode = this.api.optionsApi.getNoteOptions()?.showNoteAction ?? 'hover';
         const showCommentMode = this.api.optionsApi.getCommentOptions()?.showCommentAction ?? 'hover';
         const shouldOpenForNote = !!cellNote && openNoteMode === 'hover';
@@ -113,26 +98,19 @@ export class AnnotationsService {
         this.showPopup(cellPosition, false);
     }
     handleCellSelected(cellAddress) {
-        // if already opened - do nothing
         const openCellAddress = this.getOpenCellAddress();
         if (AnnotationsService.isSameAddress(openCellAddress, cellAddress)) {
             return;
         }
-        // if open but this has no note, close
         const cellNote = this.api.noteApi.getNoteForCell(cellAddress);
         const cellComments = this.api.commentApi.getCommentThreadForCell(cellAddress);
         const hasNotesOrComments = cellNote || cellComments;
         if (openCellAddress && !hasNotesOrComments) {
-            // hide only if in edit mode
-            // because if you select a cell and move the mouse fast over another cell
-            // with a note, you do not want to close that note
-            // the selection is debounced, so the moment goes over the new cell this event is triggered
             if (this.getEditMode()) {
                 this.hidePopup();
             }
             return;
         }
-        // if open but the new selection has a note open that one in edit mode (because of click)
         if (openCellAddress && hasNotesOrComments) {
             this.showPopup(cellAddress, true);
             return;

package/src/Utilities/Services/CalculatedColumnExpressionService.js

@@ -47,9 +47,6 @@ export class CalculatedColumnExpressionService {
     getCalculatedColumnDataType(calculatedColumnQuery) {
         try {
             if (calculatedColumnQuery.AggregatedScalarExpression) {
-                // currently AggregatedScalarExpression support only numerical values
-                // TODO AFL try to derive the type from the aggregatedColumn?
-                // we definitely don't want to aggregate everything only to get the data type
                 return 'number';
             }
             let firstRowNode = this.adaptableApi.gridApi.getFirstRowNode();
@@ -80,7 +77,6 @@ export class CalculatedColumnExpressionService {
     isCalculatedColumnQueryValid(calculatedColumnQuery) {
         if (calculatedColumnQuery.ScalarExpression) {
             try {
-                // TODO AFL add a cached validation for ScalarExpression, just like for all the other types
                 let firstRowNode = this.adaptableApi.gridApi.getFirstRowNode();
                 this.adaptableApi.internalApi
                     .getQueryLanguageService()
@@ -88,7 +84,6 @@ export class CalculatedColumnExpressionService {
                 return true;
             }
             catch (e) {
-                // no logging as this method is used only in the UI
                 return false;
             }
         }
@@ -96,7 +91,6 @@ export class CalculatedColumnExpressionService {
             const validationResult = this.adaptableApi.expressionApi.isValidAggregatedScalarExpression(calculatedColumnQuery.AggregatedScalarExpression, CalculatedColumnModuleId);
             return validationResult;
         }
-        // if query has neither a ScalarExpression nor an AggregatedScalarExpression => it can only be false
         return false;
     }
     evaluateCalculatedColumnQuery(calculatedColumn, rowNode) {
@@ -104,7 +98,6 @@ export class CalculatedColumnExpressionService {
             if (this.adaptableApi.gridApi.isGroupRowNode(rowNode)) {
                 return undefined;
             }
-            // no validation here, this function has to be as performant as possible
             if (calculatedColumn?.Query?.ScalarExpression) {
                 return this.adaptableApi.internalApi
                     .getQueryLanguageService()
@@ -122,7 +115,6 @@ export class CalculatedColumnExpressionService {
         }
     }
     createAggregatedScalarLiveValue(calculatedColumn) {
-        // if there is already an aggregated scalar, delete it (possible when editing)
         this.destroyAggregatedScalarLiveValue(calculatedColumn);
         if (calculatedColumn.Query?.AggregatedScalarExpression) {
             try {

package/src/Utilities/Services/ChartingService.js

@@ -13,7 +13,6 @@ export class ChartingService {
             case 'chartCreated':
                 const createParams = params;
                 if (allChartDefinitions.some((chartDefinition) => chartDefinition.Model?.chartId === createParams.chartId)) {
-                    // chart already exists
                 }
                 else {
                     const chartModel = models.find((model) => model?.chartId === createParams.chartId);
@@ -23,7 +22,6 @@ export class ChartingService {
             case 'chartOptionsChanged':
             case 'chartRangeSelectionChanged':
                 if (!currentChartDefinition) {
-                    // editing a non-saved chart
                     break;
                 }
                 const updatedChartDefinition = {
@@ -36,13 +34,6 @@ export class ChartingService {
         this.api.internalApi.dispatchReduxAction(InternalRedux.ChartingSetCurrentChartModels(models));
         this.emitAdaptableChartDefinitonChange();
     }
-    /**
-     * Emit events only for adaptable charting definition changes
-     * - an adaptable chart opens
-     * - an adaptable chart closes
-     * - an adaptable chart is created
-     * - an adaptable chart is deleted
-     */
     emitAdaptableChartDefinitonChange() {
         const chartingOpenState = this.api.chartingApi.getChartingOpenState();
         const chartChangedInfo = {

package/src/Utilities/Services/DataService.js

@@ -1,7 +1,6 @@
 import Emitter from '../../Utilities/Emitter';
 import { Subject } from 'rxjs';
 import Helper from '../Helpers/Helper';
-// ~ 8 hours
 const MAX_TIMEFRAME_SIZE = 86400000;
 export class DataService {
     adaptable;
@@ -58,16 +57,10 @@ export class DataService {
         this.adaptable.api.eventApi.internalApi.fireRowChangedEvent(rowDataChangedInfo);
         this.rowDataChangeLogSubject$.next(rowDataChangedInfo);
     }
-    // we need this temporary "shared memory" because the value change and the AG Grid cellChanged event are asynchronous
-    // in the first phase we keep the undone change
     logUndoChange(change) {
         const { primaryKeyValue, column, oldValue, newValue } = change;
         const undoChangeKey = this.getUndoChangeKey(primaryKeyValue, column.columnId, oldValue, newValue);
         this.undoChangeLog.set(undoChangeKey, change);
-        // normally the extractUndoChange() should be executed right after this method
-        // just to be sure that no 'zombie' undo change remains here (if something goes wrong with the undo value change)
-        // we manually extract the change after a 2 seconds
-        // why 2 and not 3 or 1? no good reason, only seems like a reasonable waiting time :)
         const UNDO_WAIT = 2000;
         const timeoutId = setTimeout(() => {
             this.adaptable.logger.warn(`Undo change was not handled within timeout: column="${change.column}", primaryKey="${change.primaryKeyValue}", from=${change.newValue} to=${change.oldValue}.`);
@@ -75,10 +68,8 @@ export class DataService {
         }, UNDO_WAIT);
         this.undoChangeTimers.set(undoChangeKey, timeoutId);
     }
-    // in the second phase, when AG-Grid triggers the cellChange event, we check if it corresponds to an undone change
     extractUndoChange(change) {
         const { primaryKeyValue, column, oldValue, newValue } = change;
-        // this is post-undo, so we have to swap the new & old values
         const undoChangeKey = this.getUndoChangeKey(primaryKeyValue, column.columnId, newValue, oldValue);
         const result = this.undoChangeLog.get(undoChangeKey);
         this.undoChangeLog.delete(undoChangeKey);
@@ -97,11 +88,9 @@ export class DataService {
         });
     }
     extractDataChangeLogEntry(cellDataChangedInfo) {
-        // create rowData snapshot
         const rowData = cellDataChangedInfo.rowData
             ? Helper.cloneObject(cellDataChangedInfo.rowData)
             : null;
-        // strip down rowNode properties as it is pretty "heavy" on the memory
         const rowNode = rowData ? { data: rowData } : null;
         return {
             ...cellDataChangedInfo,

package/src/Utilities/Services/Fdc3Service.js

@@ -8,7 +8,6 @@ export class Fdc3Service {
     uiControlsDefaultConfiguration;
     constructor(adaptableApi) {
         this.adaptableApi = adaptableApi;
-        // 1. Subscribe to FDC3 events
         const adaptableHandleIntentFn = this.getFdc3Options().intents?.handleIntent;
         const listenForIntents = this.getFdc3Options().intents?.listensFor ?? [];
         if (listenForIntents.length) {
@@ -69,7 +68,6 @@ export class Fdc3Service {
                     .then((listener) => this.contextHandlerSubscriptions.push(listener));
             });
         }
-        // 2. Init UI Controls default configuration
         this.uiControlsDefaultConfiguration = this.buildUiControlsDefaultConfiguration();
     }
     getUiControlsDefaultConfiguration() {
@@ -190,7 +188,6 @@ export class Fdc3Service {
             };
             return intentsMapping;
         }, {});
-        // merge adaptable defaults with (optional) user provided configuration
         return {
             contexts: {
                 ...adaptableDefaultContextIcons,

package/src/Utilities/Services/FlashingCellService.js

@@ -1,12 +1,7 @@
 export const FLASHING_CELL_ROW_KEY = '__ROW';
-/**
- * This service manages Flashing Cells
- */
 export class FlashingCellService {
     api;
-    // map of rowPrimaryKey -> ColIdOrRowId -> FlashingCellUuid
     gridCellsCurrentlyFlashing;
-    // map of FlashingCellUuid -> FlashingCell
     flashingCellsMapping;
     constructor(api) {
         this.api = api;
@@ -37,7 +32,6 @@ export class FlashingCellService {
         if (flashingCell.flashTarget === 'row' || flashingCell.flashTarget.includes('row')) {
             columnIds.push(FLASHING_CELL_ROW_KEY);
         }
-        // for high frequency rates, we might have multiple flashes for the same cell
         if (this.gridCellsCurrentlyFlashing[rowPrimaryKey]) {
             columnIds.forEach((colId) => {
                 if (this.gridCellsCurrentlyFlashing[rowPrimaryKey][colId] === flashingCell.Uuid) {

package/src/Utilities/Services/ModuleService.js

@@ -21,11 +21,8 @@ export class ModuleService {
         return module.isModuleEditable();
     }
     logMissingAgGridDepsInfos() {
-        // log missing core (required) AG Grid dependencies
         const agGridModulesAdapter = this.adaptableApi.internalApi.getAgGridModulesAdapter();
         if (agGridModulesAdapter.isAgGridModuleRegistered('AllEnterpriseModule')) {
-            // no need to check further if AllEnterprise is registered
-            // it may even trigger false positives (e.g. when using SSRM)
             return;
         }
         const mandatoryAgGridModuleNames = agGridModulesAdapter.getMandatoryAgGridModuleNames();
@@ -34,7 +31,6 @@ export class ModuleService {
         if (missingAgGridModuleNames.length) {
             this.adaptableApi.consoleError(`Adaptable requires these AG Grid modules: ${missingAgGridModuleNames.join(', ')}. Please register them. See: ${AgGridRequiredModulesDocsLink}`);
         }
-        // log optional missing AG Grid dependencies for all modules
         this.getModuleCollection().forEach((adaptableModule) => {
             adaptableModule.logMissingAgGridDepsInfos();
         });
@@ -56,7 +52,6 @@ export class ModuleService {
                 }
             }
         });
-        // store Settings Panel items in Internal State as they never change and we often re-use
         this.adaptableApi.internalApi.setSettingsPanelItems(settingsPanelItems);
     }
     createModuleButtonsForDashboardAndToolPanel() {
@@ -69,7 +64,6 @@ export class ModuleService {
                 }
             }
         });
-        // store Dashboard Buttons in Internal State as they never change l
         this.adaptableApi.internalApi.setDashboardModuleButtons(moduleButtons);
     }
     getTeamSharingAction(adaptableModule) {
@@ -80,7 +74,6 @@ export class ModuleService {
         return module.getTeamSharingAction();
     }
     getModuleById(adaptableModule) {
-        // @ts-ignore TODO: make modules generic
         return this.getModuleCollection().get(adaptableModule);
     }
     getModuleInfoByModule(adaptableModule) {
@@ -184,6 +177,5 @@ export class ModuleService {
         }
     }
     destroy() {
-        // TO DO
     }
 }

package/src/Utilities/Services/QueryLanguageService.js

@@ -20,7 +20,6 @@ export class QueryLanguageService {
     }
     evaluateBooleanExpression(expression, module, rowNode, evalContext) {
         if (expression == undefined) {
-            // should never happen, but just in case
             this.adaptableApi.logError('QueryLanguageService.evaluateBooleanExpression was called with an undefined expression');
             return false;
         }
@@ -37,13 +36,12 @@ export class QueryLanguageService {
         });
     }
     evaluateScalarExpression(expression, module, rowNode) {
-        // currently scalar and boolean expressions are evaluated the same
         return this.evaluateBooleanExpression(expression, module, rowNode);
     }
     evaluateAggregatedScalarExpression(expression, module, getRowNodes) {
         const aggregatedScalarFunctions = this.getModuleExpressionFunctionsMap(module).aggregatedScalarFunctions;
         return parser.evaluate(expression, {
-            node: null, // no node as this is an aggregation query
+            node: null,
             functions: aggregatedScalarFunctions,
             evaluateCustomQueryVariable: this.evaluateCustomQueryVariable,
             getRowNodes,
@@ -90,7 +88,6 @@ export class QueryLanguageService {
                 return result;
             }
             const force = config?.force ?? false;
-            // see if validation should be performed
             if (!this.adaptableApi.optionsApi.getExpressionOptions().performExpressionValidation &&
                 !force) {
                 const result = { isValid: true, errorMessage: '' };
@@ -192,7 +189,6 @@ export class QueryLanguageService {
                 this.cacheAggregatedBooleanValidation.set(cacheKey, result);
                 return result;
             }
-            // no exception,so everything seems to be fine
             const result = { isValid: true, errorMessage: '' };
             this.cacheAggregatedBooleanValidation.set(cacheKey, result);
             return result;
@@ -230,7 +226,6 @@ export class QueryLanguageService {
                 this.cacheAggregatedScalarValidation.set(cacheKey, result);
                 return result;
             }
-            // no exception,so everything seems to be fine
             const result = { isValid: true, errorMessage: '' };
             this.cacheAggregatedScalarValidation.set(cacheKey, result);
             return result;
@@ -268,12 +263,9 @@ export class QueryLanguageService {
             const columnFriendlyName = this.adaptableApi.columnApi.getFriendlyNameForColumnId(columnId);
             result = result.split(columnId).join(columnFriendlyName);
         });
-        // replace $SCOPE() with friendly name as well
         result = result.split(`$SCOPE()`).join(`$Column_Scope`);
         return result;
     }
-    // Returns the ExpressionFunctions available for the given Module as specified in the `QueryLanguageOptions.moduleExpressionFunctions`
-    // if there are no specific functions defined, it falls back to the default values
     getModuleExpressionFunctionsMap(module) {
         const expressionOptions = this.adaptableApi.optionsApi.getExpressionOptions();
         if (module) {
@@ -285,10 +277,7 @@ export class QueryLanguageService {
         const generalBooleanExpressionFunctions = this.extractMappedExpressionFunctions(booleanExpressionFunctions, expressionOptions.systemBooleanFunctions, expressionOptions.customBooleanFunctions);
         const generalScalarExpressionFunctions = this.extractMappedExpressionFunctions(scalarExpressionFunctions, expressionOptions.systemScalarFunctions, expressionOptions.customScalarFunctions);
         const generalObservableExpressionFunctions = this.extractMappedExpressionFunctions(observableExpressionFunctions, expressionOptions.systemObservableFunctions);
-        const generalAggregatedBooleanExpressionFunctions = this.extractMappedExpressionFunctions(aggregatedBooleanExpressionFunctions, expressionOptions.systemAggregatedBooleanFunctions, 
-        // right now the custom aggregated scalar functions are used in the aggregated boolean functions as well
-        // hopefully we'll simplify this in the future
-        this.getCustomAggregatedScalarFunctions());
+        const generalAggregatedBooleanExpressionFunctions = this.extractMappedExpressionFunctions(aggregatedBooleanExpressionFunctions, expressionOptions.systemAggregatedBooleanFunctions, this.getCustomAggregatedScalarFunctions());
         const generalAggregatedScalarExpressionFunctions = this.extractMappedExpressionFunctions(aggregatedScalarExpressionFunctions, expressionOptions.systemAggregatedScalarFunctions, this.getCustomAggregatedScalarFunctions());
         if (!module) {
             this.adaptableApi.logWarn(`QueryLanguageService.getModuleExpressionFunctions() was called with an undefined 'module' param, this should never happen`);
@@ -354,11 +343,9 @@ export class QueryLanguageService {
                     const groupByColumnIds = groupByParameter?.value;
                     const contextArgs = args.filter((arg) => {
                         if (typeof arg === 'object' && arg.name === 'COL') {
-                            // filter out aggregation column
                             return arg.value !== aggColumnId;
                         }
                         if (typeof arg === 'object' && arg.name === 'GROUP_BY') {
-                            // filter out groupBy column
                             return false;
                         }
                         return true;
@@ -377,7 +364,6 @@ export class QueryLanguageService {
                                             index: dataIndex,
                                             rowNode,
                                             args: contextArgs.map((arg) => {
-                                                // if col, replace with value
                                                 if (typeof arg === 'object' && arg.name === 'COL') {
                                                     return getValueForColId(arg.value, rowNode);
                                                 }
@@ -414,7 +400,6 @@ export class QueryLanguageService {
                                     aggregatedValue: aggregationValue,
                                     ...this.adaptableApi.internalApi.buildBaseContext(),
                                     args: contextArgs.map((arg) => {
-                                        // if col, replace with value
                                         if (typeof arg === 'object' && arg.name === 'COL') {
                                             return getValueForColId(arg.value, rowNode);
                                         }
@@ -434,7 +419,6 @@ export class QueryLanguageService {
                                     rowNode,
                                     ...this.adaptableApi.internalApi.buildBaseContext(),
                                     args: contextArgs.map((arg) => {
-                                        // if col, replace with value
                                         if (typeof arg === 'object' && arg.name === 'COL') {
                                             return getValueForColId(arg.value, rowNode);
                                         }
@@ -473,16 +457,13 @@ export class QueryLanguageService {
             })
             : systemFunctions;
         let generalExpressionFunctions = {};
-        // add system functions
         if (Array.isArray(systemFunctionNames)) {
-            // add only system functions specified by user
             systemFunctionNames.forEach((systemFunctionName) => {
                 generalExpressionFunctions[systemFunctionName] =
                     availableExpressionFunctions[systemFunctionName];
             });
         }
         else {
-            // add ALL system functions
             generalExpressionFunctions = { ...availableExpressionFunctions };
         }
         const customFunctionDefinitions = typeof customFunctions === 'function'
@@ -518,7 +499,6 @@ export class QueryLanguageService {
     getNodesFromExpression(input, nodeType) {
         try {
             const resultSet = new Set();
-            // @ts-ignore
             const walk = (node) => {
                 if (typeof node !== 'object') {
                     return false;
@@ -547,9 +527,6 @@ export class QueryLanguageService {
         if (!queryNodes.length) {
             return;
         }
-        // circular QUERY references can only be detected by actually evaluating the expression
-        // we just evaluate against the first row, as we don't care about the result, just about
-        // whether an exception is thrown
         const firstRowNode = this.adaptableApi.gridApi.getFirstRowNode();
         queryNodes.forEach((queryNode) => {
             this.evaluateBooleanExpression(`QUERY("${queryNode}")`, module, firstRowNode);

package/src/Utilities/Services/RowFormService.js

@@ -41,6 +41,5 @@ export class RowFormService {
         }
     }
     destroy() {
-        // nothing yet
     }
 }

package/src/Utilities/Services/RowSummaryService.js

@@ -6,9 +6,6 @@ import { AggregatedScalarLiveValue } from './AggregatedScalarLiveValue';
 import { isObjectEmpty } from '../Extensions/ObjectExtensions';
 import { throttle } from '../Helpers/TimingHelper';
 import { isEqual } from '../Extensions/ObjectExtensions';
-/**
- * The logic is extracted here to make it easier to follow
- */
 export class RowSummaryService {
     api;
     cachedCellSummary = new Map();
@@ -21,11 +18,9 @@ export class RowSummaryService {
         this.rowSummariesSubscriptions();
     }
     rowSummariesSubscriptions() {
-        // return;
         if (this.api.isDestroyed()) {
             return;
         }
-        // Currently not available for serverside model
         if (!this.api.layoutApi.internalApi.getLayoutSupportedFeatures().RowSummaries) {
             return;
         }
@@ -45,18 +40,15 @@ export class RowSummaryService {
             });
         });
         this.api.eventApi.on('LayoutChanged', (event) => {
-            // exclude filter events, those are handled in another event
             if (event.actionName.includes('FILTER')) {
                 return;
             }
             setTimeout(() => {
-                // the timeout is added so the grid has time to repond to the layout changed
                 this.throttledEvaluateRowSummary();
             }, 16);
         });
         const adaptable = this.api.internalApi.getAdaptableInstance();
         adaptable._on('AdapTableFiltersApplied', () => {
-            // we need to use this instead of layout changed so the rows have time to update
             this.throttledEvaluateRowSummary();
         });
         adaptable._on('FirstDataRendered', () => {
@@ -70,10 +62,6 @@ export class RowSummaryService {
         }
         this._throttledEvaluateRowSummary(reason);
     }
-    /**
-     *
-     * @param colId optional to evaluate only one column
-     */
     _throttledEvaluateRowSummary = throttle(this.evaluateRowSummary, 300);
     evaluateRowSummary(reason) {
         if (this._throttleAcumulatedColumnsThatChanged.size > 0) {
@@ -88,11 +76,7 @@ export class RowSummaryService {
         }
         const currentLayout = this.api.layoutApi.getCurrentLayout();
         let previousLayout = this.previousLayout;
-        // it is added here to be sure it is saved
         this.previousLayout = currentLayout;
-        /**
-         * If the previous & current layout does not have row summaries, it is safe to exit
-         */
         if (isObjectEmpty(currentLayout?.RowSummaries) && isObjectEmpty(previousLayout?.RowSummaries)) {
             return;
         }
@@ -104,9 +88,7 @@ export class RowSummaryService {
         const rowSummariesResults = rowSummaries
             .filter((rowSummary) => !rowSummary.IsSuspended)
             .map((rowSummary, index) => {
-            const { ColumnsMap, Position, 
-            // it defaults to true
-            IncludeOnlyFilteredRows = true, } = rowSummary;
+            const { ColumnsMap, Position, IncludeOnlyFilteredRows = true, } = rowSummary;
             return {
                 Position,
                 RowData: Object.entries(ColumnsMap ?? {}).reduce((acc, [columnId, expression]) => {
@@ -117,7 +99,6 @@ export class RowSummaryService {
                     let expressionLiveValue = this.cachedCellSummary.get(key);
                     if (expressionLiveValue) {
                         if (!reason) {
-                            // refresh all of them
                             expressionLiveValue.refresh();
                         }
                         else if ('columnIds' in reason && reason.columnIds.includes(columnId)) {

package/src/Utilities/Services/TeamSharingService.js

@@ -8,7 +8,6 @@ export class TeamSharingService {
         this.adaptableApi = adaptableApi;
         const teamSharingOptions = adaptableApi.optionsApi.getTeamSharingOptions();
         if (teamSharingOptions.updateInterval > 0) {
-            // convert minutes to millis
             const updateInterval = teamSharingOptions.updateInterval * 60000;
             this.updateCheckTimerId = setInterval(() => {
                 this.adaptableApi.teamSharingApi.checkForUpdates();
@@ -44,7 +43,6 @@ export class TeamSharingService {
     updateActiveSharedEntity(changedAdaptableObject, userName, sharedEntities) {
         let activeSharedEntity = sharedEntities.find((sharedEntity) => sharedEntity.Type === 'Active' && sharedEntity.Entity.Uuid === changedAdaptableObject.Uuid);
         if (!activeSharedEntity) {
-            // shared entity may have been removed in the meantime
             return [sharedEntities, []];
         }
         activeSharedEntity = {
@@ -54,16 +52,13 @@ export class TeamSharingService {
             ChangedBy: userName,
             Revision: activeSharedEntity.Revision + 1,
         };
-        // update EntityDependencyIds (references may have been removed or new ones added)
         const updatedEntityDependencyIds = [];
         const currentEntityDependencies = this.getSharedEntityDependencies(activeSharedEntity, sharedEntities);
         const newTeamSharingReferences = this.getTeamSharingReferences(changedAdaptableObject, activeSharedEntity.Module);
         const newReferenceAdaptableObjectIds = newTeamSharingReferences.map((reference) => reference.Reference.Uuid);
-        // 1. add all the existing dependencies which are still referenced
         updatedEntityDependencyIds.push(...currentEntityDependencies
             .filter((entityDependency) => newReferenceAdaptableObjectIds.includes(entityDependency.Entity.Uuid))
             .map((entity) => entity.Uuid));
-        // 2. check if there are new references which are not yet shared
         const freshReferences = newTeamSharingReferences.filter((reference) => !currentEntityDependencies
             .map((entityDependency) => entityDependency.Entity.Uuid)
             .includes(reference.Reference.Uuid));
@@ -83,7 +78,6 @@ export class TeamSharingService {
     }
     buildSharedEntityImportActions(importedSharedEntity) {
         let configOverwriteNeedsConfirmation = false;
-        // either ADD or EDIT(in which case the user will need to confirm the overwrite)
         const getSharedEntityImportAction = (sharedEntity) => {
             const { Module, Entity } = sharedEntity;
             const importInfo = this.adaptableApi.internalApi
@@ -209,7 +203,6 @@ export class TeamSharingService {
         clearTimeout(this.updateCheckTimerId);
     }
     createSharedEntity(adaptableObject, module, configuration, sharingUserName, sharingTimestamp, createdSharedEntities) {
-        // create main shared entity
         const mainSharedEntity = {
             EntityType: 'adaptableEntity',
             Uuid: createUuid(),
@@ -224,29 +217,21 @@ export class TeamSharingService {
             Revision: 1,
             Type: configuration.type,
         };
-        // load all dependencies (special cols, shared queries etc)
         const teamSharingDependencies = this.getTeamSharingReferences(adaptableObject, module);
-        // for every dependency, create recursively the corresponding shared entities
         teamSharingDependencies.forEach((teamSharingDependency) => {
             const sharedEntityDependency = this.createSharedEntity(teamSharingDependency.Reference, teamSharingDependency.Module, configuration, sharingUserName, sharingTimestamp, createdSharedEntities);
-            // update dependency IDs for the main shared entity
             mainSharedEntity.EntityDependencyIds.push(sharedEntityDependency.Uuid);
         });
-        // add main shared entity to the accumulator
         createdSharedEntities.push(mainSharedEntity);
         return mainSharedEntity;
     }
-    // returns the given rootSharedEntity with all its unique dependencies in their dependency order
     getSharedEntityDependencyTree(rootSharedEntity, allSharedEntities) {
-        // using a map to handle the case where a SharedEntity occurs multiple times in the dependency tree
         const result = new Map();
         result.set(rootSharedEntity.Uuid, rootSharedEntity);
         rootSharedEntity.EntityDependencyIds.forEach((dependencyId) => {
             const dependencySharedEntity = allSharedEntities.find((sharedEntity) => sharedEntity.Uuid === dependencyId);
-            // check for zombie dependencies (possible if state is inconsistent due to dirty reads)
             if (dependencySharedEntity) {
                 this.getSharedEntityDependencyTree(dependencySharedEntity, allSharedEntities).forEach((childDependency) => {
-                    // check if this dependency is not already present in the dependency tree
                     if (!result.has(childDependency.Uuid)) {
                         result.set(childDependency.Uuid, childDependency);
                     }

package/src/Utilities/Services/ThemeService.js

@@ -7,7 +7,6 @@ export class ThemeService {
         this.subscribe();
         if (!this.styleSheetObject) {
             this.styleSheetObject = new CSSStyleSheet();
-            // ts does not know about adoptedStyleSheets
             document.adoptedStyleSheets = [
                 ...(document.adoptedStyleSheets ?? []),
                 this.styleSheetObject,
@@ -31,8 +30,6 @@ export class ThemeService {
         let currentTheme = this.api.themeApi.getCurrentThemeObject();
         currentTheme = this.mapOsTheme(currentTheme);
         this.applyNewThemeVariables(currentTheme);
-        // this needs to be called after variables are set
-        // as it may show the warning for a custom/runtime theme
         this.showMissingThemeFiles(currentTheme);
     };
     applyNewThemeVariables(theme) {
@@ -52,8 +49,6 @@ export class ThemeService {
         this.styleSheetObject.replaceSync(str);
     }
     showMissingThemeFiles(theme) {
-        // run time defined theme
-        // because it may be an empty theme
         if (theme.Source === 'User') {
             return;
         }
@@ -77,7 +72,6 @@ export class ThemeService {
             logger.consoleWarn(`Theme "${themeName}" does not appear to be loaded. Ensure the CSS file is imported, e.g.: import "@adaptabletools/adaptable/themes/${themeName}.css".`);
         }
     }
-    // prefers-color-scheme
     getDOMPrefferedColorScheme() {
         return window.matchMedia('(prefers-color-scheme: dark)').matches ? 'dark' : 'light';
     }
@@ -91,7 +85,6 @@ export class ThemeService {
     }
     attachPrefferedColorSchemeListener() {
         const handlePrefferedColorSchemeChange = () => {
-            // we need to reapply the theme
             const currentTheme = this.api.themeApi.getCurrentThemeObject();
             if (currentTheme.Name === 'os') {
                 this.api.themeApi.applyCurrentTheme();