fix: accept typed object array packages in push config

openboot push failed with "cannot unmarshal object into Go struct field
RemoteConfig.packages of type string" when the file contained packages
in the API export format [{"name":"git","type":"formula"}] instead of
flat string arrays ["git"].

Add UnmarshalRemoteConfigFlexible() that accepts both formats and use
it in pushConfig and LoadRemoteConfigFromFile.

Fixes #12

Author: fullstackjam

internal/cli/push.go

@@ -111,8 +111,8 @@ func pushSnapshot(data []byte, slug, token, username, apiBase string) error {
 }
 
 func pushConfig(data []byte, slug, token, username, apiBase string) error {
-	var rc config.RemoteConfig
-	if err := json.Unmarshal(data, &rc); err != nil {
+	rc, err := config.UnmarshalRemoteConfigFlexible(data)
+	if err != nil {
 		return fmt.Errorf("parse config: %w", err)
 	}
 	if err := rc.Validate(); err != nil {
@@ -125,7 +125,7 @@ func pushConfig(data []byte, slug, token, username, apiBase string) error {
 	}
 
 	// Convert RemoteConfig to API format: packages as [{name, type}]
-	packages := remoteConfigToAPIPackages(&rc)
+	packages := remoteConfigToAPIPackages(rc)
 
 	reqBody := map[string]interface{}{
 		"name":        name,

internal/config/config.go

@@ -97,6 +97,89 @@ type RemoteMacOSPref struct {
 	Desc   string `json:"desc"`
 }
 
+// typedPackage represents a package entry with name and type, as returned
+// by the openboot.dev API (e.g. {"name":"git","type":"formula"}).
+type typedPackage struct {
+	Name string `json:"name"`
+	Type string `json:"type"`
+}
+
+// UnmarshalRemoteConfigFlexible parses JSON into a RemoteConfig, accepting
+// packages in either flat string array format (["git","curl"]) or typed
+// object array format ([{"name":"git","type":"formula"}]).
+func UnmarshalRemoteConfigFlexible(data []byte) (*RemoteConfig, error) {
+	// Try direct unmarshal first (flat string arrays).
+	var rc RemoteConfig
+	if err := json.Unmarshal(data, &rc); err == nil {
+		return &rc, nil
+	}
+
+	// Extract packages as typed objects and convert to flat arrays.
+	var raw map[string]json.RawMessage
+	if err := json.Unmarshal(data, &raw); err != nil {
+		return nil, err
+	}
+
+	pkgData, ok := raw["packages"]
+	if !ok {
+		return nil, fmt.Errorf("missing packages field")
+	}
+
+	var typed []typedPackage
+	if err := json.Unmarshal(pkgData, &typed); err != nil {
+		return nil, fmt.Errorf("packages must be a string array or typed object array: %w", err)
+	}
+
+	var formulae, casks, taps, npm []string
+	for _, p := range typed {
+		switch p.Type {
+		case "cask":
+			casks = append(casks, p.Name)
+		case "tap":
+			taps = append(taps, p.Name)
+		case "npm":
+			npm = append(npm, p.Name)
+		default:
+			formulae = append(formulae, p.Name)
+		}
+	}
+
+	// Replace packages with flat arrays and re-unmarshal.
+	converted := make(map[string]json.RawMessage, len(raw))
+	for k, v := range raw {
+		converted[k] = v
+	}
+	if f, err := json.Marshal(formulae); err == nil {
+		converted["packages"] = f
+	}
+	if len(casks) > 0 {
+		if c, err := json.Marshal(casks); err == nil {
+			converted["casks"] = c
+		}
+	}
+	if len(taps) > 0 {
+		if t, err := json.Marshal(taps); err == nil {
+			converted["taps"] = t
+		}
+	}
+	if len(npm) > 0 {
+		if n, err := json.Marshal(npm); err == nil {
+			converted["npm"] = n
+		}
+	}
+
+	normalised, err := json.Marshal(converted)
+	if err != nil {
+		return nil, fmt.Errorf("normalise config: %w", err)
+	}
+
+	var result RemoteConfig
+	if err := json.Unmarshal(normalised, &result); err != nil {
+		return nil, err
+	}
+	return &result, nil
+}
+
 var (
 	pkgNameRe = regexp.MustCompile(`^[a-zA-Z0-9@/_.-]+$`)
 	tapNameRe = regexp.MustCompile(`^[a-zA-Z0-9_-]+/[a-zA-Z0-9_-]+$`)
@@ -256,14 +339,14 @@ func LoadRemoteConfigFromFile(path string) (*RemoteConfig, error) {
 		return loadSnapshotAsRemoteConfig(data)
 	}
 
-	var rc RemoteConfig
-	if err := json.Unmarshal(data, &rc); err != nil {
+	rc, err := UnmarshalRemoteConfigFlexible(data)
+	if err != nil {
 		return nil, fmt.Errorf("parse remote config: %w", err)
 	}
 	if err := rc.Validate(); err != nil {
 		return nil, fmt.Errorf("invalid config: %w", err)
 	}
-	return &rc, nil
+	return rc, nil
 }
 
 // snapshotFile mirrors the subset of snapshot.Snapshot needed for conversion,

internal/config/config_test.go

@@ -345,6 +345,93 @@ func TestFetchRemoteConfig_NoAliasNoDefault(t *testing.T) {
 	assert.Contains(t, err.Error(), "config not found: testuser/default")
 }
 
+func TestUnmarshalRemoteConfigFlexible_FlatStringArrays(t *testing.T) {
+	data := []byte(`{
+		"username": "testuser",
+		"packages": ["git", "curl"],
+		"casks": ["firefox"],
+		"taps": ["homebrew/cask-fonts"],
+		"npm": ["typescript"]
+	}`)
+
+	rc, err := UnmarshalRemoteConfigFlexible(data)
+	require.NoError(t, err)
+	assert.Equal(t, []string{"git", "curl"}, rc.Packages)
+	assert.Equal(t, []string{"firefox"}, rc.Casks)
+	assert.Equal(t, []string{"homebrew/cask-fonts"}, rc.Taps)
+	assert.Equal(t, []string{"typescript"}, rc.Npm)
+}
+
+func TestUnmarshalRemoteConfigFlexible_TypedObjectArray(t *testing.T) {
+	data := []byte(`{
+		"username": "testuser",
+		"name": "My Setup",
+		"packages": [
+			{"name": "git", "type": "formula"},
+			{"name": "curl", "type": "formula"},
+			{"name": "firefox", "type": "cask"},
+			{"name": "homebrew/cask-fonts", "type": "tap"},
+			{"name": "typescript", "type": "npm"}
+		]
+	}`)
+
+	rc, err := UnmarshalRemoteConfigFlexible(data)
+	require.NoError(t, err)
+	assert.Equal(t, "testuser", rc.Username)
+	assert.Equal(t, "My Setup", rc.Name)
+	assert.Equal(t, []string{"git", "curl"}, rc.Packages)
+	assert.Equal(t, []string{"firefox"}, rc.Casks)
+	assert.Equal(t, []string{"homebrew/cask-fonts"}, rc.Taps)
+	assert.Equal(t, []string{"typescript"}, rc.Npm)
+}
+
+func TestUnmarshalRemoteConfigFlexible_TypedObjectWithDesc(t *testing.T) {
+	data := []byte(`{
+		"packages": [
+			{"name": "git", "type": "formula", "desc": "Version control"},
+			{"name": "firefox", "type": "cask", "desc": "Web browser"}
+		]
+	}`)
+
+	rc, err := UnmarshalRemoteConfigFlexible(data)
+	require.NoError(t, err)
+	assert.Equal(t, []string{"git"}, rc.Packages)
+	assert.Equal(t, []string{"firefox"}, rc.Casks)
+}
+
+func TestUnmarshalRemoteConfigFlexible_PreservesOtherFields(t *testing.T) {
+	data := []byte(`{
+		"username": "testuser",
+		"slug": "myconfig",
+		"name": "My Config",
+		"preset": "developer",
+		"dotfiles_repo": "https://github.com/testuser/dotfiles",
+		"packages": [
+			{"name": "git", "type": "formula"}
+		],
+		"shell": {"oh_my_zsh": true, "theme": "robbyrussell", "plugins": ["git"]},
+		"macos_prefs": [{"domain": "com.apple.dock", "key": "autohide", "type": "bool", "value": "true"}]
+	}`)
+
+	rc, err := UnmarshalRemoteConfigFlexible(data)
+	require.NoError(t, err)
+	assert.Equal(t, "testuser", rc.Username)
+	assert.Equal(t, "myconfig", rc.Slug)
+	assert.Equal(t, "developer", rc.Preset)
+	assert.Equal(t, "https://github.com/testuser/dotfiles", rc.DotfilesRepo)
+	assert.Equal(t, []string{"git"}, rc.Packages)
+	require.NotNil(t, rc.Shell)
+	assert.True(t, rc.Shell.OhMyZsh)
+	assert.Equal(t, "robbyrussell", rc.Shell.Theme)
+	assert.Len(t, rc.MacOSPrefs, 1)
+}
+
+func TestUnmarshalRemoteConfigFlexible_InvalidJSON(t *testing.T) {
+	data := []byte(`not json`)
+	_, err := UnmarshalRemoteConfigFlexible(data)
+	assert.Error(t, err)
+}
+
 func TestFetchRemoteConfig_ExplicitSlugNoFallback(t *testing.T) {
 	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		assert.Equal(t, "/testuser/default/config", r.URL.Path)