feat(linux): keep stored files before a format upgrade and open .sql files

.github/workflows/build-linux.yml

@@ -409,6 +409,18 @@ jobs:
       # checks in every rust-format entry.
       - name: Check the catalogues
         run: meson test -C _build --no-rebuild --suite po --print-errorlogs
+      # Reported, never failed: strings land before the translation run
+      # after the string freeze. A release refuses the gap
+      # (release-linux.yml, po/README.md).
+      - name: Report untranslated entries
+        run: |
+          status=0
+          po/check-complete.sh >> "$GITHUB_STEP_SUMMARY" || status=$?
+          case "$status" in
+            0) ;;
+            1) echo "::warning title=Translations::a language in po/LINGUAS has untranslated or fuzzy entries, which a release refuses (linux/po/README.md)" ;;
+            *) exit "$status" ;;
+          esac
       # Translators start from this template (po/README.md), since few
       # distributions ship gettext 1.0 yet.
       - name: Publish tablepro.pot

.github/workflows/release-linux.yml

@@ -166,6 +166,9 @@ jobs:
         run: |
           meson setup _build --fatal-meson-warnings
           meson compile -C _build tablepro-pot
+      # A language in po/LINGUAS ships complete (po/README.md).
+      - name: Every shipped catalogue is complete
+        run: po/check-complete.sh >> "$GITHUB_STEP_SUMMARY"
       - uses: actions/upload-artifact@cf430e030ddbb5b0abf93d22962f4752f3646cd9 # v7.0.2
         with:
           name: linux-pot

core/crates/storage/src/connections/mod.rs

@@ -814,6 +814,11 @@ impl ConnectionStore {
         // A connection saved with a tag deleted meanwhile drops it here.
         document.tidy();
         let bytes = crate::document::encode_document(&document)?;
+        crate::upgrade_copies::keep_before_write_blocking::<ConnectionsDocument>(&self.inner.path)?;
+        // Before the list itself, so a deleted connection is never gone from the list but kept in a
+        // copy, and a failed trim leaves both as they were for the next write to try again.
+        let kept = document.connections.iter().map(|saved| saved.id.to_string()).collect();
+        crate::upgrade_copies::trim_connection_copies_blocking(&self.inner.path, &kept)?;
         crate::fs::write_private_blocking(&self.inner.path, &bytes)?;
         *state = StoreState::Ready(document);
         self.publish(state);

core/crates/storage/src/document.rs

@@ -5,6 +5,7 @@ use serde::de::DeserializeOwned;
 use serde_json::Value;
 
 use crate::document_problem::{DocumentProblem, DocumentProblemKind};
+use crate::error::StorageError;
 
 /// One step of a document's upgrade chain: a file at one version
 /// rewritten into the shape of the next. Its `version` field is left for
@@ -97,6 +98,17 @@ fn corrupt(path: &Path, error: &serde_json::Error) -> DocumentProblem {
     )
 }
 
+pub(crate) fn stored_version(bytes: &[u8]) -> Option<u32> {
+    serde_json::from_slice::<VersionProbe>(bytes).ok()?.version
+}
+
+// A file an older build wrote is copied aside first, and a failed copy leaves it as it was.
+pub fn write_document_blocking<D: VersionedDocument>(path: &Path, document: &D) -> Result<(), StorageError> {
+    let bytes = encode_document(document)?;
+    crate::upgrade_copies::keep_before_write_blocking::<D>(path)?;
+    crate::fs::write_private_blocking(path, &bytes)
+}
+
 pub fn encode_document<D: VersionedDocument>(document: &D) -> Result<Vec<u8>, serde_json::Error> {
     #[derive(Serialize)]
     struct Versioned<'a, D> {

core/crates/storage/src/lib.rs

@@ -21,6 +21,7 @@ pub mod query_history;
 pub mod saved_queries;
 mod tag_name_error;
 mod unix_time;
+pub mod upgrade_copies;
 
 pub use connection_color::ConnectionColor;
 pub use connection_group::ConnectionGroup;