⚡ perf: avoid unnecessary operations

Rettend · Jan 23, 2024 · 72083b6 · 72083b6
1 parent 2ca9fdf
commit 72083b6
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/src/commands/run.ts b/src/commands/run.ts
@@ -22,7 +22,7 @@ export default defineCommand({
     },
     DEBUG: {
       type: 'string',
-      description: 'Debug level (default: 0)',
+      description: 'Debug level (0-2)',
       alias: 'd',
       default: '0',
     },
@@ -67,7 +67,6 @@ export default defineCommand({
       }
       else {
         commitMessage = fs.readFileSync(ctx.args.commit_file, 'utf-8')
-
         commitMessage = commitMessage.split('\n')[0] ?? ''
 
         const newCommitMessage = eemojify(commitMessage, config, DEBUG)

diff --git a/src/utils/emoji.ts b/src/utils/emoji.ts
@@ -37,6 +37,7 @@ export function eemojify(text: string, config: Config, DEBUG?: number): string {
     throw new Error('Invalid commit message.')
 
   let emoji = getEmoji(type, text, config, DEBUG)
+  consola.log('emoji')
 
   if (DEBUG)
     consola.log(`emoji: "${emoji}"`)
@@ -57,7 +58,8 @@ function getEmoji(type: string, text: string, config: Config, DEBUG?: number): s
   if (text.includes('!') && config.emojis.breaking)
     return config.emojis.breaking
 
-  const emojiKey = Object.keys(config.emojis).find(key => key.split('|').includes(type.toLowerCase().replace(/\(.*\)/, '').trim()))
+  type = type.toLowerCase().replace(/\(.*\)/, '').trim()
+  const emojiKey = Object.keys(config.emojis).find(key => key.split('|').includes(type))
 
   if (!emojiKey)
     return undefined
@@ -69,9 +71,10 @@ function getEmoji(type: string, text: string, config: Config, DEBUG?: number): s
       consola.log(`nested emoji: ${JSON.stringify(emoji)}`)
 
     const entries = Object.entries(emoji).filter(([key]) => key !== '.')
+    text = text.toLowerCase()
 
     for (const [key, value] of entries) {
-      if (key.split('|').some(k => text.toLowerCase().includes(k.toLowerCase())))
+      if (key.split('|').some(k => text.includes(k.toLowerCase())))
         return value
     }