From 875328c32cbe097227a2d0b60a1fe57f1aa79001 Mon Sep 17 00:00:00 2001
From: Gregor Vostrak <gregor@solidtime.io>
Date: Mon, 31 Aug 2026 13:56:18 +0200
Subject: [PATCH] fix type error and adapt formatting to new prettier version
 changes

---
 .../Common/Project/ProjectTable.vue           |   8 +-
 .../Timesheet/BreakPlacementModal.vue         |   5 +-
 .../js/Components/Timesheet/TimesheetGrid.vue |   2 +-
 .../Pages/Profile/Partials/DeleteUserForm.vue |   3 +-
 .../Pages/Teams/Partials/DeleteTeamForm.vue   |   3 +-
 resources/js/packages/api/src/index.ts        |   5 +-
 .../ui/src/Input/BillableToggleButton.vue     |   2 +-
 .../js/packages/ui/src/Input/Dropdown.vue     |   2 +-
 .../ui/src/Input/DurationHumanInput.vue       |   4 +-
 .../ui/src/Input/MultiselectDropdown.vue      |   2 +-
 .../js/packages/ui/src/Tag/TagDropdown.vue    |   2 +-
 .../src/TimeEntry/TimeEntryGroupedTable.vue   |   2 +-
 .../src/TimeEntry/TimeEntryRowTagDropdown.vue |   2 +-
 .../TimeTracker/TimeTrackerTagDropdown.vue    |   2 +-
 .../packages/ui/src/TimezoneMismatchModal.vue |   2 +-
 resources/js/packages/ui/src/utils/number.ts  |   6 +-
 resources/js/utils/commandPaletteCommands.ts  |   8 +-
 resources/js/utils/useCommandPalette.ts       | 136 ++++++++----------
 18 files changed, 87 insertions(+), 109 deletions(-)

diff --git a/resources/js/Components/Common/Project/ProjectTable.vue b/resources/js/Components/Common/Project/ProjectTable.vue
index e5190f22..49abdebd 100644
--- a/resources/js/Components/Common/Project/ProjectTable.vue
+++ b/resources/js/Components/Common/Project/ProjectTable.vue
@@ -10,13 +10,7 @@ import Pagination from '@/Components/Common/Pagination.vue';
 import LoadingSpinner from '@/packages/ui/src/LoadingSpinner.vue';
 
 export type SortColumn =
-    | 'name'
-    | 'client_name'
-    | 'spent_time'
-    | 'progress'
-    | 'billable_rate'
-    | 'status'
-    | 'visibility';
+    'name' | 'client_name' | 'spent_time' | 'progress' | 'billable_rate' | 'status' | 'visibility';
 export type { SortDirection } from '@/utils/useSortableTable';
 import { canCreateProjects } from '@/utils/permissions';
 import type { CreateProjectBody, Project, Client, CreateClientBody } from '@/packages/api/src';
