refactor: PR feedback - naming nits

Author: gpaoloni

lambdas/account-scoped/src/router.ts

@@ -19,7 +19,7 @@ import { AccountScopedRoute, FunctionRoute, HttpRequest } from './httpTypes';
 import { validateRequestMethod } from './validation/method';
 import { isAccountSID } from './twilioTypes';
 import { handleTaskRouterEvent } from './taskrouter';
-import { updateWorkersSkills } from './taskrouter/updateWorkersSkills';
+import { handleUpdateWorkersSkills } from './taskrouter/updateWorkersSkills';
 import { handleGetProfileFlagsForIdentifier } from './hrm/getProfileFlagsForIdentifier';
 import { handleToggleSwitchboardQueue } from './hrm/toggleSwitchboardQueue';
 import {
@@ -119,7 +119,7 @@ const ROUTES: Record<string, FunctionRoute> = {
   },
   updateWorkersSkills: {
     requestPipeline: [validateFlexTokenRequest({ tokenMode: 'supervisor' })],
-    handler: updateWorkersSkills,
+    handler: handleUpdateWorkersSkills,
   },
 };
 

lambdas/account-scoped/src/taskrouter/updateWorkersSkills.ts

@@ -62,7 +62,7 @@ const mergeAttributes = ({
   };
 };
 
-const handleEnableOperation = ({
+const setSkillsEnable = ({
   attributes,
   enabledSkills,
   disabledSkills,
@@ -86,7 +86,7 @@ const handleEnableOperation = ({
   });
 };
 
-const handleDisableOperation = ({
+const setSkillsDisable = ({
   attributes,
   enabledSkills,
   disabledSkills,
@@ -110,7 +110,7 @@ const handleDisableOperation = ({
   });
 };
 
-const handleAssignOperation = ({
+const setSkillsAssign = ({
   attributes,
   enabledSkills,
   disabledSkills,
@@ -138,7 +138,7 @@ const handleAssignOperation = ({
   });
 };
 
-const handleUnassignOperation = ({
+const setSkillsUnassign = ({
   attributes,
   enabledSkills,
   disabledSkills,
@@ -179,16 +179,16 @@ const updateSkillsOperation = ({
 
   switch (operation) {
     case 'enable': {
-      return handleEnableOperation(params);
+      return setSkillsEnable(params);
     }
     case 'disable': {
-      return handleDisableOperation(params);
+      return setSkillsDisable(params);
     }
     case 'assign': {
-      return handleAssignOperation(params);
+      return setSkillsAssign(params);
     }
     case 'unassign': {
-      return handleUnassignOperation(params);
+      return setSkillsUnassign(params);
     }
     default:
       return attributes;
@@ -233,7 +233,10 @@ const updateWorkerSkills = async ({
   }
 };
 
-export const updateWorkersSkills: AccountScopedHandler = async ({ body }, accountSid) => {
+export const handleUpdateWorkersSkills: AccountScopedHandler = async (
+  { body },
+  accountSid,
+) => {
   try {
     const { workers, skills, operation } = body;