ref/store: Shape Batch a bit more like Transaction and eagerly validate what we could.

1 files changed, 16 insertions, 16 deletions

diff --git a/ref/store/files/batch_queue_ops.go b/ref/store/files/batch_queue_ops.go
index a99b3c89..441bcaba 100644
--- a/ref/store/files/batch_queue_ops.go
+++ b/ref/store/files/batch_queue_ops.go
@@ -3,41 +3,41 @@ package files
 import objectid "codeberg.org/lindenii/furgit/object/id"
 
 // Create queues a detached reference creation.
-func (batch *Batch) Create(name string, newID objectid.ObjectID) {
-	batch.queue(queuedUpdate{name: name, kind: updateCreate, newID: newID})
+func (batch *Batch) Create(name string, newID objectid.ObjectID) error {
+	return batch.queue(queuedUpdate{name: name, kind: updateCreate, newID: newID})
 }
 
 // Update queues a detached reference update.
-func (batch *Batch) Update(name string, newID, oldID objectid.ObjectID) {
-	batch.queue(queuedUpdate{name: name, kind: updateReplace, newID: newID, oldID: oldID})
+func (batch *Batch) Update(name string, newID, oldID objectid.ObjectID) error {
+	return batch.queue(queuedUpdate{name: name, kind: updateReplace, newID: newID, oldID: oldID})
 }
 
 // Delete queues a detached reference deletion.
-func (batch *Batch) Delete(name string, oldID objectid.ObjectID) {
-	batch.queue(queuedUpdate{name: name, kind: updateDelete, oldID: oldID})
+func (batch *Batch) Delete(name string, oldID objectid.ObjectID) error {
+	return batch.queue(queuedUpdate{name: name, kind: updateDelete, oldID: oldID})
 }
 
 // Verify queues a detached reference verification.
-func (batch *Batch) Verify(name string, oldID objectid.ObjectID) {
-	batch.queue(queuedUpdate{name: name, kind: updateVerify, oldID: oldID})
+func (batch *Batch) Verify(name string, oldID objectid.ObjectID) error {
+	return batch.queue(queuedUpdate{name: name, kind: updateVerify, oldID: oldID})
 }
 
 // CreateSymbolic queues a symbolic reference creation.
-func (batch *Batch) CreateSymbolic(name, newTarget string) {
-	batch.queue(queuedUpdate{name: name, kind: updateCreateSymbolic, newTarget: newTarget})
+func (batch *Batch) CreateSymbolic(name, newTarget string) error {
+	return batch.queue(queuedUpdate{name: name, kind: updateCreateSymbolic, newTarget: newTarget})
 }
 
 // UpdateSymbolic queues a symbolic reference update.
-func (batch *Batch) UpdateSymbolic(name, newTarget, oldTarget string) {
-	batch.queue(queuedUpdate{name: name, kind: updateReplaceSymbolic, newTarget: newTarget, oldTarget: oldTarget})
+func (batch *Batch) UpdateSymbolic(name, newTarget, oldTarget string) error {
+	return batch.queue(queuedUpdate{name: name, kind: updateReplaceSymbolic, newTarget: newTarget, oldTarget: oldTarget})
 }
 
 // DeleteSymbolic queues a symbolic reference deletion.
-func (batch *Batch) DeleteSymbolic(name, oldTarget string) {
-	batch.queue(queuedUpdate{name: name, kind: updateDeleteSymbolic, oldTarget: oldTarget})
+func (batch *Batch) DeleteSymbolic(name, oldTarget string) error {
+	return batch.queue(queuedUpdate{name: name, kind: updateDeleteSymbolic, oldTarget: oldTarget})
 }
 
 // VerifySymbolic queues a symbolic reference verification.
-func (batch *Batch) VerifySymbolic(name, oldTarget string) {
-	batch.queue(queuedUpdate{name: name, kind: updateVerifySymbolic, oldTarget: oldTarget})
+func (batch *Batch) VerifySymbolic(name, oldTarget string) error {
+	return batch.queue(queuedUpdate{name: name, kind: updateVerifySymbolic, oldTarget: oldTarget})
 }