Add build info/expire commands and tests

Implement build detail and expiration endpoints, add CLI subcommands, and
cover new paths with HTTP/integration/output tests. Route build listings
through /v1/builds with app filtering to support sorting, update docs and
plan notes, and narrow gitignore rules for local binaries.

Author: rudrankriyam

.gitignore

@@ -7,8 +7,8 @@
 *.dll
 *.so
 *.dylib
-asc
-asc-debug
+/asc
+/asc-debug
 
 # Test binary, built with `go test -c`
 *.test

Agents.md

@@ -74,6 +74,8 @@ asc apps --json
 asc apps --sort name --json
 asc builds --app "123456789" --json
 asc builds --app "123456789" --sort -uploadedDate --json
+asc builds info --build "BUILD_ID" --json
+asc builds expire --build "BUILD_ID" --json
 
 # Utilities
 asc version

PLAN.md

@@ -24,6 +24,11 @@ A fast, AI-agent-friendly CLI for App Store Connect that enables developers to s
 - Feedback/crash/review endpoints aligned to ASC OpenAPI spec
 - Code compiles and unit tests run
 - Live API validation: feedback/crashes return data; reviews may be empty if no reviews exist
+- Apps/builds list commands working with manual pagination
+- Sorting supported for apps/builds/reviews/feedback/crashes
+- Build info and build expiration commands available
+- Installer script available for latest release downloads
+- HTTP-level client tests and CLI/output tests added
 
 ### What Doesn't Work Yet
 
@@ -175,37 +180,35 @@ asc reviews --app "123456789" --json
 
 **Goal:** Add commands for managing apps and builds
 
-#### Features
+#### Features (Current + Remaining)
 
-1. **List Apps**
+1. **List Apps** ✅
    ```bash
-   asc apps list
-   asc apps list --json
+   asc apps --json
    ```
 
-2. **List Builds**
+2. **List Builds** ✅
    ```bash
-   asc builds list --app "APP_ID"
-   asc builds list --app "APP_ID" --json
+   asc builds --app "APP_ID" --json
    ```
 
-3. **Build Details**
+3. **Build Details** ✅
    ```bash
    asc builds info --build "BUILD_ID"
    ```
 
-4. **Expire Build**
+4. **Expire Build** ✅
    ```bash
-   asc builds expire --build "BUILD_ID" --app "APP_ID"
+   asc builds expire --build "BUILD_ID"
    ```
 
 #### Technical Tasks
 
-- [ ] Implement `GET /v1/apps`
-- [ ] Implement `GET /v1/apps/{id}/builds`
-- [ ] Implement `PATCH /v1/builds/{id}`
+- [x] Implement `GET /v1/apps`
+- [x] Implement `GET /v1/apps/{id}/builds`
+- [x] Implement `GET /v1/builds/{id}`
+- [x] Implement `PATCH /v1/builds/{id}`
 - [ ] Add build expiration workflow
-- [ ] Add pagination support
 
 ---
 
@@ -410,7 +413,7 @@ github.com/goreleaser/nfpm/v2     - Packaging via `go run` (optional)
 
 **Phase 1: Foundation - IMPLEMENTED** (validated locally)
 
-Next: Add auto-pagination, more filters (build/tester/platform), and mockable integration tests
+Next: Add auto-pagination and beta management commands
 
 ## Known Issues
 

README.md

@@ -137,6 +137,12 @@ asc builds --app "123456789" --sort -uploadedDate --json
 # Fetch next page
 asc apps --next "<links.next>" --json
 asc builds --next "<links.next>" --json
