.gitea/workflows/release-beta.yml

@@ -317,10 +317,6 @@ jobs:
           CARGO_TARGET_X86_64_PC_WINDOWS_GNU_LINKER: x86_64-w64-mingw32-gcc
           OPENSSL_NO_VENDOR: "0"
           OPENSSL_STATIC: "1"
-          # Fix memset_explicit missing symbol for libsodium on MinGW
-          CFLAGS_x86_64_pc_windows_gnu: "-D_WIN32_WINNT=0x0602"
-          SODIUM_LIB_DIR: ""
-          SODIUM_STATIC: "yes"
         run: |
           npm ci --legacy-peer-deps
           CI=true npx tauri build --target x86_64-pc-windows-gnu

.gitea/workflows/test.yml

@@ -48,17 +48,8 @@ jobs:
             pkg-config
       - name: Install Rust components
         run: rustup component add rustfmt
-      - name: Install dependencies with retry
+      - name: Install dependencies
-        run: |
+        run: npm install --legacy-peer-deps
-          for i in 1 2 3; do
-            if npm install --legacy-peer-deps --prefer-offline --no-audit; then
-              exit 0
-            fi
-            echo "Attempt $i failed, retrying in 5 seconds..."
-            sleep 5
-          done
-          echo "All retry attempts failed"
-          exit 1
       - name: Update version from Git
         run: node scripts/update-version.mjs
       - run: cargo generate-lockfile --manifest-path src-tauri/Cargo.toml
@@ -170,17 +161,7 @@ jobs:
           key: ${{ runner.os }}-npm-${{ hashFiles('**/package-lock.json') }}
           restore-keys: |
             ${{ runner.os }}-npm-
-      - name: Install dependencies with retry
+      - run: npm ci --legacy-peer-deps
-        run: |
-          for i in 1 2 3; do
-            if npm ci --legacy-peer-deps --prefer-offline --no-audit; then
-              exit 0
-            fi
-            echo "Attempt $i failed, retrying in 5 seconds..."
-            sleep 5
-          done
-          echo "All retry attempts failed"
-          exit 1
       - run: npx tsc --noEmit
 
   frontend-tests:
@@ -214,15 +195,5 @@ jobs:
           key: ${{ runner.os }}-npm-${{ hashFiles('**/package-lock.json') }}
           restore-keys: |
             ${{ runner.os }}-npm-
-      - name: Install dependencies with retry
+      - run: npm ci --legacy-peer-deps
-        run: |
-          for i in 1 2 3; do
-            if npm ci --legacy-peer-deps --prefer-offline --no-audit; then
-              exit 0
-            fi
-            echo "Attempt $i failed, retrying in 5 seconds..."
-            sleep 5
-          done
-          echo "All retry attempts failed"
-          exit 1
       - run: npm run test:run

REVIEW_FIX_SUMMARY.md

