Clarify breaking changes (closes #2332)

Author: mdecimus

CHANGELOG.md

@@ -2,7 +2,7 @@
 
 All notable changes to this project will be documented in this file. This project adheres to [Semantic Versioning](http://semver.org/).
 
-## [0.14.1] - 2025-10-27
+## [0.14.1] - 2025-10-28
 
 If you are upgrading from v0.13.4 and below, this version includes **breaking changes** to the internal directory, calendar and contacts. Please read the [upgrading documentation](https://stalw.art/docs/install/upgrade) for more information on how to upgrade from previous versions.
 
@@ -30,9 +30,15 @@ If you are upgrading from v0.13.4 and below, this version includes **breaking ch
 - i18n: Swedish language support (contributed by @purung)
 
 ## Changed
+- **Breaking Database Changes** (migrated automatically on first start):
+  - Internal directory schema changed.
+  - Calendar and Contacts storage schema changed.
+  - Sieve scripts storage schema changed.
+  - Push Subscriptions storage schema changed.
 - Replaced `sieve.untrusted.limits.max-scripts` and `jmap.push.max-total` with `object-quota.*` settings. 
 - Cluster node roles now allow sharding.
 
+
 ## Fixed
 - Push Subscription: Clean-up of expired subscriptions and cluster notification of changes (#1248)
 - CalDAV: Per-user CalDAV properties (#2058)

crates/common/src/storage/index.rs

@@ -14,7 +14,7 @@ use rkyv::{
 use std::{borrow::Cow, fmt::Debug};
 use store::{
     Serialize, SerializeInfallible,
-    write::{Archive, Archiver, BatchBuilder, BlobOp, DirectoryClass, IntoOperations, TagValue},
+    write::{Archive, Archiver, BatchBuilder, BlobOp, DirectoryClass, IntoOperations},
 };
 use types::{
     acl::AclGrant,
@@ -34,10 +34,6 @@ pub enum IndexValue<'x> {
         field: Field,
         value: Vec<IndexItem<'x>>,
     },
-    Tag {
-        field: Field,
-        value: Vec<TagValue>,
-    },
     Blob {
         value: BlobHash,
     },
@@ -379,15 +375,6 @@ fn build_index(
                 }
             }
         }
-        IndexValue::Tag { field, value } => {
-            for item in value {
-                if set {
-                    batch.tag(field, item);
-                } else {
-                    batch.untag(field, item);
-                }
-            }
-        }
         IndexValue::Blob { value } => {
             if set {
                 batch.set(BlobOp::Link { hash: value }, vec![]);
@@ -524,27 +511,6 @@ fn merge_index(
                 batch.unindex(field, value.into_owned());
             }
         }
-        (
-            IndexValue::Tag {
-                field,
-                value: old_value,
-            },
-            IndexValue::Tag {
-                value: new_value, ..
-            },
-        ) => {
-            for old_tag in &old_value {
-                if !new_value.contains(old_tag) {
-                    batch.untag(field, old_tag.clone());
-                }
-            }
-
-            for new_tag in new_value {
-                if !old_value.contains(&new_tag) {
-                    batch.tag(field, new_tag);
-                }
-            }
-        }
         (IndexValue::Blob { value: old_hash }, IndexValue::Blob { value: new_hash }) => {
             batch.clear(BlobOp::Link { hash: old_hash });
             batch.set(BlobOp::Link { hash: new_hash }, vec![]);

crates/common/src/storage/tag.rs

@@ -4,26 +4,9 @@
  * SPDX-License-Identifier: AGPL-3.0-only OR LicenseRef-SEL
  */
 
+use super::BitmapHash;
 use crate::backend::MAX_TOKEN_LENGTH;
 
-use super::{BitmapClass, BitmapHash};
-
-impl BitmapClass {
-    pub fn word(token: impl AsRef<[u8]>, field: impl Into<u8>) -> Self {
-        BitmapClass::Text {
-            field: field.into(),
-            token: BitmapHash::new(token),
-        }
-    }
-
-    pub fn stemmed(token: impl AsRef<[u8]>, field: impl Into<u8>) -> Self {
-        BitmapClass::Text {
-            field: field.into() | (1 << 7),
-            token: BitmapHash::new(token),
-        }
-    }
-}
-
 impl BitmapHash {
     pub fn new(item: impl AsRef<[u8]>) -> Self {
         Self {