feat: refactor server initialization and directory creation logic

[cfc4n]

No description provided.

[Copilot]

Pull Request Overview

This PR refactors the server initialization and directory creation logic to simplify the code and centralize directory creation into a single utility. Key changes include:

• Removal of the CheckDirectory function from services/utils.go and introduction of CreateDirectory in cli/cmd/utils.go.
• Adjustments to MoLingConfig structure and updates to directory creation logic in CLI commands (root.go, perrun.go, moling_test.go) and server initialization in moling.go.

Reviewed Changes

Copilot reviewed 7 out of 9 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
services/utils.go	Removed CheckDirectory in favor of a more central directory creation logic.
services/config.go	Updated MoLingConfig, adding ListenAddr and modifying BasePath comments.
cli/cmd/utils.go	Introduced CreateDirectory to handle directory creation.
cli/cmd/root.go	Revised pre-run logic to use the new CreateDirectory and set BasePath using the current user.
cli/cmd/perrun.go	Renamed and updated pre-run function for directory creation.
cli/cmd/moling.go	Updated NewMoLingServer to remove the explicit listenAddr parameter.
cli/cmd/moling_test.go	Updated tests to match changes in directory creation and server initialization.

Files not reviewed (2)

• Makefile: Language not supported
• variables.mk: Language not supported

Comments suppressed due to low confidence (1)

cli/cmd/perrun.go:25

• [nitpick] The function name 'mlsCommandPreFunc' is not very descriptive; consider renaming it to something like 'preRunCreateDirectories' to clarify its purpose.

func mlsCommandPreFunc(cmd *cobra.Command, args []string) error {

[Copilot]

The base_path flag is documented as display-only, yet using a persistent flag allows its value to be overridden. Consider marking the flag as hidden or removing it to enforce read-only behavior.

Suggested change

	rootCmd.PersistentFlags().StringVar(&mlConfig.BasePath, "base_path", mlConfig.BasePath, "MoLing Base Data Path, automatically set by the system, cannot be changed, display only.")
	rootCmd.PersistentFlags().StringVar(&mlConfig.BasePath, "base_path", mlConfig.BasePath, "MoLing Base Data Path, automatically set by the system, cannot be changed, display only.")
	rootCmd.PersistentFlags().MarkHidden("base_path")