Scheduler — Add ESLint naming-convention rule to forbid underscore-prefixed members

packages/devextreme/eslint.config.mjs

@@ -26,6 +26,165 @@ const compat = new FlatCompat({
     allConfig: js.configs.all
 });
 
+const schedulerDOMComponentOverrides = [
+    '_createActionByOption',
+    '_defaultOptionsRules',
+    '_dimensionChanged',
+    '_dispose',
+    '_disposed',
+    '_getDefaultOptions',
+    '_init',
+    '_initTemplates',
+    '_optionChanged',
+    '_setOptionsByReference',
+    '_useTemplates',
+    '_visibilityChanged',
+];
+
+const schedulerWidgetOverrides = [
+    '_activeStateUnit',
+    '_clean',
+    '_cleanFocusState',
+    '_eventBindingTarget',
+    '_fireContentReadyAction',
+    '_focusInHandler',
+    '_focusOutHandler',
+    '_focusTarget',
+    '_initMarkup',
+    '_keyboardHandler',
+    '_render',
+    '_renderContent',
+    '_renderFocusState',
+    '_renderFocusTarget',
+    '_supportedKeys',
+    '_toggleVisibility',
+];
+
+const schedulerCollectionWidgetOverrides = [
+    '_cleanItemContainer',
+    '_clearDropDownItemsElements',
+    '_createItemByTemplate',
+    '_executeItemRenderAction',
+    '_filteredItems',
+    '_findItemElementByItem',
+    '_focusedItemIndexBeforeRender',
+    '_getItemContent',
+    '_itemClass',
+    '_itemClickHandler',
+    '_itemContainer',
+    '_moveFocus',
+    '_postprocessRenderItem',
+    '_processItemClick',
+    '_refreshActiveDescendant',
+    '_renderDirection',
+    '_renderItem',
+    '_sortedItems',
+];
+
+const schedulerR1Overrides = [
+    '_propsInfo',
+    '_value',
+    '_viewComponent',
+];
+
+const schedulerWorkspaceOverrides = [
+    '_calculateStartViewDate',
+    '_getAllDayHeight',
+    '_getCellCount',
+    '_getDateRange',
+    '_getDisplayedDate',
+    '_getGroupCount',
+    '_getIntervalDuration',
+    '_isHorizontalGroupedWorkSpace',
+    '_setVisibilityDates',
+    '_updateStateVirtualItems',
+];
+
+const schedulerLegacyMembers = [
+    '_$allDayIndicator',
+    '_$allDayPanel',
+    '_$bottomShader',
+    '_$container',
+    '_$dateTable',
+    '_$dateTableScrollableContent',
+    '_$flexContainer',
+    '_$groupTable',
+    '_$headerPanel',
+    '_$headerPanelContainer',
+    '_$mainGroup',
+    '_$recurrenceGroup',
+    '_$shader',
+    '_$thead',
+    '_$topShader',
+    '_addEvent',
+    '_appointmentTooltipOffset',
+    '_appointments',
+    '_calculateCellIndex',
+    '_callEvent',
+    '_compactAppointmentsHelper',
+    '_createAction',
+    '_createEventArgs',
+    '_createEventMap',
+    '_createOverlayContent',
+    '_createScrollable',
+    '_createToolbarConfig',
+    '_dataAccessors',
+    '_dataSource',
+    '_dataSourceChangedHandler',
+    '_dataSourceOptions',
+    '_dateTableScrollable',
+    '_draggingMode',
+    '_dxForm',
+    '_getCalendarOptionUpdater',
+    '_getCalendarOptions',
+    '_getCaption',
+    '_getCaptionOptions',
+    '_getDragBehavior',
+    '_getMaxAllowedVerticalPosition',
+    '_getNextDate',
+    '_groupPanelItems',
+    '_groupedStrategy',
+    '_ignorePreventScrollEventsDeprecation',
+    '_isAppointmentBeingUpdated',
+    '_isMobileLayout',
+    '_isScrollOptionsObject',
+    '_layoutManager',
+    '_options',
+    '_parseItem',
+    '_popup',
+    '_readOnly',
+    '_recurrenceForm',
+    '_renderAppointmentTemplate',
+    '_renderCalendar',
+    '_renderGrid',
+    '_renderOverlay',
+    '_renderToolbar',
+    '_shader',
+    '_showCalendar',
+    '_sidebarScrollable',
+    '_textCache',
+    '_timeZoneCalculator',
+    '_updateCalendarValueAndCurrentDate',
+    '_updateCurrentDate',
+    '_updateCurrentView',
+    '_updateDateByDirection',
+    '_useShortDateFormat',
+    '_weekDayButtons',
+    '_workSpace',
+    '_wrapperClassExternal',
+];
+
+const schedulerMemberAllowlist = [
+    ...schedulerDOMComponentOverrides,
+    ...schedulerWidgetOverrides,
+    ...schedulerCollectionWidgetOverrides,
+    ...schedulerR1Overrides,
+    ...schedulerWorkspaceOverrides,
+    ...schedulerLegacyMembers,
+];
+const schedulerMemberAllowlistRegex =
+    `^(_|__esModule|${schedulerMemberAllowlist.map(s => s.replace(/\$/g, '\\$')).join('|')})$`;
+
 export default [
     {
         ignores: [
@@ -556,7 +715,6 @@ export default [
                     selector: ['variable', 'function', 'parameter'],
                     format: null,
                     leadingUnderscore: 'forbid',
-                    // allow only a single underscore identifier `_` to bypass this rule
                     filter: {
                         regex: '^_$',
                         match: false,
@@ -565,13 +723,40 @@ export default [
                 {
                     selector: 'memberLike',
                     format: null,
-                    leadingUnderscore: 'allow',
+                    leadingUnderscore: 'forbid',
+                    filter: {
+                        regex: schedulerMemberAllowlistRegex,
+                        match: false,
+                    },
                 },
             ],
             'devextreme-custom/no-deferred': 'error',
             'devextreme-custom/prefer-switch-true': ['error', { minBranches: 3 }],
         },
     },
+    // Temporarily allow underscore members in appointments/ (pending refactoring)
+    {
+        files: ['js/__internal/scheduler/appointments/**/*.ts?(x)'],
+        rules: {
+            '@typescript-eslint/naming-convention': [
+                'error',
+                {
+                    selector: ['variable', 'function', 'parameter'],
+                    format: null,
+                    leadingUnderscore: 'forbid',
+                    filter: {
+                        regex: '^_$',
+                        match: false,
+                    },
+                },
+                {
+                    selector: 'memberLike',
+                    format: null,
+                    leadingUnderscore: 'allow',
+                },
+            ],
+        },
+    },
     // Allow Deferred in m_* scheduler files only
     {
         files: ['js/__internal/scheduler/**/m_*.ts?(x)'],