diff --git a/resources/js/Components/Timesheet/BreakPlacementModal.vue b/resources/js/Components/Timesheet/BreakPlacementModal.vue
index 66fda519..724d27f1 100644
--- a/resources/js/Components/Timesheet/BreakPlacementModal.vue
+++ b/resources/js/Components/Timesheet/BreakPlacementModal.vue
@@ -146,7 +146,10 @@ const changeSummary = computed<PlanLine[]>(() => {
             { times: range(plan.breakSlot), label: 'Break' },
             { times: range(plan.secondHalf), label: workLabel },
             ...plan.shifted.map((shift) => ({
-                times: moved(req.otherEntries.find((e) => e.id === shift.id)!, shift),
+                times: moved(
+                    req.otherEntries.find((e) => e.id === shift.id)!,
+                    shift
+                ),
                 label: props.entryLabel(shift.id),
             })),
         ];
diff --git a/resources/js/Components/Timesheet/TimesheetGrid.vue b/resources/js/Components/Timesheet/TimesheetGrid.vue
index 671a96a3..ff426ee2 100644
--- a/resources/js/Components/Timesheet/TimesheetGrid.vue
+++ b/resources/js/Components/Timesheet/TimesheetGrid.vue
@@ -161,7 +161,7 @@ const emit = defineEmits<{
                         :organization-billable-rate="organization?.billable_rate ?? null"
                         :no-project-value="null"
                         align="start"
-                        @changed="(p, t) => emit('add-row', p, t)">
+                        @changed="(p: string | null, t: string | null) => emit('add-row', p, t)">
                         <template #trigger>
                             <Button variant="ghost" size="sm" class="text-text-secondary">
                                 <PlusIcon class="h-4 w-4 mr-1 text-icon-default" />
diff --git a/resources/js/Pages/Profile/Partials/DeleteUserForm.vue b/resources/js/Pages/Profile/Partials/DeleteUserForm.vue
index 16014124..7ebc199f 100644
--- a/resources/js/Pages/Profile/Partials/DeleteUserForm.vue
+++ b/resources/js/Pages/Profile/Partials/DeleteUserForm.vue
@@ -35,8 +35,7 @@ async function deleteUser() {
     } catch (error) {
         if (error && typeof error === 'object' && 'response' in error) {
             const response = error.response as
-                | { status?: number; data?: { errors?: { password?: string[] } } }
-                | undefined;
+                { status?: number; data?: { errors?: { password?: string[] } } } | undefined;
             if (response?.status === 422) {
                 passwordError.value = response.data?.errors?.password?.[0] ?? 'Invalid password.';
             }
diff --git a/resources/js/Pages/Teams/Partials/DeleteTeamForm.vue b/resources/js/Pages/Teams/Partials/DeleteTeamForm.vue
index e93b3fc9..5433aac5 100644
--- a/resources/js/Pages/Teams/Partials/DeleteTeamForm.vue
+++ b/resources/js/Pages/Teams/Partials/DeleteTeamForm.vue
@@ -38,8 +38,7 @@ const deleteTeam = async () => {
     } catch (error) {
         if (error && typeof error === 'object' && 'response' in error) {
             const response = error.response as
-                | { status?: number; data?: { errors?: { password?: string[] } } }
-                | undefined;
+                { status?: number; data?: { errors?: { password?: string[] } } } | undefined;
             if (response?.status === 422) {
                 passwordError.value = response.data?.errors?.password?.[0] ?? 'Invalid password.';
             }
diff --git a/resources/js/packages/api/src/index.ts b/resources/js/packages/api/src/index.ts
index 05fc7afd..6c0370c8 100644
--- a/resources/js/packages/api/src/index.ts
+++ b/resources/js/packages/api/src/index.ts
@@ -118,7 +118,10 @@ export type DetailedInvoiceResponse = ZodiosResponseByAlias<SolidTimeApi, 'getIn
 export type DetailedInvoice = DetailedInvoiceResponse['data'];
 
 export type InvoiceIndexEntry = ZodiosResponseByAlias<SolidTimeApi, 'getInvoices'>['data'][0];
-export type InvoiceRecipient = ZodiosResponseByAlias<SolidTimeApi, 'getInvoiceRecipients'>['data'][0];
+export type InvoiceRecipient = ZodiosResponseByAlias<
+    SolidTimeApi,
+    'getInvoiceRecipients'
+>['data'][0];
 export type InvoiceRecipientBody = ZodiosBodyByAlias<SolidTimeApi, 'createInvoiceRecipient'>;
 
 export type UpdateInvoiceSettings = ZodiosBodyByAlias<SolidTimeApi, 'updateInvoiceSettings'>;
diff --git a/resources/js/packages/ui/src/Input/BillableToggleButton.vue b/resources/js/packages/ui/src/Input/BillableToggleButton.vue
index e856a823..8bfc68ee 100644
--- a/resources/js/packages/ui/src/Input/BillableToggleButton.vue
+++ b/resources/js/packages/ui/src/Input/BillableToggleButton.vue
@@ -8,7 +8,7 @@ import {
     TooltipProvider,
     TooltipTrigger,
 } from '@/packages/ui/src/tooltip';
-const active = defineModel({ default: false });
+const active = defineModel<boolean>({ default: false });
 const emit = defineEmits(['changed']);
 function toggleBillable() {
     active.value = !active.value;
diff --git a/resources/js/packages/ui/src/Input/Dropdown.vue b/resources/js/packages/ui/src/Input/Dropdown.vue
index 3283eb9c..be8c3824 100644
--- a/resources/js/packages/ui/src/Input/Dropdown.vue
+++ b/resources/js/packages/ui/src/Input/Dropdown.vue
@@ -16,7 +16,7 @@ const props = withDefaults(
 );
 
 const emit = defineEmits(['open', 'submit']);
-const open = defineModel({ default: false });
+const open = defineModel<boolean>({ default: false });
 
 function handleAutofocus(event: Event) {
     if (props.autoFocus === false) {
diff --git a/resources/js/packages/ui/src/Input/DurationHumanInput.vue b/resources/js/packages/ui/src/Input/DurationHumanInput.vue
index 2c0e44f3..f5649c41 100644
--- a/resources/js/packages/ui/src/Input/DurationHumanInput.vue
+++ b/resources/js/packages/ui/src/Input/DurationHumanInput.vue
@@ -13,11 +13,11 @@ import { type ComputedRef } from 'vue';
 
 const temporaryCustomTimerEntry = ref<string>('');
 
-const start = defineModel('start', {
+const start = defineModel<string>('start', {
     default: '',
 });
 
-const end = defineModel('end', {
+const end = defineModel<string>('end', {
     default: '',
 });
 
diff --git a/resources/js/packages/ui/src/Input/MultiselectDropdown.vue b/resources/js/packages/ui/src/Input/MultiselectDropdown.vue
index e5fbf439..0164e54d 100644
--- a/resources/js/packages/ui/src/Input/MultiselectDropdown.vue
+++ b/resources/js/packages/ui/src/Input/MultiselectDropdown.vue
@@ -20,7 +20,7 @@ const NONE_ID = 'none';
 const ROW_HEIGHT = 32;
 
 const model = defineModel<string[]>({
-    default: [],
+    default: () => [],
 });
 
 const props = defineProps<{
diff --git a/resources/js/packages/ui/src/Tag/TagDropdown.vue b/resources/js/packages/ui/src/Tag/TagDropdown.vue
index d69b371b..ca432c1b 100644
--- a/resources/js/packages/ui/src/Tag/TagDropdown.vue
+++ b/resources/js/packages/ui/src/Tag/TagDropdown.vue
@@ -32,7 +32,7 @@ const props = withDefaults(
 );
 
 const model = defineModel<string[]>({
-    default: [],
+    default: () => [],
 });
 
 const open = ref(false);
diff --git a/resources/js/packages/ui/src/TimeEntry/TimeEntryGroupedTable.vue b/resources/js/packages/ui/src/TimeEntry/TimeEntryGroupedTable.vue
index 8f004a17..0d7788fb 100644
--- a/resources/js/packages/ui/src/TimeEntry/TimeEntryGroupedTable.vue
+++ b/resources/js/packages/ui/src/TimeEntry/TimeEntryGroupedTable.vue
@@ -21,7 +21,7 @@ import TimeEntryRow from '@/packages/ui/src/TimeEntry/TimeEntryRow.vue';
 import type { TimeEntriesGroupedByType } from '@/types/time-entries';
 
 const selectedTimeEntries = defineModel<TimeEntry[]>('selected', {
-    default: [],
+    default: () => [],
 });
 
 const props = withDefaults(
diff --git a/resources/js/packages/ui/src/TimeEntry/TimeEntryRowTagDropdown.vue b/resources/js/packages/ui/src/TimeEntry/TimeEntryRowTagDropdown.vue
index 46e91a93..d24a4ad0 100644
--- a/resources/js/packages/ui/src/TimeEntry/TimeEntryRowTagDropdown.vue
+++ b/resources/js/packages/ui/src/TimeEntry/TimeEntryRowTagDropdown.vue
@@ -20,7 +20,7 @@ const emit = defineEmits<{
 }>();
 
 const model = defineModel<string[]>({
-    default: [],
+    default: () => [],
 });
 
 const timeEntryTags = computed<Tag[]>(() => {
diff --git a/resources/js/packages/ui/src/TimeTracker/TimeTrackerTagDropdown.vue b/resources/js/packages/ui/src/TimeTracker/TimeTrackerTagDropdown.vue
index 70fe77c6..8e3f3c32 100644
--- a/resources/js/packages/ui/src/TimeTracker/TimeTrackerTagDropdown.vue
+++ b/resources/js/packages/ui/src/TimeTracker/TimeTrackerTagDropdown.vue
@@ -10,7 +10,7 @@ const emit = defineEmits<{
 }>();
 
 const model = defineModel<string[]>({
-    default: [],
+    default: () => [],
 });
 const iconColorClasses = computed(() => {
     if (model.value.length > 0) {
diff --git a/resources/js/packages/ui/src/TimezoneMismatchModal.vue b/resources/js/packages/ui/src/TimezoneMismatchModal.vue
index 31cf61f3..57d8c301 100644
--- a/resources/js/packages/ui/src/TimezoneMismatchModal.vue
+++ b/resources/js/packages/ui/src/TimezoneMismatchModal.vue
@@ -7,7 +7,7 @@ import { getUserTimezone } from './utils/settings';
 import { getDayJsInstance } from './utils/time';
 import { useSessionStorage } from '@vueuse/core';
 
-const show = defineModel('show', { default: false });
+const show = defineModel<boolean>('show', { default: false });
 
 const emit = defineEmits<{
     update: [timezone: string];
diff --git a/resources/js/packages/ui/src/utils/number.ts b/resources/js/packages/ui/src/utils/number.ts
index f8d943f3..f1d39d5e 100644
--- a/resources/js/packages/ui/src/utils/number.ts
+++ b/resources/js/packages/ui/src/utils/number.ts
@@ -1,9 +1,5 @@
 export type NumberFormat =
-    | 'point-comma'
-    | 'comma-point'
-    | 'space-comma'
-    | 'space-point'
-    | 'apostrophe-point';
+    'point-comma' | 'comma-point' | 'space-comma' | 'space-point' | 'apostrophe-point';
 
 /**
  * Formats a number according to the specified format
diff --git a/resources/js/utils/commandPaletteCommands.ts b/resources/js/utils/commandPaletteCommands.ts
index 90ff8d8b..c264fe2b 100644
--- a/resources/js/utils/commandPaletteCommands.ts
+++ b/resources/js/utils/commandPaletteCommands.ts
@@ -27,13 +27,7 @@ import BillableIcon from '@/packages/ui/src/Icons/BillableIcon.vue';
 import type { Organization } from '@/types/models';
 
 export type CommandGroup =
-    | 'timer'
-    | 'active-timer'
-    | 'navigation'
-    | 'create'
-    | 'theme'
-    | 'organization'
-    | 'entity';
+    'timer' | 'active-timer' | 'navigation' | 'create' | 'theme' | 'organization' | 'entity';
 
 export interface Command {
     id: string;
diff --git a/resources/js/utils/useCommandPalette.ts b/resources/js/utils/useCommandPalette.ts
index d0fde023..26d25a41 100644
--- a/resources/js/utils/useCommandPalette.ts
+++ b/resources/js/utils/useCommandPalette.ts
@@ -385,21 +385,19 @@ export function useCommandPalette() {
             const matching = projects.value
                 .filter((p: Project) => p.name.toLowerCase().includes(query))
                 .slice(0, maxPerType)
-                .map(
-                    (p: Project): EntitySearchResult => ({
-                        id: `entity-project-${p.id}`,
-                        label: p.name,
-                        icon: ENTITY_ICONS.project,
-                        keywords: ['project'],
-                        action: () => {
-                            closePaletteAfterAction();
-                            router.visit(route('projects.show', { project: p.id }));
-                        },
-                        entityType: 'project',
-                        color: p.color,
-                        badgeClass: ENTITY_BADGE_CLASSES.project,
-                    })
-                );
+                .map((p: Project): EntitySearchResult => ({
+                    id: `entity-project-${p.id}`,
+                    label: p.name,
+                    icon: ENTITY_ICONS.project,
+                    keywords: ['project'],
+                    action: () => {
+                        closePaletteAfterAction();
+                        router.visit(route('projects.show', { project: p.id }));
+                    },
+                    entityType: 'project',
+                    color: p.color,
+                    badgeClass: ENTITY_BADGE_CLASSES.project,
+                }));
             results.push(...matching);
         }
 
@@ -407,20 +405,18 @@ export function useCommandPalette() {
             const matching = clients.value
                 .filter((c: Client) => c.name.toLowerCase().includes(query))
                 .slice(0, maxPerType)
-                .map(
-                    (c: Client): EntitySearchResult => ({
-                        id: `entity-client-${c.id}`,
-                        label: c.name,
-                        icon: ENTITY_ICONS.client,
-                        keywords: ['client'],
-                        action: () => {
-                            closePaletteAfterAction();
-                            router.visit(route('clients'));
-                        },
-                        entityType: 'client',
-                        badgeClass: ENTITY_BADGE_CLASSES.client,
-                    })
-                );
+                .map((c: Client): EntitySearchResult => ({
+                    id: `entity-client-${c.id}`,
+                    label: c.name,
+                    icon: ENTITY_ICONS.client,
+                    keywords: ['client'],
+                    action: () => {
+                        closePaletteAfterAction();
+                        router.visit(route('clients'));
+                    },
+                    entityType: 'client',
+                    badgeClass: ENTITY_BADGE_CLASSES.client,
+                }));
             results.push(...matching);
         }
 
@@ -428,22 +424,20 @@ export function useCommandPalette() {
             const matching = tasks.value
                 .filter((t: Task) => t.name.toLowerCase().includes(query))
                 .slice(0, maxPerType)
-                .map(
-                    (t: Task): EntitySearchResult => ({
-                        id: `entity-task-${t.id}`,
-                        label: t.name,
-                        icon: ENTITY_ICONS.task,
-                        keywords: ['task'],
-                        action: () => {
-                            closePaletteAfterAction();
-                            if (t.project_id) {
-                                router.visit(route('projects.show', { project: t.project_id }));
-                            }
-                        },
-                        entityType: 'task',
-                        badgeClass: ENTITY_BADGE_CLASSES.task,
-                    })
-                );
+                .map((t: Task): EntitySearchResult => ({
+                    id: `entity-task-${t.id}`,
+                    label: t.name,
+                    icon: ENTITY_ICONS.task,
+                    keywords: ['task'],
+                    action: () => {
+                        closePaletteAfterAction();
+                        if (t.project_id) {
+                            router.visit(route('projects.show', { project: t.project_id }));
+                        }
+                    },
+                    entityType: 'task',
+                    badgeClass: ENTITY_BADGE_CLASSES.task,
+                }));
             results.push(...matching);
         }
 
@@ -451,20 +445,18 @@ export function useCommandPalette() {
             const matching = tags.value
                 .filter((t: Tag) => t.name.toLowerCase().includes(query))
                 .slice(0, maxPerType)
-                .map(
-                    (t: Tag): EntitySearchResult => ({
-                        id: `entity-tag-${t.id}`,
-                        label: t.name,
-                        icon: ENTITY_ICONS.tag,
-                        keywords: ['tag'],
-                        action: () => {
-                            closePaletteAfterAction();
-                            router.visit(route('tags'));
-                        },
-                        entityType: 'tag',
-                        badgeClass: ENTITY_BADGE_CLASSES.tag,
-                    })
-                );
+                .map((t: Tag): EntitySearchResult => ({
+                    id: `entity-tag-${t.id}`,
+                    label: t.name,
+                    icon: ENTITY_ICONS.tag,
+                    keywords: ['tag'],
+                    action: () => {
+                        closePaletteAfterAction();
+                        router.visit(route('tags'));
+                    },
+                    entityType: 'tag',
+                    badgeClass: ENTITY_BADGE_CLASSES.tag,
+                }));
             results.push(...matching);
         }
 
@@ -472,20 +464,18 @@ export function useCommandPalette() {
             const matching = members.value
                 .filter((m: Member) => m.name.toLowerCase().includes(query))
                 .slice(0, maxPerType)
-                .map(
-                    (m: Member): EntitySearchResult => ({
-                        id: `entity-member-${m.id}`,
-                        label: m.name,
-                        icon: ENTITY_ICONS.member,
-                        keywords: ['member'],
-                        action: () => {
-                            closePaletteAfterAction();
-                            router.visit(route('members'));
-                        },
-                        entityType: 'member',
-                        badgeClass: ENTITY_BADGE_CLASSES.member,
-                    })
-                );
+                .map((m: Member): EntitySearchResult => ({
+                    id: `entity-member-${m.id}`,
+                    label: m.name,
+                    icon: ENTITY_ICONS.member,
+                    keywords: ['member'],
+                    action: () => {
+                        closePaletteAfterAction();
+                        router.visit(route('members'));
+                    },
+                    entityType: 'member',
+                    badgeClass: ENTITY_BADGE_CLASSES.member,
+                }));
             results.push(...matching);
         }