fix(plugin-svgo): handle potential errors from optimize (#663)

Author: mikaelbr

packages/plugin-svgo/src/index.test.ts

@@ -51,6 +51,21 @@ describe('svgo', () => {
     expect(result).toMatchSnapshot()
   })
 
+  it('throws error on invalid svg input', () => {
+    const errorSvg = `<?xml version="1.0" encoding="UTF-8"?>
+  <svg width="88px" height="88px" viewBox="0 0 88 88" version="1.1" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink">
+    <path d=M51,37 L37,51" id="Shape" class="shape"></path>
+  </svg>`
+
+    expect(() =>
+      svgo(
+        errorSvg,
+        { svgo: true, runtimeConfig: true },
+        { ...state, filePath: path.join(__dirname, '../__fixtures__/svgo') },
+      ),
+    ).toThrowError()
+  })
+
   it('uses `state.filePath` to detect configuration', () => {
     const result = svgo(
       baseSvg,

packages/plugin-svgo/src/index.ts

@@ -5,8 +5,13 @@ import type { Plugin } from '@svgr/core'
 const svgoPlugin: Plugin = (code, config, state) => {
   if (!config.svgo) return code
   const svgoConfig = getSvgoConfig(config, state)
-  const { data } = optimize(code, { ...svgoConfig, path: state.filePath })
-  return data
+  const result = optimize(code, { ...svgoConfig, path: state.filePath })
+
+  if (result.modernError) {
+    throw result.modernError
+  }
+
+  return result.data
 }
 
 export default svgoPlugin