From 8d05ade28b7be7c99555598fec22526c1bff603d Mon Sep 17 00:00:00 2001
From: Seth Falco <seth@falco.fun>
Date: Sun, 1 Oct 2023 12:56:27 +0100
Subject: [PATCH] fix(removeDesc): removeAny should be disabled by default

---
 lib/svgo-node.test.js | 12 ++++++------
 lib/svgo.test.js      | 19 ++++++++-----------
 plugins/removeDesc.js |  2 +-
 3 files changed, 15 insertions(+), 18 deletions(-)

diff --git a/lib/svgo-node.test.js b/lib/svgo-node.test.js
index 2f0d077b5..76972e7bc 100644
--- a/lib/svgo-node.test.js
+++ b/lib/svgo-node.test.js
@@ -17,7 +17,7 @@ describeLF('with LF line-endings', () => {
       <?xml version="1.0" encoding="utf-8"?>
       <svg viewBox="0 0 120 120">
         <desc>
-          Not standard description
+          Created with love
         </desc>
         <circle fill="#ff0000" cx="60" cy="60" r="50"/>
       </svg>
@@ -34,7 +34,7 @@ describeLF('with LF line-endings', () => {
       <?xml version="1.0" encoding="utf-8"?>
       <svg viewBox="0 0 120 120">
         <desc>
-          Not standard description
+          Created with love
         </desc>
         <circle fill="#ff0000" cx="60" cy="60" r="50"/>
       </svg>
@@ -53,7 +53,7 @@ describeLF('with LF line-endings', () => {
       <?xml version="1.0" encoding="utf-8"?>
       <svg viewBox="0 0 120 120">
         <desc>
-          Not standard description
+          Created with love
         </desc>
         <circle fill="#ff0000" cx="60" cy="60" r="50"/>
       </svg>
@@ -74,7 +74,7 @@ describeCRLF('with CRLF line-endings', () => {
       <?xml version="1.0" encoding="utf-8"?>
       <svg viewBox="0 0 120 120">
         <desc>
-          Not standard description
+          Created with love
         </desc>
         <circle fill="#ff0000" cx="60" cy="60" r="50"/>
       </svg>
@@ -91,7 +91,7 @@ describeCRLF('with CRLF line-endings', () => {
       <?xml version="1.0" encoding="utf-8"?>
       <svg viewBox="0 0 120 120">
         <desc>
-          Not standard description
+          Created with love
         </desc>
         <circle fill="#ff0000" cx="60" cy="60" r="50"/>
       </svg>
@@ -110,7 +110,7 @@ describeCRLF('with CRLF line-endings', () => {
       <?xml version="1.0" encoding="utf-8"?>
       <svg viewBox="0 0 120 120">
         <desc>
-          Not standard description
+          Created with love
         </desc>
         <circle fill="#ff0000" cx="60" cy="60" r="50"/>
       </svg>
diff --git a/lib/svgo.test.js b/lib/svgo.test.js
index a1570b35e..4ac94b3cd 100644
--- a/lib/svgo.test.js
+++ b/lib/svgo.test.js
@@ -11,7 +11,7 @@ test('allow to setup default preset', () => {
     <?xml version="1.0" encoding="utf-8"?>
     <svg viewBox="0 0 120 120">
       <desc>
-        Not standard description
+        Created with love
       </desc>
       <circle fill="#ff0000" cx="60" cy="60" r="50"/>
     </svg>
@@ -46,7 +46,7 @@ test('allow to disable and customize plugins in preset', () => {
           overrides: {
             removeXMLProcInst: false,
             removeDesc: {
-              removeAny: false,
+              removeAny: true,
             },
           },
         },
@@ -57,9 +57,6 @@ test('allow to disable and customize plugins in preset', () => {
   expect(data).toMatchInlineSnapshot(`
     "<?xml version="1.0" encoding="utf-8"?>
     <svg viewBox="0 0 120 120">
-      <desc>
-        Not standard description
-      </desc>
       <circle cx="60" cy="60" r="50" fill="red"/>
     </svg>
     "
@@ -104,7 +101,7 @@ describe('allow to configure EOL', () => {
       <?xml version="1.0" encoding="utf-8"?>
       <svg viewBox="0 0 120 120">
         <desc>
-          Not standard description
+          Created with love
         </desc>
         <circle fill="#ff0000" cx="60" cy="60" r="50"/>
       </svg>
@@ -123,7 +120,7 @@ describe('allow to configure EOL', () => {
       <?xml version="1.0" encoding="utf-8"?>
       <svg viewBox="0 0 120 120">
         <desc>
-          Not standard description
+          Created with love
         </desc>
         <circle fill="#ff0000" cx="60" cy="60" r="50"/>
       </svg>
@@ -142,7 +139,7 @@ describe('allow to configure EOL', () => {
       <?xml version="1.0" encoding="utf-8"?>
       <svg viewBox="0 0 120 120">
         <desc>
-          Not standard description
+          Created with love
         </desc>
         <circle cx="60" cy="60" fill="#ff0000" r="50"/>
       </svg>
@@ -163,7 +160,7 @@ describe('allow to configure final newline', () => {
       <?xml version="1.0" encoding="utf-8"?>
       <svg viewBox="0 0 120 120">
         <desc>
-          Not standard description
+          Created with love
         </desc>
         <circle cx="60" cy="60" r="50" fill="#ff0000"/>
       </svg>
@@ -180,7 +177,7 @@ describe('allow to configure final newline', () => {
       <?xml version="1.0" encoding="utf-8"?>
       <svg viewBox="0 0 120 120">
         <desc>
-          Not standard description
+          Created with love
         </desc>
         <circle fill="#ff0000" cx="60" cy="60" r="50"/>
       </svg>
@@ -199,7 +196,7 @@ describe('allow to configure final newline', () => {
       <?xml version="1.0" encoding="utf-8"?>
       <svg viewBox="0 0 120 120">
         <desc>
-          Not standard description
+          Created with love
         </desc>
         <circle fill="#ff0000" cx="60" cy="60" r="50"/>
       </svg>
diff --git a/plugins/removeDesc.js b/plugins/removeDesc.js
index 5d9e57aef..4520ed27c 100644
--- a/plugins/removeDesc.js
+++ b/plugins/removeDesc.js
@@ -19,7 +19,7 @@ const standardDescs = /^(Created with|Created using)/;
  * @type {import('./plugins-types').Plugin<'removeDesc'>}
  */
 exports.fn = (root, params) => {
-  const { removeAny = true } = params;
+  const { removeAny = false } = params;
   return {
     element: {
       enter: (node, parentNode) => {