remove Run convenience method from jail interface

Simplified the jail package interface by removing the Run() method:

## Changes
- Removed jail.Run() convenience method from jail.go
- Updated CLI to use explicit Start() -> Execute() -> Stop() pattern
- Added proper defer j.Stop() for cleanup in CLI

## Rationale
Keeping the interface explicit and minimal:
- Start() - Initialize the jail environment
- Execute() - Run command in jail (requires Start() first)
- Stop() - Cleanup resources

This makes the lifecycle clear and gives users full control over
when initialization and cleanup happen.

## Usage Pattern

Co-authored-by: f0ssel <[email protected]>

Author: blink-so[bot]

cli/cli.go

@@ -94,8 +94,14 @@ func Run(config Config, args []string) error {
 		return fmt.Errorf("failed to create jail: %v", err)
 	}
 
-	// Run the command in the jail
-	return j.Run(args, nil)
+	// Start the jail
+	if err := j.Start(); err != nil {
+		return fmt.Errorf("failed to start jail: %v", err)
+	}
+	defer j.Stop()
+
+	// Execute the command in the jail
+	return j.Execute(args, nil)
 }
 
 // setupLogging configures and returns a logger based on the log level

jail

@@ -250,17 +250,6 @@ func (j *Jail) Execute(command []string, additionalEnv map[string]string) error
 	return nil
 }
 
-// Run is a convenience method that combines Start() and Execute().
-// It starts the jail, executes the command, and automatically cleans up.
-func (j *Jail) Run(command []string, additionalEnv map[string]string) error {
-	if err := j.Start(); err != nil {
-		return err
-	}
-	defer j.Stop()
-
-	return j.Execute(command, additionalEnv)
-}
-
 // Stop gracefully shuts down the jail and cleans up resources.
 func (j *Jail) Stop() error {
 	if j.cleanedUp {
@@ -292,4 +281,4 @@ func (j *Jail) Stop() error {
 	j.cleanedUp = true
 	j.logger.Debug("Jail stopped successfully")
 	return nil
-}
+}