release v0.13.0: pack create redesign, profile resolution fix, TUI input fixes

Signed-off-by: davidfos <david.d.foster@oracle.com>

Author: dfoster-oracle

AGENTS.md

@@ -29,7 +29,7 @@ Four harness adapters (`claudecode`, `opencode`, `codex`, `cline`) handle both f
 
 - Wrap errors with `%w` — always preserve context
 - Exit codes: `cmdutil.ExitOK` (0), `ExitFail` (1), `ExitUsage` (2)
-- Tests: `t.Parallel()` where safe, `t.TempDir()` for isolation, NEVER `t.Parallel()` with `t.Setenv()`
+- Tests: `t.Parallel()` where safe, `t.TempDir()` for isolation, NEVER `t.Parallel()` with `t.Setenv()` or `t.Chdir()`
 - Use `make fmt` (not raw `gofmt -w`) for formatting
 - `--skip-settings` skips settings only; MCP configs and plugins always sync
 - Version injected via ldflags at build time (`-X main.version`, `-X main.commit`)
@@ -56,8 +56,9 @@ Four harness adapters (`claudecode`, `opencode`, `codex`, `cline`) handle both f
 ## Workflow
 
 - Before editing: read nearby code and related tests
-- After editing: `go test ./...`, then `go vet ./...`
-- Before declaring done: `make build && make test && go vet ./...` must all pass
+- After editing: `go test ./...`, then `make lint`
+- Before declaring done: `make build && make test && make lint` must all pass
+- `make lint` runs `go vet`, `staticcheck` (if installed), and `go fix ./...` (applies Go modernization fixes in-place).
 - Pre-commit: `go build ./...` → `go test ./...` → `make fmt` → check `git diff` for fmt changes → stage any → commit
 - Feature work going to main: single atomic commit, not per-task intermediaries
 - Commits use `git commit --signoff`

CHANGELOG.md

@@ -6,6 +6,21 @@ The format is based on Keep a Changelog, and releases use semantic versioning ta
 
 ## Unreleased
 
+## [0.13.0]
+
+### Changed
+
+- **`pack create` redesigned.** Takes a pack name instead of a directory path. Packs are created in the current directory and symlinked into the packs directory by default (`--link` behavior). Use `--local` to create directly inside the packs directory. Registration in sync-config is now automatic.
+- Registry search and list output uses a structured multi-line format with labeled fields (Name, Description, Owner, Repo, Path, Ref, Contact).
+- `make lint` now runs `go fix ./...` alongside `go vet` and `staticcheck`.
+
+### Fixed
+
+- Disabled packs no longer participate in override conflict resolution during profile resolution. Previously, a disabled pack declaring overrides could incorrectly suppress duplicate-resource errors between enabled packs.
+- TUI text input handles multi-byte key events correctly, preventing phantom characters from special key presses.
+- TUI pack name input no longer leaks keypresses to global hotkeys. Previously, typing characters that matched global shortcuts (e, s, r, w, digits) while entering a new pack name would trigger tab switches, sync, or other actions instead of appending to the input.
+- TUI "Save to pack" action (`.` menu) now scopes to the cursor item instead of carrying forward all selected files.
+
 ## [0.12.1]
 
 ### Changed

Makefile

@@ -32,9 +32,10 @@ fmt: ## Format Go source
 fmt-check: ## Fail if Go source is not formatted
 	@test -z "$$(gofmt -l . | grep -v '^dist/' )" || { gofmt -l . | grep -v '^dist/'; echo "Go files need formatting. Run: make fmt"; exit 1; }
 
-lint: ## Run static analysis (go vet + staticcheck if available)
+lint: ## Run static analysis (go vet + staticcheck + go fix)
 	go vet $(GO_TAGS) ./...
 	@if command -v staticcheck >/dev/null 2>&1; then staticcheck $(GO_TAGS) ./...; fi
+	go fix ./...
 
 release-tag-check: ## Validate TAG against VERSION (supports prereleases)
 	@test -n "$(TAG)" || { echo "usage: make release-tag-check TAG=vX.Y.Z[-suffix]"; exit 1; }

VERSION

@@ -1 +1 @@
-0.12.1
+0.13.0

cmd/aipack/architecture_test.go

@@ -20,7 +20,7 @@ func TestArchitecture_ServicePackagesDoNotImportCmd(t *testing.T) {
 	}
 
 	forbidden := "github.com/shrug-labs/aipack/cmd/"
