refactor: optimize code

Author: chouchouji

src/utils.ts

@@ -1,4 +1,4 @@
-import { isArray, isEmpty } from 'rattail'
+import { isArray } from 'rattail'
 import type { Alias } from './types'
 
 /**
@@ -158,9 +158,7 @@ export function normalizeAliasesToArray<T>(value: T[] | undefined) {
  * @returns {string}
  */
 export function formatUnaliasCommand(aliases: Alias[]) {
-  return aliases.reduce((acc, alias) => {
-    return `${acc} ${alias.aliasName}`
-  }, 'unalias')
+  return aliases.reduce((acc, alias) => `${acc} ${alias.aliasName}`, 'unalias')
 }
 
 /**
@@ -204,27 +202,20 @@ export function mergeAlias(source: Record<string, Alias[]>, target: Record<strin
  * @returns {Alias[]}
  */
 export function filterAliases(content: string) {
-  if (isEmpty(content)) {
+  if (content.length === 0) {
     return []
   }
 
   const aliases = content
     .split('\n')
+    .map((text) => resolveAlias(text.trim()))
     .filter(Boolean)
-    .map((text) => text.trim())
-    .reduce((acc: Alias[], text) => {
-      const alias = resolveAlias(text)
-      if (alias) {
-        const { aliasName, command } = alias
-        acc.push({
-          aliasName,
-          command,
-          frequency: 0,
-          description: '',
-        })
-      }
-      return acc
-    }, [])
+    .map((alias) => ({
+      aliasName: alias!.aliasName,
+      command: alias!.command,
+      frequency: 0,
+      description: '',
+    }))
 
   return aliases
 }

tests/utils.spec.ts

@@ -174,8 +174,15 @@ describe('test all values are not equal to target', () => {
 })
 
 describe('test filter all alias from string', () => {
-  it('no any alias', () => {
-    expect(filterAliases(`al`)).toStrictEqual([])
+  it('no any valid alias', () => {
+    expect(
+      filterAliases(`
+
+        test
+test
+        # test   
+alias c='clear' test`),
+    ).toStrictEqual([])
   })
 
   it('all valid aliases', () => {