Make everything build strings instead of printing

Refs #6

.gitignore

@@ -6,6 +6,8 @@
 *.so
 *.dylib
 
+/bin
+
 # Test binary, built with `go test -c`
 *.test
 
@@ -19,3 +21,6 @@
 .vscode/configurationCache.log
 .vscode/dryrun.log
 .vscode/targets.log
+
+# packaging
+/build

Makefile

@@ -1,27 +1,108 @@
 V ?= 0
 Q = $(if $(filter 1, $V),, @)
+BINDIR=$(CURDIR)/bin
+PREFIX=/usr
+DESTDIR=bin
+BUILDDIR=$(CURDIR)/build
+MANDIR=$(PREFIX)/share/man/man1
+
+PKGREVISION=1
+
+DEBBUILDDIR=$(BUILDDIR)/deb
+DEBTMPLDIR=$(CURDIR)/packaging/debian
+DEBDATE=$(shell date -R)
+DEBORIGSRC=lume_$(DEBVERSION).orig.tar.xz
+DEBORIGSRCDIR=lume-$(DEBVERSION)
+DEBREVISION=$(PKGREVISION)
+
+RPMVERSION=$(subst -,_,$(LUME_VERSION))
+RPMBUILDDIR=$(BUILDDIR)/rpm
+RPMTMPLDIR=$(CURDIR)/packaging/rpm
+RPMDATE=$(shell date "+%a %b %d %Y")
+RPMORIGSRC=lume-$(RPMVERSION).tar.xz
+RPMORIGSRCDIR=lume-$(RPMVERSION)
+RPMREVISION=$(PKGREVISION)
 
 ifeq ($(OS), Windows_NT)
-    EXE=lume.exe
+    EXE=$(BINDIR)/lume.exe
 	RM=del /f /q
 	BUILD_DATE=$(shell powershell Get-Date -Format "yyyy-MM-ddThh:mm:sszzz")
 else
-    EXE=lume
-    EXE=rm -f
+    EXE=$(BINDIR)/lume
+    RM=rm -f
 	BUILD_DATE=$(shell date --iso-8601=seconds)
 endif
 
 LUME_VERSION ?= $(shell git describe --tags --always)
 GIT_COMMIT := $(shell git rev-parse --short HEAD)
-LDFLAGS := $(LDFLAGS) \
+GIT_TAG=$(shell git describe --tags --abbrev=0)
+LDFLAGS = \
 	-X git.kill0.net/chill9/lume/cmd.Version=$(LUME_VERSION) \
 	-X git.kill0.net/chill9/lume/cmd.BuildDate=$(BUILD_DATE) \
 	-X git.kill0.net/chill9/lume/cmd.GitCommit=$(GIT_COMMIT)
 
+ifneq (,$(findstring -,$(LUME_VERSION)))
+	DEBVERSION=$(GIT_TAG)+git$(shell date +%Y%m%d)+$(GIT_COMMIT)
+else
+	DEBVERSION=$(LUME_VERSION)
+endif
+
 .PHONY: build
 build:
 	$(Q) go build -o $(EXE) -ldflags="$(LDFLAGS)" ./cmd/lume
 
 .PHONY: clean
-clean:
+clean: deb-clean rpm-clean
 	$(Q) $(RM) $(EXE)
