refactor(util): simplify snakeToCamel

src/goods.ts

@@ -42,9 +42,7 @@ export const parseArgv = (
 ): minimist.ParsedArgs =>
   Object.entries(minimist(args, opts)).reduce<minimist.ParsedArgs>(
     (m, [k, v]) => {
-      const kTrans = opts.camelCase
-        ? (k: string) => snakeToCamel(k.replace(/-/, '_'))
-        : identity
+      const kTrans = opts.camelCase ? snakeToCamel : identity
       const vTrans = opts.parseBoolean ? parseBool : identity
       const [_k, _v] = k === '--' || k === '_' ? [k, v] : [kTrans(k), vTrans(v)]
       m[_k] = _v

src/util.ts

@@ -286,20 +286,16 @@ export const proxyOverride = <T extends object>(
     },
   }) as T
 
-// https://stackoverflow.com/a/7888303
 export const camelToSnake = (str: string) =>
   str
     .split(/(?=[A-Z])/)
     .map((s) => s.toUpperCase())
     .join('_')
 
-// https://stackoverflow.com/a/61375162
 export const snakeToCamel = (str: string) =>
-  str
-    .toLowerCase()
-    .replace(/([-_][a-z])/g, (group) =>
-      group.toUpperCase().replace('-', '').replace('_', '')
-    )
+  str.toLowerCase().replace(/([a-z])[_-]+([a-z])/g, (_, p1, p2) => {
+    return p1 + p2.toUpperCase()
+  })
 
 export const parseBool = (v: string): boolean | string =>
   ({ true: true, false: false })[v] ?? v

test/util.test.js

@@ -145,5 +145,6 @@ describe('util', () => {
     assert.equal(snakeToCamel('NOTHROW'), 'nothrow')
     assert.equal(snakeToCamel('PREFER_LOCAL'), 'preferLocal')
     assert.equal(snakeToCamel('SOME_MORE_BIG_STR'), 'someMoreBigStr')
+    assert.equal(snakeToCamel('kebab-input-str'), 'kebabInputStr')
   })
 })