chore(deps): bump actions/checkout from 6.0.2 to 6.0.3 (#155)

* chore(deps): bump actions/checkout from 6.0.2 to 6.0.3

Bumps [actions/checkout](https://github.com/actions/checkout) from 6.0.2 to 6.0.3.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@de0fac2...df4cb1c)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: 6.0.3
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

* fix: run cargo fmt and add missing fields to test struct

* fix: correct undo/redo logic for cell edits and insert rows

* fix: add missing ConnectionProfile fields to integration_tests.rs

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Elliot <ElliotWorth@protonmail.com>

Author: dependabot[bot]

.github/workflows/ci.yml

@@ -13,7 +13,7 @@ jobs:
     name: Dependency Age Gate
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6.0.3
       - uses: dtolnay/rust-toolchain@29eef336d9b2848a0b548edc03f92a220660cdb8 # stable
       - uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e # v6.4.0
         with:
@@ -28,7 +28,7 @@ jobs:
     name: Lint (TypeScript)
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6.0.3
       - uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e # v6.4.0
         with:
           node-version: '20'
@@ -44,7 +44,7 @@ jobs:
     name: Lint (Rust)
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6.0.3
       - uses: dtolnay/rust-toolchain@29eef336d9b2848a0b548edc03f92a220660cdb8 # stable
         with:
           components: rustfmt, clippy
@@ -82,7 +82,7 @@ jobs:
           --health-timeout 5s
           --health-retries 10
     steps:
-      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6.0.3
       - uses: dtolnay/rust-toolchain@29eef336d9b2848a0b548edc03f92a220660cdb8 # stable
       - uses: Swatinem/rust-cache@c19371144df3bb44fab255c43d04cbc2ab54d1c4 # v2.9.1
         with:
@@ -106,7 +106,7 @@ jobs:
     name: Test Frontend
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6.0.3
       - uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e # v6.4.0
         with:
           node-version: '20'
@@ -133,7 +133,7 @@ jobs:
     runs-on: ubuntu-latest
     needs: [test-rust, test-frontend]
     steps:
-      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6.0.3
       - uses: dtolnay/rust-toolchain@29eef336d9b2848a0b548edc03f92a220660cdb8 # stable
       - uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e # v6.4.0
         with:

.github/workflows/release.yml

@@ -31,7 +31,7 @@ jobs:
 
     steps:
       - name: Checkout
-        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+        uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6.0.3
 
       - name: Install Linux dependencies
         if: matrix.platform == 'ubuntu-22.04'

src-tauri/crates/mas-admin/src/lib.rs

@@ -101,10 +101,7 @@ impl AdminService {
     ) -> Result<(), CoreError> {
         tracing::info!(process_id = process_id, "Killing MySQL process");
         let pool = self.connection_manager.get_pool(connection_id)?;
-        let result = sqlx::query("KILL ?")
-            .bind(process_id)
-            .execute(&pool)
-            .await;
+        let result = sqlx::query("KILL ?").bind(process_id).execute(&pool).await;
         match result {
             Ok(_) => {
                 tracing::info!(process_id = process_id, "Process killed successfully");

src-tauri/crates/mas-core/src/connection/manager.rs

@@ -35,7 +35,10 @@ impl ConnectionManager {
             "Creating connection pool"
         );
 
-        let charset = profile.charset.clone().unwrap_or_else(|| "utf8mb4".to_string());
+        let charset = profile
+            .charset
+            .clone()
+            .unwrap_or_else(|| "utf8mb4".to_string());
         let mut options = MySqlConnectOptions::new()
             .host(&profile.host)
             .port(profile.port)
@@ -64,7 +67,9 @@ impl ConnectionManager {
             .after_connect(move |conn, _meta| {
                 let charset = charset_for_after_connect.clone();
                 Box::pin(async move {
-                    sqlx::query(&format!("SET NAMES {}", charset)).execute(&mut *conn).await?;
+                    sqlx::query(&format!("SET NAMES {}", charset))
+                        .execute(&mut *conn)
+                        .await?;
                     Ok(())
                 })
             })
@@ -150,7 +155,10 @@ impl ConnectionManager {
     ) -> Result<TestConnectionResult, CoreError> {
         let start = Instant::now();
 
-        let charset = profile.charset.clone().unwrap_or_else(|| "utf8mb4".to_string());
+        let charset = profile
+            .charset
+            .clone()
+            .unwrap_or_else(|| "utf8mb4".to_string());
         let mut options = MySqlConnectOptions::new()
             .host(&profile.host)
             .port(profile.port)

src-tauri/crates/mas-core/src/connection/store.rs

@@ -55,12 +55,21 @@ impl ConnectionStore {
         db.execute("ALTER TABLE connection_profiles ADD COLUMN env TEXT", [])
             .ok();
         // Migration: add advanced settings columns if missing
-        db.execute("ALTER TABLE connection_profiles ADD COLUMN connect_timeout_secs INTEGER", [])
-            .ok();
-        db.execute("ALTER TABLE connection_profiles ADD COLUMN query_timeout_secs INTEGER", [])
-            .ok();
-        db.execute("ALTER TABLE connection_profiles ADD COLUMN charset TEXT", [])
-            .ok();
+        db.execute(
+            "ALTER TABLE connection_profiles ADD COLUMN connect_timeout_secs INTEGER",
+            [],
+        )
+        .ok();
+        db.execute(
+            "ALTER TABLE connection_profiles ADD COLUMN query_timeout_secs INTEGER",
+            [],
+        )
+        .ok();
+        db.execute(
+            "ALTER TABLE connection_profiles ADD COLUMN charset TEXT",
+            [],
+        )
+        .ok();
         tracing::debug!("Connection profiles table initialized");
         Ok(())
     }

src-tauri/crates/mas-core/tests/integration_tests.rs

@@ -42,6 +42,9 @@ fn test_profile() -> ConnectionProfile {
         pool_min: 1,
         pool_max: 5,
         read_only: false,
+        connect_timeout_secs: None,
+        query_timeout_secs: None,
+        charset: None,
         environment: None,
         created_at: Utc::now(),
         updated_at: Utc::now(),

src-tauri/crates/mas-core/tests/query_database_context_regression.rs

@@ -20,6 +20,9 @@ fn test_profile() -> ConnectionProfile {
         pool_min: 1,
         pool_max: 5,
         read_only: false,
+        connect_timeout_secs: None,
+        query_timeout_secs: None,
+        charset: None,
         environment: None,
         created_at: Utc::now(),
         updated_at: Utc::now(),

src-tauri/src/commands/mod.rs

@@ -534,7 +534,9 @@ pub async fn write_file_contents(path: String, contents: String) -> Result<(), S
                 .ok_or_else(|| "Invalid path: missing file name".to_string())?,
         )
     } else {
-        path_buf.canonicalize().map_err(|e| format!("Invalid path: {e}"))?
+        path_buf
+            .canonicalize()
+            .map_err(|e| format!("Invalid path: {e}"))?
     };
     tokio::fs::write(&resolved, &contents).await.map_err(|e| {
         tracing::error!(error = %e, path = %path, "Failed to write file");

src/components/editor/__tests__/QueryToolbar.test.tsx

@@ -386,10 +386,10 @@ describe("QueryToolbar", () => {
     expect(screen.getByText("Save").closest("button")).toBeDisabled();
   });
 
-  it("shows Running state when executing", () => {
+  it("shows Cancel button when executing", () => {
     mockIsExecuting = true;
     render(<QueryToolbar />);
-    expect(screen.getByText("Running...")).toBeInTheDocument();
+    expect(screen.getByText("Cancel")).toBeInTheDocument();
     expect(screen.queryByText("Run")).not.toBeInTheDocument();
   });
 

src/hooks/useGridEditing.ts

@@ -134,6 +134,7 @@ export function useGridEditing() {
     setDeletes(new Set());
     undoStack.current = [];
     redoStack.current = [];
+    bumpVersion();
   }, []);
 
   const applyAction = useCallback((action: EditAction): EditAction => {
@@ -143,7 +144,11 @@ export function useGridEditing() {
           const next = new Map(prev);
           const rowChanges = [...(next.get(action.rowIndex) ?? [])];
           const existing = rowChanges.findIndex((c) => c.column === action.column);
-          if (action.newValue === action.oldValue || (action.oldValue === undefined && action.newValue === action.oldValue)) {
+          if (existing >= 0 && action.newValue === rowChanges[existing].originalValue) {
+            rowChanges.splice(existing, 1);
+            if (rowChanges.length === 0) next.delete(action.rowIndex);
+            else next.set(action.rowIndex, rowChanges);
+          } else if (action.newValue === action.oldValue) {
             if (existing >= 0) {
               rowChanges.splice(existing, 1);
               if (rowChanges.length === 0) next.delete(action.rowIndex);
@@ -172,6 +177,10 @@ export function useGridEditing() {
         return { ...action, index: -1 };
       }
       case "deleteRow": {
+        if (action.rowIndex === -1) {
+          setInserts((prev) => prev.slice(0, -1));
+          return { type: "insertRow", index: 0 }; // forward action for redo
+        }
         setDeletes((prev: Set<number>) => {
           const next = new Set(prev);
           if (next.has(action.rowIndex)) next.delete(action.rowIndex);
@@ -194,7 +203,9 @@ export function useGridEditing() {
   const redo = useCallback(() => {
     const action = redoStack.current.pop();
     if (!action) return;
-    const reverse = applyAction(reverseAction(action));
+    const reverse = action.type === "insertRow"
+      ? applyAction(action)
+      : applyAction(reverseAction(action));
     undoStack.current.push(reverse);
     bumpVersion();
   }, [applyAction]);