Apply suggestions from code review

Co-authored-by: Phil Cohen <phillip@phillip.io>
cursorless-dev · Feb 4, 2025 · 54ea777 · 54ea777
1 parent 611302f
commit 54ea777
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/packages/cursorless-vscode/src/migrateSnippets.ts b/packages/cursorless-vscode/src/migrateSnippets.ts
@@ -140,18 +140,15 @@ function parseVariables(
         name,
         wrapperPhrases: phrase ? [phrase] : undefined,
         wrapperScope: variable.wrapperScopeType,
-        insertionFormatters: getFormatter(variable.formatter),
+        insertionFormatters: formatter ? getFormatter(variable.formatter) : undefined,
         // SKIP: variable.description
       };
     },
   );
 }
 
 // Convert Cursorless formatters to Talon community formatters
-function getFormatter(formatter?: string): string[] | undefined {
-  if (!formatter) {
-    return undefined;
-  }
+function getFormatter(formatter: string): string[] {
   switch (formatter) {
     case "camelCase":
       return ["PRIVATE_CAMEL_CASE"];