From a274f3102966e9d00b2d1ecc220274d0ab092ae8 Mon Sep 17 00:00:00 2001
From: Gauthier Roebroeck <gauthier.roebroeck@gmail.com>
Date: Fri, 30 Jan 2026 17:42:32 +0800
Subject: [PATCH] filters

---
 next-ui/src/components/filter/List.stories.ts     |  5 ++++-
 next-ui/src/components/filter/List.vue            | 10 +++++++---
 next-ui/src/components/filter/TriState.stories.ts |  6 ------
 next-ui/src/components/filter/TriState.vue        | 13 ++++++++++++-
 4 files changed, 23 insertions(+), 11 deletions(-)

diff --git a/next-ui/src/components/filter/List.stories.ts b/next-ui/src/components/filter/List.stories.ts
index 849cf6de..f894aa6a 100644
--- a/next-ui/src/components/filter/List.stories.ts
+++ b/next-ui/src/components/filter/List.stories.ts
@@ -22,6 +22,7 @@ const meta = {
   },
   args: {
     'onUpdate:modelValue': fn(),
+    'onUpdate:mode': fn(),
     items: [
       { title: 'Tag 1', value: '+tag1', valueExclude: '-tag1' },
       { title: 'Tag 2', value: '+tag2', valueExclude: '-tag2' },
@@ -41,16 +42,18 @@ export const Default: Story = {
 export const InitialValue: Story = {
   args: {
     modelValue: ['+tag1', '-tag2', 'crap'],
+    mode: 'allOf',
   },
 }
 
 export const Color: Story = {
   args: {
+    modelValue: ['+tag1', '-tag2', 'crap'],
     color: 'red',
   },
 }
 
-export const Objects: Story = {
+export const ValueObjects: Story = {
   args: {
     items: [
       { title: 'Tag 1', value: { include: 'tag1' }, valueExclude: { exclude: 'tag1' } },
diff --git a/next-ui/src/components/filter/List.vue b/next-ui/src/components/filter/List.vue
index 13cb69fc..454e4c01 100644
--- a/next-ui/src/components/filter/List.vue
+++ b/next-ui/src/components/filter/List.vue
@@ -19,8 +19,12 @@
 <script setup lang="ts">
 import { syncRef } from '@vueuse/core'
 import type { AnyAll } from '@/types/filter'
+import type { IncludeExclude } from '@/components/filter/TriState.vue'
 
 const model = defineModel<unknown[]>({ default: [] })
+/**
+ * Selection mode.
+ */
 const anyAll = defineModel<AnyAll>('mode', { default: 'anyOf' })
 
 //TODO: handle mode better
@@ -36,12 +40,12 @@ const { items = [], color } = defineProps<{
 }>()
 
 function initializeInternalModel() {
-  const c: Record<number, string | undefined> = {}
+  const c: Record<number, IncludeExclude> = {}
   items.forEach((_it, i) => (c[i] = undefined))
   return c
 }
 
-const internalModel = ref<Record<number, string | undefined>>(initializeInternalModel())
+const internalModel = ref<Record<number, IncludeExclude>>(initializeInternalModel())
 
 syncRef(model, internalModel, {
   direction: 'both',
@@ -68,7 +72,7 @@ syncRef(model, internalModel, {
           }
 
         return acc
-      }, {}) as Record<number, string | undefined>,
+      }, {}) as Record<number, IncludeExclude>,
     rtl: (right) =>
       items
         .map((it, i) =>
diff --git a/next-ui/src/components/filter/TriState.stories.ts b/next-ui/src/components/filter/TriState.stories.ts
index 0c7f1f8a..ce03792b 100644
--- a/next-ui/src/components/filter/TriState.stories.ts
+++ b/next-ui/src/components/filter/TriState.stories.ts
@@ -46,12 +46,6 @@ export const ValidModel: Story = {
   },
 }
 
-export const InvalidModel: Story = {
-  args: {
-    modelValue: 'nope',
-  },
-}
-
 export const BiState: Story = {
   args: {
     label: 'bi state',
diff --git a/next-ui/src/components/filter/TriState.vue b/next-ui/src/components/filter/TriState.vue
index 55b18c2b..19e903f9 100644
--- a/next-ui/src/components/filter/TriState.vue
+++ b/next-ui/src/components/filter/TriState.vue
@@ -10,7 +10,9 @@
 </template>
 
 <script setup lang="ts">
-const model = defineModel<string>()
+export type IncludeExclude = 'include' | 'exclude' | undefined
+
+const model = defineModel<IncludeExclude>()
 
 const icon = computed(() => states.value.find((it) => it.value === model.value)?.icon)
 
@@ -19,8 +21,17 @@ const {
   triState = true,
   color,
 } = defineProps<{
+  /**
+   * Label shown next to the checkbox.
+   */
   label: string
+  /**
+   * Whether the component can handle tri-state. Default to `true`.
+   */
   triState?: boolean
+  /**
+   * Base color. Applies to the checkbox when the value is not `undefined`.
+   */
   color?: string
 }>()