fix(googleicons): sort icons for glyph subsetting (#343)

Author: florian-lefebvre

src/providers/googleicons.ts

@@ -45,7 +45,7 @@ export default defineFontProvider('googleicons', async (providerOptions: Googlei
 
   async function getFontDetails(family: string, options: ResolveFontOptions<GoogleiconsFamilyOptions>) {
     // Google Icons require sorted icon names, or we will see a 400 error
-    const iconNames = (options.options?.experimental?.glyphs ?? providerOptions.experimental?.glyphs?.[family])?.join('')
+    const iconNames = (options.options?.experimental?.glyphs ?? providerOptions.experimental?.glyphs?.[family])?.toSorted().join(',')
 
     let css = ''
 

test/providers/googleicons.test.ts

@@ -52,7 +52,7 @@ describe('googleicons', () => {
     const unifont = await createUnifont([providers.googleicons({
       experimental: {
         glyphs: {
-          'Material Symbols Outlined': ['arrow_right', 'favorite', 'arrow_drop_down'],
+          'Material Symbols Outlined': ['bar_chart', 'bolt'],
         },
       },
     })])
@@ -78,8 +78,8 @@ describe('googleicons', () => {
           {
             "format": "woff2",
             "identifier": {
-              "kit": "",
-              "skey": "",
+              "kit": "kJEhBvYX7BgnkSrUwT8OhrdQw4oELdPIeeII9v6oFsLjBuVYLAPbrM4kWKE8mcwLO8rfPSFqKIVE",
+              "skey": "b8dc2088854b122f",
             },
           },
         ],
@@ -97,7 +97,7 @@ describe('googleicons', () => {
       options: {
         googleicons: {
           experimental: {
-            glyphs: ['arrow_right', 'favorite', 'arrow_drop_down'],
+            glyphs: ['bar_chart', 'bolt'],
           },
         },
       },
@@ -118,8 +118,8 @@ describe('googleicons', () => {
           {
             "format": "woff2",
             "identifier": {
-              "kit": "",
-              "skey": "",
+              "kit": "kJEhBvYX7BgnkSrUwT8OhrdQw4oELdPIeeII9v6oFsLjBuVYLAPbrM4kWKE8mcwLO8rfPSFqKIVE",
+              "skey": "b8dc2088854b122f",
             },
           },
         ],

tsconfig.json

@@ -1,8 +1,8 @@
 {
   "compilerOptions": {
-    "target": "es2022",
+    "target": "es2023",
     "lib": [
-      "es2022"
+      "es2023"
     ],
     "moduleDetection": "force",
     "module": "preserve",