meno-core 1.0.53 → 1.1.0

package/lib/client/meno-filter/MenoFilter.test.ts

@@ -50,7 +50,7 @@ describe('MenoFilter', () => {
       await filter.init();
 
       expect(filter.getAll()).toHaveLength(4);
-      expect(filter.getAll()[0]._id).toBe('1');
+      expect(filter.getAll()[0]!._id).toBe('1');
     });
 
     test('should handle missing data gracefully', async () => {
@@ -83,7 +83,7 @@ describe('MenoFilter', () => {
         Promise.resolve({
           ok: true,
           json: () => Promise.resolve(sampleItems),
-        })
+        }),
       );
       const originalFetch = (globalThis as any).fetch;
       (globalThis as any).fetch = mockFetch;
@@ -127,7 +127,7 @@ describe('MenoFilter', () => {
         Promise.resolve({
           ok: false,
           status: 404,
-        })
+        }),
       );
       const originalFetch = (globalThis as any).fetch;
       (globalThis as any).fetch = mockFetch;
@@ -157,7 +157,7 @@ describe('MenoFilter', () => {
         Promise.resolve({
           ok: true,
           json: () => Promise.resolve([{ _id: 'fetch', title: 'Fetch Data' }]),
-        })
+        }),
       );
       const originalFetch = (globalThis as any).fetch;
       (globalThis as any).fetch = mockFetch;
@@ -168,7 +168,7 @@ describe('MenoFilter', () => {
 
         // Should have inline data, not fetch data
         expect(filter.getAll()).toHaveLength(1);
-        expect(filter.getAll()[0]._id).toBe('inline');
+        expect(filter.getAll()[0]!._id).toBe('inline');
         expect(mockFetch).not.toHaveBeenCalled();
       } finally {
         (globalThis as any).fetch = originalFetch;
@@ -193,14 +193,14 @@ describe('MenoFilter', () => {
       const result = filter.filter({ category: 'tech' });
 
       expect(result).toHaveLength(3);
-      expect(result.every(item => item.category === 'tech')).toBe(true);
+      expect(result.every((item) => item.category === 'tech')).toBe(true);
     });
 
     test('should filter by multiple criteria (AND logic)', () => {
       const result = filter.filter({ category: 'tech', featured: true });
 
       expect(result).toHaveLength(1);
-      expect(result[0].title).toBe('Hello World');
+      expect(result[0]!.title).toBe('Hello World');
     });
 
     test('should skip empty/undefined criteria values', () => {
@@ -217,14 +217,14 @@ describe('MenoFilter', () => {
 
     test('should emit to subscribers on filter', () => {
       let emittedItems: CMSItemClient[] = [];
-      filter.subscribe(items => {
+      filter.subscribe((items) => {
         emittedItems = items;
       });
 
       filter.filter({ category: 'design' });
 
       expect(emittedItems).toHaveLength(1);
-      expect(emittedItems[0].category).toBe('design');
+      expect(emittedItems[0]!.category).toBe('design');
     });
   });
 
@@ -245,8 +245,8 @@ describe('MenoFilter', () => {
       const result = filter.search('world');
 
       expect(result).toHaveLength(2);
-      expect(result.map(i => i.title)).toContain('Hello World');
-      expect(result.map(i => i.title)).toContain('Goodbye World');
+      expect(result.map((i) => i.title)).toContain('Hello World');
+      expect(result.map((i) => i.title)).toContain('Goodbye World');
     });
 
     test('should search specified fields only', () => {
@@ -278,22 +278,22 @@ describe('MenoFilter', () => {
     test('should sort ascending by default', () => {
       const result = filter.sort('title');
 
-      expect(result[0].title).toBe('Coding Best Practices');
-      expect(result[3].title).toBe('Hello World');
+      expect(result[0]!.title).toBe('Coding Best Practices');
+      expect(result[3]!.title).toBe('Hello World');
     });
 
     test('should sort descending', () => {
       const result = filter.sort('title', 'desc');
 
-      expect(result[0].title).toBe('Hello World');
-      expect(result[3].title).toBe('Coding Best Practices');
+      expect(result[0]!.title).toBe('Hello World');
+      expect(result[3]!.title).toBe('Coding Best Practices');
     });
 
     test('should sort numbers correctly', () => {
       const result = filter.sort('views', 'desc');
 
-      expect(result[0].views).toBe(200);
-      expect(result[3].views).toBe(50);
+      expect(result[0]!.views).toBe(200);
+      expect(result[3]!.views).toBe(50);
     });
 
     test('should handle undefined values in sort', () => {
@@ -312,7 +312,7 @@ describe('MenoFilter', () => {
       const result = filter2.sort('views', 'asc');
 
       // Undefined values should be sorted last in ascending order
-      expect(result[result.length - 1].title).toBe('No Views');
+      expect(result[result.length - 1]!.title).toBe('No Views');
     });
   });
 
@@ -337,9 +337,7 @@ describe('MenoFilter', () => {
     });
 
     test('should find item by _filename', () => {
-      const itemsWithFilename = [
-        { _id: '1', _filename: 'hello-world', title: 'Hello' },
-      ];
+      const itemsWithFilename = [{ _id: '1', _filename: 'hello-world', title: 'Hello' }];
       const mockElement = {
         textContent: JSON.stringify(itemsWithFilename),
       };
@@ -424,7 +422,7 @@ describe('MenoFilter', () => {
       await filter.init();
 
       let receivedItems: CMSItemClient[] = [];
-      filter.subscribe(items => {
+      filter.subscribe((items) => {
         receivedItems = items;
       });
 
@@ -484,8 +482,8 @@ describe('MenoFilter', () => {
       const result = filter.getFiltered();
 
       expect(result).toHaveLength(2);
-      expect(result[0].title).toBe('Goodbye World');
-      expect(result[1].title).toBe('Hello World');
+      expect(result[0]!.title).toBe('Goodbye World');
+      expect(result[1]!.title).toBe('Hello World');
     });
   });
 
@@ -517,8 +515,8 @@ describe('MenoFilter', () => {
 
       const pageItems = filter.getItems();
       expect(pageItems).toHaveLength(3);
-      expect(pageItems[0]._id).toBe('1');
-      expect(pageItems[2]._id).toBe('3');
+      expect(pageItems[0]!._id).toBe('1');
+      expect(pageItems[2]!._id).toBe('3');
     });
 
     test('should return correct page info', async () => {
@@ -546,7 +544,7 @@ describe('MenoFilter', () => {
 
       const pageItems = filter.setPage(2);
       expect(pageItems).toHaveLength(3);
-      expect(pageItems[0]._id).toBe('4');
+      expect(pageItems[0]!._id).toBe('4');
       expect(filter.getPageInfo().current).toBe(2);
     });
 
@@ -607,7 +605,7 @@ describe('MenoFilter', () => {
 
       const lastPageItems = filter.setPage(4);
       expect(lastPageItems).toHaveLength(1); // 10 items, pages 1-3 have 3 each, page 4 has 1
-      expect(lastPageItems[0]._id).toBe('10');
+      expect(lastPageItems[0]!._id).toBe('10');
     });
 
     test('filter() should reset to page 1', async () => {
@@ -648,18 +646,18 @@ describe('MenoFilter', () => {
       await filter.init();
 
       let receivedItems: CMSItemClient[] = [];
-      filter.subscribe(items => {
+      filter.subscribe((items) => {
         receivedItems = items;
       });
 
       // Initial subscription gets first page
       expect(receivedItems).toHaveLength(3);
-      expect(receivedItems[0]._id).toBe('1');
+      expect(receivedItems[0]!._id).toBe('1');
 
       // Change page
       filter.setPage(2);
       expect(receivedItems).toHaveLength(3);
-      expect(receivedItems[0]._id).toBe('4');
+      expect(receivedItems[0]!._id).toBe('4');
     });
 
     test('getFiltered() should return all filtered items ignoring pagination', async () => {
@@ -701,7 +699,7 @@ describe('MenoFilter', () => {
       const result = filter.filter({ price: { $gt: 100 } });
 
       expect(result).toHaveLength(2);
-      expect(result.map(i => i.price)).toEqual([200, 150]);
+      expect(result.map((i) => i.price)).toEqual([200, 150]);
     });
 
     test('should filter numbers from string criteria', async () => {
@@ -718,7 +716,7 @@ describe('MenoFilter', () => {
       const result = filter.filter({ price: { $gte: '100', $lte: '150' } });
 
       expect(result).toHaveLength(2);
-      expect(result.map(i => i.price)).toEqual([100, 150]);
+      expect(result.map((i) => i.price)).toEqual([100, 150]);
     });
 
     test('should filter dates with type coercion', async () => {
@@ -735,7 +733,7 @@ describe('MenoFilter', () => {
       const result = filter.filter({ date: { $gt: '2024-01-31' } });
 
       expect(result).toHaveLength(2);
-      expect(result.map(i => i.date)).toEqual(['2024-02-15', '2024-03-15']);
+      expect(result.map((i) => i.date)).toEqual(['2024-02-15', '2024-03-15']);
     });
 
     test('should filter booleans with type coercion', async () => {
@@ -752,7 +750,7 @@ describe('MenoFilter', () => {
       const result = filter.filter({ featured: 'true' });
 
       expect(result).toHaveLength(2);
-      expect(result.every(i => i.featured === true)).toBe(true);
+      expect(result.every((i) => i.featured === true)).toBe(true);
     });
 
     test('setFieldType() should update field types', async () => {
@@ -773,7 +771,7 @@ describe('MenoFilter', () => {
       result = filter.filter({ price: { $gt: 100 } });
 
       expect(result).toHaveLength(2);
-      expect(result.map(i => i.price)).toEqual([200, 150]);
+      expect(result.map((i) => i.price)).toEqual([200, 150]);
     });
 
     test('setFieldTypes() should set multiple types', async () => {
@@ -820,7 +818,7 @@ describe('MenoFilter', () => {
       const result = filter.filterRange('price', { min: 100, max: 200 });
 
       expect(result).toHaveLength(3);
-      expect(result.map(i => i.price).sort((a, b) => (a as number) - (b as number))).toEqual([100, 150, 200]);
+      expect(result.map((i) => i.price).sort((a, b) => (a as number) - (b as number))).toEqual([100, 150, 200]);
     });
 
     test('should filter with only min', async () => {
@@ -836,7 +834,7 @@ describe('MenoFilter', () => {
       const result = filter.filterRange('price', { min: 150 });
 
       expect(result).toHaveLength(3);
-      expect(result.every(i => (i.price as number) >= 150)).toBe(true);
+      expect(result.every((i) => (i.price as number) >= 150)).toBe(true);
     });
 
     test('should filter with only max', async () => {
@@ -852,7 +850,7 @@ describe('MenoFilter', () => {
       const result = filter.filterRange('price', { max: 100 });
 
       expect(result).toHaveLength(2);
-      expect(result.every(i => (i.price as number) <= 100)).toBe(true);
+      expect(result.every((i) => (i.price as number) <= 100)).toBe(true);
     });
 
     test('should clear filter when both min and max are empty', async () => {
@@ -932,7 +930,7 @@ describe('MenoFilter', () => {
       const result = filter.search('react', ['title']);
 
       expect(result).toHaveLength(1);
-      expect(result[0].title).toBe('React Hooks Tutorial');
+      expect(result[0]!.title).toBe('React Hooks Tutorial');
     });
 
     test('should match with typos when fuzzy enabled', async () => {
@@ -956,7 +954,7 @@ describe('MenoFilter', () => {
       const result = filter.search('recat', ['title']);
 
       expect(result.length).toBeGreaterThanOrEqual(1);
-      expect(result.some(i => i.title === 'React')).toBe(true);
+      expect(result.some((i) => i.title === 'React')).toBe(true);
     });
 
     test('should use exact match for short queries (< 3 chars)', async () => {
@@ -973,7 +971,7 @@ describe('MenoFilter', () => {
       const result = filter.search('js', ['title']);
 
       expect(result).toHaveLength(1);
-      expect(result[0].title).toBe('Vue.js Components Guide');
+      expect(result[0]!.title).toBe('Vue.js Components Guide');
     });
 
     test('setFuzzySearch() should enable/disable fuzzy search', async () => {
@@ -1022,7 +1020,7 @@ describe('MenoFilter', () => {
       const result = filter.search('javascript', ['title']);
 
       expect(result).toHaveLength(1);
-      expect(result[0].title).toBe('JavaScript Best Practices');
+      expect(result[0]!.title).toBe('JavaScript Best Practices');
     });
   });
 
@@ -1065,7 +1063,7 @@ describe('MenoFilter', () => {
 
       // Should have called replaceState (default mode)
       expect(mockHistory.replaceState).toHaveBeenCalled();
-      const lastCall = mockHistory.replaceState.mock.calls[mockHistory.replaceState.mock.calls.length - 1];
+      const lastCall = mockHistory.replaceState.mock.calls[mockHistory.replaceState.mock.calls.length - 1]!;
       expect(lastCall[2]).toContain('filter.category=tech');
     });
 
@@ -1080,7 +1078,7 @@ describe('MenoFilter', () => {
       filter.filter({ category: 'design' });
 
       expect(mockHistory.pushState).toHaveBeenCalled();
-      const lastCall = mockHistory.pushState.mock.calls[mockHistory.pushState.mock.calls.length - 1];
+      const lastCall = mockHistory.pushState.mock.calls[mockHistory.pushState.mock.calls.length - 1]!;
       expect(lastCall[2]).toContain('filter.category=design');
     });
 
@@ -1094,7 +1092,7 @@ describe('MenoFilter', () => {
       filter.enableUrlSync();
       filter.sort('title', 'desc');
 
-      const lastCall = mockHistory.replaceState.mock.calls[mockHistory.replaceState.mock.calls.length - 1];
+      const lastCall = mockHistory.replaceState.mock.calls[mockHistory.replaceState.mock.calls.length - 1]!;
       // URLSearchParams encodes : as %3A
       const decodedUrl = decodeURIComponent(lastCall[2] as string);
       expect(decodedUrl).toContain('sort=title:desc');
@@ -1110,7 +1108,7 @@ describe('MenoFilter', () => {
       filter.enableUrlSync();
       filter.search('hello', ['title', 'category']);
 
-      const lastCall = mockHistory.replaceState.mock.calls[mockHistory.replaceState.mock.calls.length - 1];
+      const lastCall = mockHistory.replaceState.mock.calls[mockHistory.replaceState.mock.calls.length - 1]!;
       // URLSearchParams encodes , as %2C
       const decodedUrl = decodeURIComponent(lastCall[2] as string);
       expect(decodedUrl).toContain('search=hello');
@@ -1132,7 +1130,7 @@ describe('MenoFilter', () => {
       filter.enableUrlSync();
       filter.setPage(3);
 
-      const lastCall = mockHistory.replaceState.mock.calls[mockHistory.replaceState.mock.calls.length - 1];
+      const lastCall = mockHistory.replaceState.mock.calls[mockHistory.replaceState.mock.calls.length - 1]!;
       expect(lastCall[2]).toContain('page=3');
     });
 
@@ -1149,7 +1147,7 @@ describe('MenoFilter', () => {
       filter.enableUrlSync();
       filter.filter({ views: { $gt: 100, $lte: 200 } });
 
-      const lastCall = mockHistory.replaceState.mock.calls[mockHistory.replaceState.mock.calls.length - 1];
+      const lastCall = mockHistory.replaceState.mock.calls[mockHistory.replaceState.mock.calls.length - 1]!;
       // URLSearchParams encodes $ as %24
       const decodedUrl = decodeURIComponent(lastCall[2] as string);
       expect(decodedUrl).toContain('filter.views.$gt=100');

package/lib/client/meno-filter/MenoFilter.ts

@@ -562,7 +562,9 @@ export class MenoFilter {
       const value = item[field];
       if (value !== undefined && value !== null) {
         if (Array.isArray(value)) {
-          value.forEach((v) => values.add(v));
+          value.forEach((v) => {
+            values.add(v);
+          });
         } else {
           values.add(value);
         }
@@ -669,12 +671,12 @@ export class MenoFilter {
       case 'number': {
         if (typeof value === 'number') return value;
         const num = parseFloat(String(value));
-        return isNaN(num) ? value : num;
+        return Number.isNaN(num) ? value : num;
       }
       case 'date': {
         if (value instanceof Date) return value.getTime();
         const time = new Date(String(value)).getTime();
-        return isNaN(time) ? value : time;
+        return Number.isNaN(time) ? value : time;
       }
       case 'boolean':
         return value === true || value === 'true' || value === '1';
@@ -797,9 +799,7 @@ export class MenoFilter {
     }
 
     const queryString = params.toString();
-    const url = queryString
-      ? `${window.location.pathname}?${queryString}`
-      : window.location.pathname;
+    const url = queryString ? `${window.location.pathname}?${queryString}` : window.location.pathname;
 
     if (this.urlMode === 'push') {
       history.pushState(null, '', url);
@@ -825,6 +825,7 @@ export class MenoFilter {
       const parts = key.split('.');
       const field = parts[1];
       const op = parts[2]; // Optional operator
+      if (field === undefined) continue;
 
       if (op) {
         // Operator: filter.price.$gt=100
@@ -845,7 +846,7 @@ export class MenoFilter {
     // Parse sort
     const sortParam = params.get('sort');
     if (sortParam) {
-      const [field, order = 'asc'] = sortParam.split(':');
+      const [field = '', order = 'asc'] = sortParam.split(':');
       this.sortConfig = { field, order: order as 'asc' | 'desc' };
     }
 
@@ -863,7 +864,7 @@ export class MenoFilter {
     const pageParam = params.get('page');
     if (pageParam) {
       const page = parseInt(pageParam, 10);
-      if (!isNaN(page) && page > 0) {
+      if (!Number.isNaN(page) && page > 0) {
         this.currentPage = page;
       }
     }
@@ -872,7 +873,7 @@ export class MenoFilter {
     const perPageParam = params.get('perPage');
     if (perPageParam) {
       const perPage = parseInt(perPageParam, 10);
-      if (!isNaN(perPage) && perPage > 0) {
+      if (!Number.isNaN(perPage) && perPage > 0) {
         this.perPage = perPage;
       }
     }
@@ -1082,17 +1083,14 @@ export class MenoFilter {
 
     // Apply OR criteria
     if (this.orCriteria.length > 0) {
-      result = result.filter((item) =>
-        this.orCriteria.some((criteria) => this.matchesCriteria(item, criteria))
-      );
+      result = result.filter((item) => this.orCriteria.some((criteria) => this.matchesCriteria(item, criteria)));
     }
 
     // Apply search
     if (this.searchQuery) {
       const query = this.searchQuery;
       result = result.filter((item) => {
-        const fieldsToSearch =
-          this.searchFields.length > 0 ? this.searchFields : Object.keys(item);
+        const fieldsToSearch = this.searchFields.length > 0 ? this.searchFields : Object.keys(item);
         return fieldsToSearch.some((field) => {
           const value = item[field];
           if (typeof value !== 'string') return false;
@@ -1259,13 +1257,14 @@ export class MenoFilter {
       case '$nin':
         return Array.isArray(opValue) ? !opValue.includes(itemValue) : true;
 
-      case '$empty':
+      case '$empty': {
         const isEmpty =
           itemValue === undefined ||
           itemValue === null ||
           itemValue === '' ||
           (Array.isArray(itemValue) && itemValue.length === 0);
         return opValue ? isEmpty : !isEmpty;
+      }
 
       default:
         console.warn(`MenoFilter: Unknown operator "${op}"`);

package/lib/client/meno-filter/bindings.ts

@@ -5,7 +5,7 @@
  * to MenoFilter instances via data attributes.
  */
 
-import { MenoFilter } from './MenoFilter';
+import type { MenoFilter } from './MenoFilter';
 import { ATTR } from './constants';
 import type { FilterInstance, FilterMode } from './types';
 
@@ -42,12 +42,7 @@ function getFilterMode(element: HTMLElement): FilterMode | null {
  * Handle button click in multi-select mode
  * Toggles value in $in array
  */
-function handleMultiButtonClick(
-  filter: MenoFilter,
-  field: string,
-  value: string,
-  currentValue: unknown
-): void {
+function handleMultiButtonClick(filter: MenoFilter, field: string, value: string, currentValue: unknown): void {
   // Get current values as array
   let values: string[] = [];
   if (currentValue !== undefined) {
@@ -87,9 +82,7 @@ export function bindFilterControls(instance: FilterInstance): void {
   const { wrapper, filter, cleanup } = instance;
 
   // Filter buttons
-  const filterButtons = wrapper.querySelectorAll<HTMLElement>(
-    `[${ATTR.FILTER_FIELD}][${ATTR.FILTER_VALUE}]`
-  );
+  const filterButtons = wrapper.querySelectorAll<HTMLElement>(`[${ATTR.FILTER_FIELD}][${ATTR.FILTER_VALUE}]`);
 
   for (const btn of filterButtons) {
     if (btn.tagName === 'INPUT') continue; // Handle inputs separately
@@ -118,7 +111,6 @@ export function bindFilterControls(instance: FilterInstance): void {
           filter.addFilter(field, value);
         }
       }
-
     };
 
     btn.addEventListener('click', handler);
@@ -126,9 +118,7 @@ export function bindFilterControls(instance: FilterInstance): void {
   }
 
   // Filter checkboxes (allow multiple selections with OR logic)
-  const filterCheckboxes = wrapper.querySelectorAll<HTMLInputElement>(
-    `input[type="checkbox"][${ATTR.FILTER_FIELD}]`
-  );
+  const filterCheckboxes = wrapper.querySelectorAll<HTMLInputElement>(`input[type="checkbox"][${ATTR.FILTER_FIELD}]`);
 
   for (const checkbox of filterCheckboxes) {
     const field = checkbox.getAttribute(ATTR.FILTER_FIELD)!;
@@ -139,7 +129,7 @@ export function bindFilterControls(instance: FilterInstance): void {
       // Single mode: uncheck other checkboxes in the same field
       if (mode === 'single' && checkbox.checked) {
         const siblings = wrapper.querySelectorAll<HTMLInputElement>(
-          `input[type="checkbox"][${ATTR.FILTER_FIELD}="${field}"]`
+          `input[type="checkbox"][${ATTR.FILTER_FIELD}="${field}"]`,
         );
         for (const sibling of siblings) {
           if (sibling !== checkbox) {
@@ -156,9 +146,7 @@ export function bindFilterControls(instance: FilterInstance): void {
   }
 
   // Filter selects
-  const filterSelects = wrapper.querySelectorAll<HTMLSelectElement>(
-    `select[${ATTR.FILTER_FIELD}]`
-  );
+  const filterSelects = wrapper.querySelectorAll<HTMLSelectElement>(`select[${ATTR.FILTER_FIELD}]`);
 
   for (const select of filterSelects) {
     const field = select.getAttribute(ATTR.FILTER_FIELD)!;
@@ -177,9 +165,7 @@ export function bindFilterControls(instance: FilterInstance): void {
   }
 
   // Filter radio buttons
-  const filterRadios = wrapper.querySelectorAll<HTMLInputElement>(
-    `input[type="radio"][${ATTR.FILTER_FIELD}]`
-  );
+  const filterRadios = wrapper.querySelectorAll<HTMLInputElement>(`input[type="radio"][${ATTR.FILTER_FIELD}]`);
 
   for (const radio of filterRadios) {
     const field = radio.getAttribute(ATTR.FILTER_FIELD)!;
@@ -203,18 +189,12 @@ export function bindFilterControls(instance: FilterInstance): void {
 /**
  * Update filters based on checked checkboxes for a field
  */
-function updateCheckboxFilters(
-  wrapper: HTMLElement,
-  filter: MenoFilter,
-  field: string
-): void {
+function updateCheckboxFilters(wrapper: HTMLElement, filter: MenoFilter, field: string): void {
   const checkboxes = wrapper.querySelectorAll<HTMLInputElement>(
-    `input[type="checkbox"][${ATTR.FILTER_FIELD}="${field}"]:checked`
+    `input[type="checkbox"][${ATTR.FILTER_FIELD}="${field}"]:checked`,
   );
 
-  const values = Array.from(checkboxes).map(
-    (cb) => cb.getAttribute(ATTR.FILTER_VALUE) || cb.value
-  );
+  const values = Array.from(checkboxes).map((cb) => cb.getAttribute(ATTR.FILTER_VALUE) || cb.value);
 
   if (values.length === 0) {
     filter.removeFilter(field);
@@ -311,7 +291,7 @@ export function bindSortControls(instance: FilterInstance): void {
       }
 
       // Parse "field:order" format
-      const [field, order = 'asc'] = value.split(':');
+      const [field = '', order = 'asc'] = value.split(':');
       filter.sort(field, order as 'asc' | 'desc');
     };
 
@@ -363,17 +343,13 @@ export function bindActionControls(instance: FilterInstance): void {
  */
 export function resetFormControls(wrapper: HTMLElement): void {
   // Reset checkboxes
-  const checkboxes = wrapper.querySelectorAll<HTMLInputElement>(
-    `input[type="checkbox"][${ATTR.FILTER_FIELD}]`
-  );
+  const checkboxes = wrapper.querySelectorAll<HTMLInputElement>(`input[type="checkbox"][${ATTR.FILTER_FIELD}]`);
   for (const cb of checkboxes) {
     cb.checked = false;
   }
 
   // Reset selects
-  const selects = wrapper.querySelectorAll<HTMLSelectElement>(
-    `select[${ATTR.FILTER_FIELD}]`
-  );
+  const selects = wrapper.querySelectorAll<HTMLSelectElement>(`select[${ATTR.FILTER_FIELD}]`);
   for (const select of selects) {
     select.selectedIndex = 0;
   }
@@ -421,12 +397,13 @@ export function bindPaginationControls(instance: FilterInstance): void {
         case 'last':
           filter.setPage(filter.getPageInfo().total);
           break;
-        default:
+        default: {
           // Numeric page
           const page = parseInt(action || '1', 10);
-          if (!isNaN(page)) {
+          if (!Number.isNaN(page)) {
             filter.setPage(page);
           }
+        }
       }
     };
 
@@ -435,14 +412,12 @@ export function bindPaginationControls(instance: FilterInstance): void {
   }
 
   // Per-page select
-  const perPageSelects = wrapper.querySelectorAll<HTMLSelectElement>(
-    `[${ATTR.PER_PAGE_SELECT}]`
-  );
+  const perPageSelects = wrapper.querySelectorAll<HTMLSelectElement>(`[${ATTR.PER_PAGE_SELECT}]`);
 
   for (const select of perPageSelects) {
     const handler = () => {
       const perPage = parseInt(select.value, 10);
-      if (!isNaN(perPage) && perPage > 0) {
+      if (!Number.isNaN(perPage) && perPage > 0) {
         filter.setPerPage(perPage);
       }
     };

package/lib/client/meno-filter/init.ts

@@ -79,7 +79,7 @@ async function initFilterInstance(wrapper: HTMLElement, collection: string): Pro
     const thresholdAttr = searchInput.getAttribute(ATTR.SEARCH_THRESHOLD);
     if (thresholdAttr) {
       const parsed = parseFloat(thresholdAttr);
-      if (!isNaN(parsed)) {
+      if (!Number.isNaN(parsed)) {
         fuzzyThreshold = parsed;
       }
     }
@@ -136,7 +136,7 @@ async function initFilterInstance(wrapper: HTMLElement, collection: string): Pro
   cleanup.push(
     filter.watch(() => {
       updateUI(instance);
-    })
+    }),
   );
 }
 
@@ -165,7 +165,7 @@ function extractItemTemplate(wrapper: HTMLElement): {
     const itemTemplate = emittedTemplate.innerHTML.trim();
     // Detect itemAs from template variables (e.g., {{post.title}} → "post")
     const match = itemTemplate.match(/\{\{(\w+)\./);
-    if (match) itemAs = match[1];
+    if (match && match[1] !== undefined) itemAs = match[1];
 
     // Store SSR'd items by their data-id for potential reuse
     for (const child of Array.from(listEl.children) as HTMLElement[]) {
@@ -184,7 +184,7 @@ function extractItemTemplate(wrapper: HTMLElement): {
   if (userTemplate) {
     const itemTemplate = userTemplate.innerHTML.trim();
     const match = itemTemplate.match(/\{\{(\w+)\./);
-    if (match) itemAs = match[1];
+    if (match && match[1] !== undefined) itemAs = match[1];
     return { itemTemplate, ssrItems, itemAs };
   }