@@ -1,157 +0,0 @@
-# Review Feedback Fix Summary
-
-## Ticket Context
-**Branch**: `fix/proxmox-remote-add-error`
-**Original Issue**: Proxmox remote URLs with ports (e.g., `https://172.0.0.18:8006`) were incorrectly parsed
-
-## Automated Review Feedback
-
-The automated PR review (qwen3-coder-next via liteLLM) identified two issues:
-
-### Issue 1: Code Duplication (WARNING)
-- **Location**: `src/pages/Proxmox/RemotesPage.tsx:78-84` and `105-112`
-- **Problem**: Port parsing logic duplicated in `handleAddRemote` and `handleEditRemote`
-- **Impact**: Risk of logic drift, harder maintenance
-
-### Issue 2: Atomicity Concern (WARNING)
-- **Location**: `src/pages/Proxmox/RemotesPage.tsx:105-112`
-- **Problem**: Edit flow uses remove-then-add pattern; if add fails after remove, remote is lost
-- **Impact**: Potential data loss if second operation fails
-
-## Resolution
-
-### Fix 1: Extracted Helper Function ✅
-
-Created `parseRemoteUrl()` helper function to eliminate duplication:
-
-```typescript
-/**
- * Helper function to parse a Proxmox URL and extract hostname and port.
- * Handles URLs with or without explicit port numbers.
- *
- * @param url - The full URL (e.g., "https://172.0.0.18:8006" or "https://pve.example.com")
- * @param type - The cluster type ('pve' or 'pbs') to determine default port
- * @returns Object with hostname (stripped of protocol and port) and port number
- */
-const parseRemoteUrl = (url: string, type: 'pve' | 'pbs'): { hostname: string; port: number } => {
-  let hostname = url.replace(/^https?:\/\//, '');
-  let port = type === 'pve' ? 8006 : 8007;
-
-  const portMatch = hostname.match(/:(\d+)$/);
-  if (portMatch) {
-    port = parseInt(portMatch[1], 10);
-    hostname = hostname.replace(/:\d+$/, '');
-  }
-
-  return { hostname, port };
-};
-```
-
-**Benefits:**
-- Single source of truth
-- Prevents logic drift
-- Well-documented
-- Easy to test and maintain
-- Type-safe return value
-
-### Fix 2: Documented Known Limitation ✅
-
-Added comment in `handleEditRemote` documenting the architectural limitation:
-
-```typescript
-// Edit operation requires remove-then-add since backend doesn't support update.
-// If add fails after remove, the remote will be lost - this is a known limitation
-// until backend supports atomic update operations.
-await removeProxmoxCluster(config.id);
-await addProxmoxCluster(/* ... */);
-```
-
-**Rationale:**
-- Backend lacks atomic update operation (`updateProxmoxCluster()`)
-- Frontend rollback would be complex and error-prone
-- Proper fix belongs in backend layer
-- Risk is low-moderate (edit operations are infrequent)
-- Clear failure mode (remote disappears, error toast shown)
-- User can manually re-add if needed
-
-**Alternative considered and rejected:**
-- Implementing frontend-side rollback: Too complex, would require caching all values, handling partial failures, managing state consistency
-- Removing edit capability: Worse UX than documented limitation
-
-## Pre-existing Issue Fixed
-
-During verification, discovered missing `node_modules` dependencies causing TypeScript errors:
-- **Problem**: `sonner` and `monaco-editor` packages not installed
-- **Root cause**: ESLint peer dependency conflict preventing `npm install`
-- **Solution**: Ran `npm install --legacy-peer-deps` to resolve
-
-## Verification Results
-
-### All Checks Passing ✅
-
-**Frontend:**
-- ✅ ESLint: No issues found
-- ✅ TypeScript: No errors found (`npx tsc --noEmit`)
-- ✅ Frontend tests: 386 passed, 0 failed (45 test files)
-
-**Backend:**
-- ✅ Rust tests: 413 passed, 6 ignored, 0 failed
-- ✅ Cargo fmt: Formatting correct
-- ✅ Cargo clippy: No warnings
-
-**Code Quality:**
-- ✅ Duplication eliminated via helper function
-- ✅ Known limitation documented with clear comment
-- ✅ Dependencies resolved
-
-## Code Changes Summary
-
-**Files Modified:**
-1. `src/pages/Proxmox/RemotesPage.tsx` (+26 lines, -22 lines)
-   - Added `parseRemoteUrl()` helper function with JSDoc
-   - Refactored `handleAddRemote()` to use helper
-   - Refactored `handleEditRemote()` to use helper
-   - Added limitation comment in `handleEditRemote()`
-
-2. `package-lock.json` (dependency updates)
-   - Installed missing `sonner` and `monaco-editor` packages
-   - Used `--legacy-peer-deps` to resolve ESLint conflicts
-
-## Recommendation
-
-**APPROVE**: Both review concerns have been addressed:
-1. Code duplication eliminated with well-tested helper function
-2. Atomicity limitation documented as architectural constraint
-
-The proper long-term fix (backend `updateProxmoxCluster()` operation) should be tracked in a separate ticket.
-
-## Follow-up Tasks
-
-1. **Backend**: Implement `updateProxmoxCluster()` command in Rust
-   - Add atomic update operation to `src-tauri/src/commands/proxmox.rs`
-   - Use single SQL transaction for update
-   - Add Tauri command `#[tauri::command]`
-   - Update frontend to use new command when available
-
-2. **Dependencies**: Consider upgrading ESLint to avoid `--legacy-peer-deps`
-   - Track ESLint plugin compatibility
-   - Test with newer versions
-
-## Testing Performed
-
-- ✅ All automated tests pass
-- ✅ Linting passes
-- ✅ Type checking passes
-- ✅ Manual code review of changes
-- ✅ Helper function logic verified (preserves original behavior)
-- ✅ Comment clarity verified
-
-## Risk Assessment
-
-**Risk Level**: Low
-- Changes are refactoring with no behavior modification
-- All tests pass
-- Known limitation is clearly documented
-- Helper function is simple and well-tested
-
-**Merge Confidence**: High

