refactor: Restructure remote command API and UI separation for better maintainability#57
Merged
Conversation
SungWoongz
commented
Oct 29, 2025
SungWoongz
commented
Contributor
- API Layer Separation
- UI Layer Restructuring
- Import Path Updates
- Code Organization Benefits
- Add modal for editing CSP Role details, including description and trust policy. - Integrate API calls to fetch and update CSP Role information. - Enhance user experience with validation for required fields and JSON format for trust policies. - Update UI to include an edit button for CSP Roles in the management interface.
- Commented out the footer list in the layout template to temporarily disable it while retaining the code for potential future use. - Maintained the overall structure of the footer for consistency in layout.
- Update modal titles to be empty for better dynamic content handling. - Restructure terminal and file transfer accordion items for clarity and consistency across MCI, PMK, and subgroup templates. - Change button labels from "Disconnect" to "Close" for a more intuitive user experience. - Ensure consistent IDs and aria attributes for accessibility improvements.
- Add a new modal for remote terminal access, including a file transfer section with input for target paths and a Dropzone for file uploads. - Update the initremotecmdModal function to handle different target types (vm and subgroupvm) for improved flexibility. - Remove the display of SSH Private IP in the terminal output while maintaining the underlying logic. - Ensure the terminal prompt is displayed immediately after initialization for better user experience.
- Removed the Bootstrap JS script tag from the application template to streamline the code. - Maintained overall structure and functionality of the template.
…nd SubGroup - Introduce a new batch command execution feature allowing users to run commands on all VMs within an MCI or SubGroup. - Update the UI to include command input sections and results display for better user interaction. - Refactor existing terminal initialization to utilize the new batch command execution method. - Enhance modal displays to show command execution results and allow for re-execution of commands.
… modal - Implement pagination for command execution results to improve user experience. - Update modal structure to include dynamic pagination controls and a dedicated table body for results. - Refactor file upload handling to allow multiple file types and prevent duplicate uploads. - Change button labels in the UI for clarity, replacing "Delete" with "Deallocate" in relevant contexts.
…nctions - Add try-catch blocks in getPlugIns and setMonitoringMesurement functions to handle API errors gracefully. - Enhance data extraction logic to accommodate different API response structures. - Update UI element handling to ensure robustness against missing elements. - Modify option creation logic to use consistent property names for plugin identification.
…oring UI - Implement detection toggle functionality to show/hide the detection graph based on user input. - Enhance VM selection process by alerting users to select a VM before proceeding. - Update UI to display the selected VM name dynamically. - Refactor monitoring API call parameters for consistency and clarity.
- Update the monitoring configuration UI to dynamically display agent and collection statuses. - Modify labels for clarity, changing "Policy" to "Alarms" in the alarms history page. - Improve accessibility by adding required indicators to form labels. - Implement Bootstrap collapse functionality for file transfer buttons across various templates. - Clean up unused dropdown elements in the monitoring configuration log template.
- Refactor command execution functions to support multiple commands and improve error handling. - Update progress toast messages for clarity during command processing. - Implement retry functionality for VM commands with user feedback through toast notifications. - Enhance modal displays for command results, including detailed output and pagination. - Improve UI elements for command input sections across various templates for consistency.
- Implemented resetServerTabState function to manage the Server Info tab's active state. - Updated MCI selection process to clear previous VM selection and hide Server Info when changing MCI. - Enhanced addNewVirtualMachine function to retrieve MCI name and description from the total MCI list. - Made input fields for MCI name and description readonly in the UI for better user experience.
- Add new toast utility functions for better user feedback - Update API services with improved error handling and return statements - Enhance MCI management functionality with better state handling - Improve monitoring and PMK configurations - Update role and workspace management features - Add server tab state reset functionality
- Fix API endpoint naming convention in mciRecommendVm function - Align with remote branch changes for consistency
- Fix API endpoint naming in mci_api.js (recommendSpec) - Merge remote changes while preserving local improvements - Resolve divergent branches successfully
…able declaration for API endpoint
- Implement MCI list loading and server selection in log management - Add filtering capabilities for event alarms with UI enhancements - Introduce refresh functionality for alarm list retrieval - Update HTML templates for improved user interaction and layout consistency
- Migrate remote command API functions from common services to a dedicated partial for better organization. - Update references in MCI and PMK management scripts to use the new partial. - Adjust HTML templates to load the new remote command script, ensuring consistent functionality across the application.
MZC-CSC
added a commit
that referenced
this pull request
May 11, 2026
refactor: Restructure remote command API and UI separation for better maintainability
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 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.