feat(cmd): use aliases (#109)

Signed-off-by: Christopher Haar <[email protected]>

Author: haarchri

cmd/diff/main.go

@@ -59,9 +59,8 @@ type cli struct {
 	// order they're specified here. Keep them in alphabetical order.
 
 	// Subcommands.
-	Comp CompCmd           `cmd:"" help:"Show impact of composition changes on existing XRs."`
-	Diff DeprecatedDiffCmd `cmd:"" help:"(Deprecated) Use 'xr' subcommand instead. See what changes will be made against a live cluster when a given Crossplane resource would be applied."`
-	XR   XRCmd             `cmd:"" help:"See what changes will be made against a live cluster when a given Crossplane resource would be applied."`
+	Comp CompCmd `cmd:""         help:"Show impact of composition changes on existing XRs."`
+	XR   XRCmd   `aliases:"diff" cmd:""                                                     help:"See what changes will be made against a live cluster when a given Crossplane resource would be applied."`
 
 	Version version.Cmd `cmd:"" help:"Print the client and server version information for the current context."`
 

cmd/diff/xr.go

@@ -17,8 +17,6 @@ limitations under the License.
 package main
 
 import (
-	"fmt"
-
 	"github.com/alecthomas/kong"
 	dp "github.com/crossplane-contrib/crossplane-diff/cmd/diff/diffprocessor"
 	"k8s.io/client-go/rest"
@@ -104,46 +102,6 @@ func makeDefaultXRLoader(c *XRCmd) (ld.Loader, error) {
 	return ld.NewCompositeLoader(c.Files)
 }
 
-// DeprecatedDiffCmd represents the deprecated diff command for backward compatibility.
-type DeprecatedDiffCmd struct {
-	XRCmd
-}
-
-// Help returns help instructions for the deprecated diff command.
-func (c *DeprecatedDiffCmd) Help() string {
-	return `
-This command is deprecated. Please use 'crossplane-diff xr' instead.
-
-This command returns a diff of the in-cluster resources that would be modified if the provided Crossplane resources were applied.
-
-Similar to kubectl diff, it requires Crossplane to be operating in the live cluster found in your kubeconfig.
-
-Examples:
-  # Deprecated usage (still works but will show warning)
-  crossplane-diff diff xr.yaml
-
-  # Preferred usage
-  crossplane-diff xr xr.yaml
-`
-}
-
-// Run executes the deprecated diff command with a deprecation warning.
-func (c *DeprecatedDiffCmd) Run(k *kong.Context, log logging.Logger, appCtx *AppContext, proc dp.DiffProcessor, loader ld.Loader) error {
-	if _, err := fmt.Fprintln(k.Stderr, "Warning: The 'diff' subcommand is deprecated. Please use 'xr' instead."); err != nil {
-		return err
-	}
-
-	if _, err := fmt.Fprintln(k.Stderr, "Example: crossplane-diff xr your-file.yaml"); err != nil {
-		return err
-	}
-
-	if _, err := fmt.Fprintln(k.Stderr); err != nil {
-		return err
-	}
-
-	return c.XRCmd.Run(k, log, appCtx, proc, loader)
-}
-
 // Run executes the XR diff command.
 func (c *XRCmd) Run(k *kong.Context, log logging.Logger, appCtx *AppContext, proc dp.DiffProcessor, loader ld.Loader) error {
 	// the rest config here is provided by a function in main.go that's only invoked for commands that request it