Skip to content

Polishing of startup scripts.#6

Merged
alexheifetz merged 2 commits intomainfrom
enhance-scripts
Jun 18, 2025
Merged

Polishing of startup scripts.#6
alexheifetz merged 2 commits intomainfrom
enhance-scripts

Conversation

@alexheifetz
Copy link
Contributor

This pull request refactors the script files to centralize environment checks and Maven execution logic into reusable agent.bat and agent.sh scripts. It also removes redundant PowerShell scripts, simplifying the codebase and improving maintainability.

Centralization of Environment Checks and Maven Execution:

  • Added scripts/support/agent.bat and scripts/support/agent.sh to handle environment checks and Maven execution. These scripts ensure the AGENT_APPLICATION variable is set, verify the presence of the pom.xml file, and execute the Spring Boot application. [1] [2]

Refactoring of Existing Scripts:

  • Updated mcp_server.cmd, mcp_server.sh, shell.cmd, shell.sh, and shell_docker.cmd to delegate environment checks and Maven execution to the new agent.bat and agent.sh scripts. This reduces code duplication across scripts. [1] [2] [3] [4] [5]
  • Added new shell.sh and shell.cmd scripts for the "shell" profile, leveraging the centralized logic. [1] [2]

Removal of Redundant Scripts:

  • Removed the PowerShell script scripts/shell_docker.ps1 and the environment check script scripts/support/check_env.ps1, as their functionality is now integrated into agent.bat and agent.sh. [1] [2]

@alexheifetz alexheifetz requested a review from johnsonr June 17, 2025 18:50
@alexheifetz alexheifetz merged commit 4478398 into main Jun 18, 2025
2 checks passed
@alexheifetz alexheifetz deleted the enhance-scripts branch June 18, 2025 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant