03c4d5b2f1
Some checks failed
Test / frontend-tests (pull_request) Successful in 1m49s
Test / frontend-typecheck (pull_request) Successful in 1m59s
PR Review Automation / review (pull_request) Successful in 7m9s
Test / rust-tests (pull_request) Has been cancelled
Test / rust-clippy (pull_request) Has been cancelled
Test / rust-fmt-check (pull_request) Has been cancelled
Address automated PR review feedback: - Extract parseRemoteUrl() helper to eliminate code duplication in handleAddRemote and handleEditRemote - Add JSDoc documentation for the helper function - Document known architectural limitation in edit operation (remove-then-add pattern) - Fix pre-existing issue: install missing node_modules dependencies (sonner, monaco-editor) The edit operation uses remove-then-add because the backend lacks an atomic update command. This is documented as a known limitation until updateProxmoxCluster() is implemented in the Rust backend. Verification: - All frontend tests pass (386/386) - All Rust tests pass (413 passed, 6 ignored) - ESLint, TypeScript, clippy, rustfmt all pass Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> |
||
|---|---|---|
| .. | ||
| components | ||
| config | ||
| hooks | ||
| lib | ||
| pages | ||
| stores | ||
| styles | ||
| App.tsx | ||
| main.tsx | ||
| vite-env.d.ts | ||