Move to cobra part 3 (#250)

* Improve Some command descriptions

* Add alias and shorthands support

* Drop unused symbols

* Add shell completions for common flags

Implement dynamic shell completions for app, team, job, pool, plan,
platform, and router flags. This enables tab completion in shells for
these frequently used flags, improving CLI usability.

Also removes local platform struct in favor of appTypes.Platform.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Update license header

* Remove redundant MinArgs: 0 declarations

MinArgs defaults to 0, so explicitly setting it is unnecessary. This
cleanup removes the redundant declarations and fixes formatting.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Add shell completions for app/job commands and update shorthands

Implement Complete() method for app commands (AppCreate, AppInfo,
AppGrant, AppRevoke, AppStop, AppStart, AppRestart) and job commands
(JobInfo, JobDelete, JobTrigger, JobUpdate, JobLog) to provide
tab completion suggestions. Also fix ShorthandCommand to properly
delegate completion to wrapped commands.

Update shorthand registrations: add rollback, remove scale.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Add AutoCompleteCommand interface and gray color for env observations

- Add AutoCompleteCommand interface for commands with custom arg completion
- Implement ValidArgsFunction integration in ManagerV2 for auto-complete
- Add common aliases support for FQDN commands on root
- Add gray color support to Colorfy for subtle text styling
- Apply gray coloring to env variable observations (private, managed by)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Update tsuru/client/apps.go

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Rename autoscale commands and improve deprecated command UX

- Rename unit-autoscale-{set,unset,swap} to autoscale-{set,unset,swap}
- Register old names as deprecated aliases
- Improve DeprecatedCommand to show colored warnings with humanized
  command names (e.g., "tsuru app info" instead of "app-info")
- Add DEPRECATED notice to help output for deprecated commands
- Standardize topic descriptions (capitalization and punctuation)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Add shell completions for service commands

Implement AutoCompleteCommand interface for service-related commands
to provide shell tab-completion for service names, instance names,
and team names. Adds ServiceInstanceCompletionMixIn for reusable
completion logic across multiple service instance commands.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Use ExtractProgramName

* Add generate-doc command for markdown documentation

Add a hidden generate-doc command that uses cobra/doc to generate
markdown documentation for all CLI commands.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* fix tests

* Attempt to fix CI test on windows

* Refactor Manager.Run to return error for proper exit handling

Update both V1 and V2 Manager.Run methods to return error instead of
calling os.Exit directly. This enables better error propagation and
allows main() to handle exit codes consistently. Also simplifies
root command configuration by removing redundant runRootCmd function.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Attempt to fix CI test on windows[2]

* Enable V2 by default and fix V1-specific tests

Change the version check to default to V2 (require explicit v1 to use V1).
Remove RemovedCommand which is no longer needed with V2.
Add V2() getter to Manager and disable V2 in V1-specific tests.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Fix grammar

* Fix ServiceTemplate

* Fix test

* Drop deprecated AppDeployRebuild

---------

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Author: 3 people

go.mod

@@ -40,6 +40,7 @@ require (
 	github.com/cespare/xxhash/v2 v2.3.0 // indirect
 	github.com/containerd/containerd v1.7.29 // indirect
 	github.com/containerd/log v0.1.0 // indirect
+	github.com/cpuguy83/go-md2man/v2 v2.0.6 // indirect
 	github.com/davecgh/go-spew v1.1.1 // indirect
 	github.com/docker/cli v23.0.3+incompatible // indirect
 	github.com/docker/docker v28.0.0+incompatible // indirect
@@ -76,6 +77,7 @@ require (
 	github.com/prometheus/procfs v0.10.1 // indirect
 	github.com/robfig/cron/v3 v3.0.1 // indirect
 	github.com/rogpeppe/go-internal v1.9.0 // indirect
+	github.com/russross/blackfriday/v2 v2.1.0 // indirect
 	github.com/sagikazarmark/locafero v0.11.0 // indirect
 	github.com/sirupsen/logrus v1.9.3 // indirect
 	github.com/sourcegraph/conc v0.3.1-0.20240121214520-5f936abd7ae8 // indirect

go.sum

@@ -26,6 +26,7 @@ github.com/containerd/containerd v1.7.29 h1:90fWABQsaN9mJhGkoVnuzEY+o1XDPbg9BTC9
 github.com/containerd/containerd v1.7.29/go.mod h1:azUkWcOvHrWvaiUjSQH0fjzuHIwSPg1WL5PshGP4Szs=
 github.com/containerd/log v0.1.0 h1:TCJt7ioM2cr/tfR8GPbGf9/VRAX8D2B4PjzCpfX540I=
 github.com/containerd/log v0.1.0/go.mod h1:VRRf09a7mHDIRezVKTRCrOq78v577GXq3bSa3EhrzVo=
+github.com/cpuguy83/go-md2man/v2 v2.0.6 h1:XJtiaUW6dEEqVuZiMTn1ldk455QWwEIsMIJlo5vtkx0=
 github.com/cpuguy83/go-md2man/v2 v2.0.6/go.mod h1:oOW0eioCTA6cOiMLiUPZOpcVxMig6NIQQ7OS05n1F4g=
 github.com/creack/pty v1.1.9/go.mod h1:oKZEueFk5CKHvIhNR5MUki03XCEU+Q6VDXinZuGJ33E=
 github.com/creack/pty v1.1.11 h1:07n33Z8lZxZ2qwegKbObQohDhXDQxiMMz1NOUGYlesw=
@@ -228,6 +229,7 @@ github.com/rogpeppe/go-internal v1.2.2/go.mod h1:M8bDsm7K2OlrFYOpmOWEs/qY81heoFR
 github.com/rogpeppe/go-internal v1.3.0/go.mod h1:M8bDsm7K2OlrFYOpmOWEs/qY81heoFRclV5y23lUDJ4=
 github.com/rogpeppe/go-internal v1.9.0 h1:73kH8U+JUqXU8lRuOHeVHaa/SZPifC7BkcraZVejAe8=
 github.com/rogpeppe/go-internal v1.9.0/go.mod h1:WtVeX8xhTBvf0smdhujwtBcq4Qrzq/fJaraNFVN+nFs=
+github.com/russross/blackfriday/v2 v2.1.0 h1:JIOH55/0cWyOuilr9/qlrm0BSXldqnqwMsf35Ld67mk=
 github.com/russross/blackfriday/v2 v2.1.0/go.mod h1:+Rmxgy9KzJVeS9/2gXHxylqXiyQDYRxCVz55jmeOWTM=
 github.com/sabhiram/go-gitignore v0.0.0-20171017070213-362f9845770f h1:FQZgA673tRGrrXIP/OPMO69g81ow4XsKlN/DLH8pSic=
 github.com/sabhiram/go-gitignore v0.0.0-20171017070213-362f9845770f/go.mod h1:b18R55ulyQ/h3RaWyloPyER7fWQVZvimKKhnI5OfrJQ=

tsuru/admin/app.go

@@ -20,9 +20,9 @@ type AppRoutesRebuild struct {
 
 func (c *AppRoutesRebuild) Info() *cmd.Info {
 	return &cmd.Info{
-		Name:    "app-routes-rebuild",
-		MinArgs: 0,
-		Usage:   "app-routes-rebuild <app-name>",
+		Name: "app-routes-rebuild",
+
+		Usage: "app-routes-rebuild <app-name>",
 		Desc: `Rebuild routes for an application.
 This can be used to recover from some failure in the router that caused
 existing routes to be lost.`,

tsuru/admin/event.go

@@ -28,10 +28,9 @@ type EventBlockList struct {
 
 func (c *EventBlockList) Info() *cmd.Info {
 	return &cmd.Info{
-		Name:    "event-block-list",
-		Usage:   "event block list [-a/--active]",
-		Desc:    "Lists all event blocks",
-		MinArgs: 0,
+		Name:  "event-block-list",
+		Usage: "event block list [-a/--active]",
+		Desc:  "Lists all event blocks",
 	}
 }
 

tsuru/admin/platform.go

@@ -24,13 +24,9 @@ import (
 	"github.com/tsuru/tsuru-client/tsuru/cmd/standards"
 	"github.com/tsuru/tsuru-client/tsuru/formatter"
 	tsuruHTTP "github.com/tsuru/tsuru-client/tsuru/http"
+	appTypes "github.com/tsuru/tsuru/types/app"
 )
 
-type platform struct {
-	Name     string `json:"name"`
-	Disabled bool   `json:"disabled"`
-}
-
 type PlatformList struct {
 	fs         *pflag.FlagSet
 	simplified bool
@@ -46,7 +42,7 @@ func (p *PlatformList) Run(context *cmd.Context) error {
 	if err != nil {
 		return err
 	}
-	var platforms []platform
+	var platforms []appTypes.Platform
 	resp, err := tsuruHTTP.AuthenticatedClient.Do(request)
 	if err != nil {
 		return err
@@ -103,10 +99,9 @@ func (c *PlatformList) Flags() *pflag.FlagSet {
 }
 func (*PlatformList) Info() *cmd.Info {
 	return &cmd.Info{
-		Name:    "platform-list",
-		Usage:   "platform list",
-		Desc:    "Lists the available platforms. All platforms displayed in this list may be used to create new apps (see app-create).",
-		MinArgs: 0,
+		Name:  "platform-list",
+		Usage: "platform list",
+		Desc:  "Lists the available platforms. All platforms displayed in this list may be used to create new apps (see app-create).",
 	}
 }
 

tsuru/admin/platform_test.go

@@ -15,6 +15,7 @@ import (
 	"github.com/tsuru/tsuru-client/tsuru/cmd"
 	"github.com/tsuru/tsuru-client/tsuru/cmd/cmdtest"
 	tsuruIo "github.com/tsuru/tsuru/io"
+	appTypes "github.com/tsuru/tsuru/types/app"
 	"gopkg.in/check.v1"
 )
 
@@ -558,10 +559,10 @@ func (s *S) TestPlatformInfoRun(c *check.C) {
 	}
 	expectedMsg := "Name: teste\nStatus: enabled\nImages:\n - tsuru/teste:v2\n - tsuru/teste:v1\n"
 	res := struct {
-		Platform platform
+		Platform appTypes.Platform
 		Images   []string
 	}{
-		Platform: platform{Name: name, Disabled: false},
+		Platform: appTypes.Platform{Name: name, Disabled: false},
 		Images:   []string{"tsuru/teste:v1", "tsuru/teste:v2"},
 	}
 	result, err := json.Marshal(res)

tsuru/admin/quota.go

@@ -99,10 +99,10 @@ type AppQuotaView struct {
 
 func (*AppQuotaView) Info() *cmd.Info {
 	return &cmd.Info{
-		Name:    "app-quota-view",
-		MinArgs: 0,
-		Usage:   "app-quota-view [appname]",
-		Desc:    "Displays the current usage and limit of the given app.",
+		Name: "app-quota-view",
+
+		Usage: "app-quota-view [appname]",
+		Desc:  "Displays the current usage and limit of the given app.",
 	}
 }
 

tsuru/admin/services.go

@@ -241,11 +241,9 @@ func (c *ServiceDocGet) Info() *cmd.Info {
 type ServiceTemplate struct{}
 
 func (c *ServiceTemplate) Info() *cmd.Info {
-	usg := `service template
-e.g.: $ tsuru service template template`
 	return &cmd.Info{
 		Name:  "service-template",
-		Usage: usg,
+		Usage: "service template",
 		Desc:  "Generates a manifest template file and places it in current directory",
 	}
 }

tsuru/admin/services_test.go

@@ -237,11 +237,9 @@ func (s *S) TestServiceDocGetInfo(c *check.C) {
 
 func (s *S) TestServiceTemplateInfo(c *check.C) {
 	got := (&ServiceTemplate{}).Info()
-	usg := `service template
-e.g.: $ tsuru service template template`
 	expected := &cmd.Info{
 		Name:  "service-template",
-		Usage: usg,
+		Usage: "service template",
 		Desc:  "Generates a manifest template file and places it in current directory",
 	}
 	c.Assert(got, check.DeepEquals, expected)

tsuru/auth/login.go

@@ -42,7 +42,6 @@ func (c *Login) Info() *cmd.Info {
 		
 		All tsuru actions require the user to be authenticated (except [[tsuru login]]
 		and [[tsuru version]]).`,
-		MinArgs: 0,
 
 		V2: cmd.InfoV2{
 			OnlyAppendOnRoot: true,