fix: error handling

Author: rustyconover

CLAUDE.md

@@ -12,6 +12,8 @@ The extension is a DuckDB Community Extension using `popen()` for process pipes.
 
 ## Build Commands
 
+This extension does not use vcpkg.
+
 ```bash
 # Build release version
 GEN=ninja make release

src/shell_file_system.cpp

@@ -73,7 +73,7 @@ namespace duckdb
 
 		if (input.empty() || input.back() != '|')
 		{
-			throw std::runtime_error("Command must end with '|'.");
+			throw InvalidInputException("Command must end with '|'.");
 		}
 
 		std::string marker = "{allowed_exit_codes=";
@@ -103,16 +103,24 @@ namespace duckdb
 
 					if (!isNonNegativeInteger(token))
 					{
-						throw std::runtime_error("Invalid exit code: '" + token + "'. Must be a non-negative integer.");
+						throw InvalidInputException("Invalid exit code: '%s'. Must be a non-negative integer.", token);
 					}
 
-					int value = std::stoi(token);
+					int value;
+					try
+					{
+						value = std::stoi(token);
+					}
+					catch (const std::out_of_range &)
+					{
+						throw InvalidInputException("Exit code out of range: '%s'. Value too large.", token);
+					}
 					result.allowed_exit_codes.insert(value); // deduplicated automatically
 				}
 
 				if (result.allowed_exit_codes.empty())
 				{
-					throw std::runtime_error("No valid exit codes parsed.");
+					throw InvalidInputException("No valid exit codes parsed.");
 				}
 
 				return result;

src/shellfs_extension.cpp

@@ -23,7 +23,7 @@ namespace duckdb
 		// When writing to a PIPE ignore the SIGPIPE error and consider that the write succeeded.
 		config.AddExtensionOption("ignore_sigpipe", "Ignore SIGPIPE", LogicalType::BOOLEAN, Value(false));
 
-		QueryFarmSendTelemetry(loader, "shellfs", "2025120401");
+		QueryFarmSendTelemetry(loader, "shellfs", "2025120402");
 	}
 
 	void ShellfsExtension::Load(ExtensionLoader &loader)