Add reference details to errors

Signed-off-by: Sajay Antony <[email protected]>

cmd/oras/internal/errors/errors.go

@@ -21,12 +21,12 @@ import (
 	"oras.land/oras-go/v2/registry"
 )
 
-// NewErrInvalidReference creates a new error based on the reference string.
-func NewErrInvalidReference(ref registry.Reference) error {
-	return NewErrInvalidReferenceStr(ref.String())
+// NewErrEmptyTagOrDigest creates a new error based on the reference string.
+func NewErrEmptyTagOrDigest(ref registry.Reference) error {
+	return NewErrEmptyTagOrDigestStr(ref.String())
 }
 
-// NewErrInvalidReferenceStr creates a new error based on the reference string.
-func NewErrInvalidReferenceStr(ref string) error {
-	return fmt.Errorf("%s: invalid image reference, expecting <name:tag|name@digest>", ref)
+// NewErrEmptyTagOrDigestStr creates a new error based on the reference string.
+func NewErrEmptyTagOrDigestStr(ref string) error {
+	return fmt.Errorf("%q: no tag or digest when expecting <name:tag|name@digest>", ref)
 }

cmd/oras/internal/option/remote.go

@@ -18,6 +18,7 @@ package option
 import (
 	"context"
 	"crypto/tls"
+	"errors"
 	"fmt"
 	"io"
 	"net"
@@ -30,6 +31,7 @@ import (
 	credentials "github.com/oras-project/oras-credentials-go"
 	"github.com/sirupsen/logrus"
 	"github.com/spf13/pflag"
+	"oras.land/oras-go/v2/errdef"
 	"oras.land/oras-go/v2/registry/remote"
 	"oras.land/oras-go/v2/registry/remote/auth"
 	"oras.land/oras-go/v2/registry/remote/retry"
@@ -290,6 +292,9 @@ func (opts *Remote) NewRegistry(registry string, common Common, logger logrus.Fi
 func (opts *Remote) NewRepository(reference string, common Common, logger logrus.FieldLogger) (repo *remote.Repository, err error) {
 	repo, err = remote.NewRepository(reference)
 	if err != nil {
+		if errors.Unwrap(err) == errdef.ErrInvalidReference {
+			return nil, fmt.Errorf("%q: %v", reference, err)
+		}
 		return nil, err
 	}
 	registry := repo.Reference.Registry

cmd/oras/internal/option/target.go

@@ -177,7 +177,7 @@ func (opts *Target) NewReadonlyTarget(ctx context.Context, common Common, logger
 // EnsureReferenceNotEmpty ensures whether the tag or digest is empty.
 func (opts *Target) EnsureReferenceNotEmpty() error {
 	if opts.Reference == "" {
-		return oerrors.NewErrInvalidReferenceStr(opts.RawReference)
+		return oerrors.NewErrEmptyTagOrDigestStr(opts.RawReference)
 	}
 	return nil
 }

cmd/oras/root/manifest/delete.go

@@ -85,7 +85,7 @@ func deleteManifest(ctx context.Context, opts deleteOptions) error {
 	}
 
 	if repo.Reference.Reference == "" {
-		return oerrors.NewErrInvalidReference(repo.Reference)
+		return oerrors.NewErrEmptyTagOrDigest(repo.Reference)
 	}
 
 	// add both pull and delete scope hints for dst repository to save potential delete-scope token requests during deleting