package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "trcaa",
-  "version": "1.2.4",
+  "version": "1.1.0",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "trcaa",
-      "version": "1.2.4",
+      "version": "1.1.0",
       "dependencies": {
         "@eslint-react/eslint-plugin": "^5.8.16",
         "@monaco-editor/react": "^4.7.0",

package.json

@@ -1,7 +1,7 @@
 {
   "name": "trcaa",
   "private": true,
-  "version": "1.2.4",
+  "version": "1.2.3",
   "type": "module",
   "scripts": {
     "dev": "vite",

src-tauri/.cargo/config.toml

@@ -9,7 +9,3 @@ rustflags = ["-C", "link-arg=-Wl,--exclude-all-symbols"]
 # Use system OpenSSL instead of vendoring from source (which requires Perl modules
 # unavailable on some environments and breaks clippy/check).
 OPENSSL_NO_VENDOR = "1"
-
-# Force libsodium to use minimal mode which avoids memset_explicit on Windows
-SODIUM_USE_PKG_CONFIG = "0"
-SODIUM_STATIC = "1"

src-tauri/Cargo.lock

@@ -6528,14 +6528,13 @@ dependencies = [
 
 [[package]]
 name = "trcaa"
-version = "1.2.4"
+version = "1.2.2"
 dependencies = [
  "aes-gcm",
  "aho-corasick",
  "anyhow",
  "async-trait",
  "base64 0.22.1",
- "cc",
  "chrono",
  "dirs 5.0.1",
  "docx-rs",

src-tauri/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "trcaa"
-version = "1.2.4"
+version = "1.2.3"
 edition = "2021"
 
 [lib]
@@ -9,7 +9,6 @@ crate-type = ["staticlib", "cdylib", "rlib"]
 
 [build-dependencies]
 tauri-build = { version = "2.6", features = [] }
-cc = "1.0"
 
 [dependencies]
 tauri = { version = "2", features = [] }

src-tauri/build.rs

@@ -5,16 +5,6 @@ fn main() {
     println!("cargo:rerun-if-changed=.git/refs/heads/master");
     println!("cargo:rerun-if-changed=.git/refs/tags");
 
-    // Compile memset_explicit shim for Windows MinGW
-    if std::env::var("CARGO_CFG_TARGET_OS").unwrap_or_default() == "windows"
-        && std::env::var("CARGO_CFG_TARGET_ENV").unwrap_or_default() == "gnu"
-    {
-        cc::Build::new()
-            .file("memset_s_shim.c")
-            .compile("memset_shim");
-        println!("cargo:rerun-if-changed=memset_s_shim.c");
-    }
-
     tauri_build::build()
 }
 

src-tauri/gen/schemas/macOS-schema.json

@@ -2096,174 +2096,6 @@
                 }
               }
             },
-            {
-              "if": {
-                "properties": {
-                  "identifier": {
-                    "anyOf": [
-                      {
-                        "description": "This permission set allows opening `mailto:`, `tel:`, `https://` and `http://` urls using their default application\nas well as reveal file in directories using default file explorer\n#### This default permission set includes:\n\n- `allow-open-url`\n- `allow-reveal-item-in-dir`\n- `allow-default-urls`",
-                        "type": "string",
-                        "const": "opener:default",
-                        "markdownDescription": "This permission set allows opening `mailto:`, `tel:`, `https://` and `http://` urls using their default application\nas well as reveal file in directories using default file explorer\n#### This default permission set includes:\n\n- `allow-open-url`\n- `allow-reveal-item-in-dir`\n- `allow-default-urls`"
-                      },
-                      {
-                        "description": "This enables opening `mailto:`, `tel:`, `https://` and `http://` urls using their default application.",
-                        "type": "string",
-                        "const": "opener:allow-default-urls",
-                        "markdownDescription": "This enables opening `mailto:`, `tel:`, `https://` and `http://` urls using their default application."
-                      },
-                      {
-                        "description": "Enables the open_path command without any pre-configured scope.",
-                        "type": "string",
-                        "const": "opener:allow-open-path",
-                        "markdownDescription": "Enables the open_path command without any pre-configured scope."
-                      },
-                      {
-                        "description": "Enables the open_url command without any pre-configured scope.",
-                        "type": "string",
-                        "const": "opener:allow-open-url",
-                        "markdownDescription": "Enables the open_url command without any pre-configured scope."
-                      },
-                      {
-                        "description": "Enables the reveal_item_in_dir command without any pre-configured scope.",
-                        "type": "string",
-                        "const": "opener:allow-reveal-item-in-dir",
-                        "markdownDescription": "Enables the reveal_item_in_dir command without any pre-configured scope."
-                      },
-                      {
-                        "description": "Denies the open_path command without any pre-configured scope.",
-                        "type": "string",
-                        "const": "opener:deny-open-path",
-                        "markdownDescription": "Denies the open_path command without any pre-configured scope."
-                      },
-                      {
-                        "description": "Denies the open_url command without any pre-configured scope.",
-                        "type": "string",
-                        "const": "opener:deny-open-url",
-                        "markdownDescription": "Denies the open_url command without any pre-configured scope."
-                      },
-                      {
-                        "description": "Denies the reveal_item_in_dir command without any pre-configured scope.",
-                        "type": "string",
-                        "const": "opener:deny-reveal-item-in-dir",
-                        "markdownDescription": "Denies the reveal_item_in_dir command without any pre-configured scope."
-                      }
-                    ]
-                  }
-                }
-              },
-              "then": {
-                "properties": {
-                  "allow": {
-                    "items": {
-                      "title": "OpenerScopeEntry",
-                      "description": "Opener scope entry.",
-                      "anyOf": [
-                        {
-                          "type": "object",
-                          "required": [
-                            "url"
-                          ],
-                          "properties": {
-                            "app": {
-                              "description": "An application to open this url with, for example: firefox.",
-                              "allOf": [
-                                {
-                                  "$ref": "#/definitions/Application"
-                                }
-                              ]
-                            },
-                            "url": {
-                              "description": "A URL that can be opened by the webview when using the Opener APIs.\n\nWildcards can be used following the UNIX glob pattern.\n\nExamples:\n\n- \"https://*\" : allows all HTTPS origin\n\n- \"https://*.github.com/tauri-apps/tauri\": allows any subdomain of \"github.com\" with the \"tauri-apps/api\" path\n\n- \"https://myapi.service.com/users/*\": allows access to any URLs that begins with \"https://myapi.service.com/users/\"",
-                              "type": "string"
-                            }
-                          }
-                        },
-                        {
-                          "type": "object",
-                          "required": [
-                            "path"
-                          ],
-                          "properties": {
-                            "app": {
-                              "description": "An application to open this path with, for example: xdg-open.",
-                              "allOf": [
-                                {
-                                  "$ref": "#/definitions/Application"
-                                }
-                              ]
-                            },
-                            "path": {
-                              "description": "A path that can be opened by the webview when using the Opener APIs.\n\nThe pattern can start with a variable that resolves to a system base directory. The variables are: `$AUDIO`, `$CACHE`, `$CONFIG`, `$DATA`, `$LOCALDATA`, `$DESKTOP`, `$DOCUMENT`, `$DOWNLOAD`, `$EXE`, `$FONT`, `$HOME`, `$PICTURE`, `$PUBLIC`, `$RUNTIME`, `$TEMPLATE`, `$VIDEO`, `$RESOURCE`, `$APP`, `$LOG`, `$TEMP`, `$APPCONFIG`, `$APPDATA`, `$APPLOCALDATA`, `$APPCACHE`, `$APPLOG`.",
-                              "type": "string"
-                            }
-                          }
-                        }
-                      ]
-                    }
-                  },
-                  "deny": {
-                    "items": {
-                      "title": "OpenerScopeEntry",
-                      "description": "Opener scope entry.",
-                      "anyOf": [
-                        {
-                          "type": "object",
-                          "required": [
-                            "url"
-                          ],
-                          "properties": {
-                            "app": {
-                              "description": "An application to open this url with, for example: firefox.",
-                              "allOf": [
-                                {
-                                  "$ref": "#/definitions/Application"
-                                }
-                              ]
-                            },
-                            "url": {
-                              "description": "A URL that can be opened by the webview when using the Opener APIs.\n\nWildcards can be used following the UNIX glob pattern.\n\nExamples:\n\n- \"https://*\" : allows all HTTPS origin\n\n- \"https://*.github.com/tauri-apps/tauri\": allows any subdomain of \"github.com\" with the \"tauri-apps/api\" path\n\n- \"https://myapi.service.com/users/*\": allows access to any URLs that begins with \"https://myapi.service.com/users/\"",
-                              "type": "string"
-                            }
-                          }
-                        },
-                        {
-                          "type": "object",
-                          "required": [
-                            "path"
-                          ],
-                          "properties": {
-                            "app": {
-                              "description": "An application to open this path with, for example: xdg-open.",
-                              "allOf": [
-                                {
-                                  "$ref": "#/definitions/Application"
-                                }
-                              ]
-                            },
-                            "path": {
-                              "description": "A path that can be opened by the webview when using the Opener APIs.\n\nThe pattern can start with a variable that resolves to a system base directory. The variables are: `$AUDIO`, `$CACHE`, `$CONFIG`, `$DATA`, `$LOCALDATA`, `$DESKTOP`, `$DOCUMENT`, `$DOWNLOAD`, `$EXE`, `$FONT`, `$HOME`, `$PICTURE`, `$PUBLIC`, `$RUNTIME`, `$TEMPLATE`, `$VIDEO`, `$RESOURCE`, `$APP`, `$LOG`, `$TEMP`, `$APPCONFIG`, `$APPDATA`, `$APPLOCALDATA`, `$APPCACHE`, `$APPLOG`.",
-                              "type": "string"
-                            }
-                          }
-                        }
-                      ]
-                    }
-                  }
-                }
-              },
-              "properties": {
-                "identifier": {
-                  "description": "Identifier of the permission or permission set.",
-                  "allOf": [
-                    {
-                      "$ref": "#/definitions/Identifier"
-                    }
-                  ]
-                }
-              }
-            },
             {
               "if": {
                 "properties": {
@@ -6416,54 +6248,6 @@
           "const": "http:deny-fetch-send",
           "markdownDescription": "Denies the fetch_send command without any pre-configured scope."
         },
-        {
-          "description": "This permission set allows opening `mailto:`, `tel:`, `https://` and `http://` urls using their default application\nas well as reveal file in directories using default file explorer\n#### This default permission set includes:\n\n- `allow-open-url`\n- `allow-reveal-item-in-dir`\n- `allow-default-urls`",
-          "type": "string",
-          "const": "opener:default",
-          "markdownDescription": "This permission set allows opening `mailto:`, `tel:`, `https://` and `http://` urls using their default application\nas well as reveal file in directories using default file explorer\n#### This default permission set includes:\n\n- `allow-open-url`\n- `allow-reveal-item-in-dir`\n- `allow-default-urls`"
-        },
-        {
-          "description": "This enables opening `mailto:`, `tel:`, `https://` and `http://` urls using their default application.",
-          "type": "string",
-          "const": "opener:allow-default-urls",
-          "markdownDescription": "This enables opening `mailto:`, `tel:`, `https://` and `http://` urls using their default application."
-        },
-        {
-          "description": "Enables the open_path command without any pre-configured scope.",
-          "type": "string",
-          "const": "opener:allow-open-path",
-          "markdownDescription": "Enables the open_path command without any pre-configured scope."
-        },
-        {
-          "description": "Enables the open_url command without any pre-configured scope.",
-          "type": "string",
-          "const": "opener:allow-open-url",
-          "markdownDescription": "Enables the open_url command without any pre-configured scope."
-        },
-        {
-          "description": "Enables the reveal_item_in_dir command without any pre-configured scope.",
-          "type": "string",
-          "const": "opener:allow-reveal-item-in-dir",
-          "markdownDescription": "Enables the reveal_item_in_dir command without any pre-configured scope."
-        },
-        {
-          "description": "Denies the open_path command without any pre-configured scope.",
-          "type": "string",
-          "const": "opener:deny-open-path",
-          "markdownDescription": "Denies the open_path command without any pre-configured scope."
-        },
-        {
-          "description": "Denies the open_url command without any pre-configured scope.",
-          "type": "string",
-          "const": "opener:deny-open-url",
-          "markdownDescription": "Denies the open_url command without any pre-configured scope."
-        },
-        {
-          "description": "Denies the reveal_item_in_dir command without any pre-configured scope.",
-          "type": "string",
-          "const": "opener:deny-reveal-item-in-dir",
-          "markdownDescription": "Denies the reveal_item_in_dir command without any pre-configured scope."
-        },
         {
           "description": "This permission set configures which\nshell functionality is exposed by default.\n\n#### Granted Permissions\n\nIt allows to use the `open` functionality with a reasonable\nscope pre-configured. It will allow opening `http(s)://`,\n`tel:` and `mailto:` links.\n\n#### This default permission set includes:\n\n- `allow-open`",
           "type": "string",
@@ -6764,23 +6548,6 @@
         }
       ]
     },
-    "Application": {
-      "description": "Opener scope application.",
-      "anyOf": [
-        {
-          "description": "Open in default application.",
-          "type": "null"
-        },
-        {
-          "description": "If true, allow open with any application.",
-          "type": "boolean"
-        },
-        {
-          "description": "Allow specific application to open with.",
-          "type": "string"
-        }
-      ]
-    },
     "ShellScopeEntryAllowedArg": {
       "description": "A command argument allowed to be executed by the webview API.",
       "anyOf": [

src-tauri/memset_s_shim.c

@@ -1,28 +0,0 @@
-// Shim for memset_explicit on MinGW which doesn't provide it
-// This is needed for libsodium's secure memory clearing
-
-#if defined(_WIN32) && defined(__MINGW32__)
-
-#include <string.h>
-
-// memset_explicit is available in Windows 8+ but MinGW headers don't always declare it
-// Provide a fallback implementation using SecureZeroMemory if available,
-// or a volatile memset to prevent compiler optimization
-void *memset_explicit(void *s, int c, size_t n) {
-    // Try to use Windows API if available
-    #ifdef _WIN32_WINNT
-        #if _WIN32_WINNT >= 0x0602  // Windows 8+
-            extern void *memset_s(void *, size_t, int, size_t);
-            return memset_s(s, n, c, n);
-        #endif
-    #endif
-
-    // Fallback: use volatile to prevent optimization
-    volatile unsigned char *p = (volatile unsigned char *)s;
-    while (n--) {
-        *p++ = (unsigned char)c;
-    }
-    return s;
-}
-
-#endif

src-tauri/tauri.conf.json

@@ -1,6 +1,6 @@
 {
   "productName": "Troubleshooting and RCA Assistant",
-  "version": "1.2.4",
+  "version": "1.2.3",
   "identifier": "com.trcaa.app",
   "build": {
     "frontendDist": "../dist",

src/pages/Proxmox/RemotesPage.tsx

@@ -51,39 +51,18 @@ export function ProxmoxRemotesPage() {
     return Date.now().toString(36) + Math.random().toString(36).substr(2);
   };
 
-  /**
-   * Helper function to parse a Proxmox URL and extract hostname and port.
-   * Handles URLs with or without explicit port numbers.
-   *
-   * @param url - The full URL (e.g., "https://172.0.0.18:8006" or "https://pve.example.com")
-   * @param type - The cluster type ('pve' or 'pbs') to determine default port
-   * @returns Object with hostname (stripped of protocol and port) and port number
-   */
-  const parseRemoteUrl = (url: string, type: 'pve' | 'pbs'): { hostname: string; port: number } => {
-    let hostname = url.replace(/^https?:\/\//, '');
-    let port = type === 'pve' ? 8006 : 8007;
-
-    const portMatch = hostname.match(/:(\d+)$/);
-    if (portMatch) {
-      port = parseInt(portMatch[1], 10);
-      hostname = hostname.replace(/:\d+$/, '');
-    }
-
-    return { hostname, port };
-  };
-
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
   const handleAddRemote = async (config: any) => {
     try {
       const clusterType = config.type === 'pve' ? 've' : 'pbs';
-      const { hostname, port } = parseRemoteUrl(config.url, config.type);
+      const url = config.url.replace(/^https?:\/\//, '');
-
+      const port = config.type === 'pve' ? 8006 : 8007;
       const id = config.id || generateId();
       await addProxmoxCluster(
         id,
         config.name,
         clusterType as ClusterType,
-        { url: hostname, port },
+        { url, port },
         config.username,
         config.password || ''
       );
@@ -100,17 +79,14 @@ export function ProxmoxRemotesPage() {
   const handleEditRemote = async (config: any) => {
     try {
       const clusterType = config.type === 'pve' ? 've' : 'pbs';
-      const { hostname, port } = parseRemoteUrl(config.url, config.type);
+      const url = config.url.replace(/^https?:\/\//, '');
-
+      const port = config.type === 'pve' ? 8006 : 8007;
-      // Edit operation requires remove-then-add since backend doesn't support update.
-      // If add fails after remove, the remote will be lost - this is a known limitation
-      // until backend supports atomic update operations.
       await removeProxmoxCluster(config.id);
       await addProxmoxCluster(
         config.id,
         config.name,
         clusterType as ClusterType,
-        { url: hostname, port },
+        { url, port },
         config.username,
         config.password || ''
       );