Merge pull request #1 from dagjomar/cursor/review-project-and-implement-improvement-cb42

Cursor/review project and implement improvement cb42

Author: dagjomar

README.md

@@ -19,6 +19,8 @@ Git Stack is a simple, no-frills command-line utility that makes managing stacke
 
 - **Delete Branch Stacks:** Force-delete entire stacks of branches efficiently.
 
+- **Push Branch Stacks:** Force-push all branches in a stack to remote with a single command.
+
 ## Installation
 
 1. **Clone the Repository:**
@@ -79,6 +81,14 @@ Alternatively, to interactively delete the current branch's stack:
 
 This command prompts for confirmation before deleting the stack.
 
+- **Push a Branch Stack:**
+
+```bash
+  git stack push [base_name]
+```
+
+Force-pushes all branches in a stack to the remote repository. If no base name is provided, it uses the current branch's stack.
+
 ## TODOs
 - Create better usage examples and use cases explaining how to best us it in a normal workflow
 - Create command to manage rebasing

TODO.txt

@@ -23,7 +23,7 @@
 ## Advanced Stack Operations
 - `git stack rebase` - Rebase entire stack onto main
 - `git stack squash` - Combine all commits in a stack
-- `git stack push` - Push all stack branches in order
+- ✅ `git stack push` - Push all stack branches in order (COMPLETED)
 - `git stack clean` - Remove merged branches from stack
 - Undo/redo support for stack operations
 

bin/gitstack.sh

@@ -12,6 +12,7 @@
 #   gitstack.sh fix [stack]    # <-- fix an unhealthy stack by rebasing divergent branches
 #   gitstack.sh prev          # <-- checkout previous branch in stack (e.g. feature-2 -> feature-1)
 #   gitstack.sh next          # <-- checkout next branch in stack (e.g. feature-2 -> feature-3)
+#   gitstack.sh push [stack]  # <-- force-push all branches in a stack to remote
 #
 # Description:
 #   create      -> Creates a new branch named "<base_name>-0".
@@ -23,6 +24,7 @@
 #   fix         -> Fix an unhealthy stack by rebasing divergent branches.
 #   prev        -> Checkout previous branch in stack (e.g. feature-2 -> feature-1).
 #   next        -> Checkout next branch in stack (e.g. feature-2 -> feature-3).
+#   push        -> Force-push all branches in a stack to remote (uses current stack if none specified).
 # -----------------------------------------------------------------------------
 
 function usage() {
@@ -38,6 +40,7 @@ function usage() {
   echo "  $0 fix [stack]           (Fix an unhealthy stack by rebasing divergent branches)"
   echo "  $0 prev                  (Checkout previous branch in stack)"
   echo "  $0 next                  (Checkout next branch in stack)"
+  echo "  $0 push [stack]          (Force-push all branches in a stack to remote)"
   echo
   echo "Commands:"
   echo "  create      Creates a new branch named '<base_name>-0'."
@@ -49,6 +52,7 @@ function usage() {
   echo "  fix         Fix an unhealthy stack by rebasing divergent branches."
   echo "  prev        Checkout previous branch in stack (e.g. feature-2 -> feature-1)."
   echo "  next        Checkout next branch in stack (e.g. feature-2 -> feature-3)."
+  echo "  push        Force-push all branches in a stack to remote. Uses current stack if none specified."
   exit 1
 }
 
@@ -906,6 +910,9 @@ if [[ "${BASH_SOURCE[0]}" == "${0}" ]]; then
     next)
       next_stack
       ;;
+    push)
+      fp_stack "$@"
+      ;;
     *)
       usage
       ;;

bin/gitstack_test.sh

@@ -439,6 +439,65 @@ function test_convert_to_stack() {
   git branch -D dj/my-feature-0
 }
 
+# Test push command functionality
+function test_push_command() {
+  echo "Testing push command..."
+
+  # Set up a fake remote to test pushing
+  git init --bare "$TEST_DIR/remote.git"
+  git remote add origin "$TEST_DIR/remote.git"
+
+  # Create a test stack with multiple branches
+  git checkout main 2>/dev/null || git checkout master 2>/dev/null
+  "$SCRIPT_DIR/gitstack.sh" create push-test
+  echo "test1" > test1.txt
+  git add test1.txt
+  git commit -m "test1"
+  
+  "$SCRIPT_DIR/gitstack.sh" increment
+  echo "test2" > test2.txt
+  git add test2.txt
+  git commit -m "test2"
+  
+  "$SCRIPT_DIR/gitstack.sh" increment
+  echo "test3" > test3.txt
+  git add test3.txt
+  git commit -m "test3"
+
+  # Test push from current stack branch
+  if "$SCRIPT_DIR/gitstack.sh" push 2>&1 | grep -q "Successfully force-pushed all branches"; then
+    echo "✅ push command successfully pushed from current stack"
+  else
+    fail "push command failed to push from current stack"
+  fi
+
+  # Verify all branches were pushed to remote
+  local remote_branches
+  remote_branches=$(git ls-remote --heads origin | awk '{print $2}' | sed 's|refs/heads/||')
+  
+  if echo "$remote_branches" | grep -q "push-test-0" && \
+     echo "$remote_branches" | grep -q "push-test-1" && \
+     echo "$remote_branches" | grep -q "push-test-2"; then
+    echo "✅ All stack branches successfully pushed to remote"
+  else
+    fail "Not all stack branches were pushed to remote"
+  fi
+
+  # Test push with explicit stack name from different branch
+  git checkout main 2>/dev/null || git checkout master 2>/dev/null
+  if "$SCRIPT_DIR/gitstack.sh" push push-test 2>&1 | grep -q "Successfully force-pushed all branches"; then
+    echo "✅ push command with explicit stack name worked"
+  else
+    fail "push command with explicit stack name failed"
+  fi
+
+  # Clean up
+  git remote remove origin
+  rm -rf "$TEST_DIR/remote.git"
+  "$SCRIPT_DIR/gitstack.sh" delete -f push-test
+  rm -f test1.txt test2.txt test3.txt
+}
+
 # Run all tests
 function run_all_tests() {
   source_gitstack
@@ -450,6 +509,7 @@ function run_all_tests() {
   test_fix_command
   test_stack_navigation
   test_convert_to_stack
+  test_push_command
 }
 
 # Create a temporary test directory