fix: guard os.Args access for wasm which panics when building

[gqcn]

This pull request improves the reliability and safety of server restart and reload operations by ensuring the correct retrieval of the current executable path and process arguments. It replaces direct usage of os.Args[0] with a more robust approach using gfile.SelfPath(), and adds error handling for cases where the executable path cannot be determined. Additionally, it introduces a helper function to safely obtain process arguments, and removes an unnecessary import.

Executable Path Handling and Error Checking:

• Replaced usage of os.Args[0] with gfile.SelfPath() throughout the server admin and process management code to reliably determine the current executable path; added checks and error responses when the path cannot be determined. [1] [2] [3]
• Added early returns in initialization functions if os.Args is empty, preventing potential panics or misbehavior when the argument list is missing. [1] [2]

Process Argument Handling:

• Introduced the getCurrentProcessArgs() helper function to safely return process arguments (excluding the program name), ensuring correct behavior even if no arguments are provided. Updated process creation calls to use this helper.

Code Cleanup:

• Removed an unused import of the os package from ghttp_server_admin.go.

[Copilot]

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

This PR aims to prevent panics (notably on wasm) and make server restart/reload logic safer by avoiding direct os.Args[0] access and introducing guards/error handling when the executable path/args can’t be determined.

Changes:

• Guarded os.Args access in init paths to avoid panics when os.Args is empty.
• Switched restart/reload process spawning to use gfile.SelfPath() plus explicit empty-path error handling.
• Added a helper to safely derive current process args (getCurrentProcessArgs) and removed an unnecessary import.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File	Description
os/gfile/gfile.go	Adds os.Args length guard before deriving selfPath.
net/ghttp/ghttp_server_admin_process.go	Uses gfile.SelfPath() + helper args for restart/reload processes; adds empty-path errors.
net/ghttp/ghttp_server_admin.go	Uses gfile.SelfPath() for admin restart path; removes os import.
debug/gdebug/gdebug_caller.go	Adds os.Args length guard before deriving selfPath.

---

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

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

---

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