Calling component binding ESlint (#5912)

* update

* Change files

Author: carocao-msft

change-beta/@azure-communication-react-260b0b2c-c155-4a4a-9dd1-853630f6ee16.json

@@ -0,0 +1,9 @@
+{
+  "type": "patch",
+  "area": "improvement",
+  "workstream": "Eslint",
+  "comment": "Fix eslint errors for calling component binding",
+  "packageName": "@azure/communication-react",
+  "email": "96077406+carocao-msft@users.noreply.github.com",
+  "dependentChangeType": "patch"
+}

change/@azure-communication-react-260b0b2c-c155-4a4a-9dd1-853630f6ee16.json

@@ -0,0 +1,9 @@
+{
+  "type": "patch",
+  "area": "improvement",
+  "workstream": "Eslint",
+  "comment": "Fix eslint errors for calling component binding",
+  "packageName": "@azure/communication-react",
+  "email": "96077406+carocao-msft@users.noreply.github.com",
+  "dependentChangeType": "patch"
+}

packages/calling-component-bindings/eslint.config.ts

@@ -5,20 +5,4 @@ import {
   testFilesBaseConfig
 } from '../../common/config/ESLint/eslint.config';
 
-export default globalConfig([
-  baseConfig,
-  testFilesBaseConfig,
-  {
-    rules: {
-      '@typescript-eslint/no-explicit-any': 'off',
-      '@typescript-eslint/no-unused-vars': [
-        'warn',
-        {
-          argsIgnorePattern: '^_'
-        }
-      ],
-      'jsdoc/require-jsdoc': 'off'
-    }
-  },
-  baseGlobalIgnores
-]);
+export default globalConfig([baseConfig, testFilesBaseConfig, baseGlobalIgnores]);

packages/calling-component-bindings/src/callUtils.test.ts

@@ -28,6 +28,7 @@ const validLocalVideoStreamMock: LocalVideoStreamState = {
   view: {
     scalingMode: 'Stretch',
     isMirrored: false,
+    //eslint-disable-next-line @typescript-eslint/no-explicit-any
     target: null as any
   }
 };

packages/calling-component-bindings/src/handlers/createCommonHandlers.ts

@@ -629,7 +629,7 @@ export const createDefaultCommonCallingHandlers = memoizeOne(
     };
 
     const onSendDtmfTone = async (dtmfTone: DtmfTone): Promise<void> => await call?.sendDtmf(dtmfTone);
-
+    //eslint-disable-next-line @typescript-eslint/no-explicit-any
     const notImplemented = (): any => {
       throw new Error('Not implemented, cannot call a method from an abstract object');
     };

packages/calling-component-bindings/src/handlers/createDefaultCallingHandlersForComponent.ts

@@ -36,6 +36,7 @@ export const createDefaultCallingHandlersForComponent = <Props>(
   callAgent: CallAgentCommon | undefined,
   deviceManager: StatefulDeviceManager | undefined,
   call: CallCommon | undefined,
+  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   _Component: (props: Props) => ReactElement | null
 ): Common<CommonCallingHandlers, Props> => {
   if (!callAgent && !call && !deviceManager) {

packages/calling-component-bindings/src/handlers/createHandlers.test.ts

@@ -12,28 +12,28 @@ import { CallingHandlers } from './createHandlers';
 function TestCallClientComponent(props: CallingHandlers): ReactElement | null {
   return null;
 }
-
 class MockCallClient {
+  //eslint-disable-next-line @typescript-eslint/no-explicit-any
   getState(): any {
     throw new Error('Method not implemented.');
   }
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
+  // eslint-disable-next-line @typescript-eslint/no-unused-vars, @typescript-eslint/no-explicit-any
   onStateChange(handler: (state: any) => void): void {
     throw new Error('Method not implemented.');
   }
   createView(
     // eslint-disable-next-line @typescript-eslint/no-unused-vars
     callId: string,
-    // eslint-disable-next-line @typescript-eslint/no-unused-vars
+    //eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/no-unused-vars
     participantId: any,
-    // eslint-disable-next-line @typescript-eslint/no-unused-vars
+    //eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/no-unused-vars
     stream: any,
-    // eslint-disable-next-line @typescript-eslint/no-unused-vars
+    //eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/no-unused-vars
     options?: any
   ): Promise<CreateViewResult | undefined> {
     throw new Error('Method not implemented.');
   }
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
+  //eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/no-unused-vars
   disposeView(callId: string, participantId: any, stream: any): void {
     throw new Error('Method not implemented.');
   }

packages/calling-component-bindings/src/handlers/createTeamsCallHandlers.ts

@@ -171,6 +171,7 @@ export const createTeamsCallingHandlersForComponent = <Props>(
   callAgent: undefined | TeamsCallAgent,
   deviceManager: StatefulDeviceManager | undefined,
   call: undefined | TeamsCall,
+  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   _Component: (props: Props) => ReactElement | null
 ): Common<TeamsCallingHandlers, Props> => {
   return createDefaultTeamsCallingHandlers(callClient, callAgent, deviceManager, call);
@@ -184,6 +185,7 @@ const teamsSingleParticipantTrampoline = (
   if (participants.length !== 1) {
     throw new Error('Only one participant is supported in Teams call!');
   } else {
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     return callAgent.startCall(participants[0] as any, options);
   }
 };

packages/calling-component-bindings/src/hooks/useHandlers.ts

@@ -22,7 +22,7 @@ import { CallAgentContext, CallClientContext, CallContext, useDeviceManager } fr
  */
 // eslint-disable-next-line @typescript-eslint/explicit-function-return-type
 export const useHandlers = <PropsT>(component: (props: PropsT) => ReactElement | null) => {
-  const callClient: StatefulCallClient = (useContext(CallClientContext) as any)?.callClient;
+  const callClient: StatefulCallClient = useContext(CallClientContext)?.callClient as StatefulCallClient;
   const deviceManager = useDeviceManager();
   const call = useContext(CallContext)?.call;
   const callAgent = useContext(CallAgentContext)?.callAgent;

packages/calling-component-bindings/src/hooks/usePropsFor.ts

@@ -75,8 +75,10 @@ import {
  *
  * @public
  */
+//eslint-disable-next-line @typescript-eslint/no-explicit-any
 export const usePropsFor = <Component extends (props: any) => JSX.Element>(
   component: Component
+  //eslint-disable-next-line @typescript-eslint/no-explicit-any
 ): GetSelector<Component> extends (props: any) => any
   ? ReturnType<GetSelector<Component>> &
       Common<CommonCallingHandlers & _ComponentCallingHandlers, Parameters<Component>[0]>
@@ -85,8 +87,10 @@ export const usePropsFor = <Component extends (props: any) => JSX.Element>(
   const props = useSelector(selector);
   const handlers = useHandlers<Parameters<Component>[0]>(component);
   if (props !== undefined) {
+    //eslint-disable-next-line @typescript-eslint/no-explicit-any
     return { ...props, ...handlers } as any;
   }
+  //eslint-disable-next-line @typescript-eslint/no-explicit-any
   return undefined as any;
 };
 
@@ -106,6 +110,7 @@ const emptySelector: EmptySelector = (): Record<string, never> => ({});
  *
  * @public
  */
+//eslint-disable-next-line @typescript-eslint/no-explicit-any
 export type GetSelector<Component extends (props: any) => JSX.Element | undefined> =
   AreEqual<Component, typeof VideoGallery> extends true
     ? VideoGallerySelector
@@ -153,12 +158,13 @@ export type GetSelector<Component extends (props: any) => JSX.Element | undefine
  *
  * @public
  */
+//eslint-disable-next-line @typescript-eslint/no-explicit-any
 export const getSelector = <Component extends (props: any) => JSX.Element | undefined>(
   component: Component
 ): GetSelector<Component> => {
   return findSelector(component);
 };
-
+//eslint-disable-next-line @typescript-eslint/no-explicit-any
 const findSelector = (component: (props: any) => JSX.Element | undefined): any => {
   // Dialpad only has handlers currently and doesn't require any props from the stateful layer so return the emptySelector
   if (component === Dialpad) {