From 3f6da132992ba5c53fe5960de5ba3355c003872b Mon Sep 17 00:00:00 2001
From: mkelvers <mikkelelvers@outlook.com>
Date: Wed, 13 May 2026 13:46:44 +0200
Subject: [PATCH] fix: remove explicit transaction from migration 012

Goose wraps each migration in a transaction automatically.
Explicit BEGIN TRANSACTION/COMMIT caused a nested transaction
error in SQLite.
---
 internal/database/migrations/012_remove_recovery_key.sql | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/internal/database/migrations/012_remove_recovery_key.sql b/internal/database/migrations/012_remove_recovery_key.sql
index 5f33e4e..d525494 100644
--- a/internal/database/migrations/012_remove_recovery_key.sql
+++ b/internal/database/migrations/012_remove_recovery_key.sql
@@ -1,8 +1,6 @@
 -- +goose Up
 PRAGMA foreign_keys = OFF;
 
-BEGIN TRANSACTION;
-
 CREATE TABLE user_new (
   id TEXT PRIMARY KEY,
   username TEXT NOT NULL UNIQUE,
@@ -18,8 +16,6 @@ DROP TABLE user;
 
 ALTER TABLE user_new RENAME TO user;
 
-COMMIT;
-
 PRAGMA foreign_keys = ON;
 
 -- +goose Down