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

package/src/AdaptableState/Common/AdaptableForm.js

@@ -11,18 +11,9 @@ exports.isAdaptableFormFieldHidden = isAdaptableFormFieldHidden;
 exports.isAdaptableFormFieldGroupHidden = isAdaptableFormFieldGroupHidden;
 exports.isAdaptableFormFieldDisabled = isAdaptableFormFieldDisabled;
 exports.validateAdaptableForm = validateAdaptableForm;
-/**
- * Returns true if the supplied entry from a form's `fields` array is a
- * group rather than a single field or an inline-row array of fields.
- */
 function isAdaptableFormFieldGroup(entry) {
     return !!entry && !Array.isArray(entry) && entry.kind === 'group';
 }
-/**
- * Flattens a form's `fields` array (which may contain field groups and
- * inline-row arrays) into a single flat list of fields. Useful when
- * computing defaults, validation or per-field lookups.
- */
 function flattenAdaptableFormFields(formDef) {
     const out = [];
     for (const entry of formDef?.fields ?? []) {
@@ -79,10 +70,6 @@ function resolveSelectPlaceholder(field, options) {
 function getDefaultAdaptableFormData(formDef) {
     return flattenAdaptableFormFields(formDef).reduce((data, field) => {
         if (field.fieldType === 'select' && field.multi) {
-            // Multi-selects always work in terms of arrays. Allow the consumer to
-            // supply an explicit defaultValue (which may already be an array) but
-            // fall back to an empty array rather than '' so downstream code never
-            // has to guard against a string value.
             data[field.name] = Array.isArray(field.defaultValue)
                 ? field.defaultValue
                 : field.defaultValue == null || field.defaultValue === ''
@@ -95,15 +82,6 @@ function getDefaultAdaptableFormData(formDef) {
         return data;
     }, {});
 }
-/**
- * Resolves a field's `options` to the current array of options.
- *
- * - When `options` is already an array, it's returned as-is.
- * - When it's a synchronous function, the function is invoked.
- * - When the function returns a Promise this helper returns `undefined` -
- *   asynchronous resolution is the caller's responsibility (the
- *   `AdaptableFormComponent` resolves promises via internal state).
- */
 function resolveAdaptableFormFieldOptionsSync(field, formData, context) {
     const opts = field.options;
     if (!opts)
@@ -115,45 +93,24 @@ function resolveAdaptableFormFieldOptionsSync(field, formData, context) {
         return undefined;
     return result;
 }
-/**
- * Returns true if the given field should be hidden given the current form
- * data and context. Resolves both boolean and function forms of `hidden`.
- */
 function isAdaptableFormFieldHidden(field, formData, context) {
     if (typeof field.hidden === 'function') {
         return !!field.hidden(formData, context);
     }
     return !!field.hidden;
 }
-/**
- * Returns true if the given group should be hidden given the current
- * form data and context.
- */
 function isAdaptableFormFieldGroupHidden(group, formData, context) {
     if (typeof group.hidden === 'function') {
         return !!group.hidden(formData, context);
     }
     return !!group.hidden;
 }
-/**
- * Returns true if the given field should be disabled given the current
- * form data and context. Resolves both boolean and function forms of
- * `disabled`.
- */
 function isAdaptableFormFieldDisabled(field, formData, context) {
     if (typeof field.disabled === 'function') {
         return !!field.disabled(formData, context);
     }
     return !!field.disabled;
 }
-/**
- * Validates an entire AdaptableForm. Hidden fields and fields inside
- * hidden groups are skipped. Returns a map of field-name to error
- * message; fields without entries are valid.
- *
- * Validation order per field: `required` -> built-in (min/max/length/
- * pattern) -> custom `validate`. The first failing check wins.
- */
 function validateAdaptableForm(formDef, formData, context) {
     const errors = {};
     for (const entry of formDef?.fields ?? []) {
@@ -188,11 +145,9 @@ function validateOne(field, formData, context, errors) {
 }
 function validateAdaptableFormField(field, formData, context) {
     const value = formData[field.name];
-    // textOutput is read-only; never produces validation errors
     if (field.fieldType === 'textOutput') {
         return null;
     }
-    // 1) required
     if (field.required) {
         const isEmpty = value == null ||
             value === '' ||
@@ -202,9 +157,7 @@ function validateAdaptableFormField(field, formData, context) {
             return `${field.label} is required`;
         }
     }
-    // empty optional values skip subsequent built-ins
     const isProvided = value != null && value !== '';
-    // 2) built-in numeric / length / pattern checks
     if (isProvided) {
         if (field.fieldType === 'number' || field.fieldType === 'slider') {
             const numValue = Number(value);
@@ -243,12 +196,10 @@ function validateAdaptableFormField(field, formData, context) {
                     }
                 }
                 catch {
-                    // invalid pattern - silently ignore so we don't crash the form
                 }
             }
         }
     }
-    // 3) custom validation
     if (field.validate) {
         const customError = field.validate(value, formData, context);
         if (customError) {

package/src/AdaptableState/Common/AdaptableFormatPresets.js

@@ -1,10 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.resolveDisplayFormat = exports.isAdaptableNumericFormatPreset = exports.NUMERIC_FORMAT_PRESETS = void 0;
-/**
- * Concrete options associated with each numeric preset. The wizard and
- * runtime resolver both pull from this single source of truth.
- */
 exports.NUMERIC_FORMAT_PRESETS = {
     Percentage: {
         FractionDigits: 2,
@@ -158,19 +154,10 @@ exports.NUMERIC_FORMAT_PRESETS = {
         Notation: 'scientific',
     },
 };
-/**
- * Type guard: `true` if the value is a known numeric preset name.
- */
 const isAdaptableNumericFormatPreset = (value) => {
     return typeof value === 'string' && value in exports.NUMERIC_FORMAT_PRESETS;
 };
 exports.isAdaptableNumericFormatPreset = isAdaptableNumericFormatPreset;
-/**
- * Resolve a `DisplayFormat` (preset name or concrete object) into a
- * concrete `AdaptableFormat`. Returns `undefined` if the input is
- * `undefined`. A bare preset name is expanded into a `NumberFormatter`
- * with the corresponding options.
- */
 const resolveDisplayFormat = (displayFormat) => {
     if (displayFormat == null) {
         return undefined;

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;