Add Critical log before process exit on SQL file errors, update changelog

vbilopav · claude · vbilopav · commit e1213ca8a89d · 2026-03-26T09:10:18.000+01:00
Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/changelog.md b/changelog.md
@@ -230,6 +230,10 @@ Override with the `@path` annotation: `-- @path /custom/path/{id}`
 | `ParseErrorMode.Exit` (default) | Logs error, exits process | Fail-fast — catches SQL errors at startup |
 | `ParseErrorMode.Skip` | Logs error, skips file, continues | Production — tolerate partial failures |
 
+All SQL file errors are logged at `Error` level. In `Exit` mode, a `Critical` log explains the exit and how to switch to `Skip` mode.
+
+A warning is logged when the configured file pattern matches no files.
+
 Errors caught at startup:
 - Parse errors (malformed SQL, unclosed strings/quotes)
 - Describe errors (PostgreSQL syntax errors, invalid table/column references)
diff --git a/plugins/NpgsqlRest.SqlFileSource/SqlFileSource.cs b/plugins/NpgsqlRest.SqlFileSource/SqlFileSource.cs
@@ -57,6 +57,7 @@ public class SqlFileSource(SqlFileSourceOptions options) : IEndpointSource
                     NpgsqlRestOptions.Logger?.LogError("SqlFileSource: Error processing file {FilePath}: {Error}", filePath, ex.Message);
                     if (options.ErrorMode == ParseErrorMode.Exit)
                     {
+                        NpgsqlRestOptions.Logger?.LogCritical("SqlFileSource: Exiting due to SQL file error. Set ErrorMode to Skip to continue past errors.");
                         Environment.Exit(1);
                     }
                 }
@@ -94,6 +95,7 @@ private static (Routine, IRoutineSourceParameterFormatter)? ProcessFile(
             }
             if (options.ErrorMode == ParseErrorMode.Exit)
             {
+                NpgsqlRestOptions.Logger?.LogCritical("SqlFileSource: Exiting due to SQL file error. Set ErrorMode to Skip to continue past errors.");
                 Environment.Exit(1);
             }
             return null;
@@ -122,6 +124,7 @@ private static (Routine, IRoutineSourceParameterFormatter)? ProcessFile(
                 NpgsqlRestOptions.Logger?.LogError("SqlFileSource: {FilePath}: Describe failed: {Error}", filePath, describeResult.Error);
                 if (options.ErrorMode == ParseErrorMode.Exit)
                 {
+                    NpgsqlRestOptions.Logger?.LogCritical("SqlFileSource: Exiting due to SQL file error. Set ErrorMode to Skip to continue past errors.");
                     Environment.Exit(1);
                 }
                 return null;
@@ -143,6 +146,7 @@ private static (Routine, IRoutineSourceParameterFormatter)? ProcessFile(
                             NpgsqlRestOptions.Logger?.LogError("SqlFileSource: {FilePath}: {Error}", filePath, error);
                             if (options.ErrorMode == ParseErrorMode.Exit)
                             {
+                                NpgsqlRestOptions.Logger?.LogCritical("SqlFileSource: Exiting due to SQL file error. Set ErrorMode to Skip to continue past errors.");
                                 Environment.Exit(1);
                             }
                             return null;

Original file line number	Diff line number	Diff line change
`@@ -57,6 +57,7 @@ public class SqlFileSource(SqlFileSourceOptions options) : IEndpointSource`
`57`	`57`	`NpgsqlRestOptions.Logger?.LogError("SqlFileSource: Error processing file {FilePath}: {Error}", filePath, ex.Message);`
`58`	`58`	`if (options.ErrorMode == ParseErrorMode.Exit)`
`59`	`59`	`{`
	`60`	`+ NpgsqlRestOptions.Logger?.LogCritical("SqlFileSource: Exiting due to SQL file error. Set ErrorMode to Skip to continue past errors.");`
`60`	`61`	`Environment.Exit(1);`
`61`	`62`	`}`
`62`	`63`	`}`
`@@ -94,6 +95,7 @@ private static (Routine, IRoutineSourceParameterFormatter)? ProcessFile(`
`94`	`95`	`}`
`95`	`96`	`if (options.ErrorMode == ParseErrorMode.Exit)`
`96`	`97`	`{`
	`98`	`+ NpgsqlRestOptions.Logger?.LogCritical("SqlFileSource: Exiting due to SQL file error. Set ErrorMode to Skip to continue past errors.");`
`97`	`99`	`Environment.Exit(1);`
`98`	`100`	`}`
`99`	`101`	`return null;`
`@@ -122,6 +124,7 @@ private static (Routine, IRoutineSourceParameterFormatter)? ProcessFile(`
`122`	`124`	`NpgsqlRestOptions.Logger?.LogError("SqlFileSource: {FilePath}: Describe failed: {Error}", filePath, describeResult.Error);`
`123`	`125`	`if (options.ErrorMode == ParseErrorMode.Exit)`
`124`	`126`	`{`
	`127`	`+ NpgsqlRestOptions.Logger?.LogCritical("SqlFileSource: Exiting due to SQL file error. Set ErrorMode to Skip to continue past errors.");`
`125`	`128`	`Environment.Exit(1);`
`126`	`129`	`}`
`127`	`130`	`return null;`
`@@ -143,6 +146,7 @@ private static (Routine, IRoutineSourceParameterFormatter)? ProcessFile(`
`143`	`146`	`NpgsqlRestOptions.Logger?.LogError("SqlFileSource: {FilePath}: {Error}", filePath, error);`
`144`	`147`	`if (options.ErrorMode == ParseErrorMode.Exit)`
`145`	`148`	`{`
	`149`	`+ NpgsqlRestOptions.Logger?.LogCritical("SqlFileSource: Exiting due to SQL file error. Set ErrorMode to Skip to continue past errors.");`
`146`	`150`	`Environment.Exit(1);`
`147`	`151`	`}`
`148`	`152`	`return null;`