fix(cleanupAttrs): handle numeric attribute values (#2219)

[Zhang-Wei-666]

Fixes #2219.

cleanupAttrs assumes attribute values are strings and crashes when an upstream step assigns numeric values such as 0 to width or height.

This change coerces non-string attribute values to strings before applying whitespace cleanup and adds a regression test covering numeric width / height values on <svg> elements without a viewBox.

[Copilot]

Pull request overview

Fixes a crash in the cleanupAttrs plugin when upstream plugins assign non-string attribute values (e.g., numeric width/height), addressing issue #2219 and adding a regression test to cover the scenario.

Changes:

• Update cleanupAttrs to coerce non-string attribute values to strings before applying newline/trim/space cleanup.
• Add a regression test ensuring cleanupAttrs doesn’t crash when width/height are numeric.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
plugins/cleanupAttrs.js	Coerces attribute values to strings before running whitespace cleanup logic.
lib/svgo.test.js	Adds a regression test for numeric width/height values passed through cleanupAttrs.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[KTibow]

Converting to draft because it's unclear how to reproduce #2219, and artificially making node.attributes.width = 0 is the wrong kind of reproduction to use in a test.