chore: deduplicate RTL warnings in uno-preset-rtl

Author: danielroe

test/unit/uno-preset-rtl.spec.ts

@@ -1,11 +1,12 @@
 import { afterEach, beforeEach, describe, expect, it, vi, type MockInstance } from 'vitest'
-import { presetRtl } from '../../uno-preset-rtl'
+import { presetRtl, resetRtlWarnings } from '../../uno-preset-rtl'
 import { createGenerator } from 'unocss'
 
 describe('uno-preset-rtl', () => {
   let warnSpy: MockInstance
 
   beforeEach(() => {
+    resetRtlWarnings()
     warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {})
   })
 

uno-preset-rtl.ts

@@ -1,6 +1,22 @@
 import type { CSSEntries, DynamicMatcher, Preset, RuleContext } from 'unocss'
 import { cornerMap, directionSize, h } from '@unocss/preset-wind4/utils'
 
+// Track warnings to avoid duplicates
+const warnedClasses = new Set<string>()
+
+function warnOnce(message: string, key: string) {
+  if (!warnedClasses.has(key)) {
+    warnedClasses.add(key)
+    // oxlint-disable-next-line no-console -- warn logging
+    console.warn(message)
+  }
+}
+
+/** Reset warning state (for testing) */
+export function resetRtlWarnings() {
+  warnedClasses.clear()
+}
+
 const directionMap: Record<string, string[]> = {
   'l': ['-left'],
   'r': ['-right'],
@@ -30,9 +46,9 @@ function directionSizeRTL(
     const defaultMap = { l: 'is', r: 'ie' }
     const map = prefixMap || defaultMap
     const replacement = map[direction as 'l' | 'r']
-    // oxlint-disable-next-line no-console -- warn logging
-    console.warn(
+    warnOnce(
       `[RTL] Avoid using '${match}'. Use '${match.replace(direction === 'l' ? 'l' : 'r', replacement)}' instead.`,
+      match,
     )
     return matcher([match, replacement, size], context)
   }
@@ -83,9 +99,9 @@ export function presetRtl(): Preset {
         ([, direction, size], context) => {
           if (!size) return undefined
           const replacement = direction === 'left' ? 'inset-is' : 'inset-ie'
-          // oxlint-disable-next-line no-console -- warn logging
-          console.warn(
+          warnOnce(
             `[RTL] Avoid using '${direction}-${size}'. Use '${replacement}-${size}' instead.`,
+            `${direction}-${size}`,
           )
           return directionSize('inset')(['', direction === 'left' ? 'is' : 'ie', size], context)
         },
@@ -95,8 +111,10 @@ export function presetRtl(): Preset {
         /^text-(left|right)$/,
         ([, direction]) => {
           const replacement = direction === 'left' ? 'start' : 'end'
-          // oxlint-disable-next-line no-console -- warn logging
-          console.warn(`[RTL] Avoid using 'text-${direction}'. Use 'text-${replacement}' instead.`)
+          warnOnce(
+            `[RTL] Avoid using 'text-${direction}'. Use 'text-${replacement}' instead.`,
+            `text-${direction}`,
+          )
           return { 'text-align': replacement }
         },
         { autocomplete: 'text-(left|right)' },
@@ -112,9 +130,9 @@ export function presetRtl(): Preset {
           }
           const replacement = replacementMap[direction]
           if (!replacement) return undefined
-          // oxlint-disable-next-line no-console -- warn logging
-          console.warn(
+          warnOnce(
             `[RTL] Avoid using 'rounded-${direction}'. Use 'rounded-${replacement}' instead.`,
+            `rounded-${direction}`,
           )
           return handlerRounded(['', replacement, size ?? 'DEFAULT'], context)
         },
@@ -124,9 +142,9 @@ export function presetRtl(): Preset {
         args => {
           const [_, direction, size] = args
           const replacement = direction === 'l' ? 'is' : 'ie'
-          // oxlint-disable-next-line no-console -- warn logging
-          console.warn(
+          warnOnce(
             `[RTL] Avoid using 'border-${direction}'. Use 'border-${replacement}' instead.`,
+            `border-${direction}`,
           )
           return handlerBorderSize(['', replacement, size || '1'])
         },