secure all .toLowerCase() calls (after a crash report) (#1235)

anastasiya1155 · web-flow · commit d15680701e58 · 2024-02-20T12:33:16.000-06:00
diff --git a/client/src/CommandBar/steps/AddToStudio.tsx b/client/src/CommandBar/steps/AddToStudio.tsx
@@ -156,7 +156,7 @@ const AddToStudio = (props: Props) => {
     const newSectionsToShow: CommandBarSectionType[] = [];
     initialSections.forEach((s) => {
       const items = (s.items as CommandBarItemGeneralType[]).filter((item) => {
-        return item.label.toLowerCase().includes(inputValue.toLowerCase());
+        return item.label?.toLowerCase().includes(inputValue?.toLowerCase());
       });
 
       if (items.length) {
diff --git a/client/src/CommandBar/steps/Documentation/index.tsx b/client/src/CommandBar/steps/Documentation/index.tsx
@@ -177,10 +177,11 @@ const Documentation = ({}: Props) => {
     }
     const newSections: CommandBarSectionType[] = [];
     sections.forEach((s) => {
-      const newItems = s.items.filter((i) =>
-        ('label' in i ? i.label : i.componentProps.doc.name)
-          .toLowerCase()
-          .includes(inputValue.toLowerCase()),
+      const newItems = s.items.filter(
+        (i) =>
+          ('label' in i ? i.label : i.componentProps.doc.name)
+            ?.toLowerCase()
+            .includes(inputValue?.toLowerCase()),
       );
       if (newItems.length) {
         newSections.push({ ...s, items: newItems });
diff --git a/client/src/CommandBar/steps/Initial.tsx b/client/src/CommandBar/steps/Initial.tsx
@@ -434,8 +434,8 @@ const InitialCommandBar = ({ shouldShowTutorial }: Props) => {
     }
     const newSections: CommandBarSectionType[] = [];
     initialSections.forEach((s) => {
-      const newItems = (s.items as CommandBarItemGeneralType[]).filter((i) =>
-        i.label.toLowerCase().includes(inputValue.toLowerCase()),
+      const newItems = (s.items as CommandBarItemGeneralType[]).filter(
+        (i) => i.label?.toLowerCase().includes(inputValue?.toLowerCase()),
       );
       if (newItems.length) {
         newSections.push({
diff --git a/client/src/CommandBar/steps/LocalRepos.tsx b/client/src/CommandBar/steps/LocalRepos.tsx
@@ -140,10 +140,11 @@ const LocalRepos = ({}: Props) => {
     }
     const newSections: CommandBarSectionType[] = [];
     sections.forEach((s) => {
-      const newItems = s.items.filter((i) =>
-        ('label' in i ? i.label : i.componentProps.repo.shortName)
-          .toLowerCase()
-          .includes(inputValue.toLowerCase()),
+      const newItems = s.items.filter(
+        (i) =>
+          ('label' in i ? i.label : i.componentProps.repo.shortName)
+            ?.toLowerCase()
+            .includes(inputValue?.toLowerCase()),
       );
       if (newItems.length) {
         newSections.push({ ...s, items: newItems });
diff --git a/client/src/CommandBar/steps/ManageRepos/index.tsx b/client/src/CommandBar/steps/ManageRepos/index.tsx
@@ -172,9 +172,9 @@ const ManageRepos = ({ shouldShowTutorial }: Props) => {
     ) => {
       return 'componentProps' in item
         ? item.componentProps.repo.shortName
-            .toLowerCase()
-            .includes(inputValue.toLowerCase())
-        : item.label.toLowerCase().includes(inputValue.toLowerCase());
+            ?.toLowerCase()
+            .includes(inputValue?.toLowerCase())
+        : item.label?.toLowerCase().includes(inputValue?.toLowerCase());
     };
 
     sections.forEach((s) => {
diff --git a/client/src/CommandBar/steps/PrivateRepos/index.tsx b/client/src/CommandBar/steps/PrivateRepos/index.tsx
@@ -91,8 +91,8 @@ const PrivateReposStep = ({ shouldShowTutorial }: Props) => {
     sections.forEach((s) => {
       const items = (s.items as CommandBarItemCustomType[]).filter((item) => {
         return item.componentProps.repo.shortName
-          .toLowerCase()
-          .includes(inputValue.toLowerCase());
+          ?.toLowerCase()
+          .includes(inputValue?.toLowerCase());
       });
 
       if (items.length) {
diff --git a/client/src/Project/CurrentTabContent/ChatTab/Input/index.tsx b/client/src/Project/CurrentTabContent/ChatTab/Input/index.tsx
@@ -178,7 +178,7 @@ const ConversationInput = ({
         .filter(
           (t): t is FileTabType =>
             t.type === TabTypesEnum.FILE &&
-            (!search || t.path.toLowerCase().includes(search.toLowerCase())),
+            (!search || t.path?.toLowerCase().includes(search?.toLowerCase())),
         )
         .map((t) => ({ path: t.path, repo: t.repoRef }));
       filesResults.push(
diff --git a/client/src/Project/CurrentTabContent/StudioTab/Conversation/Input/TemplatesDropdown.tsx b/client/src/Project/CurrentTabContent/StudioTab/Conversation/Input/TemplatesDropdown.tsx
@@ -39,8 +39,8 @@ const TemplatesDropdown = ({ templates, onTemplateSelected }: Props) => {
       setTemplatesToShow(templates);
     } else {
       setTemplatesToShow(
-        templates.filter((t) =>
-          t.name.toLowerCase().includes(inputValue.toLowerCase()),
+        templates.filter(
+          (t) => t.name?.toLowerCase().includes(inputValue?.toLowerCase()),
         ),
       );
     }
diff --git a/client/src/Project/LeftSidebar/NavPanel/Repo/RepoDropdown.tsx b/client/src/Project/LeftSidebar/NavPanel/Repo/RepoDropdown.tsx
@@ -113,35 +113,38 @@ const RepoDropdown = ({
     if (!search) {
       return indexedBranches;
     }
-    return indexedBranches.filter((b) =>
-      b
-        .replace(/^origin\//, '')
-        .toLowerCase()
-        .includes(search.toLowerCase()),
+    return indexedBranches.filter(
+      (b) =>
+        b
+          .replace(/^origin\//, '')
+          ?.toLowerCase()
+          .includes(search?.toLowerCase()),
     );
   }, [indexedBranches, search]);
 
   const indexingBranchesToShow = useMemo(() => {
     if (!search) {
       return branchesToSync;
     }
-    return branchesToSync.filter((b) =>
-      b
-        .replace(/^origin\//, '')
-        .toLowerCase()
-        .includes(search.toLowerCase()),
+    return branchesToSync.filter(
+      (b) =>
+        b
+          .replace(/^origin\//, '')
+          ?.toLowerCase()
+          .includes(search?.toLowerCase()),
     );
   }, [branchesToSync, search]);
 
   const notIndexedBranchesToShow = useMemo(() => {
     if (!search) {
       return notSyncedBranches;
     }
-    return notSyncedBranches.filter((b) =>
-      b
-        .replace(/^origin\//, '')
-        .toLowerCase()
-        .includes(search.toLowerCase()),
+    return notSyncedBranches.filter(
+      (b) =>
+        b
+          .replace(/^origin\//, '')
+          ?.toLowerCase()
+          .includes(search?.toLowerCase()),
     );
   }, [notSyncedBranches, search]);
 
diff --git a/client/src/Project/LeftSidebar/NavPanel/Repo/index.tsx b/client/src/Project/LeftSidebar/NavPanel/Repo/index.tsx
@@ -72,7 +72,7 @@ const RepoNav = ({
       }
       return resp?.entries.sort((a, b) => {
         if ((a.entry_data === 'Directory') === (b.entry_data === 'Directory')) {
-          return a.name.toLowerCase() < b.name.toLowerCase() ? -1 : 1;
+          return a.name?.toLowerCase() < b.name?.toLowerCase() ? -1 : 1;
         } else {
           return a.entry_data === 'Directory' ? -1 : 1;
         }
diff --git a/client/src/Project/LeftSidebar/RegexSearchPanel/Results/FileResult.tsx b/client/src/Project/LeftSidebar/RegexSearchPanel/Results/FileResult.tsx
@@ -41,7 +41,7 @@ const FileResult = ({
       }
       return resp?.entries.sort((a, b) => {
         if ((a.entry_data === 'Directory') === (b.entry_data === 'Directory')) {
-          return a.name.toLowerCase() < b.name.toLowerCase() ? -1 : 1;
+          return a.name?.toLowerCase() < b.name?.toLowerCase() ? -1 : 1;
         } else {
           return a.entry_data === 'Directory' ? -1 : 1;
         }
diff --git a/client/src/Project/LeftSidebar/RegexSearchPanel/Results/RepoResult.tsx b/client/src/Project/LeftSidebar/RegexSearchPanel/Results/RepoResult.tsx
@@ -25,7 +25,7 @@ const RepoResult = ({ repoRef, isExpandable, index }: Props) => {
       }
       return resp?.entries.sort((a, b) => {
         if ((a.entry_data === 'Directory') === (b.entry_data === 'Directory')) {
-          return a.name.toLowerCase() < b.name.toLowerCase() ? -1 : 1;
+          return a.name?.toLowerCase() < b.name?.toLowerCase() ? -1 : 1;
         } else {
           return a.entry_data === 'Directory' ? -1 : 1;
         }
diff --git a/client/src/components/MarkdownWithCode/FolderChip.tsx b/client/src/components/MarkdownWithCode/FolderChip.tsx
@@ -22,7 +22,7 @@ const FolderChip = ({ onClick, path, repoRef }: Props) => {
       }
       return resp?.entries.sort((a, b) => {
         if ((a.entry_data === 'Directory') === (b.entry_data === 'Directory')) {
-          return a.name.toLowerCase() < b.name.toLowerCase() ? -1 : 1;
+          return a.name?.toLowerCase() < b.name?.toLowerCase() ? -1 : 1;
         } else {
           return a.entry_data === 'Directory' ? -1 : 1;
         }
diff --git a/client/src/hooks/useCodeSearch.ts b/client/src/hooks/useCodeSearch.ts
@@ -61,7 +61,7 @@ export const useCodeSearch = ({
     }
     const lines = code.split('\n');
     const results = lines.reduce(function (prev: number[], cur, i) {
-      if (cur.toLowerCase().includes(deferredSearchTerm.toLowerCase())) {
+      if (cur?.toLowerCase().includes(deferredSearchTerm?.toLowerCase())) {
         prev.push(i);
       }
       return prev;
diff --git a/client/src/utils/index.ts b/client/src/utils/index.ts
@@ -79,7 +79,7 @@ export const getFileExtensionForLang = (lang: string, lowercased?: boolean) => {
   // @ts-ignore
   let ext = langs[lang]?.[0];
   if (lowercased) {
-    const key = Object.keys(langs).find((key) => key.toLowerCase() === lang);
+    const key = Object.keys(langs).find((key) => key?.toLowerCase() === lang);
     if (key) {
       // @ts-ignore
       ext = langs[key]?.[0];
@@ -97,7 +97,7 @@ export const getPrettyLangName = (lang: string) => {
     case 'tsx':
       return 'TypeScript';
     default:
-      return Object.keys(langs).find((key) => key.toLowerCase() === lang);
+      return Object.keys(langs).find((key) => key?.toLowerCase() === lang);
   }
 };
 
diff --git a/client/src/utils/keyboardUtils.ts b/client/src/utils/keyboardUtils.ts
@@ -21,7 +21,7 @@ export const checkEventKeys = (e: KeyboardEvent, shortcut?: string[]) => {
   if (!shortcut) {
     return false;
   }
-  const keys = shortcut.map((k) => k.toLowerCase());
+  const keys = shortcut.map((k) => k?.toLowerCase());
   if (
     (keys.includes('cmd') && !(e.metaKey || e.ctrlKey)) ||
     (!keys.includes('cmd') && (e.metaKey || e.ctrlKey))
@@ -53,9 +53,9 @@ export const checkEventKeys = (e: KeyboardEvent, shortcut?: string[]) => {
     if (
       (e.altKey &&
         keys.includes(
-          e.code.replace('Key', '').replace('Digit', '').toLowerCase(),
+          e.code.replace('Key', '').replace('Digit', '')?.toLowerCase(),
         )) ||
-      (e.shiftKey && keys.includes(e.code.replace('Digit', '').toLowerCase()))
+      (e.shiftKey && keys.includes(e.code.replace('Digit', '')?.toLowerCase()))
     ) {
       return true;
     }
diff --git a/client/src/utils/mappers.ts b/client/src/utils/mappers.ts
@@ -44,19 +44,19 @@ export const mapReposBySections = (githubRepos: RepoType[]) => {
         ? 1
         : b === 'Local'
         ? -1
-        : a.toLowerCase() < b.toLowerCase()
+        : a?.toLowerCase() < b?.toLowerCase()
         ? -1
         : 1,
     )
     .forEach((k) => {
       result.push({
         org: k,
         items: byOrg[k].sort((a, b) =>
-          a.folderName.toLowerCase() < b.folderName.toLowerCase()
+          a.folderName?.toLowerCase() < b.folderName?.toLowerCase()
             ? -1
-            : a.folderName.toLowerCase() > b.folderName.toLowerCase()
+            : a.folderName?.toLowerCase() > b.folderName?.toLowerCase()
             ? 1
-            : a.shortName.toLowerCase() < b.shortName.toLowerCase()
+            : a.shortName?.toLowerCase() < b.shortName?.toLowerCase()
             ? -1
             : 1,
         ),

Original file line number	Diff line number	Diff line change
`@@ -178,7 +178,7 @@ const ConversationInput = ({`
`178`	`178`	`.filter(`
`179`	`179`	`(t): t is FileTabType =>`
`180`	`180`	`t.type === TabTypesEnum.FILE &&`
`181`		`- (!search \|\| t.path.toLowerCase().includes(search.toLowerCase())),`
	`181`	`+ (!search \|\| t.path?.toLowerCase().includes(search?.toLowerCase())),`
`182`	`182`	`)`
`183`	`183`	`.map((t) => ({ path: t.path, repo: t.repoRef }));`
`184`	`184`	`filesResults.push(`
Original file line number	Diff line number	Diff line change
`@@ -72,7 +72,7 @@ const RepoNav = ({`
`72`	`72`	`}`
`73`	`73`	`return resp?.entries.sort((a, b) => {`
`74`	`74`	`if ((a.entry_data === 'Directory') === (b.entry_data === 'Directory')) {`
`75`		`- return a.name.toLowerCase() < b.name.toLowerCase() ? -1 : 1;`
	`75`	`+ return a.name?.toLowerCase() < b.name?.toLowerCase() ? -1 : 1;`
`76`	`76`	`} else {`
`77`	`77`	`return a.entry_data === 'Directory' ? -1 : 1;`
`78`	`78`	`}`