diff --git a/src/components/admin/admin-sections-browser.tsx b/src/components/admin/admin-sections-browser.tsx
index 821c9ae1..2f6001f5 100644
--- a/src/components/admin/admin-sections-browser.tsx
+++ b/src/components/admin/admin-sections-browser.tsx
@@ -377,7 +377,10 @@ export function AdminSectionsBrowser({ portSlug }: AdminSectionsBrowserProps) {
   return (
     <div className="space-y-6">
       <div className="relative max-w-md">
-        <Search className="pointer-events-none absolute left-3 top-1/2 h-4 w-4 -translate-y-1/2 text-muted-foreground" />
+        <Search
+          className="pointer-events-none absolute left-3 top-1/2 h-4 w-4 -translate-y-1/2 text-muted-foreground"
+          aria-hidden
+        />
         <Input
           type="search"
           inputMode="search"
@@ -466,7 +469,10 @@ function SectionCard({
         )}
       >
         <CardHeader className="flex flex-row items-start gap-3 space-y-0 pb-2">
-          <Icon className="h-5 w-5 mt-0.5 text-muted-foreground group-hover:text-primary" />
+          <Icon
+            className="h-5 w-5 mt-0.5 text-muted-foreground group-hover:text-primary"
+            aria-hidden
+          />
           <div className="flex-1">
             <CardTitle className="text-base">{section.label}</CardTitle>
             {groupTitle ? (
diff --git a/src/components/admin/ai-budget-card.tsx b/src/components/admin/ai-budget-card.tsx
index 89db298a..e5833bb4 100644
--- a/src/components/admin/ai-budget-card.tsx
+++ b/src/components/admin/ai-budget-card.tsx
@@ -88,7 +88,7 @@ function AiBudgetCardBody({
           <CardTitle>AI cost guardrails</CardTitle>
         </CardHeader>
         <CardContent className="flex items-center gap-2 text-sm text-muted-foreground">
-          <Loader2 className="h-4 w-4 animate-spin" /> Loading…
+          <Loader2 className="h-4 w-4 animate-spin" aria-hidden /> Loading…
         </CardContent>
       </Card>
     );
@@ -198,7 +198,9 @@ function AiBudgetCardBody({
 
         <div className="flex gap-2">
           <Button onClick={() => save.mutate()} disabled={save.isPending}>
-            {save.isPending ? <Loader2 className="mr-1.5 h-3 w-3 animate-spin" /> : null}
+            {save.isPending ? (
+              <Loader2 className="mr-1.5 h-3 w-3 animate-spin" aria-hidden />
+            ) : null}
             Save guardrails
           </Button>
         </div>
diff --git a/src/components/admin/audit/audit-log-card.tsx b/src/components/admin/audit/audit-log-card.tsx
index a9ede4d7..141ce0ac 100644
--- a/src/components/admin/audit/audit-log-card.tsx
+++ b/src/components/admin/audit/audit-log-card.tsx
@@ -44,11 +44,11 @@ const ACTION_BADGE_COLORS: Record<string, string> = {
 };
 
 function ActionIcon({ action }: { action: string }) {
-  if (action === 'create') return <Plus className="h-5 w-5" />;
-  if (action === 'update') return <Pencil className="h-5 w-5" />;
-  if (action === 'delete') return <Trash2 className="h-5 w-5" />;
-  if (action === 'viewed') return <Eye className="h-5 w-5" />;
-  return <Activity className="h-5 w-5" />;
+  if (action === 'create') return <Plus className="h-5 w-5" aria-hidden />;
+  if (action === 'update') return <Pencil className="h-5 w-5" aria-hidden />;
+  if (action === 'delete') return <Trash2 className="h-5 w-5" aria-hidden />;
+  if (action === 'viewed') return <Eye className="h-5 w-5" aria-hidden />;
+  return <Activity className="h-5 w-5" aria-hidden />;
 }
 
 function actionVerb(action: string): string {
@@ -114,7 +114,7 @@ export function AuditLogCard({ entry }: AuditLogCardProps) {
 
           {/* Timestamp + IP meta line */}
           <div className="mt-1 flex flex-wrap items-center gap-x-2 gap-y-0.5 text-xs text-muted-foreground">
-            <ListCardMeta icon={<Clock className="h-3 w-3" />}>
+            <ListCardMeta icon={<Clock className="h-3 w-3" aria-hidden />}>
               {formatDistanceToNow(new Date(entry.createdAt), { addSuffix: true })}
             </ListCardMeta>
             {entry.ipAddress ? (
diff --git a/src/components/admin/audit/audit-log-list.tsx b/src/components/admin/audit/audit-log-list.tsx
index 5c982112..9efd511d 100644
--- a/src/components/admin/audit/audit-log-list.tsx
+++ b/src/components/admin/audit/audit-log-list.tsx
@@ -351,7 +351,10 @@ export function AuditLogList() {
             Search
           </Label>
           <div className="relative w-72">
-            <Search className="absolute left-3 top-1/2 -translate-y-1/2 h-4 w-4 text-muted-foreground" />
+            <Search
+              className="absolute left-3 top-1/2 -translate-y-1/2 h-4 w-4 text-muted-foreground"
+              aria-hidden
+            />
             <Input
               id="audit-search"
               className="pl-9 h-9"
diff --git a/src/components/admin/backup-admin-panel.tsx b/src/components/admin/backup-admin-panel.tsx
index 8ee44bef..aee8d433 100644
--- a/src/components/admin/backup-admin-panel.tsx
+++ b/src/components/admin/backup-admin-panel.tsx
@@ -102,7 +102,7 @@ export function BackupAdminPanel() {
         <CardHeader className="flex flex-row items-start justify-between gap-3 space-y-0">
           <div>
             <CardTitle className="text-base flex items-center gap-2">
-              <Database className="h-4 w-4" />
+              <Database className="h-4 w-4" aria-hidden />
               Run a backup now
             </CardTitle>
             <CardDescription>
@@ -112,11 +112,11 @@ export function BackupAdminPanel() {
           </div>
           <div className="flex gap-2">
             <Button variant="ghost" size="sm" onClick={load} disabled={loading || running}>
-              <RefreshCw className="mr-1.5 h-3.5 w-3.5" />
+              <RefreshCw className="mr-1.5 h-3.5 w-3.5" aria-hidden />
               Refresh
             </Button>
             <Button onClick={() => setConfirmOpen(true)} disabled={running}>
-              {running && <Loader2 className="mr-1.5 h-4 w-4 animate-spin" />}
+              {running && <Loader2 className="mr-1.5 h-4 w-4 animate-spin" aria-hidden />}
               {running ? 'Running…' : 'Run backup'}
             </Button>
           </div>
@@ -136,7 +136,7 @@ export function BackupAdminPanel() {
         <CardContent>
           {loading ? (
             <div className="flex items-center gap-2 text-sm text-muted-foreground">
-              <Loader2 className="h-4 w-4 animate-spin" /> Loading…
+              <Loader2 className="h-4 w-4 animate-spin" aria-hidden /> Loading…
             </div>
           ) : jobs.length === 0 ? (
             <p className="text-sm text-muted-foreground">No backups yet.</p>
@@ -168,7 +168,7 @@ export function BackupAdminPanel() {
                     </span>
                     {j.status === 'completed' && (
                       <Button size="sm" variant="outline" onClick={() => download(j.id)}>
-                        <Download className="mr-1.5 h-3.5 w-3.5" />
+                        <Download className="mr-1.5 h-3.5 w-3.5" aria-hidden />
                         Download
                       </Button>
                     )}
@@ -184,7 +184,7 @@ export function BackupAdminPanel() {
         <DialogContent>
           <DialogHeader>
             <DialogTitle className="flex items-center gap-2">
-              <AlertTriangle className="h-5 w-5 text-amber-600" />
+              <AlertTriangle className="h-5 w-5 text-amber-600" aria-hidden />
               Run a fresh backup now?
             </DialogTitle>
             <DialogDescription>
diff --git a/src/components/admin/branding/pdf-logo-uploader.tsx b/src/components/admin/branding/pdf-logo-uploader.tsx
index be9d040b..99b4f7b3 100644
--- a/src/components/admin/branding/pdf-logo-uploader.tsx
+++ b/src/components/admin/branding/pdf-logo-uploader.tsx
@@ -207,7 +207,7 @@ export function PdfLogoUploader() {
 
         {loading ? (
           <div className="flex items-center gap-2 text-sm text-muted-foreground">
-            <Loader2 className="h-4 w-4 animate-spin" /> Loading current logo…
+            <Loader2 className="h-4 w-4 animate-spin" aria-hidden /> Loading current logo…
           </div>
         ) : current ? (
           <div className="flex items-start gap-4 rounded-md border p-4">
@@ -232,7 +232,7 @@ export function PdfLogoUploader() {
                   Test with sample PDF
                 </Button>
                 <Button variant="outline" size="sm" onClick={clear} disabled={working}>
-                  <Trash2 className="mr-1 h-3 w-3" /> Remove
+                  <Trash2 className="mr-1 h-3 w-3" aria-hidden /> Remove
                 </Button>
               </div>
             </div>
@@ -292,9 +292,9 @@ export function PdfLogoUploader() {
             <div className="flex gap-2">
               <Button onClick={upload} disabled={working}>
                 {working ? (
-                  <Loader2 className="mr-1 h-3 w-3 animate-spin" />
+                  <Loader2 className="mr-1 h-3 w-3 animate-spin" aria-hidden />
                 ) : (
-                  <Upload className="mr-1 h-3 w-3" />
+                  <Upload className="mr-1 h-3 w-3" aria-hidden />
                 )}
                 Save logo
               </Button>
@@ -316,7 +316,7 @@ export function PdfLogoUploader() {
                 onClick={() => fileInputRef.current?.click()}
                 disabled={working}
               >
-                <RefreshCw className="mr-1 h-3 w-3" /> Choose a different file
+                <RefreshCw className="mr-1 h-3 w-3" aria-hidden /> Choose a different file
               </Button>
             </div>
           </div>
diff --git a/src/components/admin/brochures-admin-panel.tsx b/src/components/admin/brochures-admin-panel.tsx
index bcdaec94..0d356ada 100644
--- a/src/components/admin/brochures-admin-panel.tsx
+++ b/src/components/admin/brochures-admin-panel.tsx
@@ -69,13 +69,13 @@ export function BrochuresAdminPanel() {
     <div className="space-y-4">
       <div className="flex justify-end">
         <Button onClick={() => setCreateOpen(true)}>
-          <Plus className="mr-2 h-4 w-4" /> New brochure
+          <Plus className="mr-2 h-4 w-4" aria-hidden /> New brochure
         </Button>
       </div>
 
       {brochuresQuery.isLoading && (
         <div className="flex items-center gap-2 text-sm text-muted-foreground">
-          <Loader2 className="h-4 w-4 animate-spin" /> Loading…
+          <Loader2 className="h-4 w-4 animate-spin" aria-hidden /> Loading…
         </div>
       )}
 
@@ -179,10 +179,10 @@ function BrochureCard({ brochure, onChange }: { brochure: BrochureRow; onChange:
       <CardHeader>
         <CardTitle className="flex items-center justify-between text-base">
           <span className="flex items-center gap-2">
-            <FileText className="h-4 w-4" /> {brochure.label}
+            <FileText className="h-4 w-4" aria-hidden /> {brochure.label}
             {brochure.isDefault && (
               <span className="flex items-center gap-1 rounded bg-primary/10 px-2 py-0.5 text-xs text-primary">
-                <Star className="h-3 w-3" /> default
+                <Star className="h-3 w-3" aria-hidden /> default
               </span>
             )}
             {brochure.archivedAt && (
@@ -222,9 +222,9 @@ function BrochureCard({ brochure, onChange }: { brochure: BrochureRow; onChange:
                 <Button asChild variant="outline" size="sm" disabled={uploading}>
                   <span>
                     {uploading ? (
-                      <Loader2 className="mr-2 h-3 w-3 animate-spin" />
+                      <Loader2 className="mr-2 h-3 w-3 animate-spin" aria-hidden />
                     ) : (
-                      <Upload className="mr-2 h-3 w-3" />
+                      <Upload className="mr-2 h-3 w-3" aria-hidden />
                     )}
                     Upload version
                   </span>
@@ -237,7 +237,7 @@ function BrochureCard({ brochure, onChange }: { brochure: BrochureRow; onChange:
                   onClick={() => setDefaultMutation.mutate()}
                   disabled={setDefaultMutation.isPending}
                 >
-                  <Star className="mr-2 h-3 w-3" /> Mark default
+                  <Star className="mr-2 h-3 w-3" aria-hidden /> Mark default
                 </Button>
               )}
               <Button
@@ -246,7 +246,7 @@ function BrochureCard({ brochure, onChange }: { brochure: BrochureRow; onChange:
                 onClick={() => archiveMutation.mutate()}
                 disabled={archiveMutation.isPending}
               >
-                <Archive className="mr-2 h-3 w-3" /> Archive
+                <Archive className="mr-2 h-3 w-3" aria-hidden /> Archive
               </Button>
             </>
           )}
@@ -331,7 +331,9 @@ function CreateBrochureDialog({
             disabled={!label.trim() || createMutation.isPending}
             onClick={() => createMutation.mutate()}
           >
-            {createMutation.isPending && <Loader2 className="mr-2 h-4 w-4 animate-spin" />}
+            {createMutation.isPending && (
+              <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />
+            )}
             Create
           </Button>
         </DialogFooter>
diff --git a/src/components/admin/custom-fields/custom-field-form.tsx b/src/components/admin/custom-fields/custom-field-form.tsx
index 17b520a3..7ca55d83 100644
--- a/src/components/admin/custom-fields/custom-field-form.tsx
+++ b/src/components/admin/custom-fields/custom-field-form.tsx
@@ -263,7 +263,7 @@ function CustomFieldFormBody({ open, onOpenChange, field, onSuccess }: CustomFie
                   maxLength={100}
                 />
                 <Button type="button" variant="outline" size="sm" onClick={addOption}>
-                  <Plus className="h-4 w-4" />
+                  <Plus className="h-4 w-4" aria-hidden />
                 </Button>
               </div>
               {selectOptions.length > 0 && (
diff --git a/src/components/admin/custom-fields/custom-fields-manager.tsx b/src/components/admin/custom-fields/custom-fields-manager.tsx
index c778de8a..03442d6d 100644
--- a/src/components/admin/custom-fields/custom-fields-manager.tsx
+++ b/src/components/admin/custom-fields/custom-fields-manager.tsx
@@ -124,7 +124,7 @@ export function CustomFieldsManager() {
       cell: ({ row }) => (
         <div className="flex items-center justify-end gap-1">
           <Button variant="ghost" size="sm" onClick={() => handleEdit(row.original)}>
-            <Pencil className="h-4 w-4" />
+            <Pencil className="h-4 w-4" aria-hidden />
             <span className="sr-only">Edit</span>
           </Button>
           <ConfirmationDialog
@@ -135,7 +135,7 @@ export function CustomFieldsManager() {
                 className="text-destructive hover:text-destructive"
                 disabled={deletingId === row.original.id}
               >
-                <Trash2 className="h-4 w-4" />
+                <Trash2 className="h-4 w-4" aria-hidden />
                 <span className="sr-only">Delete</span>
               </Button>
             }
@@ -159,7 +159,7 @@ export function CustomFieldsManager() {
         description="Define custom fields for clients and records"
         actions={
           <Button onClick={handleCreate}>
-            <Plus className="mr-1.5 h-4 w-4" />
+            <Plus className="mr-1.5 h-4 w-4" aria-hidden />
             New Field
           </Button>
         }
@@ -228,7 +228,7 @@ export function CustomFieldsManager() {
                         onClick={() => handleEdit(original)}
                         aria-label="Edit field"
                       >
-                        <Pencil className="h-4 w-4" />
+                        <Pencil className="h-4 w-4" aria-hidden />
                       </Button>
                       <ConfirmationDialog
                         trigger={
@@ -239,7 +239,7 @@ export function CustomFieldsManager() {
                             disabled={deletingId === original.id}
                             aria-label="Delete field"
                           >
-                            <Trash2 className="h-4 w-4" />
+                            <Trash2 className="h-4 w-4" aria-hidden />
                           </Button>
                         }
                         title="Delete Custom Field"
diff --git a/src/components/admin/documenso/documenso-test-button.tsx b/src/components/admin/documenso/documenso-test-button.tsx
index aca6db8e..bb5e9f34 100644
--- a/src/components/admin/documenso/documenso-test-button.tsx
+++ b/src/components/admin/documenso/documenso-test-button.tsx
@@ -47,17 +47,17 @@ export function DocumensoTestButton() {
       {result &&
         (result.ok ? (
           <span className="flex items-center text-xs text-green-600">
-            <CheckCircle2 className="mr-1 h-3.5 w-3.5" />
+            <CheckCircle2 className="mr-1 h-3.5 w-3.5" aria-hidden />
             HTTP {result.status}
           </span>
         ) : (
           <span className="flex items-center text-xs text-destructive">
-            <XCircle className="mr-1 h-3.5 w-3.5" />
+            <XCircle className="mr-1 h-3.5 w-3.5" aria-hidden />
             {result.error ?? `HTTP ${result.status}`}
           </span>
         ))}
       <Button variant="outline" onClick={runTest} disabled={pending}>
-        {pending && <Loader2 className="mr-2 h-4 w-4 animate-spin" />}
+        {pending && <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />}
         Test connection
       </Button>
     </div>
diff --git a/src/components/admin/document-templates/template-list.tsx b/src/components/admin/document-templates/template-list.tsx
index 1bf7a2d2..820cace1 100644
--- a/src/components/admin/document-templates/template-list.tsx
+++ b/src/components/admin/document-templates/template-list.tsx
@@ -94,7 +94,7 @@ export function TemplateList() {
       header: 'Name',
       cell: ({ row }) => (
         <div className="flex items-center gap-2">
-          <FileText className="h-4 w-4 shrink-0 text-muted-foreground" />
+          <FileText className="h-4 w-4 shrink-0 text-muted-foreground" aria-hidden />
           <span className="font-medium">{row.original.name}</span>
         </div>
       ),
@@ -147,7 +147,7 @@ export function TemplateList() {
             title="Version history"
             onClick={() => handleViewHistory(row.original)}
           >
-            <History className="h-4 w-4" />
+            <History className="h-4 w-4" aria-hidden />
           </Button>
           <Button
             variant="ghost"
@@ -155,7 +155,7 @@ export function TemplateList() {
             title="Edit template"
             onClick={() => handleEditTemplate(row.original)}
           >
-            <Pencil className="h-4 w-4" />
+            <Pencil className="h-4 w-4" aria-hidden />
           </Button>
           <Button
             variant="ghost"
@@ -168,7 +168,7 @@ export function TemplateList() {
           <ConfirmationDialog
             trigger={
               <Button variant="ghost" size="icon" title="Delete template">
-                <Trash2 className="h-4 w-4 text-destructive" />
+                <Trash2 className="h-4 w-4 text-destructive" aria-hidden />
               </Button>
             }
             title="Delete Template"
@@ -189,7 +189,7 @@ export function TemplateList() {
         description="Manage reusable document templates with TipTap content and PDF generation"
         actions={
           <Button onClick={handleNewTemplate}>
-            <Plus className="mr-2 h-4 w-4" />
+            <Plus className="mr-2 h-4 w-4" aria-hidden />
             New Template
           </Button>
         }
@@ -237,7 +237,7 @@ export function TemplateList() {
                   title="Version history"
                   onClick={() => handleViewHistory(original)}
                 >
-                  <History className="h-4 w-4" />
+                  <History className="h-4 w-4" aria-hidden />
                 </Button>
                 <Button
                   variant="ghost"
@@ -245,12 +245,12 @@ export function TemplateList() {
                   title="Edit template"
                   onClick={() => handleEditTemplate(original)}
                 >
-                  <Pencil className="h-4 w-4" />
+                  <Pencil className="h-4 w-4" aria-hidden />
                 </Button>
                 <ConfirmationDialog
                   trigger={
                     <Button variant="ghost" size="icon" title="Delete template">
-                      <Trash2 className="h-4 w-4 text-destructive" />
+                      <Trash2 className="h-4 w-4 text-destructive" aria-hidden />
                     </Button>
                   }
                   title="Delete Template"
diff --git a/src/components/admin/document-templates/template-preview.tsx b/src/components/admin/document-templates/template-preview.tsx
index 10de7773..c406cd64 100644
--- a/src/components/admin/document-templates/template-preview.tsx
+++ b/src/components/admin/document-templates/template-preview.tsx
@@ -59,7 +59,7 @@ export function TemplatePreview({ content, templateName }: TemplatePreviewProps)
   return (
     <>
       <Button variant="outline" size="sm" onClick={handlePreview}>
-        <Eye className="mr-1.5 h-3.5 w-3.5" />
+        <Eye className="mr-1.5 h-3.5 w-3.5" aria-hidden />
         Preview PDF
       </Button>
 
@@ -70,7 +70,7 @@ export function TemplatePreview({ content, templateName }: TemplatePreviewProps)
               <DialogTitle>Preview - {templateName}</DialogTitle>
               {pdfBase64 && (
                 <Button variant="ghost" size="sm" onClick={handleOpenInNewTab} className="mr-6">
-                  <ExternalLink className="mr-1.5 h-3.5 w-3.5" />
+                  <ExternalLink className="mr-1.5 h-3.5 w-3.5" aria-hidden />
                   Open in new tab
                 </Button>
               )}
diff --git a/src/components/admin/document-templates/template-version-history.tsx b/src/components/admin/document-templates/template-version-history.tsx
index 69c697dc..2efd16bb 100644
--- a/src/components/admin/document-templates/template-version-history.tsx
+++ b/src/components/admin/document-templates/template-version-history.tsx
@@ -81,7 +81,7 @@ export function TemplateVersionHistory({
   if (versions.length === 0) {
     return (
       <div className="flex flex-col items-center gap-2 rounded-md border border-dashed p-6 text-center">
-        <Clock className="h-6 w-6 text-muted-foreground" />
+        <Clock className="h-6 w-6 text-muted-foreground" aria-hidden />
         <p className="text-sm text-muted-foreground">
           No previous versions found. Versions are saved whenever you update the template content.
         </p>
@@ -129,7 +129,7 @@ export function TemplateVersionHistory({
               onClick={() => handleRollback(v.version)}
               disabled={rollingBack === v.version}
             >
-              <RotateCcw className="mr-1.5 h-3.5 w-3.5" />
+              <RotateCcw className="mr-1.5 h-3.5 w-3.5" aria-hidden />
               {rollingBack === v.version ? 'Restoring…' : 'Restore'}
             </Button>
           </div>
diff --git a/src/components/admin/email-templates-admin.tsx b/src/components/admin/email-templates-admin.tsx
index 2227a407..22e602a8 100644
--- a/src/components/admin/email-templates-admin.tsx
+++ b/src/components/admin/email-templates-admin.tsx
@@ -150,7 +150,7 @@ function EmailTemplatesAdminBody({
                       onClick={() => save(row, 'save')}
                       disabled={savingKey === row.key || !dirty}
                     >
-                      <Save className="h-3.5 w-3.5 mr-1.5" /> Save
+                      <Save className="h-3.5 w-3.5 mr-1.5" aria-hidden /> Save
                     </Button>
                     {overridden ? (
                       <Button
@@ -159,7 +159,7 @@ function EmailTemplatesAdminBody({
                         onClick={() => save(row, 'reset')}
                         disabled={savingKey === row.key}
                       >
-                        <RotateCcw className="h-3.5 w-3.5 mr-1.5" /> Reset to default
+                        <RotateCcw className="h-3.5 w-3.5 mr-1.5" aria-hidden /> Reset to default
                       </Button>
                     ) : null}
                     {message?.key === row.key ? (
diff --git a/src/components/admin/forms/form-template-form.tsx b/src/components/admin/forms/form-template-form.tsx
index 92ee5321..276c9844 100644
--- a/src/components/admin/forms/form-template-form.tsx
+++ b/src/components/admin/forms/form-template-form.tsx
@@ -139,7 +139,7 @@ function FormTemplateFormBody({ open, onOpenChange, template, onSaved }: Props)
             <div className="flex items-center justify-between">
               <Label className="text-sm font-medium">Fields</Label>
               <Button variant="outline" size="sm" onClick={addField}>
-                <Plus className="h-3.5 w-3.5 mr-1" />
+                <Plus className="h-3.5 w-3.5 mr-1" aria-hidden />
                 Add field
               </Button>
             </div>
@@ -154,7 +154,7 @@ function FormTemplateFormBody({ open, onOpenChange, template, onSaved }: Props)
                       className="text-destructive h-7 w-7"
                       onClick={() => removeField(i)}
                     >
-                      <Trash2 className="h-3.5 w-3.5" />
+                      <Trash2 className="h-3.5 w-3.5" aria-hidden />
                     </Button>
                   )}
                 </div>
diff --git a/src/components/admin/forms/form-template-list.tsx b/src/components/admin/forms/form-template-list.tsx
index 190b48f9..6c7fdb83 100644
--- a/src/components/admin/forms/form-template-list.tsx
+++ b/src/components/admin/forms/form-template-list.tsx
@@ -57,7 +57,7 @@ export function FormTemplateList() {
               setFormOpen(true);
             }}
           >
-            <Plus className="h-4 w-4 mr-1.5" />
+            <Plus className="h-4 w-4 mr-1.5" aria-hidden />
             New template
           </Button>
         }
@@ -95,12 +95,12 @@ export function FormTemplateList() {
                   setFormOpen(true);
                 }}
               >
-                <Pencil className="h-4 w-4" />
+                <Pencil className="h-4 w-4" aria-hidden />
               </Button>
               <ConfirmationDialog
                 trigger={
                   <Button variant="ghost" size="icon" className="text-destructive">
-                    <Trash2 className="h-4 w-4" />
+                    <Trash2 className="h-4 w-4" aria-hidden />
                   </Button>
                 }
                 title="Delete form template"
diff --git a/src/components/admin/invitations/invitations-manager.tsx b/src/components/admin/invitations/invitations-manager.tsx
index 285ad5d8..b28a2543 100644
--- a/src/components/admin/invitations/invitations-manager.tsx
+++ b/src/components/admin/invitations/invitations-manager.tsx
@@ -89,7 +89,7 @@ export function InvitationsManager() {
           </p>
         </div>
         <Button onClick={() => setSheetOpen(true)}>
-          <Plus className="h-4 w-4 mr-1.5" />
+          <Plus className="h-4 w-4 mr-1.5" aria-hidden />
           Send invite
         </Button>
       </div>
@@ -98,7 +98,7 @@ export function InvitationsManager() {
         <p className="text-sm text-muted-foreground">Loading…</p>
       ) : invites.length === 0 ? (
         <div className="rounded-lg border border-dashed p-8 text-center text-muted-foreground">
-          <Mail className="mx-auto h-6 w-6 mb-2" />
+          <Mail className="mx-auto h-6 w-6 mb-2" aria-hidden />
           <p className="text-sm">No invitations issued yet.</p>
         </div>
       ) : (
@@ -142,7 +142,7 @@ export function InvitationsManager() {
                           disabled={resendMutation.isPending || !!i.usedAt}
                           title="Resend invite"
                         >
-                          <RotateCw className="h-3.5 w-3.5" />
+                          <RotateCw className="h-3.5 w-3.5" aria-hidden />
                         </Button>
                         {i.status === 'pending' && (
                           <ConfirmationDialog
@@ -153,7 +153,7 @@ export function InvitationsManager() {
                                 className="text-destructive"
                                 title="Revoke"
                               >
-                                <Trash2 className="h-3.5 w-3.5" />
+                                <Trash2 className="h-3.5 w-3.5" aria-hidden />
                               </Button>
                             }
                             title="Revoke invitation?"
@@ -230,7 +230,9 @@ export function InvitationsManager() {
                 Cancel
               </Button>
               <Button type="submit" disabled={!email.trim() || createMutation.isPending}>
-                {createMutation.isPending && <Loader2 className="mr-2 h-4 w-4 animate-spin" />}
+                {createMutation.isPending && (
+                  <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />
+                )}
                 Send invite
               </Button>
             </SheetFooter>
diff --git a/src/components/admin/ocr-settings-form.tsx b/src/components/admin/ocr-settings-form.tsx
index e6835aea..53c6acae 100644
--- a/src/components/admin/ocr-settings-form.tsx
+++ b/src/components/admin/ocr-settings-form.tsx
@@ -138,7 +138,7 @@ function SettingsBlockBody({
           <CardTitle>{title}</CardTitle>
         </CardHeader>
         <CardContent className="flex items-center gap-2 text-sm text-muted-foreground">
-          <Loader2 className="h-4 w-4 animate-spin" /> Loading…
+          <Loader2 className="h-4 w-4 animate-spin" aria-hidden /> Loading…
         </CardContent>
       </Card>
     );
@@ -250,7 +250,11 @@ function SettingsBlockBody({
               onClick={() => setShowKey((v) => !v)}
               aria-label={showKey ? 'Hide key' : 'Show key'}
             >
-              {showKey ? <EyeOff className="h-4 w-4" /> : <Eye className="h-4 w-4" />}
+              {showKey ? (
+                <EyeOff className="h-4 w-4" aria-hidden />
+              ) : (
+                <Eye className="h-4 w-4" aria-hidden />
+              )}
             </Button>
           </div>
           <p className="text-xs text-muted-foreground">
@@ -264,7 +268,9 @@ function SettingsBlockBody({
             disabled={save.isPending}
             data-testid={`save-${scope}`}
           >
-            {save.isPending ? <Loader2 className="mr-1.5 h-3 w-3 animate-spin" /> : null}
+            {save.isPending ? (
+              <Loader2 className="mr-1.5 h-3 w-3 animate-spin" aria-hidden />
+            ) : null}
             Save settings
           </Button>
           <Button
@@ -273,7 +279,9 @@ function SettingsBlockBody({
             onClick={() => test.mutate()}
             disabled={test.isPending || apiKey.length === 0}
           >
-            {test.isPending ? <Loader2 className="mr-1.5 h-3 w-3 animate-spin" /> : null}
+            {test.isPending ? (
+              <Loader2 className="mr-1.5 h-3 w-3 animate-spin" aria-hidden />
+            ) : null}
             Test connection
           </Button>
           {hasKey ? (
@@ -290,13 +298,13 @@ function SettingsBlockBody({
 
           {testStatus?.ok ? (
             <span className="inline-flex items-center gap-1 text-sm text-green-700">
-              <CheckCircle2 className="h-4 w-4" />
+              <CheckCircle2 className="h-4 w-4" aria-hidden />
               Connection OK
             </span>
           ) : null}
           {testStatus && !testStatus.ok ? (
             <span className="inline-flex items-center gap-1 text-sm text-destructive">
-              <XCircle className="h-4 w-4" />
+              <XCircle className="h-4 w-4" aria-hidden />
               {testStatus.reason}
             </span>
           ) : null}
diff --git a/src/components/admin/onboarding-checklist.tsx b/src/components/admin/onboarding-checklist.tsx
index b54672c7..e39a2c63 100644
--- a/src/components/admin/onboarding-checklist.tsx
+++ b/src/components/admin/onboarding-checklist.tsx
@@ -228,11 +228,11 @@ export function OnboardingChecklist() {
                     }
                   >
                     {done ? (
-                      <Check className="h-4 w-4" />
+                      <Check className="h-4 w-4" aria-hidden />
                     ) : loading ? (
-                      <Loader2 className="h-4 w-4 animate-spin" />
+                      <Loader2 className="h-4 w-4 animate-spin" aria-hidden />
                     ) : (
-                      <Circle className="h-4 w-4" />
+                      <Circle className="h-4 w-4" aria-hidden />
                     )}
                   </span>
                   <div className="flex-1 min-w-0">
@@ -243,7 +243,7 @@ export function OnboardingChecklist() {
                           className="text-sm font-medium hover:underline inline-flex items-center gap-1"
                         >
                           {idx + 1}. {step.label}
-                          <ExternalLink className="h-3 w-3 opacity-50" />
+                          <ExternalLink className="h-3 w-3 opacity-50" aria-hidden />
                         </Link>
                         <p className="mt-0.5 text-xs text-muted-foreground">{step.description}</p>
                         {auto && (
@@ -265,7 +265,7 @@ export function OnboardingChecklist() {
                           onClick={() => toggleManual(step.id)}
                         >
                           {saving === step.id ? (
-                            <Loader2 className="h-3.5 w-3.5 animate-spin" />
+                            <Loader2 className="h-3.5 w-3.5 animate-spin" aria-hidden />
                           ) : manual ? (
                             'Mark incomplete'
                           ) : (
diff --git a/src/components/admin/ports/port-list.tsx b/src/components/admin/ports/port-list.tsx
index e025463e..8bf85cd8 100644
--- a/src/components/admin/ports/port-list.tsx
+++ b/src/components/admin/ports/port-list.tsx
@@ -95,7 +95,7 @@ export function PortList() {
       header: '',
       cell: ({ row }) => (
         <Button variant="ghost" size="sm" onClick={() => handleEditPort(row.original)}>
-          <Pencil className="h-4 w-4" />
+          <Pencil className="h-4 w-4" aria-hidden />
           <span className="sr-only">Edit</span>
         </Button>
       ),
@@ -111,7 +111,7 @@ export function PortList() {
         description="Manage marina ports and their configuration"
         actions={
           <Button onClick={handleNewPort}>
-            <Plus className="mr-1.5 h-4 w-4" />
+            <Plus className="mr-1.5 h-4 w-4" aria-hidden />
             New Port
           </Button>
         }
@@ -154,7 +154,7 @@ export function PortList() {
               aria-label={`Edit ${original.name}`}
               className="shrink-0"
             >
-              <Pencil className="h-4 w-4" />
+              <Pencil className="h-4 w-4" aria-hidden />
             </Button>
           </div>
         )}
diff --git a/src/components/admin/queue-detail-table.tsx b/src/components/admin/queue-detail-table.tsx
index 29a50ef4..071a8ba1 100644
--- a/src/components/admin/queue-detail-table.tsx
+++ b/src/components/admin/queue-detail-table.tsx
@@ -159,7 +159,7 @@ export function QueueDetailTable({ queueName }: QueueDetailTableProps) {
                           disabled={retryMutation.isPending}
                           onClick={() => retryMutation.mutate(job.id)}
                         >
-                          <RotateCcw className="h-3.5 w-3.5" />
+                          <RotateCcw className="h-3.5 w-3.5" aria-hidden />
                         </Button>
                       )}
                       <Button
@@ -170,7 +170,7 @@ export function QueueDetailTable({ queueName }: QueueDetailTableProps) {
                         disabled={deleteMutation.isPending}
                         onClick={() => deleteMutation.mutate(job.id)}
                       >
-                        <Trash2 className="h-3.5 w-3.5" />
+                        <Trash2 className="h-3.5 w-3.5" aria-hidden />
                       </Button>
                     </div>
                   </TableCell>
diff --git a/src/components/admin/residential-stages-admin.tsx b/src/components/admin/residential-stages-admin.tsx
index b4cacb6a..072807e2 100644
--- a/src/components/admin/residential-stages-admin.tsx
+++ b/src/components/admin/residential-stages-admin.tsx
@@ -128,7 +128,7 @@ export function ResidentialStagesAdmin() {
   if (loading) {
     return (
       <div className="flex items-center gap-2 text-sm text-muted-foreground">
-        <Loader2 className="h-4 w-4 animate-spin" /> Loading stages…
+        <Loader2 className="h-4 w-4 animate-spin" aria-hidden /> Loading stages…
       </div>
     );
   }
@@ -149,7 +149,7 @@ export function ResidentialStagesAdmin() {
               key={`${stage.id}-${idx}`}
               className="flex items-center gap-3 rounded-md border p-3"
             >
-              <GripVertical className="h-4 w-4 text-muted-foreground" />
+              <GripVertical className="h-4 w-4 text-muted-foreground" aria-hidden />
               <div className="flex-1 grid grid-cols-1 sm:grid-cols-3 gap-2">
                 <div className="space-y-1">
                   <Label className="text-[11px] uppercase tracking-wide text-muted-foreground">
@@ -223,13 +223,13 @@ export function ResidentialStagesAdmin() {
                 onClick={() => remove(idx)}
                 className="text-destructive"
               >
-                <Trash2 className="h-3.5 w-3.5" />
+                <Trash2 className="h-3.5 w-3.5" aria-hidden />
               </Button>
             </div>
           ))}
 
           <Button type="button" variant="outline" size="sm" onClick={add}>
-            <Plus className="mr-1.5 h-3.5 w-3.5" />
+            <Plus className="mr-1.5 h-3.5 w-3.5" aria-hidden />
             Add stage
           </Button>
         </CardContent>
@@ -245,9 +245,9 @@ export function ResidentialStagesAdmin() {
       <div className="flex items-center gap-3">
         <Button onClick={() => attemptSave(false)} disabled={saving}>
           {saving ? (
-            <Loader2 className="mr-1.5 h-4 w-4 animate-spin" />
+            <Loader2 className="mr-1.5 h-4 w-4 animate-spin" aria-hidden />
           ) : (
-            <Save className="mr-1.5 h-4 w-4" />
+            <Save className="mr-1.5 h-4 w-4" aria-hidden />
           )}
           Save stages
         </Button>
@@ -318,7 +318,7 @@ export function ResidentialStagesAdmin() {
               }}
               disabled={saving}
             >
-              {saving && <Loader2 className="mr-1.5 h-4 w-4 animate-spin" />}
+              {saving && <Loader2 className="mr-1.5 h-4 w-4 animate-spin" aria-hidden />}
               Reassign + save
             </Button>
           </DialogFooter>
diff --git a/src/components/admin/roles/role-list.tsx b/src/components/admin/roles/role-list.tsx
index cac036ef..cca13c30 100644
--- a/src/components/admin/roles/role-list.tsx
+++ b/src/components/admin/roles/role-list.tsx
@@ -87,7 +87,7 @@ export function RoleList() {
           <span className="font-medium">{formatRole(row.original.name)}</span>
           {row.original.isSystem && (
             <Badge variant="outline" className="text-xs">
-              <Lock className="mr-1 h-3 w-3" />
+              <Lock className="mr-1 h-3 w-3" aria-hidden />
               System
             </Badge>
           )}
@@ -126,7 +126,7 @@ export function RoleList() {
       cell: ({ row }) => (
         <div className="flex items-center justify-end gap-1">
           <Button variant="ghost" size="sm" onClick={() => handleEditRole(row.original)}>
-            <Pencil className="h-4 w-4" />
+            <Pencil className="h-4 w-4" aria-hidden />
             <span className="sr-only">Edit</span>
           </Button>
           {!row.original.isSystem && (
@@ -137,7 +137,7 @@ export function RoleList() {
                   size="sm"
                   className="text-destructive hover:text-destructive"
                 >
-                  <Trash2 className="h-4 w-4" />
+                  <Trash2 className="h-4 w-4" aria-hidden />
                   <span className="sr-only">Delete</span>
                 </Button>
               }
@@ -162,7 +162,7 @@ export function RoleList() {
         description="Manage roles and their permissions"
         actions={
           <Button onClick={handleNewRole}>
-            <Plus className="mr-1.5 h-4 w-4" />
+            <Plus className="mr-1.5 h-4 w-4" aria-hidden />
             New Role
           </Button>
         }
@@ -209,7 +209,7 @@ export function RoleList() {
                   onClick={() => handleEditRole(original)}
                   aria-label="Edit role"
                 >
-                  <Pencil className="h-4 w-4" />
+                  <Pencil className="h-4 w-4" aria-hidden />
                 </Button>
                 {!original.isSystem ? (
                   <ConfirmationDialog
@@ -220,7 +220,7 @@ export function RoleList() {
                         className="text-destructive hover:text-destructive"
                         aria-label="Delete role"
                       >
-                        <Trash2 className="h-4 w-4" />
+                        <Trash2 className="h-4 w-4" aria-hidden />
                       </Button>
                     }
                     title="Delete Role"
diff --git a/src/components/admin/sales-email-config-card.tsx b/src/components/admin/sales-email-config-card.tsx
index fea0b7a8..0d33302b 100644
--- a/src/components/admin/sales-email-config-card.tsx
+++ b/src/components/admin/sales-email-config-card.tsx
@@ -174,7 +174,7 @@ export function SalesEmailConfigCard() {
     return (
       <Card>
         <CardContent className="flex items-center gap-2 py-6 text-sm text-muted-foreground">
-          <Loader2 className="h-4 w-4 animate-spin" /> Loading sales email config…
+          <Loader2 className="h-4 w-4 animate-spin" aria-hidden /> Loading sales email config…
         </CardContent>
       </Card>
     );
@@ -387,7 +387,7 @@ export function SalesEmailConfigCard() {
 
       <div className="flex justify-end">
         <Button onClick={handleSave} disabled={saving}>
-          {saving && <Loader2 className="mr-2 h-4 w-4 animate-spin" />}
+          {saving && <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />}
           Save sales email settings
         </Button>
       </div>
diff --git a/src/components/admin/settings/settings-manager.tsx b/src/components/admin/settings/settings-manager.tsx
index 58156eb2..528319fd 100644
--- a/src/components/admin/settings/settings-manager.tsx
+++ b/src/components/admin/settings/settings-manager.tsx
@@ -408,7 +408,7 @@ export function SettingsManager() {
                           saveSetting(setting.key, values[setting.key] ?? setting.defaultValue)
                         }
                       >
-                        <Save className="h-3.5 w-3.5" />
+                        <Save className="h-3.5 w-3.5" aria-hidden />
                       </Button>
                     </div>
                   </div>
@@ -454,7 +454,7 @@ export function SettingsManager() {
                       saveSetting(setting.key, values[setting.key] ?? setting.defaultValue)
                     }
                   >
-                    <Save className="h-3.5 w-3.5" />
+                    <Save className="h-3.5 w-3.5" aria-hidden />
                   </Button>
                 </div>
               </div>
@@ -542,7 +542,7 @@ export function SettingsManager() {
                       size="sm"
                       className="text-destructive hover:text-destructive"
                     >
-                      <Trash2 className="h-4 w-4" />
+                      <Trash2 className="h-4 w-4" aria-hidden />
                     </Button>
                   }
                   title="Delete Setting"
@@ -569,7 +569,7 @@ export function SettingsManager() {
                 className="flex-1"
               />
               <Button variant="outline" onClick={handleAddCustom} disabled={!customKey.trim()}>
-                <Plus className="mr-1 h-4 w-4" />
+                <Plus className="mr-1 h-4 w-4" aria-hidden />
                 Add
               </Button>
             </div>
diff --git a/src/components/admin/shared/settings-form-card.tsx b/src/components/admin/shared/settings-form-card.tsx
index 022225d2..a7bbbda0 100644
--- a/src/components/admin/shared/settings-form-card.tsx
+++ b/src/components/admin/shared/settings-form-card.tsx
@@ -147,7 +147,7 @@ export function SettingsFormCard({ title, description, fields, extra }: Settings
         </CardHeader>
         <CardContent>
           <div className="flex items-center text-sm text-muted-foreground">
-            <Loader2 className="mr-2 h-4 w-4 animate-spin" />
+            <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />
             Loading…
           </div>
         </CardContent>
@@ -176,7 +176,7 @@ export function SettingsFormCard({ title, description, fields, extra }: Settings
         <div className="flex items-center justify-end gap-2 pt-2 border-t">
           {extra}
           <Button onClick={handleSave} disabled={saving || !dirty}>
-            {saving && <Loader2 className="mr-2 h-4 w-4 animate-spin" />}
+            {saving && <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />}
             Save changes
           </Button>
         </div>
diff --git a/src/components/admin/storage-admin-panel.tsx b/src/components/admin/storage-admin-panel.tsx
index 315c5a25..49918082 100644
--- a/src/components/admin/storage-admin-panel.tsx
+++ b/src/components/admin/storage-admin-panel.tsx
@@ -174,7 +174,7 @@ export function StorageAdminPanel() {
   if (status.isLoading) {
     return (
       <div className="flex items-center gap-2 text-sm text-muted-foreground">
-        <Loader2 className="h-4 w-4 animate-spin" /> Loading storage status…
+        <Loader2 className="h-4 w-4 animate-spin" aria-hidden /> Loading storage status…
       </div>
     );
   }
@@ -216,18 +216,22 @@ export function StorageAdminPanel() {
               onClick={() => testMutation.mutate()}
               disabled={testMutation.isPending}
             >
-              {testMutation.isPending && <Loader2 className="mr-2 h-4 w-4 animate-spin" />}
+              {testMutation.isPending && (
+                <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />
+              )}
               Test S3 connection
             </Button>
             {testResult && (
               <div className="rounded-md border p-3 text-sm">
                 {testResult.ok ? (
                   <div className="flex items-center gap-2 text-emerald-600">
-                    <CheckCircle2 className="h-4 w-4" /> Connection OK — round-trip succeeded.
+                    <CheckCircle2 className="h-4 w-4" aria-hidden /> Connection OK — round-trip
+                    succeeded.
                   </div>
                 ) : (
                   <div className="flex items-center gap-2 text-destructive">
-                    <XCircle className="h-4 w-4" /> {testResult.error ?? 'Connection failed'}
+                    <XCircle className="h-4 w-4" aria-hidden />{' '}
+                    {testResult.error ?? 'Connection failed'}
                   </div>
                 )}
               </div>
@@ -248,9 +252,9 @@ export function StorageAdminPanel() {
         <Card className="lg:col-span-2">
           <CardHeader className="flex flex-row items-start gap-3 space-y-0 pb-2">
             {s.backend === 's3' ? (
-              <ServerCog className="mt-0.5 h-5 w-5 text-muted-foreground" />
+              <ServerCog className="mt-0.5 h-5 w-5 text-muted-foreground" aria-hidden />
             ) : (
-              <HardDrive className="mt-0.5 h-5 w-5 text-muted-foreground" />
+              <HardDrive className="mt-0.5 h-5 w-5 text-muted-foreground" aria-hidden />
             )}
             <div>
               <CardTitle className="text-base">Active backend: {s.backend}</CardTitle>
@@ -283,7 +287,9 @@ export function StorageAdminPanel() {
                   disabled={dryRunMutation.isPending}
                   onClick={() => openConfirm('switch-and-migrate')}
                 >
-                  {dryRunMutation.isPending && <Loader2 className="mr-2 h-4 w-4 animate-spin" />}
+                  {dryRunMutation.isPending && (
+                    <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />
+                  )}
                   Switch + migrate existing files to {otherBackend}
                 </Button>
                 <Button variant="outline" onClick={() => openConfirm('switch-only')}>
@@ -294,7 +300,7 @@ export function StorageAdminPanel() {
                   onClick={() => status.refetch()}
                   disabled={status.isFetching}
                 >
-                  <RefreshCw className="mr-2 h-4 w-4" /> Refresh
+                  <RefreshCw className="mr-2 h-4 w-4" aria-hidden /> Refresh
                 </Button>
               </div>
               <p className="text-xs text-muted-foreground">
@@ -337,7 +343,7 @@ export function StorageAdminPanel() {
         <DialogContent>
           <DialogHeader>
             <DialogTitle className="flex items-center gap-2">
-              <AlertTriangle className="h-5 w-5 text-amber-600" />
+              <AlertTriangle className="h-5 w-5 text-amber-600" aria-hidden />
               {confirmMode === 'switch-and-migrate'
                 ? `Switch + migrate to ${otherBackend}?`
                 : `Switch active backend to ${otherBackend}?`}
@@ -382,7 +388,9 @@ export function StorageAdminPanel() {
                 })
               }
             >
-              {migrateMutation.isPending && <Loader2 className="mr-2 h-4 w-4 animate-spin" />}
+              {migrateMutation.isPending && (
+                <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />
+              )}
               {confirmMode === 'switch-and-migrate' ? 'Migrate now' : 'Switch now'}
             </Button>
           </DialogFooter>
diff --git a/src/components/admin/system-monitoring-dashboard.tsx b/src/components/admin/system-monitoring-dashboard.tsx
index 3748f3b2..fdb1d4a8 100644
--- a/src/components/admin/system-monitoring-dashboard.tsx
+++ b/src/components/admin/system-monitoring-dashboard.tsx
@@ -60,9 +60,9 @@ export function SystemMonitoringDashboard() {
           <>
             <div className="flex items-center gap-2 mb-3">
               {health.overall === 'healthy' ? (
-                <CheckCircle2 className="h-4 w-4 text-green-500" />
+                <CheckCircle2 className="h-4 w-4 text-green-500" aria-hidden />
               ) : (
-                <AlertTriangle className="h-4 w-4 text-yellow-500" />
+                <AlertTriangle className="h-4 w-4 text-yellow-500" aria-hidden />
               )}
               <span className="text-sm text-muted-foreground">
                 All services checked at {new Date(health.checkedAt).toLocaleTimeString()}
@@ -88,7 +88,7 @@ export function SystemMonitoringDashboard() {
         <Card>
           <CardHeader className="pb-2">
             <CardTitle className="text-xs text-muted-foreground font-medium uppercase tracking-wide flex items-center gap-1.5">
-              <Wifi className="h-3.5 w-3.5" />
+              <Wifi className="h-3.5 w-3.5" aria-hidden />
               Active Connections
             </CardTitle>
           </CardHeader>
@@ -100,7 +100,7 @@ export function SystemMonitoringDashboard() {
         <Card>
           <CardHeader className="pb-2">
             <CardTitle className="text-xs text-muted-foreground font-medium uppercase tracking-wide flex items-center gap-1.5">
-              <AlertTriangle className="h-3.5 w-3.5" />
+              <AlertTriangle className="h-3.5 w-3.5" aria-hidden />
               Total Failed Jobs
             </CardTitle>
           </CardHeader>
@@ -114,7 +114,7 @@ export function SystemMonitoringDashboard() {
         <Card>
           <CardHeader className="pb-2">
             <CardTitle className="text-xs text-muted-foreground font-medium uppercase tracking-wide flex items-center gap-1.5">
-              <Activity className="h-3.5 w-3.5" />
+              <Activity className="h-3.5 w-3.5" aria-hidden />
               Active Jobs
             </CardTitle>
           </CardHeader>
diff --git a/src/components/admin/tags/tag-list.tsx b/src/components/admin/tags/tag-list.tsx
index c580ade5..a65bbdaa 100644
--- a/src/components/admin/tags/tag-list.tsx
+++ b/src/components/admin/tags/tag-list.tsx
@@ -87,13 +87,13 @@ export function TagList() {
       cell: ({ row }) => (
         <div className="flex items-center justify-end gap-1">
           <Button variant="ghost" size="sm" onClick={() => handleEditTag(row.original)}>
-            <Pencil className="h-4 w-4" />
+            <Pencil className="h-4 w-4" aria-hidden />
             <span className="sr-only">Edit</span>
           </Button>
           <ConfirmationDialog
             trigger={
               <Button variant="ghost" size="sm" className="text-destructive hover:text-destructive">
-                <Trash2 className="h-4 w-4" />
+                <Trash2 className="h-4 w-4" aria-hidden />
                 <span className="sr-only">Delete</span>
               </Button>
             }
@@ -117,7 +117,7 @@ export function TagList() {
         description="Manage tags used across clients and records"
         actions={
           <Button onClick={handleNewTag}>
-            <Plus className="mr-1.5 h-4 w-4" />
+            <Plus className="mr-1.5 h-4 w-4" aria-hidden />
             New Tag
           </Button>
         }
@@ -152,7 +152,7 @@ export function TagList() {
                 onClick={() => handleEditTag(original)}
                 aria-label={`Edit ${original.name}`}
               >
-                <Pencil className="h-4 w-4" />
+                <Pencil className="h-4 w-4" aria-hidden />
               </Button>
               <ConfirmationDialog
                 trigger={
@@ -162,7 +162,7 @@ export function TagList() {
                     className="text-destructive hover:text-destructive"
                     aria-label={`Delete ${original.name}`}
                   >
-                    <Trash2 className="h-4 w-4" />
+                    <Trash2 className="h-4 w-4" aria-hidden />
                   </Button>
                 }
                 title="Delete Tag"
diff --git a/src/components/admin/users/user-card.tsx b/src/components/admin/users/user-card.tsx
index 4f78b875..4f36035c 100644
--- a/src/components/admin/users/user-card.tsx
+++ b/src/components/admin/users/user-card.tsx
@@ -73,7 +73,7 @@ export function UserCard({
               onClick={(e) => e.stopPropagation()}
               aria-label={`Actions for ${user.displayName}`}
             >
-              <MoreHorizontal className="h-4 w-4" />
+              <MoreHorizontal className="h-4 w-4" aria-hidden />
             </Button>
           </DropdownMenuTrigger>
           <DropdownMenuContent align="end">
@@ -83,7 +83,7 @@ export function UserCard({
                 onEdit(user);
               }}
             >
-              <Pencil className="mr-2 h-3.5 w-3.5" />
+              <Pencil className="mr-2 h-3.5 w-3.5" aria-hidden />
               Edit
             </DropdownMenuItem>
             <ConfirmationDialog
@@ -91,12 +91,12 @@ export function UserCard({
                 <DropdownMenuItem onSelect={(e) => e.preventDefault()} disabled={isToggling}>
                   {user.isActive ? (
                     <>
-                      <PowerOff className="mr-2 h-3.5 w-3.5" />
+                      <PowerOff className="mr-2 h-3.5 w-3.5" aria-hidden />
                       Disable sign-in
                     </>
                   ) : (
                     <>
-                      <Power className="mr-2 h-3.5 w-3.5 text-emerald-600" />
+                      <Power className="mr-2 h-3.5 w-3.5 text-emerald-600" aria-hidden />
                       Enable sign-in
                     </>
                   )}
@@ -115,7 +115,7 @@ export function UserCard({
             <ConfirmationDialog
               trigger={
                 <DropdownMenuItem className="text-destructive" onSelect={(e) => e.preventDefault()}>
-                  <Trash2 className="mr-2 h-3.5 w-3.5" />
+                  <Trash2 className="mr-2 h-3.5 w-3.5" aria-hidden />
                   Remove
                 </DropdownMenuItem>
               }
@@ -155,16 +155,18 @@ export function UserCard({
 
           {/* Role + last login meta */}
           <div className="mt-1 flex flex-wrap items-center gap-x-2 gap-y-0.5 text-xs text-muted-foreground">
-            <ListCardMeta icon={<Shield className="h-3 w-3" />}>
+            <ListCardMeta icon={<Shield className="h-3 w-3" aria-hidden />}>
               {formatRole(user.role.name)}
             </ListCardMeta>
 
             {user.lastLoginAt ? (
-              <ListCardMeta icon={<Clock className="h-3 w-3" />}>
+              <ListCardMeta icon={<Clock className="h-3 w-3" aria-hidden />}>
                 {formatDistanceToNow(new Date(user.lastLoginAt), { addSuffix: true })}
               </ListCardMeta>
             ) : (
-              <ListCardMeta icon={<Clock className="h-3 w-3" />}>Never logged in</ListCardMeta>
+              <ListCardMeta icon={<Clock className="h-3 w-3" aria-hidden />}>
+                Never logged in
+              </ListCardMeta>
             )}
           </div>
 
diff --git a/src/components/admin/users/user-list.tsx b/src/components/admin/users/user-list.tsx
index 19bf19c3..8585df70 100644
--- a/src/components/admin/users/user-list.tsx
+++ b/src/components/admin/users/user-list.tsx
@@ -130,7 +130,7 @@ export function UserList() {
               onClick={() => handleEditUser(row.original)}
               title="Edit user"
             >
-              <Pencil className="h-4 w-4" />
+              <Pencil className="h-4 w-4" aria-hidden />
               <span className="sr-only">Edit</span>
             </Button>
           </PermissionGate>
@@ -149,9 +149,9 @@ export function UserList() {
                   }
                 >
                   {row.original.isActive ? (
-                    <PowerOff className="h-4 w-4" />
+                    <PowerOff className="h-4 w-4" aria-hidden />
                   ) : (
-                    <Power className="h-4 w-4" />
+                    <Power className="h-4 w-4" aria-hidden />
                   )}
                   <span className="sr-only">{row.original.isActive ? 'Disable' : 'Enable'}</span>
                 </Button>
@@ -176,7 +176,7 @@ export function UserList() {
                   title="Remove from port"
                   className="text-destructive hover:text-destructive"
                 >
-                  <Trash2 className="h-4 w-4" />
+                  <Trash2 className="h-4 w-4" aria-hidden />
                   <span className="sr-only">Remove</span>
                 </Button>
               }
@@ -201,7 +201,7 @@ export function UserList() {
         description="Manage users and their roles for this port"
         actions={
           <Button onClick={handleNewUser}>
-            <Plus className="mr-1.5 h-4 w-4" />
+            <Plus className="mr-1.5 h-4 w-4" aria-hidden />
             New User
           </Button>
         }
diff --git a/src/components/admin/vocabularies/vocabularies-manager.tsx b/src/components/admin/vocabularies/vocabularies-manager.tsx
index 7d1b4a78..3f277f39 100644
--- a/src/components/admin/vocabularies/vocabularies-manager.tsx
+++ b/src/components/admin/vocabularies/vocabularies-manager.tsx
@@ -166,7 +166,7 @@ export function VocabulariesManager() {
       <div>
         <PageHeader title="Vocabularies" description="Per-port pick lists used across the CRM." />
         <div className="flex items-center justify-center py-12 text-muted-foreground">
-          <Loader2 className="mr-2 h-4 w-4 animate-spin" /> Loading…
+          <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden /> Loading…
         </div>
       </div>
     );
@@ -208,7 +208,7 @@ export function VocabulariesManager() {
                               disabled={saving === def.key}
                               title="Restore the shipped defaults"
                             >
-                              <RotateCcw className="mr-1 h-3.5 w-3.5" />
+                              <RotateCcw className="mr-1 h-3.5 w-3.5" aria-hidden />
                               Reset
                             </Button>
                             <Button
@@ -217,9 +217,9 @@ export function VocabulariesManager() {
                               disabled={!v.dirty || saving === def.key}
                             >
                               {saving === def.key ? (
-                                <Loader2 className="mr-1 h-3.5 w-3.5 animate-spin" />
+                                <Loader2 className="mr-1 h-3.5 w-3.5 animate-spin" aria-hidden />
                               ) : (
-                                <Save className="mr-1 h-3.5 w-3.5" />
+                                <Save className="mr-1 h-3.5 w-3.5" aria-hidden />
                               )}
                               Save
                             </Button>
@@ -244,7 +244,7 @@ export function VocabulariesManager() {
                                     onClick={() => moveEntry(def.key, index, -1)}
                                     disabled={index === 0}
                                   >
-                                    <GripVertical className="h-3.5 w-3.5 rotate-90" />
+                                    <GripVertical className="h-3.5 w-3.5 rotate-90" aria-hidden />
                                   </button>
                                 </div>
                                 <Input
@@ -259,7 +259,7 @@ export function VocabulariesManager() {
                                   onClick={() => removeEntry(def.key, index)}
                                   aria-label={`Remove ${entry}`}
                                 >
-                                  <Trash2 className="h-3.5 w-3.5" />
+                                  <Trash2 className="h-3.5 w-3.5" aria-hidden />
                                 </Button>
                               </li>
                             ))}
@@ -285,7 +285,7 @@ export function VocabulariesManager() {
                             onClick={() => addEntry(def.key)}
                             disabled={!v.newEntry.trim()}
                           >
-                            <Plus className="mr-1 h-3.5 w-3.5" /> Add
+                            <Plus className="mr-1 h-3.5 w-3.5" aria-hidden /> Add
                           </Button>
                         </div>
                       </CardContent>
diff --git a/src/components/admin/website-analytics/umami-test-button.tsx b/src/components/admin/website-analytics/umami-test-button.tsx
index 7638bb12..9e3d9445 100644
--- a/src/components/admin/website-analytics/umami-test-button.tsx
+++ b/src/components/admin/website-analytics/umami-test-button.tsx
@@ -50,17 +50,17 @@ export function UmamiTestButton() {
       {result &&
         (result.ok ? (
           <span className="flex items-center text-xs text-green-600">
-            <CheckCircle2 className="mr-1 h-3.5 w-3.5" />
+            <CheckCircle2 className="mr-1 h-3.5 w-3.5" aria-hidden />
             Connected ({result.visitors ?? 0} active)
           </span>
         ) : (
           <span className="flex items-center text-xs text-destructive">
-            <XCircle className="mr-1 h-3.5 w-3.5" />
+            <XCircle className="mr-1 h-3.5 w-3.5" aria-hidden />
             {result.error ?? 'Failed'}
           </span>
         ))}
       <Button type="button" size="sm" variant="outline" onClick={runTest} disabled={pending}>
-        {pending ? <Loader2 className="mr-2 h-3.5 w-3.5 animate-spin" /> : null}
+        {pending ? <Loader2 className="mr-2 h-3.5 w-3.5 animate-spin" aria-hidden /> : null}
         Test connection
       </Button>
     </div>
diff --git a/src/components/berths/berth-card.tsx b/src/components/berths/berth-card.tsx
index e8fba621..7052bc8b 100644
--- a/src/components/berths/berth-card.tsx
+++ b/src/components/berths/berth-card.tsx
@@ -107,7 +107,7 @@ export function BerthCard({ berth }: BerthCardProps) {
               onClick={(e) => e.stopPropagation()}
               aria-label={`Actions for berth ${berth.mooringNumber}`}
             >
-              <MoreHorizontal className="h-4 w-4" />
+              <MoreHorizontal className="h-4 w-4" aria-hidden />
             </Button>
           </DropdownMenuTrigger>
           <DropdownMenuContent align="end">
@@ -117,7 +117,7 @@ export function BerthCard({ berth }: BerthCardProps) {
                 router.push(`/${portSlug}/berths/${berth.id}`);
               }}
             >
-              <Activity className="mr-2 h-3.5 w-3.5" />
+              <Activity className="mr-2 h-3.5 w-3.5" aria-hidden />
               View details
             </DropdownMenuItem>
             <DropdownMenuItem
@@ -126,7 +126,7 @@ export function BerthCard({ berth }: BerthCardProps) {
                 router.push(`/${portSlug}/berths/${berth.id}?edit=true`);
               }}
             >
-              <Pencil className="mr-2 h-3.5 w-3.5" />
+              <Pencil className="mr-2 h-3.5 w-3.5" aria-hidden />
               Edit
             </DropdownMenuItem>
           </DropdownMenuContent>
diff --git a/src/components/berths/berth-columns.tsx b/src/components/berths/berth-columns.tsx
index 55d4ce35..992464fe 100644
--- a/src/components/berths/berth-columns.tsx
+++ b/src/components/berths/berth-columns.tsx
@@ -147,7 +147,7 @@ function ActionsCell({ row }: { row: { original: BerthRow } }) {
           className="h-8 w-8"
           onClick={(e) => e.stopPropagation()}
         >
-          <MoreHorizontal className="h-4 w-4" />
+          <MoreHorizontal className="h-4 w-4" aria-hidden />
           <span className="sr-only">Open menu</span>
         </Button>
       </DropdownMenuTrigger>
@@ -158,7 +158,7 @@ function ActionsCell({ row }: { row: { original: BerthRow } }) {
             router.push(`/${params.portSlug}/berths/${berth.id}`);
           }}
         >
-          <Activity className="mr-2 h-4 w-4" />
+          <Activity className="mr-2 h-4 w-4" aria-hidden />
           View details
         </DropdownMenuItem>
         <DropdownMenuItem
@@ -167,7 +167,7 @@ function ActionsCell({ row }: { row: { original: BerthRow } }) {
             router.push(`/${params.portSlug}/berths/${berth.id}?edit=true`);
           }}
         >
-          <Pencil className="mr-2 h-4 w-4" />
+          <Pencil className="mr-2 h-4 w-4" aria-hidden />
           Edit
         </DropdownMenuItem>
       </DropdownMenuContent>
diff --git a/src/components/berths/berth-deal-documents-tab.tsx b/src/components/berths/berth-deal-documents-tab.tsx
index adc94f4d..67b4ef0b 100644
--- a/src/components/berths/berth-deal-documents-tab.tsx
+++ b/src/components/berths/berth-deal-documents-tab.tsx
@@ -65,7 +65,7 @@ export function BerthDealDocumentsTab({ berthId }: { berthId: string }) {
                   className="flex flex-wrap items-center justify-between gap-2 py-2.5 text-sm"
                 >
                   <div className="flex min-w-0 items-center gap-2">
-                    <FileText className="h-4 w-4 shrink-0 text-muted-foreground" />
+                    <FileText className="h-4 w-4 shrink-0 text-muted-foreground" aria-hidden />
                     <span className="truncate font-medium">{doc.title}</span>
                     <span className="text-xs text-muted-foreground">{doc.documentType}</span>
                   </div>
@@ -76,7 +76,7 @@ export function BerthDealDocumentsTab({ berthId }: { berthId: string }) {
                       href={`/${portSlug}/interests/${doc.interestId}` as any}
                       className="inline-flex items-center gap-1 text-xs text-primary hover:underline"
                     >
-                      Open <ExternalLink className="h-3 w-3" />
+                      Open <ExternalLink className="h-3 w-3" aria-hidden />
                     </Link>
                   </div>
                 </li>
diff --git a/src/components/berths/berth-detail-header.tsx b/src/components/berths/berth-detail-header.tsx
index 6b4697e5..4b56feb3 100644
--- a/src/components/berths/berth-detail-header.tsx
+++ b/src/components/berths/berth-detail-header.tsx
@@ -290,11 +290,11 @@ export function BerthDetailHeader({ berth }: BerthDetailHeaderProps) {
           <div className="flex flex-wrap items-center gap-2 sm:shrink-0">
             <PermissionGate resource="berths" action="edit">
               <Button variant="outline" size="sm" onClick={() => setStatusOpen(true)}>
-                <RefreshCw className="mr-1.5 h-4 w-4" />
+                <RefreshCw className="mr-1.5 h-4 w-4" aria-hidden />
                 Change Status
               </Button>
               <Button size="sm" onClick={() => setEditOpen(true)}>
-                <Pencil className="mr-1.5 h-4 w-4" />
+                <Pencil className="mr-1.5 h-4 w-4" aria-hidden />
                 Edit
               </Button>
             </PermissionGate>
@@ -369,7 +369,7 @@ function InterestLinkPicker({
           ) : (
             '— No interest —'
           )}
-          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" />
+          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" aria-hidden />
         </Button>
       </PopoverTrigger>
       <PopoverContent className="w-(--radix-popper-anchor-width) min-w-[320px] p-0" align="start">
@@ -420,7 +420,7 @@ function InterestLinkPicker({
                     {stageLabel(opt.pipelineStage)}
                   </span>
                   {value === opt.id ? (
-                    <Check className="h-3.5 w-3.5 text-muted-foreground" />
+                    <Check className="h-3.5 w-3.5 text-muted-foreground" aria-hidden />
                   ) : null}
                 </CommandItem>
               ))}
diff --git a/src/components/berths/berth-interest-pulse.tsx b/src/components/berths/berth-interest-pulse.tsx
index 97df770f..d8c8bb39 100644
--- a/src/components/berths/berth-interest-pulse.tsx
+++ b/src/components/berths/berth-interest-pulse.tsx
@@ -84,7 +84,7 @@ export function BerthInterestPulse({ berthId }: { berthId: string }) {
       <Card>
         <CardHeader className="pb-3">
           <CardTitle className="flex items-center gap-1.5 text-sm font-medium">
-            <Users className="size-3.5" />
+            <Users className="size-3.5" aria-hidden />
             Interested parties
           </CardTitle>
         </CardHeader>
@@ -99,7 +99,7 @@ export function BerthInterestPulse({ berthId }: { berthId: string }) {
     <Card>
       <CardHeader className="flex flex-row items-center justify-between pb-3 space-y-0">
         <CardTitle className="flex items-center gap-1.5 text-sm font-medium">
-          <Users className="size-3.5" />
+          <Users className="size-3.5" aria-hidden />
           Interested parties
           <span className="ml-1 rounded-full bg-muted px-1.5 py-0.5 text-[10px] font-medium text-muted-foreground">
             {active.length}
@@ -161,7 +161,10 @@ export function BerthInterestPulse({ berthId }: { berthId: string }) {
                       </p>
                     ) : null}
                   </div>
-                  <ChevronRight className="size-4 shrink-0 text-muted-foreground/60 transition-transform group-hover:translate-x-0.5" />
+                  <ChevronRight
+                    className="size-4 shrink-0 text-muted-foreground/60 transition-transform group-hover:translate-x-0.5"
+                    aria-hidden
+                  />
                 </Link>
               </li>
             );
diff --git a/src/components/berths/berth-reservations-tab.tsx b/src/components/berths/berth-reservations-tab.tsx
index a5c4ae84..91c9a63f 100644
--- a/src/components/berths/berth-reservations-tab.tsx
+++ b/src/components/berths/berth-reservations-tab.tsx
@@ -48,7 +48,7 @@ export function BerthReservationsTab({ berthId }: BerthReservationsTabProps) {
         <h3 className="text-lg font-semibold">Reservations</h3>
         <PermissionGate resource="reservations" action="create">
           <Button size="sm" onClick={() => setReserveOpen(true)}>
-            <Plus className="mr-1.5 h-4 w-4" />
+            <Plus className="mr-1.5 h-4 w-4" aria-hidden />
             Reserve this berth
           </Button>
         </PermissionGate>
diff --git a/src/components/berths/berth-status-suggestion-dialog.tsx b/src/components/berths/berth-status-suggestion-dialog.tsx
index 20562206..f4727d46 100644
--- a/src/components/berths/berth-status-suggestion-dialog.tsx
+++ b/src/components/berths/berth-status-suggestion-dialog.tsx
@@ -60,7 +60,7 @@ export function BerthStatusSuggestionDialog({
           <Badge variant="outline" className="text-base px-4 py-1.5">
             {currentStatus.replace(/_/g, ' ')}
           </Badge>
-          <ArrowRight className="h-5 w-5 text-muted-foreground" />
+          <ArrowRight className="h-5 w-5 text-muted-foreground" aria-hidden />
           <Badge variant="default" className="text-base px-4 py-1.5">
             {suggestedStatus.replace(/_/g, ' ')}
           </Badge>
@@ -73,7 +73,9 @@ export function BerthStatusSuggestionDialog({
             Dismiss
           </Button>
           <Button onClick={() => applyMutation.mutate()} disabled={applyMutation.isPending}>
-            {applyMutation.isPending && <Loader2 className="mr-1.5 h-4 w-4 animate-spin" />}
+            {applyMutation.isPending && (
+              <Loader2 className="mr-1.5 h-4 w-4 animate-spin" aria-hidden />
+            )}
             Apply Change
           </Button>
         </DialogFooter>
diff --git a/src/components/berths/waiting-list-manager.tsx b/src/components/berths/waiting-list-manager.tsx
index 0fe7e7de..5b07367c 100644
--- a/src/components/berths/waiting-list-manager.tsx
+++ b/src/components/berths/waiting-list-manager.tsx
@@ -68,7 +68,7 @@ function SortableEntry({
         {...listeners}
         className="cursor-grab active:cursor-grabbing text-muted-foreground"
       >
-        <GripVertical className="h-4 w-4" />
+        <GripVertical className="h-4 w-4" aria-hidden />
       </button>
 
       <span className="text-sm font-mono w-6 text-center text-muted-foreground">
@@ -88,7 +88,7 @@ function SortableEntry({
         onClick={() => onRemove(entry.id)}
         className="text-muted-foreground hover:text-destructive transition-colors"
       >
-        <Trash2 className="h-4 w-4" />
+        <Trash2 className="h-4 w-4" aria-hidden />
       </button>
     </div>
   );
@@ -189,7 +189,7 @@ export function WaitingListManager({ berthId }: WaitingListManagerProps) {
       <div className="flex items-center justify-between">
         <span className="text-sm font-medium">Waiting List ({entries.length})</span>
         <Button size="sm" variant="outline" onClick={() => setShowAddForm((v) => !v)}>
-          <Plus className="mr-1.5 h-4 w-4" />
+          <Plus className="mr-1.5 h-4 w-4" aria-hidden />
           Add
         </Button>
       </div>
@@ -217,7 +217,9 @@ export function WaitingListManager({ berthId }: WaitingListManagerProps) {
           />
           <div className="flex gap-2">
             <Button size="sm" onClick={handleAdd} disabled={addMutation.isPending}>
-              {addMutation.isPending && <Loader2 className="mr-1.5 h-4 w-4 animate-spin" />}
+              {addMutation.isPending && (
+                <Loader2 className="mr-1.5 h-4 w-4 animate-spin" aria-hidden />
+              )}
               Add to List
             </Button>
             <Button size="sm" variant="ghost" onClick={() => setShowAddForm(false)}>
diff --git a/src/components/clients/bulk-archive-wizard.tsx b/src/components/clients/bulk-archive-wizard.tsx
index a72d2b3d..769df525 100644
--- a/src/components/clients/bulk-archive-wizard.tsx
+++ b/src/components/clients/bulk-archive-wizard.tsx
@@ -126,7 +126,7 @@ function BulkArchiveWizardBody({ open, onOpenChange, clientIds, onSuccess }: Pro
 
         {preflight.isLoading ? (
           <div className="py-8 text-center text-sm text-muted-foreground">
-            <Loader2 className="h-5 w-5 animate-spin mx-auto mb-2" />
+            <Loader2 className="h-5 w-5 animate-spin mx-auto mb-2" aria-hidden />
             Checking each client…
           </div>
         ) : preflight.error ? (
@@ -156,7 +156,7 @@ function BulkArchiveWizardBody({ open, onOpenChange, clientIds, onSuccess }: Pro
                 {blocked.length > 0 && (
                   <div className="rounded-md border border-red-300 bg-red-50 p-3 text-xs text-red-900 space-y-1">
                     <div className="font-medium flex items-center gap-1.5">
-                      <AlertTriangle className="h-4 w-4" /> Blocked
+                      <AlertTriangle className="h-4 w-4" aria-hidden /> Blocked
                     </div>
                     {blocked.slice(0, 5).map((b) => (
                       <div key={b.clientId}>
@@ -236,7 +236,7 @@ function BulkArchiveWizardBody({ open, onOpenChange, clientIds, onSuccess }: Pro
             {stage === 'confirm' && (
               <div className="space-y-2 text-sm">
                 <div className="rounded-md border border-emerald-300 bg-emerald-50 p-3 text-emerald-900 flex items-start gap-2">
-                  <CheckCircle2 className="h-4 w-4 mt-0.5" />
+                  <CheckCircle2 className="h-4 w-4 mt-0.5" aria-hidden />
                   <div>
                     Ready to archive <strong>{archivable.length}</strong> client
                     {archivable.length === 1 ? '' : 's'}
@@ -268,7 +268,7 @@ function BulkArchiveWizardBody({ open, onOpenChange, clientIds, onSuccess }: Pro
                 }
               }}
             >
-              Continue <ArrowRight className="h-4 w-4 ml-1" />
+              Continue <ArrowRight className="h-4 w-4 ml-1" aria-hidden />
             </Button>
           )}
 
@@ -279,18 +279,18 @@ function BulkArchiveWizardBody({ open, onOpenChange, clientIds, onSuccess }: Pro
                 disabled={carouselIndex === 0}
                 onClick={() => setCarouselIndex((i) => Math.max(0, i - 1))}
               >
-                <ArrowLeft className="h-4 w-4 mr-1" /> Back
+                <ArrowLeft className="h-4 w-4 mr-1" aria-hidden /> Back
               </Button>
               {carouselIndex < highStakes.length - 1 ? (
                 <Button
                   disabled={(reasons[currentHighStakes?.clientId ?? '']?.trim().length ?? 0) < 5}
                   onClick={() => setCarouselIndex((i) => i + 1)}
                 >
-                  Next <ArrowRight className="h-4 w-4 ml-1" />
+                  Next <ArrowRight className="h-4 w-4 ml-1" aria-hidden />
                 </Button>
               ) : (
                 <Button disabled={!allHighStakesReasoned} onClick={() => setStage('confirm')}>
-                  Review <ArrowRight className="h-4 w-4 ml-1" />
+                  Review <ArrowRight className="h-4 w-4 ml-1" aria-hidden />
                 </Button>
               )}
             </>
@@ -304,7 +304,7 @@ function BulkArchiveWizardBody({ open, onOpenChange, clientIds, onSuccess }: Pro
             >
               {archiveMutation.isPending ? (
                 <>
-                  <Loader2 className="h-4 w-4 animate-spin mr-1.5" /> Archiving…
+                  <Loader2 className="h-4 w-4 animate-spin mr-1.5" aria-hidden /> Archiving…
                 </>
               ) : (
                 `Archive ${archivable.length}`
diff --git a/src/components/clients/bulk-hard-delete-dialog.tsx b/src/components/clients/bulk-hard-delete-dialog.tsx
index 54e23f9a..342bf3eb 100644
--- a/src/components/clients/bulk-hard-delete-dialog.tsx
+++ b/src/components/clients/bulk-hard-delete-dialog.tsx
@@ -111,7 +111,7 @@ function BulkHardDeleteDialogBody({ onOpenChange, clientIds, onDeleted }: Props)
     <>
       <DialogHeader>
         <DialogTitle className="flex items-center gap-2 text-destructive">
-          <AlertTriangle className="h-5 w-5" />
+          <AlertTriangle className="h-5 w-5" aria-hidden />
           Permanently delete {clientIds.length} client{clientIds.length === 1 ? '' : 's'}
         </DialogTitle>
         <DialogDescription>
@@ -136,7 +136,7 @@ function BulkHardDeleteDialogBody({ onOpenChange, clientIds, onDeleted }: Props)
       {stage === 'confirm' && (
         <div className="space-y-3">
           <div className="flex items-start gap-2 rounded-md border border-blue-300 bg-blue-50 p-3 text-xs text-blue-900">
-            <Mail className="h-4 w-4 shrink-0 mt-0.5" />
+            <Mail className="h-4 w-4 shrink-0 mt-0.5" aria-hidden />
             <div>
               Code sent to <span className="font-mono">{maskedEmail}</span>. Enter both fields
               below.
@@ -214,7 +214,7 @@ function BulkHardDeleteDialogBody({ onOpenChange, clientIds, onDeleted }: Props)
           >
             {requestCode.isPending ? (
               <>
-                <Loader2 className="h-4 w-4 animate-spin mr-1.5" /> Sending…
+                <Loader2 className="h-4 w-4 animate-spin mr-1.5" aria-hidden /> Sending…
               </>
             ) : (
               'Send confirmation code'
@@ -229,7 +229,7 @@ function BulkHardDeleteDialogBody({ onOpenChange, clientIds, onDeleted }: Props)
           >
             {bulkDelete.isPending ? (
               <>
-                <Loader2 className="h-4 w-4 animate-spin mr-1.5" /> Deleting…
+                <Loader2 className="h-4 w-4 animate-spin mr-1.5" aria-hidden /> Deleting…
               </>
             ) : (
               `Permanently delete ${clientIds.length}`
diff --git a/src/components/clients/client-card.tsx b/src/components/clients/client-card.tsx
index 1b0cb3d8..736d868b 100644
--- a/src/components/clients/client-card.tsx
+++ b/src/components/clients/client-card.tsx
@@ -61,16 +61,16 @@ export function ClientCard({ client, portSlug, onEdit, onArchive }: ClientCardPr
               onClick={(e) => e.stopPropagation()}
               aria-label={`Actions for ${client.fullName}`}
             >
-              <MoreHorizontal className="h-4 w-4" />
+              <MoreHorizontal className="h-4 w-4" aria-hidden />
             </Button>
           </DropdownMenuTrigger>
           <DropdownMenuContent align="end">
             <DropdownMenuItem onClick={() => onEdit(client)}>
-              <Pencil className="mr-2 h-3.5 w-3.5" />
+              <Pencil className="mr-2 h-3.5 w-3.5" aria-hidden />
               Edit
             </DropdownMenuItem>
             <DropdownMenuItem className="text-destructive" onClick={() => onArchive(client)}>
-              <Archive className="mr-2 h-3.5 w-3.5" />
+              <Archive className="mr-2 h-3.5 w-3.5" aria-hidden />
               Archive
             </DropdownMenuItem>
           </DropdownMenuContent>
diff --git a/src/components/clients/client-columns.tsx b/src/components/clients/client-columns.tsx
index a73db953..675202ae 100644
--- a/src/components/clients/client-columns.tsx
+++ b/src/components/clients/client-columns.tsx
@@ -121,7 +121,7 @@ export function getClientColumns({
             className="inline-flex items-center gap-1.5 text-sm text-foreground hover:text-primary hover:underline"
             title={`Email ${value}`}
           >
-            <Mail className="h-3 w-3 shrink-0 text-muted-foreground" />
+            <Mail className="h-3 w-3 shrink-0 text-muted-foreground" aria-hidden />
             <span className="truncate">{value}</span>
           </a>
         );
@@ -147,7 +147,7 @@ export function getClientColumns({
               className="inline-flex items-center gap-1.5 text-foreground hover:text-primary hover:underline"
               title={`Call ${value}`}
             >
-              <Phone className="h-3 w-3 shrink-0 text-muted-foreground" />
+              <Phone className="h-3 w-3 shrink-0 text-muted-foreground" aria-hidden />
               <span>{value}</span>
             </a>
             {waDigits && (
@@ -160,7 +160,7 @@ export function getClientColumns({
                 title={`WhatsApp ${value}`}
                 aria-label={`WhatsApp ${value}`}
               >
-                <MessageCircle className="h-3.5 w-3.5" />
+                <MessageCircle className="h-3.5 w-3.5" aria-hidden />
               </a>
             )}
           </span>
@@ -303,16 +303,16 @@ export function getClientColumns({
               className="h-7 w-7"
               onClick={(e) => e.stopPropagation()}
             >
-              <MoreHorizontal className="h-4 w-4" />
+              <MoreHorizontal className="h-4 w-4" aria-hidden />
             </Button>
           </DropdownMenuTrigger>
           <DropdownMenuContent align="end">
             <DropdownMenuItem onClick={() => onEdit(row.original)}>
-              <Pencil className="mr-2 h-3.5 w-3.5" />
+              <Pencil className="mr-2 h-3.5 w-3.5" aria-hidden />
               Edit
             </DropdownMenuItem>
             <DropdownMenuItem className="text-destructive" onClick={() => onArchive(row.original)}>
-              <Archive className="mr-2 h-3.5 w-3.5" />
+              <Archive className="mr-2 h-3.5 w-3.5" aria-hidden />
               Archive
             </DropdownMenuItem>
           </DropdownMenuContent>
diff --git a/src/components/clients/client-detail-header.tsx b/src/components/clients/client-detail-header.tsx
index b910b770..8190bd18 100644
--- a/src/components/clients/client-detail-header.tsx
+++ b/src/components/clients/client-detail-header.tsx
@@ -167,7 +167,7 @@ export function ClientDetailHeader({ client }: ClientDetailHeaderProps) {
                   title="Permanently delete client"
                   className="shrink-0 rounded-md p-1.5 text-muted-foreground/70 transition-colors hover:bg-destructive/10 hover:text-destructive"
                 >
-                  <Trash2 className="size-4" />
+                  <Trash2 className="size-4" aria-hidden />
                 </button>
               </PermissionGate>
             )}
@@ -182,7 +182,11 @@ export function ClientDetailHeader({ client }: ClientDetailHeaderProps) {
                 isArchived ? 'hover:text-emerald-600' : 'hover:text-destructive',
               )}
             >
-              {isArchived ? <RotateCcw className="size-4" /> : <Archive className="size-4" />}
+              {isArchived ? (
+                <RotateCcw className="size-4" aria-hidden />
+              ) : (
+                <Archive className="size-4" aria-hidden />
+              )}
             </button>
           </div>
         </div>
diff --git a/src/components/clients/client-form.tsx b/src/components/clients/client-form.tsx
index 5e981d0a..c35b8a5b 100644
--- a/src/components/clients/client-form.tsx
+++ b/src/components/clients/client-form.tsx
@@ -224,7 +224,7 @@ export function ClientForm({ open, onOpenChange, client, onUseExistingClient }:
                 size="sm"
                 onClick={() => append({ channel: 'email', value: '', isPrimary: false })}
               >
-                <Plus className="mr-1 h-3.5 w-3.5" />
+                <Plus className="mr-1 h-3.5 w-3.5" aria-hidden />
                 Add Contact
               </Button>
             </div>
@@ -333,7 +333,7 @@ export function ClientForm({ open, onOpenChange, client, onUseExistingClient }:
                         className="h-8 text-destructive hover:text-destructive"
                         onClick={() => remove(index)}
                       >
-                        <Trash2 className="mr-1 h-3.5 w-3.5" />
+                        <Trash2 className="mr-1 h-3.5 w-3.5" aria-hidden />
                         Remove
                       </Button>
                     )}
@@ -419,7 +419,7 @@ export function ClientForm({ open, onOpenChange, client, onUseExistingClient }:
             </Button>
             <Button type="submit" disabled={isSubmitting || mutation.isPending}>
               {(isSubmitting || mutation.isPending) && (
-                <Loader2 className="mr-2 h-4 w-4 animate-spin" />
+                <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />
               )}
               {isEdit ? 'Save changes' : 'Create Client'}
             </Button>
diff --git a/src/components/clients/client-interests-tab.tsx b/src/components/clients/client-interests-tab.tsx
index f376f25e..b047d364 100644
--- a/src/components/clients/client-interests-tab.tsx
+++ b/src/components/clients/client-interests-tab.tsx
@@ -77,7 +77,10 @@ function InterestRowItem({
             <p className="mt-0.5 truncate text-xs text-muted-foreground">{yachtLabel}</p>
           ) : null}
         </div>
-        <ChevronRight className="size-4 shrink-0 text-muted-foreground transition-transform group-hover:translate-x-0.5" />
+        <ChevronRight
+          className="size-4 shrink-0 text-muted-foreground transition-transform group-hover:translate-x-0.5"
+          aria-hidden
+        />
       </div>
 
       <div className="mt-3">
@@ -364,9 +367,9 @@ function InterestPreviewSheet({
 function InterestSkeleton() {
   return (
     <div className="rounded-xl border border-border bg-card p-4 shadow-sm">
-      <Skeleton className="h-4 w-32" />
-      <Skeleton className="mt-2 h-3 w-24" />
-      <Skeleton className="mt-3 h-2 w-48" />
+      <Skeleton className="h-4 w-32" aria-hidden />
+      <Skeleton className="mt-2 h-3 w-24" aria-hidden />
+      <Skeleton className="mt-3 h-2 w-48" aria-hidden />
     </div>
   );
 }
@@ -421,7 +424,7 @@ export function ClientInterestsTab({ clientId }: ClientInterestsTabProps) {
     <div className="space-y-6">
       <div className="flex justify-end">
         <Button size="sm" onClick={() => setCreateOpen(true)}>
-          <Plus className="mr-1.5 size-3.5" />
+          <Plus className="mr-1.5 size-3.5" aria-hidden />
           Add interest
         </Button>
       </div>
diff --git a/src/components/clients/client-list.tsx b/src/components/clients/client-list.tsx
index 28b22e1a..f9c493c6 100644
--- a/src/components/clients/client-list.tsx
+++ b/src/components/clients/client-list.tsx
@@ -167,7 +167,7 @@ export function ClientList() {
             far-right edge, which is where reps look first. */}
         <PermissionGate resource="clients" action="create">
           <Button size="sm" className="ml-auto" onClick={() => setCreateOpen(true)}>
-            <Plus className="mr-1.5 h-4 w-4" />
+            <Plus className="mr-1.5 h-4 w-4" aria-hidden />
             New Client
           </Button>
         </PermissionGate>
diff --git a/src/components/clients/client-pipeline-summary.tsx b/src/components/clients/client-pipeline-summary.tsx
index 11018326..3df8a95e 100644
--- a/src/components/clients/client-pipeline-summary.tsx
+++ b/src/components/clients/client-pipeline-summary.tsx
@@ -124,8 +124,8 @@ function HeroVariant({ clientId, portSlug }: { clientId: string; portSlug: strin
   if (isLoading) {
     return (
       <div className="space-y-2">
-        <Skeleton className="h-4 w-40" />
-        <Skeleton className="h-2 w-48" />
+        <Skeleton className="h-4 w-40" aria-hidden />
+        <Skeleton className="h-2 w-48" aria-hidden />
       </div>
     );
   }
@@ -143,7 +143,7 @@ function HeroVariant({ clientId, portSlug }: { clientId: string; portSlug: strin
           href={`/${portSlug}/interests/new` as Route}
           className="inline-flex items-center gap-1 text-xs font-medium text-primary hover:underline"
         >
-          Start interest <ArrowRight className="size-3" />
+          Start interest <ArrowRight className="size-3" aria-hidden />
         </Link>
       </div>
     );
@@ -182,7 +182,10 @@ function HeroVariant({ clientId, portSlug }: { clientId: string; portSlug: strin
           >
             {STAGE_LABELS[stage]}
           </span>
-          <ChevronRight className="size-3.5 shrink-0 text-muted-foreground transition-transform group-hover:translate-x-0.5" />
+          <ChevronRight
+            className="size-3.5 shrink-0 text-muted-foreground transition-transform group-hover:translate-x-0.5"
+            aria-hidden
+          />
         </div>
         <div className="mt-1.5">
           <StageStepper current={stage} size="xs" />
@@ -214,8 +217,8 @@ function PanelVariant({ clientId, portSlug }: { clientId: string; portSlug: stri
   if (isLoading) {
     return (
       <div className="space-y-2">
-        <Skeleton className="h-4 w-40" />
-        <Skeleton className="h-2 w-48" />
+        <Skeleton className="h-4 w-40" aria-hidden />
+        <Skeleton className="h-2 w-48" aria-hidden />
       </div>
     );
   }
@@ -233,7 +236,7 @@ function PanelVariant({ clientId, portSlug }: { clientId: string; portSlug: stri
           href={`/${portSlug}/interests/new` as Route}
           className="inline-flex items-center gap-1 text-xs font-medium text-primary hover:underline"
         >
-          Start interest <ArrowRight className="size-3" />
+          Start interest <ArrowRight className="size-3" aria-hidden />
         </Link>
       </div>
     );
@@ -289,7 +292,10 @@ function PanelVariant({ clientId, portSlug }: { clientId: string; portSlug: stri
                     <StageStepper current={stage} size="xs" />
                   </div>
                 </div>
-                <ChevronRight className="size-3.5 shrink-0 text-muted-foreground transition-transform group-hover:translate-x-0.5" />
+                <ChevronRight
+                  className="size-3.5 shrink-0 text-muted-foreground transition-transform group-hover:translate-x-0.5"
+                  aria-hidden
+                />
               </Link>
             </li>
           );
diff --git a/src/components/clients/client-yachts-tab.tsx b/src/components/clients/client-yachts-tab.tsx
index a3ab63ee..60965eba 100644
--- a/src/components/clients/client-yachts-tab.tsx
+++ b/src/components/clients/client-yachts-tab.tsx
@@ -42,7 +42,7 @@ export function ClientYachtsTab({ clientId, yachts }: ClientYachtsTabProps) {
         <h3 className="text-sm font-medium">Client-owned yachts</h3>
         <PermissionGate resource="yachts" action="create">
           <Button size="sm" onClick={() => setCreateOpen(true)}>
-            <Plus className="mr-1.5 h-4 w-4" />
+            <Plus className="mr-1.5 h-4 w-4" aria-hidden />
             Add yacht
           </Button>
         </PermissionGate>
diff --git a/src/components/clients/contacts-editor.tsx b/src/components/clients/contacts-editor.tsx
index f39b7ebb..258ab949 100644
--- a/src/components/clients/contacts-editor.tsx
+++ b/src/components/clients/contacts-editor.tsx
@@ -141,7 +141,7 @@ export function ContactsEditor({ clientId, contacts }: { clientId: string; conta
           onClick={() => setAdding(true)}
           className="w-full justify-center"
         >
-          <Plus className="h-3.5 w-3.5 mr-1.5" />
+          <Plus className="h-3.5 w-3.5 mr-1.5" aria-hidden />
           Add contact
         </Button>
       )}
@@ -201,7 +201,7 @@ function ContactRow({
       {/* Top / left: channel + value */}
       <div className="flex min-w-0 flex-1 items-center gap-2">
         <ChannelPicker value={contact.channel} onChange={changeChannel}>
-          <Icon className="h-3.5 w-3.5 text-muted-foreground" />
+          <Icon className="h-3.5 w-3.5 text-muted-foreground" aria-hidden />
         </ChannelPicker>
         <div className="min-w-0 flex-1">
           {contact.channel === 'phone' || contact.channel === 'whatsapp' ? (
@@ -282,7 +282,7 @@ function ContactRow({
             title="Remove"
             className="rounded p-1 text-muted-foreground/50 transition-all hover:bg-background/60 hover:text-destructive sm:opacity-0 sm:group-hover:opacity-100"
           >
-            <Trash2 className="h-3.5 w-3.5" />
+            <Trash2 className="h-3.5 w-3.5" aria-hidden />
           </button>
         </div>
       ) : null}
@@ -439,7 +439,7 @@ function NewContactForm({
 
       <div className="ml-auto flex gap-2">
         <Button type="button" size="sm" onClick={submit} disabled={submitDisabled}>
-          {saving ? <Loader2 className="h-3.5 w-3.5 animate-spin" /> : 'Save'}
+          {saving ? <Loader2 className="h-3.5 w-3.5 animate-spin" aria-hidden /> : 'Save'}
         </Button>
         <Button type="button" size="sm" variant="ghost" onClick={onCancel} disabled={saving}>
           Cancel
diff --git a/src/components/clients/gdpr-export-button.tsx b/src/components/clients/gdpr-export-button.tsx
index 973ea8d1..57bb965e 100644
--- a/src/components/clients/gdpr-export-button.tsx
+++ b/src/components/clients/gdpr-export-button.tsx
@@ -111,7 +111,7 @@ export function GdprExportButton({ clientId }: { clientId: string }) {
     <Dialog open={open} onOpenChange={setOpen}>
       <DialogTrigger asChild>
         <Button variant="outline" size="sm" className="h-8">
-          <FileDown className="mr-1.5 h-3.5 w-3.5" />
+          <FileDown className="mr-1.5 h-3.5 w-3.5" aria-hidden />
           GDPR export
         </Button>
       </DialogTrigger>
@@ -154,9 +154,9 @@ export function GdprExportButton({ clientId }: { clientId: string }) {
 
           <Button onClick={() => request.mutate()} disabled={request.isPending}>
             {request.isPending ? (
-              <Loader2 className="mr-1.5 h-3.5 w-3.5 animate-spin" />
+              <Loader2 className="mr-1.5 h-3.5 w-3.5 animate-spin" aria-hidden />
             ) : (
-              <FileDown className="mr-1.5 h-3.5 w-3.5" />
+              <FileDown className="mr-1.5 h-3.5 w-3.5" aria-hidden />
             )}
             Queue export
           </Button>
@@ -180,7 +180,7 @@ export function GdprExportButton({ clientId }: { clientId: string }) {
                       </div>
                       {r.sentTo ? (
                         <div className="text-xs text-muted-foreground inline-flex items-center gap-1">
-                          <Mail className="h-3 w-3" />
+                          <Mail className="h-3 w-3" aria-hidden />
                           Sent to {r.sentTo}
                         </div>
                       ) : null}
@@ -195,7 +195,7 @@ export function GdprExportButton({ clientId }: { clientId: string }) {
                         size="sm"
                         onClick={() => downloadById(r.id)}
                       >
-                        <Download className="h-3.5 w-3.5" />
+                        <Download className="h-3.5 w-3.5" aria-hidden />
                       </Button>
                     ) : null}
                   </li>
diff --git a/src/components/clients/hard-delete-dialog.tsx b/src/components/clients/hard-delete-dialog.tsx
index 0f680d81..6db65f3c 100644
--- a/src/components/clients/hard-delete-dialog.tsx
+++ b/src/components/clients/hard-delete-dialog.tsx
@@ -94,7 +94,7 @@ function HardDeleteDialogBody({ onOpenChange, clientId, clientName, onDeleted }:
     <>
       <DialogHeader>
         <DialogTitle className="flex items-center gap-2 text-destructive">
-          <AlertTriangle className="h-5 w-5" />
+          <AlertTriangle className="h-5 w-5" aria-hidden />
           Permanently delete {clientName}
         </DialogTitle>
         <DialogDescription>
@@ -126,7 +126,7 @@ function HardDeleteDialogBody({ onOpenChange, clientId, clientName, onDeleted }:
       ) : (
         <div className="space-y-3">
           <div className="flex items-start gap-2 rounded-md border border-blue-300 bg-blue-50 p-3 text-xs text-blue-900">
-            <Mail className="h-4 w-4 shrink-0 mt-0.5" />
+            <Mail className="h-4 w-4 shrink-0 mt-0.5" aria-hidden />
             <div className="flex-1">
               <div>
                 Code sent to <span className="font-mono">{maskedEmail}</span>. It expires in 10
@@ -185,7 +185,7 @@ function HardDeleteDialogBody({ onOpenChange, clientId, clientName, onDeleted }:
           >
             {requestCode.isPending ? (
               <>
-                <Loader2 className="h-4 w-4 animate-spin mr-1.5" /> Sending…
+                <Loader2 className="h-4 w-4 animate-spin mr-1.5" aria-hidden /> Sending…
               </>
             ) : (
               'Send confirmation code'
@@ -199,7 +199,7 @@ function HardDeleteDialogBody({ onOpenChange, clientId, clientName, onDeleted }:
           >
             {hardDelete.isPending ? (
               <>
-                <Loader2 className="h-4 w-4 animate-spin mr-1.5" /> Deleting…
+                <Loader2 className="h-4 w-4 animate-spin mr-1.5" aria-hidden /> Deleting…
               </>
             ) : (
               'Permanently delete'
diff --git a/src/components/clients/portal-invite-button.tsx b/src/components/clients/portal-invite-button.tsx
index c24959e0..4d3efb0e 100644
--- a/src/components/clients/portal-invite-button.tsx
+++ b/src/components/clients/portal-invite-button.tsx
@@ -75,7 +75,7 @@ export function PortalInviteButton({
           setOpen(true);
         }}
       >
-        <UserPlus className="mr-1.5 h-3.5 w-3.5" />
+        <UserPlus className="mr-1.5 h-3.5 w-3.5" aria-hidden />
         Invite to portal
       </Button>
 
@@ -96,7 +96,7 @@ export function PortalInviteButton({
 
           {success ? (
             <div className="py-4 flex items-center gap-3 text-sm text-green-700">
-              <Check className="h-5 w-5" />
+              <Check className="h-5 w-5" aria-hidden />
               Activation email sent to <strong>{email}</strong>.
             </div>
           ) : (
@@ -138,7 +138,7 @@ export function PortalInviteButton({
                 <Button onClick={submit} disabled={loading || !email}>
                   {loading ? (
                     <>
-                      <Loader2 className="h-4 w-4 mr-2 animate-spin" />
+                      <Loader2 className="h-4 w-4 mr-2 animate-spin" aria-hidden />
                       Sending…
                     </>
                   ) : (
diff --git a/src/components/clients/send-documents-dialog.tsx b/src/components/clients/send-documents-dialog.tsx
index 7e560c5c..27d85de7 100644
--- a/src/components/clients/send-documents-dialog.tsx
+++ b/src/components/clients/send-documents-dialog.tsx
@@ -82,11 +82,11 @@ export function SendDocumentsDialog({
           <div className="space-y-3">
             <div>
               <h3 className="mb-2 flex items-center gap-2 text-sm font-semibold">
-                <Mail className="h-4 w-4" /> Brochures
+                <Mail className="h-4 w-4" aria-hidden /> Brochures
               </h3>
               {brochuresQuery.isLoading && (
                 <div className="flex items-center gap-2 text-sm text-muted-foreground">
-                  <Loader2 className="h-4 w-4 animate-spin" /> Loading brochures…
+                  <Loader2 className="h-4 w-4 animate-spin" aria-hidden /> Loading brochures…
                 </div>
               )}
               {!brochuresQuery.isLoading && usableBrochures.length === 0 && (
@@ -103,7 +103,7 @@ export function SendDocumentsDialog({
                     onClick={() => setActiveSend({ kind: 'brochure', brochureId: b.id })}
                   >
                     <span className="flex items-center gap-2">
-                      <FileText className="h-4 w-4" />
+                      <FileText className="h-4 w-4" aria-hidden />
                       {b.label}
                       {b.isDefault && (
                         <span className="rounded bg-primary/10 px-2 py-0.5 text-xs text-primary">
diff --git a/src/components/clients/smart-archive-dialog.tsx b/src/components/clients/smart-archive-dialog.tsx
index fc9bde5e..1273d266 100644
--- a/src/components/clients/smart-archive-dialog.tsx
+++ b/src/components/clients/smart-archive-dialog.tsx
@@ -283,7 +283,7 @@ function SmartArchiveDialogBody({
 
         {isLoading ? (
           <div className="py-8 text-center text-sm text-muted-foreground">
-            <Loader2 className="h-5 w-5 animate-spin mx-auto mb-2" />
+            <Loader2 className="h-5 w-5 animate-spin mx-auto mb-2" aria-hidden />
             Loading dossier…
           </div>
         ) : error || !dossier ? (
@@ -296,7 +296,7 @@ function SmartArchiveDialogBody({
               <Card className="border-red-300 bg-red-50">
                 <CardHeader className="pb-2">
                   <CardTitle className="text-sm font-medium text-red-800 flex items-center gap-2">
-                    <AlertTriangle className="h-4 w-4" /> Cannot archive
+                    <AlertTriangle className="h-4 w-4" aria-hidden /> Cannot archive
                   </CardTitle>
                 </CardHeader>
                 <CardContent className="text-sm text-red-700 space-y-1">
@@ -311,7 +311,7 @@ function SmartArchiveDialogBody({
               <Card className="border-amber-300 bg-amber-50">
                 <CardHeader className="pb-2">
                   <CardTitle className="text-sm font-medium text-amber-900 flex items-center gap-2">
-                    <AlertTriangle className="h-4 w-4" />
+                    <AlertTriangle className="h-4 w-4" aria-hidden />
                     Late-stage deal — confirmation required
                   </CardTitle>
                 </CardHeader>
@@ -328,7 +328,8 @@ function SmartArchiveDialogBody({
               <Card>
                 <CardHeader className="pb-2">
                   <CardTitle className="text-sm font-medium flex items-center gap-2">
-                    <FileText className="h-4 w-4" /> Pipeline interests ({dossier.interests.length})
+                    <FileText className="h-4 w-4" aria-hidden /> Pipeline interests (
+                    {dossier.interests.length})
                   </CardTitle>
                 </CardHeader>
                 <CardContent className="text-xs space-y-1">
@@ -371,7 +372,7 @@ function SmartArchiveDialogBody({
               <Card>
                 <CardHeader className="pb-2">
                   <CardTitle className="text-sm font-medium flex items-center gap-2">
-                    <Anchor className="h-4 w-4" /> Berths ({dossier.berths.length})
+                    <Anchor className="h-4 w-4" aria-hidden /> Berths ({dossier.berths.length})
                   </CardTitle>
                 </CardHeader>
                 <CardContent className="space-y-3">
@@ -425,7 +426,7 @@ function SmartArchiveDialogBody({
               <Card>
                 <CardHeader className="pb-2">
                   <CardTitle className="text-sm font-medium flex items-center gap-2">
-                    <Ship className="h-4 w-4" /> Yachts owned ({dossier.yachts.length})
+                    <Ship className="h-4 w-4" aria-hidden /> Yachts owned ({dossier.yachts.length})
                   </CardTitle>
                 </CardHeader>
                 <CardContent className="space-y-2">
@@ -459,7 +460,7 @@ function SmartArchiveDialogBody({
               <Card>
                 <CardHeader className="pb-2">
                   <CardTitle className="text-sm font-medium flex items-center gap-2">
-                    <Anchor className="h-4 w-4" /> Active reservations (
+                    <Anchor className="h-4 w-4" aria-hidden /> Active reservations (
                     {dossier.reservations.length})
                   </CardTitle>
                 </CardHeader>
@@ -493,7 +494,8 @@ function SmartArchiveDialogBody({
               <Card>
                 <CardHeader className="pb-2">
                   <CardTitle className="text-sm font-medium flex items-center gap-2">
-                    <Receipt className="h-4 w-4" /> Outstanding invoices ({dossier.invoices.length})
+                    <Receipt className="h-4 w-4" aria-hidden /> Outstanding invoices (
+                    {dossier.invoices.length})
                   </CardTitle>
                 </CardHeader>
                 <CardContent className="space-y-2">
@@ -527,7 +529,7 @@ function SmartArchiveDialogBody({
               <Card>
                 <CardHeader className="pb-2">
                   <CardTitle className="text-sm font-medium flex items-center gap-2">
-                    <FileText className="h-4 w-4" /> In-flight signing requests
+                    <FileText className="h-4 w-4" aria-hidden /> In-flight signing requests
                   </CardTitle>
                 </CardHeader>
                 <CardContent className="space-y-2">
@@ -559,7 +561,7 @@ function SmartArchiveDialogBody({
             <Card className="border-muted">
               <CardHeader className="pb-2">
                 <CardTitle className="text-sm font-medium flex items-center gap-2">
-                  <Users className="h-4 w-4" /> Automatically handled
+                  <Users className="h-4 w-4" aria-hidden /> Automatically handled
                 </CardTitle>
               </CardHeader>
               <CardContent className="text-xs text-muted-foreground space-y-1">
@@ -600,7 +602,7 @@ function SmartArchiveDialogBody({
             onClick={() => archiveMutation.mutate()}
           >
             {archiveMutation.isPending ? (
-              <Loader2 className="h-3.5 w-3.5 animate-spin mr-1.5" />
+              <Loader2 className="h-3.5 w-3.5 animate-spin mr-1.5" aria-hidden />
             ) : null}
             Archive
           </Button>
diff --git a/src/components/clients/smart-restore-dialog.tsx b/src/components/clients/smart-restore-dialog.tsx
index f501972e..41e20812 100644
--- a/src/components/clients/smart-restore-dialog.tsx
+++ b/src/components/clients/smart-restore-dialog.tsx
@@ -53,10 +53,10 @@ interface Props {
 }
 
 function iconFor(kind: string) {
-  if (kind.startsWith('berth_')) return <Anchor className="h-3 w-3" />;
-  if (kind.startsWith('yacht_')) return <Ship className="h-3 w-3" />;
-  if (kind.startsWith('documenso_')) return <FileText className="h-3 w-3" />;
-  return <Wrench className="h-3 w-3" />;
+  if (kind.startsWith('berth_')) return <Anchor className="h-3 w-3" aria-hidden />;
+  if (kind.startsWith('yacht_')) return <Ship className="h-3 w-3" aria-hidden />;
+  if (kind.startsWith('documenso_')) return <FileText className="h-3 w-3" aria-hidden />;
+  return <Wrench className="h-3 w-3" aria-hidden />;
 }
 
 export function SmartRestoreDialog(props: Props) {
@@ -135,7 +135,7 @@ function SmartRestoreDialogBody({ open, onOpenChange, clientId, clientName, onSu
 
         {dossierQuery.isLoading ? (
           <div className="py-8 text-center text-sm text-muted-foreground">
-            <Loader2 className="h-5 w-5 animate-spin mx-auto mb-2" />
+            <Loader2 className="h-5 w-5 animate-spin mx-auto mb-2" aria-hidden />
             Loading restore dossier…
           </div>
         ) : dossierQuery.error || !dossier ? (
@@ -153,7 +153,7 @@ function SmartRestoreDialogBody({ open, onOpenChange, clientId, clientName, onSu
               <Card className="border-emerald-300 bg-emerald-50">
                 <CardHeader className="pb-2">
                   <CardTitle className="text-sm font-medium text-emerald-900 flex items-center gap-2">
-                    <CheckCircle2 className="h-4 w-4" /> Auto-reversed (
+                    <CheckCircle2 className="h-4 w-4" aria-hidden /> Auto-reversed (
                     {dossier.autoReversible.length})
                   </CardTitle>
                 </CardHeader>
@@ -174,7 +174,7 @@ function SmartRestoreDialogBody({ open, onOpenChange, clientId, clientName, onSu
               <Card className="border-amber-300 bg-amber-50">
                 <CardHeader className="pb-2">
                   <CardTitle className="text-sm font-medium text-amber-900 flex items-center gap-2">
-                    <AlertTriangle className="h-4 w-4" /> Opt-in to undo (
+                    <AlertTriangle className="h-4 w-4" aria-hidden /> Opt-in to undo (
                     {dossier.reversibleWithPrompt.length})
                   </CardTitle>
                 </CardHeader>
@@ -207,7 +207,8 @@ function SmartRestoreDialogBody({ open, onOpenChange, clientId, clientName, onSu
               <Card className="border-slate-300 bg-slate-50">
                 <CardHeader className="pb-2">
                   <CardTitle className="text-sm font-medium text-slate-900 flex items-center gap-2">
-                    <Lock className="h-4 w-4" /> Cannot be undone ({dossier.locked.length})
+                    <Lock className="h-4 w-4" aria-hidden /> Cannot be undone (
+                    {dossier.locked.length})
                   </CardTitle>
                 </CardHeader>
                 <CardContent className="text-xs space-y-1.5 text-slate-700">
@@ -236,7 +237,7 @@ function SmartRestoreDialogBody({ open, onOpenChange, clientId, clientName, onSu
           >
             {restoreMutation.isPending ? (
               <>
-                <Loader2 className="h-4 w-4 animate-spin mr-1.5" /> Restoring…
+                <Loader2 className="h-4 w-4 animate-spin mr-1.5" aria-hidden /> Restoring…
               </>
             ) : (
               'Restore client'
diff --git a/src/components/companies/add-membership-dialog.tsx b/src/components/companies/add-membership-dialog.tsx
index 2ad4ca1d..dc3def88 100644
--- a/src/components/companies/add-membership-dialog.tsx
+++ b/src/components/companies/add-membership-dialog.tsx
@@ -208,7 +208,7 @@ export function AddMembershipDialog({ open, onOpenChange, companyId }: AddMember
             </Button>
             <Button type="submit" disabled={isSubmitting || mutation.isPending}>
               {(isSubmitting || mutation.isPending) && (
-                <Loader2 className="mr-2 h-4 w-4 animate-spin" />
+                <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />
               )}
               Add member
             </Button>
diff --git a/src/components/companies/company-card.tsx b/src/components/companies/company-card.tsx
index 573a2bc3..cc172e98 100644
--- a/src/components/companies/company-card.tsx
+++ b/src/components/companies/company-card.tsx
@@ -66,22 +66,22 @@ export function CompanyCard({ company, portSlug, onEdit, onArchive }: CompanyCar
               onClick={(e) => e.stopPropagation()}
               aria-label={`Actions for ${company.name}`}
             >
-              <MoreHorizontal className="h-4 w-4" />
+              <MoreHorizontal className="h-4 w-4" aria-hidden />
             </Button>
           </DropdownMenuTrigger>
           <DropdownMenuContent align="end">
             <DropdownMenuItem asChild>
               <Link href={`/${portSlug}/companies/${company.id}`}>
-                <Eye className="mr-2 h-3.5 w-3.5" />
+                <Eye className="mr-2 h-3.5 w-3.5" aria-hidden />
                 View
               </Link>
             </DropdownMenuItem>
             <DropdownMenuItem onClick={() => onEdit(company)}>
-              <Pencil className="mr-2 h-3.5 w-3.5" />
+              <Pencil className="mr-2 h-3.5 w-3.5" aria-hidden />
               Edit
             </DropdownMenuItem>
             <DropdownMenuItem className="text-destructive" onClick={() => onArchive(company)}>
-              <Archive className="mr-2 h-3.5 w-3.5" />
+              <Archive className="mr-2 h-3.5 w-3.5" aria-hidden />
               Archive
             </DropdownMenuItem>
           </DropdownMenuContent>
@@ -89,7 +89,7 @@ export function CompanyCard({ company, portSlug, onEdit, onArchive }: CompanyCar
       }
     >
       <div className="flex items-start gap-3">
-        <ListCardAvatar icon={<Building2 className="h-5 w-5" />} />
+        <ListCardAvatar icon={<Building2 className="h-5 w-5" aria-hidden />} />
         <div className="min-w-0 flex-1">
           {/* Title row + spacer for actions button */}
           <div className="flex items-start justify-between gap-2">
@@ -108,10 +108,14 @@ export function CompanyCard({ company, portSlug, onEdit, onArchive }: CompanyCar
           {country || company.taxId ? (
             <div className="mt-0.5 flex flex-wrap items-center gap-x-2 gap-y-0.5 text-xs text-muted-foreground">
               {country ? (
-                <ListCardMeta icon={<MapPin className="h-3 w-3" />}>{country}</ListCardMeta>
+                <ListCardMeta icon={<MapPin className="h-3 w-3" aria-hidden />}>
+                  {country}
+                </ListCardMeta>
               ) : null}
               {company.taxId ? (
-                <ListCardMeta icon={<Hash className="h-3 w-3" />}>{company.taxId}</ListCardMeta>
+                <ListCardMeta icon={<Hash className="h-3 w-3" aria-hidden />}>
+                  {company.taxId}
+                </ListCardMeta>
               ) : null}
             </div>
           ) : null}
diff --git a/src/components/companies/company-columns.tsx b/src/components/companies/company-columns.tsx
index cf777dde..43ab7976 100644
--- a/src/components/companies/company-columns.tsx
+++ b/src/components/companies/company-columns.tsx
@@ -157,7 +157,7 @@ export function getCompanyColumns({
               className="h-7 w-7"
               onClick={(e) => e.stopPropagation()}
             >
-              <MoreHorizontal className="h-4 w-4" />
+              <MoreHorizontal className="h-4 w-4" aria-hidden />
             </Button>
           </DropdownMenuTrigger>
           <DropdownMenuContent align="end">
@@ -166,16 +166,16 @@ export function getCompanyColumns({
                 // eslint-disable-next-line @typescript-eslint/no-explicit-any
                 href={`/${portSlug}/companies/${row.original.id}` as any}
               >
-                <Eye className="mr-2 h-3.5 w-3.5" />
+                <Eye className="mr-2 h-3.5 w-3.5" aria-hidden />
                 View
               </Link>
             </DropdownMenuItem>
             <DropdownMenuItem onClick={() => onEdit(row.original)}>
-              <Pencil className="mr-2 h-3.5 w-3.5" />
+              <Pencil className="mr-2 h-3.5 w-3.5" aria-hidden />
               Edit
             </DropdownMenuItem>
             <DropdownMenuItem className="text-destructive" onClick={() => onArchive(row.original)}>
-              <Archive className="mr-2 h-3.5 w-3.5" />
+              <Archive className="mr-2 h-3.5 w-3.5" aria-hidden />
               Archive
             </DropdownMenuItem>
           </DropdownMenuContent>
diff --git a/src/components/companies/company-detail-header.tsx b/src/components/companies/company-detail-header.tsx
index e22d1f5b..4daa7f65 100644
--- a/src/components/companies/company-detail-header.tsx
+++ b/src/components/companies/company-detail-header.tsx
@@ -108,7 +108,7 @@ export function CompanyDetailHeader({ company }: CompanyDetailHeaderProps) {
           <div className="flex flex-wrap items-center gap-2">
             <PermissionGate resource="companies" action="edit">
               <Button variant="outline" size="sm" onClick={() => setEditOpen(true)}>
-                <Pencil className="mr-1.5 h-3.5 w-3.5" />
+                <Pencil className="mr-1.5 h-3.5 w-3.5" aria-hidden />
                 Edit
               </Button>
             </PermissionGate>
@@ -119,7 +119,7 @@ export function CompanyDetailHeader({ company }: CompanyDetailHeaderProps) {
                 onClick={() => setArchiveOpen(true)}
                 disabled={isArchived}
               >
-                <Archive className="mr-1.5 h-3.5 w-3.5" />
+                <Archive className="mr-1.5 h-3.5 w-3.5" aria-hidden />
                 Archive
               </Button>
             </PermissionGate>
diff --git a/src/components/companies/company-form.tsx b/src/components/companies/company-form.tsx
index dae13853..818d6373 100644
--- a/src/components/companies/company-form.tsx
+++ b/src/components/companies/company-form.tsx
@@ -412,7 +412,7 @@ export function CompanyForm({ open, onOpenChange, company }: CompanyFormProps) {
                     size="sm"
                     onClick={() => setClientFormOpen(true)}
                   >
-                    <Plus className="mr-1 h-3.5 w-3.5" /> New client
+                    <Plus className="mr-1 h-3.5 w-3.5" aria-hidden /> New client
                   </Button>
                   <Button
                     type="button"
@@ -420,7 +420,7 @@ export function CompanyForm({ open, onOpenChange, company }: CompanyFormProps) {
                     size="sm"
                     onClick={() => setYachtFormOpen(true)}
                   >
-                    <Plus className="mr-1 h-3.5 w-3.5" /> New yacht
+                    <Plus className="mr-1 h-3.5 w-3.5" aria-hidden /> New yacht
                   </Button>
                 </div>
               </div>
@@ -459,7 +459,7 @@ export function CompanyForm({ open, onOpenChange, company }: CompanyFormProps) {
             </Button>
             <Button type="submit" disabled={isSubmitting || mutation.isPending}>
               {(isSubmitting || mutation.isPending) && (
-                <Loader2 className="mr-2 h-4 w-4 animate-spin" />
+                <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />
               )}
               {isEdit ? 'Save changes' : 'Create Company'}
             </Button>
@@ -669,7 +669,7 @@ function EntityMultiPicker({
             )}
           >
             {placeholder}
-            <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" />
+            <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" aria-hidden />
           </Button>
         </PopoverTrigger>
         <PopoverContent className="w-(--radix-popper-anchor-width) min-w-[280px] p-0" align="start">
diff --git a/src/components/companies/company-list.tsx b/src/components/companies/company-list.tsx
index 469044f2..0ade2916 100644
--- a/src/components/companies/company-list.tsx
+++ b/src/components/companies/company-list.tsx
@@ -126,7 +126,7 @@ export function CompanyList() {
         actions={
           <PermissionGate resource="companies" action="create">
             <Button size="sm" onClick={() => setCreateOpen(true)}>
-              <Plus className="mr-1.5 h-4 w-4" />
+              <Plus className="mr-1.5 h-4 w-4" aria-hidden />
               New Company
             </Button>
           </PermissionGate>
diff --git a/src/components/companies/company-members-tab.tsx b/src/components/companies/company-members-tab.tsx
index c0e1938f..ac4b3ab1 100644
--- a/src/components/companies/company-members-tab.tsx
+++ b/src/components/companies/company-members-tab.tsx
@@ -159,7 +159,7 @@ export function CompanyMembersTab({ companyId, portSlug }: CompanyMembersTabProp
 
         <PermissionGate resource="memberships" action="manage">
           <Button size="sm" onClick={() => setAddOpen(true)}>
-            <Plus className="mr-1.5 h-4 w-4" />
+            <Plus className="mr-1.5 h-4 w-4" aria-hidden />
             Add Member
           </Button>
         </PermissionGate>
@@ -167,7 +167,7 @@ export function CompanyMembersTab({ companyId, portSlug }: CompanyMembersTabProp
 
       {isLoading ? (
         <div className="flex items-center justify-center py-12">
-          <Loader2 className="h-6 w-6 animate-spin text-muted-foreground" />
+          <Loader2 className="h-6 w-6 animate-spin text-muted-foreground" aria-hidden />
         </div>
       ) : members.length === 0 ? (
         <EmptyState
@@ -226,7 +226,7 @@ export function CompanyMembersTab({ companyId, portSlug }: CompanyMembersTabProp
                         <DropdownMenu>
                           <DropdownMenuTrigger asChild>
                             <Button variant="ghost" size="icon" className="h-7 w-7">
-                              <MoreHorizontal className="h-4 w-4" />
+                              <MoreHorizontal className="h-4 w-4" aria-hidden />
                             </Button>
                           </DropdownMenuTrigger>
                           <DropdownMenuContent align="end">
@@ -235,7 +235,7 @@ export function CompanyMembersTab({ companyId, portSlug }: CompanyMembersTabProp
                                 onClick={() => setPrimaryMutation.mutate(m.id)}
                                 disabled={setPrimaryMutation.isPending}
                               >
-                                <Star className="mr-2 h-3.5 w-3.5" />
+                                <Star className="mr-2 h-3.5 w-3.5" aria-hidden />
                                 Set Primary
                               </DropdownMenuItem>
                             )}
@@ -245,7 +245,7 @@ export function CompanyMembersTab({ companyId, portSlug }: CompanyMembersTabProp
                                 onClick={() => endMutation.mutate(m.id)}
                                 disabled={endMutation.isPending}
                               >
-                                <XCircle className="mr-2 h-3.5 w-3.5" />
+                                <XCircle className="mr-2 h-3.5 w-3.5" aria-hidden />
                                 Remove from company
                               </DropdownMenuItem>
                             )}
diff --git a/src/components/companies/company-owned-yachts-tab.tsx b/src/components/companies/company-owned-yachts-tab.tsx
index 6cd70b07..6c4d83f7 100644
--- a/src/components/companies/company-owned-yachts-tab.tsx
+++ b/src/components/companies/company-owned-yachts-tab.tsx
@@ -81,7 +81,7 @@ export function CompanyOwnedYachtsTab({ companyId, portSlug }: CompanyOwnedYacht
   if (isLoading) {
     return (
       <div className="flex items-center justify-center py-12">
-        <Loader2 className="h-6 w-6 animate-spin text-muted-foreground" />
+        <Loader2 className="h-6 w-6 animate-spin text-muted-foreground" aria-hidden />
       </div>
     );
   }
diff --git a/src/components/companies/company-picker.tsx b/src/components/companies/company-picker.tsx
index f6cf9e8f..c2c50d5c 100644
--- a/src/components/companies/company-picker.tsx
+++ b/src/components/companies/company-picker.tsx
@@ -65,7 +65,7 @@ export function CompanyPicker({
           className={cn('w-full justify-between', !value && 'text-muted-foreground')}
         >
           <span className="truncate">{selectedLabel}</span>
-          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" />
+          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" aria-hidden />
         </Button>
       </PopoverTrigger>
       <PopoverContent className="w-[320px] p-0" align="start">
diff --git a/src/components/dashboard/active-deals-tile.tsx b/src/components/dashboard/active-deals-tile.tsx
index 41738f2c..18fa0b6b 100644
--- a/src/components/dashboard/active-deals-tile.tsx
+++ b/src/components/dashboard/active-deals-tile.tsx
@@ -46,7 +46,7 @@ export function ActiveDealsTile() {
             Active deals
           </p>
           {isLoading ? (
-            <Skeleton className="mt-1 h-7 w-12" />
+            <Skeleton className="mt-1 h-7 w-12" aria-hidden />
           ) : (
             <p className="text-2xl font-bold leading-tight text-foreground">
               {data?.activeInterests ?? 0}
diff --git a/src/components/dashboard/berth-status-chart.tsx b/src/components/dashboard/berth-status-chart.tsx
index 973978da..35d7e377 100644
--- a/src/components/dashboard/berth-status-chart.tsx
+++ b/src/components/dashboard/berth-status-chart.tsx
@@ -56,7 +56,7 @@ export function BerthStatusChart() {
       </CardHeader>
       <CardContent>
         {isLoading ? (
-          <Skeleton className="h-[240px] w-full" />
+          <Skeleton className="h-[240px] w-full" aria-hidden />
         ) : chartData.length === 0 ? (
           <p className="py-8 text-center text-sm text-muted-foreground">No berths yet.</p>
         ) : (
diff --git a/src/components/dashboard/chart-card.tsx b/src/components/dashboard/chart-card.tsx
index ebcc2fb8..b7042575 100644
--- a/src/components/dashboard/chart-card.tsx
+++ b/src/components/dashboard/chart-card.tsx
@@ -114,18 +114,18 @@ export function ChartCard({
               aria-label="Chart options"
               data-testid="chart-menu"
             >
-              <MoreHorizontal className="h-4 w-4" />
+              <MoreHorizontal className="h-4 w-4" aria-hidden />
             </Button>
           </DropdownMenuTrigger>
           <DropdownMenuContent align="end" className="w-44">
             {toCsv ? (
               <DropdownMenuItem onSelect={onDownloadCsv}>
-                <Download className="mr-2 h-4 w-4" />
+                <Download className="mr-2 h-4 w-4" aria-hidden />
                 Download CSV
               </DropdownMenuItem>
             ) : null}
             <DropdownMenuItem onSelect={onDownloadPng}>
-              <ImageIcon className="mr-2 h-4 w-4" />
+              <ImageIcon className="mr-2 h-4 w-4" aria-hidden />
               Download PNG
             </DropdownMenuItem>
           </DropdownMenuContent>
diff --git a/src/components/dashboard/customize-widgets-menu.tsx b/src/components/dashboard/customize-widgets-menu.tsx
index bada2462..7f01045e 100644
--- a/src/components/dashboard/customize-widgets-menu.tsx
+++ b/src/components/dashboard/customize-widgets-menu.tsx
@@ -41,7 +41,7 @@ export function CustomizeWidgetsMenu() {
     <Dialog open={open} onOpenChange={setOpen}>
       <DialogTrigger asChild>
         <Button variant="outline" size="sm" className="gap-1.5">
-          <LayoutGrid className="h-4 w-4" />
+          <LayoutGrid className="h-4 w-4" aria-hidden />
           Customize
         </Button>
       </DialogTrigger>
diff --git a/src/components/dashboard/hot-deals-card.tsx b/src/components/dashboard/hot-deals-card.tsx
index b292d571..17120772 100644
--- a/src/components/dashboard/hot-deals-card.tsx
+++ b/src/components/dashboard/hot-deals-card.tsx
@@ -69,9 +69,9 @@ export function HotDealsCard() {
       <CardContent>
         {isLoading ? (
           <div className="space-y-3">
-            <Skeleton className="h-12 w-full" />
-            <Skeleton className="h-12 w-full" />
-            <Skeleton className="h-12 w-full" />
+            <Skeleton className="h-12 w-full" aria-hidden />
+            <Skeleton className="h-12 w-full" aria-hidden />
+            <Skeleton className="h-12 w-full" aria-hidden />
           </div>
         ) : deals.length === 0 ? (
           <p className="py-4 text-center text-sm text-muted-foreground">
diff --git a/src/components/dashboard/kpi-cards.tsx b/src/components/dashboard/kpi-cards.tsx
index 7b6a5f57..b44cf145 100644
--- a/src/components/dashboard/kpi-cards.tsx
+++ b/src/components/dashboard/kpi-cards.tsx
@@ -26,8 +26,8 @@ function KpiTileSkeleton() {
   return (
     <div className="relative overflow-hidden rounded-xl border border-border bg-card p-3 shadow-sm sm:p-5">
       <div className="absolute inset-x-0 top-0 h-1 bg-muted" aria-hidden />
-      <Skeleton className="h-3 w-20" />
-      <Skeleton className="mt-2 h-6 w-24 sm:mt-3 sm:h-7" />
+      <Skeleton className="h-3 w-20" aria-hidden />
+      <Skeleton className="mt-2 h-6 w-24 sm:mt-3 sm:h-7" aria-hidden />
     </div>
   );
 }
diff --git a/src/components/dashboard/my-reminders-rail.tsx b/src/components/dashboard/my-reminders-rail.tsx
index 97ebf544..8c885f79 100644
--- a/src/components/dashboard/my-reminders-rail.tsx
+++ b/src/components/dashboard/my-reminders-rail.tsx
@@ -82,7 +82,7 @@ export function MyRemindersRail() {
       <CardHeader className="flex flex-row items-start justify-between gap-2 space-y-0 pb-3">
         <div className="space-y-0.5">
           <CardTitle className="flex items-center gap-1.5 text-base">
-            <AlarmClock className="size-4" />
+            <AlarmClock className="size-4" aria-hidden />
             Reminders
           </CardTitle>
           {overdueCount > 0 ? (
@@ -150,7 +150,10 @@ export function MyRemindersRail() {
                           ? 'in ' + formatDistanceToNowStrict(due)
                           : 'now'}
                     </span>
-                    <ChevronRight className="size-3.5 shrink-0 text-muted-foreground/60 transition-transform group-hover:translate-x-0.5" />
+                    <ChevronRight
+                      className="size-3.5 shrink-0 text-muted-foreground/60 transition-transform group-hover:translate-x-0.5"
+                      aria-hidden
+                    />
                   </Link>
                 </li>
               );
diff --git a/src/components/dashboard/pipeline-value-tile.tsx b/src/components/dashboard/pipeline-value-tile.tsx
index f9b8861e..5f7c250f 100644
--- a/src/components/dashboard/pipeline-value-tile.tsx
+++ b/src/components/dashboard/pipeline-value-tile.tsx
@@ -38,7 +38,7 @@ export function PipelineValueTile() {
             Pipeline value
           </p>
           {isLoading ? (
-            <Skeleton className="mt-1 h-7 w-24" />
+            <Skeleton className="mt-1 h-7 w-24" aria-hidden />
           ) : (
             <p
               className="truncate text-2xl font-bold leading-tight text-foreground"
diff --git a/src/components/dashboard/source-conversion-chart.tsx b/src/components/dashboard/source-conversion-chart.tsx
index dd2a874b..70969c2a 100644
--- a/src/components/dashboard/source-conversion-chart.tsx
+++ b/src/components/dashboard/source-conversion-chart.tsx
@@ -46,9 +46,9 @@ export function SourceConversionChart() {
       <CardContent>
         {isLoading ? (
           <div className="space-y-3">
-            <Skeleton className="h-8 w-full" />
-            <Skeleton className="h-8 w-full" />
-            <Skeleton className="h-8 w-full" />
+            <Skeleton className="h-8 w-full" aria-hidden />
+            <Skeleton className="h-8 w-full" aria-hidden />
+            <Skeleton className="h-8 w-full" aria-hidden />
           </div>
         ) : rows.length === 0 ? (
           <p className="py-4 text-center text-sm text-muted-foreground">
diff --git a/src/components/dashboard/timezone-drift-banner.tsx b/src/components/dashboard/timezone-drift-banner.tsx
index 53976565..b5ee8306 100644
--- a/src/components/dashboard/timezone-drift-banner.tsx
+++ b/src/components/dashboard/timezone-drift-banner.tsx
@@ -103,7 +103,7 @@ export function TimezoneDriftBanner() {
   return (
     <div className="flex flex-wrap items-center justify-between gap-3 rounded-md border border-amber-300 bg-amber-50 px-4 py-2.5 text-sm text-amber-900 sm:flex-nowrap">
       <div className="flex items-start gap-2 min-w-0">
-        <Clock className="mt-0.5 h-4 w-4 shrink-0" />
+        <Clock className="mt-0.5 h-4 w-4 shrink-0" aria-hidden />
         <div className="min-w-0">
           <p className="font-medium leading-tight">
             You appear to be in{' '}
diff --git a/src/components/dashboard/website-glance-tile.tsx b/src/components/dashboard/website-glance-tile.tsx
index 893a9115..2139b21c 100644
--- a/src/components/dashboard/website-glance-tile.tsx
+++ b/src/components/dashboard/website-glance-tile.tsx
@@ -52,7 +52,7 @@ export function WebsiteGlanceTile() {
               Website today
             </div>
             {loading ? (
-              <Skeleton className="mt-2 h-7 w-20" />
+              <Skeleton className="mt-2 h-7 w-20" aria-hidden />
             ) : (
               <div className="mt-1 flex items-baseline gap-2 text-lg font-semibold tabular-nums sm:mt-2 sm:text-2xl">
                 {today.toLocaleString()}
diff --git a/src/components/documents/aggregated-section.tsx b/src/components/documents/aggregated-section.tsx
index 21de9582..21175f0f 100644
--- a/src/components/documents/aggregated-section.tsx
+++ b/src/components/documents/aggregated-section.tsx
@@ -56,7 +56,7 @@ export function AggregatedSection<K extends AggregatedItemKind['kind']>({
         <h3 className="flex items-center gap-2 text-sm font-semibold text-foreground">
           {icon}
           {title}
-          <Loader2 className="ml-1 h-3.5 w-3.5 animate-spin text-muted-foreground" />
+          <Loader2 className="ml-1 h-3.5 w-3.5 animate-spin text-muted-foreground" aria-hidden />
         </h3>
       </section>
     );
@@ -108,7 +108,7 @@ function GroupBlock<K extends AggregatedItemKind['kind']>({
   // The server always sets exactly one of `files` / `workflows` per group;
   // unify them into a single list for rendering. The discriminated-union
   // generic on `AggregatedSection` keeps the row type correct upstream.
-  const items = ((group.files ?? group.workflows ?? []) as unknown) as ItemOfKind<K>[];
+  const items = (group.files ?? group.workflows ?? []) as unknown as ItemOfKind<K>[];
   return (
     <div className="px-3 py-2">
       <header className="mb-1 text-[0.7rem] font-medium uppercase tracking-wide text-muted-foreground">
diff --git a/src/components/documents/create-document-wizard.tsx b/src/components/documents/create-document-wizard.tsx
index 9fa8860b..2faf68d1 100644
--- a/src/components/documents/create-document-wizard.tsx
+++ b/src/components/documents/create-document-wizard.tsx
@@ -192,7 +192,7 @@ export function CreateDocumentWizard({ portSlug }: CreateDocumentWizardProps) {
         actions={
           <Button asChild variant="outline">
             <Link href={`/${portSlug}/documents`}>
-              <ArrowLeft className="mr-1.5 h-4 w-4" /> Back
+              <ArrowLeft className="mr-1.5 h-4 w-4" aria-hidden /> Back
             </Link>
           </Button>
         }
@@ -360,7 +360,7 @@ export function CreateDocumentWizard({ portSlug }: CreateDocumentWizardProps) {
               Signers
             </h2>
             <Button size="sm" variant="outline" onClick={addSigner}>
-              <Plus className="mr-1.5 h-3.5 w-3.5" /> Add signer
+              <Plus className="mr-1.5 h-3.5 w-3.5" aria-hidden /> Add signer
             </Button>
           </div>
           <ul className="space-y-2">
@@ -405,7 +405,7 @@ export function CreateDocumentWizard({ portSlug }: CreateDocumentWizardProps) {
                   onClick={() => removeSigner(idx)}
                   className="text-muted-foreground hover:text-destructive"
                 >
-                  <Trash2 className="h-3.5 w-3.5" />
+                  <Trash2 className="h-3.5 w-3.5" aria-hidden />
                 </button>
               </li>
             ))}
diff --git a/src/components/documents/document-detail.tsx b/src/components/documents/document-detail.tsx
index 50b49a24..e0c68a4f 100644
--- a/src/components/documents/document-detail.tsx
+++ b/src/components/documents/document-detail.tsx
@@ -133,7 +133,7 @@ export function DocumentDetail({ documentId, portSlug }: DocumentDetailProps) {
           actions={
             <Button asChild variant="outline">
               <Link href={`/${portSlug}/documents`}>
-                <ArrowLeft className="mr-1.5 h-4 w-4" />
+                <ArrowLeft className="mr-1.5 h-4 w-4" aria-hidden />
                 Back to documents
               </Link>
             </Button>
@@ -225,18 +225,18 @@ export function DocumentDetail({ documentId, portSlug }: DocumentDetailProps) {
           <div className="flex items-center gap-2">
             <Button asChild variant="outline" size="sm">
               <Link href={`/${portSlug}/documents`}>
-                <ArrowLeft className="mr-1.5 h-4 w-4" /> Back
+                <ArrowLeft className="mr-1.5 h-4 w-4" aria-hidden /> Back
               </Link>
             </Button>
             {isComplete && doc.signedFileId ? (
               <>
                 <Button asChild size="sm">
                   <Link href={`/api/v1/files/${doc.signedFileId}/download`}>
-                    <Download className="mr-1.5 h-4 w-4" /> Download signed PDF
+                    <Download className="mr-1.5 h-4 w-4" aria-hidden /> Download signed PDF
                   </Link>
                 </Button>
                 <Button size="sm" variant="outline" onClick={handleEmailSignedPdf}>
-                  <Mail className="mr-1.5 h-4 w-4" /> Email signatories
+                  <Mail className="mr-1.5 h-4 w-4" aria-hidden /> Email signatories
                 </Button>
               </>
             ) : null}
@@ -299,7 +299,7 @@ export function DocumentDetail({ documentId, portSlug }: DocumentDetailProps) {
                             variant="outline"
                             onClick={() => handleRemind(signer.id)}
                           >
-                            <Bell className="mr-1.5 h-3 w-3" /> Remind
+                            <Bell className="mr-1.5 h-3 w-3" aria-hidden /> Remind
                           </Button>
                           {signer.signingUrl ? (
                             <button
@@ -370,7 +370,7 @@ export function DocumentDetail({ documentId, portSlug }: DocumentDetailProps) {
                       }}
                       className="text-muted-foreground hover:text-destructive"
                     >
-                      <Trash2 className="h-3.5 w-3.5" />
+                      <Trash2 className="h-3.5 w-3.5" aria-hidden />
                     </button>
                   </li>
                 ))}
diff --git a/src/components/documents/document-list.tsx b/src/components/documents/document-list.tsx
index 9f99fd06..ac01e69b 100644
--- a/src/components/documents/document-list.tsx
+++ b/src/components/documents/document-list.tsx
@@ -95,7 +95,7 @@ function DocRow({ doc, onDelete, onSend }: DocRowProps) {
               )}
               <PermissionGate resource="documents" action="manage_folders">
                 <DropdownMenuItem onSelect={() => setMoveOpen(true)}>
-                  <FolderInput className="mr-2 h-4 w-4" />
+                  <FolderInput className="mr-2 h-4 w-4" aria-hidden />
                   Move to folder…
                 </DropdownMenuItem>
               </PermissionGate>
diff --git a/src/components/documents/document-template-picker.tsx b/src/components/documents/document-template-picker.tsx
index 3415633d..ef1b5b6b 100644
--- a/src/components/documents/document-template-picker.tsx
+++ b/src/components/documents/document-template-picker.tsx
@@ -79,7 +79,7 @@ export function DocumentTemplatePicker({
           className={cn('w-full justify-between', !value && 'text-muted-foreground')}
         >
           <span className="truncate">{selectedLabel}</span>
-          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" />
+          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" aria-hidden />
         </Button>
       </PopoverTrigger>
       <PopoverContent className="w-[320px] p-0" align="start">
diff --git a/src/components/documents/documents-hub.tsx b/src/components/documents/documents-hub.tsx
index 4b6a89ec..daffd38a 100644
--- a/src/components/documents/documents-hub.tsx
+++ b/src/components/documents/documents-hub.tsx
@@ -311,7 +311,11 @@ function FlatFolderListing({ portSlug, folderId }: FlatFolderListingProps) {
             onClick={() => setExpandedDocId(expanded ? null : doc.id)}
             className="flex min-h-[44px] min-w-[44px] items-center justify-center text-muted-foreground transition-transform"
           >
-            {expanded ? <ChevronDown className="h-4 w-4" /> : <ChevronRight className="h-4 w-4" />}
+            {expanded ? (
+              <ChevronDown className="h-4 w-4" aria-hidden />
+            ) : (
+              <ChevronRight className="h-4 w-4" aria-hidden />
+            )}
           </button>
           <Link
             href={`/${portSlug}/documents/${doc.id}`}
@@ -409,18 +413,18 @@ function FlatFolderListing({ portSlug, folderId }: FlatFolderListingProps) {
         </ul>
       ) : documents.length === 0 ? (
         <EmptyState
-          icon={<FileText className="h-7 w-7" />}
+          icon={<FileText className="h-7 w-7" aria-hidden />}
           title="No documents in this folder"
           body="Upload a file, generate a signing flow, or move existing documents here."
           actions={
             <>
               <Button onClick={() => setUploadOpen(true)}>
-                <Upload className="mr-1.5 h-4 w-4" />
+                <Upload className="mr-1.5 h-4 w-4" aria-hidden />
                 Upload file
               </Button>
               <Button asChild variant="outline">
                 <Link href={`/${portSlug}/documents/new`}>
-                  <Plus className="mr-1.5 h-4 w-4" />
+                  <Plus className="mr-1.5 h-4 w-4" aria-hidden />
                   Generate for signing
                 </Link>
               </Button>
@@ -560,7 +564,7 @@ function FolderDropZone({ folderId, entityType, entityId, children }: FolderDrop
       {(dragActive || uploading) && (
         <div className="pointer-events-none absolute inset-0 flex items-center justify-center rounded-md border-2 border-dashed border-primary bg-primary/5 backdrop-blur-[1px] z-10">
           <div className="flex flex-col items-center gap-2 text-sm font-medium text-primary">
-            <Upload className="h-8 w-8" />
+            <Upload className="h-8 w-8" aria-hidden />
             {uploading ? 'Uploading…' : 'Drop to upload to this folder'}
           </div>
         </div>
diff --git a/src/components/documents/entity-folder-view.tsx b/src/components/documents/entity-folder-view.tsx
index ee14efda..06ceea24 100644
--- a/src/components/documents/entity-folder-view.tsx
+++ b/src/components/documents/entity-folder-view.tsx
@@ -50,7 +50,7 @@ export function EntityFolderView({ portSlug, entityType, entityId }: Props) {
     <div className="space-y-4">
       <AggregatedSection<'workflows'>
         title="Signing in progress"
-        icon={<ClipboardSignature className="h-4 w-4 text-muted-foreground" />}
+        icon={<ClipboardSignature className="h-4 w-4 text-muted-foreground" aria-hidden />}
         groups={workflowGroups}
         loading={workflowsLoading}
         emptyMessage="No workflows in flight for this entity."
@@ -68,7 +68,7 @@ export function EntityFolderView({ portSlug, entityType, entityId }: Props) {
 
       <AggregatedSection<'files'>
         title="Files"
-        icon={<FileText className="h-4 w-4 text-muted-foreground" />}
+        icon={<FileText className="h-4 w-4 text-muted-foreground" aria-hidden />}
         groups={fileGroups}
         loading={filesLoading}
         emptyMessage="No files for this entity yet."
@@ -86,7 +86,7 @@ export function EntityFolderView({ portSlug, entityType, entityId }: Props) {
                     className="min-h-[44px] gap-1 px-2 text-xs text-brand"
                     onClick={() => setDetailsId(signedFromDocumentId)}
                   >
-                    <Eye className="h-3 w-3" />
+                    <Eye className="h-3 w-3" aria-hidden />
                     View signing details
                   </Button>
                 ) : null}
diff --git a/src/components/documents/eoi-generate-dialog.tsx b/src/components/documents/eoi-generate-dialog.tsx
index d0b3ab12..9f4391a7 100644
--- a/src/components/documents/eoi-generate-dialog.tsx
+++ b/src/components/documents/eoi-generate-dialog.tsx
@@ -259,7 +259,7 @@ export function EoiGenerateDialog({
       <DialogContent className="sm:max-w-lg">
         <DialogHeader>
           <DialogTitle className="flex items-center gap-2">
-            <FileSignature className="size-4" />
+            <FileSignature className="size-4" aria-hidden />
             Generate Expression of Interest
           </DialogTitle>
           <DialogDescription>
@@ -290,9 +290,9 @@ export function EoiGenerateDialog({
 
           {ctxLoading ? (
             <div className="space-y-2">
-              <Skeleton className="h-4 w-full" />
-              <Skeleton className="h-4 w-3/4" />
-              <Skeleton className="h-4 w-2/3" />
+              <Skeleton className="h-4 w-full" aria-hidden />
+              <Skeleton className="h-4 w-3/4" aria-hidden />
+              <Skeleton className="h-4 w-2/3" aria-hidden />
             </div>
           ) : ctx ? (
             <div className="space-y-3 rounded-md border bg-muted/20 p-3">
@@ -339,9 +339,9 @@ export function EoiGenerateDialog({
                       className="inline-flex items-center gap-1 text-xs text-primary hover:underline"
                       onClick={() => onOpenChange(false)}
                     >
-                      <Pencil className="size-3" />
+                      <Pencil className="size-3" aria-hidden />
                       Edit client details
-                      <ExternalLink className="size-3" />
+                      <ExternalLink className="size-3" aria-hidden />
                     </Link>
                     <Link
                       // eslint-disable-next-line @typescript-eslint/no-explicit-any
@@ -349,9 +349,9 @@ export function EoiGenerateDialog({
                       className="inline-flex items-center gap-1 text-xs text-primary hover:underline"
                       onClick={() => onOpenChange(false)}
                     >
-                      <Pencil className="size-3" />
+                      <Pencil className="size-3" aria-hidden />
                       Manage linked berths
-                      <ExternalLink className="size-3" />
+                      <ExternalLink className="size-3" aria-hidden />
                     </Link>
                   </div>
                 </div>
@@ -365,7 +365,7 @@ export function EoiGenerateDialog({
 
           {!ctxLoading && ctx && !requiredMet && (
             <p className="flex items-start gap-2 rounded-md border border-amber-200 bg-amber-50 px-3 py-2 text-xs text-amber-900">
-              <AlertTriangle className="size-3.5 shrink-0 mt-0.5" />
+              <AlertTriangle className="size-3.5 shrink-0 mt-0.5" aria-hidden />
               Add the missing required details on the client&apos;s record before generating the
               EOI.
             </p>
@@ -481,7 +481,7 @@ function PreviewRow({
                 className="ml-1 inline-flex items-center rounded p-0.5 text-muted-foreground hover:bg-muted/60 hover:text-foreground"
                 aria-label={`Edit ${label}`}
               >
-                <Pencil className="h-3 w-3" />
+                <Pencil className="h-3 w-3" aria-hidden />
               </button>
             ) : null}
           </>
diff --git a/src/components/documents/folder-actions-menu.tsx b/src/components/documents/folder-actions-menu.tsx
index a4357523..bad32a67 100644
--- a/src/components/documents/folder-actions-menu.tsx
+++ b/src/components/documents/folder-actions-menu.tsx
@@ -87,14 +87,14 @@ export function FolderActionsMenu({ selectedFolderId, onAfterDelete }: FolderAct
             setCreateOpen(true);
           }}
         >
-          <FolderPlus className="mr-2 h-4 w-4" />
+          <FolderPlus className="mr-2 h-4 w-4" aria-hidden />
           New folder {isFolderSelected ? 'inside this' : 'at root'}
         </Button>
         {isFolderSelected ? (
           <DropdownMenu>
             <DropdownMenuTrigger asChild>
               <Button variant="ghost" size="icon">
-                <MoreHorizontal className="h-4 w-4" />
+                <MoreHorizontal className="h-4 w-4" aria-hidden />
                 <span className="sr-only">More folder actions</span>
               </Button>
             </DropdownMenuTrigger>
@@ -111,7 +111,7 @@ export function FolderActionsMenu({ selectedFolderId, onAfterDelete }: FolderAct
                           setRenameOpen(true);
                         }}
                       >
-                        <Pencil className="mr-2 h-4 w-4" />
+                        <Pencil className="mr-2 h-4 w-4" aria-hidden />
                         Rename
                       </DropdownMenuItem>
                     </span>
@@ -130,7 +130,7 @@ export function FolderActionsMenu({ selectedFolderId, onAfterDelete }: FolderAct
                             onSelect={(e) => e.preventDefault()}
                             className="text-destructive"
                           >
-                            <Trash2 className="mr-2 h-4 w-4" />
+                            <Trash2 className="mr-2 h-4 w-4" aria-hidden />
                             Delete
                           </DropdownMenuItem>
                         }
diff --git a/src/components/documents/folder-breadcrumb.tsx b/src/components/documents/folder-breadcrumb.tsx
index 0fa24b80..185b5ef0 100644
--- a/src/components/documents/folder-breadcrumb.tsx
+++ b/src/components/documents/folder-breadcrumb.tsx
@@ -42,7 +42,7 @@ export function FolderBreadcrumb({ selectedFolderId, onSelect }: FolderBreadcrum
         onClick={() => onSelect(undefined)}
         className="flex min-h-[44px] items-center gap-1 py-2 hover:text-foreground"
       >
-        <Home className="h-3.5 w-3.5" />
+        <Home className="h-3.5 w-3.5" aria-hidden />
         <span>All</span>
       </button>
       {path.length === 0 && selectedFolderId === null ? (
diff --git a/src/components/documents/folder-tree-sidebar.tsx b/src/components/documents/folder-tree-sidebar.tsx
index f2bc1015..a808bcf0 100644
--- a/src/components/documents/folder-tree-sidebar.tsx
+++ b/src/components/documents/folder-tree-sidebar.tsx
@@ -37,7 +37,7 @@ export function FolderTreeSidebar({ selectedFolderId, onSelect, footer }: Folder
       <Sheet open={mobileOpen} onOpenChange={setMobileOpen}>
         <SheetTrigger asChild>
           <Button variant="outline" size="sm" className="min-h-[44px]">
-            <FolderTree className="mr-2 h-4 w-4" />
+            <FolderTree className="mr-2 h-4 w-4" aria-hidden />
             Show folders
           </Button>
         </SheetTrigger>
@@ -130,7 +130,7 @@ function PseudoRow({
       )}
       onClick={onClick}
     >
-      <Icon className="mr-2 h-4 w-4" />
+      <Icon className="mr-2 h-4 w-4" aria-hidden />
       {label}
     </Button>
   );
@@ -183,9 +183,9 @@ function FolderRow({
           )}
         >
           {open && hasChildren ? (
-            <FolderOpen className="h-4 w-4 shrink-0" />
+            <FolderOpen className="h-4 w-4 shrink-0" aria-hidden />
           ) : (
-            <Folder className="h-4 w-4 shrink-0" />
+            <Folder className="h-4 w-4 shrink-0" aria-hidden />
           )}
           <span className="truncate">
             {node.name}
diff --git a/src/components/documents/hub-root-view.tsx b/src/components/documents/hub-root-view.tsx
index fb1ca963..9bb3c473 100644
--- a/src/components/documents/hub-root-view.tsx
+++ b/src/components/documents/hub-root-view.tsx
@@ -51,7 +51,7 @@ export function HubRootView({ portSlug }: Props) {
     <div className="space-y-4">
       <section className="rounded-md border bg-white">
         <h3 className="flex items-center gap-2 border-b px-3 py-2 text-sm font-semibold">
-          <ClipboardSignature className="h-4 w-4 text-muted-foreground" />
+          <ClipboardSignature className="h-4 w-4 text-muted-foreground" aria-hidden />
           Signing in progress
         </h3>
         {workflowsLoading ? (
@@ -76,7 +76,7 @@ export function HubRootView({ portSlug }: Props) {
 
       <section className="rounded-md border bg-white">
         <h3 className="flex items-center gap-2 border-b px-3 py-2 text-sm font-semibold">
-          <FileText className="h-4 w-4 text-muted-foreground" />
+          <FileText className="h-4 w-4 text-muted-foreground" aria-hidden />
           Recent files
         </h3>
         {filesLoading ? (
diff --git a/src/components/documents/move-to-folder-dialog.tsx b/src/components/documents/move-to-folder-dialog.tsx
index 678c6d01..c1e89f5e 100644
--- a/src/components/documents/move-to-folder-dialog.tsx
+++ b/src/components/documents/move-to-folder-dialog.tsx
@@ -119,7 +119,7 @@ function DialogBody({
             }
           }}
         >
-          <FolderInput className="mr-1.5 h-4 w-4" />
+          <FolderInput className="mr-1.5 h-4 w-4" aria-hidden />
           Move
         </Button>
       </DialogFooter>
diff --git a/src/components/documents/new-document-menu.tsx b/src/components/documents/new-document-menu.tsx
index 88472b09..7c9a1ff6 100644
--- a/src/components/documents/new-document-menu.tsx
+++ b/src/components/documents/new-document-menu.tsx
@@ -62,14 +62,14 @@ export function NewDocumentMenu({
       <DropdownMenu>
         <DropdownMenuTrigger asChild>
           <Button size={size}>
-            <Plus className="mr-1.5 h-4 w-4" />
+            <Plus className="mr-1.5 h-4 w-4" aria-hidden />
             New document
-            <ChevronDown className="ml-1.5 h-4 w-4 opacity-80" />
+            <ChevronDown className="ml-1.5 h-4 w-4 opacity-80" aria-hidden />
           </Button>
         </DropdownMenuTrigger>
         <DropdownMenuContent align="end" className="w-64">
           <DropdownMenuItem onSelect={() => setUploadOpen(true)} className="gap-2 py-2.5">
-            <Upload className="h-4 w-4" />
+            <Upload className="h-4 w-4" aria-hidden />
             <div className="flex flex-col">
               <span>Upload file</span>
               <span className="text-xs text-muted-foreground">
@@ -79,7 +79,7 @@ export function NewDocumentMenu({
           </DropdownMenuItem>
           <DropdownMenuItem asChild className="gap-2 py-2.5">
             <Link href={`/${portSlug}/documents/new`}>
-              <FileSignature className="h-4 w-4" />
+              <FileSignature className="h-4 w-4" aria-hidden />
               <div className="flex flex-col">
                 <span>Generate document for signing</span>
                 <span className="text-xs text-muted-foreground">
diff --git a/src/components/documents/signing-details-dialog.tsx b/src/components/documents/signing-details-dialog.tsx
index b6b6d35e..5078196e 100644
--- a/src/components/documents/signing-details-dialog.tsx
+++ b/src/components/documents/signing-details-dialog.tsx
@@ -64,7 +64,7 @@ export function SigningDetailsDialog({ documentId, open, onOpenChange }: Props)
         </DialogHeader>
         {isLoading || !data ? (
           <div className="flex items-center justify-center py-12 text-muted-foreground">
-            <Loader2 className="mr-2 h-4 w-4 animate-spin" />
+            <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />
             Loading...
           </div>
         ) : (
diff --git a/src/components/email/email-accounts-list.tsx b/src/components/email/email-accounts-list.tsx
index 8ad9967d..42d117c2 100644
--- a/src/components/email/email-accounts-list.tsx
+++ b/src/components/email/email-accounts-list.tsx
@@ -117,7 +117,7 @@ export function EmailAccountsList() {
           </p>
         </div>
         <Button onClick={() => setSheetOpen(true)}>
-          <Plus className="h-4 w-4 mr-1.5" />
+          <Plus className="h-4 w-4 mr-1.5" aria-hidden />
           Add account
         </Button>
       </div>
@@ -126,14 +126,14 @@ export function EmailAccountsList() {
         <p className="text-sm text-muted-foreground">Loading…</p>
       ) : accounts.length === 0 ? (
         <div className="rounded-lg border border-dashed p-8 text-center text-muted-foreground">
-          <Mail className="mx-auto h-6 w-6 mb-2" />
+          <Mail className="mx-auto h-6 w-6 mb-2" aria-hidden />
           <p className="text-sm">No email accounts connected.</p>
         </div>
       ) : (
         <div className="rounded-lg border divide-y">
           {accounts.map((a) => (
             <div key={a.id} className="flex items-center gap-3 p-3">
-              <Mail className="h-5 w-5 text-muted-foreground" />
+              <Mail className="h-5 w-5 text-muted-foreground" aria-hidden />
               <div className="flex-1 min-w-0">
                 <div className="text-sm font-medium truncate">{a.emailAddress}</div>
                 <div className="text-xs text-muted-foreground">
@@ -152,12 +152,12 @@ export function EmailAccountsList() {
                 disabled={syncMutation.isPending}
                 title="Sync now"
               >
-                <RefreshCw className="h-4 w-4" />
+                <RefreshCw className="h-4 w-4" aria-hidden />
               </Button>
               <ConfirmationDialog
                 trigger={
                   <Button variant="ghost" size="icon" className="text-destructive">
-                    <Trash2 className="h-4 w-4" />
+                    <Trash2 className="h-4 w-4" aria-hidden />
                   </Button>
                 }
                 title="Remove account"
diff --git a/src/components/email/email-draft-button.tsx b/src/components/email/email-draft-button.tsx
index 6900499d..b1c2f6ed 100644
--- a/src/components/email/email-draft-button.tsx
+++ b/src/components/email/email-draft-button.tsx
@@ -101,12 +101,12 @@ export function EmailDraftButton({
       <Button variant="outline" size="sm" onClick={handleGenerateDraft} disabled={isLoading}>
         {isLoading ? (
           <>
-            <Loader2 className="mr-1.5 h-3.5 w-3.5 animate-spin" />
+            <Loader2 className="mr-1.5 h-3.5 w-3.5 animate-spin" aria-hidden />
             Generating...
           </>
         ) : (
           <>
-            <Mail className="mr-1.5 h-3.5 w-3.5" />
+            <Mail className="mr-1.5 h-3.5 w-3.5" aria-hidden />
             Draft Email
           </>
         )}
@@ -156,12 +156,12 @@ export function EmailDraftButton({
                 <Button variant="outline" size="sm" onClick={handleCopy}>
                   {copied ? (
                     <>
-                      <Check className="mr-1.5 h-3.5 w-3.5 text-green-600" />
+                      <Check className="mr-1.5 h-3.5 w-3.5 text-green-600" aria-hidden />
                       Copied
                     </>
                   ) : (
                     <>
-                      <Copy className="mr-1.5 h-3.5 w-3.5" />
+                      <Copy className="mr-1.5 h-3.5 w-3.5" aria-hidden />
                       Copy to clipboard
                     </>
                   )}
@@ -174,7 +174,7 @@ export function EmailDraftButton({
                 >
                   {isLoading ? (
                     <>
-                      <Loader2 className="mr-1.5 h-3.5 w-3.5 animate-spin" />
+                      <Loader2 className="mr-1.5 h-3.5 w-3.5 animate-spin" aria-hidden />
                       Regenerating...
                     </>
                   ) : (
diff --git a/src/components/email/email-threads-list.tsx b/src/components/email/email-threads-list.tsx
index b2c1e56a..2d653020 100644
--- a/src/components/email/email-threads-list.tsx
+++ b/src/components/email/email-threads-list.tsx
@@ -35,11 +35,11 @@ export function EmailThreadsList() {
         {Array.from({ length: 4 }).map((_, i) => (
           <div key={i} className="p-3 space-y-2">
             <div className="flex items-center justify-between gap-2">
-              <Skeleton className="h-4 w-1/3" />
-              <Skeleton className="h-3 w-16" />
+              <Skeleton className="h-4 w-1/3" aria-hidden />
+              <Skeleton className="h-3 w-16" aria-hidden />
             </div>
-            <Skeleton className="h-3 w-1/2" />
-            <Skeleton className="h-3 w-2/3" />
+            <Skeleton className="h-3 w-1/2" aria-hidden />
+            <Skeleton className="h-3 w-2/3" aria-hidden />
           </div>
         ))}
       </div>
diff --git a/src/components/expenses/expense-card.tsx b/src/components/expenses/expense-card.tsx
index bb8978eb..77999f2e 100644
--- a/src/components/expenses/expense-card.tsx
+++ b/src/components/expenses/expense-card.tsx
@@ -108,22 +108,22 @@ export function ExpenseCard({ expense, portSlug, onEdit, onArchive }: ExpenseCar
               onClick={(e) => e.stopPropagation()}
               aria-label={`Actions for expense: ${title}`}
             >
-              <MoreHorizontal className="h-4 w-4" />
+              <MoreHorizontal className="h-4 w-4" aria-hidden />
             </Button>
           </DropdownMenuTrigger>
           <DropdownMenuContent align="end">
             <DropdownMenuItem asChild>
               <Link href={`/${portSlug}/expenses/${expense.id}`}>
-                <Eye className="mr-2 h-3.5 w-3.5" />
+                <Eye className="mr-2 h-3.5 w-3.5" aria-hidden />
                 View
               </Link>
             </DropdownMenuItem>
             <DropdownMenuItem onClick={() => onEdit(expense)}>
-              <Pencil className="mr-2 h-3.5 w-3.5" />
+              <Pencil className="mr-2 h-3.5 w-3.5" aria-hidden />
               Edit
             </DropdownMenuItem>
             <DropdownMenuItem className="text-destructive" onClick={() => onArchive(expense)}>
-              <Archive className="mr-2 h-3.5 w-3.5" />
+              <Archive className="mr-2 h-3.5 w-3.5" aria-hidden />
               Archive
             </DropdownMenuItem>
           </DropdownMenuContent>
@@ -131,7 +131,7 @@ export function ExpenseCard({ expense, portSlug, onEdit, onArchive }: ExpenseCar
       }
     >
       <div className="flex items-start gap-3">
-        <ListCardAvatar icon={<Receipt className="h-5 w-5" />} />
+        <ListCardAvatar icon={<Receipt className="h-5 w-5" aria-hidden />} />
         <div className="min-w-0 flex-1">
           {/* Title row + spacer for actions button */}
           <div className="flex items-start justify-between gap-2">
@@ -159,7 +159,9 @@ export function ExpenseCard({ expense, portSlug, onEdit, onArchive }: ExpenseCar
           {/* Date meta */}
           {dateFormatted ? (
             <div className="mt-0.5 flex flex-wrap items-center gap-x-2 text-xs text-muted-foreground">
-              <ListCardMeta icon={<Calendar className="h-3 w-3" />}>{dateFormatted}</ListCardMeta>
+              <ListCardMeta icon={<Calendar className="h-3 w-3" aria-hidden />}>
+                {dateFormatted}
+              </ListCardMeta>
             </div>
           ) : null}
 
diff --git a/src/components/expenses/expense-columns.tsx b/src/components/expenses/expense-columns.tsx
index 089d39f6..d909200c 100644
--- a/src/components/expenses/expense-columns.tsx
+++ b/src/components/expenses/expense-columns.tsx
@@ -166,22 +166,22 @@ export function getExpenseColumns({
               className="h-7 w-7"
               onClick={(e) => e.stopPropagation()}
             >
-              <MoreHorizontal className="h-4 w-4" />
+              <MoreHorizontal className="h-4 w-4" aria-hidden />
             </Button>
           </DropdownMenuTrigger>
           <DropdownMenuContent align="end">
             <DropdownMenuItem asChild>
               <Link href={`/${portSlug}/expenses/${row.original.id}`}>
-                <Eye className="mr-2 h-3.5 w-3.5" />
+                <Eye className="mr-2 h-3.5 w-3.5" aria-hidden />
                 View
               </Link>
             </DropdownMenuItem>
             <DropdownMenuItem onClick={() => onEdit(row.original)}>
-              <Pencil className="mr-2 h-3.5 w-3.5" />
+              <Pencil className="mr-2 h-3.5 w-3.5" aria-hidden />
               Edit
             </DropdownMenuItem>
             <DropdownMenuItem className="text-destructive" onClick={() => onArchive(row.original)}>
-              <Archive className="mr-2 h-3.5 w-3.5" />
+              <Archive className="mr-2 h-3.5 w-3.5" aria-hidden />
               Archive
             </DropdownMenuItem>
           </DropdownMenuContent>
diff --git a/src/components/expenses/expense-detail.tsx b/src/components/expenses/expense-detail.tsx
index 125e198d..73fc1dec 100644
--- a/src/components/expenses/expense-detail.tsx
+++ b/src/components/expenses/expense-detail.tsx
@@ -48,7 +48,7 @@ function ReceiptThumbnail({ fileId }: { fileId: string }) {
   if (isLoading) {
     return (
       <div className="flex h-32 items-center justify-center rounded border bg-muted/40 text-xs text-muted-foreground">
-        <Loader2 className="mr-2 h-3 w-3 animate-spin" /> Loading preview…
+        <Loader2 className="mr-2 h-3 w-3 animate-spin" aria-hidden /> Loading preview…
       </div>
     );
   }
@@ -69,7 +69,7 @@ function ReceiptThumbnail({ fileId }: { fileId: string }) {
         </a>
       ) : (
         <div className="flex h-32 items-center justify-center text-muted-foreground">
-          <FileText className="h-8 w-8" />
+          <FileText className="h-8 w-8" aria-hidden />
         </div>
       )}
       <div className="mt-2 flex items-center justify-between text-xs text-muted-foreground">
@@ -78,7 +78,7 @@ function ReceiptThumbnail({ fileId }: { fileId: string }) {
           href={`/api/v1/files/${fileId}/download`}
           className="inline-flex items-center gap-1 text-primary hover:underline"
         >
-          <Download className="h-3 w-3" /> Download
+          <Download className="h-3 w-3" aria-hidden /> Download
         </a>
       </div>
     </div>
@@ -131,7 +131,7 @@ export function ExpenseDetail({ expenseId, onEdit, onArchived }: ExpenseDetailPr
   if (isLoading) {
     return (
       <div className="flex items-center justify-center p-12">
-        <Loader2 className="h-6 w-6 animate-spin text-muted-foreground" />
+        <Loader2 className="h-6 w-6 animate-spin text-muted-foreground" aria-hidden />
       </div>
     );
   }
@@ -162,7 +162,7 @@ export function ExpenseDetail({ expenseId, onEdit, onArchived }: ExpenseDetailPr
           {onEdit && (
             <PermissionGate resource="expenses" action="edit">
               <Button variant="outline" size="sm" onClick={onEdit}>
-                <Edit className="mr-1.5 h-4 w-4" />
+                <Edit className="mr-1.5 h-4 w-4" aria-hidden />
                 Edit
               </Button>
             </PermissionGate>
@@ -174,7 +174,7 @@ export function ExpenseDetail({ expenseId, onEdit, onArchived }: ExpenseDetailPr
               className="text-destructive"
               onClick={() => setArchiveOpen(true)}
             >
-              <Archive className="mr-1.5 h-4 w-4" />
+              <Archive className="mr-1.5 h-4 w-4" aria-hidden />
               Archive
             </Button>
           </PermissionGate>
@@ -259,7 +259,7 @@ export function ExpenseDetail({ expenseId, onEdit, onArchived }: ExpenseDetailPr
         <Card>
           <CardHeader>
             <CardTitle className="text-sm font-medium flex items-center gap-2">
-              <Receipt className="h-4 w-4" />
+              <Receipt className="h-4 w-4" aria-hidden />
               Receipts ({expense.receiptFileIds.length})
             </CardTitle>
           </CardHeader>
diff --git a/src/components/expenses/expense-duplicate-banner.tsx b/src/components/expenses/expense-duplicate-banner.tsx
index f4402145..8790748a 100644
--- a/src/components/expenses/expense-duplicate-banner.tsx
+++ b/src/components/expenses/expense-duplicate-banner.tsx
@@ -75,7 +75,7 @@ export function ExpenseDuplicateBanner({ expense }: Props) {
       )}
     >
       <div className="flex min-w-0 items-start gap-2">
-        <AlertTriangle className="mt-0.5 h-4 w-4 shrink-0" />
+        <AlertTriangle className="mt-0.5 h-4 w-4 shrink-0" aria-hidden />
         <div className="min-w-0">
           <p className="font-medium">Looks like a duplicate</p>
           <p className="mt-0.5 text-xs text-amber-800">
diff --git a/src/components/expenses/expense-form-dialog.tsx b/src/components/expenses/expense-form-dialog.tsx
index 3480a4cd..b4c636ef 100644
--- a/src/components/expenses/expense-form-dialog.tsx
+++ b/src/components/expenses/expense-form-dialog.tsx
@@ -363,7 +363,7 @@ export function ExpenseFormDialog({ open, onOpenChange, expense }: ExpenseFormDi
                   disabled={noReceipt}
                   onClick={() => fileInputRef.current?.click()}
                 >
-                  <Upload className="mr-2 h-4 w-4" />
+                  <Upload className="mr-2 h-4 w-4" aria-hidden />
                   Upload receipt image or PDF
                 </Button>
               )}
@@ -393,7 +393,7 @@ export function ExpenseFormDialog({ open, onOpenChange, expense }: ExpenseFormDi
 
               {noReceipt && (
                 <div className="flex gap-2 rounded-md border border-amber-300 bg-amber-50 p-2 text-xs text-amber-900 dark:border-amber-900 dark:bg-amber-950/40 dark:text-amber-200">
-                  <AlertTriangle className="h-4 w-4 shrink-0" />
+                  <AlertTriangle className="h-4 w-4 shrink-0" aria-hidden />
                   <span>
                     Expenses without a receipt may not be reimbursed by the parent company. The PDF
                     export will flag this expense.
@@ -416,7 +416,7 @@ export function ExpenseFormDialog({ open, onOpenChange, expense }: ExpenseFormDi
               disabled={isSubmitting || mutation.isPending || isUploading || !canSubmit}
             >
               {(isSubmitting || mutation.isPending) && (
-                <Loader2 className="mr-2 h-4 w-4 animate-spin" />
+                <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />
               )}
               {isEdit ? 'Save changes' : 'Create Expense'}
             </Button>
diff --git a/src/components/expenses/trip-label-combobox.tsx b/src/components/expenses/trip-label-combobox.tsx
index 7508ab9b..9bc1858f 100644
--- a/src/components/expenses/trip-label-combobox.tsx
+++ b/src/components/expenses/trip-label-combobox.tsx
@@ -73,7 +73,7 @@ export function TripLabelCombobox({
           className={cn('w-full justify-between font-normal', !value && 'text-muted-foreground')}
         >
           <span className="truncate">{value || placeholder}</span>
-          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" />
+          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" aria-hidden />
         </Button>
       </PopoverTrigger>
       <PopoverContent className="w-(--radix-popper-anchor-width) min-w-[260px] p-0">
@@ -102,7 +102,7 @@ export function TripLabelCombobox({
             {showCreate ? (
               <CommandGroup heading="New">
                 <CommandItem value={`__create__${trimmed}`} onSelect={() => commit(trimmed)}>
-                  <Plus className="mr-2 h-4 w-4" />
+                  <Plus className="mr-2 h-4 w-4" aria-hidden />
                   <span className="truncate">Create &ldquo;{trimmed}&rdquo;</span>
                 </CommandItem>
               </CommandGroup>
diff --git a/src/components/interests/add-berth-to-interest-dialog.tsx b/src/components/interests/add-berth-to-interest-dialog.tsx
index bdc690cb..9f15549f 100644
--- a/src/components/interests/add-berth-to-interest-dialog.tsx
+++ b/src/components/interests/add-berth-to-interest-dialog.tsx
@@ -81,7 +81,7 @@ export function AddBerthToInterestDialog({
             title="Pitching specifically"
             description="The client is pitched on this exact berth."
             consequence="This berth will appear as under interest on the public map."
-            icon={<Eye className="size-4" />}
+            icon={<Eye className="size-4" aria-hidden />}
           />
           <RoleCard
             value="exploring"
@@ -89,7 +89,7 @@ export function AddBerthToInterestDialog({
             title="Just exploring"
             description="The berth is being considered or covered by the EOI bundle, but not pitched specifically."
             consequence="This berth is hidden from the public map."
-            icon={<EyeOff className="size-4" />}
+            icon={<EyeOff className="size-4" aria-hidden />}
           />
         </RadioGroup>
 
@@ -109,7 +109,9 @@ export function AddBerthToInterestDialog({
             Cancel
           </Button>
           <Button type="button" onClick={handleSubmit} disabled={mutation.isPending}>
-            {mutation.isPending ? <Loader2 className="mr-1.5 size-3.5 animate-spin" /> : null}
+            {mutation.isPending ? (
+              <Loader2 className="mr-1.5 size-3.5 animate-spin" aria-hidden />
+            ) : null}
             Add berth to interest
           </Button>
         </DialogFooter>
diff --git a/src/components/interests/berth-recommender-panel.tsx b/src/components/interests/berth-recommender-panel.tsx
index de4c5903..490c109d 100644
--- a/src/components/interests/berth-recommender-panel.tsx
+++ b/src/components/interests/berth-recommender-panel.tsx
@@ -156,7 +156,7 @@ function RecommendationCard({ rec, portSlug, onAdd }: RecommendationCardProps) {
             </span>
             {showHeat ? (
               <span className="inline-flex items-center gap-1 rounded-md border border-rose-200 bg-rose-50 px-2 py-0.5 text-xs font-medium text-rose-800">
-                <Flame className="size-3" />
+                <Flame className="size-3" aria-hidden />
                 Heat {Math.round(rec.heat!.total)}
               </span>
             ) : null}
@@ -174,9 +174,9 @@ function RecommendationCard({ rec, portSlug, onAdd }: RecommendationCardProps) {
           </div>
         </div>
         {expanded ? (
-          <ChevronUp className="size-4 shrink-0 text-muted-foreground" />
+          <ChevronUp className="size-4 shrink-0 text-muted-foreground" aria-hidden />
         ) : (
-          <ChevronDown className="size-4 shrink-0 text-muted-foreground" />
+          <ChevronDown className="size-4 shrink-0 text-muted-foreground" aria-hidden />
         )}
       </button>
 
@@ -213,7 +213,7 @@ function RecommendationCard({ rec, portSlug, onAdd }: RecommendationCardProps) {
                 onAdd(rec);
               }}
             >
-              <Plus className="mr-1.5 size-3.5" />
+              <Plus className="mr-1.5 size-3.5" aria-hidden />
               Add to interest
             </Button>
             <Button asChild size="sm" variant="outline">
@@ -375,7 +375,7 @@ export function BerthRecommenderPanel({
         <div className="flex flex-wrap items-start justify-between gap-2">
           <div className="min-w-0 space-y-1">
             <CardTitle className="flex items-center gap-2 text-base">
-              <Sparkles className="size-4 text-brand-600" />
+              <Sparkles className="size-4 text-brand-600" aria-hidden />
               Recommendations for {formatDesired(desiredLengthFt, desiredWidthFt, desiredDraftFt)}
             </CardTitle>
             {!hasDimensions ? (
@@ -392,7 +392,7 @@ export function BerthRecommenderPanel({
               onClick={() => setFiltersOpen((v) => !v)}
               disabled={!hasDimensions}
             >
-              <Filter className="mr-1.5 size-3.5" />
+              <Filter className="mr-1.5 size-3.5" aria-hidden />
               {filtersOpen ? 'Hide filters' : 'Add filters'}
             </Button>
             <Button
diff --git a/src/components/interests/external-eoi-upload-dialog.tsx b/src/components/interests/external-eoi-upload-dialog.tsx
index 0a9b136f..f1bd7087 100644
--- a/src/components/interests/external-eoi-upload-dialog.tsx
+++ b/src/components/interests/external-eoi-upload-dialog.tsx
@@ -146,9 +146,9 @@ export function ExternalEoiUploadDialog({ open, onOpenChange, interestId, onSucc
           </Button>
           <Button onClick={() => mutation.mutate()} disabled={!file || mutation.isPending}>
             {mutation.isPending ? (
-              <Loader2 className="h-3.5 w-3.5 animate-spin mr-1.5" />
+              <Loader2 className="h-3.5 w-3.5 animate-spin mr-1.5" aria-hidden />
             ) : (
-              <Upload className="h-3.5 w-3.5 mr-1.5" />
+              <Upload className="h-3.5 w-3.5 mr-1.5" aria-hidden />
             )}
             Upload
           </Button>
diff --git a/src/components/interests/inline-stage-picker.tsx b/src/components/interests/inline-stage-picker.tsx
index 74eded5d..ebe1dd7e 100644
--- a/src/components/interests/inline-stage-picker.tsx
+++ b/src/components/interests/inline-stage-picker.tsx
@@ -217,9 +217,9 @@ export function InlineStagePicker({
           >
             <span>{STAGE_LABELS[stage]}</span>
             {mutation.isPending ? (
-              <Loader2 className="size-3 animate-spin" />
+              <Loader2 className="size-3 animate-spin" aria-hidden />
             ) : showChevron ? (
-              <ChevronDown className="size-3 opacity-70" />
+              <ChevronDown className="size-3 opacity-70" aria-hidden />
             ) : null}
           </button>
         </PopoverTrigger>
@@ -234,7 +234,7 @@ export function InlineStagePicker({
             // strongly nudged for the audit log.
             <div className="p-3 space-y-3">
               <div className="flex items-start gap-2">
-                <AlertTriangle className="size-4 shrink-0 text-amber-600 mt-0.5" />
+                <AlertTriangle className="size-4 shrink-0 text-amber-600 mt-0.5" aria-hidden />
                 <div className="text-sm">
                   <p className="font-medium text-foreground">Override transition</p>
                   <p className="text-xs text-muted-foreground">
@@ -270,7 +270,7 @@ export function InlineStagePicker({
                   disabled={mutation.isPending}
                   className="gap-1"
                 >
-                  <ChevronLeft className="size-3.5" />
+                  <ChevronLeft className="size-3.5" aria-hidden />
                   Back
                 </Button>
                 <Button
@@ -279,7 +279,9 @@ export function InlineStagePicker({
                   onClick={commitOverride}
                   disabled={mutation.isPending}
                 >
-                  {mutation.isPending && <Loader2 className="size-3.5 animate-spin mr-1" />}
+                  {mutation.isPending && (
+                    <Loader2 className="size-3.5 animate-spin mr-1" aria-hidden />
+                  )}
                   Confirm override
                 </Button>
               </div>
@@ -324,9 +326,12 @@ export function InlineStagePicker({
                       />
                       <span className="flex-1">{STAGE_LABELS[s]}</span>
                       {isPending ? (
-                        <Loader2 className="size-3.5 animate-spin text-muted-foreground" />
+                        <Loader2
+                          className="size-3.5 animate-spin text-muted-foreground"
+                          aria-hidden
+                        />
                       ) : isCurrent ? (
-                        <Check className="size-3.5 text-muted-foreground" />
+                        <Check className="size-3.5 text-muted-foreground" aria-hidden />
                       ) : isOverride && canOverride ? (
                         <span
                           className="text-[10px] uppercase tracking-wide text-amber-600"
@@ -376,7 +381,7 @@ export function InlineStagePicker({
                 if (openConfirmTarget) void unlinkAllAndOpen(openConfirmTarget);
               }}
             >
-              {unlinking && <Loader2 className="mr-1.5 size-3.5 animate-spin" />}
+              {unlinking && <Loader2 className="mr-1.5 size-3.5 animate-spin" aria-hidden />}
               Unlink {linkedBerthCount} {linkedBerthCount === 1 ? 'berth' : 'berths'} & reset
             </AlertDialogAction>
           </AlertDialogFooter>
diff --git a/src/components/interests/interest-card.tsx b/src/components/interests/interest-card.tsx
index 9c4a8c3e..ce830b77 100644
--- a/src/components/interests/interest-card.tsx
+++ b/src/components/interests/interest-card.tsx
@@ -73,16 +73,16 @@ export function InterestCard({ interest, portSlug, onEdit, onArchive }: Interest
               onClick={(e) => e.stopPropagation()}
               aria-label={`Actions for ${clientName}'s interest`}
             >
-              <MoreHorizontal className="h-4 w-4" />
+              <MoreHorizontal className="h-4 w-4" aria-hidden />
             </Button>
           </DropdownMenuTrigger>
           <DropdownMenuContent align="end">
             <DropdownMenuItem onClick={() => onEdit(interest)}>
-              <Pencil className="mr-2 h-3.5 w-3.5" />
+              <Pencil className="mr-2 h-3.5 w-3.5" aria-hidden />
               Edit
             </DropdownMenuItem>
             <DropdownMenuItem className="text-destructive" onClick={() => onArchive(interest)}>
-              <Archive className="mr-2 h-3.5 w-3.5" />
+              <Archive className="mr-2 h-3.5 w-3.5" aria-hidden />
               Archive
             </DropdownMenuItem>
           </DropdownMenuContent>
@@ -104,7 +104,7 @@ export function InterestCard({ interest, portSlug, onEdit, onArchive }: Interest
                   aria-label={`${notesCount} note${notesCount === 1 ? '' : 's'}`}
                   className="inline-flex shrink-0 items-center text-muted-foreground"
                 >
-                  <MessageSquare className="size-3.5" />
+                  <MessageSquare className="size-3.5" aria-hidden />
                 </span>
               ) : null}
             </div>
diff --git a/src/components/interests/interest-columns.tsx b/src/components/interests/interest-columns.tsx
index 298df995..69599779 100644
--- a/src/components/interests/interest-columns.tsx
+++ b/src/components/interests/interest-columns.tsx
@@ -134,7 +134,7 @@ export function getInterestColumns({
                 aria-label={`${notesCount} note${notesCount === 1 ? '' : 's'}`}
                 className="inline-flex items-center text-muted-foreground"
               >
-                <MessageSquare className="size-3.5" />
+                <MessageSquare className="size-3.5" aria-hidden />
               </span>
             ) : null}
           </div>
@@ -294,16 +294,16 @@ export function getInterestColumns({
               className="h-7 w-7"
               onClick={(e) => e.stopPropagation()}
             >
-              <MoreHorizontal className="h-4 w-4" />
+              <MoreHorizontal className="h-4 w-4" aria-hidden />
             </Button>
           </DropdownMenuTrigger>
           <DropdownMenuContent align="end">
             <DropdownMenuItem onClick={() => onEdit(row.original)}>
-              <Pencil className="mr-2 h-3.5 w-3.5" />
+              <Pencil className="mr-2 h-3.5 w-3.5" aria-hidden />
               Edit
             </DropdownMenuItem>
             <DropdownMenuItem className="text-destructive" onClick={() => onArchive(row.original)}>
-              <Archive className="mr-2 h-3.5 w-3.5" />
+              <Archive className="mr-2 h-3.5 w-3.5" aria-hidden />
               Archive
             </DropdownMenuItem>
           </DropdownMenuContent>
diff --git a/src/components/interests/interest-contact-log-tab.tsx b/src/components/interests/interest-contact-log-tab.tsx
index 4fa5acc9..de6e3fdb 100644
--- a/src/components/interests/interest-contact-log-tab.tsx
+++ b/src/components/interests/interest-contact-log-tab.tsx
@@ -114,15 +114,15 @@ export function InterestContactLogTab({ interestId }: InterestContactLogTabProps
           </p>
         </div>
         <Button size="sm" onClick={() => setComposeOpen(true)} className="gap-1.5">
-          <Plus className="size-4" />
+          <Plus className="size-4" aria-hidden />
           Log contact
         </Button>
       </div>
 
       {isLoading ? (
         <div className="space-y-2">
-          <Skeleton className="h-20 rounded-lg" />
-          <Skeleton className="h-20 rounded-lg" />
+          <Skeleton className="h-20 rounded-lg" aria-hidden />
+          <Skeleton className="h-20 rounded-lg" aria-hidden />
         </div>
       ) : entries.length === 0 ? (
         <EmptyState onAdd={() => setComposeOpen(true)} />
@@ -183,7 +183,7 @@ function ContactLogRow({
             channelMeta.tone,
           )}
         >
-          <Icon className="size-3.5" />
+          <Icon className="size-3.5" aria-hidden />
         </span>
         <div className="min-w-0 flex-1 space-y-1.5">
           <div className="flex items-center gap-2 flex-wrap">
@@ -201,7 +201,7 @@ function ContactLogRow({
           <p className="text-sm text-foreground whitespace-pre-wrap">{entry.summary}</p>
           {entry.followUpAt && (
             <p className="inline-flex items-center gap-1.5 rounded-md border border-amber-200 bg-amber-50 px-2 py-0.5 text-xs text-amber-900">
-              <Bell className="size-3" />
+              <Bell className="size-3" aria-hidden />
               Follow up {format(new Date(entry.followUpAt), 'MMM d, yyyy')} (reminder created)
             </p>
           )}
@@ -209,12 +209,12 @@ function ContactLogRow({
         <DropdownMenu>
           <DropdownMenuTrigger asChild>
             <Button variant="ghost" size="icon" className="h-7 w-7" aria-label="Row actions">
-              <MoreVertical className="h-4 w-4" />
+              <MoreVertical className="h-4 w-4" aria-hidden />
             </Button>
           </DropdownMenuTrigger>
           <DropdownMenuContent align="end">
             <DropdownMenuItem onClick={() => onEdit(entry)}>
-              <Pencil className="mr-2 size-3.5" />
+              <Pencil className="mr-2 size-3.5" aria-hidden />
               Edit
             </DropdownMenuItem>
             <DropdownMenuItem
@@ -229,7 +229,7 @@ function ContactLogRow({
                 if (ok) deleteMutation.mutate();
               }}
             >
-              <Trash2 className="mr-2 size-3.5" />
+              <Trash2 className="mr-2 size-3.5" aria-hidden />
               Delete
             </DropdownMenuItem>
           </DropdownMenuContent>
@@ -246,7 +246,7 @@ function EmptyState({ onAdd }: { onAdd: () => void }) {
   return (
     <div className="rounded-xl border border-dashed bg-muted/20 p-8 text-center">
       <div className="mx-auto flex size-12 items-center justify-center rounded-full bg-background text-muted-foreground">
-        <Phone className="size-5" />
+        <Phone className="size-5" aria-hidden />
       </div>
       <h3 className="mt-3 text-sm font-medium text-foreground">No contact logged yet</h3>
       <p className="mt-1 text-xs text-muted-foreground">
@@ -254,7 +254,7 @@ function EmptyState({ onAdd }: { onAdd: () => void }) {
         picks up the deal.
       </p>
       <Button size="sm" onClick={onAdd} className="mt-4 gap-1.5">
-        <Plus className="size-3.5" />
+        <Plus className="size-3.5" aria-hidden />
         Log first contact
       </Button>
     </div>
diff --git a/src/components/interests/interest-contract-tab.tsx b/src/components/interests/interest-contract-tab.tsx
index 98da4ddc..25354433 100644
--- a/src/components/interests/interest-contract-tab.tsx
+++ b/src/components/interests/interest-contract-tab.tsx
@@ -106,7 +106,7 @@ export function InterestContractTab({ interestId, clientId: _clientId }: Interes
   return (
     <div className="space-y-5">
       {docsLoading ? (
-        <Skeleton className="h-44 w-full rounded-lg" />
+        <Skeleton className="h-44 w-full rounded-lg" aria-hidden />
       ) : activeDoc ? (
         <ActiveContractCard
           doc={activeDoc}
@@ -145,7 +145,7 @@ export function InterestContractTab({ interestId, clientId: _clientId }: Interes
                     className="text-xs text-primary hover:underline inline-flex items-center gap-1"
                   >
                     Open
-                    <ExternalLink className="size-3" />
+                    <ExternalLink className="size-3" aria-hidden />
                   </Link>
                 )}
               </li>
@@ -232,7 +232,7 @@ function ActiveContractCard({
       <header className="flex items-start justify-between gap-3">
         <div className="min-w-0 flex-1 space-y-1">
           <div className="flex items-center gap-2 flex-wrap">
-            <FileSignature className="size-4 text-foreground" />
+            <FileSignature className="size-4 text-foreground" aria-hidden />
             <h2 className="truncate text-base font-semibold text-foreground">{doc.title}</h2>
             <StatusBadge status={doc.status} />
           </div>
@@ -261,7 +261,11 @@ function ActiveContractCard({
               onClick={() => remindAllMutation.mutate()}
               className="gap-1.5 [&_svg]:size-3.5"
             >
-              {remindAllMutation.isPending ? <Loader2 className="animate-spin" /> : <RefreshCw />}
+              {remindAllMutation.isPending ? (
+                <Loader2 className="animate-spin" aria-hidden />
+              ) : (
+                <RefreshCw />
+              )}
               Remind all
             </Button>
           )}
@@ -274,7 +278,7 @@ function ActiveContractCard({
         </h3>
         {signersLoading ? (
           <div className="flex items-center gap-2 text-sm text-muted-foreground">
-            <Loader2 className="size-3.5 animate-spin" /> Loading signers…
+            <Loader2 className="size-3.5 animate-spin" aria-hidden /> Loading signers…
           </div>
         ) : signers.length === 0 ? (
           <p className="text-sm text-muted-foreground italic">
@@ -287,7 +291,7 @@ function ActiveContractCard({
 
       <footer className="mt-3 flex flex-wrap items-center justify-between gap-2 text-xs text-muted-foreground">
         <p className="flex items-center gap-1.5">
-          <AlertTriangle className="size-3 text-amber-600" />
+          <AlertTriangle className="size-3 text-amber-600" aria-hidden />
           Reminders are rate-limited (max once per 7 days per signer).
         </p>
         <div className="flex items-center gap-1">
@@ -338,7 +342,7 @@ function EmptyContractState({
   return (
     <section className="rounded-xl border border-dashed bg-muted/20 p-8 text-center">
       <div className="mx-auto flex size-14 items-center justify-center rounded-full bg-background text-muted-foreground">
-        <FileSignature className="size-6" />
+        <FileSignature className="size-6" aria-hidden />
       </div>
       <h2 className="mt-4 text-base font-semibold text-foreground">
         No contract in flight for this interest
@@ -349,11 +353,11 @@ function EmptyContractState({
       </p>
       <div className="mt-5 flex flex-wrap items-center justify-center gap-2">
         <Button onClick={onUploadForSigning} size="sm" className="gap-1.5">
-          <FileSignature className="size-4" />
+          <FileSignature className="size-4" aria-hidden />
           Upload draft for signing
         </Button>
         <Button onClick={onUploadSigned} variant="outline" size="sm" className="gap-1.5">
-          <Upload className="size-4" />
+          <Upload className="size-4" aria-hidden />
           Upload paper-signed copy
         </Button>
       </div>
@@ -372,7 +376,7 @@ function StatusBadge({ status }: { status: DocumentRow['status'] }) {
         STATUS_TONES[status],
       )}
     >
-      {status === 'completed' && <CheckCircle2 className="mr-1 size-3" />}
+      {status === 'completed' && <CheckCircle2 className="mr-1 size-3" aria-hidden />}
       {STATUS_LABELS[status]}
     </Badge>
   );
diff --git a/src/components/interests/interest-detail-header.tsx b/src/components/interests/interest-detail-header.tsx
index f3c0439e..88825eca 100644
--- a/src/components/interests/interest-detail-header.tsx
+++ b/src/components/interests/interest-detail-header.tsx
@@ -231,7 +231,7 @@ export function InterestDetailHeader({ portSlug, interest }: InterestDetailHeade
                     interest.activeReminderCount === 1 ? '' : 's'
                   }`}
                 >
-                  <AlarmClock className="size-3" />
+                  <AlarmClock className="size-3" aria-hidden />
                   {interest.activeReminderCount}
                 </span>
               ) : null}
@@ -336,7 +336,7 @@ export function InterestDetailHeader({ portSlug, interest }: InterestDetailHeade
                     'hover:bg-foreground/5 disabled:opacity-50',
                   )}
                 >
-                  <RefreshCcw className="size-3.5" />
+                  <RefreshCcw className="size-3.5" aria-hidden />
                   Reopen
                 </button>
               ) : (
@@ -358,7 +358,7 @@ export function InterestDetailHeader({ portSlug, interest }: InterestDetailHeade
                       'hover:bg-emerald-100',
                     )}
                   >
-                    <Trophy className="size-3.5" />
+                    <Trophy className="size-3.5" aria-hidden />
                     <span className="hidden sm:inline">Mark won</span>
                   </button>
                   <button
@@ -372,7 +372,7 @@ export function InterestDetailHeader({ portSlug, interest }: InterestDetailHeade
                       'hover:bg-rose-50',
                     )}
                   >
-                    <XCircle className="size-3.5" />
+                    <XCircle className="size-3.5" aria-hidden />
                     <span className="hidden sm:inline">Close as lost</span>
                   </button>
                 </>
@@ -395,7 +395,7 @@ export function InterestDetailHeader({ portSlug, interest }: InterestDetailHeade
                   'hover:bg-foreground/5 hover:text-foreground',
                 )}
               >
-                <Pencil className="size-4" />
+                <Pencil className="size-4" aria-hidden />
               </button>
             </PermissionGate>
             <PermissionGate resource="interests" action="delete">
@@ -410,7 +410,11 @@ export function InterestDetailHeader({ portSlug, interest }: InterestDetailHeade
                   isArchived ? 'hover:text-foreground' : 'hover:text-destructive',
                 )}
               >
-                {isArchived ? <RotateCcw className="size-4" /> : <Archive className="size-4" />}
+                {isArchived ? (
+                  <RotateCcw className="size-4" aria-hidden />
+                ) : (
+                  <Archive className="size-4" aria-hidden />
+                )}
               </button>
             </PermissionGate>
           </div>
diff --git a/src/components/interests/interest-documents-tab.tsx b/src/components/interests/interest-documents-tab.tsx
index c04ed4f4..075a052a 100644
--- a/src/components/interests/interest-documents-tab.tsx
+++ b/src/components/interests/interest-documents-tab.tsx
@@ -110,7 +110,7 @@ export function InterestDocumentsTab({ interestId }: InterestDocumentsTabProps)
           emptyState={
             <div className="flex flex-col items-center gap-3 rounded-lg border border-dashed border-border bg-muted/20 px-6 py-10 text-center">
               <div className="flex size-10 items-center justify-center rounded-full bg-background text-muted-foreground">
-                <FileSignature className="size-5" />
+                <FileSignature className="size-5" aria-hidden />
               </div>
               <div className="space-y-1">
                 <p className="text-sm font-medium text-foreground">No documents yet</p>
diff --git a/src/components/interests/interest-eoi-tab.tsx b/src/components/interests/interest-eoi-tab.tsx
index 317b6d4e..f6793b20 100644
--- a/src/components/interests/interest-eoi-tab.tsx
+++ b/src/components/interests/interest-eoi-tab.tsx
@@ -106,7 +106,7 @@ export function InterestEoiTab({ interestId, clientId }: InterestEoiTabProps) {
   return (
     <div className="space-y-5">
       {docsLoading ? (
-        <Skeleton className="h-44 w-full rounded-lg" />
+        <Skeleton className="h-44 w-full rounded-lg" aria-hidden />
       ) : activeDoc ? (
         <ActiveEoiCard
           doc={activeDoc}
@@ -148,7 +148,7 @@ export function InterestEoiTab({ interestId, clientId }: InterestEoiTabProps) {
                     className="text-xs text-primary hover:underline inline-flex items-center gap-1"
                   >
                     Open
-                    <ExternalLink className="size-3" />
+                    <ExternalLink className="size-3" aria-hidden />
                   </Link>
                 )}
               </li>
@@ -221,7 +221,7 @@ function ActiveEoiCard({
       <header className="flex items-start justify-between gap-3">
         <div className="min-w-0 flex-1 space-y-1">
           <div className="flex items-center gap-2 flex-wrap">
-            <FileSignature className="size-4 text-foreground" />
+            <FileSignature className="size-4 text-foreground" aria-hidden />
             <h2 className="truncate text-base font-semibold text-foreground">{doc.title}</h2>
             <StatusBadge status={doc.status} />
           </div>
@@ -250,7 +250,11 @@ function ActiveEoiCard({
               onClick={() => remindAllMutation.mutate()}
               className="gap-1.5 [&_svg]:size-3.5"
             >
-              {remindAllMutation.isPending ? <Loader2 className="animate-spin" /> : <RefreshCw />}
+              {remindAllMutation.isPending ? (
+                <Loader2 className="animate-spin" aria-hidden />
+              ) : (
+                <RefreshCw />
+              )}
               Remind all
             </Button>
           )}
@@ -263,7 +267,7 @@ function ActiveEoiCard({
         </h3>
         {signersLoading ? (
           <div className="flex items-center gap-2 text-sm text-muted-foreground">
-            <Loader2 className="size-3.5 animate-spin" /> Loading signers…
+            <Loader2 className="size-3.5 animate-spin" aria-hidden /> Loading signers…
           </div>
         ) : signers.length === 0 ? (
           <p className="text-sm text-muted-foreground italic">
@@ -276,7 +280,7 @@ function ActiveEoiCard({
 
       <footer className="mt-3 flex flex-wrap items-center justify-between gap-2 text-xs text-muted-foreground">
         <p className="flex items-center gap-1.5">
-          <AlertTriangle className="size-3 text-amber-600" />
+          <AlertTriangle className="size-3 text-amber-600" aria-hidden />
           Reminders are rate-limited (max once per 7 days per signer).
         </p>
         <div className="flex items-center gap-1">
@@ -327,7 +331,7 @@ function EmptyEoiState({
   return (
     <section className="rounded-xl border border-dashed bg-muted/20 p-8 text-center">
       <div className="mx-auto flex size-14 items-center justify-center rounded-full bg-background text-muted-foreground">
-        <FileSignature className="size-6" />
+        <FileSignature className="size-6" aria-hidden />
       </div>
       <h2 className="mt-4 text-base font-semibold text-foreground">
         No EOI in flight for this interest
@@ -338,11 +342,11 @@ function EmptyEoiState({
       </p>
       <div className="mt-5 flex flex-wrap items-center justify-center gap-2">
         <Button onClick={onGenerate} size="sm" className="gap-1.5">
-          <FileSignature className="size-4" />
+          <FileSignature className="size-4" aria-hidden />
           Generate EOI
         </Button>
         <Button onClick={onUploadSigned} variant="outline" size="sm" className="gap-1.5">
-          <Upload className="size-4" />
+          <Upload className="size-4" aria-hidden />
           Upload paper-signed copy
         </Button>
       </div>
@@ -359,7 +363,7 @@ function StatusBadge({ status }: { status: DocumentRow['status'] }) {
         STATUS_TONES[status],
       )}
     >
-      {status === 'completed' && <CheckCircle2 className="mr-1 size-3" />}
+      {status === 'completed' && <CheckCircle2 className="mr-1 size-3" aria-hidden />}
       {STATUS_LABELS[status]}
     </Badge>
   );
diff --git a/src/components/interests/interest-form.tsx b/src/components/interests/interest-form.tsx
index 389a6c33..84dfbe24 100644
--- a/src/components/interests/interest-form.tsx
+++ b/src/components/interests/interest-form.tsx
@@ -278,7 +278,7 @@ export function InterestForm({ open, onOpenChange, defaultClientId, interest }:
                     disabled={isEdit}
                   >
                     {selectedClient?.label ?? interest?.clientName ?? 'Select client...'}
-                    <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" />
+                    <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" aria-hidden />
                   </Button>
                 </PopoverTrigger>
                 <PopoverContent className="w-(--radix-popper-anchor-width) min-w-[280px] p-0">
@@ -336,7 +336,7 @@ export function InterestForm({ open, onOpenChange, defaultClientId, interest }:
                     )}
                   >
                     {selectedBerth?.label ?? interest?.berthMooringNumber ?? 'Select berth...'}
-                    <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" />
+                    <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" aria-hidden />
                   </Button>
                 </PopoverTrigger>
                 <PopoverContent className="w-(--radix-popper-anchor-width) min-w-[280px] p-0">
@@ -398,7 +398,7 @@ export function InterestForm({ open, onOpenChange, defaultClientId, interest }:
                     className="h-7 text-xs"
                     onClick={() => setCreateYachtOpen(true)}
                   >
-                    <Plus className="mr-1 h-3 w-3" />
+                    <Plus className="mr-1 h-3 w-3" aria-hidden />
                     Add new
                   </Button>
                 )}
@@ -581,7 +581,7 @@ export function InterestForm({ open, onOpenChange, defaultClientId, interest }:
             </Button>
             <Button type="submit" disabled={isSubmitting || mutation.isPending}>
               {(isSubmitting || mutation.isPending) && (
-                <Loader2 className="mr-2 h-4 w-4 animate-spin" />
+                <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />
               )}
               {isEdit ? 'Save changes' : 'Create Interest'}
             </Button>
diff --git a/src/components/interests/interest-list.tsx b/src/components/interests/interest-list.tsx
index f65a0f60..b2609487 100644
--- a/src/components/interests/interest-list.tsx
+++ b/src/components/interests/interest-list.tsx
@@ -179,7 +179,7 @@ export function InterestList() {
                 className="rounded-none"
                 onClick={() => setViewMode('table')}
               >
-                <LayoutList className="h-4 w-4" />
+                <LayoutList className="h-4 w-4" aria-hidden />
               </Button>
               <Button
                 size="sm"
@@ -187,12 +187,12 @@ export function InterestList() {
                 className="rounded-none"
                 onClick={() => setViewMode('board')}
               >
-                <Kanban className="h-4 w-4" />
+                <Kanban className="h-4 w-4" aria-hidden />
               </Button>
             </div>
             <PermissionGate resource="interests" action="create">
               <Button size="sm" onClick={() => setCreateOpen(true)}>
-                <Plus className="mr-1.5 h-4 w-4" />
+                <Plus className="mr-1.5 h-4 w-4" aria-hidden />
                 New Interest
               </Button>
             </PermissionGate>
@@ -350,7 +350,7 @@ export function InterestList() {
           aria-label="New interest"
           className="fixed bottom-[calc(env(safe-area-inset-bottom)+86px)] right-4 z-40 inline-flex h-12 w-12 items-center justify-center rounded-full bg-primary text-primary-foreground shadow-lg transition-transform hover:scale-105 active:scale-95 lg:hidden"
         >
-          <Plus className="h-6 w-6" />
+          <Plus className="h-6 w-6" aria-hidden />
         </button>
       </PermissionGate>
 
diff --git a/src/components/interests/interest-outcome-dialog.tsx b/src/components/interests/interest-outcome-dialog.tsx
index daaea08f..6a198735 100644
--- a/src/components/interests/interest-outcome-dialog.tsx
+++ b/src/components/interests/interest-outcome-dialog.tsx
@@ -83,9 +83,9 @@ export function InterestOutcomeDialog({ interestId, open, onOpenChange, mode }:
         <DialogHeader>
           <DialogTitle className="flex items-center gap-2">
             {mode === 'won' ? (
-              <Trophy className="h-4 w-4 text-emerald-600" />
+              <Trophy className="h-4 w-4 text-emerald-600" aria-hidden />
             ) : (
-              <XCircle className="h-4 w-4 text-rose-600" />
+              <XCircle className="h-4 w-4 text-rose-600" aria-hidden />
             )}
             {mode === 'won' ? 'Mark interest as won' : 'Close interest as lost'}
           </DialogTitle>
@@ -148,7 +148,7 @@ export function InterestOutcomeDialog({ interestId, open, onOpenChange, mode }:
                 : 'bg-rose-600 hover:bg-rose-700'
             }
           >
-            {mutation.isPending && <Loader2 className="mr-2 h-4 w-4 animate-spin" />}
+            {mutation.isPending && <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />}
             {mode === 'won' ? 'Mark as won' : 'Close as lost'}
           </Button>
         </DialogFooter>
diff --git a/src/components/interests/interest-picker.tsx b/src/components/interests/interest-picker.tsx
index 4ba8ff55..4ca10047 100644
--- a/src/components/interests/interest-picker.tsx
+++ b/src/components/interests/interest-picker.tsx
@@ -73,7 +73,7 @@ export function InterestPicker({
           className={cn('w-full justify-between', !value && 'text-muted-foreground')}
         >
           <span className="truncate">{selectedLabel}</span>
-          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" />
+          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" aria-hidden />
         </Button>
       </PopoverTrigger>
       <PopoverContent className="w-[320px] p-0" align="start">
diff --git a/src/components/interests/interest-reservation-tab.tsx b/src/components/interests/interest-reservation-tab.tsx
index 22757ae6..654f5df6 100644
--- a/src/components/interests/interest-reservation-tab.tsx
+++ b/src/components/interests/interest-reservation-tab.tsx
@@ -109,7 +109,7 @@ export function InterestReservationTab({
   return (
     <div className="space-y-5">
       {docsLoading ? (
-        <Skeleton className="h-44 w-full rounded-lg" />
+        <Skeleton className="h-44 w-full rounded-lg" aria-hidden />
       ) : activeDoc ? (
         <ActiveReservationCard
           doc={activeDoc}
@@ -148,7 +148,7 @@ export function InterestReservationTab({
                     className="text-xs text-primary hover:underline inline-flex items-center gap-1"
                   >
                     Open
-                    <ExternalLink className="size-3" />
+                    <ExternalLink className="size-3" aria-hidden />
                   </Link>
                 )}
               </li>
@@ -235,7 +235,7 @@ function ActiveReservationCard({
       <header className="flex items-start justify-between gap-3">
         <div className="min-w-0 flex-1 space-y-1">
           <div className="flex items-center gap-2 flex-wrap">
-            <FileSignature className="size-4 text-foreground" />
+            <FileSignature className="size-4 text-foreground" aria-hidden />
             <h2 className="truncate text-base font-semibold text-foreground">{doc.title}</h2>
             <StatusBadge status={doc.status} />
           </div>
@@ -264,7 +264,11 @@ function ActiveReservationCard({
               onClick={() => remindAllMutation.mutate()}
               className="gap-1.5 [&_svg]:size-3.5"
             >
-              {remindAllMutation.isPending ? <Loader2 className="animate-spin" /> : <RefreshCw />}
+              {remindAllMutation.isPending ? (
+                <Loader2 className="animate-spin" aria-hidden />
+              ) : (
+                <RefreshCw />
+              )}
               Remind all
             </Button>
           )}
@@ -277,7 +281,7 @@ function ActiveReservationCard({
         </h3>
         {signersLoading ? (
           <div className="flex items-center gap-2 text-sm text-muted-foreground">
-            <Loader2 className="size-3.5 animate-spin" /> Loading signers…
+            <Loader2 className="size-3.5 animate-spin" aria-hidden /> Loading signers…
           </div>
         ) : signers.length === 0 ? (
           <p className="text-sm text-muted-foreground italic">
@@ -290,7 +294,7 @@ function ActiveReservationCard({
 
       <footer className="mt-3 flex flex-wrap items-center justify-between gap-2 text-xs text-muted-foreground">
         <p className="flex items-center gap-1.5">
-          <AlertTriangle className="size-3 text-amber-600" />
+          <AlertTriangle className="size-3 text-amber-600" aria-hidden />
           Reminders are rate-limited (max once per 7 days per signer).
         </p>
         <div className="flex items-center gap-1">
@@ -341,7 +345,7 @@ function EmptyReservationState({
   return (
     <section className="rounded-xl border border-dashed bg-muted/20 p-8 text-center">
       <div className="mx-auto flex size-14 items-center justify-center rounded-full bg-background text-muted-foreground">
-        <FileSignature className="size-6" />
+        <FileSignature className="size-6" aria-hidden />
       </div>
       <h2 className="mt-4 text-base font-semibold text-foreground">
         No reservation in flight for this interest
@@ -352,11 +356,11 @@ function EmptyReservationState({
       </p>
       <div className="mt-5 flex flex-wrap items-center justify-center gap-2">
         <Button onClick={onUploadForSigning} size="sm" className="gap-1.5">
-          <FileSignature className="size-4" />
+          <FileSignature className="size-4" aria-hidden />
           Upload draft for signing
         </Button>
         <Button onClick={onUploadSigned} variant="outline" size="sm" className="gap-1.5">
-          <Upload className="size-4" />
+          <Upload className="size-4" aria-hidden />
           Upload paper-signed copy
         </Button>
       </div>
@@ -375,7 +379,7 @@ function StatusBadge({ status }: { status: DocumentRow['status'] }) {
         STATUS_TONES[status],
       )}
     >
-      {status === 'completed' && <CheckCircle2 className="mr-1 size-3" />}
+      {status === 'completed' && <CheckCircle2 className="mr-1 size-3" aria-hidden />}
       {STATUS_LABELS[status]}
     </Badge>
   );
diff --git a/src/components/interests/interest-stage-picker.tsx b/src/components/interests/interest-stage-picker.tsx
index 4c71b4ef..61dab782 100644
--- a/src/components/interests/interest-stage-picker.tsx
+++ b/src/components/interests/interest-stage-picker.tsx
@@ -115,7 +115,7 @@ export function InterestStagePicker({
                   : 'border-red-300 bg-red-50 text-red-900'
               }`}
             >
-              <AlertTriangle className="h-4 w-4 mt-0.5 shrink-0" />
+              <AlertTriangle className="h-4 w-4 mt-0.5 shrink-0" aria-hidden />
               {canOverride ? (
                 <span>
                   This is not a normal forward transition. Override is enabled — supply a reason
@@ -175,7 +175,7 @@ export function InterestStagePicker({
               !reasonValid
             }
           >
-            {mutation.isPending && <Loader2 className="mr-2 h-4 w-4 animate-spin" />}
+            {mutation.isPending && <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />}
             {overrideEffective ? 'Override stage' : 'Confirm'}
           </Button>
         </DialogFooter>
diff --git a/src/components/interests/interest-tabs.tsx b/src/components/interests/interest-tabs.tsx
index 468d5890..73b8be0b 100644
--- a/src/components/interests/interest-tabs.tsx
+++ b/src/components/interests/interest-tabs.tsx
@@ -340,7 +340,7 @@ function MilestoneSection({
           return (
             <li key={step.label} className="flex items-start gap-2 text-sm">
               {done ? (
-                <CheckCircle2 className="mt-0.5 size-4 shrink-0 text-emerald-600" />
+                <CheckCircle2 className="mt-0.5 size-4 shrink-0 text-emerald-600" aria-hidden />
               ) : (
                 <Circle
                   className={cn(
@@ -650,7 +650,7 @@ function OverviewTab({
           <div className="flex flex-wrap items-center gap-x-3 gap-y-1.5">
             <Button asChild size="sm" className="h-7 px-2.5 text-xs">
               <Link href={`/${portSlug}/invoices/new?interestId=${interestId}&kind=deposit`}>
-                <Plus className="size-3.5" />
+                <Plus className="size-3.5" aria-hidden />
                 Create deposit invoice
               </Link>
             </Button>
@@ -710,7 +710,7 @@ function OverviewTab({
             <span className="text-[10px] font-semibold uppercase tracking-wide">Past</span>
             {pastMilestones.map((m) => (
               <span key={m.key} className="inline-flex items-center gap-1.5">
-                <CheckCircle2 className="size-3 text-emerald-600" />
+                <CheckCircle2 className="size-3 text-emerald-600" aria-hidden />
                 <span className="font-medium text-foreground">{m.title}</span>
                 <span>·</span>
                 <span>{m.pastSummary}</span>
diff --git a/src/components/interests/interest-timeline.tsx b/src/components/interests/interest-timeline.tsx
index 17a4ddaf..c68466b4 100644
--- a/src/components/interests/interest-timeline.tsx
+++ b/src/components/interests/interest-timeline.tsx
@@ -46,17 +46,23 @@ function eventIcon(event: TimelineEvent) {
 
   if (type === 'outcome_set') {
     const outcome = (event.metadata as Record<string, unknown>).outcome as string | undefined;
-    if (outcome === 'won') return <Trophy className="h-4 w-4 text-emerald-600" />;
-    if (outcome && LOST_OUTCOMES.has(outcome)) return <XCircle className="h-4 w-4 text-rose-600" />;
-    return <XCircle className="h-4 w-4 text-rose-600" />;
+    if (outcome === 'won') return <Trophy className="h-4 w-4 text-emerald-600" aria-hidden />;
+    if (outcome && LOST_OUTCOMES.has(outcome))
+      return <XCircle className="h-4 w-4 text-rose-600" aria-hidden />;
+    return <XCircle className="h-4 w-4 text-rose-600" aria-hidden />;
   }
-  if (type === 'outcome_cleared') return <RefreshCcw className="h-4 w-4 text-blue-500" />;
-  if (event.type === 'document_event') return <FileText className="h-4 w-4 text-sky-600" />;
-  if (event.action === 'create') return <PlusCircle className="h-4 w-4 text-green-500" />;
-  if (event.action === 'archive') return <Archive className="h-4 w-4 text-orange-500" />;
-  if (event.action === 'restore') return <RotateCcw className="h-4 w-4 text-blue-500" />;
-  if (type === 'stage_change') return <Clock className="h-4 w-4 text-purple-500" />;
-  return <Pencil className="h-4 w-4 text-muted-foreground" />;
+  if (type === 'outcome_cleared')
+    return <RefreshCcw className="h-4 w-4 text-blue-500" aria-hidden />;
+  if (event.type === 'document_event')
+    return <FileText className="h-4 w-4 text-sky-600" aria-hidden />;
+  if (event.action === 'create')
+    return <PlusCircle className="h-4 w-4 text-green-500" aria-hidden />;
+  if (event.action === 'archive')
+    return <Archive className="h-4 w-4 text-orange-500" aria-hidden />;
+  if (event.action === 'restore')
+    return <RotateCcw className="h-4 w-4 text-blue-500" aria-hidden />;
+  if (type === 'stage_change') return <Clock className="h-4 w-4 text-purple-500" aria-hidden />;
+  return <Pencil className="h-4 w-4 text-muted-foreground" aria-hidden />;
 }
 
 function actorLabel(event: TimelineEvent): string | null {
@@ -121,7 +127,7 @@ export function InterestTimeline({ interestId }: InterestTimelineProps) {
                 {event.description}
                 {isAuto ? (
                   <span className="ml-2 inline-flex items-center gap-1 rounded-full bg-muted px-1.5 py-0.5 align-middle text-[10px] font-medium uppercase tracking-wide text-muted-foreground">
-                    <Bot className="h-3 w-3" />
+                    <Bot className="h-3 w-3" aria-hidden />
                     Auto
                   </span>
                 ) : null}
diff --git a/src/components/interests/linked-berths-list.tsx b/src/components/interests/linked-berths-list.tsx
index 6776c9bf..fad6fade 100644
--- a/src/components/interests/linked-berths-list.tsx
+++ b/src/components/interests/linked-berths-list.tsx
@@ -220,7 +220,7 @@ function BypassDialog({ row, open, onOpenChange, onSubmit, isPending }: BypassDi
             onClick={() => onSubmit(reason.trim().length > 0 ? reason.trim() : null)}
             disabled={isPending || reason.trim().length === 0}
           >
-            {isPending ? <Loader2 className="mr-1.5 size-3.5 animate-spin" /> : null}
+            {isPending ? <Loader2 className="mr-1.5 size-3.5 animate-spin" aria-hidden /> : null}
             Save bypass
           </Button>
         </DialogFooter>
@@ -266,7 +266,7 @@ function LinkedBerthRowItem({ row, portSlug, eoiStatus, onUpdate, onRemove, isPe
             <StatusPill status={statusToPill(row.status)}>{formatStatus(row.status)}</StatusPill>
             {row.isPrimary ? (
               <span className="inline-flex items-center gap-1 rounded-md border border-brand-200 bg-brand-50 px-2 py-0.5 text-xs font-medium text-brand-800">
-                <Star className="size-3" />
+                <Star className="size-3" aria-hidden />
                 Primary
               </span>
             ) : null}
@@ -287,7 +287,7 @@ function LinkedBerthRowItem({ row, portSlug, eoiStatus, onUpdate, onRemove, isPe
               onClick={() => onUpdate(row.berthId, { isPrimary: true })}
               disabled={isPending}
             >
-              <Star className="mr-1.5 size-3.5" />
+              <Star className="mr-1.5 size-3.5" aria-hidden />
               Set as primary
             </Button>
           ) : null}
@@ -300,7 +300,7 @@ function LinkedBerthRowItem({ row, portSlug, eoiStatus, onUpdate, onRemove, isPe
             className="text-destructive hover:text-destructive"
             aria-label={`Remove berth ${row.mooringNumber ?? row.berthId}`}
           >
-            <Trash2 className="size-3.5" />
+            <Trash2 className="size-3.5" aria-hidden />
           </Button>
         </div>
       </div>
@@ -334,7 +334,7 @@ function LinkedBerthRowItem({ row, portSlug, eoiStatus, onUpdate, onRemove, isPe
                     className="inline-flex h-5 w-5 items-center justify-center rounded-full text-muted-foreground hover:bg-muted/60 hover:text-foreground"
                     aria-label="What does Specifically pitching do?"
                   >
-                    <HelpCircle className="h-3.5 w-3.5" />
+                    <HelpCircle className="h-3.5 w-3.5" aria-hidden />
                   </button>
                 </TooltipTrigger>
                 <TooltipContent side="top" className="max-w-xs text-[11px] leading-snug">
@@ -369,7 +369,7 @@ function LinkedBerthRowItem({ row, portSlug, eoiStatus, onUpdate, onRemove, isPe
                     className="inline-flex h-5 w-5 items-center justify-center rounded-full text-muted-foreground hover:bg-muted/60 hover:text-foreground"
                     aria-label="What does Mark in EOI bundle do?"
                   >
-                    <HelpCircle className="h-3.5 w-3.5" />
+                    <HelpCircle className="h-3.5 w-3.5" aria-hidden />
                   </button>
                 </TooltipTrigger>
                 <TooltipContent side="top" className="max-w-xs text-[11px] leading-snug">
@@ -484,7 +484,7 @@ export function LinkedBerthsList({ interestId }: LinkedBerthsListProps) {
     <Card>
       <CardHeader>
         <CardTitle className="flex items-center gap-2 text-base">
-          <Anchor className="size-4 text-brand-600" />
+          <Anchor className="size-4 text-brand-600" aria-hidden />
           Linked berths{rows.length > 0 ? ` (${rows.length})` : ''}
         </CardTitle>
       </CardHeader>
diff --git a/src/components/interests/recommendation-list.tsx b/src/components/interests/recommendation-list.tsx
index 9a7d5971..c5af8a85 100644
--- a/src/components/interests/recommendation-list.tsx
+++ b/src/components/interests/recommendation-list.tsx
@@ -58,9 +58,9 @@ export function RecommendationList({ interestId }: RecommendationListProps) {
           disabled={generateMutation.isPending}
         >
           {generateMutation.isPending ? (
-            <Loader2 className="mr-1.5 h-4 w-4 animate-spin" />
+            <Loader2 className="mr-1.5 h-4 w-4 animate-spin" aria-hidden />
           ) : (
-            <Sparkles className="mr-1.5 h-4 w-4" />
+            <Sparkles className="mr-1.5 h-4 w-4" aria-hidden />
           )}
           Generate Recommendations
         </Button>
diff --git a/src/components/interests/send-from-interest-button.tsx b/src/components/interests/send-from-interest-button.tsx
index b3cc0c20..3ef77d3d 100644
--- a/src/components/interests/send-from-interest-button.tsx
+++ b/src/components/interests/send-from-interest-button.tsx
@@ -28,7 +28,7 @@ export function SendFromInterestButton({
   return (
     <>
       <Button variant="outline" size="sm" onClick={() => setOpen(true)}>
-        <Send className="mr-2 h-4 w-4" /> Send documents
+        <Send className="mr-2 h-4 w-4" aria-hidden /> Send documents
       </Button>
       <SendDocumentsDialog
         open={open}
diff --git a/src/components/interests/stage-legend.tsx b/src/components/interests/stage-legend.tsx
index 7c83d675..781138f8 100644
--- a/src/components/interests/stage-legend.tsx
+++ b/src/components/interests/stage-legend.tsx
@@ -22,7 +22,7 @@ export function StageLegend() {
           className="h-8 gap-1.5 text-muted-foreground"
           aria-label="What do the colors mean?"
         >
-          <Info className="h-3.5 w-3.5" />
+          <Info className="h-3.5 w-3.5" aria-hidden />
           <span className="hidden sm:inline">Legend</span>
         </Button>
       </PopoverTrigger>
diff --git a/src/components/layout/breadcrumbs.tsx b/src/components/layout/breadcrumbs.tsx
index c1dc7478..668f3163 100644
--- a/src/components/layout/breadcrumbs.tsx
+++ b/src/components/layout/breadcrumbs.tsx
@@ -123,7 +123,7 @@ export function Breadcrumbs() {
             </BreadcrumbItem>
             {!crumb.isLast && (
               <BreadcrumbSeparator className="text-muted-foreground/40">
-                <ChevronRight className="w-3 h-3" />
+                <ChevronRight className="w-3 h-3" aria-hidden />
               </BreadcrumbSeparator>
             )}
           </Fragment>
diff --git a/src/components/layout/inbox.tsx b/src/components/layout/inbox.tsx
index f6c5afac..fa275914 100644
--- a/src/components/layout/inbox.tsx
+++ b/src/components/layout/inbox.tsx
@@ -123,7 +123,7 @@ export function Inbox() {
           aria-label={`Inbox${combined > 0 ? ` (${combined} unread)` : ''}`}
           data-testid="inbox-bell"
         >
-          <InboxIcon className="h-5 w-5" />
+          <InboxIcon className="h-5 w-5" aria-hidden />
           {combined > 0 ? (
             <span
               key={combined}
diff --git a/src/components/layout/mobile/mobile-topbar.tsx b/src/components/layout/mobile/mobile-topbar.tsx
index 5fe92120..ffcc96f6 100644
--- a/src/components/layout/mobile/mobile-topbar.tsx
+++ b/src/components/layout/mobile/mobile-topbar.tsx
@@ -54,7 +54,7 @@ export function MobileTopbar() {
             'text-white/95 active:bg-white/10 transition-colors',
           )}
         >
-          <ChevronLeft className="size-[22px] stroke-[2.25]" />
+          <ChevronLeft className="size-[22px] stroke-[2.25]" aria-hidden />
         </button>
       ) : (
         <div
diff --git a/src/components/layout/sidebar.tsx b/src/components/layout/sidebar.tsx
index 7d08a3b6..b1d672c9 100644
--- a/src/components/layout/sidebar.tsx
+++ b/src/components/layout/sidebar.tsx
@@ -304,9 +304,9 @@ function SidebarContent({
               )}
             >
               {collapsed ? (
-                <ChevronRight className="w-4 h-4" />
+                <ChevronRight className="w-4 h-4" aria-hidden />
               ) : (
-                <ChevronLeft className="w-4 h-4" />
+                <ChevronLeft className="w-4 h-4" aria-hidden />
               )}
             </button>
           )}
@@ -334,9 +334,9 @@ function SidebarContent({
                           className="text-slate-400 hover:text-slate-700 transition-colors"
                         >
                           {adminExpanded ? (
-                            <ChevronUp className="w-3 h-3" />
+                            <ChevronUp className="w-3 h-3" aria-hidden />
                           ) : (
-                            <ChevronDown className="w-3 h-3" />
+                            <ChevronDown className="w-3 h-3" aria-hidden />
                           )}
                         </button>
                       )}
@@ -355,7 +355,7 @@ function SidebarContent({
                       ))}
                     </ul>
                   )}
-                  <Separator className="mt-3 bg-slate-200" />
+                  <Separator className="mt-3 bg-slate-200" aria-hidden />
                 </div>
               );
             })}
diff --git a/src/components/layout/topbar.tsx b/src/components/layout/topbar.tsx
index 63687517..9ca5066a 100644
--- a/src/components/layout/topbar.tsx
+++ b/src/components/layout/topbar.tsx
@@ -68,7 +68,7 @@ export function Topbar({ ports, user }: TopbarProps) {
               'text-muted-foreground hover:text-foreground hover:bg-accent transition-colors',
             )}
           >
-            <ChevronLeft className="h-4 w-4" />
+            <ChevronLeft className="h-4 w-4" aria-hidden />
           </button>
         )}
         <Breadcrumbs />
@@ -92,7 +92,7 @@ export function Topbar({ ports, user }: TopbarProps) {
               size="sm"
               className="bg-gradient-brand hover:opacity-90 text-white gap-1.5 shadow-sm transition-all duration-base ease-smooth hover:scale-[1.02]"
             >
-              <Plus className="w-4 h-4" />
+              <Plus className="w-4 h-4" aria-hidden />
               <span className="hidden sm:inline">New</span>
             </Button>
           </DropdownMenuTrigger>
diff --git a/src/components/layout/user-menu.tsx b/src/components/layout/user-menu.tsx
index 39130c9e..bddd1d2a 100644
--- a/src/components/layout/user-menu.tsx
+++ b/src/components/layout/user-menu.tsx
@@ -91,7 +91,7 @@ export function UserMenu({ trigger, align = 'end', user, ports }: UserMenuProps)
                 regardless of how many ports the operator has access to. */}
             <DropdownMenuSub>
               <DropdownMenuSubTrigger>
-                <Building2 className="w-4 h-4 mr-2" />
+                <Building2 className="w-4 h-4 mr-2" aria-hidden />
                 Switch port
               </DropdownMenuSubTrigger>
               <DropdownMenuSubContent className="w-56">
@@ -116,31 +116,31 @@ export function UserMenu({ trigger, align = 'end', user, ports }: UserMenuProps)
 
         {/* eslint-disable-next-line @typescript-eslint/no-explicit-any */}
         <DropdownMenuItem onClick={() => router.push(`${base}/settings/profile` as any)}>
-          <User className="w-4 h-4 mr-2" />
+          <User className="w-4 h-4 mr-2" aria-hidden />
           Profile
         </DropdownMenuItem>
         {/* eslint-disable-next-line @typescript-eslint/no-explicit-any */}
         <DropdownMenuItem onClick={() => router.push(`${base}/settings` as any)}>
-          <Settings className="w-4 h-4 mr-2" />
+          <Settings className="w-4 h-4 mr-2" aria-hidden />
           Settings
         </DropdownMenuItem>
         <DropdownMenuItem
           // eslint-disable-next-line @typescript-eslint/no-explicit-any
           onClick={() => router.push(`${base}/settings#notifications` as any)}
         >
-          <Bell className="w-4 h-4 mr-2" />
+          <Bell className="w-4 h-4 mr-2" aria-hidden />
           Notification preferences
         </DropdownMenuItem>
         <DropdownMenuSeparator />
         <DropdownMenuItem onClick={handleToggleDarkMode}>
           {darkMode ? (
             <>
-              <Sun className="w-4 h-4 mr-2" />
+              <Sun className="w-4 h-4 mr-2" aria-hidden />
               Light Mode
             </>
           ) : (
             <>
-              <Moon className="w-4 h-4 mr-2" />
+              <Moon className="w-4 h-4 mr-2" aria-hidden />
               Dark Mode
             </>
           )}
@@ -150,7 +150,7 @@ export function UserMenu({ trigger, align = 'end', user, ports }: UserMenuProps)
           className="text-destructive focus:text-destructive"
           onClick={() => router.push('/api/auth/sign-out')}
         >
-          <LogOut className="w-4 h-4 mr-2" />
+          <LogOut className="w-4 h-4 mr-2" aria-hidden />
           Sign Out
         </DropdownMenuItem>
       </DropdownMenuContent>
diff --git a/src/components/reminders/reminder-card.tsx b/src/components/reminders/reminder-card.tsx
index 496f45e9..6f121f70 100644
--- a/src/components/reminders/reminder-card.tsx
+++ b/src/components/reminders/reminder-card.tsx
@@ -136,7 +136,7 @@ export function ReminderCard({
                 onClick={(e) => e.stopPropagation()}
                 aria-label={`Actions for reminder: ${reminder.title}`}
               >
-                <MoreHorizontal className="h-4 w-4" />
+                <MoreHorizontal className="h-4 w-4" aria-hidden />
               </Button>
             </DropdownMenuTrigger>
             <DropdownMenuContent align="end">
@@ -146,7 +146,7 @@ export function ReminderCard({
                   onComplete(reminder.id);
                 }}
               >
-                <CheckCircle2 className="mr-2 h-3.5 w-3.5 text-green-600" />
+                <CheckCircle2 className="mr-2 h-3.5 w-3.5 text-green-600" aria-hidden />
                 Complete
               </DropdownMenuItem>
               <DropdownMenuItem
@@ -155,7 +155,7 @@ export function ReminderCard({
                   onSnooze(reminder.id);
                 }}
               >
-                <Clock className="mr-2 h-3.5 w-3.5" />
+                <Clock className="mr-2 h-3.5 w-3.5" aria-hidden />
                 Snooze
               </DropdownMenuItem>
               <DropdownMenuItem
@@ -164,7 +164,7 @@ export function ReminderCard({
                   onEdit(reminder);
                 }}
               >
-                <Bell className="mr-2 h-3.5 w-3.5" />
+                <Bell className="mr-2 h-3.5 w-3.5" aria-hidden />
                 Edit
               </DropdownMenuItem>
               <DropdownMenuItem
@@ -174,7 +174,7 @@ export function ReminderCard({
                   onDismiss(reminder.id);
                 }}
               >
-                <XCircle className="mr-2 h-3.5 w-3.5" />
+                <XCircle className="mr-2 h-3.5 w-3.5" aria-hidden />
                 Dismiss
               </DropdownMenuItem>
             </DropdownMenuContent>
@@ -183,7 +183,7 @@ export function ReminderCard({
       }
     >
       <div className="flex items-start gap-3">
-        <ListCardAvatar icon={<Bell className="h-5 w-5" />} />
+        <ListCardAvatar icon={<Bell className="h-5 w-5" aria-hidden />} />
         <div className="min-w-0 flex-1">
           {/* Title row + spacer for actions button */}
           <div className="flex items-start justify-between gap-2">
diff --git a/src/components/reminders/reminder-list.tsx b/src/components/reminders/reminder-list.tsx
index a6624a08..e87e8aba 100644
--- a/src/components/reminders/reminder-list.tsx
+++ b/src/components/reminders/reminder-list.tsx
@@ -153,7 +153,7 @@ export function ReminderList({ embedded = false }: ReminderListProps = {}) {
                   Auto
                 </Badge>
               )}
-              {overdue && <AlertTriangle className="h-3.5 w-3.5 text-destructive" />}
+              {overdue && <AlertTriangle className="h-3.5 w-3.5 text-destructive" aria-hidden />}
             </div>
             {row.original.client && (
               <span className="text-xs text-muted-foreground">
@@ -193,7 +193,7 @@ export function ReminderList({ embedded = false }: ReminderListProps = {}) {
         const Icon = config.icon;
         return (
           <div className="flex items-center gap-1.5 text-sm">
-            <Icon className="h-3.5 w-3.5" />
+            <Icon className="h-3.5 w-3.5" aria-hidden />
             {config.label}
           </div>
         );
@@ -218,7 +218,7 @@ export function ReminderList({ embedded = false }: ReminderListProps = {}) {
                     className="text-green-600 hover:text-green-700"
                     onClick={() => handleComplete(row.original.id)}
                   >
-                    <CheckCircle2 className="h-4 w-4" />
+                    <CheckCircle2 className="h-4 w-4" aria-hidden />
                   </Button>
                 </TooltipTrigger>
                 <TooltipContent>Mark complete</TooltipContent>
@@ -231,7 +231,7 @@ export function ReminderList({ embedded = false }: ReminderListProps = {}) {
                     aria-label="Snooze"
                     onClick={() => setSnoozingId(row.original.id)}
                   >
-                    <Clock className="h-4 w-4" />
+                    <Clock className="h-4 w-4" aria-hidden />
                   </Button>
                 </TooltipTrigger>
                 <TooltipContent>Snooze</TooltipContent>
@@ -248,7 +248,7 @@ export function ReminderList({ embedded = false }: ReminderListProps = {}) {
                       setFormOpen(true);
                     }}
                   >
-                    <Pencil className="h-4 w-4" />
+                    <Pencil className="h-4 w-4" aria-hidden />
                   </Button>
                 </TooltipTrigger>
                 <TooltipContent>Edit</TooltipContent>
@@ -262,7 +262,7 @@ export function ReminderList({ embedded = false }: ReminderListProps = {}) {
                     className="text-muted-foreground hover:text-foreground"
                     onClick={() => handleDismiss(row.original.id)}
                   >
-                    <XCircle className="h-4 w-4" />
+                    <XCircle className="h-4 w-4" aria-hidden />
                   </Button>
                 </TooltipTrigger>
                 <TooltipContent>Dismiss</TooltipContent>
@@ -289,7 +289,7 @@ export function ReminderList({ embedded = false }: ReminderListProps = {}) {
                 setFormOpen(true);
               }}
             >
-              <Plus className="mr-1.5 h-4 w-4" />
+              <Plus className="mr-1.5 h-4 w-4" aria-hidden />
               New Reminder
             </Button>
           }
@@ -303,7 +303,7 @@ export function ReminderList({ embedded = false }: ReminderListProps = {}) {
               setFormOpen(true);
             }}
           >
-            <Plus className="mr-1.5 h-4 w-4" />
+            <Plus className="mr-1.5 h-4 w-4" aria-hidden />
             New Reminder
           </Button>
         </div>
@@ -371,7 +371,7 @@ export function ReminderList({ embedded = false }: ReminderListProps = {}) {
         )}
         emptyState={
           <div className="text-center py-8">
-            <Bell className="mx-auto h-8 w-8 text-muted-foreground mb-2" />
+            <Bell className="mx-auto h-8 w-8 text-muted-foreground mb-2" aria-hidden />
             <p className="text-muted-foreground">No reminders.</p>
             <Button
               variant="link"
diff --git a/src/components/reservations/berth-reserve-dialog.tsx b/src/components/reservations/berth-reserve-dialog.tsx
index b6cbe915..1fa0b7f0 100644
--- a/src/components/reservations/berth-reserve-dialog.tsx
+++ b/src/components/reservations/berth-reserve-dialog.tsx
@@ -227,7 +227,9 @@ export function BerthReserveDialog({ open, onOpenChange, berthId }: BerthReserve
                 createMutation.mutate(data);
               })}
             >
-              {createMutation.isPending && <Loader2 className="mr-2 h-4 w-4 animate-spin" />}
+              {createMutation.isPending && (
+                <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />
+              )}
               Create reservation
             </Button>
             <Button
@@ -239,7 +241,7 @@ export function BerthReserveDialog({ open, onOpenChange, berthId }: BerthReserve
               })}
             >
               {createAndActivateMutation.isPending && (
-                <Loader2 className="mr-2 h-4 w-4 animate-spin" />
+                <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />
               )}
               Create and activate
             </Button>
diff --git a/src/components/reservations/reservation-detail.tsx b/src/components/reservations/reservation-detail.tsx
index 32e2b464..06bb0f3c 100644
--- a/src/components/reservations/reservation-detail.tsx
+++ b/src/components/reservations/reservation-detail.tsx
@@ -165,7 +165,7 @@ export function ReservationDetail({ reservationId, portSlug }: ReservationDetail
         actions={
           <Button asChild variant="outline">
             <Link href={`/${portSlug}/berths`}>
-              <ArrowLeft className="mr-1.5 h-4 w-4" /> Back
+              <ArrowLeft className="mr-1.5 h-4 w-4" aria-hidden /> Back
             </Link>
           </Button>
         }
@@ -195,13 +195,13 @@ export function ReservationDetail({ reservationId, portSlug }: ReservationDetail
             {completedAgreement.signedFileId ? (
               <Button asChild size="sm" variant="outline">
                 <Link href={`/api/v1/files/${completedAgreement.signedFileId}/download`}>
-                  <Download className="mr-1.5 h-4 w-4" /> Download signed PDF
+                  <Download className="mr-1.5 h-4 w-4" aria-hidden /> Download signed PDF
                 </Link>
               </Button>
             ) : null}
             <Button asChild size="sm" variant="outline">
               <Link href={`/${portSlug}/documents/${completedAgreement.id}`}>
-                <Mail className="mr-1.5 h-4 w-4" /> View document
+                <Mail className="mr-1.5 h-4 w-4" aria-hidden /> View document
               </Link>
             </Button>
           </div>
@@ -233,7 +233,7 @@ export function ReservationDetail({ reservationId, portSlug }: ReservationDetail
           <div className="flex flex-wrap items-center gap-2">
             <Button asChild size="sm">
               <Link href={`/${portSlug}/documents/${activeAgreement.id}`}>
-                <FileSignature className="mr-1.5 h-4 w-4" /> View document
+                <FileSignature className="mr-1.5 h-4 w-4" aria-hidden /> View document
               </Link>
             </Button>
             <Button
@@ -250,7 +250,7 @@ export function ReservationDetail({ reservationId, portSlug }: ReservationDetail
                 }
               }}
             >
-              <Bell className="mr-1.5 h-4 w-4" /> Remind signers
+              <Bell className="mr-1.5 h-4 w-4" aria-hidden /> Remind signers
             </Button>
           </div>
         </div>
@@ -259,7 +259,7 @@ export function ReservationDetail({ reservationId, portSlug }: ReservationDetail
 
     return (
       <EmptyState
-        icon={<FileSignature className="h-7 w-7" />}
+        icon={<FileSignature className="h-7 w-7" aria-hidden />}
         title="No reservation agreement yet"
         body="Generate an agreement for the parties to sign before activation."
         actions={
@@ -295,13 +295,13 @@ export function ReservationDetail({ reservationId, portSlug }: ReservationDetail
           <div className="flex items-center gap-2">
             {res.status === 'active' && (
               <Button variant="outline" size="sm" onClick={() => setEndDialogOpen(true)}>
-                <StopCircle className="mr-1.5 h-4 w-4" />
+                <StopCircle className="mr-1.5 h-4 w-4" aria-hidden />
                 End reservation
               </Button>
             )}
             <Button asChild variant="outline">
               <Link href={`/${portSlug}/berths`}>
-                <ArrowLeft className="mr-1.5 h-4 w-4" /> Back to berths
+                <ArrowLeft className="mr-1.5 h-4 w-4" aria-hidden /> Back to berths
               </Link>
             </Button>
           </div>
diff --git a/src/components/residential/residential-client-detail-header.tsx b/src/components/residential/residential-client-detail-header.tsx
index 35ae990b..24fa4b39 100644
--- a/src/components/residential/residential-client-detail-header.tsx
+++ b/src/components/residential/residential-client-detail-header.tsx
@@ -33,7 +33,7 @@ export function ResidentialClientDetailHeader({ client, onSaveName }: Props) {
     <DetailHeaderStrip>
       <div className="space-y-1">
         <div className="text-xs font-semibold uppercase tracking-wide text-muted-foreground inline-flex items-center gap-1.5">
-          <Home className="h-3 w-3" /> Residential client
+          <Home className="h-3 w-3" aria-hidden /> Residential client
         </div>
         <h1 className="truncate text-2xl font-bold tracking-tight text-foreground">
           <InlineEditableField value={client.fullName} onSave={onSaveName} />
diff --git a/src/components/residential/residential-client-tabs.tsx b/src/components/residential/residential-client-tabs.tsx
index 21c3f021..8ba5fc72 100644
--- a/src/components/residential/residential-client-tabs.tsx
+++ b/src/components/residential/residential-client-tabs.tsx
@@ -263,7 +263,7 @@ function InterestsTab({
       <div className="flex items-center justify-between mb-4">
         <h2 className="text-lg font-semibold">Interests</h2>
         <Button size="sm" onClick={() => setOpen(true)}>
-          <Plus className="h-3.5 w-3.5 mr-1.5" />
+          <Plus className="h-3.5 w-3.5 mr-1.5" aria-hidden />
           New interest
         </Button>
       </div>
diff --git a/src/components/residential/residential-clients-list.tsx b/src/components/residential/residential-clients-list.tsx
index 81abd59e..122df1e1 100644
--- a/src/components/residential/residential-clients-list.tsx
+++ b/src/components/residential/residential-clients-list.tsx
@@ -72,7 +72,7 @@ export function ResidentialClientsList() {
         description="Inquiries and clients for the residential side"
         actions={
           <Button size="sm" onClick={() => setCreateOpen(true)}>
-            <Plus className="mr-1.5 h-3.5 w-3.5" />
+            <Plus className="mr-1.5 h-3.5 w-3.5" aria-hidden />
             New
           </Button>
         }
diff --git a/src/components/shared/addresses-editor.tsx b/src/components/shared/addresses-editor.tsx
index 3f4a41d0..f51174ae 100644
--- a/src/components/shared/addresses-editor.tsx
+++ b/src/components/shared/addresses-editor.tsx
@@ -105,7 +105,7 @@ export function AddressesEditor({ endpoint, invalidateKey, addresses }: Addresse
           className="w-full justify-center"
           data-testid="add-address-button"
         >
-          <Plus className="h-3.5 w-3.5 mr-1.5" />
+          <Plus className="h-3.5 w-3.5 mr-1.5" aria-hidden />
           Add address
         </Button>
       )}
@@ -135,7 +135,7 @@ function AddressCard({
   return (
     <div className="group rounded-lg border bg-muted/30 p-3 text-sm space-y-2">
       <div className="flex items-center gap-2">
-        <MapPin className="h-3.5 w-3.5 text-muted-foreground shrink-0" />
+        <MapPin className="h-3.5 w-3.5 text-muted-foreground shrink-0" aria-hidden />
         <div className="flex-1 min-w-0">
           <InlineEditableField
             value={address.label}
@@ -168,7 +168,7 @@ function AddressCard({
           className="p-1 rounded text-muted-foreground/50 hover:text-destructive hover:bg-background/60 opacity-0 group-hover:opacity-100 transition-all"
           data-testid="address-remove-button"
         >
-          <Trash2 className="h-3.5 w-3.5" />
+          <Trash2 className="h-3.5 w-3.5" aria-hidden />
         </button>
       </div>
 
@@ -441,7 +441,7 @@ function NewAddressForm({
             Cancel
           </Button>
           <Button type="button" size="sm" onClick={submit} disabled={saving}>
-            {saving ? <Loader2 className="h-3.5 w-3.5 animate-spin" /> : 'Save'}
+            {saving ? <Loader2 className="h-3.5 w-3.5 animate-spin" aria-hidden /> : 'Save'}
           </Button>
         </div>
       </div>
diff --git a/src/components/shared/archive-confirm-dialog.tsx b/src/components/shared/archive-confirm-dialog.tsx
index 785388ee..0d8d262c 100644
--- a/src/components/shared/archive-confirm-dialog.tsx
+++ b/src/components/shared/archive-confirm-dialog.tsx
@@ -58,7 +58,7 @@ export function ArchiveConfirmDialog({
               isArchived ? '' : 'bg-destructive text-destructive-foreground hover:bg-destructive/90'
             }
           >
-            {isLoading && <Loader2 className="mr-2 h-4 w-4 animate-spin" />}
+            {isLoading && <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />}
             {action}
           </AlertDialogAction>
         </AlertDialogFooter>
diff --git a/src/components/shared/berth-picker.tsx b/src/components/shared/berth-picker.tsx
index ed2ab963..c25f3e41 100644
--- a/src/components/shared/berth-picker.tsx
+++ b/src/components/shared/berth-picker.tsx
@@ -136,7 +136,7 @@ export function BerthPicker({
           className={cn('w-full justify-between', !value && 'text-muted-foreground')}
         >
           <span className="truncate">{selectedLabel}</span>
-          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" />
+          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" aria-hidden />
         </Button>
       </PopoverTrigger>
       <PopoverContent className="w-[320px] p-0" align="start">
diff --git a/src/components/shared/client-picker.tsx b/src/components/shared/client-picker.tsx
index 3e6d6e7f..252ff51b 100644
--- a/src/components/shared/client-picker.tsx
+++ b/src/components/shared/client-picker.tsx
@@ -70,7 +70,7 @@ export function ClientPicker({
           className={cn('w-full justify-between', !value && 'text-muted-foreground')}
         >
           <span className="truncate">{selectedLabel}</span>
-          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" />
+          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" aria-hidden />
         </Button>
       </PopoverTrigger>
       <PopoverContent className="w-[320px] p-0" align="start">
diff --git a/src/components/shared/column-picker.tsx b/src/components/shared/column-picker.tsx
index cc4f4673..be02ed43 100644
--- a/src/components/shared/column-picker.tsx
+++ b/src/components/shared/column-picker.tsx
@@ -72,7 +72,7 @@ export function ColumnPicker({
             cards (no columns to pick). The desktop table view shows
             this trigger from the `sm:` breakpoint up. */}
         <Button variant="outline" size="sm" className="hidden sm:inline-flex gap-1.5 h-8">
-          <Columns3 className="h-3.5 w-3.5" />
+          <Columns3 className="h-3.5 w-3.5" aria-hidden />
           <span>Columns</span>
         </Button>
       </DropdownMenuTrigger>
@@ -102,7 +102,7 @@ export function ColumnPicker({
                   isVisible ? 'bg-primary border-primary text-primary-foreground' : 'border-border'
                 }`}
               >
-                {isVisible && <Check className="h-3 w-3" />}
+                {isVisible && <Check className="h-3 w-3" aria-hidden />}
               </span>
               <span className="flex-1">{col.label}</span>
               {col.alwaysVisible && (
@@ -125,7 +125,7 @@ export function ColumnPicker({
           <>
             <DropdownMenuSeparator />
             <DropdownMenuItem onClick={onSaveView}>
-              <Bookmark className="mr-2 h-3.5 w-3.5" />
+              <Bookmark className="mr-2 h-3.5 w-3.5" aria-hidden />
               Save current view
             </DropdownMenuItem>
           </>
diff --git a/src/components/shared/country-combobox.tsx b/src/components/shared/country-combobox.tsx
index 01135d7e..90a09487 100644
--- a/src/components/shared/country-combobox.tsx
+++ b/src/components/shared/country-combobox.tsx
@@ -119,7 +119,7 @@ export function CountryCombobox({
           ) : (
             <span className="truncate">{placeholder}</span>
           )}
-          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" />
+          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" aria-hidden />
         </Button>
       </PopoverTrigger>
       <PopoverContent className="w-(--radix-popper-anchor-width) min-w-[280px] p-0" align="start">
diff --git a/src/components/shared/custom-fields-section.tsx b/src/components/shared/custom-fields-section.tsx
index 1ab8b7e0..6d4c1e8f 100644
--- a/src/components/shared/custom-fields-section.tsx
+++ b/src/components/shared/custom-fields-section.tsx
@@ -108,9 +108,9 @@ export function CustomFieldsSection({ entityType, entityId }: CustomFieldsSectio
         <div className="flex items-center justify-between">
           <CardTitle className="text-base">Custom Fields</CardTitle>
           {collapsed ? (
-            <ChevronRight className="h-4 w-4 text-muted-foreground" />
+            <ChevronRight className="h-4 w-4 text-muted-foreground" aria-hidden />
           ) : (
-            <ChevronDown className="h-4 w-4 text-muted-foreground" />
+            <ChevronDown className="h-4 w-4 text-muted-foreground" aria-hidden />
           )}
         </div>
       </CardHeader>
diff --git a/src/components/shared/data-table.tsx b/src/components/shared/data-table.tsx
index b6f86402..ad5431fa 100644
--- a/src/components/shared/data-table.tsx
+++ b/src/components/shared/data-table.tsx
@@ -206,11 +206,11 @@ export function DataTable<TData>({
   }
 
   function getSortIcon(columnId: string) {
-    if (sort?.field !== columnId) return <ArrowUpDown className="ml-1 h-3.5 w-3.5" />;
+    if (sort?.field !== columnId) return <ArrowUpDown className="ml-1 h-3.5 w-3.5" aria-hidden />;
     return sort.direction === 'asc' ? (
-      <ArrowUp className="ml-1 h-3.5 w-3.5" />
+      <ArrowUp className="ml-1 h-3.5 w-3.5" aria-hidden />
     ) : (
-      <ArrowDown className="ml-1 h-3.5 w-3.5" />
+      <ArrowDown className="ml-1 h-3.5 w-3.5" aria-hidden />
     );
   }
 
@@ -279,7 +279,7 @@ export function DataTable<TData>({
               <TableRow>
                 <TableCell colSpan={allColumns.length} className="h-40 text-center">
                   <div className="flex items-center justify-center gap-2">
-                    <Loader2 className="h-5 w-5 animate-spin text-muted-foreground" />
+                    <Loader2 className="h-5 w-5 animate-spin text-muted-foreground" aria-hidden />
                     <span className="text-muted-foreground">Loading...</span>
                   </div>
                 </TableCell>
@@ -350,7 +350,7 @@ export function DataTable<TData>({
         <ul className="lg:hidden flex flex-col gap-2">
           {isLoading ? (
             <li className="rounded-md border bg-card p-6 text-center">
-              <Loader2 className="mx-auto h-5 w-5 animate-spin text-muted-foreground" />
+              <Loader2 className="mx-auto h-5 w-5 animate-spin text-muted-foreground" aria-hidden />
             </li>
           ) : rows.length === 0 ? (
             <li className="rounded-md border bg-card p-6 text-center text-sm text-muted-foreground">
diff --git a/src/components/shared/detail-layout.tsx b/src/components/shared/detail-layout.tsx
index 6eafc136..1d00d855 100644
--- a/src/components/shared/detail-layout.tsx
+++ b/src/components/shared/detail-layout.tsx
@@ -32,7 +32,7 @@ export function DetailLayout({ header, tabs, defaultTab, isLoading, actions }: D
   if (isLoading) {
     return (
       <div className="flex items-center justify-center py-20">
-        <Loader2 className="h-8 w-8 animate-spin text-muted-foreground" />
+        <Loader2 className="h-8 w-8 animate-spin text-muted-foreground" aria-hidden />
       </div>
     );
   }
diff --git a/src/components/shared/empty-state.tsx b/src/components/shared/empty-state.tsx
index acdc5398..e29415b3 100644
--- a/src/components/shared/empty-state.tsx
+++ b/src/components/shared/empty-state.tsx
@@ -24,7 +24,7 @@ export function EmptyState({ icon: Icon, title, description, action, className }
     >
       {Icon && (
         <div className="w-12 h-12 rounded-full bg-muted flex items-center justify-center mb-4">
-          <Icon className="w-6 h-6 text-muted-foreground" />
+          <Icon className="w-6 h-6 text-muted-foreground" aria-hidden />
         </div>
       )}
       <h3 className="text-base font-semibold text-foreground mb-1">{title}</h3>
diff --git a/src/components/shared/filter-bar.tsx b/src/components/shared/filter-bar.tsx
index cc6cc29c..45dd0d6c 100644
--- a/src/components/shared/filter-bar.tsx
+++ b/src/components/shared/filter-bar.tsx
@@ -98,14 +98,14 @@ export function FilterBar({ filters, values, onChange, onClear }: FilterBarProps
       <Popover>
         <PopoverTrigger asChild>
           <Button variant="outline" size="sm" className="h-8">
-            <Filter className="mr-1.5 h-3.5 w-3.5" />
+            <Filter className="mr-1.5 h-3.5 w-3.5" aria-hidden />
             Filters
             {activeCount > 0 && (
               <Badge variant="secondary" className="ml-1.5 px-1.5 py-0 text-xs">
                 {activeCount}
               </Badge>
             )}
-            <ChevronDown className="ml-1 h-3.5 w-3.5" />
+            <ChevronDown className="ml-1 h-3.5 w-3.5" aria-hidden />
           </Button>
         </PopoverTrigger>
         <PopoverContent className="w-80 p-4" align="start">
diff --git a/src/components/shared/image-cropper-dialog.tsx b/src/components/shared/image-cropper-dialog.tsx
index 3ed292f0..78eb9797 100644
--- a/src/components/shared/image-cropper-dialog.tsx
+++ b/src/components/shared/image-cropper-dialog.tsx
@@ -131,7 +131,7 @@ export function ImageCropperDialog({
           <Button onClick={handleUpload} disabled={uploading || !pixels}>
             {uploading ? (
               <>
-                <Loader2 className="mr-1.5 h-4 w-4 animate-spin" />
+                <Loader2 className="mr-1.5 h-4 w-4 animate-spin" aria-hidden />
                 Uploading…
               </>
             ) : (
diff --git a/src/components/shared/inline-country-field.tsx b/src/components/shared/inline-country-field.tsx
index 5e539e3e..c78afce8 100644
--- a/src/components/shared/inline-country-field.tsx
+++ b/src/components/shared/inline-country-field.tsx
@@ -72,7 +72,7 @@ export function InlineCountryField({
             if (open) pickedRef.current = false;
           }}
         />
-        {saving && <Loader2 className="h-3 w-3 animate-spin text-muted-foreground" />}
+        {saving && <Loader2 className="h-3 w-3 animate-spin text-muted-foreground" aria-hidden />}
       </div>
     );
   }
@@ -98,7 +98,10 @@ export function InlineCountryField({
         {display ?? emptyText}
       </span>
       {!disabled && (
-        <Pencil className="h-3 w-3 opacity-0 transition-opacity group-hover:opacity-50" />
+        <Pencil
+          className="h-3 w-3 opacity-0 transition-opacity group-hover:opacity-50"
+          aria-hidden
+        />
       )}
     </button>
   );
diff --git a/src/components/shared/inline-editable-field.tsx b/src/components/shared/inline-editable-field.tsx
index 1a3b9598..582a5806 100644
--- a/src/components/shared/inline-editable-field.tsx
+++ b/src/components/shared/inline-editable-field.tsx
@@ -145,7 +145,7 @@ function InlineEditableFieldBody(props: InlineEditableFieldProps) {
             ))}
           </SelectContent>
         </Select>
-        {saving && <Loader2 className="h-3 w-3 animate-spin text-muted-foreground" />}
+        {saving && <Loader2 className="h-3 w-3 animate-spin text-muted-foreground" aria-hidden />}
       </div>
     );
   }
@@ -175,7 +175,7 @@ function InlineEditableFieldBody(props: InlineEditableFieldProps) {
           disabled={saving || disabled}
           className="h-8 text-sm w-auto"
         />
-        {saving && <Loader2 className="h-3 w-3 animate-spin text-muted-foreground" />}
+        {saving && <Loader2 className="h-3 w-3 animate-spin text-muted-foreground" aria-hidden />}
       </div>
     );
   }
@@ -219,7 +219,7 @@ function InlineEditableFieldBody(props: InlineEditableFieldProps) {
           rows={props.rows ?? 4}
           className="text-sm"
         />
-        {saving && <Loader2 className="h-3 w-3 animate-spin text-muted-foreground" />}
+        {saving && <Loader2 className="h-3 w-3 animate-spin text-muted-foreground" aria-hidden />}
       </div>
     );
   }
@@ -260,7 +260,7 @@ function InlineEditableFieldBody(props: InlineEditableFieldProps) {
         disabled={saving}
         className="h-7 text-sm"
       />
-      {saving && <Loader2 className="h-3 w-3 animate-spin text-muted-foreground" />}
+      {saving && <Loader2 className="h-3 w-3 animate-spin text-muted-foreground" aria-hidden />}
     </div>
   );
 }
diff --git a/src/components/shared/inline-phone-field.tsx b/src/components/shared/inline-phone-field.tsx
index 5818c1cd..d7655fa5 100644
--- a/src/components/shared/inline-phone-field.tsx
+++ b/src/components/shared/inline-phone-field.tsx
@@ -115,7 +115,7 @@ export function InlinePhoneField({
               'transition-colors hover:bg-primary/90 disabled:opacity-50',
             )}
           >
-            {saving ? <Loader2 className="size-3.5 animate-spin" /> : 'Save'}
+            {saving ? <Loader2 className="size-3.5 animate-spin" aria-hidden /> : 'Save'}
           </button>
         </div>
       </div>
@@ -146,7 +146,10 @@ export function InlinePhoneField({
         {display ?? emptyText}
       </span>
       {!disabled && (
-        <Pencil className="h-3 w-3 opacity-20 transition-opacity group-hover:opacity-60" />
+        <Pencil
+          className="h-3 w-3 opacity-20 transition-opacity group-hover:opacity-60"
+          aria-hidden
+        />
       )}
     </button>
   );
diff --git a/src/components/shared/inline-tag-editor.tsx b/src/components/shared/inline-tag-editor.tsx
index a0a5f5b2..65ceb1ee 100644
--- a/src/components/shared/inline-tag-editor.tsx
+++ b/src/components/shared/inline-tag-editor.tsx
@@ -91,7 +91,7 @@ export function InlineTagEditor({
               variant="ghost"
               className="h-6 px-2 text-xs text-muted-foreground hover:text-foreground"
             >
-              <Plus className="h-3 w-3 mr-1" />
+              <Plus className="h-3 w-3 mr-1" aria-hidden />
               {currentTags.length === 0 ? 'Add tag' : 'Add'}
             </Button>
           </PopoverTrigger>
@@ -119,7 +119,7 @@ export function InlineTagEditor({
                       style={{ backgroundColor: t.color }}
                     />
                     <span className="flex-1 truncate">{t.name}</span>
-                    {checked && <Check className="h-3.5 w-3.5 text-primary" />}
+                    {checked && <Check className="h-3.5 w-3.5 text-primary" aria-hidden />}
                   </button>
                 );
               })}
diff --git a/src/components/shared/inline-timezone-field.tsx b/src/components/shared/inline-timezone-field.tsx
index 9043479b..ec1fad90 100644
--- a/src/components/shared/inline-timezone-field.tsx
+++ b/src/components/shared/inline-timezone-field.tsx
@@ -71,7 +71,7 @@ export function InlineTimezoneField({
             if (open) pickedRef.current = false;
           }}
         />
-        {saving && <Loader2 className="h-3 w-3 animate-spin text-muted-foreground" />}
+        {saving && <Loader2 className="h-3 w-3 animate-spin text-muted-foreground" aria-hidden />}
       </div>
     );
   }
@@ -95,7 +95,10 @@ export function InlineTimezoneField({
         {display ?? emptyText}
       </span>
       {!disabled && (
-        <Pencil className="h-3 w-3 opacity-0 transition-opacity group-hover:opacity-50" />
+        <Pencil
+          className="h-3 w-3 opacity-0 transition-opacity group-hover:opacity-50"
+          aria-hidden
+        />
       )}
     </button>
   );
diff --git a/src/components/shared/interest-picker.tsx b/src/components/shared/interest-picker.tsx
index 19ab635c..ad3c0989 100644
--- a/src/components/shared/interest-picker.tsx
+++ b/src/components/shared/interest-picker.tsx
@@ -95,7 +95,7 @@ export function InterestPicker({
           className={cn('w-full justify-between', !value && 'text-muted-foreground')}
         >
           <span className="truncate">{selectedLabel}</span>
-          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" />
+          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" aria-hidden />
         </Button>
       </PopoverTrigger>
       <PopoverContent className="w-[320px] p-0" align="start">
diff --git a/src/components/shared/loading-skeleton.tsx b/src/components/shared/loading-skeleton.tsx
index 69c4d1b5..0db802dc 100644
--- a/src/components/shared/loading-skeleton.tsx
+++ b/src/components/shared/loading-skeleton.tsx
@@ -42,14 +42,14 @@ export function CardSkeleton({ className }: LoadingSkeletonProps) {
   return (
     <div className={cn('border border-border rounded-lg p-5 space-y-3 bg-background', className)}>
       <div className="flex items-center justify-between">
-        <Skeleton className="h-5 w-1/3" />
-        <Skeleton className="h-5 w-16 rounded-full" />
+        <Skeleton className="h-5 w-1/3" aria-hidden />
+        <Skeleton className="h-5 w-16 rounded-full" aria-hidden />
       </div>
-      <Skeleton className="h-4 w-full" />
-      <Skeleton className="h-4 w-4/5" />
+      <Skeleton className="h-4 w-full" aria-hidden />
+      <Skeleton className="h-4 w-4/5" aria-hidden />
       <div className="pt-2 flex gap-2">
-        <Skeleton className="h-8 w-20 rounded-md" />
-        <Skeleton className="h-8 w-20 rounded-md" />
+        <Skeleton className="h-8 w-20 rounded-md" aria-hidden />
+        <Skeleton className="h-8 w-20 rounded-md" aria-hidden />
       </div>
     </div>
   );
@@ -63,13 +63,13 @@ export function FormSkeleton({ fields = 4 }: { fields?: number }) {
     <div className="space-y-5">
       {Array.from({ length: fields }).map((_, i) => (
         <div key={i} className="space-y-1.5">
-          <Skeleton className="h-4 w-24" />
-          <Skeleton className="h-9 w-full rounded-md" />
+          <Skeleton className="h-4 w-24" aria-hidden />
+          <Skeleton className="h-9 w-full rounded-md" aria-hidden />
         </div>
       ))}
       <div className="flex gap-3 pt-2">
-        <Skeleton className="h-9 w-24 rounded-md" />
-        <Skeleton className="h-9 w-20 rounded-md" />
+        <Skeleton className="h-9 w-24 rounded-md" aria-hidden />
+        <Skeleton className="h-9 w-20 rounded-md" aria-hidden />
       </div>
     </div>
   );
@@ -97,10 +97,10 @@ export function PageSkeleton() {
       {/* Page header */}
       <div className="flex items-center justify-between">
         <div className="space-y-1.5">
-          <Skeleton className="h-7 w-48" />
-          <Skeleton className="h-4 w-72" />
+          <Skeleton className="h-7 w-48" aria-hidden />
+          <Skeleton className="h-4 w-72" aria-hidden />
         </div>
-        <Skeleton className="h-9 w-28 rounded-md" />
+        <Skeleton className="h-9 w-28 rounded-md" aria-hidden />
       </div>
       {/* Content */}
       <TableSkeleton />
diff --git a/src/components/shared/notes-list.tsx b/src/components/shared/notes-list.tsx
index 2bb4aa17..ebfe65fa 100644
--- a/src/components/shared/notes-list.tsx
+++ b/src/components/shared/notes-list.tsx
@@ -217,9 +217,9 @@ export function NotesList({ entityType, entityId, currentUserId, aggregate }: No
             onClick={() => createMutation.mutate(newNote.trim())}
           >
             {createMutation.isPending ? (
-              <Loader2 className="mr-1.5 h-4 w-4 animate-spin" />
+              <Loader2 className="mr-1.5 h-4 w-4 animate-spin" aria-hidden />
             ) : (
-              <Send className="mr-1.5 h-4 w-4" />
+              <Send className="mr-1.5 h-4 w-4" aria-hidden />
             )}
             Add Note
           </Button>
@@ -280,7 +280,7 @@ export function NotesList({ entityType, entityId, currentUserId, aggregate }: No
                         {SOURCE_LABEL[note.source]} · {note.sourceLabel}
                       </span>
                     )}
-                  {note.isLocked && <Lock className="h-3 w-3 text-muted-foreground" />}
+                  {note.isLocked && <Lock className="h-3 w-3 text-muted-foreground" aria-hidden />}
                   {canEdit(note) && (
                     <span className="text-xs text-muted-foreground">{getTimeRemaining(note)}</span>
                   )}
@@ -322,7 +322,7 @@ export function NotesList({ entityType, entityId, currentUserId, aggregate }: No
                       setEditContent(note.content);
                     }}
                   >
-                    <Pencil className="h-3.5 w-3.5" />
+                    <Pencil className="h-3.5 w-3.5" aria-hidden />
                   </Button>
                   <Button
                     variant="ghost"
@@ -330,7 +330,7 @@ export function NotesList({ entityType, entityId, currentUserId, aggregate }: No
                     className="h-7 w-7 text-destructive"
                     onClick={() => deleteMutation.mutate(note.id)}
                   >
-                    <Trash2 className="h-3.5 w-3.5" />
+                    <Trash2 className="h-3.5 w-3.5" aria-hidden />
                   </Button>
                 </div>
               )}
diff --git a/src/components/shared/owner-picker.tsx b/src/components/shared/owner-picker.tsx
index cdb361fa..2a697ffb 100644
--- a/src/components/shared/owner-picker.tsx
+++ b/src/components/shared/owner-picker.tsx
@@ -121,7 +121,7 @@ export function OwnerPicker({
             )}
             {selectedLabel}
           </span>
-          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" />
+          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" aria-hidden />
         </Button>
       </PopoverTrigger>
       <PopoverContent className="w-[320px] p-0" align="start">
diff --git a/src/components/shared/saved-views-dropdown.tsx b/src/components/shared/saved-views-dropdown.tsx
index b00776f2..8bcf63d7 100644
--- a/src/components/shared/saved-views-dropdown.tsx
+++ b/src/components/shared/saved-views-dropdown.tsx
@@ -35,7 +35,7 @@ export function SavedViewsDropdown({ entityType, onApplyView }: SavedViewsDropdo
     <DropdownMenu>
       <DropdownMenuTrigger asChild>
         <Button variant="outline" size="sm" className="h-8">
-          <Bookmark className="mr-1.5 h-3.5 w-3.5" />
+          <Bookmark className="mr-1.5 h-3.5 w-3.5" aria-hidden />
           Views
         </Button>
       </DropdownMenuTrigger>
@@ -54,7 +54,9 @@ export function SavedViewsDropdown({ entityType, onApplyView }: SavedViewsDropdo
           >
             <span className="truncate">{view.name}</span>
             <div className="flex items-center gap-1">
-              {activeViewId === view.id && <Check className="h-3.5 w-3.5 text-primary" />}
+              {activeViewId === view.id && (
+                <Check className="h-3.5 w-3.5 text-primary" aria-hidden />
+              )}
               <button
                 className="p-0.5 rounded hover:bg-muted"
                 onClick={(e) => {
@@ -62,7 +64,7 @@ export function SavedViewsDropdown({ entityType, onApplyView }: SavedViewsDropdo
                   deleteView(view.id);
                 }}
               >
-                <Trash2 className="h-3 w-3 text-muted-foreground" />
+                <Trash2 className="h-3 w-3 text-muted-foreground" aria-hidden />
               </button>
             </div>
           </DropdownMenuItem>
diff --git a/src/components/shared/send-document-dialog.tsx b/src/components/shared/send-document-dialog.tsx
index f28251a4..e689e1c0 100644
--- a/src/components/shared/send-document-dialog.tsx
+++ b/src/components/shared/send-document-dialog.tsx
@@ -241,7 +241,7 @@ function SendDocumentDialogInner({
             )}
             {previewQuery.isLoading && canPreview && (
               <div className="flex items-center gap-2 text-sm text-muted-foreground">
-                <Loader2 className="h-4 w-4 animate-spin" /> Rendering preview…
+                <Loader2 className="h-4 w-4 animate-spin" aria-hidden /> Rendering preview…
               </div>
             )}
           </div>
@@ -284,7 +284,9 @@ function SendDocumentDialogInner({
                 Back
               </Button>
               <Button onClick={() => sendMutation.mutate()} disabled={!canSend}>
-                {sendMutation.isPending && <Loader2 className="mr-2 h-4 w-4 animate-spin" />}
+                {sendMutation.isPending && (
+                  <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />
+                )}
                 Confirm and send
               </Button>
             </>
diff --git a/src/components/shared/subdivision-combobox.tsx b/src/components/shared/subdivision-combobox.tsx
index 9b676dd5..ffe8d661 100644
--- a/src/components/shared/subdivision-combobox.tsx
+++ b/src/components/shared/subdivision-combobox.tsx
@@ -87,7 +87,7 @@ export function SubdivisionCombobox({
           data-testid={testId}
         >
           <span className="truncate text-sm">{triggerLabel}</span>
-          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" />
+          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" aria-hidden />
         </Button>
       </PopoverTrigger>
       <PopoverContent className="w-[320px] p-0" align="start">
diff --git a/src/components/shared/tag-picker.tsx b/src/components/shared/tag-picker.tsx
index 80864121..59ef9939 100644
--- a/src/components/shared/tag-picker.tsx
+++ b/src/components/shared/tag-picker.tsx
@@ -74,7 +74,7 @@ export function TagPicker({
                 ? `${selectedIds.length} tag${selectedIds.length > 1 ? 's' : ''} selected`
                 : placeholder}
             </span>
-            <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" />
+            <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" aria-hidden />
           </Button>
         </PopoverTrigger>
         <PopoverContent className="w-(--radix-popover-trigger-width) p-0" align="start">
diff --git a/src/components/shared/timezone-combobox.tsx b/src/components/shared/timezone-combobox.tsx
index 3c682e45..36c29485 100644
--- a/src/components/shared/timezone-combobox.tsx
+++ b/src/components/shared/timezone-combobox.tsx
@@ -94,7 +94,7 @@ export function TimezoneCombobox({
           data-testid={testId}
         >
           <span className="truncate text-sm">{selectedLabel}</span>
-          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" />
+          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" aria-hidden />
         </Button>
       </PopoverTrigger>
       <PopoverContent className="w-(--radix-popper-anchor-width) min-w-[360px] p-0" align="start">
diff --git a/src/components/yachts/yacht-card.tsx b/src/components/yachts/yacht-card.tsx
index 6f03c863..ea81e19f 100644
--- a/src/components/yachts/yacht-card.tsx
+++ b/src/components/yachts/yacht-card.tsx
@@ -69,22 +69,22 @@ export function YachtCard({ yacht, portSlug, onEdit, onArchive }: YachtCardProps
               onClick={(e) => e.stopPropagation()}
               aria-label={`Actions for ${yacht.name}`}
             >
-              <MoreHorizontal className="h-4 w-4" />
+              <MoreHorizontal className="h-4 w-4" aria-hidden />
             </Button>
           </DropdownMenuTrigger>
           <DropdownMenuContent align="end">
             <DropdownMenuItem asChild>
               <Link href={`/${portSlug}/yachts/${yacht.id}`}>
-                <Eye className="mr-2 h-3.5 w-3.5" />
+                <Eye className="mr-2 h-3.5 w-3.5" aria-hidden />
                 View
               </Link>
             </DropdownMenuItem>
             <DropdownMenuItem onClick={() => onEdit(yacht)}>
-              <Pencil className="mr-2 h-3.5 w-3.5" />
+              <Pencil className="mr-2 h-3.5 w-3.5" aria-hidden />
               Edit
             </DropdownMenuItem>
             <DropdownMenuItem className="text-destructive" onClick={() => onArchive(yacht)}>
-              <Archive className="mr-2 h-3.5 w-3.5" />
+              <Archive className="mr-2 h-3.5 w-3.5" aria-hidden />
               Archive
             </DropdownMenuItem>
           </DropdownMenuContent>
@@ -92,7 +92,7 @@ export function YachtCard({ yacht, portSlug, onEdit, onArchive }: YachtCardProps
       }
     >
       <div className="flex items-start gap-3">
-        <ListCardAvatar icon={<Ship className="h-5 w-5" />} />
+        <ListCardAvatar icon={<Ship className="h-5 w-5" aria-hidden />} />
         <div className="min-w-0 flex-1">
           {/* Title row + spacer for actions button */}
           <div className="flex items-start justify-between gap-2">
diff --git a/src/components/yachts/yacht-columns.tsx b/src/components/yachts/yacht-columns.tsx
index fe3240d2..16d5d709 100644
--- a/src/components/yachts/yacht-columns.tsx
+++ b/src/components/yachts/yacht-columns.tsx
@@ -149,7 +149,7 @@ export function getYachtColumns({
               className="h-7 w-7"
               onClick={(e) => e.stopPropagation()}
             >
-              <MoreHorizontal className="h-4 w-4" />
+              <MoreHorizontal className="h-4 w-4" aria-hidden />
             </Button>
           </DropdownMenuTrigger>
           <DropdownMenuContent align="end">
@@ -158,16 +158,16 @@ export function getYachtColumns({
                 // eslint-disable-next-line @typescript-eslint/no-explicit-any
                 href={`/${portSlug}/yachts/${row.original.id}` as any}
               >
-                <Eye className="mr-2 h-3.5 w-3.5" />
+                <Eye className="mr-2 h-3.5 w-3.5" aria-hidden />
                 View
               </Link>
             </DropdownMenuItem>
             <DropdownMenuItem onClick={() => onEdit(row.original)}>
-              <Pencil className="mr-2 h-3.5 w-3.5" />
+              <Pencil className="mr-2 h-3.5 w-3.5" aria-hidden />
               Edit
             </DropdownMenuItem>
             <DropdownMenuItem className="text-destructive" onClick={() => onArchive(row.original)}>
-              <Archive className="mr-2 h-3.5 w-3.5" />
+              <Archive className="mr-2 h-3.5 w-3.5" aria-hidden />
               Archive
             </DropdownMenuItem>
           </DropdownMenuContent>
diff --git a/src/components/yachts/yacht-detail-header.tsx b/src/components/yachts/yacht-detail-header.tsx
index 161fc984..b365c1f9 100644
--- a/src/components/yachts/yacht-detail-header.tsx
+++ b/src/components/yachts/yacht-detail-header.tsx
@@ -171,7 +171,7 @@ export function YachtDetailHeader({ yacht }: YachtDetailHeaderProps) {
           {/* Actions */}
           <div className="flex flex-wrap items-center gap-2">
             <Button variant="outline" size="sm" onClick={() => setEditOpen(true)}>
-              <Pencil className="mr-1.5 h-3.5 w-3.5" />
+              <Pencil className="mr-1.5 h-3.5 w-3.5" aria-hidden />
               Edit
             </Button>
             <PermissionGate resource="yachts" action="transfer">
@@ -181,7 +181,7 @@ export function YachtDetailHeader({ yacht }: YachtDetailHeaderProps) {
                 onClick={() => setTransferOpen(true)}
                 disabled={isArchived}
               >
-                <ArrowRightLeft className="mr-1.5 h-3.5 w-3.5" />
+                <ArrowRightLeft className="mr-1.5 h-3.5 w-3.5" aria-hidden />
                 Transfer
               </Button>
             </PermissionGate>
@@ -191,7 +191,7 @@ export function YachtDetailHeader({ yacht }: YachtDetailHeaderProps) {
               onClick={() => setArchiveOpen(true)}
               disabled={isArchived}
             >
-              <Archive className="mr-1.5 h-3.5 w-3.5" />
+              <Archive className="mr-1.5 h-3.5 w-3.5" aria-hidden />
               Archive
             </Button>
           </div>
diff --git a/src/components/yachts/yacht-form.tsx b/src/components/yachts/yacht-form.tsx
index 55d299b9..ba924c0f 100644
--- a/src/components/yachts/yacht-form.tsx
+++ b/src/components/yachts/yacht-form.tsx
@@ -359,7 +359,7 @@ export function YachtForm({ open, onOpenChange, yacht, initialOwner }: YachtForm
             </Button>
             <Button type="submit" disabled={isSubmitting || mutation.isPending}>
               {(isSubmitting || mutation.isPending) && (
-                <Loader2 className="mr-2 h-4 w-4 animate-spin" />
+                <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />
               )}
               {isEdit ? 'Save changes' : 'Create Yacht'}
             </Button>
diff --git a/src/components/yachts/yacht-list.tsx b/src/components/yachts/yacht-list.tsx
index 863386cd..3457039e 100644
--- a/src/components/yachts/yacht-list.tsx
+++ b/src/components/yachts/yacht-list.tsx
@@ -116,7 +116,7 @@ export function YachtList() {
         actions={
           <PermissionGate resource="yachts" action="create">
             <Button size="sm" onClick={() => setCreateOpen(true)}>
-              <Plus className="mr-1.5 h-4 w-4" />
+              <Plus className="mr-1.5 h-4 w-4" aria-hidden />
               New Yacht
             </Button>
           </PermissionGate>
diff --git a/src/components/yachts/yacht-ownership-history.tsx b/src/components/yachts/yacht-ownership-history.tsx
index c64fd7a3..076ef50e 100644
--- a/src/components/yachts/yacht-ownership-history.tsx
+++ b/src/components/yachts/yacht-ownership-history.tsx
@@ -64,7 +64,7 @@ export function YachtOwnershipHistory({ yachtId }: YachtOwnershipHistoryProps) {
   if (isLoading) {
     return (
       <div className="flex items-center justify-center py-12">
-        <Loader2 className="h-6 w-6 animate-spin text-muted-foreground" />
+        <Loader2 className="h-6 w-6 animate-spin text-muted-foreground" aria-hidden />
       </div>
     );
   }
diff --git a/src/components/yachts/yacht-picker.tsx b/src/components/yachts/yacht-picker.tsx
index fb8a076b..2bddc9e0 100644
--- a/src/components/yachts/yacht-picker.tsx
+++ b/src/components/yachts/yacht-picker.tsx
@@ -88,7 +88,7 @@ export function YachtPicker({
           className={cn('w-full justify-between', !value && 'text-muted-foreground')}
         >
           <span className="truncate">{selectedLabel}</span>
-          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" />
+          <ChevronsUpDown className="ml-2 h-4 w-4 shrink-0 opacity-50" aria-hidden />
         </Button>
       </PopoverTrigger>
       <PopoverContent className="w-[320px] p-0" align="start">
diff --git a/src/components/yachts/yacht-transfer-dialog.tsx b/src/components/yachts/yacht-transfer-dialog.tsx
index e89ec26b..3234d12b 100644
--- a/src/components/yachts/yacht-transfer-dialog.tsx
+++ b/src/components/yachts/yacht-transfer-dialog.tsx
@@ -190,7 +190,7 @@ export function YachtTransferDialog({
             </Button>
             <Button type="submit" disabled={isSubmitting || mutation.isPending}>
               {(isSubmitting || mutation.isPending) && (
-                <Loader2 className="mr-2 h-4 w-4 animate-spin" />
+                <Loader2 className="mr-2 h-4 w-4 animate-spin" aria-hidden />
               )}
               Transfer
             </Button>