fix(smart-rename): name of useRef should not need pascalCase

pionxzh · Jul 6, 2024 · 0e586d5 · 0e586d5
1 parent 548f121
commit 0e586d5
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 5 deletions.
diff --git a/packages/ast-utils/src/case.ts b/packages/ast-utils/src/case.ts
@@ -1,5 +1,10 @@
 /**
  * PascalCase
+ *
+ * @example
+ * pascalCase('foo') // Foo
+ * pascalCase('foo-bar') // FooBar
+ * pascalCase('foo_bar') // FooBar
  */
 export function pascalCase(str: string): string {
     return str.replace(/(?:^|[-_])(\w)/g, (_, c) => c.toUpperCase())

diff --git a/packages/unminify/src/transformations/__tests__/smart-rename.spec.ts b/packages/unminify/src/transformations/__tests__/smart-rename.spec.ts
@@ -236,14 +236,14 @@ const d = useRef();
 const ef = o.useRef(null);
 
 const g = o.useRef(a, b); // invalid parameters
-const ButtonRef = o.useRef(null); // name is not minified
+const buttonRef = o.useRef(null); // name is not minified
 `,
   `
-const DRef = useRef();
-const EfRef = o.useRef(null);
+const dRef = useRef();
+const efRef = o.useRef(null);
 
 const g = o.useRef(a, b); // invalid parameters
-const ButtonRef = o.useRef(null); // name is not minified
+const buttonRef = o.useRef(null); // name is not minified
 `,
 )
 

diff --git a/packages/unminify/src/transformations/smart-rename.ts b/packages/unminify/src/transformations/smart-rename.ts
@@ -202,7 +202,7 @@ function handleReactRename(j: JSCodeshift, root: Collection) {
 
             // rename the identifier
             const oldName = id.name
-            const newName = generateName(`${pascalCase(oldName)}Ref`, scope)
+            const newName = generateName(`${oldName}Ref`, scope)
             renameIdentifier(j, scope, oldName, newName)
         })