refactor: clean up unused const

Add the MissingIDError type for when the ID of a resource is not
provided where required.

internal/executor/add.go

@@ -32,6 +32,13 @@ func (a *AddExecutor) Execute() error {
 }
 
 func (a *AddExecutor) addToList(gtsClient *client.Client) error {
+	if a.listID == "" {
+		return MissingIDError{
+			resource: resourceList,
+			action:   "add to",
+		}
+	}
+
 	funcMap := map[string]func(*client.Client) error{
 		resourceAccount: a.addAccountsToList,
 	}
@@ -48,10 +55,6 @@ func (a *AddExecutor) addToList(gtsClient *client.Client) error {
 }
 
 func (a *AddExecutor) addAccountsToList(gtsClient *client.Client) error {
-	if a.listID == "" {
-		return FlagNotSetError{flagText: flagListID}
-	}
-
 	if a.accountNames.Empty() {
 		return NoAccountSpecifiedError{}
 	}
@@ -138,7 +141,10 @@ func (a *AddExecutor) addToBookmarks(gtsClient *client.Client) error {
 
 func (a *AddExecutor) addStatusToBookmarks(gtsClient *client.Client) error {
 	if a.statusID == "" {
-		return FlagNotSetError{flagText: flagStatusID}
+		return MissingIDError{
+			resource: resourceStatus,
+			action:   "add to your bookmarks",
+		}
 	}
 
 	if err := gtsClient.AddStatusToBookmarks(a.statusID); err != nil {
@@ -152,7 +158,10 @@ func (a *AddExecutor) addStatusToBookmarks(gtsClient *client.Client) error {
 
 func (a *AddExecutor) addToStatus(gtsClient *client.Client) error {
 	if a.statusID == "" {
-		return FlagNotSetError{flagText: flagStatusID}
+		return MissingIDError{
+			resource: resourceStatus,
+			action:   "add to",
+		}
 	}
 
 	funcMap := map[string]func(*client.Client) error{

internal/executor/consts.go

@@ -0,0 +1,29 @@
+package executor
+
+const (
+	flagFrom string = "from"
+	flagTo   string = "to"
+	flagType string = "type"
+
+	resourceAccount         string = "account"
+	resourceBlocked         string = "blocked"
+	resourceBookmarks       string = "bookmarks"
+	resourceBoost           string = "boost"
+	resourceFollowers       string = "followers"
+	resourceFollowing       string = "following"
+	resourceFollowRequest   string = "follow-request"
+	resourceInstance        string = "instance"
+	resourceLike            string = "like"
+	resourceLiked           string = "liked"
+	resourceList            string = "list"
+	resourceMedia           string = "media"
+	resourceMediaAttachment string = "media-attachment"
+	resourceMutedAccounts   string = "muted-accounts"
+	resourceNote            string = "note"
+	resourcePoll            string = "poll"
+	resourceStatus          string = "status"
+	resourceStar            string = "star"
+	resourceStarred         string = "starred"
+	resourceTimeline        string = "timeline"
+	resourceVote            string = "vote"
+)

internal/executor/create.go

@@ -34,12 +34,12 @@ func (c *CreateExecutor) Execute() error {
 
 func (c *CreateExecutor) createList(gtsClient *client.Client) error {
 	if c.listTitle == "" {
-		return FlagNotSetError{flagText: flagListTitle}
+		return Error{"please provide the title of the list that you want to create"}
 	}
 
 	parsedListRepliesPolicy, err := model.ParseListRepliesPolicy(c.listRepliesPolicy)
 	if err != nil {
-		return err
+		return err //nolint:wrapcheck
 	}
 
 	form := client.CreateListForm{

internal/executor/delete.go

@@ -33,7 +33,10 @@ func (d *DeleteExecutor) Execute() error {
 
 func (d *DeleteExecutor) deleteList(gtsClient *client.Client) error {
 	if d.listID == "" {
-		return FlagNotSetError{flagText: flagListID}
+		return MissingIDError{
+			resource: resourceList,
+			action:   "delete",
+		}
 	}
 
 	if err := gtsClient.DeleteList(d.listID); err != nil {
@@ -47,7 +50,10 @@ func (d *DeleteExecutor) deleteList(gtsClient *client.Client) error {
 
 func (d *DeleteExecutor) deleteStatus(gtsClient *client.Client) error {
 	if d.statusID == "" {
-		return FlagNotSetError{flagText: flagStatusID}
+		return MissingIDError{
+			resource: resourceStatus,
+			action:   "delete",
+		}
 	}
 
 	status, err := gtsClient.GetStatus(d.statusID)

internal/executor/edit.go

@@ -33,7 +33,10 @@ func (e *EditExecutor) Execute() error {
 
 func (e *EditExecutor) editList(gtsClient *client.Client) error {
 	if e.listID == "" {
-		return FlagNotSetError{flagText: flagListID}
+		return MissingIDError{
+			resource: resourceList,
+			action: "edit",
+		}
 	}
 
 	list, err := gtsClient.GetList(e.listID)

internal/executor/errors.go

@@ -117,3 +117,12 @@ func (e UnexpectedNumValuesError) Error() string {
 		e.expected,
 	)
 }
+
+type MissingIDError struct {
+	resource string
+	action   string
+}
+
+func (e MissingIDError) Error() string {
+	return "please provide the ID of the " + e.resource + " you want to " + e.action
+}

internal/executor/flags.go

@@ -1,14 +0,0 @@
-package executor
-
-const (
-	flagAttachmentID = "attachment-id"
-	flagContent      = "content"
-	flagFrom         = "from"
-	flagInstance     = "instance"
-	flagListID       = "list-id"
-	flagListTitle    = "list-title"
-	flagStatusID     = "status-id"
-	flagTag          = "tag"
-	flagTo           = "to"
-	flagType         = "type"
-)

internal/executor/login.go

@@ -13,7 +13,7 @@ func (l *LoginExecutor) Execute() error {
 	var err error
 
 	if l.instance == "" {
-		return FlagNotSetError{flagText: flagInstance}
+		return Error{"please specify the instance that you want to log into"}
 	}
 
 	instance := l.instance

internal/executor/mute.go

@@ -47,7 +47,10 @@ func (m *MuteExecutor) muteAccount(gtsClient *client.Client) error {
 
 func (m *MuteExecutor) muteStatus(gtsClient *client.Client) error {
 	if m.statusID == "" {
-		return FlagNotSetError{flagText: flagStatusID}
+		return MissingIDError{
+			resource: resourceStatus,
+			action: "mute",
+		}
 	}
 
 	status, err := gtsClient.GetStatus(m.statusID)

internal/executor/remove.go

@@ -49,7 +49,10 @@ func (r *RemoveExecutor) removeFromList(gtsClient *client.Client) error {
 
 func (r *RemoveExecutor) removeAccountsFromList(gtsClient *client.Client) error {
 	if r.listID == "" {
-		return FlagNotSetError{flagText: flagListID}
+		return MissingIDError{
+			resource: resourceList,
+			action:   "remove from",
+		}
 	}
 
 	if r.accountNames.Empty() {
@@ -125,7 +128,10 @@ func (r *RemoveExecutor) removeFromBookmarks(gtsClient *client.Client) error {
 
 func (r *RemoveExecutor) removeStatusFromBookmarks(gtsClient *client.Client) error {
 	if r.statusID == "" {
-		return FlagNotSetError{flagText: flagStatusID}
+		return MissingIDError{
+			resource: resourceStatus,
+			action:   "remove",
+		}
 	}
 
 	if err := gtsClient.RemoveStatusFromBookmarks(r.statusID); err != nil {
@@ -139,7 +145,10 @@ func (r *RemoveExecutor) removeStatusFromBookmarks(gtsClient *client.Client) err
 
 func (r *RemoveExecutor) removeFromStatus(gtsClient *client.Client) error {
 	if r.statusID == "" {
-		return FlagNotSetError{flagText: flagStatusID}
+		return MissingIDError{
+			resource: resourceStatus,
+			action:   "remove from",
+		}
 	}
 
 	funcMap := map[string]func(*client.Client) error{

internal/executor/resources.go

@@ -1,25 +0,0 @@
-package executor
-
-const (
-	resourceAccount         = "account"
-	resourceBlocked         = "blocked"
-	resourceBookmarks       = "bookmarks"
-	resourceBoost           = "boost"
-	resourceFollowers       = "followers"
-	resourceFollowing       = "following"
-	resourceFollowRequest   = "follow-request"
-	resourceInstance        = "instance"
-	resourceLike            = "like"
-	resourceLiked           = "liked"
-	resourceList            = "list"
-	resourceMedia           = "media"
-	resourceMediaAttachment = "media-attachment"
-	resourceMutedAccounts   = "muted-accounts"
-	resourceNote            = "note"
-	resourcePoll            = "poll"
-	resourceStatus          = "status"
-	resourceStar            = "star"
-	resourceStarred         = "starred"
-	resourceTimeline        = "timeline"
-	resourceVote            = "vote"
-)

internal/executor/show.go

@@ -118,7 +118,10 @@ func (s *ShowExecutor) showAccount(gtsClient *client.Client) error {
 
 func (s *ShowExecutor) showStatus(gtsClient *client.Client) error {
 	if s.statusID == "" {
-		return FlagNotSetError{flagText: flagStatusID}
+		return MissingIDError{
+			resource: resourceStatus,
+			action:   "view",
+		}
 	}
 
 	status, err := gtsClient.GetStatus(s.statusID)
@@ -157,7 +160,10 @@ func (s *ShowExecutor) showTimeline(gtsClient *client.Client) error {
 		timeline, err = gtsClient.GetPublicTimeline(s.limit)
 	case model.TimelineCategoryList:
 		if s.listID == "" {
-			return FlagNotSetError{flagText: flagListID}
+			return MissingIDError{
+				resource: resourceList,
+				action:   "view the timeline in",
+			}
 		}
 
 		var list model.List
@@ -170,7 +176,7 @@ func (s *ShowExecutor) showTimeline(gtsClient *client.Client) error {
 		timeline, err = gtsClient.GetListTimeline(list.ID, list.Title, s.limit)
 	case model.TimelineCategoryTag:
 		if s.tag == "" {
-			return FlagNotSetError{flagText: flagTag}
+			return Error{"please provide the name of the tag"}
 		}
 
 		timeline, err = gtsClient.GetTagTimeline(s.tag, s.limit)
@@ -410,10 +416,6 @@ func (s *ShowExecutor) showMutedAccounts(gtsClient *client.Client) error {
 }
 
 func (s *ShowExecutor) showMediaAttachment(gtsClient *client.Client) error {
-	if len(s.attachmentIDs) == 0 {
-		return FlagNotSetError{flagText: flagAttachmentID}
-	}
-
 	if len(s.attachmentIDs) != 1 {
 		return fmt.Errorf(
 			"unexpected number of attachment IDs received: want 1, got %d",
@@ -453,7 +455,10 @@ func (s *ShowExecutor) showMedia(gtsClient *client.Client) error {
 
 func (s *ShowExecutor) showMediaFromStatus(gtsClient *client.Client) error {
 	if s.statusID == "" {
-		return FlagNotSetError{flagText: flagStatusID}
+		return MissingIDError{
+			resource: resourceStatus,
+			action:   "view the media from",
+		}
 	}
 
 	status, err := gtsClient.GetStatus(s.statusID)

internal/executor/unmute.go

@@ -42,7 +42,10 @@ func (m *UnmuteExecutor) unmuteAccount(gtsClient *client.Client) error {
 
 func (m *UnmuteExecutor) unmuteStatus(gtsClient *client.Client) error {
 	if m.statusID == "" {
-		return FlagNotSetError{flagText: flagStatusID}
+		return MissingIDError{
+			resource: resourceStatus,
+			action:   "unmute",
+		}
 	}
 
 	status, err := gtsClient.GetStatus(m.statusID)