fix(compose-from-case): use any contact with email + show number/status

Walk Case.contactsIds for the first contact with a non-empty emailAddress
(falling back to the Account's emailAddress) instead of blindly using
contactsIds[0] and giving up if it has no email — Outlook was opening
the new mail with no To recipient when the first linked contact was
e.g. opposing counsel or a witness without a stored address.

Also hydrate the "כתוב מתיק" picker rows with number/status/court#
through GetCaseAsync (same pattern as SaveAttachmentsViewModel) and
switch the dialog to the DataGrid layout so it matches the file-
attachments picker the user expected.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

src/OutlookAddin.UI/Dialogs/ComposeFromCaseDialog.xaml

@@ -1,9 +1,10 @@
 <Window x:Class="MarcusLaw.OutlookAddin.UI.Dialogs.ComposeFromCaseDialog"
         xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
         xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
+        xmlns:conv="clr-namespace:MarcusLaw.OutlookAddin.UI.Converters"
         Title="כתוב מייל מתיק"
-        Height="480" Width="600"
-        MinHeight="320" MinWidth="500"
+        Height="520" Width="680"
+        MinHeight="360" MinWidth="540"
         FlowDirection="RightToLeft"
         WindowStartupLocation="CenterOwner"
         ShowInTaskbar="False"
@@ -11,6 +12,7 @@
         Background="#F8FAFC"
         FontFamily="Segoe UI, David, Frank Ruehl, Arial">
     <Window.Resources>
+        <conv:CaseStatusToHebrewConverter x:Key="CaseStatusHe" />
         <SolidColorBrush x:Key="Primary"      Color="#2563EB" />
         <SolidColorBrush x:Key="PrimaryHover" Color="#1D4ED8" />
         <SolidColorBrush x:Key="Border"       Color="#E2E8F0" />
@@ -134,24 +136,30 @@
         </Border>
 
         <Border Grid.Row="1" Style="{StaticResource Card}">
-            <ListBox ItemsSource="{Binding Cases}"
+            <DataGrid ItemsSource="{Binding Cases}"
                       SelectedItem="{Binding SelectedCase}"
+                      AutoGenerateColumns="False"
+                      HeadersVisibility="Column"
+                      GridLinesVisibility="Horizontal"
+                      HorizontalGridLinesBrush="#E2E8F0"
+                      RowBackground="White"
+                      AlternatingRowBackground="#F8FAFC"
                       BorderThickness="0"
-                     Background="Transparent"
-                     ItemContainerStyle="{StaticResource ResultItem}"
-                     Padding="4">
-                <ListBox.ItemTemplate>
-                    <DataTemplate>
-                        <StackPanel>
-                            <TextBlock FontSize="13">
-                                <Run Text="[" /><Run Text="{Binding Number, Mode=OneWay}" /><Run Text="] " />
-                                <Run Text="{Binding Name, Mode=OneWay}" FontWeight="SemiBold" />
-                            </TextBlock>
-                            <TextBlock Text="{Binding Status}" Foreground="{StaticResource Muted}" FontSize="11" />
-                        </StackPanel>
-                    </DataTemplate>
-                </ListBox.ItemTemplate>
-            </ListBox>
+                      CanUserAddRows="False"
+                      CanUserDeleteRows="False"
+                      CanUserResizeRows="False"
+                      CanUserReorderColumns="False"
+                      SelectionMode="Single"
+                      SelectionUnit="FullRow"
+                      IsReadOnly="True"
+                      RowHeight="30">
+                <DataGrid.Columns>
+                    <DataGridTextColumn Header="מס׳ תיק" Binding="{Binding Number}" Width="80" />
+                    <DataGridTextColumn Header="שם" Binding="{Binding Name}" Width="*" />
+                    <DataGridTextColumn Header="מס׳ בבית משפט" Binding="{Binding CourtCaseNumber}" Width="140" />
+                    <DataGridTextColumn Header="סטטוס" Binding="{Binding Status, Converter={StaticResource CaseStatusHe}}" Width="110" />
+                </DataGrid.Columns>
+            </DataGrid>
         </Border>
 
         <TextBlock Grid.Row="2" Text="{Binding StatusMessage}"

src/OutlookAddin.UI/ViewModels/ComposeFromCaseViewModel.cs

@@ -1,4 +1,5 @@
 using System;
+using System.Collections.Generic;
 using System.Collections.ObjectModel;
 using System.Threading;
 using System.Threading.Tasks;
@@ -74,20 +75,36 @@ namespace MarcusLaw.OutlookAddin.UI.ViewModels
                 ct.ThrowIfCancellationRequested();
 
                 Cases.Clear();
-                int kept = 0;
+                var caseHits = new List<EspoEntityRef>();
                 foreach (var hit in search.List)
                 {
-                    if (!string.Equals(hit.EntityType, "Case", StringComparison.OrdinalIgnoreCase)) continue;
-                    Cases.Add(new CaseEntity
-                    {
-                        Id = hit.Id,
-                        Name = hit.Name,
-                    });
-                    kept++;
+                    if (string.Equals(hit.EntityType, "Case", StringComparison.OrdinalIgnoreCase))
+                        caseHits.Add(hit);
                 }
-                StatusMessage = kept == 0
-                    ? "אין תוצאות מסוג תיק (Case)"
-                    : $"נמצאו {kept} תיקים";
+
+                if (caseHits.Count == 0)
+                {
+                    StatusMessage = "אין תוצאות מסוג תיק (Case)";
+                    return;
+                }
+
+                // GlobalSearch only returns id+name. Hydrate each hit with
+                // number+status (and the Hebrew case name when present) so the
+                // picker shows the same columns the file-attachments picker
+                // does.
+                var detailTasks = new List<Task<CaseEntity?>>(caseHits.Count);
+                foreach (var hit in caseHits)
+                {
+                    detailTasks.Add(LoadCaseRowAsync(hit, ct));
+                }
+                var details = await Task.WhenAll(detailTasks).ConfigureAwait(true);
+                ct.ThrowIfCancellationRequested();
+
+                foreach (var row in details)
+                {
+                    if (row != null) Cases.Add(row);
+                }
+                StatusMessage = $"נמצאו {Cases.Count} תיקים";
             }
             catch (OperationCanceledException) { /* superseded */ }
             catch (EspoCrmAuthorizationException)
