feat: cover dynamic values in weak prop codemod and simplify it

.prettierignore

@@ -1 +1,2 @@
+codemods/__testfixtures__
 package-lock.json

src/codemods/__testfixtures__/weak-to-secondary/dynamic-value.input.tsx

@@ -0,0 +1,7 @@
+import { Text } from '@freenow/wave';
+
+export const TextTest = () => (
+    <Text weak={Date.now() % 2 === 0} fontSize={1}>
+        Just a text
+    </Text>
+);

src/codemods/__testfixtures__/weak-to-secondary/dynamic-value.output.tsx

@@ -0,0 +1,7 @@
+import { Text } from '@freenow/wave';
+
+export const TextTest = () => (
+    <Text secondary={Date.now() % 2 === 0} fontSize={1}>
+        Just a text
+    </Text>
+);

src/codemods/__tests__/weak-to-secondary-test.ts

@@ -1,7 +1,7 @@
 jest.autoMockOff();
 const { defineTest } = require('jscodeshift/dist/testUtils');
 
-const tests = ['basic-usage', 'local-rename', 'boolean-true', 'boolean-false'];
+const tests = ['basic-usage', 'local-rename', 'boolean-true', 'boolean-false', 'dynamic-value'];
 
 describe('weak-to-secondary', () => {
     tests.forEach(test =>

src/codemods/weak-to-secondary.ts

@@ -1,4 +1,4 @@
-import { API, FileInfo } from 'jscodeshift';
+import { API, ASTPath, FileInfo, JSXIdentifier } from 'jscodeshift';
 import { Options } from 'recast';
 
 module.exports = (file: FileInfo, api: API, options: Options) => {
@@ -38,22 +38,28 @@ module.exports = (file: FileInfo, api: API, options: Options) => {
             .forEach(attr => {
                 const mutableAttr = j(attr);
 
-                // In case it's implicitly true (<Text weak>)
-                if (!attr.node.value) {
-                    // Replace with secondary prop
-                    mutableAttr.replaceWith(secondaryProp);
-                }
+                // Find the identifier (where the prop name is kept)
+                const identifier: ASTPath<JSXIdentifier> = mutableAttr
+                    .find(j.JSXIdentifier, {
+                        name: 'weak'
+                    })
+                    .get(0).node;
 
-                // In case it has a value (weak={false} or weak={true})
+                // In case it has a boolean value (weak={false} or weak={true})
                 if (
                     attr.node.value?.type === 'JSXExpressionContainer' &&
                     attr.node.value.expression.type === 'BooleanLiteral'
                 ) {
                     // If weak={true} replace with secondary prop
                     if (attr.node.value.expression.value) mutableAttr.replaceWith(secondaryProp);
-                    // Else (weak={false}) remove altogether
+                    // Otherwise (weak={false}) remove altogether
                     else mutableAttr.remove();
+                    return;
                 }
+
+                // For other cases, e.g. implicit value (weak), dynamic value (weak={Date.now() % 2 === 0})
+                // Replace the name of the prop
+                identifier.name = 'secondary';
             });
     });