typescript config and clean up

Signed-off-by: kohinoor98 <[email protected]>

Author: kohinoor98

.eslintrc.js

@@ -1,12 +1,17 @@
-/*
- * Copyright OpenSearch Contributors
- * SPDX-License-Identifier: Apache-2.0
- */
-
 const LICENSE_HEADER = `
 /*
- * Copyright OpenSearch Contributors
- * SPDX-License-Identifier: Apache-2.0
+ *   Copyright OpenSearch Contributors
+ *
+ *   Licensed under the Apache License, Version 2.0 (the "License").
+ *   You may not use this file except in compliance with the License.
+ *   A copy of the License is located at
+ *
+ *       http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *   or in the "license" file accompanying this file. This file is distributed
+ *   on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either
+ *   express or implied. See the License for the specific language governing
+ *   permissions and limitations under the License.
  */
 `;
 
@@ -15,6 +20,28 @@ module.exports = {
   extends: ["@elastic/eslint-config-kibana", "plugin:@elastic/eui/recommended"],
   rules: {
     // "@osd/eslint/require-license-header": "off"
+    "import/no-default-export": "off",
+    "@typescript-eslint/naming-convention": [
+      "error",
+      {
+        selector: "default",
+        format: ["camelCase", "UPPER_CASE", "PascalCase", "snake_case"],
+        leadingUnderscore: "allow",
+        trailingUnderscore: "allow",
+      },
+    ],
+    "@osd/eslint/no-restricted-paths": [
+      "error",
+      {
+        basePath: __dirname,
+        zones: [
+          {
+            target: ["(public|server)/**/*"],
+            from: ["../../packages/**/*", "packages/**/*"],
+          },
+        ],
+      },
+    ],
   },
   overrides: [
     {

.lintstagedrc

@@ -1,3 +1,9 @@
 {
-  "*.{ts,tsx,js,jsx,json,css,md}": ["prettier --write", "git add"]
+  "*.{ts,tsx}": [
+    "npx eslint --fix",
+    "prettier --write"
+  ],
+  "*.{js,jsx,json,css,md}": [
+    "prettier --write"
+  ]
 }

cypress/support/index.d.ts

@@ -1,9 +1,24 @@
+/*
+ *   Copyright OpenSearch Contributors
+ *
+ *   Licensed under the Apache License, Version 2.0 (the "License").
+ *   You may not use this file except in compliance with the License.
+ *   A copy of the License is located at
+ *
+ *       http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *   or in the "license" file accompanying this file. This file is distributed
+ *   on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either
+ *   express or implied. See the License for the specific language governing
+ *   permissions and limitations under the License.
+ */
+
 /*
  * Copyright OpenSearch Contributors
  * SPDX-License-Identifier: Apache-2.0
  */
 
-/// <reference types="cypress" />
+// / <reference types="cypress" />
 
 declare namespace Cypress {
   interface Chainable<Subject> {

models/interfaces.ts

@@ -1,3 +1,18 @@
+/*
+ *   Copyright OpenSearch Contributors
+ *
+ *   Licensed under the Apache License, Version 2.0 (the "License").
+ *   You may not use this file except in compliance with the License.
+ *   A copy of the License is located at
+ *
+ *       http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *   or in the "license" file accompanying this file. This file is distributed
+ *   on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either
+ *   express or implied. See the License for the specific language governing
+ *   permissions and limitations under the License.
+ */
+
 /*
  * Copyright OpenSearch Contributors
  * SPDX-License-Identifier: Apache-2.0
@@ -41,13 +56,13 @@ export type DiffableMappingsPropertiesObject = Record<
   }
 >;
 
-export type MappingsProperties = {
+export type MappingsProperties = Array<{
   fieldName: string;
   type: string;
   path?: string;
   analyzer?: string;
   properties?: MappingsProperties;
-}[];
+}>;
 
 export interface IndexItem {
   index: string;
@@ -525,14 +540,14 @@ export interface Transform {
   enabled_at: number | null;
   updated_at: number;
   metadata_id: string | null;
-  aggregations: Map<String, any>;
+  aggregations: Map<string, any>;
   page_size: number;
   schedule: IntervalSchedule | CronSchedule;
   schema_version: number;
   source_index: string;
   target_index: string;
-  roles: String[];
-  data_selection_query: Map<String, any>;
+  roles: string[];
+  data_selection_query: Map<string, any>;
 }
 
 export interface TransformMetadata {
@@ -571,15 +586,15 @@ export interface CronSchedule {
   };
 }
 
-//Frontend dimension data model
+// Frontend dimension data model
 export interface DimensionItem {
   sequence: number;
   field: FieldItem;
   aggregationMethod: string;
   interval?: number;
 }
 
-//Frontend metric data model
+// Frontend metric data model
 export interface MetricItem {
   source_field: FieldItem;
   all: boolean;
@@ -620,18 +635,23 @@ interface HistogramItem {
   };
 }
 
-//Backend dimension data model
+// Backend dimension data model
 export type RollupDimensionItem = DateHistogramItem | TermsItem | HistogramItem;
 
-//Backend metric data model
+// Backend metric data model
 export interface RollupMetricItem {
   source_field: string;
   metrics: [
     {
+      // eslint-disable-next-line @typescript-eslint/ban-types
       min?: Object;
+      // eslint-disable-next-line @typescript-eslint/ban-types
       max?: Object;
+      // eslint-disable-next-line @typescript-eslint/ban-types
       sum?: Object;
+      // eslint-disable-next-line @typescript-eslint/ban-types
       avg?: Object;
+      // eslint-disable-next-line @typescript-eslint/ban-types
       value_count?: Object;
     }
   ];

package.json

@@ -18,6 +18,7 @@
     "osd": "node ../../scripts/osd",
     "opensearch": "node ../../scripts/opensearch",
     "lint": "node ../../scripts/eslint . && node ../../scripts/stylelint",
+    "eslint:fix": "npx eslint '**/*.{ts,tsx}' --fix",
     "plugin-helpers": "node ../../scripts/plugin_helpers",
     "test:jest": "../../node_modules/.bin/jest --config ./test/jest.config.js",
     "build": "yarn plugin-helpers build",
@@ -73,4 +74,4 @@
   "engines": {
     "yarn": "^1.21.1"
   }
-}
+}

public/JobHandler/callbacks/force_merge.test.tsx

@@ -1,12 +1,27 @@
+/*
+ *   Copyright OpenSearch Contributors
+ *
+ *   Licensed under the Apache License, Version 2.0 (the "License").
+ *   You may not use this file except in compliance with the License.
+ *   A copy of the License is located at
+ *
+ *       http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *   or in the "license" file accompanying this file. This file is distributed
+ *   on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either
+ *   express or implied. See the License for the specific language governing
+ *   permissions and limitations under the License.
+ */
+
 /*
  * Copyright OpenSearch Contributors
  * SPDX-License-Identifier: Apache-2.0
  */
 
+import { CoreSetup } from "opensearch-dashboards/public";
 import { callbackForForceMerge, callbackForForceMergeTimeout } from "./force_merge";
 import { coreServicesMock, httpClientMock } from "../../../test/mocks";
 import { ListenType } from "../../lib/JobScheduler";
-import { CoreSetup } from "opensearch-dashboards/public";
 
 const getMockFn = (response = {}, ok = true) => {
   return jest.fn().mockResolvedValue({
@@ -33,7 +48,7 @@ const core = ({
 describe("callbackForForceMerge spec", () => {
   it("callback when error", async () => {
     httpClientMock.fetch = getMockFn({}, false);
-    let result = await callbackForForceMerge(forceMergeMetaData, {
+    const result = await callbackForForceMerge(forceMergeMetaData, {
       core,
     });
     expect(result).toBe(false);

public/JobHandler/callbacks/force_merge.tsx

@@ -1,3 +1,18 @@
+/*
+ *   Copyright OpenSearch Contributors
+ *
+ *   Licensed under the Apache License, Version 2.0 (the "License").
+ *   You may not use this file except in compliance with the License.
+ *   A copy of the License is located at
+ *
+ *       http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *   or in the "license" file accompanying this file. This file is distributed
+ *   on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either
+ *   express or implied. See the License for the specific language governing
+ *   permissions and limitations under the License.
+ */
+
 /*
  * Copyright OpenSearch Contributors
  * SPDX-License-Identifier: Apache-2.0
@@ -15,11 +30,11 @@ type ForceMergeTaskResult = TaskResult<{
     successful: number;
     total: number;
     failed: number;
-    failures?: {
+    failures?: Array<{
       index: string;
       status: string;
       shard: number;
-    }[];
+    }>;
   };
 }>;
 

public/JobHandler/callbacks/open.test.tsx

@@ -1,11 +1,26 @@
+/*
+ *   Copyright OpenSearch Contributors
+ *
+ *   Licensed under the Apache License, Version 2.0 (the "License").
+ *   You may not use this file except in compliance with the License.
+ *   A copy of the License is located at
+ *
+ *       http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *   or in the "license" file accompanying this file. This file is distributed
+ *   on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either
+ *   express or implied. See the License for the specific language governing
+ *   permissions and limitations under the License.
+ */
+
 /*
  * Copyright OpenSearch Contributors
  * SPDX-License-Identifier: Apache-2.0
  */
+import { CoreSetup } from "opensearch-dashboards/public";
 import { callbackForOpen, callbackForOpenTimeout } from "./open";
 import { coreServicesMock, httpClientMock } from "../../../test/mocks";
 import { ListenType } from "../../lib/JobScheduler";
-import { CoreSetup } from "opensearch-dashboards/public";
 
 const getMockFn = (response = {}, ok = true) => {
   return jest.fn().mockResolvedValue({
@@ -32,7 +47,7 @@ const core = ({
 describe("callbackForOpen spec", () => {
   it("callback when error", async () => {
     httpClientMock.fetch = getMockFn({}, false);
-    let result = await callbackForOpen(openMetaData, {
+    const result = await callbackForOpen(openMetaData, {
       core,
     });
     expect(result).toBe(false);

public/JobHandler/callbacks/open.tsx

@@ -1,7 +1,18 @@
 /*
- * Copyright OpenSearch Contributors
- * SPDX-License-Identifier: Apache-2.0
+ *   Copyright OpenSearch Contributors
+ *
+ *   Licensed under the Apache License, Version 2.0 (the "License").
+ *   You may not use this file except in compliance with the License.
+ *   A copy of the License is located at
+ *
+ *       http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *   or in the "license" file accompanying this file. This file is distributed
+ *   on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either
+ *   express or implied. See the License for the specific language governing
+ *   permissions and limitations under the License.
  */
+
 import React from "react";
 import { CallbackType, TaskResult } from "../interface";
 import { OpenJobMetaData } from "../../models/interfaces";

public/JobHandler/callbacks/reindex.test.tsx

@@ -1,11 +1,26 @@
+/*
+ *   Copyright OpenSearch Contributors
+ *
+ *   Licensed under the Apache License, Version 2.0 (the "License").
+ *   You may not use this file except in compliance with the License.
+ *   A copy of the License is located at
+ *
+ *       http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *   or in the "license" file accompanying this file. This file is distributed
+ *   on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either
+ *   express or implied. See the License for the specific language governing
+ *   permissions and limitations under the License.
+ */
+
 /*
  * Copyright OpenSearch Contributors
  * SPDX-License-Identifier: Apache-2.0
  */
+import { CoreSetup } from "opensearch-dashboards/public";
 import { callbackForReindex, callbackForReindexTimeout } from "./reindex";
 import { coreServicesMock, httpClientMock } from "../../../test/mocks";
 import { ListenType } from "../../lib/JobScheduler";
-import { CoreSetup } from "opensearch-dashboards/public";
 
 const getMockFn = (response = {}, ok = true) => {
   return jest.fn().mockResolvedValue({
@@ -35,7 +50,7 @@ const core = ({
 describe("callbackForOpen spec", () => {
   it("callback when error", async () => {
     httpClientMock.fetch = getMockFn({}, false);
-    let result = await callbackForReindex(reindexMetaData, {
+    const result = await callbackForReindex(reindexMetaData, {
       core,
     });
     expect(result).toBe(false);