Skip to content

Add context management for remote cmd - #234

Merged
cb-github-robot merged 1 commit into
cloud-barista:mainfrom
seokho-son:main
Jan 24, 2026
Merged

cb-github-robot merged 1 commit into
cloud-barista:mainfrom
seokho-son:main

Conversation

@seokho-son

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings January 24, 2026 18:56

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds task management functionality for remote command execution, allowing users to view, monitor, and cancel running command tasks. It enhances the remote command and file transfer dialogs with MCI selection dropdowns and timeout configuration.

Changes:

  • Added a new "Cmd Status" button to the MCI context menu for viewing task execution history
  • Enhanced executeRemoteCmd and transferFileToMci functions to include MCI selector dropdowns and fetch available MCIs dynamically
  • Implemented new task management modal with auto-refresh, task status display, and cancellation capability
Comments suppressed due to low confidence (1)

index.js:11184

  • Similar to the executeRemoteCmd function, the subgroupid and vmid values here are directly interpolated into the URL query string without encoding. If these IDs contain special characters, the URL could be malformed. Consider using encodeURIComponent for proper URL encoding.
        if (radioValue === "SubGroup") {
          url += `?subGroupId=${subgroupid}`;
        } else if (radioValue === "VM") {
          url += `?vmId=${vmid}`;

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread index.js Outdated
Comment thread index.js
Comment thread index.js
Comment thread index.js Outdated
Comment thread index.js Outdated
Comment thread index.js Outdated
Comment thread index.js Outdated
Comment thread index.js Outdated
Comment thread index.js
Comment thread index.js
Signed-off-by: Seokho Son <shsongist@gmail.com>
@seokho-son

Copy link
Copy Markdown
Member Author

/approve

@cb-github-robot
cb-github-robot merged commit 6d94495 into cloud-barista:main Jan 24, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants