fix: resolve build/test failures and relax eslint callback rules

Author: sonofmagic

eslint.config.js

@@ -4,4 +4,10 @@ export default icebreaker(
   {
     ignores: ['**/fixtures/**', 'website/public/_pagefind'],
   },
+  {
+    rules: {
+      'dot-notation': 'off',
+      'prefer-arrow-callback': 'off',
+    },
+  },
 )

packages/config/src/defaults.ts

@@ -32,7 +32,7 @@ export function getDefaultTransformerConfig(): TransformerOptions {
       include: defaultPipelineInclude,
       exclude: defaultPipelineExclude,
     },
-    disabled: process.env.NODE_ENV === 'development',
+    disabled: process.env['NODE_ENV'] === 'development',
     registry: {
       file: '.tw-patch/tw-class-list.json',
       mapping: {

packages/shared/test/classGenerator.behavior.test.ts

@@ -1,4 +1,5 @@
 import { ClassGenerator } from '@/classGenerator'
+import process from 'node:process'
 
 describe('ClassGenerator behaviour', () => {
   it('respects include/exclude filters and ignore rules', () => {
@@ -51,21 +52,21 @@ describe('ClassGenerator behaviour', () => {
   })
 
   it('skips logging when reserved name is encountered without log flag', () => {
-    const logSpy = vi.spyOn(console, 'log').mockImplementation(() => undefined)
+    const writeSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true)
     const generator = new ClassGenerator({
       reserveClassName: [/^tw-a$/u],
     })
 
     const result = generator.generateClassName('foo')
     expect(result.name).toBe('tw-b')
     expect(generator.newClassSize).toBe(2)
-    expect(logSpy).not.toHaveBeenCalled()
+    expect(writeSpy).not.toHaveBeenCalled()
 
-    logSpy.mockRestore()
+    writeSpy.mockRestore()
   })
 
   it('falls back to default generator and skips reserved results', () => {
-    const logSpy = vi.spyOn(console, 'log').mockImplementation(() => undefined)
+    const writeSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true)
     const generator = new ClassGenerator({
       reserveClassName: [/^tw-a$/u],
       log: true,
@@ -74,8 +75,8 @@ describe('ClassGenerator behaviour', () => {
     const result = generator.generateClassName('foo')
     expect(result.name).toBe('tw-b')
     expect(generator.newClassSize).toBe(2)
-    expect(logSpy).toHaveBeenCalledWith('The class name has been reserved. tw-a')
+    expect(writeSpy).toHaveBeenCalledWith('The class name has been reserved. tw-a\n')
 
-    logSpy.mockRestore()
+    writeSpy.mockRestore()
   })
 })

packages/tailwindcss-patch/src/cache/store.ts

@@ -308,12 +308,12 @@ export class CacheStore {
       return undefined
     }
 
-    const values = toStringArray(record.values)
+    const values = toStringArray(record['values'])
     if (values.length === 0) {
       return undefined
     }
 
-    const contextRecord = asObject(record.context)
+    const contextRecord = asObject(record['context'])
     if (!contextRecord) {
       return undefined
     }
@@ -361,7 +361,7 @@ export class CacheStore {
         optionsHash,
       },
       values,
-      updatedAt: typeof record.updatedAt === 'string' ? record.updatedAt : new Date(0).toISOString(),
+      updatedAt: typeof record['updatedAt'] === 'string' ? record['updatedAt'] : new Date(0).toISOString(),
     }
 
     return normalized
@@ -380,11 +380,11 @@ export class CacheStore {
       return { kind: 'invalid' }
     }
 
-    if (record.schemaVersion !== CACHE_SCHEMA_VERSION) {
+    if (record['schemaVersion'] !== CACHE_SCHEMA_VERSION) {
       return { kind: 'invalid' }
     }
 
-    const contextsRecord = asObject(record.contexts)
+    const contextsRecord = asObject(record['contexts'])
     if (!contextsRecord) {
       return { kind: 'invalid' }
     }
@@ -406,7 +406,7 @@ export class CacheStore {
       kind: 'v2',
       data: {
         schemaVersion: CACHE_SCHEMA_VERSION,
-        updatedAt: typeof record.updatedAt === 'string' ? record.updatedAt : new Date(0).toISOString(),
+        updatedAt: typeof record['updatedAt'] === 'string' ? record['updatedAt'] : new Date(0).toISOString(),
         contexts,
       },
     }

packages/unplugin-tailwindcss-mangle/src/utils.ts

@@ -40,17 +40,17 @@ export function getGroupedEntries<T>(
       return 'other'
     }
   })
-  if (!groupedEntries.css) {
-    groupedEntries.css = []
+  if (!groupedEntries['css']) {
+    groupedEntries['css'] = []
   }
-  if (!groupedEntries.html) {
-    groupedEntries.html = []
+  if (!groupedEntries['html']) {
+    groupedEntries['html'] = []
   }
-  if (!groupedEntries.js) {
-    groupedEntries.js = []
+  if (!groupedEntries['js']) {
+    groupedEntries['js'] = []
   }
-  if (!groupedEntries.other) {
-    groupedEntries.other = []
+  if (!groupedEntries['other']) {
+    groupedEntries['other'] = []
   }
   return groupedEntries as Record<'css' | 'html' | 'js' | 'other', [string, T][]>
 }

packages/unplugin-tailwindcss-mangle/test/transform.test.ts

@@ -34,7 +34,7 @@ const {
 })
 
 vi.mock('@tailwindcss-mangle/core', () => {
-  const Context = vi.fn(() => {
+  const Context = vi.fn(function MockContext() {
     return mockCtx
   })
   return {

packages/unplugin-tailwindcss-mangle/test/webpack.test.ts

@@ -28,7 +28,7 @@ const { mockCtx, mockCssHandler, mockHtmlHandler, mockJsHandler } = vi.hoisted((
 })
 
 vi.mock('@tailwindcss-mangle/core', () => {
-  const Context = vi.fn(() => {
+  const Context = vi.fn(function MockContext() {
     return mockCtx
   })
   return {