Add case

Author: owjs3901

src/commands/devup/__tests__/index.test.ts

@@ -370,6 +370,56 @@ describe('devup commands', () => {
     expect(Array.isArray(parsed.theme?.typography?.heading)).toBe(true)
   })
 
+  test('exportDevup filters out empty typography entries', async () => {
+    getColorCollectionSpy = spyOn(
+      getColorCollectionModule,
+      'getDevupColorCollection',
+    ).mockResolvedValue(null)
+    styleNameToTypographySpy = spyOn(
+      styleNameToTypographyModule,
+      'styleNameToTypography',
+    ).mockReturnValue({ level: 0, name: 'heading' })
+    textSegmentToTypographySpy = spyOn(
+      textSegmentToTypographyModule,
+      'textSegmentToTypography',
+    ).mockReturnValue(null)
+
+    ;(globalThis as { figma?: unknown }).figma = {
+      util: { rgba: (v: unknown) => v },
+      variables: {},
+      loadAllPagesAsync: async () => {},
+      getLocalTextStylesAsync: async () => [
+        {
+          id: 'id',
+          name: 'heading/1',
+          fontName: { family: 'Inter', style: 'Regular' },
+        } as unknown as TextStyle,
+      ],
+      root: {
+        findAllWithCriteria: () => [
+          {
+            textStyleId: 'id',
+            getStyledTextSegments: () => [{ textStyleId: 'id' }],
+          } as unknown as TextNode,
+        ],
+      },
+      getStyleByIdAsync: async () =>
+        ({
+          id: 'id',
+          name: 'heading/1',
+          fontName: { family: 'Inter', style: 'Regular' },
+        }) as unknown as TextStyle,
+    } as unknown as typeof figma
+
+    await exportDevup('json')
+
+    const payload = JSON.parse(
+      (downloadFileMock.mock.calls[0] as unknown as { args: [string, string] })
+        ?.args?.[1] ?? '{}',
+    )
+    expect(payload.theme?.typography).toBeUndefined()
+  })
+
   test('importDevup creates colors and typography from json', async () => {
     getColorCollectionSpy = spyOn(
       getColorCollectionModule,

src/commands/devup/index.ts

@@ -120,27 +120,20 @@ export async function exportDevup(
     devup.theme.typography = Object.entries(typography).reduce(
       (acc, [key, value]) => {
         const filtered = value.filter((v) => v !== null)
-        if (filtered.length === 0) return acc
+        if (filtered.length === 0) {
+          return acc
+        }
         if (filtered.length === 1) {
           acc[key] = filtered[0]
           return acc
         }
         if (value[0] === null) {
           acc[key] = [filtered[0]]
-          let init = false
-          for (let i = 0; i < value.length; i += 1) {
-            if (value[i] === null) {
-              if (init) {
-                acc[key].push(null)
-              } else {
-                if (!init) {
-                  acc[key].push(null)
-                  init = true
-                } else {
-                  acc[key].push(value[i])
-                }
-              }
-            }
+          for (let i = 1; i < value.length; i += 1) {
+            acc[key].push(value[i])
+          }
+          while (acc[key][acc[key].length - 1] === null) {
+            acc[key].pop()
           }
           return acc
         }