Function TestExtractStringArg_AllShapes

cli/plugins/args_extract_test.go:20–121

TestExtractStringArg_AllShapes is the regression matrix for the args extractor used by every plugin's DescribeCall. The original coverage of extractStringArg sat at 25 percent because only the JSON-envelope path was exercised by the websearch tests. This matrix pins all branches: empty, flag-style,

(t *testing.T)

// path was exercised by the websearch tests. This matrix pins all
// branches: empty, flag-style, positional-fallback, multi-key fallback.
func TestExtractStringArg_AllShapes(t *testing.T) {
	cases := []struct {
		name string
		args []string
		keys []string
		want string
	}{
		{
			name: "empty args returns empty",
			args: nil,
			keys: []string{"query"},
			want: "",
		},
		{
			name: "flat JSON top-level match",
			args: []string{`{"query":"golang errgroup"}`},
			keys: []string{"query"},
			want: "golang errgroup",
		},
		{
			name: "nested @coder envelope match",
			args: []string{`{"cmd":"search","args":{"query":"login"}}`},
			keys: []string{"query"},
			want: "login",
		},
		{
			name: "first key in keys list wins",
			args: []string{`{"q":"second","query":"first"}`},
			keys: []string{"query", "q"},
			want: "first",
		},
		{
			name: "second key is fallback when first missing",
			args: []string{`{"q":"fallback"}`},
			keys: []string{"query", "q"},
			want: "fallback",
		},
		{
			name: "flag style --key value",
			args: []string{"search", "--query", "kubernetes basics"},
			keys: []string{"query"},
			want: "kubernetes basics",
		},
		{
			name: "flag style --key=value",
			args: []string{"--query=golang"},
			keys: []string{"query"},
			want: "golang",
		},
		{
			name: "double-quoted flag value",
			args: []string{"--query", `"quoted value"`},
			keys: []string{"query"},
			want: "quoted value",
		},
		{
			name: "single-quoted flag value",
			args: []string{"--query", "'single quoted'"},