Merge branch 'rule-builder': a row builder for the tagging rules

Item 76 replaces the four free-text fields with a row builder: field and
operator dropdowns per condition, +/- to add and remove them, a match
all/any choice, and a separate "but not" block. The stored format does
not change, so mailctl needs no edit. The query string stays
authoritative and remains visible, and a rule the builder cannot
represent opens in a text mode every rule carries.

Along the way, items 75, 77 and 80, and a data-loss defect released in
0.16.0 (item 79): opening the dialog and pressing Save destroyed the
first rule with nothing edited. That one damaged a real rule in the
user's own file, which was repaired by hand.

Four defects in this work were found by hand rather than by the suite,
and each is recorded where it was missed: a lost note, a one-way text
mode toggle, a geometry save on a path neither button takes, and a rule
list squeezed to one row by a long rule. Two Qt traps and one about the
user's compositor went into CLAUDE.md.

1 files changed, 17 insertions, 0 deletions

diff --git a/src/notmuchworker.h b/src/notmuchworker.h
index b3fbed3..9736ab8 100644
--- a/src/notmuchworker.h
+++ b/src/notmuchworker.h
@@ -151,6 +151,18 @@ public slots:
     /// called when the dialog is opened and never on a timer.
     void requestDatabaseStats(quint64 generation);
 
+    /// Every Maildir folder under the database root, as paths relative to it.
+    ///
+    /// From the DISK, not from the index: a folder mbsync created and nothing
+    /// has landed in yet is still a folder a tagging rule may target, and one
+    /// derived from indexed message paths would not offer it.
+    ///
+    /// Here rather than in MainWindow because the database root is
+    /// notmuch's `database.path` and this class owns the only handle that can
+    /// answer for it. Duplicating the path into config is exactly the second
+    /// source of truth the design refuses.
+    void requestFolders();
+
 signals:
     void threadsReady(const QVector<ThreadSummary> &threads, quint64 generation);
     void queryFinished(int totalThreads, quint64 generation);
@@ -175,6 +187,11 @@ signals:
     /// renders as unknown rather than as zero.
     void databaseStatsReady(const DatabaseStats &stats, quint64 generation);
 
+    /// Maildir folders relative to the database root, sorted. No generation:
+    /// the tree on disk does not change under a query, and the one consumer
+    /// asks once when its dialog opens.
+    void foldersReady(const QStringList &folders);
+
     void errorOccurred(const QString &message);
 
 private: