cmd/machine/describe.go (1)

28-29: Use the cobra command's context instead of context.Background().

Using context.Background() ignores any context set on the cobra command, which can affect signal handling and cancellation propagation.

♻️ Proposed fix

 describeCmd := &cobra.Command{
 	Use:   "describe [name]",
 	Short: "Retrieves the description of an existing machine",
-	RunE: func(_ *cobra.Command, args []string) error {
-		return cmd.Run(context.Background(), args)
+	RunE: func(cobraCmd *cobra.Command, args []string) error {
+		return cmd.Run(cobraCmd.Context(), args)
 	},
 }

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@cmd/machine/describe.go` around lines 28 - 29, The RunE handler uses
context.Background() which ignores the cobra command's context; change the
handler signature to accept the cobra command (e.g. func(c *cobra.Command, args
[]string) error) and pass c.Context() into the existing cmd.Run call (replace
context.Background() with c.Context()), ensuring you don't shadow the
package-level cmd variable while updating the parameter name.

cmd/machine/describe.go (1)

26-31: Consider using cobra command's context for signal propagation.

Using context.Background() in the RunE handler ignores any context that cobra may provide (e.g., for handling cancellation signals like SIGINT). Consider using the command's context:

♻️ Suggested improvement

 	describeCmd := &cobra.Command{
 		Use:   "describe [name]",
 		Short: "Retrieves the description of an existing machine",
-		RunE: func(_ *cobra.Command, args []string) error {
-			return cmd.Run(context.Background(), args)
+		RunE: func(cobraCmd *cobra.Command, args []string) error {
+			return cmd.Run(cobraCmd.Context(), args)
 		},
 	}

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@cmd/machine/describe.go` around lines 26 - 31, The handler currently calls
cmd.Run with context.Background(), which ignores cobra's command context and
signal cancellation; change the RunE signature to accept the command (e.g.,
func(c *cobra.Command, args []string) error) and call cmd.Run(c.Context(), args)
so describeCmd uses the cobra-provided context for cancellation/propagation
(also remove the now-unused context.Background() usage/import).

cmd/machine/describe.go (1)

26-32: Consider adding argument validation to the command.

The Use: "describe [name]" suggests an optional single argument, but without explicit Args validation, extra arguments are silently ignored (per GetMachine behavior in pkg/workspace/machine.go:147-170). Adding validation improves CLI UX by providing clear error messages for invalid invocations.

Proposed fix

 	describeCmd := &cobra.Command{
 		Use:   "describe [name]",
 		Short: "Retrieves the description of an existing machine",
+		Args:  cobra.MaximumNArgs(1),
 		RunE: func(_ *cobra.Command, args []string) error {
 			return cmd.Run(context.Background(), args)
 		},
 	}

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@cmd/machine/describe.go` around lines 26 - 32, The describe command lacks
argument validation: add an Args validator on the describeCmd (the cobra.Command
created in cmd/machine/describe.go) to enforce exactly one positional argument
(or adjust to zero/one if intended) so extra arguments are rejected; update the
command declaration near describeCmd and ensure RunE (which calls cmd.Run) only
runs when the Args check passes so GetMachine is not called with unexpected
extra args.