fix: gather verbs, walk oom, admin path bypass, roulette refund

1 files changed, 14 insertions, 0 deletions

diff --git a/internal/player/hardening_removeitem_test.go b/internal/player/hardening_removeitem_test.go
index 5cb1e4c..b891d4e 100644
--- a/internal/player/hardening_removeitem_test.go
+++ b/internal/player/hardening_removeitem_test.go
@@ -21,3 +21,17 @@ func TestRemoveItemRejectsNonPositive(t *testing.T) {
 		t.Fatalf("stack mutated by a zero removal: %+v", slot)
 	}
 }
+
+// Regression: RemoveItem used to drain every stack and only then return false,
+// so a caller that rolled back on failure lost the items anyway.
+func TestRemoveItemIsAllOrNothing(t *testing.T) {
+	p := New("alice")
+	p.SetInvSlot(0, &InventorySlot{ItemID: "coin", Quantity: 2})
+	p.SetInvSlot(1, &InventorySlot{ItemID: "coin", Quantity: 3})
+	if p.RemoveItem("coin", 10) {
+		t.Fatal("removal of more than held reported success")
+	}
+	if got := p.CountItem("coin"); got != 5 {
+		t.Fatalf("failed removal drained stacks: count = %d, want 5", got)
+	}
+}