feat: load the contact store once and feed both consumers

MainWindow reads contactsDir() once while building its UI, holds the
result in m_contacts, and hands it to the query bar's completer right
after that is constructed and to every ComposeWindow as it opens. An
empty contactsDir() skips the call, so a machine with no address book
pays nothing and warns about nothing.

The directory is not watched: a restart picks up a vdirsyncer update,
and a QFileSystemWatcher would be a live-index feature nobody asked for.

No action is added, so none of the five places in "Adding an action is
FIVE places" applies: there is no name in KeyMap::knownActions(), no
default binding, no icon-table entry and no menu entry to add.

1 files changed, 16 insertions, 0 deletions

diff --git a/src/mainwindow.cpp b/src/mainwindow.cpp
index 2443ead..4c37a83 100644
--- a/src/mainwindow.cpp
+++ b/src/mainwindow.cpp
@@ -770,6 +770,17 @@ void MainWindow::buildUi()
     m_queryEdit->installEventFilter(this);
     m_queryCompleter = new QueryCompleter(m_queryEdit, m_config, this);
 
+    // The address book, read once for the whole session. A vdirsyncer
+    // directory is a handful of small vCards, so the parse is paid at startup
+    // rather than per keystroke, and a restart is the refresh route: watching
+    // the directory would be a live-index feature nobody asked for. An empty
+    // contactsDir() is the feature switched off, so the loader is not even
+    // called, and no warning is raised for a machine with no address book.
+    if (!m_config.contactsDir().isEmpty()) {
+        m_contacts = ContactStore::loadDirectory(m_config.contactsDir());
+        m_queryCompleter->setContacts(m_contacts);
+    }
+
     m_markReadTimer = new QTimer(this);
     // Named so a test can observe whether it is armed without the window
     // having to expose the timer or the decision that armed it.
@@ -1254,6 +1265,11 @@ void MainWindow::openComposer(const ComposeContext &context)
     composer->setAttribute(Qt::WA_DeleteOnClose);
     m_composers.append(QPointer<ComposeWindow>(composer));
 
+    // The same list the query bar offers, from the once-per-session load. The
+    // composer builds its own model over it, so a recipient field completes on
+    // the address book the query bar completes from.
+    composer->setContacts(m_contacts);
+
     // Compaction, and ONLY compaction. The QPointer above is what keeps
     // composersBlockingQuit() safe against a destroyed window, since it nulls
     // on destruction; this drops the entry so the list does not accumulate