Update extension for DuckDB v1.4 compatibility (#35)

* Initial plan

* Update extension for DuckDB v1.4 compatibility

- Changed Load method signature from Load(DuckDB &db) to Load(ExtensionLoader &loader)
- Replaced ExtensionUtil::RegisterFunction with loader.RegisterFunction
- Replaced old C API entry points with DUCKDB_CPP_EXTENSION_ENTRY macro
- Updated vcpkg.json with overlay-triplets configuration
- Updated workflow to use v1.4-andium branch

Co-authored-by: lmangani <1423657+lmangani@users.noreply.github.com>

* Complete DuckDB v1.4 compatibility update

Co-authored-by: lmangani <1423657+lmangani@users.noreply.github.com>

* Fix ExtensionUtil removal and GetDatabase API changes

- Remove ExtensionUtil include (class removed in DuckDB v1.4)
- Change loader.GetDatabase() to loader.GetDatabaseInstance()
- Update both chsql_extension.cpp and chsql_system.cpp

Co-authored-by: lmangani <1423657+lmangani@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: lmangani <1423657+lmangani@users.noreply.github.com>

Author: Copilot

.github/workflows/MainDistributionPipeline.yml

@@ -12,18 +12,18 @@ concurrency:
   cancel-in-progress: true
 
 jobs:
-  #  duckdb-next-build:
+  duckdb-next-build:
+    name: Build extension binaries
+    uses: duckdb/extension-ci-tools/.github/workflows/_extension_distribution.yml@main
+    with:
+      duckdb_version: v1.4-andium
+      ci_tools_version: main
+      extension_name: chsql
+
+  #  duckdb-stable-build:
   #    name: Build extension binaries
   #    uses: duckdb/extension-ci-tools/.github/workflows/_extension_distribution.yml@main
   #    with:
   #      duckdb_version: main
   #      ci_tools_version: main
   #      extension_name: chsql
-
-  duckdb-stable-build:
-    name: Build extension binaries
-    uses: duckdb/extension-ci-tools/.github/workflows/_extension_distribution.yml@main
-    with:
-      duckdb_version: main
-      ci_tools_version: main
-      extension_name: chsql

_codeql_detected_source_root

@@ -0,0 +1 @@
+.

chsql/src/chsql_extension.cpp

@@ -5,7 +5,6 @@
 #include "duckdb/common/exception.hpp"
 #include "duckdb/common/string_util.hpp"
 #include "duckdb/function/scalar_function.hpp"
-#include "duckdb/main/extension_util.hpp"
 #include "duckdb/parser/parsed_data/create_scalar_function_info.hpp"
 #include "duckdb/catalog/default/default_functions.hpp"
 #include "duckdb/catalog/default/default_table_functions.hpp"
@@ -186,35 +185,35 @@ static const DefaultTableMacro chsql_table_macros[] = {
 	};
   // clang-format on
 
-  static void LoadInternal(DatabaseInstance &instance)
+  static void LoadInternal(ExtensionLoader &loader)
   {
     // Macros
     for (idx_t index = 0; chsql_macros[index].name != nullptr; index++)
     {
       auto info = DefaultFunctionGenerator::CreateInternalMacroInfo(chsql_macros[index]);
-      ExtensionUtil::RegisterFunction(instance, *info);
+      loader.RegisterFunction(*info);
     }
     // Table Macros
     for (idx_t index = 0; chsql_table_macros[index].name != nullptr; index++)
     {
       auto table_info = DefaultTableFunctionGenerator::CreateTableMacroInfo(chsql_table_macros[index]);
-      ExtensionUtil::RegisterFunction(instance, *table_info);
+      loader.RegisterFunction(*table_info);
     }
-    ExtensionUtil::RegisterFunction(instance, ReadParquetOrderedFunction());
+    loader.RegisterFunction(ReadParquetOrderedFunction());
     // Flock
-    ExtensionUtil::RegisterFunction(instance, DuckFlockTableFunction());
+    loader.RegisterFunction(DuckFlockTableFunction());
     // System Table
-    RegisterSystemFunctions(instance);
+    RegisterSystemFunctions(loader);
     // Register Views
-    Connection con(Catalog::GetSystemCatalog(instance).GetDatabase());
+    Connection con(Catalog::GetSystemCatalog(loader.GetDatabaseInstance()).GetDatabase());
     con.BeginTransaction();
     CreateSystemViews(con);
     con.Commit();
   }
 
-  void ChsqlExtension::Load(DuckDB &db)
+  void ChsqlExtension::Load(ExtensionLoader &loader)
   {
-    LoadInternal(*db.instance);
+    LoadInternal(loader);
   }
   std::string ChsqlExtension::Name()
   {
@@ -235,15 +234,9 @@ static const DefaultTableMacro chsql_table_macros[] = {
 extern "C"
 {
 
-  DUCKDB_EXTENSION_API void chsql_init(duckdb::DatabaseInstance &db)
+  DUCKDB_CPP_EXTENSION_ENTRY(chsql, loader)
   {
-    duckdb::DuckDB db_wrapper(db);
-    db_wrapper.LoadExtension<duckdb::ChsqlExtension>();
-  }
-
-  DUCKDB_EXTENSION_API const char *chsql_version()
-  {
-    return duckdb::DuckDB::LibraryVersion();
+    duckdb::LoadInternal(loader);
   }
 }
 

chsql/src/chsql_system.cpp

@@ -1,7 +1,6 @@
 #include "chsql_system.hpp"
 #include "duckdb/function/table_function.hpp"
 #include "duckdb/function/scalar_function.hpp"
-#include "duckdb/main/extension_util.hpp"
 #include "duckdb/main/client_context.hpp"
 #include "duckdb/catalog/catalog.hpp"
 #include "duckdb/main/connection.hpp"
@@ -559,30 +558,30 @@ static unique_ptr<FunctionData> SystemDisksBind(ClientContext &context, TableFun
 }
 
 // -- Registration function
-void RegisterSystemFunctions(DatabaseInstance &instance) {
+void RegisterSystemFunctions(ExtensionLoader &loader) {
     // Register system.databases table function
     auto databases_func = TableFunction("system_databases", {}, SystemDatabasesFunction, SystemDatabasesBind);
-    ExtensionUtil::RegisterFunction(instance, databases_func);
+    loader.RegisterFunction(databases_func);
 
     // Register system.tables table function
     auto tables_func = TableFunction("system_tables", {}, SystemTablesFunction, SystemTablesBind);
-    ExtensionUtil::RegisterFunction(instance, tables_func);
+    loader.RegisterFunction(tables_func);
 
     // Register system.columns table function
     auto columns_func = TableFunction("system_columns", {}, SystemColumnsFunction, SystemColumnsBind);
-    ExtensionUtil::RegisterFunction(instance, columns_func);
+    loader.RegisterFunction(columns_func);
 
     // Register system.functions table function
     auto functions_func = TableFunction("system_functions", {}, SystemFunctionsFunction, SystemFunctionsBind);
-    ExtensionUtil::RegisterFunction(instance, functions_func);
+    loader.RegisterFunction(functions_func);
 
     // Register system.uptime scalar function
     auto uptime_func = ScalarFunction("uptime", {}, LogicalType::BIGINT, SystemUptimeFunction);
-    ExtensionUtil::RegisterFunction(instance, uptime_func);
+    loader.RegisterFunction(uptime_func);
 
     // Register system.disks table function
     auto disks_func = TableFunction("system_disks", {}, SystemDisksFunction, SystemDisksBind);
-    ExtensionUtil::RegisterFunction(instance, disks_func);
+    loader.RegisterFunction(disks_func);
 }
 
 void CreateSystemViews(Connection &con) {

chsql/src/include/chsql_extension.hpp

@@ -8,7 +8,7 @@ namespace duckdb
 	class ChsqlExtension : public Extension
 	{
 	public:
-		void Load(DuckDB &db) override;
+		void Load(ExtensionLoader &loader) override;
 		std::string Name() override;
 		std::string Version() const override;
 	};

chsql/src/include/chsql_system.hpp

@@ -4,7 +4,7 @@
 
 namespace duckdb {
 
-void RegisterSystemFunctions(DatabaseInstance &instance);
+void RegisterSystemFunctions(ExtensionLoader &loader);
 void CreateSystemViews(Connection &con);
 
 } // namespace duckdb

chsql/vcpkg.json

@@ -1,3 +1,11 @@
 {
-  "dependencies": []
+  "dependencies": [],
+  "vcpkg-configuration": {
+    "overlay-ports": [
+      "../extension-ci-tools/vcpkg_ports"
+    ],
+    "overlay-triplets": [
+      "../extension-ci-tools/toolchains"
+    ]
+  }
 }