fixed typed code coverage

Author: cobbvanth

lib/index.js

@@ -289,61 +289,99 @@ function transform(state, node) {
       }
 
       case 'mdxJsxFlowElement': {
-        // Map the attributes array into name/value pairs
+        /**
+         * A typedef for the MDX node shape.
+         * (Adjust property types as needed for your use case.)
+         *
+         * @typedef {Object} MdxJsxFlowElement
+         * @property {string} name
+         * @property {Record<string, unknown>} [properties]
+         * @property {Array<{
+         *   type: 'mdxJsxAttribute',
+         *   name: string,
+         *   value: unknown
+         * }>} [attributes]
+         */
+
         /** @type {Record<string, unknown>} */
         const attributesMap = {}
 
-        if (Array.isArray(unsafe.attributes)) {
-          unsafe.attributes.forEach((attr) => {
+        /**
+         * Cast `unsafe` to our `MdxJsxFlowElement`.
+         * If `unsafe` truly may be something else, you may want extra runtime checks.
+         *
+         * @type {MdxJsxFlowElement}
+         */
+        const typedUnsafe = /** @type {MdxJsxFlowElement} */ (unsafe)
+
+        if (Array.isArray(typedUnsafe.attributes)) {
+          for (const attribute of typedUnsafe.attributes) {
             if (
-              attr &&
-              typeof attr === 'object' &&
-              'name' in attr &&
-              'value' in attr
+              attribute &&
+              typeof attribute === 'object' &&
+              'name' in attribute &&
+              'value' in attribute
             ) {
-              attributesMap[attr.name] = attr.value
+              attributesMap[attribute.name] = attribute.value
             }
-          })
+          }
         }
 
-        const mdxProps = {
-          ...unsafe,
-          tagName: unsafe.name,
+        /**
+         * Here we define `mdxProps` to include everything from `typedUnsafe`
+         * plus a new `tagName` field.
+         *
+         * @typedef {MdxJsxFlowElement & {
+         *   tagName: string,
+         * }} MdxProps
+         */
+
+        /** @type {MdxProps} */
+        const mdxProperties = {
+          ...typedUnsafe,
+          tagName: typedUnsafe.name,
           properties: {
-            ...unsafe.properties,
-            ...attributesMap // Spread the mapped attributes directly into properties
+            ...typedUnsafe.properties,
+            ...attributesMap
           }
         }
 
-        return element(state, mdxProps)
+        return element(state, mdxProperties)
       }
 
       case 'mdxJsxTextElement': {
         /** @type {Record<string, unknown>} */
         const attributesMap = {}
-        if (Array.isArray(unsafe.attributes)) {
-          unsafe.attributes.forEach((attr) => {
+
+        /** @type {{ attributes?: Array<{ type: 'mdxJsxAttribute', name: string, value: unknown }> }} */
+        const typedUnsafe = unsafe
+
+        if (Array.isArray(typedUnsafe.attributes)) {
+          for (const attribute of typedUnsafe.attributes) {
             if (
-              attr &&
-              typeof attr === 'object' &&
-              'name' in attr &&
-              'value' in attr
+              attribute &&
+              typeof attribute === 'object' &&
+              'name' in attribute &&
+              'value' in attribute
             ) {
-              attributesMap[attr.name] = attr.value
+              attributesMap[attribute.name] = attribute.value
             }
-          })
+          }
         }
 
-        const mdxProps = {
-          ...unsafe,
-          tagName: unsafe.name,
+        const mdxProperties = {
+          .../** @type {{ name: string, properties?: Record<string, unknown> }} */ (
+            unsafe
+          ),
+          tagName: /** @type {{ name: string }} */ (unsafe).name,
           properties: {
-            ...unsafe.properties,
+            .../** @type {{ properties?: Record<string, unknown> }} */ (unsafe)
+              .properties,
             ...attributesMap
           }
         }
 
-        return element(state, mdxProps)
+        return element(state, mdxProperties)
       }
 
       default: