2 files changed, 107 insertions, 0 deletions

diff --git a/internal/store/buckets.go b/internal/store/buckets.go
index 8bbcca9..1e3ce9a 100644
--- a/internal/store/buckets.go
+++ b/internal/store/buckets.go
@@ -89,6 +89,51 @@ func (s *Store) RemoveBucketItem(taskID string) error {
 	return checkRowsAffected(result)
 }
 
+// GetBucketItems returns every task in a bucket's pool (dormant and active
+// both), active first, then by priority -- backs the bucket-management view.
+func (s *Store) GetBucketItems(bucketID string) ([]models.Task, error) {
+	rows, err := s.db.Query(`
+		SELECT id, content, description, project_name, project_id, due_date, priority, completed, labels, created_at,
+		       recurrence_freq, recurrence_interval, recurrence_weekdays, recurrence_series_id, next_occurrence_override, estimated_minutes,
+		       chain_id, chain_position, chain_unlocked,
+		       bucket_id, bucket_state, bucket_last_active_at
+		FROM native_tasks
+		WHERE bucket_id = ?
+		ORDER BY (bucket_state = 'active') DESC, priority DESC
+	`, bucketID)
+	if err != nil {
+		return nil, err
+	}
+	defer func() { _ = rows.Close() }()
+	return scanNativeTasks(rows)
+}
+
+// DeleteBucket removes a bucket and returns every member task to being a
+// plain, unbucketed task (clearing bucket_id/bucket_state/
+// bucket_last_active_at) rather than deleting them. Returns
+// ErrNativeTaskNotFound if id doesn't match any row.
+func (s *Store) DeleteBucket(id string) error {
+	tx, err := s.db.Begin()
+	if err != nil {
+		return err
+	}
+	defer func() { _ = tx.Rollback() }()
+
+	if _, err := tx.Exec(`
+		UPDATE native_tasks SET bucket_id = '', bucket_state = '', bucket_last_active_at = NULL, updated_at = ? WHERE bucket_id = ?
+	`, config.Now(), id); err != nil {
+		return err
+	}
+	result, err := tx.Exec(`DELETE FROM maintenance_buckets WHERE id = ?`, id)
+	if err != nil {
+		return err
+	}
+	if err := checkRowsAffected(result); err != nil {
+		return err
+	}
+	return tx.Commit()
+}
+
 // selectBucketCycle activates the top pick_n dormant items in the bucket's
 // pool, scored by staleness (never-activated items first, then oldest
 // bucket_last_active_at) with task priority as a tiebreaker. Activated items
diff --git a/internal/store/buckets_web_test.go b/internal/store/buckets_web_test.go
new file mode 100644
index 0000000..564b09a
--- /dev/null
+++ b/internal/store/buckets_web_test.go
@@ -0,0 +1,62 @@
+package store
+
+import (
+	"testing"
+	"time"
+)
+
+func TestGetBucketItems_ReturnsActiveFirst(t *testing.T) {
+	s := newNativeTasksTestStore(t)
+	bucket, err := s.CreateBucket("Gutters", 30, 5)
+	if err != nil {
+		t.Fatal(err)
+	}
+	createDormantTask(t, s, "dormant-1", bucket.ID, 1, nil)
+	createDormantTask(t, s, "dormant-2", bucket.ID, 1, nil)
+	if _, err := s.selectBucketCycle(bucket.ID, time.Now()); err != nil {
+		t.Fatal(err)
+	}
+
+	items, err := s.GetBucketItems(bucket.ID)
+	if err != nil {
+		t.Fatalf("GetBucketItems: %v", err)
+	}
+	if len(items) != 2 {
+		t.Fatalf("len(items) = %d, want 2", len(items))
+	}
+	if items[0].BucketState != "active" {
+		t.Errorf("items[0].BucketState = %q, want active (active items sort first)", items[0].BucketState)
+	}
+}
+
+func TestDeleteBucket_UnbucketsItemsAndRemovesBucket(t *testing.T) {
+	s := newNativeTasksTestStore(t)
+	bucket, err := s.CreateBucket("Gutters", 30, 1)
+	if err != nil {
+		t.Fatal(err)
+	}
+	createDormantTask(t, s, "item-1", bucket.ID, 1, nil)
+
+	if err := s.DeleteBucket(bucket.ID); err != nil {
+		t.Fatalf("DeleteBucket: %v", err)
+	}
+
+	if _, err := s.GetBucketByID(bucket.ID); err != ErrNativeTaskNotFound {
+		t.Errorf("bucket should be gone, err = %v", err)
+	}
+	task, err := s.GetNativeTaskByID("item-1")
+	if err != nil {
+		t.Fatal(err)
+	}
+	if task.BucketID != "" || task.BucketState != "" {
+		t.Errorf("task = %+v, want unbucketed (task itself kept, not deleted)", task)
+	}
+}
+
+func TestDeleteBucket_UnknownID_ReturnsErrNotFound(t *testing.T) {
+	s := newNativeTasksTestStore(t)
+
+	if err := s.DeleteBucket("does-not-exist"); err != ErrNativeTaskNotFound {
+		t.Errorf("err = %v, want ErrNativeTaskNotFound", err)
+	}
+}