@@ -101,6 +118,27 @@ namespace MarcusLaw.OutlookAddin.UI.ViewModels
             }
         }
 
+        private async Task<CaseEntity?> LoadCaseRowAsync(EspoEntityRef hit, CancellationToken ct)
+        {
+            var row = new CaseEntity { Id = hit.Id, Name = hit.Name };
+            if (string.IsNullOrEmpty(hit.Id)) return row;
+            try
+            {
+                var detail = await _client.GetCaseAsync(
+                    hit.Id, "id,name,number,status,cCourtCaseNumber", ct).ConfigureAwait(false);
+                row.Number = detail.Number;
+                row.Status = detail.Status;
+                row.CourtCaseNumber = detail.CourtCaseNumber;
+                if (!string.IsNullOrEmpty(detail.Name)) row.Name = detail.Name;
+            }
+            catch (OperationCanceledException) { throw; }
+            catch (Exception ex)
+            {
+                _logger.Warning(ex, "ComposeFromCase: hydrating case {CaseId} failed", hit.Id);
+            }
+            return row;
+        }
+
         [RelayCommand(CanExecute = nameof(CanSubmit))]
         private void Submit()
         {

src/OutlookAddin/Services/ComposeService.cs

@@ -42,21 +42,7 @@ namespace OutlookAddin.Services
             if (string.IsNullOrWhiteSpace(caseId)) throw new ArgumentException("caseId required", nameof(caseId));
 
             var caseEntity = await _client.GetCaseAsync(caseId, "id,name,number,status,contactsIds,accountId").ConfigureAwait(true);
-            ContactEntity? primaryContact = null;
-            if (caseEntity.ContactsIds != null && caseEntity.ContactsIds.Count > 0)
-            {
-                try
-                {
-                    primaryContact = await _client.GetContactAsync(
-                        caseEntity.ContactsIds[0],
-                        "id,name,emailAddress",
-                        default).ConfigureAwait(true);
-                }
-                catch (Exception ex)
-                {
-                    _logger.Warning(ex, "ComposeService: failed to load primary contact for case {CaseId}", caseId);
-                }
-            }
+            var recipientEmail = await ResolveRecipientEmailAsync(caseEntity, caseId).ConfigureAwait(true);
 
             var guid = Guid.NewGuid().ToString("N");
             StampUserProperty(mail, EspoCaseIdProperty, caseEntity.Id);
@@ -65,14 +51,14 @@ namespace OutlookAddin.Services
 
             try
             {
-                if (!string.IsNullOrWhiteSpace(primaryContact?.EmailAddress))
+                if (!string.IsNullOrWhiteSpace(recipientEmail))
                 {
                     var existingTo = mail.To ?? string.Empty;
-                    if (!existingTo.Contains(primaryContact!.EmailAddress!))
+                    if (!existingTo.Contains(recipientEmail!))
                     {
                         mail.To = string.IsNullOrEmpty(existingTo)
-                            ? primaryContact.EmailAddress
-                            : existingTo + "; " + primaryContact.EmailAddress;
+                            ? recipientEmail
+                            : existingTo + "; " + recipientEmail;
                     }
                 }
 
@@ -107,6 +93,61 @@ namespace OutlookAddin.Services
                 new FilingTarget("Case", caseEntity.Id, caseEntity.Name ?? caseEntity.Id));
         }
 
