From 6595993ee897a10c2ddcea31f4a414fccd0cefa6 Mon Sep 17 00:00:00 2001
From: Bart Veneman <bart.veneman@drukwerkdeal.nl>
Date: Sun, 22 Jun 2025 20:39:18 +0200
Subject: [PATCH] feat: analyze types of atrules composition

---
 src/atrules/atrules.test.js |  12 ++++-
 src/index.js                | 105 ++++++++++++++++++------------------
 src/index.test.js           |   5 +-
 3 files changed, 68 insertions(+), 54 deletions(-)

diff --git a/src/atrules/atrules.test.js b/src/atrules/atrules.test.js
index 240dee4..86e4f5a 100644
--- a/src/atrules/atrules.test.js
+++ b/src/atrules/atrules.test.js
@@ -18,8 +18,16 @@ AtRules('counts total atrules', () => {
       a {}
     }
   `
-  let actual = analyze(css).atrules.total
-  assert.is(actual, 4)
+  let actual = analyze(css).atrules
+  assert.is(actual.total, 4)
+  assert.is(actual.totalUnique, 4)
+  assert.equal(actual.unique, {
+    'import': 1,
+    'layer': 1,
+    'media': 1,
+    'supports': 1,
+  })
+  assert.equal(actual.uniquenessRatio, 4 / 4)
 })
 
 AtRules('calculates complexity', () => {
diff --git a/src/index.js b/src/index.js
index 12122de..21f19a8 100644
--- a/src/index.js
+++ b/src/index.js
@@ -110,7 +110,7 @@ export function analyze(css, options = {}) {
   let linesOfCode = ast.loc.end.line - ast.loc.start.line + 1
 
   // Atrules
-  let totalAtRules = 0
+  let atrules = new Collection(useLocations)
   let atRuleComplexities = new AggregateCollection()
   /** @type {Record<string, string>[]} */
   let fontfaces = []
@@ -209,7 +209,7 @@ export function analyze(css, options = {}) {
     enter(node) {
       switch (node.type) {
         case Atrule: {
-          totalAtRules++
+          atrules.p(node.name, node.loc)
           atruleNesting.push(nestingDepth)
           uniqueAtruleNesting.p(nestingDepth, node.loc)
 
@@ -795,10 +795,11 @@ export function analyze(css, options = {}) {
   let declarationComplexity = declarationComplexities.aggregate()
   let propertyComplexity = propertyComplexities.aggregate()
   let valueComplexity = valueComplexities.aggregate()
+  let atruleCount = atrules.c()
 
   return {
     stylesheet: {
-      sourceLinesOfCode: totalAtRules + totalSelectors + totalDeclarations + keyframeSelectors.size(),
+      sourceLinesOfCode: atruleCount.total + totalSelectors + totalDeclarations + keyframeSelectors.size(),
       linesOfCode,
       size: cssLen,
       complexity: atRuleComplexity.sum + selectorComplexity.sum + declarationComplexity.sum + propertyComplexity.sum + valueComplexity.sum,
@@ -819,55 +820,57 @@ export function analyze(css, options = {}) {
         },
       },
     },
-    atrules: {
-      fontface: assign({
-        total: fontFacesCount,
-        totalUnique: fontFacesCount,
-        unique: fontfaces,
-        uniquenessRatio: fontFacesCount === 0 ? 0 : 1,
-      }, useLocations ? {
-        uniqueWithLocations: fontfaces_with_loc.c().uniqueWithLocations,
-      } : {}),
-      import: imports.c(),
-      media: assign(
-        medias.c(),
-        {
-          browserhacks: mediaBrowserhacks.c(),
-          features: mediaFeatures.c(),
-        }
-      ),
-      charset: charsets.c(),
-      supports: assign(
-        supports.c(),
-        {
-          browserhacks: supportsBrowserhacks.c(),
-        },
-      ),
-      keyframes: assign(
-        keyframes.c(), {
-        prefixed: assign(
-          prefixedKeyframes.c(), {
-          ratio: ratio(prefixedKeyframes.size(), keyframes.size())
+    atrules: assign(
+      atruleCount,
+      {
+        fontface: assign({
+          total: fontFacesCount,
+          totalUnique: fontFacesCount,
+          unique: fontfaces,
+          uniquenessRatio: fontFacesCount === 0 ? 0 : 1,
+        }, useLocations ? {
+          uniqueWithLocations: fontfaces_with_loc.c().uniqueWithLocations,
+        } : {}),
+        import: imports.c(),
+        media: assign(
+          medias.c(),
+          {
+            browserhacks: mediaBrowserhacks.c(),
+            features: mediaFeatures.c(),
+          }
+        ),
+        charset: charsets.c(),
+        supports: assign(
+          supports.c(),
+          {
+            browserhacks: supportsBrowserhacks.c(),
+          },
+        ),
+        keyframes: assign(
+          keyframes.c(), {
+          prefixed: assign(
+            prefixedKeyframes.c(), {
+            ratio: ratio(prefixedKeyframes.size(), keyframes.size())
+          }),
         }),
-      }),
-      container: assign(
-        containers.c(),
-        {
-          names: containerNames.c(),
-        }
-      ),
-      layer: layers.c(),
-      property: registeredProperties.c(),
-      total: totalAtRules,
-      complexity: atRuleComplexity,
-      nesting: assign(
-        atruleNesting.aggregate(),
-        {
-          items: atruleNesting.toArray(),
-        },
-        uniqueAtruleNesting.c(),
-      ),
-    },
+        container: assign(
+          containers.c(),
+          {
+            names: containerNames.c(),
+          }
+        ),
+        layer: layers.c(),
+        property: registeredProperties.c(),
+        complexity: atRuleComplexity,
+        nesting: assign(
+          atruleNesting.aggregate(),
+          {
+            items: atruleNesting.toArray(),
+          },
+          uniqueAtruleNesting.c(),
+        ),
+      }
+    ),
     rules: {
       total: totalRules,
       empty: {
diff --git a/src/index.test.js b/src/index.test.js
index c7111a6..19f053c 100644
--- a/src/index.test.js
+++ b/src/index.test.js
@@ -87,6 +87,10 @@ Api("handles empty input gracefully", () => {
       complexity: 0,
     },
     atrules: {
+      total: 0,
+      totalUnique: 0,
+      unique: {},
+      uniquenessRatio: 0,
       fontface: {
         total: 0,
         totalUnique: 0,
@@ -172,7 +176,6 @@ Api("handles empty input gracefully", () => {
         unique: {},
         uniquenessRatio: 0,
       },
-      total: 0,
       complexity: {
         min: 0,
         max: 0,