rm unstack

Author: skarim

README.md

@@ -326,37 +326,6 @@ gh stack view --short
 gh stack view --json
 ```
 
-### `gh stack unstack`
-
-Remove a stack from local tracking and optionally delete it on GitHub.
-
-```
-gh stack unstack [branch] [flags]
-```
-
-If no branch is specified, uses the current branch to find the stack. By default, the stack is removed from both local tracking and GitHub. Use `--local` to only remove the local tracking entry.
-
-| Flag | Description |
-|------|-------------|
-| `--local` | Only delete the stack locally (keep it on GitHub) |
-
-| Argument | Description |
-|----------|-------------|
-| `[branch]` | A branch in the stack to delete (defaults to the current branch) |
-
-**Examples:**
-
-```sh
-# Remove the stack from local tracking and GitHub
-gh stack unstack
-
-# Only remove local tracking
-gh stack unstack --local
-
-# Specify a branch to identify the stack
-gh stack unstack feature-auth
-```
-
 ### `gh stack merge`
 
 Merge a stack of PRs.

cmd/root.go

@@ -35,7 +35,6 @@ func RootCmd() *cobra.Command {
 	root.AddCommand(PushCmd(cfg))
 	root.AddCommand(SubmitCmd(cfg))
 	root.AddCommand(SyncCmd(cfg))
-	root.AddCommand(UnstackCmd(cfg))
 	root.AddCommand(MergeCmd(cfg))
 
 	// Helper commands

cmd/root_test.go

@@ -8,7 +8,7 @@ import (
 
 func TestRootCmd_SubcommandRegistration(t *testing.T) {
 	root := RootCmd()
-	expected := []string{"init", "add", "checkout", "push", "sync", "unstack", "merge", "view", "rebase", "up", "down", "top", "bottom", "alias", "feedback"}
+	expected := []string{"init", "add", "checkout", "push", "sync", "merge", "view", "rebase", "up", "down", "top", "bottom", "alias", "feedback"}
 
 	registered := make(map[string]bool)
 	for _, cmd := range root.Commands() {

cmd/unstack.go

@@ -10,7 +10,6 @@ type ClientOps interface {
 	CreatePR(base, head, title, body string, draft bool) (*PullRequest, error)
 	CreateStack(prNumbers []int) (int, error)
 	UpdateStack(stackID string, prNumbers []int) error
-	DeleteStack() error
 }
 
 // Compile-time check that Client satisfies ClientOps.

cmd/unstack_test.go

@@ -266,12 +266,6 @@ func (c *Client) FindPRDetailsForBranch(branch string) (*PRDetails, error) {
 	}, nil
 }
 
-// DeleteStack deletes a stack on GitHub.
-// TODO: Implement once the stack API is available.
-func (c *Client) DeleteStack() error {
-	return fmt.Errorf("deleting a stack on GitHub is not yet supported by the API")
-}
-
 // CreateStack creates a stack on GitHub from an ordered list of PR numbers.
 // The PR numbers must be ordered from bottom to top of the stack and must
 // form a valid base-to-head chain. Returns the server-assigned stack ID.

internal/github/client_interface.go

@@ -1,7 +1,5 @@
 package github
 
-import "fmt"
-
 // MockClient is a test double for GitHub API operations.
 // Each field is an optional function that, when set, handles the corresponding
 // ClientOps method call. When nil, a reasonable default is returned.
@@ -12,7 +10,6 @@ type MockClient struct {
 	CreatePRFn      func(string, string, string, string, bool) (*PullRequest, error)
 	CreateStackFn   func([]int) (int, error)
 	UpdateStackFn   func(string, []int) error
-	DeleteStackFn   func() error
 }
 
 // Compile-time check that MockClient satisfies ClientOps.
@@ -46,13 +43,6 @@ func (m *MockClient) CreatePR(base, head, title, body string, draft bool) (*Pull
 	return nil, nil
 }
 
-func (m *MockClient) DeleteStack() error {
-	if m.DeleteStackFn != nil {
-		return m.DeleteStackFn()
-	}
-	return fmt.Errorf("deleting a stack on GitHub is not yet supported by the API")
-}
-
 func (m *MockClient) CreateStack(prNumbers []int) (int, error) {
 	if m.CreateStackFn != nil {
 		return m.CreateStackFn(prNumbers)

internal/github/github.go

@@ -359,7 +359,7 @@ func TestValidateNoDuplicateBranch(t *testing.T) {
 	})
 }
 
-// --- RemoveStackForBranch: used by unstack ---
+// --- RemoveStackForBranch ---
 
 func TestRemoveStackForBranch(t *testing.T) {
 	t.Run("found and removed", func(t *testing.T) {

internal/github/mock_client.go

@@ -142,7 +142,6 @@ Small, incidental fixes (e.g., fixing a typo you noticed) can go in the current
 | Switch to top/bottom branch | `gh stack top` / `gh stack bottom` |
 | Check out by PR | `gh stack checkout 42` |
 | Check out by branch | `gh stack checkout feature-auth` |
-| Remove stack | `gh stack unstack --local` |
 
 ---
 
@@ -357,12 +356,6 @@ echo "$output" | jq -r '.currentBranch'
 echo "$output" | jq '[.branches[] | .isMerged] | all'
 ```
 
-### Clean up after all PRs are merged
-
-```bash
-gh stack unstack --local
-```
-
 ---
 
 ## Commands
@@ -716,30 +709,6 @@ Resolves the target against locally tracked stacks. Accepts a PR number, PR URL,
 
 ---
 
-### Remove a stack — `gh stack unstack`
-
-Remove a stack from local tracking. Use `--local` to avoid warnings about unsupported server-side deletion.
-
-```
-gh stack unstack [branch] [flags]
-```
-
-```bash
-# Remove from local tracking
-gh stack unstack --local
-
-# Specify a branch to identify which stack
-gh stack unstack feature-auth --local
-```
-
-| Flag | Description |
-|------|-------------|
-| `--local` | Only delete the stack locally (recommended) |
-
-| Argument | Description |
-|----------|-------------|
-| `[branch]` | A branch in the stack (defaults to the current branch) |
-
 ---
 
 ## Output conventions
@@ -768,6 +737,5 @@ gh stack unstack feature-auth --local
 2. **Stack disambiguation cannot be bypassed.** If the current branch is the trunk of multiple stacks, commands error with code 6. Check out a non-shared branch first.
 3. **Multiple remotes require `--remote` or config.** If more than one remote is configured, pass `--remote <name>` or set `remote.pushDefault` in git config before running `push`, `sync`, or `rebase`.
 4. **Merging PRs:** Merging Stacked PRs from the CLI is not supported yet. Direct users to open the PR URL in a browser to merge PRs.
-5. **Server-side stack deletion is not supported.** Use `unstack --local`.
-6. **Server-side stack discovery is not supported.** `checkout` only works with locally tracked stacks.
-7. **PR title and body are auto-generated.** There is no flag to set a custom PR title or body during `submit`. The title and body are generated from commit messages plus a footer. Use `gh pr edit` to modify PR title and body after creation.
+5. **Server-side stack discovery is not supported.** `checkout` only works with locally tracked stacks.
+6. **PR title and body are auto-generated.** There is no flag to set a custom PR title or body during `submit`. The title and body are generated from commit messages plus a footer. Use `gh pr edit` to modify PR title and body after creation.