fix(apisecurity/tags/list): remove pagination and limit from tags list

Author: anthony-gomez-fastly

CHANGELOG.md

@@ -5,9 +5,11 @@
 ### Breaking:
 
 ### Bug Fixes:
+
 - fix(stats): `stats historical` now returns write errors instead of silently swallowing them [#1678](https://github.com/fastly/cli/pull/1678)
 
 ### Enhancements:
+
 - feat(stats): add `--field` flag to `stats historical` to filter to a single stats field. [#1678](https://github.com/fastly/cli/pull/1678)
 - feat(stats): add `stats aggregate` subcommand for cross-service aggregated stats. [#1678](https://github.com/fastly/cli/pull/1678)
 - feat(stats): add `stats usage` subcommand for bandwidth/request usage, with `--by-service` breakdown. [#1678](https://github.com/fastly/cli/pull/1678)
@@ -19,6 +21,7 @@
 - feat(service/version): add support for service validation. [#1695](https://github.com/fastly/cli/pull/1695)
 
 ### Dependencies:
+
 - build(deps): `golang.org/x/net` from 0.50.0 to 0.51.0 ([#1674](https://github.com/fastly/cli/pull/1674))
 - build(deps): `actions/upload-artifact` from 6 to 7 ([#1675](https://github.com/fastly/cli/pull/1675))
 - build(deps): `actions/download-artifact` from 7 to 8 ([#1675](https://github.com/fastly/cli/pull/1675))

pkg/commands/apisecurity/tags/list.go

@@ -20,10 +20,6 @@ type ListCommand struct {
 
 	// Required.
 	serviceName argparser.OptionalServiceNameID
-
-	// Optional.
-	limit argparser.OptionalInt
-	page  argparser.OptionalInt
 }
 
 // NewListCommand returns a usable command registered under the parent.
@@ -48,8 +44,6 @@ func NewListCommand(parent argparser.Registerer, g *global.Data) *ListCommand {
 		Description: argparser.FlagServiceNameDesc,
 		Dst:         &c.serviceName.Value,
 	})
-	c.CmdClause.Flag("limit", "Maximum number of tags to return per page").Action(c.limit.Set).IntVar(&c.limit.Value)
-	c.CmdClause.Flag("page", "Page number to return").Action(c.page.Set).IntVar(&c.page.Value)
 	c.RegisterFlagBool(c.JSONFlag())
 
 	return &c
@@ -82,13 +76,6 @@ func (c *ListCommand) Exec(_ io.Reader, out io.Writer) error {
 		ServiceID: &serviceID,
 	}
 
-	if c.limit.WasSet {
-		input.Limit = &c.limit.Value
-	}
-	if c.page.WasSet {
-		input.Page = &c.page.Value
-	}
-
 	tags, err := operations.ListTags(context.TODO(), fc, input)
 	if err != nil {
 		c.Globals.ErrLog.Add(err)

pkg/commands/apisecurity/tags/tags_test.go

@@ -312,7 +312,7 @@ func TestTagsList(t *testing.T) {
 		},
 		{
 			Name: "validate API success with pagination",
-			Args: fmt.Sprintf("--service-id %s --limit 10 --page 2", serviceID),
+			Args: fmt.Sprintf("--service-id %s", serviceID),
 			Client: &http.Client{
 				Transport: &testutil.MockRoundTripper{
 					Response: &http.Response{