+
+# Build details
+asc builds info --build "BUILD_ID" --json
+
+# Expire a build (irreversible)
+asc builds expire --build "BUILD_ID" --json
 ```
 
 ### Utilities

cmd/commands.go

@@ -381,8 +381,16 @@ Examples:
   asc builds --app "123456789" --limit 10 --json
   asc builds --app "123456789" --sort -uploadedDate --json
   asc builds --app "123456789" --output table
-  asc builds --next "<links.next>" --json`,
+  asc builds --next "<links.next>" --json
+
+Subcommands:
+  info    Show build details
+  expire  Expire a build for TestFlight`,
 		FlagSet: fs,
+		Subcommands: []*ffcli.Command{
+			BuildsInfoCommand(),
+			BuildsExpireCommand(),
+		},
 		Exec: func(ctx context.Context, args []string) error {
 			if *limit != 0 && (*limit < 1 || *limit > 200) {
 				return fmt.Errorf("builds: --limit must be between 1 and 200")
@@ -431,6 +439,104 @@ Examples:
 	}
 }
 
+// BuildsInfoCommand returns a build detail subcommand.
+func BuildsInfoCommand() *ffcli.Command {
+	fs := flag.NewFlagSet("builds info", flag.ExitOnError)
+
+	buildID := fs.String("build", "", "Build ID")
+	output := fs.String("output", "json", "Output format: json (default), table, markdown")
+	jsonFlag := fs.Bool("json", false, "Output in JSON format (shorthand)")
+	pretty := fs.Bool("pretty", false, "Pretty-print JSON output")
+
+	return &ffcli.Command{
+		Name:       "info",
+		ShortUsage: "asc builds info [flags]",
+		ShortHelp:  "Show build details.",
+		LongHelp: `Show build details.
+
+Examples:
+  asc builds info --build "BUILD_ID" --json`,
+		FlagSet: fs,
+		Exec: func(ctx context.Context, args []string) error {
+			if strings.TrimSpace(*buildID) == "" {
+				fmt.Fprintln(os.Stderr, "Error: --build is required")
+				fs.Usage()
+				return flag.ErrHelp
+			}
+
+			client, err := getASCClient()
+			if err != nil {
+				return fmt.Errorf("builds info: %w", err)
+			}
+
+			requestCtx, cancel := contextWithTimeout(ctx)
+			defer cancel()
+
+			build, err := client.GetBuild(requestCtx, strings.TrimSpace(*buildID))
+			if err != nil {
+				return fmt.Errorf("builds info: failed to fetch: %w", err)
+			}
+
+			format := *output
+			if *jsonFlag {
+				format = "json"
+			}
+
+			return printOutput(build, format, *pretty)
+		},
+	}
+}
+
+// BuildsExpireCommand returns a build expiration subcommand.
+func BuildsExpireCommand() *ffcli.Command {
+	fs := flag.NewFlagSet("builds expire", flag.ExitOnError)
+
+	buildID := fs.String("build", "", "Build ID")
+	output := fs.String("output", "json", "Output format: json (default), table, markdown")
+	jsonFlag := fs.Bool("json", false, "Output in JSON format (shorthand)")
+	pretty := fs.Bool("pretty", false, "Pretty-print JSON output")
+
+	return &ffcli.Command{
+		Name:       "expire",
+		ShortUsage: "asc builds expire [flags]",
+		ShortHelp:  "Expire a build for TestFlight.",
+		LongHelp: `Expire a build for TestFlight.
+
+This action is irreversible for the specified build.
+
+Examples:
+  asc builds expire --build "BUILD_ID" --json`,
+		FlagSet: fs,
+		Exec: func(ctx context.Context, args []string) error {
+			if strings.TrimSpace(*buildID) == "" {
+				fmt.Fprintln(os.Stderr, "Error: --build is required")
+				fs.Usage()
+				return flag.ErrHelp
+			}
+
+			client, err := getASCClient()
+			if err != nil {
+				return fmt.Errorf("builds expire: %w", err)
+			}
+
+			requestCtx, cancel := contextWithTimeout(ctx)
+			defer cancel()
+
+			build, err := client.ExpireBuild(requestCtx, strings.TrimSpace(*buildID))
+			if err != nil {
+				return fmt.Errorf("builds expire: failed to expire: %w", err)
+			}
+
+			format := *output
+			if *jsonFlag {
+				format = "json"
+			}
+
+			return printOutput(build, format, *pretty)
+		},
+	}
+}
+
 // VersionCommand returns a version subcommand
 func VersionCommand(version string) *ffcli.Command {
 	return &ffcli.Command{

cmd/commands_test.go

@@ -128,3 +128,45 @@ func TestUnknownCommandPrintsHelpError(t *testing.T) {
 		t.Fatalf("expected unknown command message, got %q", stderr)
 	}
 }
+
+func TestBuildsInfoRequiresBuildID(t *testing.T) {
+	root := RootCommand("1.2.3")
+
+	stdout, stderr := captureOutput(t, func() {
+		if err := root.Parse([]string{"builds", "info"}); err != nil {
+			t.Fatalf("parse error: %v", err)
+		}
+		err := root.Run(context.Background())
+		if !errors.Is(err, flag.ErrHelp) {
+			t.Fatalf("expected ErrHelp, got %v", err)
+		}
+	})
+
+	if stdout != "" {
+		t.Fatalf("expected empty stdout, got %q", stdout)
+	}
+	if !strings.Contains(stderr, "--build is required") {
+		t.Fatalf("expected missing build error, got %q", stderr)
+	}
+}
+
+func TestBuildsExpireRequiresBuildID(t *testing.T) {
+	root := RootCommand("1.2.3")
+
+	stdout, stderr := captureOutput(t, func() {
+		if err := root.Parse([]string{"builds", "expire"}); err != nil {
+			t.Fatalf("parse error: %v", err)
+		}
+		err := root.Run(context.Background())
+		if !errors.Is(err, flag.ErrHelp) {
+			t.Fatalf("expected ErrHelp, got %v", err)
+		}
+	})
+
+	if stdout != "" {
+		t.Fatalf("expected empty stdout, got %q", stdout)
+	}
+	if !strings.Contains(stderr, "--build is required") {
+		t.Fatalf("expected missing build error, got %q", stderr)
+	}
+}

internal/asc/client.go

@@ -49,6 +49,12 @@ type Response[T any] struct {
 	Links Links         `json:"links,omitempty"`
 }
 
+// SingleResponse is a generic ASC API response wrapper for single resources.
+type SingleResponse[T any] struct {
+	Data  Resource[T] `json:"data"`
+	Links Links       `json:"links,omitempty"`
+}
+
 // FeedbackAttributes describes beta feedback screenshot submissions.
 type FeedbackAttributes struct {
 	CreatedDate    string `json:"createdDate"`
@@ -97,6 +103,9 @@ type AppsResponse = Response[AppAttributes]
 // BuildsResponse is the response from builds endpoint.
 type BuildsResponse = Response[BuildAttributes]
 
+// BuildResponse is the response from build detail/updates.
+type BuildResponse = SingleResponse[BuildAttributes]
+
 type listQuery struct {
 	limit   int
 	nextURL string
@@ -740,11 +749,12 @@ func (c *Client) GetBuilds(ctx context.Context, appID string, opts ...BuildsOpti
 		opt(query)
 	}
 
-	path := fmt.Sprintf("/v1/apps/%s/builds", appID)
+	path := "/v1/builds"
 	if query.nextURL != "" {
 		path = query.nextURL
 	} else {
 		values := url.Values{}
+		values.Set("filter[app]", appID)
 		if query.sort != "" {
 			values.Set("sort", query.sort)
 		}
@@ -769,6 +779,56 @@ func (c *Client) GetBuilds(ctx context.Context, appID string, opts ...BuildsOpti
 	return &response, nil
 }
 
+// GetBuild retrieves a single build by ID.
+func (c *Client) GetBuild(ctx context.Context, buildID string) (*BuildResponse, error) {
+	path := fmt.Sprintf("/v1/builds/%s", buildID)
+	data, err := c.do(ctx, "GET", path, nil)
+	if err != nil {
+		return nil, err
+	}
+
+	var response BuildResponse
+	if err := json.Unmarshal(data, &response); err != nil {
+		return nil, fmt.Errorf("failed to parse response: %w", err)
+	}
+
+	return &response, nil
+}
+
+// ExpireBuild expires a build for TestFlight testing.
+func (c *Client) ExpireBuild(ctx context.Context, buildID string) (*BuildResponse, error) {
+	payload := struct {
+		Data struct {
+			Type       string `json:"type"`
+			ID         string `json:"id"`
+			Attributes struct {
+				Expired bool `json:"expired"`
+			} `json:"attributes"`
+		} `json:"data"`
+	}{}
+	payload.Data.Type = "builds"
+	payload.Data.ID = buildID
+	payload.Data.Attributes.Expired = true
+
+	body, err := BuildRequestBody(payload)
+	if err != nil {
+		return nil, err
+	}
+
+	path := fmt.Sprintf("/v1/builds/%s", buildID)
+	data, err := c.do(ctx, "PATCH", path, body)
+	if err != nil {
+		return nil, err
+	}
+
+	var response BuildResponse
+	if err := json.Unmarshal(data, &response); err != nil {
+		return nil, fmt.Errorf("failed to parse response: %w", err)
+	}
+
+	return &response, nil
+}
+
 // Links represents pagination links
 type Links struct {
 	Self string `json:"self,omitempty"`
@@ -802,6 +862,8 @@ func PrintMarkdown(data interface{}) error {
 		return printAppsMarkdown(v)
 	case *BuildsResponse:
 		return printBuildsMarkdown(v)
+	case *BuildResponse:
+		return printBuildsMarkdown(&BuildsResponse{Data: []Resource[BuildAttributes]{v.Data}})
 	default:
 		return PrintJSON(data)
 	}
@@ -820,6 +882,8 @@ func PrintTable(data interface{}) error {
 		return printAppsTable(v)
 	case *BuildsResponse:
 		return printBuildsTable(v)
+	case *BuildResponse:
+		return printBuildsTable(&BuildsResponse{Data: []Resource[BuildAttributes]{v.Data}})
 	default:
 		return PrintJSON(data)
 	}