+        /// <summary>
+        /// Find a usable recipient email for the case. EspoCRM's Case.contactsIds
+        /// is a many-to-many bag (primary + co-counsel + witnesses + …) and the
+        /// first entry isn't guaranteed to be the actual client — and even when
+        /// it is, the contact may have no emailAddress stored. So walk the list
+        /// and return the first non-empty address, then fall back to the
+        /// linked Account's email.
+        /// </summary>
+        private async Task<string?> ResolveRecipientEmailAsync(CaseEntity caseEntity, string caseId)
+        {
+            if (caseEntity.ContactsIds != null)
+            {
+                foreach (var contactId in caseEntity.ContactsIds)
+                {
+                    if (string.IsNullOrWhiteSpace(contactId)) continue;
+                    try
+                    {
+                        var contact = await _client.GetContactAsync(
+                            contactId,
+                            "id,name,emailAddress,emailAddressData",
+                            default).ConfigureAwait(true);
+                        if (!string.IsNullOrWhiteSpace(contact?.EmailAddress))
+                        {
+                            return contact!.EmailAddress;
+                        }
+                    }
+                    catch (Exception ex)
+                    {
+                        _logger.Warning(ex, "ComposeService: failed to load contact {ContactId} for case {CaseId}", contactId, caseId);
+                    }
+                }
+            }
+
+            if (!string.IsNullOrWhiteSpace(caseEntity.AccountId))
+            {
+                try
+                {
+                    var account = await _client.GetAccountAsync(
+                        caseEntity.AccountId!,
+                        "id,name,emailAddress",
+                        default).ConfigureAwait(true);
+                    if (!string.IsNullOrWhiteSpace(account?.EmailAddress))
+                    {
+                        return account!.EmailAddress;
+                    }
+                }
+                catch (Exception ex)
+                {
+                    _logger.Warning(ex, "ComposeService: failed to load account {AccountId} for case {CaseId}", caseEntity.AccountId, caseId);
+                }
+            }
+
+            return null;
+        }
+
         public static string? ReadEspoCaseId(Outlook.MailItem mail) => TryReadUserProperty(mail, EspoCaseIdProperty);
         public static string? ReadEspoCaseName(Outlook.MailItem mail) => TryReadUserProperty(mail, EspoCaseNameProperty);
         public static string? ReadAddInGuid(Outlook.MailItem mail) => TryReadUserProperty(mail, AddInGuidProperty);