Refactoring for JSON support in output

api/resource_platform.go

@@ -1,6 +1,28 @@
 package api
 
 func (c *Client) PlatformRegions() ([]Region, error) {
+	query := `
+		query {
+			platform {
+				regions {
+					name
+					code
+				}
+			}
+		}
+	`
+
+	req := c.NewRequest(query)
+
+	data, err := c.Run(req)
+	if err != nil {
+		return nil, err
+	}
+
+	return data.Platform.Regions, nil
+}
+
+func (c *Client) PlatformRegionsAll() ([]Region, error) {
 	query := `
 		query {
 			platform {

api/types.go

@@ -394,10 +394,10 @@ type CheckState struct {
 }
 
 type Region struct {
-	Code      string
-	Name      string
-	Latitude  float32
-	Longitude float32
+	Code      string  `json:"code"`
+	Name      string  `json:"name"`
+	Latitude  float32 `json:"latitude,omitempty"`
+	Longitude float32 `json:"longitude,omitempty"`
 }
 
 type AutoscalingConfig struct {

cmd/appInfo.go

@@ -6,7 +6,6 @@ import (
 
 	"github.com/superfly/flyctl/docstrings"
 
-	"github.com/logrusorgru/aurora"
 	"github.com/superfly/flyctl/cmd/presenters"
 )
 
@@ -21,26 +20,23 @@ func runAppInfo(ctx *CmdContext) error {
 		return err
 	}
 
-	fmt.Println(aurora.Bold("App"))
-	err = ctx.RenderEx(&presenters.AppInfo{App: *app}, presenters.Options{HideHeader: true, Vertical: true})
+	err = ctx.Frender(ctx.Out, PresenterOption{Presentable: &presenters.AppInfo{App: *app}, HideHeader: true, Vertical: true, Title: "App"})
 	if err != nil {
 		return err
 	}
 
-	fmt.Println(aurora.Bold("Services"))
-	err = ctx.Render(&presenters.Services{Services: app.Services})
+	err = ctx.Frender(ctx.Out, PresenterOption{Presentable: &presenters.Services{Services: app.Services}, Title: "Services"})
 	if err != nil {
 		return err
 	}
 
-	fmt.Println(aurora.Bold("IP Addresses"))
-	err = ctx.Render(&presenters.IPAddresses{IPAddresses: app.IPAddresses.Nodes})
+	err = ctx.Frender(ctx.Out, PresenterOption{Presentable: &presenters.IPAddresses{IPAddresses: app.IPAddresses.Nodes}, Title: "IP Adresses"})
 	if err != nil {
 		return err
 	}
 
 	if !app.Deployed {
-		fmt.Println(`App has not been deployed yet. Try running "flyctl deploy --image flyio/hellofly"`)
+		fmt.Fprintln(ctx.Out, `App has not been deployed yet. Try running "flyctl deploy --image flyio/hellofly"`)
 	}
 
 	return nil

cmd/apps.go

@@ -222,7 +222,7 @@ func runAppsCreate(ctx *CmdContext) error {
 
 	fmt.Println("New app created")
 
-	err = ctx.RenderEx(&presenters.AppInfo{App: *app}, presenters.Options{HideHeader: true, Vertical: true})
+	err = ctx.Frender(ctx.Out, PresenterOption{Presentable: &presenters.AppInfo{App: *app}, HideHeader: true, Vertical: true})
 	if err != nil {
 		return err
 	}

cmd/certificates.go

@@ -31,9 +31,9 @@ func newCertificatesCommand() *Command {
 	create.Command.Args = cobra.ExactArgs(1)
 
 	certsDeleteStrings := docstrings.Get("certs.delete")
-	delete := BuildCommand(cmd, runCertDelete, certsDeleteStrings.Usage, certsDeleteStrings.Short, certsDeleteStrings.Long, os.Stdout, requireSession, requireAppName)
-	delete.Command.Args = cobra.ExactArgs(1)
-	delete.AddBoolFlag(BoolFlagOpts{Name: "yes", Shorthand: "y", Description: "accept all confirmations"})
+	deleteCmd := BuildCommand(cmd, runCertDelete, certsDeleteStrings.Usage, certsDeleteStrings.Short, certsDeleteStrings.Long, os.Stdout, requireSession, requireAppName)
+	deleteCmd.Command.Args = cobra.ExactArgs(1)
+	deleteCmd.AddBoolFlag(BoolFlagOpts{Name: "yes", Shorthand: "y", Description: "accept all confirmations"})
 
 	certsShowStrings := docstrings.Get("certs.show")
 	show := BuildCommand(cmd, runCertShow, certsShowStrings.Usage, certsShowStrings.Short, certsShowStrings.Long, os.Stdout, requireSession, requireAppName)
@@ -63,7 +63,7 @@ func runCertShow(ctx *CmdContext) error {
 		return err
 	}
 
-	return ctx.RenderEx(&presenters.Certificate{Certificate: cert}, presenters.Options{Vertical: true})
+	return ctx.Frender(ctx.Out, PresenterOption{Presentable: &presenters.Certificate{Certificate: cert}, Vertical: true})
 }
 
 func runCertCheck(ctx *CmdContext) error {
@@ -74,7 +74,7 @@ func runCertCheck(ctx *CmdContext) error {
 		return err
 	}
 
-	return ctx.RenderEx(&presenters.Certificate{Certificate: cert}, presenters.Options{Vertical: true})
+	return ctx.Frender(ctx.Out, PresenterOption{Presentable: &presenters.Certificate{Certificate: cert}, Vertical: true})
 }
 
 func runCertAdd(ctx *CmdContext) error {
@@ -85,7 +85,7 @@ func runCertAdd(ctx *CmdContext) error {
 		return err
 	}
 
-	return ctx.RenderEx(&presenters.Certificate{Certificate: cert}, presenters.Options{Vertical: true})
+	return ctx.Frender(ctx.Out, PresenterOption{Presentable: &presenters.Certificate{Certificate: cert}, Vertical: true})
 }
 
 func runCertDelete(ctx *CmdContext) error {

cmd/command.go

@@ -164,20 +164,21 @@ func (ctx *CmdContext) Render(presentable presenters.Presentable) error {
 	return presenter.Render()
 }
 
-// RenderEx - Render a presentable structure via the context with additional options
-func (ctx *CmdContext) RenderEx(presentable presenters.Presentable, options presenters.Options) error {
-	presenter := &presenters.Presenter{
-		Item: presentable,
-		Out:  os.Stdout,
-		Opts: options,
-	}
-
-	return presenter.Render()
-}
+//// RenderEx - Render a presentable structure via the context with additional options
+//func (ctx *CmdContext) RenderEx(presentable presenters.Presentable, options presenters.Options) error {
+//	presenter := &presenters.Presenter{
+//		Item: presentable,
+//		Out:  os.Stdout,
+//		Opts: options,
+//	}
+//
+//	return presenter.Render()
+//}
 
 // PresenterOption - options for RenderEx, RenderView, render etc...
 type PresenterOption struct {
 	Presentable presenters.Presentable
+	AsJSON      bool
 	Vertical    bool
 	HideHeader  bool
 	Title       string
@@ -191,10 +192,11 @@ func (ctx *CmdContext) render(out io.Writer, views ...PresenterOption) error {
 			Opts: presenters.Options{
 				Vertical:   v.Vertical,
 				HideHeader: v.HideHeader,
+				AsJSON:     v.AsJSON,
 			},
 		}
 
-		if v.Title != "" {
+		if v.Title != "" && !v.AsJSON {
 			fmt.Fprintln(out, aurora.Bold(v.Title))
 		}
 
@@ -208,6 +210,13 @@ func (ctx *CmdContext) render(out io.Writer, views ...PresenterOption) error {
 
 // Frender - render a view to a Writer
 func (ctx *CmdContext) Frender(w io.Writer, views ...PresenterOption) error {
+	// If JSON output wanted, set in all views
+	if ctx.GlobalConfig.GetBool(flyctl.ConfigJSONOutput) {
+		for i, _ := range views {
+			views[i].AsJSON = true
+		}
+	}
+
 	return ctx.render(w, views...)
 }
 

cmd/platform.go

@@ -1,6 +1,8 @@
 package cmd
 
 import (
+	"fmt"
+	"github.com/superfly/flyctl/flyctl"
 	"os"
 
 	"github.com/spf13/cobra"
@@ -34,6 +36,8 @@ func runPlatformRegions(ctx *CmdContext) error {
 		return err
 	}
 
+	fmt.Println(ctx.GlobalConfig.GetBool(flyctl.ConfigJSONOutput))
+
 	return ctx.Frender(ctx.Out, PresenterOption{
 		Presentable: &presenters.Regions{Regions: regions},
 	})

cmd/presenters/alloc_checks.go

@@ -8,6 +8,10 @@ type AllocationChecks struct {
 	Checks []api.CheckState
 }
 
+func (p *AllocationChecks) APIStruct() interface{} {
+	return nil
+}
+
 func (p *AllocationChecks) FieldNames() []string {
 	return []string{"ID", "Service", "State", "Output"}
 }

cmd/presenters/alloc_events.go

@@ -10,6 +10,10 @@ type AllocationEvents struct {
 	Events []api.AllocationEvent
 }
 
+func (p *AllocationEvents) APIStruct() interface{} {
+	return p.Events
+}
+
 func (p *AllocationEvents) FieldNames() []string {
 	return []string{"Timestamp", "Type", "Message"}
 }

cmd/presenters/allocations.go

@@ -12,6 +12,10 @@ type Allocations struct {
 	Allocations []*api.AllocationStatus
 }
 
+func (p *Allocations) APIStruct() interface{} {
+	return p.Allocations
+}
+
 func (p *Allocations) FieldNames() []string {
 	return []string{"ID", "Version", "Region", "Desired", "Status", "Health Checks", "Restarts", "Created"}
 }

cmd/presenters/appInfo.go

@@ -10,6 +10,10 @@ type AppInfo struct {
 	App api.App
 }
 
+func (p *AppInfo) APIStruct() interface{} {
+	return p.App
+}
+
 func (p *AppInfo) FieldNames() []string {
 	return []string{"Name", "Owner", "Version", "Status", "Hostname"}
 }

cmd/presenters/apps.go

@@ -9,6 +9,10 @@ type Apps struct {
 	Apps []api.App
 }
 
+func (p *Apps) APIStruct() interface{} {
+	return nil
+}
+
 func (p *Apps) FieldNames() []string {
 	return []string{"Name", "Owner", "Status", "Latest Deploy"}
 }

cmd/presenters/builds.go

@@ -6,6 +6,10 @@ type Builds struct {
 	Builds []api.Build
 }
 
+func (p *Builds) APIStruct() interface{} {
+	return nil
+}
+
 func (p *Builds) FieldNames() []string {
 	return []string{"ID", "Status", "User", "Created At", "Updated At"}
 }

cmd/presenters/certificate.go

@@ -11,6 +11,10 @@ type Certificate struct {
 	Certificate *api.AppCertificate
 }
 
+func (p *Certificate) APIStruct() interface{} {
+	return p.Certificate
+}
+
 func (p *Certificate) FieldNames() []string {
 	return []string{"Hostname", "Configured", "Issued", "Certificate Authority", "DNS Provider", "DNS Validation Instructions", "DNS Validation Hostname", "DNS Validation Target", "Source", "Created At", "Status"}
 }

cmd/presenters/certificates.go

@@ -8,6 +8,10 @@ type Certificates struct {
 	Certificates []api.AppCertificate
 }
 
+func (p *Certificates) APIStruct() interface{} {
+	return nil
+}
+
 func (p *Certificates) FieldNames() []string {
 	return []string{"Hostname", "Created At", "Status"}
 }

cmd/presenters/deploymentStatus.go

@@ -10,6 +10,10 @@ type DeploymentStatus struct {
 	Status *api.DeploymentStatus
 }
 
+func (p *DeploymentStatus) APIStruct() interface{} {
+	return p.Status
+}
+
 func (p *DeploymentStatus) FieldNames() []string {
 	return []string{"ID", "Version", "Status", "Description", "Allocations"}
 }

cmd/presenters/history.go

@@ -8,6 +8,10 @@ type AppHistory struct {
 	AppChanges []api.AppChange
 }
 
+func (p *AppHistory) APIStruct() interface{} {
+	return nil
+}
+
 func (p *AppHistory) FieldNames() []string {
 	return []string{"Type", "Status", "Description", "User", "Date"}
 }

cmd/presenters/ipAddresses.go

@@ -8,6 +8,10 @@ type IPAddresses struct {
 	IPAddresses []api.IPAddress
 }
 
+func (p *IPAddresses) APIStruct() interface{} {
+	return nil
+}
+
 func (p *IPAddresses) FieldNames() []string {
 	return []string{"Type", "Address", "Created At"}
 }

cmd/presenters/presenter.go

@@ -1,6 +1,7 @@
 package presenters
 
 import (
+	"encoding/json"
 	"fmt"
 	"io"
 
@@ -11,6 +12,7 @@ import (
 type Presentable interface {
 	FieldNames() []string
 	Records() []map[string]string
+	APIStruct() interface{}
 }
 
 // Presenter - A self managing presenter which can be rendered in multiple ways
@@ -24,10 +26,15 @@ type Presenter struct {
 type Options struct {
 	Vertical   bool
 	HideHeader bool
+	AsJSON     bool
 }
 
 // Render - Renders a presenter as a field list or table
 func (p *Presenter) Render() error {
+	if p.Opts.AsJSON {
+		return p.renderJSON()
+	}
+
 	if p.Opts.Vertical {
 		return p.renderFieldList()
 	}
@@ -89,3 +96,14 @@ func (p *Presenter) renderFieldList() error {
 
 	return nil
 }
+
+func (p *Presenter) renderJSON() error {
+	data := p.Item.APIStruct()
+	if data == nil {
+		return fmt.Errorf("JSON output not available")
+	}
+	prettyJSON, err := json.MarshalIndent(p.Item.APIStruct(), "", "    ")
+	fmt.Fprintln(p.Out, string(prettyJSON))
+
+	return err
+}

cmd/presenters/regionAutoscaleConfigs.go

@@ -10,6 +10,10 @@ type AutoscalingRegionConfigs struct {
 	Regions []api.AutoscalingRegionConfig
 }
 
+func (p *AutoscalingRegionConfigs) APIStruct() interface{} {
+	return nil
+}
+
 func (p *AutoscalingRegionConfigs) FieldNames() []string {
 	return []string{"Region", "Min Count", "Weight"}
 }

cmd/presenters/regions.go

@@ -8,6 +8,10 @@ type Regions struct {
 	Regions []api.Region
 }
 
+func (p *Regions) APIStruct() interface{} {
+	return p.Regions
+}
+
 func (p *Regions) FieldNames() []string {
 	return []string{"Code", "Name"}
 }