[Bug] The SVGs in /build/svg/ don't have the `octicon` classes attached

[fregante]

Describe the bug

The classes:

• ✅ show up in the SVG generated by toSVG(): https://github.com/primer/octicons/tree/main/lib/octicons_node#class
• ✅ show up in the React components (line 65) https://www.unpkg.com/browse/@primer/octicons-react@19.8.0/dist/index.esm.js
• ❌ but they don't appear in the static SVGs: https://www.unpkg.com/@primer/octicons@19.8.0/build/svg/alert-16.svg

Expected behavior

Could you include the octicon octicon-something classes in the built SVGs?

I only need this in @primer/octicons, but the issue also appears in @primer/octicons-react (however I don't understand the point of the built SVG files in that package since they're not allowed by the exports map)

Context

I want to import the raw SVGs in my extensions

• https://github.com/refined-github/refined-github/issues/6878
• https://github.com/refined-github/refined-github/pull/7152

[fregante]

So this is because the icons are plainly copied from the ./icons folder, which understandably doesn't have any extraneous attributes.

This issue doesn't affect me anymore though, I'm using https://github.com/fregante/octicons-plain-react

[github-actions[bot]]

Hi! This issue has been marked as stale because it has been open with no activity for 180 days. You can comment on the issue or remove the stale label to keep it open. If you do nothing, this issue will be closed in 7 days.