+
+install-man:
+	install -p -D -m 0644 lume.1 $(DESTDIR)$(MANDIR)/lume.1
+
+.PHONY: install
+install: install-man
+	$(Q) install -p -D -m 0755 $(EXE) $(DESTDIR)${PREFIX}/bin/lume
+	$(Q) install -p -D -m 0644 .lumerc.sample $(DESTDIR)${PREFIX}/share/lume/lumerc
+
+.PHONY: deb
+deb: deb-clean
+	$(Q) mkdir -p $(DEBBUILDDIR)
+	$(Q) git archive --format tar --prefix lume-$(DEBVERSION)/ $(LUME_VERSION) | xz > $(DEBBUILDDIR)/$(DEBORIGSRC)
+	$(Q) tar xf $(DEBBUILDDIR)/$(DEBORIGSRC) -C $(DEBBUILDDIR)
+	$(Q) mkdir $(DEBBUILDDIR)/$(DEBORIGSRCDIR)/debian
+	$(Q) mkdir $(DEBBUILDDIR)/$(DEBORIGSRCDIR)/debian/source
+	$(Q) sed -e 's/__VERSION__/$(DEBVERSION)/g' -e 's/__REVISION__/$(DEBREVISION)/g' $(DEBTMPLDIR)/rules > $(DEBBUILDDIR)/$(DEBORIGSRCDIR)/debian/rules
+	$(Q) chmod 0755 $(DEBBUILDDIR)/$(DEBORIGSRCDIR)/debian/rules
+	$(Q) sed -e 's/__VERSION__/$(DEBVERSION)/g' -e 's/__DATE__/$(DEBDATE)/g' -e 's/__REVISION__/$(DEBREVISION)/g' $(DEBTMPLDIR)/changelog > $(DEBBUILDDIR)/$(DEBORIGSRCDIR)/debian/changelog
+	$(Q) echo 10 > $(DEBBUILDDIR)/$(DEBORIGSRCDIR)/debian/compat
+	$(Q) echo "3.0 (quilt)" > $(DEBBUILDDIR)/$(DEBORIGSRCDIR)/debian/source/format
+	$(Q) cp $(DEBTMPLDIR)/control $(DEBBUILDDIR)/$(DEBORIGSRCDIR)/debian/control
+	$(Q) cp $(DEBTMPLDIR)/copyright $(DEBBUILDDIR)/$(DEBORIGSRCDIR)/debian/copyright
+	$(Q) cp $(DEBTMPLDIR)/lume.manpages $(DEBBUILDDIR)/$(DEBORIGSRCDIR)/debian/lume.manpages
+	$(Q) cd $(DEBBUILDDIR)/$(DEBORIGSRCDIR) && dpkg-buildpackage -us -uc
+	$(Q) mv $(DEBBUILDDIR)/*.dsc $(BUILDDIR)
+	$(Q) mv $(DEBBUILDDIR)/*.changes $(BUILDDIR)
+	$(Q) mv $(DEBBUILDDIR)/*.buildinfo $(BUILDDIR)
+	$(Q) mv $(DEBBUILDDIR)/*.deb $(BUILDDIR)
+	$(Q) mv $(DEBBUILDDIR)/*.tar.* $(BUILDDIR)
+
+.PHONY: rpm
+rpm: rpm-clean
+	$(Q) mkdir -p $(RPMBUILDDIR)/SPECS
+	$(Q) mkdir -p $(RPMBUILDDIR)/SOURCES
+	$(Q) sed -e 's/__VERSION__/$(RPMVERSION)/g' -e 's/__DATE__/$(RPMDATE)/g' -e 's/__REVISION__/$(RPMREVISION)/g' $(RPMTMPLDIR)/lume.spec > $(RPMBUILDDIR)/SPECS/lume.spec
+	$(Q) git archive --format tar --prefix $(RPMORIGSRCDIR)/ $(LUME_VERSION) | xz > $(RPMBUILDDIR)/SOURCES/$(RPMORIGSRC)
+	$(Q) rpmbuild --define "_topdir $(RPMBUILDDIR)"  -ba $(RPMBUILDDIR)/SPECS/lume.spec
+	$(Q) mv $(RPMBUILDDIR)/RPMS/*/*.rpm $(BUILDDIR)
+	$(Q) mv $(RPMBUILDDIR)/SRPMS/*.rpm $(BUILDDIR)
+
+deb-clean:
+	$(Q) rm -rf $(DEBBUILDDIR)
+	$(Q) rm -f $(BUILDDIR)/*.dsc
+	$(Q) rm -f $(BUILDDIR)/*.changes
+	$(Q) rm -f $(BUILDDIR)/*.buildinfo
+	$(Q) rm -f $(BUILDDIR)/*.deb
+	$(Q) rm -f $(BUILDDIR)/*.tar.*
+
+rpm-clean:
+	$(Q) rm -rf $(RPMBUILDDIR)
+	$(Q) rm -f $(BUILDDIR)/*.rpm

cmd/breathe.go

@@ -0,0 +1,109 @@
+package lumecmd
+
+import (
+	"flag"
+	"fmt"
+
+	"git.kill0.net/chill9/lifx-go"
+)
+
+func NewCmdBreathe() Command {
+	return Command{
+		Name: "breathe",
+		Func: BreatheCmd,
+		Flags: func() *flag.FlagSet {
+			fs := flag.NewFlagSet("breathe", flag.ExitOnError)
+
+			selector := fs.String("selector", defaultSelector, "Set the selector")
+			fs.StringVar(selector, "s", defaultSelector, "Set the selector")
+
+			color := fs.String("color", defaultColor, "The color to use for the breathe effect")
+			fs.StringVar(color, "c", defaultColor, "The color to use for the breathe effect")
+
+			fs.String("from-color", defaultColor, "The color to start the effect from")
+
+			fs.Float64("period", lifx.DefaultBreathePeriod, "The time in seconds for one cycle of the effect")
+
+			fs.Float64("cycles", lifx.DefaultBreatheCycles, "The number of times to repeat the effect")
+
+			fs.Bool("persist", lifx.DefaultBreathePersist, "If false set the light back to its previous value when effect ends, if true leave the last effect color")
+
+			fs.Bool("power-on", lifx.DefaultBreathePowerOn, "If true, turn the bulb on if it is not already on")
+
+			fs.Float64("peak", lifx.DefaultBreathePeak, "Defines where in a period the target color is at its maximum (min: 0.0, max: 1.0)")
+
+			return fs
+		}(),
+		Use:   "[--selector <selector>] --color <color> [--from-color <color>] [--period <period>] [--cycles <cycles>] [--persist <persist>] [--power-on] [--peak <peak>]",
+		Short: "The breathe effect",
+	}
+}
+
+func BreatheCmd(ctx Context) (int, error) {
+	var p Printer
+
+	c := ctx.Client
+	breathe := lifx.NewBreathe()
+	selector := ctx.Flags.String("selector")
+	format, err := getOutputFormatFromFlags(ctx.Flags)
+	if err != nil {
+		return ExitFailure, err
+	}
+
+	if format == "" && ctx.Config.OutputFormat != "" {
+		format = ctx.Config.OutputFormat
+	}
+
+	color := ctx.Flags.String("color")
+	if color != "" {
+		breathe.Color = lifx.NamedColor(color)
+	}
+
+	from_color := ctx.Flags.String("from-color")
+	if from_color != "" {
+		breathe.FromColor = lifx.NamedColor(from_color)
+	}
+
+	periodFlag := ctx.Flags.String("period")
+	if periodFlag != "" {
+		period := ctx.Flags.Float64("period")
+		breathe.Period = period
+	}
+
+	cyclesFlag := ctx.Flags.String("cycles")
+	if cyclesFlag != "" {
+		cycles := ctx.Flags.Float64("cycles")
+		breathe.Cycles = cycles
+	}
+
+	persist := ctx.Flags.Bool("persist")
+	breathe.Persist = persist
+
+	power_on := ctx.Flags.Bool("power-on")
+	breathe.PowerOn = power_on
+
+	peakFlag := ctx.Flags.String("peak")
+	if peakFlag != "" {
+		peak := ctx.Flags.Float64("peak")
+		breathe.Peak = peak
+	}
+
+	if color == "" {
+		printCmdHelp(ctx.Name)
+		return ExitFailure, nil
+	}
+
+	if err := breathe.Valid(); err != nil {
+		return ExitFailure, err
+	}
+
+	r, err := c.Breathe(selector, breathe)
+	if err != nil {
+		return ExitFailure, err
+	}
+
+	p = NewPrinter(format)
+	fmt.Print(p.Results(r.Results))
+
+	return ExitSuccess, nil
+}

cmd/command.go

@@ -1,6 +1,7 @@
 package lumecmd
 
 import (
+	"errors"
 	"flag"
 	"fmt"
 	"strconv"
@@ -13,7 +14,7 @@ const (
 	ExitFailure
 )
 
-type CmdArgs struct {
+type Context struct {
 	Flags  Flags
 	Args   []string
 	Client *lifx.Client
@@ -27,7 +28,7 @@ type Flags struct {
 
 type Command struct {
 	Name  string
-	Func  func(CmdArgs) (int, error)
+	Func  func(Context) (int, error)
 	Flags *flag.FlagSet
 	Use   string
 	Short string
@@ -81,6 +82,13 @@ func RegisterCommand(cmd Command) error {
 	if _, ok := commandRegistry[cmd.Name]; ok {
 		return fmt.Errorf("%s command is already registered")
 	}
+
+	if cmd.Flags == nil {
+		cmd.Flags = flag.NewFlagSet(cmd.Name, flag.ExitOnError)
+	}
+
+	mergeGlobalFlags(cmd.Flags)
+
 	commandRegistry[cmd.Name] = cmd
 	return nil
 }
@@ -89,3 +97,26 @@ func GetCommand(name string) (Command, bool) {
 	cmd, ok := commandRegistry[name]
 	return cmd, ok
 }
+
+func mergeGlobalFlags(fs *flag.FlagSet) {
+	fs.Bool("debug", false, "Enable debug mode")
+
+	formatTable := fs.Bool("table", false, "Format output as an ASCII table")
+	fs.BoolVar(formatTable, "t", false, "Format output as an ASCII table")
+
+	fs.Bool("simple", false, "Format output simply")
+}
+
+func getOutputFormatFromFlags(fs Flags) (string, error) {
+	formatSimple := fs.Bool("simple")
+	formatTable := fs.Bool("table")
+
+	switch {
+	case formatSimple && formatTable:
+		return "", errors.New("only one output format permitted")
+	case formatTable:
+		return "table", nil
+	default:
+		return "simple", nil
+	}
+}

cmd/config.go

@@ -11,25 +11,48 @@ import (
 )
 
 const lumercFile string = ".lumerc"
+const lumeConfigFile string = "lume.conf"
+const defaultPowerIndicator rune = '●'
 
 type Config struct {
 	AccessToken  string               `toml:"access_token"`
 	OutputFormat string               `toml:"output_format"`
 	Colors       map[string][]float32 `toml:"colors"`
 	userAgent    string
+	Debug        bool   `toml:"debug"`
+	Indicator    string `toml:"indicator"`
 }
 
 var (
 	DefaultConfig = Config{
 		userAgent: initUserAgent(),
 	}
+	globalConfig *Config = NewConfig()
 )
 
+func NewConfig() *Config {
+	c := new(Config)
+	c.userAgent = initUserAgent()
+	c.Debug = false
+	c.OutputFormat = "simple"
+	c.Indicator = string(defaultPowerIndicator)
+	return c
+}
+
+func GetConfig() *Config {
+	return globalConfig
+}
+
 // Validate configuration struct
 func (c *Config) Validate() error {
 	var err error
+
 	if c.AccessToken == "" {
-		err = errors.New("access_token is not set")
+		return errors.New("access_token is not set")
+	}
+
+	if len([]rune(c.Indicator)) != 1 {
+		return errors.New("indicator must be a single rune")
 	}
 
 	if err = c.validateColors(); err != nil {
@@ -60,20 +83,20 @@ func (c *Config) validateColors() (err error) {
 	return err
 }
 
-func LoadConfig(s string) (*Config, error) {
-	var err error
-	var c *Config = &Config{}
-
-	*c = DefaultConfig
-
-	if _, err := toml.Decode(s, &c); err != nil {
-		err = fmt.Errorf("fatal: failed to parse; %w", err)
-	}
-
+func (c *Config) MergeWithEnv() {
 	envAccessToken := os.Getenv("LIFX_ACCESS_TOKEN")
 	if envAccessToken != "" {
 		c.AccessToken = envAccessToken
 	}
+}
+
+func LoadConfig(s string) (*Config, error) {
+	var err error
+	var c *Config = GetConfig()
+
+	if _, err := toml.Decode(s, &c); err != nil {
+		err = fmt.Errorf("fatal: failed to parse; %w", err)
+	}
 
 	return c, err
 }
@@ -81,27 +104,21 @@ func LoadConfig(s string) (*Config, error) {
 func LoadConfigFile(configPath string) (*Config, error) {
 	var err error
 
-	var c *Config = &Config{}
-
-	*c = DefaultConfig
+	var c *Config = GetConfig()
 
 	if _, err := toml.DecodeFile(configPath, &c); err != nil {
 		err = fmt.Errorf("fatal: failed to parse %s; %w", configPath, err)
 	}
 
-	envAccessToken := os.Getenv("LIFX_ACCESS_TOKEN")
-	if envAccessToken != "" {
-		c.AccessToken = envAccessToken
-	}
-
 	return c, err
 }
 
 func getConfigPath() string {
-	var tryPath, configPath string
+	var tryPath, configPath, homeDir, cwd string
+	var err error
 
 	// ~/.lumerc
-	homeDir, err := os.UserHomeDir()
+	homeDir, err = os.UserHomeDir()
 	if err == nil {
 		tryPath = path.Join(homeDir, lumercFile)
 		if _, err := os.Stat(tryPath); !os.IsNotExist(err) {
@@ -109,8 +126,17 @@ func getConfigPath() string {
 		}
 	}
 
+	// ~/.config/lume/lume.conf
+	homeDir, err = os.UserHomeDir()
+	if err == nil {
+		tryPath = path.Join(homeDir, ".config/lume", lumeConfigFile)
+		if _, err := os.Stat(tryPath); !os.IsNotExist(err) {
+			configPath = tryPath
+		}
+	}
+
 	// ./.lumerc
-	cwd, err := os.Getwd()
+	cwd, err = os.Getwd()
 	if err == nil {
 		tryPath = path.Join(cwd, lumercFile)
 		if _, err := os.Stat(tryPath); !os.IsNotExist(err) {

cmd/help.go

@@ -3,6 +3,7 @@ package lumecmd
 import (
 	"fmt"
 	"sort"
+	"strings"
 )
 
 func NewCmdHelp() Command {
@@ -14,17 +15,23 @@ func NewCmdHelp() Command {
 	}
 }
 
-func HelpCmd(args CmdArgs) (int, error) {
-	if len(args.Args) == 0 {
-		printHelp(commandRegistry)
-	} else if len(args.Args) >= 1 {
-		printCmdHelp(args.Args[0])
+func HelpCmd(ctx Context) (int, error) {
+	if len(ctx.Args) == 0 {
+		fmt.Print(printHelp(commandRegistry))
+	} else if len(ctx.Args) >= 1 {
+		if cmdHelp, err := printCmdHelp(ctx.Args[0]); err == nil {
+			fmt.Print(cmdHelp)
+		} else {
+			fmt.Print(err)
+		}
 	}
 
 	return ExitSuccess, nil
 }
 
-func printHelp(commands map[string]Command) {
+func printHelp(commands map[string]Command) string {
+	var b strings.Builder
+
 	var maxLen, cmdLen int
 	var keys []string
 
@@ -36,34 +43,45 @@ func printHelp(commands map[string]Command) {
 		}
 	}
 
-	fmt.Printf("usage:\n  lume <command> [<args...>]")
-	fmt.Println()
-	fmt.Println("\ncommands:")
+	fmt.Fprintf(&b, "usage:\n  lume <command> [<args...>]")
+	fmt.Fprintln(&b)
+	fmt.Fprintln(&b, "\ncommands:")
 
 	sort.Strings(keys)
+
 	for _, k := range keys {
 		c := commands[k]
-		fmt.Printf("  %-*s    %s\n", maxLen, c.Name, c.Short)
+		fmt.Fprintf(&b, "  %-*s    %s\n", maxLen, c.Name, c.Short)
 	}
+
+	return b.String()
 }
 
-func printCmdHelp(name string) error {
+func printCmdHelp(name string) (string, error) {
+	var b strings.Builder
+
 	subCmd, ok := commandRegistry[name]
+
 	if !ok {
-		return fmt.Errorf("unknown commnnd: %s\n", name)
+		return "", fmt.Errorf("unknown commnnd: %s\n", name)
 	}
 
 	if subCmd.Use != "" {
-		fmt.Printf("usage:\n  lume %s %s\n", subCmd.Name, subCmd.Use)
+		fmt.Fprintf(&b, "usage:\n  lume %s %s\n", subCmd.Name, subCmd.Use)
 	} else {
-		fmt.Printf("usage:\n  lume %s\n", subCmd.Name)
+		fmt.Fprintf(&b, "usage:\n  lume %s\n", subCmd.Name)
 	}
 
 	if subCmd.Flags != nil {
-		fmt.Println()
-		fmt.Print("flags:\n")
+		out := subCmd.Flags.Output()
+		defer subCmd.Flags.SetOutput(out)
+
+		fmt.Fprintln(&b)
+		fmt.Fprint(&b, "flags:\n")
+
+		subCmd.Flags.SetOutput(&b)
 		subCmd.Flags.PrintDefaults()
 	}
 
-	return nil
+	return b.String(), nil
 }

cmd/ls.go

@@ -2,6 +2,7 @@ package lumecmd
 
 import (
 	"flag"
+	"fmt"
 )
 
 func NewCmdLs() Command {
@@ -14,8 +15,6 @@ func NewCmdLs() Command {
 			selector := fs.String("selector", defaultSelector, "Set the selector")
 			fs.StringVar(selector, "s", defaultSelector, "Set the selector")
 
-			fs.String("format", defaultOutputFormat, "Set the output format")
-
 			return fs
 		}(),
 		Use:   "[--selector=<selector>]",
@@ -23,24 +22,28 @@ func NewCmdLs() Command {
 	}
 }
 
-func LsCmd(args CmdArgs) (int, error) {
+func LsCmd(ctx Context) (int, error) {
 	var p Printer
 
-	c := args.Client
-	selector := args.Flags.String("selector")
-	format := args.Flags.String("format")
+	c := ctx.Client
+	selector := ctx.Flags.String("selector")
+	format, err := getOutputFormatFromFlags(ctx.Flags)
+	if err != nil {
+		return ExitFailure, err
+	}
 
-	if format == "" && args.Config.OutputFormat != "" {
-		format = args.Config.OutputFormat
+	if format == "" && ctx.Config.OutputFormat != "" {
+		format = ctx.Config.OutputFormat
 	}
 
 	lights, err := c.ListLights(selector)
+
 	if err != nil {
 		return ExitFailure, err
 	}
 
 	p = NewPrinter(format)
-	p.Lights(lights)
+	fmt.Print(p.Lights(lights))
 
 	return ExitSuccess, nil
 }

cmd/main.go

@@ -4,7 +4,6 @@ import (
 	"errors"
 	"flag"
 	"fmt"
-	"os"
 
 	"git.kill0.net/chill9/lifx-go"
 )
@@ -20,6 +19,8 @@ func init() {
 	RegisterCommand(NewCmdShow())
 	RegisterCommand(NewCmdToggle())
 	RegisterCommand(NewCmdVersion())
+	RegisterCommand(NewCmdBreathe())
+	RegisterCommand(NewCmdValidate())
 }
 
 var Version string
@@ -27,9 +28,11 @@ var BuildDate string
 var GitCommit string
 
 func Main(args []string) (int, error) {
-	var config *Config
+	var config *Config = GetConfig()
 	var err error
 
+	flag.Parse()
+
 	if len(args) == 1 {
 		args = append(args, "help")
 	}
@@ -43,26 +46,21 @@ func Main(args []string) (int, error) {
 	if config, err = LoadConfigFile(configPath); err != nil {
 		return ExitFailure, err
 	}
-
-	envAccessToken := os.Getenv("LIFX_ACCESS_TOKEN")
-	if envAccessToken != "" {
-		config.AccessToken = envAccessToken
-	}
+	config.MergeWithEnv()
 
 	if err = config.Validate(); err != nil {
 		return ExitFailure, fmt.Errorf("fatal: %s", err)
 	}
 
-	flag.Parse()
-
 	command := args[1]
 
 	c := lifx.NewClient(
 		config.AccessToken,
 		lifx.WithUserAgent(config.userAgent),
+		lifx.WithDebug(config.Debug),
 	)
 
-	cmdArgs := CmdArgs{
+	ctx := Context{
 		Client: c,
 		Config: *config,
 		Args:   args[2:],
@@ -77,11 +75,11 @@ func Main(args []string) (int, error) {
 	fs := cmd.Flags
 	if fs != nil {
 		fs.Parse(args[2:])
-		cmdArgs.Flags = Flags{FlagSet: fs}
+		ctx.Flags = Flags{FlagSet: fs}
 	}
-	cmdArgs.Name = command
+	ctx.Name = command
 
-	exitCode, err := cmd.Func(cmdArgs)
+	exitCode, err := cmd.Func(ctx)
 	if err != nil {
 		err = fmt.Errorf("fatal: %s", err)
 	}

cmd/poweroff.go

@@ -2,6 +2,7 @@ package lumecmd
 
 import (
 	"flag"
+	"fmt"
 
 	"git.kill0.net/chill9/lifx-go"
 )
@@ -19,8 +20,6 @@ func NewCmdPoweroff() Command {
 			selector := fs.String("selector", defaultSelector, "Set the selector")
 			fs.StringVar(selector, "s", defaultSelector, "Set the selector")
 
-			fs.String("format", defaultOutputFormat, "Set the output format")
-
 			return fs
 		}(),
 		Use:   "[--selector <selector>] [--duration <sec>]",
@@ -28,17 +27,20 @@ func NewCmdPoweroff() Command {
 	}
 }
 
-func PoweroffCmd(args CmdArgs) (int, error) {
+func PoweroffCmd(ctx Context) (int, error) {
 	var p Printer
 
-	c := args.Client
-	duration := args.Flags.Float64("duration")
-	selector := args.Flags.String("selector")
-	format := args.Flags.String("format")
+	c := ctx.Client
+	duration := ctx.Flags.Float64("duration")
+	selector := ctx.Flags.String("selector")
 	state := lifx.State{Power: "off", Duration: duration}
+	format, err := getOutputFormatFromFlags(ctx.Flags)
+	if err != nil {
+		return ExitFailure, err
+	}
 
-	if format == "" && args.Config.OutputFormat != "" {
-		format = args.Config.OutputFormat
+	if format == "" && ctx.Config.OutputFormat != "" {
+		format = ctx.Config.OutputFormat
 	}
 
 	r, err := c.SetState(selector, state)
@@ -47,7 +49,7 @@ func PoweroffCmd(args CmdArgs) (int, error) {
 	}
 
 	p = NewPrinter(format)
-	p.Results(r.Results)
+	fmt.Print(p.Results(r.Results))
 
 	return ExitSuccess, nil
 }

cmd/poweron.go

@@ -2,6 +2,7 @@ package lumecmd
 
 import (
 	"flag"
+	"fmt"
 
 	"git.kill0.net/chill9/lifx-go"
 )
@@ -19,8 +20,6 @@ func NewCmdPoweron() Command {
 			selector := fs.String("selector", defaultSelector, "Set the selector")
 			fs.StringVar(selector, "s", defaultSelector, "Set the selector")
 
-			fs.String("format", defaultOutputFormat, "Set the output format")
-
 			return fs
 		}(),
 		Use:   "[--selector <selector>] [--duration <sec>]",
@@ -28,17 +27,20 @@ func NewCmdPoweron() Command {
 	}
 }
 
-func PoweronCmd(args CmdArgs) (int, error) {
+func PoweronCmd(ctx Context) (int, error) {
 	var p Printer
 
-	c := args.Client
-	duration := args.Flags.Float64("duration")
-	selector := args.Flags.String("selector")
-	format := args.Flags.String("format")
+	c := ctx.Client
+	duration := ctx.Flags.Float64("duration")
+	selector := ctx.Flags.String("selector")
 	state := lifx.State{Power: "on", Duration: duration}
+	format, err := getOutputFormatFromFlags(ctx.Flags)
+	if err != nil {
+		return ExitFailure, err
+	}
 
-	if format == "" && args.Config.OutputFormat != "" {
-		format = args.Config.OutputFormat
+	if format == "" && ctx.Config.OutputFormat != "" {
+		format = ctx.Config.OutputFormat
 	}
 
 	r, err := c.SetState(selector, state)
@@ -47,7 +49,7 @@ func PoweronCmd(args CmdArgs) (int, error) {
 	}
 
 	p = NewPrinter(format)
-	p.Results(r.Results)
+	fmt.Print(p.Results(r.Results))
 
 	return ExitSuccess, nil
 }

cmd/print.go

@@ -2,7 +2,8 @@ package lumecmd
 
 import (
 	"fmt"
-	"os"
+	"io"
+	"strings"
 	"time"
 
 	"git.kill0.net/chill9/lifx-go"
@@ -11,8 +12,8 @@ import (
 )
 
 type Printer interface {
-	Results(results []lifx.Result)
-	Lights(lights []lifx.Light)
+	Results(results []lifx.Result) string
+	Lights(lights []lifx.Light) string
 }
 
 type defaultPrinter struct{}
@@ -28,17 +29,19 @@ func NewPrinter(format string) Printer {
 	}
 }
 
-func (dp *defaultPrinter) Results(results []lifx.Result) {
+func (dp *defaultPrinter) Results(results []lifx.Result) string {
+	var b strings.Builder
+
 	sortResults(results)
 
-	table := tablewriter.NewWriter(os.Stdout)
+	table := tablewriter.NewWriter(&b)
 	_, rows := makeResultsTable(results)
 
 	for _, v := range rows {
 		table.Append(v)
 	}
 
-	fmt.Printf("total %d\n", len(results))
+	fmt.Fprintf(&b, "total %d\n", len(results))
 	table.SetAlignment(tablewriter.ALIGN_LEFT)
 	table.SetAutoWrapText(false)
 	table.SetBorder(false)
@@ -49,12 +52,16 @@ func (dp *defaultPrinter) Results(results []lifx.Result) {
 	table.SetRowSeparator("")
 	table.SetTablePadding(" ")
 	table.Render()
+
+	return b.String()
 }
 
-func (tp *tablePrinter) Results(results []lifx.Result) {
+func (tp *tablePrinter) Results(results []lifx.Result) string {
+	var b strings.Builder
+
 	sortResults(results)
 
-	table := tablewriter.NewWriter(os.Stdout)
+	table := tablewriter.NewWriter(&b)
 	hdr, rows := makeResultsTable(results)
 
 	for _, v := range rows {
@@ -63,19 +70,23 @@ func (tp *tablePrinter) Results(results []lifx.Result) {
 
 	table.SetHeader(hdr)
 	table.Render()
+
+	return b.String()
 }
 
-func (dp *defaultPrinter) Lights(lights []lifx.Light) {
+func (dp *defaultPrinter) Lights(lights []lifx.Light) string {
+	var b strings.Builder
+
 	sortLights(lights)
 
-	table := tablewriter.NewWriter(os.Stdout)
+	table := tablewriter.NewWriter(&b)
 	_, rows := makeLightsTable(lights)
 
 	for _, v := range rows {
 		table.Append(v)
 	}
 
-	fmt.Printf("total %d\n", len(lights))
+	fmt.Fprintf(&b, "total %d\n", len(lights))
 	table.SetAlignment(tablewriter.ALIGN_LEFT)
 	table.SetAutoWrapText(false)
 	table.SetBorder(false)
@@ -86,12 +97,16 @@ func (dp *defaultPrinter) Lights(lights []lifx.Light) {
 	table.SetRowSeparator("")
 	table.SetTablePadding(" ")
 	table.Render()
+
+	return b.String()
 }
 
-func (tp *tablePrinter) Lights(lights []lifx.Light) {
+func (tp *tablePrinter) Lights(lights []lifx.Light) string {
+	var b strings.Builder
+
 	sortLights(lights)
 
-	table := tablewriter.NewWriter(os.Stdout)
+	table := tablewriter.NewWriter(&b)
 	hdr, rows := makeLightsTable(lights)
 
 	for _, v := range rows {
@@ -100,6 +115,17 @@ func (tp *tablePrinter) Lights(lights []lifx.Light) {
 
 	table.SetHeader(hdr)
 	table.Render()
+
+	return b.String()
+}
+
+func ColorizeIndicator(s string) string {
+	c := color.New(color.FgRed)
+	if s == "on" {
+		c = color.New(color.FgGreen)
+	}
+
+	return c.Sprint(GetConfig().Indicator)
 }
 
 func ColorizePower(s string) string {
@@ -129,11 +155,17 @@ func PrintfWithIndent(indent int, format string, a ...interface{}) (n int, err e
 	return fmt.Printf(format, a...)
 }
 
+func FprintfWithIndent(w io.Writer, indent int, format string, a ...interface{}) (n int, err error) {
+	format = fmt.Sprintf("%*s%s", indent, "", format)
+	return fmt.Fprintf(w, format, a...)
+}
+
 func makeLightsTable(lights []lifx.Light) (hdr []string, rows [][]string) {
-	hdr = []string{"ID", "Location", "Group", "Label", "Last Seen", "Power"}
+	hdr = []string{"", "ID", "Location", "Group", "Label", "Last Seen", "Power"}
 
 	for _, l := range lights {
 		rows = append(rows, []string{
+			fmt.Sprint(ColorizeIndicator(l.Power)),
 			fmt.Sprint(l.Id),
 			fmt.Sprint(l.Location.Name),
 			fmt.Sprint(l.Group.Name),

cmd/setcolor.go

@@ -41,8 +41,6 @@ func NewCmdSetColor() Command {
 			fast := fs.Bool("fast", defaultFast, "fast state")
 			fs.BoolVar(fast, "f", defaultFast, "fast state")
 
-			fs.String("format", defaultOutputFormat, "Set the output format")
-
 			return fs
 		}(),
 		Use:   "[--selector <selector>] [--power (on|off)] [--hue <hue>] [--saturation <saturation>] [--rgb <rbg>] [--name <color>] [--brightness <brightness>] [--duration <sec>] [--fast]",
@@ -50,30 +48,33 @@ func NewCmdSetColor() Command {
 	}
 }
 
-func SetColorCmd(args CmdArgs) (int, error) {
+func SetColorCmd(ctx Context) (int, error) {
 	var p Printer
 
-	c := args.Client
+	c := ctx.Client
 	state := lifx.State{}
-	selector := args.Flags.String("selector")
-	format := args.Flags.String("format")
-
-	if format == "" && args.Config.OutputFormat != "" {
-		format = args.Config.OutputFormat
+	selector := ctx.Flags.String("selector")
+	format, err := getOutputFormatFromFlags(ctx.Flags)
+	if err != nil {
+		return ExitFailure, err
 	}
 
-	power := args.Flags.String("power")
+	if format == "" && ctx.Config.OutputFormat != "" {
+		format = ctx.Config.OutputFormat
+	}
+
+	power := ctx.Flags.String("power")
 	if power != "" {
 		state.Power = power
 	}
 
-	hueFlag := args.Flags.String("hue")
-	saturationFlag := args.Flags.String("saturation")
-	rgbFlag := args.Flags.String("rgb")
-	name := args.Flags.String("name")
+	hueFlag := ctx.Flags.String("hue")
+	saturationFlag := ctx.Flags.String("saturation")
+	rgbFlag := ctx.Flags.String("rgb")
+	name := ctx.Flags.String("name")
 
 	if (hueFlag == "" || saturationFlag == "") && rgbFlag == "" && name == "" {
-		printCmdHelp(args.Name)
+		printCmdHelp(ctx.Name)
 		return ExitFailure, nil
 	}
 
@@ -81,12 +82,12 @@ func SetColorCmd(args CmdArgs) (int, error) {
 		color := lifx.HSBKColor{}
 
 		if hueFlag != "" {
-			hue := args.Flags.Float32("hue")
+			hue := ctx.Flags.Float32("hue")
 			color.H = lifx.Float32Ptr(hue)
 		}
 
 		if saturationFlag != "" {
-			saturation := args.Flags.Float32("saturation")
+			saturation := ctx.Flags.Float32("saturation")
 			color.S = lifx.Float32Ptr(saturation)
 		}
 		state.Color = color
@@ -98,7 +99,7 @@ func SetColorCmd(args CmdArgs) (int, error) {
 		}
 		state.Color = color
 	} else if name != "" {
-		hsb, ok := args.Config.Colors[name]
+		hsb, ok := ctx.Config.Colors[name]
 		if !ok {
 			return ExitFailure, fmt.Errorf("%s is not a defined color", name)
 		}
@@ -109,16 +110,16 @@ func SetColorCmd(args CmdArgs) (int, error) {
 		state.Color = color
 	}
 
-	brightnessFlag := args.Flags.String("brightness")
+	brightnessFlag := ctx.Flags.String("brightness")
 	if brightnessFlag != "" {
-		brightness := args.Flags.Float64("brightness")
+		brightness := ctx.Flags.Float64("brightness")
 		state.Brightness = brightness
 	}
 
-	duration := args.Flags.Float64("duration")
+	duration := ctx.Flags.Float64("duration")
 	state.Duration = duration
 
-	fast := args.Flags.Bool("fast")
+	fast := ctx.Flags.Bool("fast")
 	state.Fast = fast
 
 	r, err := c.SetState(selector, state)
@@ -129,7 +130,7 @@ func SetColorCmd(args CmdArgs) (int, error) {
 
 	if !fast {
 		p = NewPrinter(format)
-		p.Results(r.Results)
+		fmt.Print(p.Results(r.Results))
 	}
 
 	return ExitSuccess, nil

cmd/setstate.go

@@ -2,6 +2,7 @@ package lumecmd
 
 import (
 	"flag"
+	"fmt"
 
 	"git.kill0.net/chill9/lifx-go"
 )
@@ -34,8 +35,6 @@ func NewCmdSetState() Command {
 			fast := fs.Bool("fast", defaultFast, "fast state")
 			fs.BoolVar(fast, "f", defaultFast, "fast state")
 
-			fs.String("format", defaultOutputFormat, "Set the output format")
-
 			return fs
 		}(),
 		Use:   "[--selector <selector>] [--power (on|off)] [--color <color>] [--brightness <brightness>] [--duration <sec>] [--infrared <infrared>] [--fast]",
@@ -43,48 +42,51 @@ func NewCmdSetState() Command {
 	}
 }
 
-func SetStateCmd(args CmdArgs) (int, error) {
+func SetStateCmd(ctx Context) (int, error) {
 	var p Printer
 
-	c := args.Client
+	c := ctx.Client
 	state := lifx.State{}
-	selector := args.Flags.String("selector")
-	format := args.Flags.String("format")
-
-	if format == "" && args.Config.OutputFormat != "" {
-		format = args.Config.OutputFormat
+	selector := ctx.Flags.String("selector")
+	format, err := getOutputFormatFromFlags(ctx.Flags)
+	if err != nil {
+		return ExitFailure, err
 	}
 
-	power := args.Flags.String("power")
+	if format == "" && ctx.Config.OutputFormat != "" {
+		format = ctx.Config.OutputFormat
+	}
+
+	power := ctx.Flags.String("power")
 	if power != "" {
 		state.Power = power
 	}
 
-	color := args.Flags.String("color")
+	color := ctx.Flags.String("color")
 	if color != "" {
 		state.Color = lifx.NamedColor(color)
 	}
 
-	brightnessFlag := args.Flags.String("brightness")
+	brightnessFlag := ctx.Flags.String("brightness")
 	if brightnessFlag != "" {
-		brightness := args.Flags.Float64("brightness")
+		brightness := ctx.Flags.Float64("brightness")
 		state.Brightness = brightness
 	}
 
-	duration := args.Flags.Float64("duration")
+	duration := ctx.Flags.Float64("duration")
 	state.Duration = duration
 
-	infraredFlag := args.Flags.String("infrared")
+	infraredFlag := ctx.Flags.String("infrared")
 	if infraredFlag != "" {
-		infrared := args.Flags.Float64("infrared")
+		infrared := ctx.Flags.Float64("infrared")
 		state.Infrared = infrared
 	}
 
-	fast := args.Flags.Bool("fast")
+	fast := ctx.Flags.Bool("fast")
 	state.Fast = fast
 
 	if power == "" && color == "" && brightnessFlag == "" && infraredFlag == "" {
-		printCmdHelp(args.Name)
+		printCmdHelp(ctx.Name)
 		return ExitFailure, nil
 	}
 
@@ -95,7 +97,7 @@ func SetStateCmd(args CmdArgs) (int, error) {
 
 	if !fast {
 		p = NewPrinter(format)
-		p.Results(r.Results)
+		fmt.Print(p.Results(r.Results))
 	}
 
 	return ExitSuccess, nil

cmd/setwhite.go

@@ -2,6 +2,7 @@ package lumecmd
 
 import (
 	"flag"
+	"fmt"
 
 	"git.kill0.net/chill9/lifx-go"
 )
@@ -37,8 +38,6 @@ func NewCmdSetWhite() Command {
 			fast := fs.Bool("fast", defaultFast, "fast state")
 			fs.BoolVar(fast, "f", defaultFast, "fast state")
 
-			fs.String("format", defaultOutputFormat, "Set the output format")
-
 			return fs
 		}(),
 		Use:   "[--selector <selector>] [--power (on|off)] [--kelvin <kelvin>] [--name <color>] [--brightness <brightness>] [--duration <sec>] [--infrared] [--fast]",
@@ -46,26 +45,29 @@ func NewCmdSetWhite() Command {
 	}
 }
 
-func SetWhiteCmd(args CmdArgs) (int, error) {
+func SetWhiteCmd(ctx Context) (int, error) {
 	var p Printer
 
-	c := args.Client
+	c := ctx.Client
 	state := lifx.State{}
-	selector := args.Flags.String("selector")
-	format := args.Flags.String("format")
-
-	if format == "" && args.Config.OutputFormat != "" {
-		format = args.Config.OutputFormat
+	selector := ctx.Flags.String("selector")
+	format, err := getOutputFormatFromFlags(ctx.Flags)
+	if err != nil {
+		return ExitFailure, err
 	}
 
-	power := args.Flags.String("power")
+	if format == "" && ctx.Config.OutputFormat != "" {
+		format = ctx.Config.OutputFormat
+	}
+
+	power := ctx.Flags.String("power")
 	if power != "" {
 		state.Power = power
 	}
 
-	kelvinFlag := args.Flags.String("kelvin")
+	kelvinFlag := ctx.Flags.String("kelvin")
 	if kelvinFlag != "" {
-		kelvin := args.Flags.Int16("kelvin")
+		kelvin := ctx.Flags.Int16("kelvin")
 		color, err := lifx.NewWhite(kelvin)
 		if err != nil {
 			return ExitFailure, err
@@ -73,9 +75,9 @@ func SetWhiteCmd(args CmdArgs) (int, error) {
 		state.Color = color
 	}
 
-	name := args.Flags.String("name")
+	name := ctx.Flags.String("name")
 	if name != "" {
-		name := args.Flags.String("name")
+		name := ctx.Flags.String("name")
 		color, err := lifx.NewWhiteString(name)
 		if err != nil {
 			return ExitFailure, err
@@ -83,26 +85,26 @@ func SetWhiteCmd(args CmdArgs) (int, error) {
 		state.Color = color
 	}
 
-	brightnessFlag := args.Flags.String("brightness")
+	brightnessFlag := ctx.Flags.String("brightness")
 	if brightnessFlag != "" {
-		brightness := args.Flags.Float64("brightness")
+		brightness := ctx.Flags.Float64("brightness")
 		state.Brightness = brightness
 	}
 
-	duration := args.Flags.Float64("duration")
+	duration := ctx.Flags.Float64("duration")
 	state.Duration = duration
 
-	infraredFlag := args.Flags.String("infrared")
+	infraredFlag := ctx.Flags.String("infrared")
 	if infraredFlag != "" {
-		infrared := args.Flags.Float64("infrared")
+		infrared := ctx.Flags.Float64("infrared")
 		state.Infrared = infrared
 	}
 
-	fast := args.Flags.Bool("fast")
+	fast := ctx.Flags.Bool("fast")
 	state.Fast = fast
 
 	if power == "" && kelvinFlag == "" && name == "" && brightnessFlag == "" && infraredFlag == "" {
-		printCmdHelp(args.Name)
+		printCmdHelp(ctx.Name)
 		return ExitFailure, nil
 	}
 
@@ -113,7 +115,7 @@ func SetWhiteCmd(args CmdArgs) (int, error) {
 
 	if !fast {
 		p = NewPrinter(format)
-		p.Results(r.Results)
+		fmt.Print(p.Results(r.Results))
 	}
 
 	return ExitSuccess, nil

cmd/show.go

@@ -3,6 +3,7 @@ package lumecmd
 import (
 	"flag"
 	"fmt"
+	"strings"
 )
 
 const Tabstop int = 2
@@ -24,10 +25,12 @@ func NewCmdShow() Command {
 	}
 }
 
-func ShowCmd(args CmdArgs) (int, error) {
+func ShowCmd(ctx Context) (int, error) {
 	var indent int
-	c := args.Client
-	selector := args.Flags.String("selector")
+	var b strings.Builder
+
+	c := ctx.Client
+	selector := ctx.Flags.String("selector")
 	lights, err := c.ListLights(selector)
 
 	if err != nil {
@@ -38,48 +41,52 @@ func ShowCmd(args CmdArgs) (int, error) {
 
 	for i, l := range lights {
 		indent = 0
-		fmt.Printf(
-			"Light ID: %s, %s, Power: %s\n",
+		fmt.Fprintf(
+			&b,
+			"%s Light ID: %s, %s, Power: %s\n",
+			ColorizeIndicator(l.Power),
 			l.Id,
 			connected(l.Connected),
 			ColorizePower(l.Power),
 		)
-		indent += Tabstop
-		PrintfWithIndent(indent, "Label: %s, ID: %s\n", l.Label, l.Id)
-		PrintfWithIndent(indent, "UUID: %s\n", l.UUID)
-		PrintfWithIndent(indent, "Location: %s, ID: %s\n", l.Location.Name, l.Location.Id)
-		PrintfWithIndent(indent, "Group: %s, ID: %s\n", l.Group.Name, l.Group.Id)
-		PrintfWithIndent(indent, "Color: Hue: %.1f, Saturation: %.1f%%, Kelvin: %d\n",
+		indent += Tabstop + 2
+		FprintfWithIndent(&b, indent, "Label: %s, ID: %s\n", l.Label, l.Id)
+		FprintfWithIndent(&b, indent, "UUID: %s\n", l.UUID)
+		FprintfWithIndent(&b, indent, "Location: %s, ID: %s\n", l.Location.Name, l.Location.Id)
+		FprintfWithIndent(&b, indent, "Group: %s, ID: %s\n", l.Group.Name, l.Group.Id)
+		FprintfWithIndent(&b, indent, "Color: Hue: %.1f, Saturation: %.1f%%, Kelvin: %d\n",
 			*l.Color.H, *l.Color.S, *l.Color.K)
-		PrintfWithIndent(indent, "Brightness: %.1f%%\n", l.Brightness*100)
+		FprintfWithIndent(&b, indent, "Brightness: %.1f%%\n", l.Brightness*100)
 		if l.Effect != "" {
-			PrintfWithIndent(indent, "Effect: %s\n", l.Effect)
+			FprintfWithIndent(&b, indent, "Effect: %s\n", l.Effect)
 		}
-		PrintfWithIndent(indent, "Product: %s\n", l.Product.Name)
-		PrintfWithIndent(indent, "Capabilities: ")
-		fmt.Printf("Color: %s, ", YesNo(l.Product.Capabilities.HasColor))
-		fmt.Printf("Variable Color Temp: %s, ", YesNo(l.Product.Capabilities.HasVariableColorTemp))
-		fmt.Printf("IR: %s, ", YesNo(l.Product.Capabilities.HasIR))
-		fmt.Printf("Chain: %s, ", YesNo(l.Product.Capabilities.HasChain))
-		fmt.Printf("Multizone: %s, ", YesNo(l.Product.Capabilities.HasMultizone))
-		fmt.Printf("Min Kelvin: %.1f, ", l.Product.Capabilities.MinKelvin)
-		fmt.Printf("Max Kelvin: %.1f ", l.Product.Capabilities.MaxKelvin)
-		fmt.Println()
+		FprintfWithIndent(&b, indent, "Product: %s\n", l.Product.Name)
+		FprintfWithIndent(&b, indent, "Capabilities: ")
+		fmt.Fprintf(&b, "Color: %s, ", YesNo(l.Product.Capabilities.HasColor))
+		fmt.Fprintf(&b, "Variable Color Temp: %s, ", YesNo(l.Product.Capabilities.HasVariableColorTemp))
+		fmt.Fprintf(&b, "IR: %s, ", YesNo(l.Product.Capabilities.HasIR))
+		fmt.Fprintf(&b, "Chain: %s, ", YesNo(l.Product.Capabilities.HasChain))
+		fmt.Fprintf(&b, "Multizone: %s, ", YesNo(l.Product.Capabilities.HasMultizone))
+		fmt.Fprintf(&b, "Min Kelvin: %.1f, ", l.Product.Capabilities.MinKelvin)
+		fmt.Fprintf(&b, "Max Kelvin: %.1f ", l.Product.Capabilities.MaxKelvin)
+		fmt.Fprintln(&b)
 		// List applicable selectors (most to least specific)
-		PrintfWithIndent(indent, "Selectors:\n")
+		FprintfWithIndent(&b, indent, "Selectors:\n")
 		indent += Tabstop
-		PrintfWithIndent(indent, "id:%s\n", l.Id)
-		PrintfWithIndent(indent, "label:%s\n", l.Label)
-		PrintfWithIndent(indent, "group_id:%s\n", l.Group.Id)
-		PrintfWithIndent(indent, "group:%s\n", l.Group.Name)
-		PrintfWithIndent(indent, "location_id:%s\n", l.Location.Id)
-		PrintfWithIndent(indent, "location:%s\n", l.Location.Name)
+		FprintfWithIndent(&b, indent, "id:%s\n", l.Id)
+		FprintfWithIndent(&b, indent, "label:%s\n", l.Label)
+		FprintfWithIndent(&b, indent, "group_id:%s\n", l.Group.Id)
+		FprintfWithIndent(&b, indent, "group:%s\n", l.Group.Name)
+		FprintfWithIndent(&b, indent, "location_id:%s\n", l.Location.Id)
+		FprintfWithIndent(&b, indent, "location:%s\n", l.Location.Name)
 		indent -= Tabstop
-		PrintfWithIndent(indent, "Last Seen: %s (%.1fs ago)\n", l.LastSeen, l.SecondsLastSeen)
+		FprintfWithIndent(&b, indent, "Last Seen: %s (%.1fs ago)\n", l.LastSeen, l.SecondsLastSeen)
 
 		if i < len(lights)-1 {
-			fmt.Println()
+			fmt.Fprintln(&b)
 		}
+
+		fmt.Print(b.String())
 	}
 	return ExitSuccess, nil
 }

cmd/toggle.go

@@ -2,6 +2,7 @@ package lumecmd
 
 import (
 	"flag"
+	"fmt"
 )
 
 func NewCmdToggle() Command {
@@ -17,8 +18,6 @@ func NewCmdToggle() Command {
 			selector := fs.String("selector", defaultSelector, "Set the selector")
 			fs.StringVar(selector, "s", defaultSelector, "Set the selector")
 
-			fs.String("format", defaultOutputFormat, "Set the output format")
-
 			return fs
 		}(),
 		Use:   "[--selector <selector>] [--duration <sec>]",
@@ -26,16 +25,19 @@ func NewCmdToggle() Command {
 	}
 }
 
-func ToggleCmd(args CmdArgs) (int, error) {
+func ToggleCmd(ctx Context) (int, error) {
 	var p Printer
 
-	c := args.Client
-	duration := args.Flags.Float64("duration")
-	selector := args.Flags.String("selector")
-	format := args.Flags.String("format")
+	c := ctx.Client
+	duration := ctx.Flags.Float64("duration")
+	selector := ctx.Flags.String("selector")
+	format, err := getOutputFormatFromFlags(ctx.Flags)
+	if err != nil {
+		return ExitFailure, err
+	}
 
-	if format == "" && args.Config.OutputFormat != "" {
-		format = args.Config.OutputFormat
+	if format == "" && ctx.Config.OutputFormat != "" {
+		format = ctx.Config.OutputFormat
 	}
 
 	r, err := c.Toggle(selector, duration)
@@ -44,7 +46,7 @@ func ToggleCmd(args CmdArgs) (int, error) {
 	}
 
 	p = NewPrinter(format)
-	p.Results(r.Results)
+	fmt.Print(p.Results(r.Results))
 
 	return ExitSuccess, nil
 }

cmd/util.go

@@ -60,3 +60,9 @@ func YesNo(v bool) string {
 	}
 	return "no"
 }
+
+func Debugf(format string, a ...interface{}) {
+	if GetConfig().Debug {
+		fmt.Printf(format, a...)
+	}
+}

cmd/validate.go

@@ -0,0 +1,51 @@
+package lumecmd
+
+import (
+	"errors"
+	"flag"
+	"fmt"
+	"strings"
+
+	"git.kill0.net/chill9/lifx-go"
+)
+
+func NewCmdValidate() Command {
+	return Command{
+		Name: "validate",
+		Func: ValidateCmd,
+		Flags: func() *flag.FlagSet {
+			fs := flag.NewFlagSet("validate", flag.ExitOnError)
+
+			return fs
+		}(),
+		Use:   "<command>",
+		Short: "Validate a color string",
+	}
+}
+
+func ValidateCmd(ctx Context) (int, error) {
+	var b strings.Builder
+	c := ctx.Client
+
+	if len(ctx.Args) != 1 {
+		fmt.Print(printCmdHelp(ctx.Name))
+		return ExitFailure, nil
+	}
+
+	color := lifx.NamedColor(ctx.Args[0])
+
+	i, err := c.ValidateColor(color)
+	if err != nil {
+		return ExitFailure, err
+	}
+
+	if validColor, ok := i.(*lifx.HSBKColor); ok {
+		fmt.Fprintln(&b, validColor)
+	} else {
+		return ExitFailure, errors.New("go type %T but wanted *HSBKColor")
+	}
+
+	fmt.Print(b.String())
+
+	return ExitSuccess, nil
+}

cmd/version.go

@@ -2,6 +2,7 @@ package lumecmd
 
 import (
 	"fmt"
+	"runtime"
 	"strings"
 )
 
@@ -15,19 +16,23 @@ func NewCmdVersion() Command {
 	}
 }
 
-func VersionCmd(args CmdArgs) (int, error) {
+func VersionCmd(ctx Context) (int, error) {
 	var b strings.Builder
 
-	fmt.Fprintf(&b, "lume %s", Version)
-	b.WriteString(" ")
+	fmt.Fprintf(&b, "lume, version %s\n", Version)
+
 	if GitCommit != "" {
-		fmt.Fprintf(&b, "(git: %s)", GitCommit)
-		b.WriteString(" ")
-	}
-	if BuildDate != "" {
-		fmt.Fprintf(&b, "build_date: %s", BuildDate)
+		fmt.Fprintf(&b, "  revision:   %s\n", GitCommit)
 	}
 
-	fmt.Println(b.String())
+	if BuildDate != "" {
+		fmt.Fprintf(&b, "  build date: %s\n", BuildDate)
+	}
+
+	fmt.Fprintf(&b, "  go version: %s\n", runtime.Version())
+	fmt.Fprintf(&b, "  platform:   %s/%s\n", runtime.GOOS, runtime.GOARCH)
+
+	fmt.Print(b.String())
+
 	return ExitSuccess, nil
 }

go.mod

@@ -3,9 +3,11 @@ module git.kill0.net/chill9/lume
 go 1.15
 
 require (
-	git.kill0.net/chill9/lifx-go v0.0.0-20210215004437-f86c28b0a5ef
+	git.kill0.net/chill9/lifx-go v0.0.0-20210418161634-4c1678b62c73
 	github.com/BurntSushi/toml v0.3.1
 	github.com/fatih/color v1.10.0
+	github.com/mattn/go-runewidth v0.0.10 // indirect
 	github.com/olekukonko/tablewriter v0.0.5
-	golang.org/x/sys v0.0.0-20210110051926-789bb1bd4061
+	github.com/rivo/uniseg v0.2.0 // indirect
+	golang.org/x/sys v0.0.0-20210326220804-49726bf1d181
 )

go.sum

@@ -1,5 +1,11 @@
 git.kill0.net/chill9/lifx-go v0.0.0-20210215004437-f86c28b0a5ef h1:8yyXAk+qiRvrowTeHHAOBZqGXXCr8SxWFydez61ZGp8=
 git.kill0.net/chill9/lifx-go v0.0.0-20210215004437-f86c28b0a5ef/go.mod h1:ZFKIcwdJ4Nqlrkn/eUHbeLt0NVhFsfxBREkVoA+jzUc=
+git.kill0.net/chill9/lifx-go v0.0.0-20210323044657-dbe1c40e1621 h1:koWq2W08HjmvsNm4D8kD0OjMZKCJ/3681P9XFMTj77I=
+git.kill0.net/chill9/lifx-go v0.0.0-20210323044657-dbe1c40e1621/go.mod h1:jInpjEqTBhrFpQKk7zPIWISvgjjfS2djXeKB3yB/8dY=
+git.kill0.net/chill9/lifx-go v0.0.0-20210329222320-2107a0586447 h1:tN+zR5aszesrZRrhS3uOqAIWLcADCIH7GFJ6SOQS9r0=
+git.kill0.net/chill9/lifx-go v0.0.0-20210329222320-2107a0586447/go.mod h1:jInpjEqTBhrFpQKk7zPIWISvgjjfS2djXeKB3yB/8dY=
+git.kill0.net/chill9/lifx-go v0.0.0-20210418161634-4c1678b62c73 h1:fteCAelwAcfam2Q8eeJFyK4+sXGOpR6Me5YMKBi+MYY=
+git.kill0.net/chill9/lifx-go v0.0.0-20210418161634-4c1678b62c73/go.mod h1:jInpjEqTBhrFpQKk7zPIWISvgjjfS2djXeKB3yB/8dY=
 github.com/BurntSushi/toml v0.3.1 h1:WXkYYl6Yr3qBf1K79EBnL4mak0OimBfB0XUf9Vl28OQ=
 github.com/BurntSushi/toml v0.3.1/go.mod h1:xHWCNGjB5oqiDr8zfno3MHue2Ht5sIBksp03qcyfWMU=
 github.com/fatih/color v1.10.0 h1:s36xzo75JdqLaaWoiEHk767eHiwo0598uUxyfiPkDsg=
@@ -10,9 +16,18 @@ github.com/mattn/go-isatty v0.0.12 h1:wuysRhFDzyxgEmMf5xjvJ2M9dZoWAXNNr5LSBS7uHX
 github.com/mattn/go-isatty v0.0.12/go.mod h1:cbi8OIDigv2wuxKPP5vlRcQ1OAZbq2CE4Kysco4FUpU=
 github.com/mattn/go-runewidth v0.0.9 h1:Lm995f3rfxdpd6TSmuVCHVb/QhupuXlYr8sCI/QdE+0=
 github.com/mattn/go-runewidth v0.0.9/go.mod h1:H031xJmbD/WCDINGzjvQ9THkh0rPKHF+m2gUSrubnMI=
+github.com/mattn/go-runewidth v0.0.10 h1:CoZ3S2P7pvtP45xOtBw+/mDL2z0RKI576gSkzRRpdGg=
+github.com/mattn/go-runewidth v0.0.10/go.mod h1:RAqKPSqVFrSLVXbA8x7dzmKdmGzieGRCM46jaSJTDAk=
 github.com/olekukonko/tablewriter v0.0.5 h1:P2Ga83D34wi1o9J6Wh1mRuqd4mF/x/lgBS7N7AbDhec=
 github.com/olekukonko/tablewriter v0.0.5/go.mod h1:hPp6KlRPjbx+hW8ykQs1w3UBbZlj6HuIJcUGPhkA7kY=
+github.com/rivo/uniseg v0.1.0/go.mod h1:J6wj4VEh+S6ZtnVlnTBMWIodfgj8LQOQFoIToxlJtxc=
+github.com/rivo/uniseg v0.2.0 h1:S1pD9weZBuJdFmowNwbpi7BJ8TNftyUImj/0WQi72jY=
+github.com/rivo/uniseg v0.2.0/go.mod h1:J6wj4VEh+S6ZtnVlnTBMWIodfgj8LQOQFoIToxlJtxc=
 golang.org/x/sys v0.0.0-20200116001909-b77594299b42/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
 golang.org/x/sys v0.0.0-20200223170610-d5e6a3e2c0ae/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
 golang.org/x/sys v0.0.0-20210110051926-789bb1bd4061 h1:DQmQoKxQWtyybCtX/3dIuDBcAhFszqq8YiNeS6sNu1c=
 golang.org/x/sys v0.0.0-20210110051926-789bb1bd4061/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
+golang.org/x/sys v0.0.0-20210320140829-1e4c9ba3b0c4 h1:EZ2mChiOa8udjfp6rRmswTbtZN/QzUQp4ptM4rnjHvc=
+golang.org/x/sys v0.0.0-20210320140829-1e4c9ba3b0c4/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=
+golang.org/x/sys v0.0.0-20210326220804-49726bf1d181 h1:64ChN/hjER/taL4YJuA+gpLfIMT+/NFherRZixbxOhg=
+golang.org/x/sys v0.0.0-20210326220804-49726bf1d181/go.mod h1:h1NjWce9XRLGQEsW7wpKNCjG9DtNlClVuFLEZdDNbEs=

lume.1

@@ -0,0 +1,186 @@
+.Dd $Mdocdate$
+
+.Dt lume \&1 "User Commands"
+
+.Sh NAME
+.Nm lume
+.Nd CLI tool for the LIFX HTTP API
+
+.Sh SYNOPSIS
+.Nm lume
+.Ar COMMAND
+.Op Ar OPTIONS ...
+
+.Sh COMMANDS
+.Bl -tag -width Ds
+.It Xo Ic breathe
+.Op Fl s Ar selector | Fl Fl selector Ns = Ns Ar selector
+.Fl c | Fl Fl color
+.Op Fl Fl from-color Ns = Ns Ar color
+.Op Fl Fl cycles Ns = Ns Ar cycles
+.Op Fl Fl peak Ns = Ns Ar peak
+.Op Fl Fl period Ns = Ns Ar period
+.Op Fl Fl persist
+.Op Fl Fl power-on
+.Xc
+.It Xo Ic help
+.Op Ar COMMAND
+.Xc
+Print the help message. If a command is specified, then show the subcommand's help message.
+.It Xo Ic ls
+.Op Fl s Ar selector | Fl Fl selector Ns = Ns Ar selector
+.Op Fl Fl simple | Fl Fl table
+.Xc
+List the lights and their basic state
+.It Xo Ic poweroff
+.Op Fl s Ar selector | Fl Fl selector Ns = Ns Ar selector
+.Op Fl Fl simple | Fl Fl table
+.Op Fl d | Fl Fl duration
+.Xc
+Power off lights
+.It Xo Ic poweron
+.Op Fl s Ar selector | Fl Fl selector Ns = Ns Ar selector
+.Op Fl Fl simple | Fl Fl table
+.Op Fl d | Fl Fl duration
+.Xc
+Power off lights
+.It Xo Ic set-color
+.Op Fl b Ar brightness | Fl Fl brightness Ns = Ns Ar brightness
+.Op Fl d | Fl Fl duration
+.Op Fl f Ar fast | Fl Fl fast Ns = Ns Ar fast
+.Op Fl H Ar hue | Fl Fl hue Ns = Ns Ar hue
+.Op Fl n Ar name | Fl Fl name Ns = Ns Ar name
+.Op Fl p Ar power | Fl Fl power Ns = Ns Ar power
+.Op Fl r Ar rgb | Fl Fl rgb Ns = Ns Ar rgb
+.Op Fl S Ar saturation | Fl Fl saturation Ns = Ns Ar saturation
+.Op Fl s Ar selector | Fl Fl selector Ns = Ns Ar selector
+.Op Fl Fl simple | Fl Fl table
+.Xc
+Set light color
+.It Xo Ic set-state
+.Op Fl b Ar brightness | Fl Fl brightness Ns = Ns Ar brightness
+.Op Fl c | Fl Fl color
+.Op Fl d | Fl Fl duration
+.Op Fl f Ar fast | Fl Fl fast Ns = Ns Ar fast
+.Op Fl i Ar infrared | Fl Fl infrared Ns = Ns Ar infrared
+.Op Fl p Ar power | Fl Fl power Ns = Ns Ar power
+.Op Fl s Ar selector | Fl Fl selector Ns = Ns Ar selector
+.Op Fl Fl simple | Fl Fl table
+.Xc
+Set light properties
+.It Xo Ic set-white
+.Op Fl b Ar brightness | Fl Fl brightness Ns = Ns Ar brightness
+.Op Fl d | Fl Fl duration
+.Op Fl f Ar fast | Fl Fl fast Ns = Ns Ar fast
+.Op Fl i Ar infrared | Fl Fl infrared Ns = Ns Ar infrared
+.Op Fl k Ar kelvin | Fl Fl kelvin Ns = Ns Ar kelvin
+.Op Fl n Ar name | Fl Fl name Ns = Ns Ar name
+.Op Fl p Ar power | Fl Fl power Ns = Ns Ar power
+.Op Fl s Ar selector | Fl Fl selector Ns = Ns Ar selector
+.Op Fl Fl simple | Fl Fl table
+.Xc
+Set light white levels
+.It Xo Ic show
+.Op Fl s Ar selector | Fl Fl selector Ns = Ns Ar selector
+.Xc
+Show extended details about the lights
+.It Xo Ic toggle
+.Op Fl d | Fl Fl duration
+.Op Fl s Ar selector | Fl Fl selector Ns = Ns Ar selector
+.Op Fl Fl simple | Fl Fl table
+.Xc
+Toggle the power
+.It Xo Ic validate
+.Ar color_string
+.Xc
+Validate a color string
+.It Xo Ic version
+.Xc
+Print the version
+.El
+
+.Sh OPTIONS 
+.Bl -tag -width Ds
+.It Fl b , Fl Fl brightness Ns = Ns Ar brightness
+The brightness level from 0.0 to 1.0. Overrides any brightness set in color (if any)
+.It Fl c , Fl Fl color Ns = Ns Ar color
+This color to use for the action.
+.Pp
+When used with the
+.Nm breathe
+effect, this
+is the "to" color.
+.It Fl Fl cycles Ns = Ns Ar cycles
+The number of times to repeat the effect. Defaults to 1.0 cycle.
+.It Fl d , Fl Fl duration Ns = Ns Ar duration
+The time in seconds to spend performing the action. Range: 0.0 – 3155760000.0 (100 years).
+.It Fl f , Fl Fl fast
+Execute the action fast without any state checks or waiting for the result from the HTTP API
+.It Fl Fl from-color Ns = Ns Ar color
+The color to start the
+.Nm breathe
+effect from. If this parameter is omitted
+then the color the bulb is currently set to is used instead.
+.It Fl H , Fl Fl hue Ns = Ns Ar hue
+Sets the hue. Range 0 - 360.
+.It Fl i , Fl Fl infrared Ns = Ns Ar infrared
+Sets the maximum  brightness of the infrared channel from 0.0 to 1.0.
+.It Fl k , Fl Fl kelvin Ns = Ns Ar kelvin
+Set the kelvin value. The saturation is automatically set to 0.
+.It Fl n , Fl Fl name Ns = Ns Ar name
+Set the color using a named color from
+.Pa lumerc
+.It Fl Fl peak Ns = Ns Ar peak
+Defines where in a period the target color is at its maximum. Minimum 0.0,
+maximum 1.0. Defaults to 0.5.
+.It Fl Fl period Ns = Ns Ar period
+The time in seconds for one cycle of the
+.Nm breathe
+effect. Defaults to 1.0
+second.
+.It Fl Fl persist
+If false set the light back to its previous value when effect ends, if true
+leave the last effect color.
+.It Fl p , Fl Fl power Ns = Ns Ar [ on | off ]
+Set the power state
+.It Fl Fl power-on
+If true, turn the bulb on if it is not already on.
+.It Fl r , Fl Fl rgb Ns = Ns Ar R,G,B
+Set the color via a comma delimited R,G,B string. Values range from 0 - 255.
+.It Fl S , Fl Fl saturation Ns = Ns Ar saturation
+Set the saturation. Range 0.0 - 1.0.
+.It Fl s , Fl Fl selector Ns = Ns Ar selector
+The selector is used to group lights together belonging in the same account
+.Pp
+Selectors can be in the following format: all, label:[value], id:[value], group_id:[value], group:[value], location_id:[value], location:[value], scene_id:[value]
+.Pp
+The default selector is "all"
+.Sh FILES
+.Bl -tag -width "~/.config/lume/lume.conf" -compact
+.It Pa ~/.lumerc
+Default
+.Nm
+configuration file
+.It Pa ~/.config/lume/lume.conf
+XDG config home
+.Nm
+configuration file
+.Sh EXAMPLES
+Sample
+.Pa lumerc
+file:
+.Bd -literal -offset indent
+access_token = "token"
+# indicator = "●"
+# output_format = "table"
+
+[colors]
+purple_candy = [ 280.0, 0.29, 0.71 ]
+wasabi = [ 120.0, 1.0, 0.7 ]
+honeydew = [ 120.0, 1.0, 0.97 ]
+green_mist = [ 92.0, 0.72, 0.75 ]
+pea = [ 90.0, 0.42, 0.47 ]
+cat_eye = [ 76.0, 0.74, 0.61 ]
+seagreen = [ 160.0, 1.0, 0.50 ]
+blue_mist = [ 202.0, 0.97, 0.75 ]
+.Ed

packaging/debian/changelog

@@ -0,0 +1,5 @@
+lume (__VERSION__-__REVISION__) unstable; urgency=medium
+
+  * Package generated with make deb
+
+ -- Ryan Cavicchioni <ryan@cavi.cc>  __DATE__

packaging/debian/control

@@ -0,0 +1,14 @@
+Source: lume
+Section: unknown
+Priority: optional
+Maintainer: Ryan Cavicchioni <ryan@cavi.cc>
+Build-Depends: debhelper (>= 10)
+Standards-Version: 4.1.2
+Homepage: https://git.kill0.net/chill9/lume
+Vcs-Git: https://git.kill0.net/chill9/lume.git
+Vcs-Browser: https://git.kill0.net/chill9/lume.git
+
+Package: lume
+Architecture: any
+Depends: ${shlibs:Depends}, ${misc:Depends}
+Description: A CLI tool for the LIFX HTTP API

packaging/debian/copyright

@@ -0,0 +1,28 @@
+Format: https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/
+Upstream-Name: lume
+Source: https://git.kill0.net/chill9/lume
+
+Files: *
+Copyright: 2021 Ryan Cavicchioni <ryan@cavi.cc>
+License: MPL-2.0
+
+License: MPL-2.0
+ Licensed under the Mozilla Public License License, Version 2.0 (the "License");
+ you may not use this file except in compliance with the License.  You may
+ obtain a copy of the License at
+ .
+ https://www.mozilla.org/en-US/MPL/2.0/
+ .
+ Unless required by applicable law or agreed to in writing, software
+ distributed under the License is distributed on an "AS IS" BASIS,
+ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ See the License for the specific language governing permissions and
+ limitations under the License.
+ .
+ On Debian systems, the complete text of the Mozilla Public License version 2.0
+ license can be found in "/usr/share/common-licenses/MPL-2.0".
+
+# Please also look if there are files or directories which have a
+# different copyright/license attached and list them here.
+# Please avoid picking licenses with terms that are more restrictive than the
+# packaged work, as it may make Debian's contributions unacceptable upstream.

packaging/debian/lume.manpages

@@ -0,0 +1 @@
+lume.1

packaging/debian/rules

@@ -0,0 +1,19 @@
+#!/usr/bin/make -f
+
+DISTRIBUTION = $(shell lsb_release -sr)
+VERSION = __VERSION__
+REVISION = __REVISION__
+PACKAGEVERSION = $(VERSION)-$(REVISION)
+
+%:
+	dh $@
+
+override_dh_auto_clean:
+override_dh_auto_test:
+override_dh_auto_build:
+	make
+override_dh_auto_install:
+	make install DESTDIR=debian/lume
+
+override_dh_gencontrol:
+	dh_gencontrol -- -v$(PACKAGEVERSION)

packaging/rpm/lume.spec

@@ -0,0 +1,33 @@
+Name:           lume
+Version:        __VERSION__
+Release:        __REVISION__%{?dist}
+Summary:        A CLI tool for the LIFX HTTP API
+
+License:        MPL
+URL:            https://git.kill0.net/chill9/lume
+Source:         %{name}-%{version}.tar.xz
+
+%global debug_package %{nil}
+
+%description
+
+%prep
+%setup
+
+%build
+%make_build
+
+%install
+%make_install DESTDIR=%{buildroot}
+
+
+%files
+%{_bindir}/lume
+%license LICENSE
+/usr/share/lume/lumerc
+%doc %{_mandir}/man1/lume.1.*
+
+
+%changelog
+* __DATE__ Ryan Cavicchioni <ryan@cavi.cc>
+- lume __VERSION__