fix: update @trivago/prettier-plugin-sort-imports to fix babel vuln…

…erable to arbitrary code execution (#1002) * fix: `babel` vulnerable to arbitrary code execution when compiling specifically crafted malicious code * style: run prettier format --------- Co-authored-by: Sarin-Udompanish <[email protected]>
Refinitiv · Oct 26, 2023 · e96385f · e96385f
1 parent b039d0c
commit e96385f
Show file tree

Hide file tree

Showing 69 changed files with 206 additions and 196 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/packages/configurations/package.json b/packages/configurations/package.json
@@ -22,7 +22,7 @@
   "author": "LSEG",
   "license": "Apache-2.0",
   "peerDependencies": {
-    "@trivago/prettier-plugin-sort-imports": "4.1.0",
+    "@trivago/prettier-plugin-sort-imports": "^4.2.1",
     "@typescript-eslint/eslint-plugin": "^5.37.0",
     "@typescript-eslint/parser": "^5.37.0",
     "eslint": "^8.49.0",

diff --git a/packages/configurations/prettier-config.json b/packages/configurations/prettier-config.json
@@ -9,8 +9,7 @@
     "^@refinitiv-ui/core",
     "^@refinitiv-ui/elements",
     "^@refinitiv-ui/(.*)$",
-    "^[./]",
-    "<TS_TYPES>"
+    "^[./]"
   ],
   "importOrderSeparation": true,
   "importOrderSortSpecifiers": true,

diff --git a/packages/core/src/decorators/custom-element.ts b/packages/core/src/decorators/custom-element.ts
@@ -1,8 +1,7 @@
+import type { ElementConstructor } from '../interfaces/ElementConstructor';
 import { CustomStyleRegistry } from '../registries/CustomStyleRegistry.js';
 import { ElementRegistry } from '../registries/ElementRegistry.js';
 
-import type { ElementConstructor } from '../interfaces/ElementConstructor';
-
 interface ElementOptions {
   /**
    * External theme is required for this element.

diff --git a/packages/core/src/elements/BasicElement.ts b/packages/core/src/elements/BasicElement.ts
@@ -3,11 +3,10 @@ import { CSSResultArray, LitElement, unsafeCSS } from 'lit';
 import { property } from '../decorators/property.js';
 import { ElementRegistry } from '../registries/ElementRegistry.js';
 import { FocusRegistry } from '../registries/FocusRegistry.js';
+import type { CSSValue } from '../types/base';
 import { FocusableHelper } from '../utils/focusableHelper.js';
 import { BasicElementSymbol } from '../utils/helpers.js';
 
-import type { CSSValue } from '../types/base';
-
 const CSS_VARIABLE_REGEXP = /^--\w/;
 const CSS_VARIABLE_REPLACE_REGEXP = /['"]([^'"]+?)['"]/g;
 const NOTIFY_REGEXP = /([a-zA-Z])(?=[A-Z])/g;

diff --git a/packages/core/src/elements/ControlElement.ts b/packages/core/src/elements/ControlElement.ts
@@ -1,10 +1,10 @@
+import type { PropertyValues } from 'lit';
+
 import { property } from '../decorators/property.js';
+import type { ControlProperties as IControlProperties } from '../interfaces/ControlProperties';
 import { WarningNotice } from '../notices/WarningNotice.js';
 import { BasicElement } from './BasicElement.js';
 
-import type { ControlProperties as IControlProperties } from '../interfaces/ControlProperties';
-import type { PropertyValues } from 'lit';
-
 /**
  * Control element base class.
  * Usually used for creating form-style elements.

diff --git a/packages/core/src/index.ts b/packages/core/src/index.ts
@@ -2,17 +2,17 @@
  * Export element base classes
  */
 import { BasicElement } from './elements/BasicElement.js';
-import { CustomStyleRegistry } from './registries/CustomStyleRegistry.js';
-import { NativeStyleRegistry } from './registries/NativeStyleRegistry.js';
-import { global } from './utils/global.js';
 
 /**
  * Export common interfaces
  */
 import type { MultiValue } from './interfaces/MultiValue';
 import type { StyleInfo } from './interfaces/StyleInfo';
 import type { StyleMap } from './interfaces/StyleMap';
+import { CustomStyleRegistry } from './registries/CustomStyleRegistry.js';
+import { NativeStyleRegistry } from './registries/NativeStyleRegistry.js';
 import type { FocusedChangedEvent } from './types/events';
+import { global } from './utils/global.js';
 
 /* eslint-disable @typescript-eslint/no-unsafe-argument */
 /* eslint-disable @typescript-eslint/no-unsafe-assignment */

diff --git a/packages/core/src/registries/ElementRegistry.ts b/packages/core/src/registries/ElementRegistry.ts
@@ -1,9 +1,8 @@
 import { DuplicateElementError } from '../errors/DuplicateElementError.js';
+import type { ElementConstructor } from '../interfaces/ElementConstructor';
 import { ready } from '../utils/elementReady.js';
 import { CustomStyleRegistry } from './CustomStyleRegistry.js';
 
-import type { ElementConstructor } from '../interfaces/ElementConstructor';
-
 class ElementRegistrationItem {
   creations = 0;
   connections = 0;

diff --git a/packages/core/src/registries/FocusRegistry.ts b/packages/core/src/registries/FocusRegistry.ts
@@ -1,7 +1,6 @@
-import { isBasicElement } from '../utils/helpers.js';
-
 import type { BasicElement } from '../elements/BasicElement';
 import type { FocusedChangedEvent } from '../types/events';
+import { isBasicElement } from '../utils/helpers.js';
 
 const register = new Set<BasicElement>(); /* Track all active elements */
 const focusedMap = new Map<BasicElement, 'visible' | ''>(); /* Track all focused elements */

diff --git a/packages/core/src/utils/focusableHelper.ts b/packages/core/src/utils/focusableHelper.ts
@@ -1,6 +1,5 @@
-import { isBasicElement } from './helpers.js';
-
 import type { BasicElement } from '../elements/BasicElement';
+import { isBasicElement } from './helpers.js';
 
 type DelegatedList = {
   element: BasicElement;

diff --git a/packages/elements/src/autosuggest/helpers/renderer.ts b/packages/elements/src/autosuggest/helpers/renderer.ts
@@ -1,8 +1,7 @@
 import { uuid } from '@refinitiv-ui/utils/uuid.js';
 
-import { updateElementContent } from './utils.js';
-
 import type { AutosuggestItem, AutosuggestQuery, AutosuggestRenderer, Suggestion } from './types';
+import { updateElementContent } from './utils.js';
 
 /**
  * Build item element from data object

diff --git a/packages/elements/src/autosuggest/index.ts b/packages/elements/src/autosuggest/index.ts
@@ -10,14 +10,12 @@ import { TranslateDirective, TranslatePropertyKey, translate } from '@refinitiv-
 import { AnimationTaskRunner, TimeoutTaskRunner } from '@refinitiv-ui/utils/async.js';
 import { isMobile } from '@refinitiv-ui/utils/browser.js';
 
+import type { TapEvent } from '../events';
 import '../item/index.js';
 import '../loader/index.js';
 import { Overlay } from '../overlay/index.js';
 import { VERSION } from '../version.js';
 import { renderer } from './helpers/renderer.js';
-import { escapeRegExp, itemHighlightable, queryWordSelect } from './helpers/utils.js';
-
-import type { TapEvent } from '../events';
 import type {
   AutosuggestHighlightItemEvent,
   AutosuggestHighlightable,
@@ -40,6 +38,7 @@ import type {
   SuggestionsFetchRequestedEvent,
   SuggestionsQueryEvent
 } from './helpers/types';
+import { escapeRegExp, itemHighlightable, queryWordSelect } from './helpers/utils.js';
 
 export type {
   AutosuggestTargetElement,

diff --git a/packages/elements/src/calendar/index.ts b/packages/elements/src/calendar/index.ts
@@ -56,7 +56,9 @@ import {
   up
 } from '@refinitiv-ui/utils/navigation.js';
 
+import type { Button } from '../button';
 import '../button/index.js';
+import type { TapEvent } from '../events';
 import { VERSION } from '../version.js';
 import {
   CalendarLocaleScope,
@@ -68,10 +70,6 @@ import {
   YEAR_VIEW
 } from './constants.js';
 import './locales.js';
-import { ViewFormatTranslateParams, monthInfo, monthsNames, toCalendarCell, weekdaysNames } from './utils.js';
-
-import type { Button } from '../button';
-import type { TapEvent } from '../events';
 import type {
   BeforeCellRenderEvent,
   CalendarFilter,
@@ -83,6 +81,7 @@ import type {
   Row,
   WeekdayName
 } from './types';
+import { ViewFormatTranslateParams, monthInfo, monthsNames, toCalendarCell, weekdaysNames } from './utils.js';
 
 export { CalendarRenderView };
 export type { CalendarCell } from './types';

diff --git a/packages/elements/src/card/index.ts b/packages/elements/src/card/index.ts
@@ -8,14 +8,13 @@ import '@refinitiv-ui/phrasebook/locale/en/card.js';
 import { Translate, translate } from '@refinitiv-ui/translate';
 import { isSlotEmpty } from '@refinitiv-ui/utils/is-slot-empty.js';
 
+import type { Button } from '../button';
 import '../button/index.js';
+import type { ItemTriggerEvent, OpenedChangedEvent } from '../events';
 import '../label/index.js';
+import type { OverlayMenu, OverlayMenuData } from '../overlay-menu';
 import '../overlay-menu/index.js';
 import { VERSION } from '../version.js';
-
-import type { Button } from '../button';
-import type { ItemTriggerEvent, OpenedChangedEvent } from '../events';
-import type { OverlayMenu, OverlayMenuData } from '../overlay-menu';
 import type { CardConfig } from './helpers/types';
 
 export type { CardConfig };

diff --git a/packages/elements/src/chart/elements/chart.ts b/packages/elements/src/chart/elements/chart.ts
@@ -1,4 +1,15 @@
 import { Chart as ChartJS } from 'chart.js';
+import type {
+  ChartConfiguration,
+  ChartDataset,
+  ChartOptions,
+  ChartType,
+  Color,
+  LegendItem,
+  LineControllerDatasetOptions,
+  Plugin,
+  UpdateMode
+} from 'chart.js';
 import 'chartjs-adapter-date-fns';
 
 import {
@@ -20,18 +31,6 @@ import '../../header/index.js';
 import { VERSION } from '../../version.js';
 import { DatasetColors, MergeObject, merge } from '../helpers/index.js';
 
-import type {
-  ChartConfiguration,
-  ChartDataset,
-  ChartOptions,
-  ChartType,
-  Color,
-  LegendItem,
-  LineControllerDatasetOptions,
-  Plugin,
-  UpdateMode
-} from 'chart.js';
-
 const CSS_COLOR_PREFIX = '--chart-color-';
 
 /* Make ChartJS to know our plugin