fix: persist work mode config and resolve packages from local path

WorkOn/WorkOff modified config in memory but never saved to disk,
so WorkStatusList always reported empty. Add SaveConfigFile and
ConfigService.Save to persist changes.

Also fix package resolution in work mode: readManifest and packageDir
now check WorkMode config and use the local path when a tap is in
work mode, so install/remove/upgrade/info/search all work correctly
against the local checkout.

Author: jlrickert

pkg/dots/config.go

@@ -3,6 +3,7 @@ package dots
 import (
 	"fmt"
 	"os"
+	"path/filepath"
 
 	"gopkg.in/yaml.v3"
 )
@@ -77,6 +78,18 @@ func LoadConfigFile(path string) (*Config, error) {
 	return ParseConfig(data)
 }
 
+// SaveConfigFile writes a Config to disk as YAML.
+func SaveConfigFile(path string, cfg *Config) error {
+	data, err := yaml.Marshal(cfg)
+	if err != nil {
+		return fmt.Errorf("marshal config: %w", err)
+	}
+	if err := os.MkdirAll(filepath.Dir(path), 0o755); err != nil {
+		return err
+	}
+	return os.WriteFile(path, data, 0o644)
+}
+
 // MergeConfig merges override into base. Non-zero override fields win.
 func MergeConfig(base, override *Config) *Config {
 	result := *base

pkg/dotsctl/config_service.go

@@ -39,6 +39,15 @@ func (s *ConfigService) Config(cache bool) (*dots.Config, error) {
 	return cfg, nil
 }
 
+// Save writes the current config to disk and updates the cache.
+func (s *ConfigService) Save(cfg *dots.Config) error {
+	if err := dots.SaveConfigFile(s.ConfigPath, cfg); err != nil {
+		return err
+	}
+	s.cached = cfg
+	return nil
+}
+
 // InvalidateCache clears the cached config.
 func (s *ConfigService) InvalidateCache() {
 	s.cached = nil

pkg/dotsctl/dots_info.go

@@ -31,7 +31,7 @@ func (d *Dots) Info(ctx context.Context, pkgRef string) (*InfoResult, error) {
 	}
 
 	// Read manifest if available
-	manifestData, err := d.Repo.ReadManifest(ctx, tap, pkg)
+	manifestData, err := d.readManifest(ctx, tap, pkg)
 	if err == nil {
 		manifest, err := dots.ParseManifest(manifestData)
 		if err == nil {

pkg/dotsctl/dots_install.go

@@ -3,6 +3,7 @@ package dotsctl
 import (
 	"context"
 	"fmt"
+	"os"
 	"strings"
 	"time"
 
@@ -34,7 +35,7 @@ func (d *Dots) Install(ctx context.Context, opts InstallOptions) (*InstallResult
 	}
 
 	// Read and parse manifest
-	manifestData, err := d.Repo.ReadManifest(ctx, tap, pkg)
+	manifestData, err := d.readManifest(ctx, tap, pkg)
 	if err != nil {
 		return nil, fmt.Errorf("read manifest: %w", err)
 	}
@@ -193,7 +194,29 @@ func (d *Dots) resolveStrategy(override, pkgStrategy dots.LinkStrategy) dots.Lin
 	return dots.LinkSymlink
 }
 
+// readManifest reads a package manifest, checking work mode paths first.
+func (d *Dots) readManifest(ctx context.Context, tap, pkg string) ([]byte, error) {
+	cfg, _ := d.ConfigService.Config(true)
+	if cfg != nil {
+		if localPath, ok := cfg.WorkMode[tap]; ok {
+			manifestPath := localPath + "/" + pkg + "/Dotfile.yaml"
+			data, err := os.ReadFile(manifestPath)
+			if err != nil {
+				return nil, &dots.PackageNotFoundError{Tap: tap, Package: pkg}
+			}
+			return data, nil
+		}
+	}
+	return d.Repo.ReadManifest(ctx, tap, pkg)
+}
+
 func (d *Dots) packageDir(tap, pkg string) string {
+	cfg, _ := d.ConfigService.Config(true)
+	if cfg != nil {
+		if localPath, ok := cfg.WorkMode[tap]; ok {
+			return localPath + "/" + pkg
+		}
+	}
 	return d.PathService.TapsDir() + "/" + tap + "/" + pkg
 }
 

pkg/dotsctl/dots_remove.go

@@ -41,7 +41,7 @@ func (d *Dots) Remove(ctx context.Context, opts RemoveOptions) error {
 
 	// Run pre_remove hook if manifest is available
 	pkgDir := d.packageDir(tap, pkg)
-	manifestData, _ := d.Repo.ReadManifest(ctx, tap, pkg)
+	manifestData, _ := d.readManifest(ctx, tap, pkg)
 	if manifestData != nil {
 		manifest, err := dots.ParseManifest(manifestData)
 		if err == nil {

pkg/dotsctl/dots_search.go

@@ -73,7 +73,7 @@ func (d *Dots) Search(ctx context.Context, query string) ([]SearchResult, error)
 }
 
 func (d *Dots) readAndParseManifest(ctx context.Context, tap, pkg string) (*dots.Manifest, error) {
-	data, err := d.Repo.ReadManifest(ctx, tap, pkg)
+	data, err := d.readManifest(ctx, tap, pkg)
 	if err != nil {
 		return nil, err
 	}

pkg/dotsctl/dots_upgrade.go

@@ -38,7 +38,7 @@ func (d *Dots) Upgrade(ctx context.Context, opts UpgradeOptions) error {
 
 	// Run pre_upgrade hook
 	pkgDir := d.packageDir(tap, pkg)
-	manifestData, err := d.Repo.ReadManifest(ctx, tap, pkg)
+	manifestData, err := d.readManifest(ctx, tap, pkg)
 	if err != nil {
 		return fmt.Errorf("read manifest: %w", err)
 	}

pkg/dotsctl/dots_work.go

@@ -29,6 +29,10 @@ func (d *Dots) WorkOn(ctx context.Context, opts WorkOnOptions) error {
 	}
 	cfg.WorkMode[opts.Tap] = opts.LocalPath
 
+	if err := d.ConfigService.Save(cfg); err != nil {
+		return fmt.Errorf("save config: %w", err)
+	}
+
 	// Re-link all installed packages from this tap using local path
 	lockfile, err := d.Repo.ReadLockfile(ctx)
 	if err == nil {
@@ -54,6 +58,10 @@ func (d *Dots) WorkOff(ctx context.Context, tap string) error {
 	}
 	delete(cfg.WorkMode, tap)
 
+	if err := d.ConfigService.Save(cfg); err != nil {
+		return fmt.Errorf("save config: %w", err)
+	}
+
 	// Re-link packages from internal clone
 	lockfile, err := d.Repo.ReadLockfile(ctx)
 	if err == nil {