Merge pull request '[agent/codex:gpt-5.4-mini] Read ~/spec/code/core/go/cli/RFC.md fully. Find ONE feature ...' (#21) from agent/update-the-code-against-the-ax-design-pr into dev

cmd/core/pkgcmd/cmd_search.go

@@ -34,16 +34,14 @@ func addPkgSearchCommand(parent *cobra.Command) {
 		Use:   "search",
 		Short: i18n.T("cmd.pkg.search.short"),
 		Long:  i18n.T("cmd.pkg.search.long"),
+		Args:  cobra.RangeArgs(0, 1),
 		RunE: func(cmd *cobra.Command, args []string) error {
 			org := searchOrg
-			pattern := searchPattern
+			pattern := resolvePkgSearchPattern(searchPattern, args)
 			limit := searchLimit
 			if org == "" {
 				org = "host-uk"
 			}
-			if pattern == "" {
-				pattern = "*"
-			}
 			if limit == 0 {
 				limit = 50
 			}
@@ -223,6 +221,16 @@ func formatPkgSearchUpdatedAt(raw string) string {
 	return cli.FormatAge(updatedAt)
 }
 
+func resolvePkgSearchPattern(flagPattern string, args []string) string {
+	if flagPattern != "" {
+		return flagPattern
+	}
+	if len(args) > 0 && strings.TrimSpace(args[0]) != "" {
+		return args[0]
+	}
+	return "*"
+}
+
 // matchGlob does simple glob matching with * wildcards
 func matchGlob(pattern, name string) bool {
 	if pattern == "*" || pattern == "" {

cmd/core/pkgcmd/cmd_search_test.go

@@ -0,0 +1,24 @@
+package pkgcmd
+
+import (
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+)
+
+func TestResolvePkgSearchPattern_Good(t *testing.T) {
+	t.Run("uses flag pattern when set", func(t *testing.T) {
+		got := resolvePkgSearchPattern("core-*", []string{"api"})
+		assert.Equal(t, "core-*", got)
+	})
+
+	t.Run("uses positional pattern when flag is empty", func(t *testing.T) {
+		got := resolvePkgSearchPattern("", []string{"api"})
+		assert.Equal(t, "api", got)
+	})
+
+	t.Run("defaults to wildcard when nothing is provided", func(t *testing.T) {
+		got := resolvePkgSearchPattern("", nil)
+		assert.Equal(t, "*", got)
+	})
+}