feat: Multi Server Management #354
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…have persisted to database
…during get requests
|
Note Reviews pausedUse the following commands to manage reviews:
WalkthroughAdds a full “Servers” feature across API, storage, validation, middleware, WebSocket, and UI. Introduces SSH/context-aware Docker and SSH services, per-request Docker usage, server-scoped realtime flows, migrations for servers table and status enforcement, file-manager context propagation, i18n strings, Redux slice and RTK Query APIs, and a Servers settings UI with selector. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
actor U as User
participant V as View: ServerSelector
participant R as RTK Query
participant API as ServersController
participant SVC as ServersService
participant ST as ServerStorage
participant DB as DB
U->>V: Select server (activate/deactivate)
V->>R: PATCH /v1/server/status
R->>API: UpdateServerStatus
API->>SVC: UpdateServerStatus(req, userID)
SVC->>ST: BeginTx / GetServer / UpdateStatus
ST->>DB: Update status (trigger ensures single active)
DB-->>ST: OK
ST-->>SVC: Commit
SVC-->>API: Updated server
API-->>R: Response
R-->>V: Updated server
V-->>U: UI reflects active/inactive
sequenceDiagram
autonumber
actor C as Client (WebSocket)
participant WS as SocketServer
participant ST as ServerStorage
participant API as Handlers
participant D as DashboardMonitor/Terminal
C->>WS: GET /ws?token=...&server=SERVER_ID
WS->>ST: Validate server ownership
ST-->>WS: Server or error
WS->>API: readLoop(conn, server)
API->>D: New...(conn, logger, server)
D-->>API: Start using server-scoped SSH/Docker
sequenceDiagram
autonumber
participant Ctrl as ContainerController
participant DS as DockerService
participant DK as Docker Daemon
Ctrl->>Ctrl: getDockerService(ctx)
Ctrl->>DS: NewDockerServiceWithContext(ctx)
DS->>DS: (optional) Setup SSH tunnel
Ctrl->>DS: Perform operation (e.g., ListAllContainers)
DS->>DK: API call
DK-->>DS: Result
DS-->>Ctrl: Result
Ctrl->>DS: Close()
Estimated code review effort🎯 5 (Critical) | ⏱️ ~150 minutes Possibly related PRs
Suggested labels
Poem
✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
|
@coderabbitai pause |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
This comment was marked as abuse.
This comment was marked as abuse.
Sorry, something went wrong.
This comment was marked as abuse.
This comment was marked as abuse.
Sorry, something went wrong.
✅ Actions performedReviews paused. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary by CodeRabbit