-	for _, line := range strings.Split(strings.TrimSpace(string(out)), "\n") {
+	for line := range strings.SplitSeq(strings.TrimSpace(string(out)), "\n") {
 		if strings.TrimSpace(line) == "" {
 			continue
 		}
@@ -60,7 +60,7 @@ func TestArchitecture_NoDeletedPackages(t *testing.T) {
 		"github.com/shrug-labs/aipack/internal/app/doctor",
 		"github.com/shrug-labs/aipack/internal/app/seed",
 	}
-	for _, line := range strings.Split(strings.TrimSpace(string(out)), "\n") {
+	for line := range strings.SplitSeq(strings.TrimSpace(string(out)), "\n") {
 		if strings.TrimSpace(line) == "" {
 			continue
 		}
@@ -95,7 +95,7 @@ func TestArchitecture_HarnessAndRenderDoNotImportConfig(t *testing.T) {
 		if err != nil {
 			continue // package may have no Go files
 		}
-		for _, line := range strings.Split(strings.TrimSpace(string(out)), "\n") {
+		for line := range strings.SplitSeq(strings.TrimSpace(string(out)), "\n") {
 			parts := strings.SplitN(line, " ", 2)
 			if len(parts) < 2 {
 				continue
@@ -114,7 +114,6 @@ func TestCLIExitCodes_HelpReturnsZero(t *testing.T) {
 
 	commands := []string{"doctor", "save", "init", "clean", "sync", "render", "version", "manage", "pack", "profile", "search", "query", "restore", "trace", "status", "install"}
 	for _, cmd := range commands {
-		cmd := cmd
 		t.Run(cmd, func(t *testing.T) {
 			t.Parallel()
 			_, _, code := runApp(t, cmd, "--help")

cmd/aipack/install_test.go

@@ -23,7 +23,7 @@ func TestInstall_HiddenFromTopLevelHelp(t *testing.T) {
 	}
 	// "install" (the hidden alias for "pack install") should not appear as a
 	// visible top-level command. "install-completions" is a separate, visible command.
-	for _, line := range strings.Split(stdout, "\n") {
+	for line := range strings.SplitSeq(stdout, "\n") {
 		trimmed := strings.TrimSpace(line)
 		if strings.HasPrefix(trimmed, "install") && !strings.HasPrefix(trimmed, "install-completions") {
 			t.Fatalf("install should be hidden from top-level help, but found: %q", line)

cmd/aipack/pack.go

@@ -38,29 +38,48 @@ Packs are installed under ~/.config/aipack/packs/<name>/.`
 // --- pack create ---
 
 type PackCreateCmd struct {
-	Dir  string `arg:"" help:"Directory path to create the pack in"`
-	Name string `help:"Pack name for pack.json (default: directory basename)" name:"name"`
+	Name      string `arg:"" help:"Pack name"`
+	ConfigDir string `help:"Config directory (default: ~/.config/aipack)" name:"config-dir" type:"path"`
+	Local     bool   `help:"Create pack inside the packs directory instead of the current directory" name:"local"`
 }
 
 func (c *PackCreateCmd) Help() string {
 	return `Scaffolds a new pack directory with a pack.json manifest and standard
-subdirectories: rules/, agents/, workflows/, skills/, mcp/, configs/.
+subdirectories, then installs and registers it so it is immediately available
+for profiles, sync, and save.
+
+By default, the pack is created in the current directory and symlinked into
+the packs directory (--link behavior). Use --local to create the pack directly
+inside the packs directory instead.
 
 Examples:
-  # Create a new pack
-  aipack pack create ./my-new-pack
+  # Create a pack in CWD, symlink into packs dir (default)
+  aipack pack create my-new-pack
 
-  # Create with a custom name (overrides directory basename)
-  aipack pack create ./path/to/dir --name custom-pack-name
+  # Create a pack directly in the packs dir
+  aipack pack create my-new-pack --local
 
 See also: pack install, pack show`
 }
 
 func (c *PackCreateCmd) Run(ctx context.Context, g *Globals) error {
-	if err := app.PackCreate(app.PackCreateRequest{Dir: c.Dir, Name: c.Name}); err != nil {
+	cfgDir, err := cmdutil.EnsureConfigDir(c.ConfigDir, config.HomeDir(), g.Stderr)
+	if err != nil {
 		return err
 	}
-	fmt.Fprintf(g.Stdout, "Created pack at %s\n", c.Dir)
+
+	if err := app.PackCreate(app.PackCreateRequest{
+		Name:      c.Name,
+		ConfigDir: cfgDir,
+		Local:     c.Local,
+	}); err != nil {
+		return err
+	}
+	if c.Local {
+		fmt.Fprintf(g.Stdout, "Created pack %q in %s\n", c.Name, filepath.Join(cfgDir, "packs", c.Name))
+	} else {
+		fmt.Fprintf(g.Stdout, "Created pack %q in ./%s (linked)\n", c.Name, c.Name)
+	}
 	return nil
 }
 

cmd/aipack/pack_test.go

@@ -28,7 +28,7 @@ func TestPackList_JSON_Empty(t *testing.T) {
 		t.Fatalf("pack list --json exit=%d, want %d; stderr=%s", code, cmdutil.ExitOK, stderr)
 	}
 
-	var entries []interface{}
+	var entries []any
 	if err := json.Unmarshal([]byte(stdout), &entries); err != nil {
 		t.Fatalf("invalid JSON: %v\noutput=%s", err, stdout)
 	}
@@ -57,7 +57,7 @@ func TestPackList_JSON_WithPack(t *testing.T) {
 		t.Fatalf("pack list --json exit=%d, want %d; stderr=%s", code, cmdutil.ExitOK, stderr)
 	}
 
-	var entries []map[string]interface{}
+	var entries []map[string]any
 	if err := json.Unmarshal([]byte(stdout), &entries); err != nil {
 		t.Fatalf("invalid JSON: %v\noutput=%s", err, stdout)
 	}
@@ -120,7 +120,7 @@ func TestPackShow_NotInstalled(t *testing.T) {
 
 func writePackManifestCmd(t *testing.T, dir string, name string) {
 	t.Helper()
-	m := map[string]interface{}{
+	m := map[string]any{
 		"schema_version": 1,
 		"name":           name,
 		"version":        "1.0.0",

cmd/aipack/registry.go

@@ -254,26 +254,30 @@ func (c *RegistrySourcesCmd) Run(ctx context.Context, g *Globals) error {
 }
 
 func printRegistryResults(g *Globals, results []app.RegistrySearchResult) {
-	for _, r := range results {
-		installed := ""
+	for i, r := range results {
+		if i > 0 {
+			fmt.Fprintln(g.Stdout)
+		}
+		status := ""
 		if r.Installed {
-			installed = " [installed]"
+			status = " [installed]"
 		}
-		desc := ""
+		fmt.Fprintf(g.Stdout, "Name:        %s%s\n", r.Name, status)
 		if r.Description != "" {
-			desc = " — " + r.Description
+			fmt.Fprintf(g.Stdout, "Description: %s\n", r.Description)
+		}
+		if r.Owner != "" {
+			fmt.Fprintf(g.Stdout, "Owner:       %s\n", r.Owner)
 		}
-		fmt.Fprintf(g.Stdout, "  %s%s%s\n", r.Name, installed, desc)
-		details := []string{r.Repo}
+		fmt.Fprintf(g.Stdout, "Repo:        %s\n", r.Repo)
 		if r.Path != "" {
-			details = append(details, "path: "+r.Path)
+			fmt.Fprintf(g.Stdout, "Path:        %s\n", r.Path)
 		}
 		if r.Ref != "" {
-			details = append(details, "ref: "+r.Ref)
+			fmt.Fprintf(g.Stdout, "Ref:         %s\n", r.Ref)
 		}
-		if r.Owner != "" {
-			details = append(details, "owner: "+r.Owner)
+		if r.Contact != "" {
+			fmt.Fprintf(g.Stdout, "Contact:     %s\n", r.Contact)
 		}
-		fmt.Fprintf(g.Stdout, "    %s\n", strings.Join(details, ", "))
 	}
 }

cmd/aipack/save.go

@@ -313,7 +313,7 @@ func (c *SaveCmd) runToPack(ctx context.Context, g *Globals) error {
 
 func parseCategories(raw string) ([]domain.PackCategory, error) {
 	var cats []domain.PackCategory
-	for _, p := range strings.Split(raw, ",") {
+	for p := range strings.SplitSeq(raw, ",") {
 		p = strings.TrimSpace(p)
 		if p == "" {
 			continue