fix(clients): resolve Tauri 2.0 configuration and build issues

doublegate · claude · doublegate · commit 808e83357322 · 2026-01-21T11:27:26.000-05:00
wraith-chat fixes:
- Fix Tauri 2.0 plugin initialization error by removing invalid plugin
  configs from tauri.conf.json and adding capabilities/default.json
- Fix SQLCipher database key persistence by adding proper keyring
  features (sync-secret-service, crypto-rust) for Linux Secret Service
- Add database key mismatch detection with automatic backup recovery
- Add KDE Wayland workaround for Error 71 (EPROTO)
- Fix TypeScript unused import errors in frontend components
- Add missing tsconfig.node.json for Vite build
- Update Tauri plugin dependencies to v2.6.0 for compatibility

wraith-sync fixes:
- Fix Tauri 2.0 plugin initialization error (same as wraith-chat)
- Add KDE Wayland workaround for Error 71

Both clients now build and run correctly on Linux with KDE Plasma.

Co-Authored-By: Claude Opus 4.5 &lt;noreply@anthropic.com&gt;
diff --git a/clients/wraith-chat/frontend/dist/index.html b/clients/wraith-chat/frontend/dist/index.html
@@ -1 +1,13 @@
-<\!DOCTYPE html><html><body>Placeholder</body></html>
+<!DOCTYPE html>
+<html lang="en">
+  <head>
+    <meta charset="UTF-8" />
+    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
+    <title>WRAITH Chat</title>
+    <script type="module" crossorigin src="/assets/index-BQjXt0nW.js"></script>
+    <link rel="stylesheet" crossorigin href="/assets/index-BrCg67_n.css">
+  </head>
+  <body>
+    <div id="root"></div>
+  </body>
+</html>
diff --git a/clients/wraith-chat/frontend/src/App.tsx b/clients/wraith-chat/frontend/src/App.tsx
@@ -1,8 +1,7 @@
 // Main App Component
 
-import React, { useEffect, useState } from 'react';
+import { useEffect, useState } from 'react';
 import { useConversationStore } from './stores/conversationStore';
-import { useMessageStore } from './stores/messageStore';
 import { useContactStore } from './stores/contactStore';
 import { useNodeStore } from './stores/nodeStore';
 import ConversationList from './components/ConversationList';
diff --git a/clients/wraith-chat/frontend/src/components/ConversationList.tsx b/clients/wraith-chat/frontend/src/components/ConversationList.tsx
@@ -1,6 +1,5 @@
 // Conversation List Component
 
-import React from 'react';
 import { useConversationStore } from '../stores/conversationStore';
 import { formatDistanceToNow } from 'date-fns';
 
diff --git a/clients/wraith-chat/frontend/src/components/GroupSettings.tsx b/clients/wraith-chat/frontend/src/components/GroupSettings.tsx
@@ -1,6 +1,6 @@
 // Group Settings Component - Sprint 17.7
 
