Make driver option required (#138)

rayakame · web-flow · commit bb4a7f25c832 · 2025-12-10T10:41:43.000+01:00
* work

* fragment
diff --git a/.changes/unreleased/Breaking-20251210-092132.yaml b/.changes/unreleased/Breaking-20251210-092132.yaml
@@ -0,0 +1,6 @@
+kind: Breaking
+body: Config option `driver` is now required instead of defaulting to asyncpg
+time: 2025-12-10T09:21:32.2253488Z
+custom:
+    Author: rayakame
+    PR: "138"
diff --git a/.changie.yaml b/.changie.yaml
@@ -29,6 +29,8 @@ kinds:
       auto: patch
     - label: Dependencys
       auto: patch
+    - label: Breaking
+      auto: patch
 newlines:
     afterChangelogHeader: 1
     beforeChangelogVersion: 1
diff --git a/internal/core/config.go b/internal/core/config.go
@@ -44,7 +44,7 @@ func ParseConfig(req *plugin.GenerateRequest) (*Config, error) {
 		return nil, fmt.Errorf("unmarshalling plugin options: %w", err)
 	}
 	if config.SqlDriver == "" {
-		config.SqlDriver = SQLDriverAioSQLite
+		return nil, fmt.Errorf("invalid options: driver must not be empty")
 	}
 	val, err := isDriverAsync(config.SqlDriver)
 	if err != nil {

Original file line number	Diff line number	Diff line change
`@@ -44,7 +44,7 @@ func ParseConfig(req plugin.GenerateRequest) (Config, error) {`
`44`	`44`	`return nil, fmt.Errorf("unmarshalling plugin options: %w", err)`
`45`	`45`	`}`
`46`	`46`	`if config.SqlDriver == "" {`
`47`		`- config.SqlDriver = SQLDriverAioSQLite`
	`47`	`+ return nil, fmt.Errorf("invalid options: driver must not be empty")`
`48`	`48`	`}`
`49`	`49`	`val, err := isDriverAsync(config.SqlDriver)`
`50`	`50`	`if err != nil {`