v0.0.25: misc cleanup and fixes

Author: geemus

.github/workflows/release.yml

@@ -138,6 +138,11 @@ jobs:
       - shell: bash
         run: |
           cp cmd/anchor/dist/windows/anchor.${{ env.RELEASE_VERSION }}.nupkg ./
+      - uses: actions/upload-artifact@v4
+        with:
+          name: anchor.${{ env.RELEASE_VERSION }}.nupkg
+          overwrite: true
+          path: cmd/anchor/dist/windows/anchor.${{ env.RELEASE_VERSION }}.nupkg
 
       - shell: pwsh
         run: |

README.md

@@ -52,7 +52,7 @@ Available via [Chocolatey][] or as a downloadable binary from the [releases page
 
 Install:
 ```
-chocolatey install anchor --version=0.0.23
+chocolatey install anchor --version=0.0.22
 ```
 
 ### Install from source

api/api.go

@@ -17,7 +17,6 @@ import (
 
 	"github.com/anchordotdev/cli"
 	"github.com/anchordotdev/cli/keyring"
-	"github.com/anchordotdev/cli/version"
 	"golang.org/x/exp/slices"
 )
 
@@ -370,7 +369,7 @@ type userAgentSetter struct {
 }
 
 func (s userAgentSetter) RoundTrip(req *http.Request) (*http.Response, error) {
-	req.Header.Set("User-Agent", version.UserAgent())
+	req.Header.Set("User-Agent", cli.UserAgent())
 
 	return s.RoundTripper.RoundTrip(req)
 }

auth/models/whoami.go

@@ -39,7 +39,7 @@ func (m *WhoAmIChecker) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 	switch msg := msg.(type) {
 	case UserWhoAmIMsg:
 		m.whoami = string(msg)
-		return m, tea.Quit
+		return m, nil
 	}
 
 	var cmd tea.Cmd

auth/testdata/TestWhoAmI/signed-in.golden

@@ -2,4 +2,7 @@
 # Identify Current Anchor.dev Account `anchor auth whoami`
 ─── WhoAmIChecker ──────────────────────────────────────────────────────────────
 # Identify Current Anchor.dev Account `anchor auth whoami`
-    * Identifying Anchor.dev account… *
+    * Identifying Anchor.dev account… ⠋
+─── WhoAmIChecker ──────────────────────────────────────────────────────────────
+# Identify Current Anchor.dev Account `anchor auth whoami`
+    - Identified Anchor.dev account: [email protected]

cli.go

@@ -2,11 +2,41 @@ package cli
 
 import (
 	"context"
+	"fmt"
+	"runtime"
 
 	"github.com/anchordotdev/cli/ui"
 	"github.com/spf13/cobra"
 )
 
+var Version = struct {
+	Version, Commit, Date string
+
+	Os, Arch string
+}{
+	Version: "dev",
+	Commit:  "none",
+	Date:    "unknown",
+	Os:      runtime.GOOS,
+	Arch:    runtime.GOARCH,
+}
+
+func IsDevVersion() bool {
+	return Version.Version == "dev"
+}
+
+func UserAgent() string {
+	return "Anchor CLI " + VersionString()
+}
+
+func ReleaseTagName() string {
+	return fmt.Sprintf("v%s", Version.Version)
+}
+
+func VersionString() string {
+	return fmt.Sprintf("%s (%s/%s) Commit: %s BuildDate: %s", Version.Version, Version.Os, Version.Arch, Version.Commit, Version.Date)
+}
+
 type Config struct {
 	JSON           bool `desc:"Only print JSON output to STDOUT." flag:"json,j" env:"JSON_OUTPUT" toml:"json-output"`
 	NonInteractive bool `desc:"Run without ever asking for user input." flag:"non-interactive,n" env:"NON_INTERACTIVE" toml:"non-interactive"`

cmd.go

@@ -245,12 +245,20 @@ func NewCmd[T UIer](parent *cobra.Command, name string, fn func(*cobra.Command))
 			panic(err)
 		}
 
-		cmd.RunE = func(cmd *cobra.Command, _ []string) error {
+		cmd.RunE = func(cmd *cobra.Command, args []string) error {
 			cfg := ConfigFromCmd(cmd)
 			if cfg.Test.SkipRunE {
 				return nil
 			}
 
+			var t T
+
+			switch any(t).(type) {
+			case ShowHelp:
+				cmd.HelpFunc()(cmd, args)
+				return nil
+			}
+
 			ctx, cancel := context.WithCancelCause(cmd.Context())
 			defer cancel(nil)
 
@@ -266,7 +274,6 @@ func NewCmd[T UIer](parent *cobra.Command, name string, fn func(*cobra.Command))
 				errc <- err
 			}()
 
-			var t T
 			if err := t.UI().RunTUI(ctx, drv); err != nil && err != context.Canceled {
 				prg.Quit()
 

cmd/anchor/main.go

@@ -21,7 +21,9 @@ var (
 
 func init() {
 	if version != "" {
-		versionpkg.Set(version, commit, date)
+		cli.Version.Commit = commit
+		cli.Version.Date = date
+		cli.Version.Version = version
 	}
 	cli.CmdRoot.PersistentPreRunE = versionpkg.VersionCheck
 }

go.mod

@@ -19,7 +19,6 @@ require (
 	github.com/muesli/termenv v0.15.2
 	github.com/oapi-codegen/runtime v1.1.1
 	github.com/spf13/cobra v1.8.0
-	github.com/spf13/pflag v1.0.5
 	github.com/stretchr/testify v1.8.4
 	github.com/zalando/go-keyring v0.2.4
 	golang.org/x/crypto v0.21.0
@@ -64,6 +63,7 @@ require (
 	github.com/rivo/uniseg v0.4.7 // indirect
 	github.com/rogpeppe/go-internal v1.10.0 // indirect
 	github.com/sahilm/fuzzy v0.1.1-0.20230530133925-c48e322e2a8f // indirect
+	github.com/spf13/pflag v1.0.5 // indirect
 	golang.org/x/mod v0.16.0 // indirect
 	golang.org/x/net v0.23.0 // indirect
 	golang.org/x/oauth2 v0.16.0 // indirect

lcl/config_test.go

@@ -112,20 +112,6 @@ func TestLclConfig(t *testing.T) {
 			t.Skip("diagnostic unsupported in mock mode")
 		}
 
-		teatest.WaitFor(
-			t, drv.Out,
-			func(bts []byte) bool {
-				if len(errc) > 0 {
-					t.Fatal(<-errc)
-				}
-
-				expect := "Entered hello-world.lcl.host domain for lcl.host diagnostic certificate."
-				return bytes.Contains(bts, []byte(expect))
-			},
-			teatest.WithCheckInterval(time.Millisecond*100),
-			teatest.WithDuration(time.Second*3),
-		)
-
 		teatest.WaitFor(
 			t, drv.Out,
 			func(bts []byte) bool {