feat: openboot edit shows interactive config picker

Instead of requiring --slug or relying on sync source, edit now fetches
the user's configs and presents a select list. --slug still skips the
picker for direct access.

Author: fullstackjam

internal/cli/edit.go

@@ -5,22 +5,20 @@ import (
 	"os/exec"
 
 	"github.com/openbootdotdev/openboot/internal/auth"
-	syncpkg "github.com/openbootdotdev/openboot/internal/sync"
 	"github.com/openbootdotdev/openboot/internal/ui"
 	"github.com/spf13/cobra"
 )
 
 var editCmd = &cobra.Command{
 	Use:   "edit",
-	Short: "Open your config on openboot.dev in the browser",
-	Long: `Open your remote config on openboot.dev in the default browser.
+	Short: "Open a config on openboot.dev in the browser",
+	Long: `Pick a config from your openboot.dev account and open it in the browser.
 
-Resolves the config slug from the current sync source (set when you last ran
-'openboot install' or 'openboot push'). Use --slug to open a specific config.`,
-	Example: `  # Open your linked config in the browser
+Use --slug to skip the picker and open a specific config directly.`,
+	Example: `  # Pick a config interactively
   openboot edit
 
-  # Open a specific config
+  # Open a specific config directly
   openboot edit --slug my-config`,
 	SilenceUsage: true,
 	RunE: func(cmd *cobra.Command, args []string) error {
@@ -30,7 +28,7 @@ Resolves the config slug from the current sync source (set when you last ran
 }
 
 func init() {
-	editCmd.Flags().String("slug", "", "config slug to open (default: current sync source)")
+	editCmd.Flags().String("slug", "", "config slug to open (skips the picker)")
 	rootCmd.AddCommand(editCmd)
 }
 
@@ -49,12 +47,14 @@ func runEdit(slugOverride string) error {
 
 	slug := slugOverride
 	if slug == "" {
-		if source, loadErr := syncpkg.LoadSource(); loadErr == nil && source != nil && source.Slug != "" {
-			slug = source.Slug
+		slug, err = pickConfig(stored.Token, auth.GetAPIBase())
+		if err != nil {
+			return err
+		}
+		if slug == "" {
+			ui.Info("No config selected.")
+			return nil
 		}
-	}
-	if slug == "" {
-		return fmt.Errorf("no config slug — use --slug or run 'openboot install <config>' first")
 	}
 
 	url := fmt.Sprintf("https://openboot.dev/%s/%s", stored.Username, slug)
@@ -66,3 +66,41 @@ func runEdit(slugOverride string) error {
 	ui.Success(fmt.Sprintf("Opened %s", url))
 	return nil
 }
+
+// pickConfig fetches the user's configs and shows an interactive select list.
+// Returns the chosen slug, or "" if the user has no configs.
+func pickConfig(token, apiBase string) (string, error) {
+	configs, _ := fetchUserConfigs(token, apiBase)
+
+	if len(configs) == 0 {
+		return "", fmt.Errorf("no configs found — run 'openboot push' to create one")
+	}
+
+	options := make([]string, 0, len(configs))
+	for _, c := range configs {
+		label := c.Slug
+		if c.Name != "" && c.Name != c.Slug {
+			label = fmt.Sprintf("%s — %s", c.Slug, c.Name)
+		}
+		options = append(options, label)
+	}
+
+	fmt.Println()
+	choice, err := ui.SelectOption("Which config would you like to edit?", options)
+	if err != nil {
+		return "", fmt.Errorf("select config: %w", err)
+	}
+
+	// Extract slug from "slug — Name" label
+	slug := splitBefore(choice, " — ")
+	return slug, nil
+}
+
+func splitBefore(s, sep string) string {
+	for i := 0; i < len(s)-len(sep)+1; i++ {
+		if s[i:i+len(sep)] == sep {
+			return s[:i]
+		}
+	}
+	return s
+}

internal/cli/edit_test.go

@@ -1,6 +1,9 @@
 package cli
 
 import (
+	"encoding/json"
+	"net/http"
+	"net/http/httptest"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
@@ -16,34 +19,56 @@ func TestRunEdit_NotAuthenticated(t *testing.T) {
 	assert.Contains(t, err.Error(), "not logged in")
 }
 
-func TestRunEdit_NoSlug_NoSyncSource(t *testing.T) {
+func TestRunEdit_WithSlugFlag_OpensDirectly(t *testing.T) {
 	setupTestAuth(t, true)
 
+	// With --slug, no API call is made (no picker needed).
+	// exec.Command("open", url) succeeds on macOS; error is from "open" only.
+	err := runEdit("my-config")
+
+	if err != nil {
+		assert.Contains(t, err.Error(), "open browser")
+	}
+}
+
+func TestRunEdit_NoConfigs_ReturnsError(t *testing.T) {
+	setupTestAuth(t, true)
+
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		json.NewEncoder(w).Encode(map[string]any{"configs": []any{}})
+	}))
+	defer server.Close()
+
+	t.Setenv("OPENBOOT_API_URL", server.URL)
+
 	err := runEdit("")
 
 	assert.Error(t, err)
-	assert.Contains(t, err.Error(), "no config slug")
+	assert.Contains(t, err.Error(), "no configs found")
 }
 
-func TestRunEdit_SlugFromSyncSource(t *testing.T) {
-	tmpDir := setupTestAuth(t, true)
-	writeSyncSource(t, tmpDir, "my-setup")
+func TestPickConfig_NoConfigs_ReturnsError(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		json.NewEncoder(w).Encode(map[string]any{"configs": []any{}})
+	}))
+	defer server.Close()
 
-	// exec.Command("open", url) will fail in CI / test environment — that's fine,
-	// we just verify the slug resolution works and the error is from "open", not auth/slug.
-	err := runEdit("")
+	_, err := pickConfig("test-token", server.URL)
 
-	// The only error allowed is from the "open" binary itself (not found in some envs).
-	if err != nil {
-		assert.Contains(t, err.Error(), "open browser")
-	}
+	assert.Error(t, err)
+	assert.Contains(t, err.Error(), "no configs found")
+}
+
+func TestSplitBefore(t *testing.T) {
+	assert.Equal(t, "my-setup", splitBefore("my-setup — My Mac Setup", " — "))
+	assert.Equal(t, "work-mac", splitBefore("work-mac — Work Machine", " — "))
+	assert.Equal(t, "no-sep", splitBefore("no-sep", " — "))
 }
 
 func TestEditCmd_CommandStructure(t *testing.T) {
 	assert.Equal(t, "edit", editCmd.Use)
 	assert.NotEmpty(t, editCmd.Short)
 	assert.NotEmpty(t, editCmd.Long)
 	assert.NotNil(t, editCmd.RunE)
-
 	assert.NotNil(t, editCmd.Flags().Lookup("slug"))
 }