-import React, { useEffect, useState, useCallback } from 'react';
+import { useEffect, useState } from 'react';
 import {
   useGroupStore,
   formatMemberCount,
diff --git a/clients/wraith-chat/frontend/src/components/MessageBubble.tsx b/clients/wraith-chat/frontend/src/components/MessageBubble.tsx
@@ -1,6 +1,5 @@
 // Message Bubble Component
 
-import React from 'react';
 import type { Message } from '../types';
 import { format } from 'date-fns';
 
diff --git a/clients/wraith-chat/frontend/src/components/VoiceCall.tsx b/clients/wraith-chat/frontend/src/components/VoiceCall.tsx
@@ -1,12 +1,11 @@
 // Voice Call Component - Sprint 17.5
 
-import React, { useEffect, useState, useCallback } from 'react';
+import { useEffect, useState, useCallback } from 'react';
 import {
   useCallStore,
   formatCallDuration,
   getCallStateText,
   type CallInfo,
-  type CallState,
 } from '../stores/callStore';
 
 interface VoiceCallProps {
diff --git a/clients/wraith-chat/frontend/tsconfig.node.json b/clients/wraith-chat/frontend/tsconfig.node.json
@@ -0,0 +1,11 @@
+{
+  "compilerOptions": {
+    "composite": true,
+    "skipLibCheck": true,
+    "module": "ESNext",
+    "moduleResolution": "bundler",
+    "allowSyntheticDefaultImports": true,
+    "strict": true
+  },
+  "include": ["vite.config.ts"]
+}
diff --git a/clients/wraith-chat/src-tauri/Cargo.toml b/clients/wraith-chat/src-tauri/Cargo.toml
@@ -16,12 +16,12 @@ path = "src/main.rs"
 
 [dependencies]
 # Tauri 2.0
-tauri = "2.0"
-tauri-plugin-dialog = "2.0"
-tauri-plugin-fs = "2.0"
-tauri-plugin-shell = "2.0"
-tauri-plugin-log = "2.0"
-tauri-plugin-notification = "2.0"
+tauri = { version = "2", features = [] }
+tauri-plugin-dialog = "2"
+tauri-plugin-fs = "2"
+tauri-plugin-shell = "2"
+tauri-plugin-log = "2"
+tauri-plugin-notification = "2"
 
 # WRAITH Protocol
 wraith-core = { path = "../../../crates/wraith-core" }
@@ -63,7 +63,9 @@ chrono = "0.4"
 uuid = { version = "1", features = ["v4"] }
 
 # Secure key storage (cross-platform keyring)
-keyring = "3"
+# sync-secret-service: Uses D-Bus Secret Service API (libsecret/GNOME Keyring)
+# crypto-rust: Pure Rust crypto for secret encryption over D-Bus
+keyring = { version = "3", features = ["sync-secret-service", "crypto-rust"] }
 
 # Voice Calling (Sprint 17.5)
 audiopus = "0.3.0-rc.0"
@@ -85,4 +87,4 @@ default = ["custom-protocol"]
 custom-protocol = ["tauri/custom-protocol"]
 
 [build-dependencies]
-tauri-build = { version = "2.0", features = [] }
+tauri-build = { version = "2", features = [] }
diff --git a/clients/wraith-chat/src-tauri/capabilities/default.json b/clients/wraith-chat/src-tauri/capabilities/default.json
@@ -0,0 +1,24 @@
+{
+  "$schema": "../gen/schemas/desktop-schema.json",
+  "identifier": "default",
+  "description": "enables the default permissions for WRAITH Chat",
+  "windows": [
+    "main"
+  ],
+  "permissions": [
+    "core:default",
+    "dialog:default",
+    "dialog:allow-open",
+    "dialog:allow-save",
+    "dialog:allow-message",
+    "dialog:allow-ask",
+    "fs:default",
+    "fs:allow-read-dir",
+    "fs:allow-read-file",
+    "fs:allow-write-file",
+    "shell:default",
+    "shell:allow-open",
+    "notification:default",
+    "notification:allow-notify"
+  ]
+}
diff --git a/clients/wraith-chat/src-tauri/gen/schemas/capabilities.json b/clients/wraith-chat/src-tauri/gen/schemas/capabilities.json
@@ -1 +1 @@
-{}
+{"default":{"identifier":"default","description":"enables the default permissions for WRAITH Chat","local":true,"windows":["main"],"permissions":["core:default","dialog:default","dialog:allow-open","dialog:allow-save","dialog:allow-message","dialog:allow-ask","fs:default","fs:allow-read-dir","fs:allow-read-file","fs:allow-write-file","shell:default","shell:allow-open","notification:default","notification:allow-notify"]}}
diff --git a/clients/wraith-chat/src-tauri/src/database.rs b/clients/wraith-chat/src-tauri/src/database.rs
@@ -1,11 +1,23 @@
 // SQLCipher Database for Encrypted Message Storage
 
-use anyhow::{Context, Result};
+use anyhow::{Context, Result, bail};
 use chrono::Utc;
 use rusqlite::{Connection, OptionalExtension, params};
 use serde::{Deserialize, Serialize};
 use std::path::Path;
 
+/// Error indicating the database exists but cannot be decrypted with the given key
+#[derive(Debug)]
+pub struct DatabaseKeyMismatchError;
+
+impl std::fmt::Display for DatabaseKeyMismatchError {
+    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+        write!(f, "Database exists but cannot be decrypted with the current key")
+    }
+}
+
+impl std::error::Error for DatabaseKeyMismatchError {}
+
 /// Database connection manager
 pub struct Database {
     /// The underlying SQLCipher connection
@@ -14,8 +26,14 @@ pub struct Database {
 
 impl Database {
     /// Open or create encrypted database
+    ///
+    /// Returns `DatabaseKeyMismatchError` if the database exists but the key is wrong.
+    /// The caller can handle this by backing up and recreating the database.
     pub fn open<P: AsRef<Path>>(path: P, password: &str) -> Result<Self> {
-        let conn = Connection::open(path)?;
+        let path_ref = path.as_ref();
+        let db_exists = path_ref.exists();
+
+        let conn = Connection::open(path_ref)?;
 
         // Set SQLCipher encryption key
         conn.pragma_update(None, "key", password)?;
@@ -26,6 +44,28 @@ impl Database {
         conn.pragma_update(None, "cipher_hmac_algorithm", "HMAC_SHA512")?;
         conn.pragma_update(None, "cipher_kdf_algorithm", "PBKDF2_HMAC_SHA512")?;
 
+        // Verify the key works by attempting a simple query
+        // This will fail with "file is not a database" if the key is wrong
+        if db_exists {
+            match conn.query_row("SELECT count(*) FROM sqlite_master", [], |_| Ok(())) {
+                Ok(_) => {
+                    log::debug!("Database key verified successfully");
+                }
+                Err(e) => {
+                    let error_msg = e.to_string();
+                    if error_msg.contains("file is not a database")
+                        || error_msg.contains("not a database")
+                        || error_msg.contains("hmac check failed")
+                    {
+                        log::error!("Database key mismatch: existing database cannot be decrypted");
+                        bail!(DatabaseKeyMismatchError);
+                    }
+                    // Other errors should be propagated normally
+                    return Err(e).context("Failed to verify database key");
+                }
+            }
+        }
+
         let db = Self { conn };
         db.create_tables()?;
 
diff --git a/clients/wraith-chat/src-tauri/src/lib.rs b/clients/wraith-chat/src-tauri/src/lib.rs
@@ -125,7 +125,44 @@ fn setup_app<R: Runtime>(app: &mut tauri::App<R>) -> Result<(), Box<dyn std::err
 
     log::info!("Opening database at: {:?}", db_path);
 
-    let db = database::Database::open(db_path, &password)?;
+    // Try to open the database, handling key mismatch errors
+    let db = match database::Database::open(&db_path, &password) {
+        Ok(db) => db,
+        Err(e) => {
+            // Check if this is a key mismatch error
+            if e.downcast_ref::<database::DatabaseKeyMismatchError>().is_some() {
+                log::warn!(
+                    "Database key mismatch detected. The encryption key has changed since the database was created."
+                );
+
+                // Backup the old database with a timestamp to avoid overwriting previous backups
+                let timestamp = chrono::Utc::now().format("%Y%m%d_%H%M%S");
+                let backup_filename = format!("wraith_chat.db.backup.{}", timestamp);
+                let backup_path = app_dir.join(&backup_filename);
+
+                if let Err(backup_err) = std::fs::rename(&db_path, &backup_path) {
+                    log::error!("Failed to backup old database: {}", backup_err);
+                    return Err(Box::new(std::io::Error::other(format!(
+                        "Database key mismatch and failed to backup: {}. \
+                        Please manually remove the database at {:?}",
+                        backup_err, db_path
+                    ))));
+                }
+
+                log::info!(
+                    "Old database backed up to {:?}. Creating new database with current key.",
+                    backup_path
+                );
+
+                // Try to open a fresh database
+                database::Database::open(&db_path, &password)?
+            } else {
+                // Other errors should be propagated
+                return Err(e.into());
+            }
+        }
+    };
+
     let state = Arc::new(state::AppState::new(db));
 
     app.manage(state);
diff --git a/clients/wraith-chat/src-tauri/src/main.rs b/clients/wraith-chat/src-tauri/src/main.rs
@@ -2,5 +2,75 @@
 #![cfg_attr(not(debug_assertions), windows_subsystem = "windows")]
 
 fn main() {
+    // Workaround for Wayland Error 71 (Protocol error) on KDE Plasma 6
+    // See: https://github.com/tauri-apps/tauri/issues/10702
+    //      https://github.com/tauri-apps/tao/issues/977
+    //
+    // WebKitGTK has compatibility issues with Wayland on KDE Plasma 6, causing
+    // "Error 71 (Protocol error) dispatching to Wayland display" crashes.
+    // This is an upstream issue blocked by tao/webkit2gtk compatibility.
+    //
+    // Solution: Automatically fallback to X11 via XWayland if:
+    // 1. We're on Linux
+    // 2. GDK_BACKEND is not already set (respect user preference)
+    // 3. We're in a Wayland session
+    // 4. We're on KDE Plasma 6 (or any Wayland compositor with issues)
+    #[cfg(target_os = "linux")]
+    {
+        use std::env;
+
+        // Only set GDK_BACKEND if not already configured by user
+        if env::var("GDK_BACKEND").is_err() {
+            // Check if we're in a Wayland session
+            if let Ok(session_type) = env::var("XDG_SESSION_TYPE") {
+                if session_type == "wayland" {
+                    // Check for KDE Plasma (common source of Error 71)
+                    let is_kde = env::var("KDE_SESSION_VERSION").is_ok()
+                        || env::var("KDE_FULL_SESSION").is_ok()
+                        || env::var("DESKTOP_SESSION")
+                            .map(|s| s.contains("plasma") || s.contains("kde"))
+                            .unwrap_or(false);
+
+                    if is_kde {
+                        eprintln!(
+                            "Detected KDE Plasma on Wayland - forcing X11 backend to avoid Error 71"
+                        );
+                        eprintln!("See: https://github.com/tauri-apps/tauri/issues/10702");
+                        // SAFETY: We're in main() before any threads are spawned,
+                        // so there's no risk of data races with other threads reading env vars
+                        unsafe {
+                            env::set_var("GDK_BACKEND", "x11");
+                        }
+                    } else {
+                        // For other Wayland compositors, prefer Wayland but fallback to X11
+                        // This allows GDK to try Wayland first, then X11 if issues occur
+                        // SAFETY: We're in main() before any threads are spawned,
+                        // so there's no risk of data races with other threads reading env vars
+                        unsafe {
+                            env::set_var("GDK_BACKEND", "wayland,x11");
+                        }
+                    }
+                }
+            }
+        }
+
+        // Workaround for GBM (Generic Buffer Management) errors
+        // See: https://github.com/tauri-apps/tauri/issues/13493
+        //      https://github.com/winfunc/opcode/issues/26
+        //
+        // WebKitGTK's hardware-accelerated compositing can fail with:
+        // "Failed to create GBM buffer of size WxH: Invalid argument"
+        //
+        // This occurs due to incompatibility between WebKitGTK, Mesa, and GPU drivers
+        // (especially NVIDIA). Disabling compositing mode forces WebKit to use a
+        // simpler, more compatible rendering path.
+        if env::var("WEBKIT_DISABLE_COMPOSITING_MODE").is_err() {
+            // SAFETY: We're in main() before any threads are spawned
+            unsafe {
+                env::set_var("WEBKIT_DISABLE_COMPOSITING_MODE", "1");
+            }
+        }
+    }
+
     wraith_chat_lib::run();
 }
diff --git a/clients/wraith-chat/src-tauri/tauri.conf.json b/clients/wraith-chat/src-tauri/tauri.conf.json
@@ -1,4 +1,5 @@
 {
+  "$schema": "https://schema.tauri.app/config/2",
   "productName": "WRAITH Chat",
   "version": "1.7.0",
   "identifier": "com.wraith.chat",
@@ -50,20 +51,5 @@
       "csp": "default-src 'self'; style-src 'self' 'unsafe-inline'; script-src 'self' 'unsafe-inline'"
     }
   },
-  "plugins": {
-    "dialog": {},
-    "fs": {
-      "scope": ["$APPDATA/*", "$APPDATA/**"]
-    },
-    "shell": {
-      "open": true
-    },
-    "log": {
-      "targets": ["stdout", "logfile"],
-      "maxFileSize": 5242880
-    },
-    "notification": {
-      "all": true
-    }
-  }
+  "plugins": {}
 }
diff --git a/clients/wraith-sync/src-tauri/capabilities/default.json b/clients/wraith-sync/src-tauri/capabilities/default.json
@@ -0,0 +1,24 @@
+{
+  "$schema": "../gen/schemas/desktop-schema.json",
+  "identifier": "default",
+  "description": "enables the default permissions for WRAITH Sync",
+  "windows": [
+    "main"
+  ],
+  "permissions": [
+    "core:default",
+    "dialog:default",
+    "dialog:allow-open",
+    "dialog:allow-save",
+    "dialog:allow-message",
+    "dialog:allow-ask",
+    "fs:default",
+    "fs:allow-read-dir",
+    "fs:allow-read-file",
+    "fs:allow-write-file",
+    "shell:default",
+    "shell:allow-open",
+    "notification:default",
+    "notification:allow-notify"
+  ]
+}
diff --git a/clients/wraith-sync/src-tauri/gen/schemas/capabilities.json b/clients/wraith-sync/src-tauri/gen/schemas/capabilities.json
@@ -1 +1 @@
-{}
+{"default":{"identifier":"default","description":"enables the default permissions for WRAITH Sync","local":true,"windows":["main"],"permissions":["core:default","dialog:default","dialog:allow-open","dialog:allow-save","dialog:allow-message","dialog:allow-ask","fs:default","fs:allow-read-dir","fs:allow-read-file","fs:allow-write-file","shell:default","shell:allow-open","notification:default","notification:allow-notify"]}}
diff --git a/clients/wraith-sync/src-tauri/src/lib.rs b/clients/wraith-sync/src-tauri/src/lib.rs
@@ -36,7 +36,7 @@ pub fn run() {
         .plugin(tauri_plugin_dialog::init())
         .plugin(tauri_plugin_fs::init())
         .plugin(tauri_plugin_shell::init())
-        .plugin(tauri_plugin_log::Builder::default().build())
+        // Note: tauri_plugin_log removed - using tracing_subscriber directly
         .plugin(tauri_plugin_notification::init())
         .setup(|app| {
             setup_app(app)?;
diff --git a/clients/wraith-sync/src-tauri/src/main.rs b/clients/wraith-sync/src-tauri/src/main.rs
diff --git a/clients/wraith-sync/src-tauri/tauri.conf.json b/clients/wraith-sync/src-tauri/tauri.conf.json

Original file line number	Diff line number	Diff line change
`@@ -1 +1 @@`
`1`		`-{}`
	`1`	`+{"default":{"identifier":"default","description":"enables the default permissions for WRAITH Chat","local":true,"windows":["main"],"permissions":["core:default","dialog:default","dialog:allow-open","dialog:allow-save","dialog:allow-message","dialog:allow-ask","fs:default","fs:allow-read-dir","fs:allow-read-file","fs:allow-write-file","shell:default","shell:allow-open","notification:default","notification:allow-notify"]}}`