@sqg/sqg 0.8.0 → 0.10.0

package/dist/sqg.mjs

@@ -680,6 +680,8 @@ var EnumType = class {
 var SQLQuery = class {
 	columns;
 	allColumns;
+	/** Database-reported parameter types (variable name → SQL type), set by database adapters */
+	parameterTypes;
 	constructor(filename, id, rawQuery, queryAnonymous, queryNamed, queryPositional, type, isOne, isPluck, variables, config) {
 		this.filename = filename;
 		this.id = id;
@@ -964,6 +966,25 @@ async function initializeDatabase(queries, execQueries) {
 
 //#endregion
 //#region src/db/duckdb.ts
+function convertType(type) {
+	if (type instanceof DuckDBListType) return new ListType(convertType(type.valueType));
+	if (type instanceof DuckDBStructType) return new StructType(type.entryTypes.map((t, index) => ({
+		name: type.entryNames[index],
+		type: convertType(t),
+		nullable: true
+	})));
+	if (type instanceof DuckDBMapType) return new MapType({
+		name: "key",
+		type: convertType(type.keyType),
+		nullable: true
+	}, {
+		name: "value",
+		type: convertType(type.valueType),
+		nullable: true
+	});
+	if (type instanceof DuckDBEnumType) return new EnumType(type.values);
+	return type.toString();
+}
 const duckdb = new class {
 	db;
 	connection;
@@ -1003,6 +1024,15 @@ const duckdb = new class {
 				return ` ${part.value} `;
 			}).join("");
 			const stmt = await connection.prepare(sql);
+			if (stmt.parameterCount > 0) {
+				const paramTypes = /* @__PURE__ */ new Map();
+				for (let i = 0; i < stmt.parameterCount; i++) {
+					const paramType = stmt.parameterType(i + 1);
+					paramTypes.set(statement.parameters[i].name, convertType(paramType));
+				}
+				query.parameterTypes = paramTypes;
+				consola.debug("Parameter types:", Object.fromEntries(paramTypes));
+			}
 			for (let i = 0; i < stmt.parameterCount; i++) stmt.bindValue(i + 1, statement.parameters[i].value);
 			if (query.isQuery) {
 				const result = await stmt.stream();
@@ -1010,25 +1040,6 @@ const duckdb = new class {
 				const columnTypes = result.columnTypes();
 				consola.debug("Columns:", columnNames);
 				consola.debug("Types:", columnTypes.map((t) => `${t.toString()} / ${t.constructor.name}`));
-				function convertType(type) {
-					if (type instanceof DuckDBListType) return new ListType(convertType(type.valueType));
-					if (type instanceof DuckDBStructType) return new StructType(type.entryTypes.map((t, index) => ({
-						name: type.entryNames[index],
-						type: convertType(t),
-						nullable: true
-					})));
-					if (type instanceof DuckDBMapType) return new MapType({
-						name: "key",
-						type: convertType(type.keyType),
-						nullable: true
-					}, {
-						name: "value",
-						type: convertType(type.valueType),
-						nullable: true
-					});
-					if (type instanceof DuckDBEnumType) return new EnumType(type.values);
-					return type.toString();
-				}
 				query.columns = columnNames.map((name, index) => ({
 					name,
 					type: convertType(columnTypes[index]),
@@ -1051,33 +1062,17 @@ const duckdb = new class {
 		for (const table of tables) {
 			consola.info(`Introspecting table schema: ${table.tableName}`);
 			try {
-				const rows = (await connection.runAndReadAll(`DESCRIBE ${table.tableName}`)).getRows();
-				function convertType(type) {
-					if (type instanceof DuckDBListType) return new ListType(convertType(type.valueType));
-					if (type instanceof DuckDBStructType) return new StructType(type.entryTypes.map((t, index) => ({
-						name: type.entryNames[index],
-						type: convertType(t),
-						nullable: true
-					})));
-					if (type instanceof DuckDBMapType) return new MapType({
-						name: "key",
-						type: convertType(type.keyType),
-						nullable: true
-					}, {
-						name: "value",
-						type: convertType(type.valueType),
-						nullable: true
-					});
-					if (type instanceof DuckDBEnumType) return new EnumType(type.values);
-					return type.toString();
-				}
-				table.columns = rows.map((row) => {
-					return {
-						name: row[0],
-						type: row[1],
-						nullable: row[2] !== "NO"
-					};
-				});
+				const descRows = (await connection.runAndReadAll(`DESCRIBE ${table.tableName}`)).getRows();
+				const nullabilityMap = /* @__PURE__ */ new Map();
+				for (const row of descRows) nullabilityMap.set(row[0], row[2] !== "NO");
+				const stream = await (await connection.prepare(`SELECT * FROM ${table.tableName} LIMIT 0`)).stream();
+				const columnNames = stream.columnNames();
+				const columnTypes = stream.columnTypes();
+				table.columns = columnNames.map((name, index) => ({
+					name,
+					type: convertType(columnTypes[index]),
+					nullable: nullabilityMap.get(name) ?? true
+				}));
 				consola.debug(`Table ${table.tableName} columns:`, table.columns);
 				consola.success(`Introspected table: ${table.tableName} (${table.columns.length} columns)`);
 			} catch (error) {
@@ -1205,6 +1200,23 @@ const postgres = new class {
 				if (value.startsWith("'") && value.endsWith("'") || value.startsWith("\"") && value.endsWith("\"")) return value.slice(1, -1);
 				return value;
 			});
+			if (statement.parameters.length > 0) try {
+				await db.query(`DEALLOCATE ALL`);
+				await db.query(`PREPARE sqg_param_check AS ${statement.sql}`);
+				const paramTypeResult = await db.query(`SELECT unnest(parameter_types)::oid AS oid FROM pg_prepared_statements WHERE name = 'sqg_param_check'`);
+				await db.query(`DEALLOCATE sqg_param_check`);
+				if (paramTypeResult.rows.length === statement.parameters.length) {
+					const paramTypes = /* @__PURE__ */ new Map();
+					for (let i = 0; i < statement.parameters.length; i++) {
+						const typeName = getTypeName(Number(paramTypeResult.rows[i].oid));
+						paramTypes.set(statement.parameters[i].name, typeName);
+					}
+					query.parameterTypes = paramTypes;
+					consola.debug("Parameter types:", Object.fromEntries(paramTypes));
+				}
+			} catch (e) {
+				consola.debug(`Parameter type introspection failed for ${query.id}, using heuristic:`, e.message);
+			}
 			let result;
 			try {
 				await db.query("BEGIN");
@@ -1964,6 +1976,24 @@ var JavaDuckDBArrowGenerator = class extends BaseGenerator {
 
 //#endregion
 //#region src/generators/typescript-generator.ts
+/** Resolve a ColumnType to its DuckDB type constant name (e.g. "VARCHAR", "INTEGER") for use in generated code. */
+function resolveElementType(baseType) {
+	if (baseType instanceof ListType) return `new DuckDBListType(${resolveElementType(baseType.baseType)})`;
+	return {
+		VARCHAR: "VARCHAR",
+		TEXT: "VARCHAR",
+		INTEGER: "INTEGER",
+		INT: "INTEGER",
+		BIGINT: "BIGINT",
+		DOUBLE: "DOUBLE",
+		FLOAT: "FLOAT",
+		BOOLEAN: "BOOLEAN",
+		DATE: "DATE",
+		TIMESTAMP: "TIMESTAMP",
+		SMALLINT: "SMALLINT",
+		TINYINT: "TINYINT"
+	}[baseType.toString().toUpperCase()] || "VARCHAR";
+}
 var TsGenerator = class extends BaseGenerator {
 	constructor(template) {
 		super(template, new TypeScriptTypeMapper());
@@ -1983,6 +2013,7 @@ var TsGenerator = class extends BaseGenerator {
 	async beforeGenerate(_projectDir, _gen, _queries, _tables) {
 		Handlebars.registerHelper("quote", (value) => this.quote(value));
 		Handlebars.registerHelper("appendMethod", (column) => {
+			if (column.type instanceof ListType) return "List";
 			const typeStr = column.type?.toString().toUpperCase() || "";
 			if (typeStr === "INTEGER" || typeStr === "INT" || typeStr === "INT4" || typeStr === "SIGNED") return "Integer";
 			if (typeStr === "SMALLINT" || typeStr === "INT2" || typeStr === "SHORT") return "SmallInt";
@@ -2004,7 +2035,15 @@ var TsGenerator = class extends BaseGenerator {
 			if (typeStr === "INTERVAL") return "Interval";
 			return "Varchar";
 		});
+		Handlebars.registerHelper("appendListTypeArg", (column) => {
+			if (!(column.type instanceof ListType)) return "";
+			return `, new DuckDBListType(${resolveElementType(column.type.baseType)})`;
+		});
 		Handlebars.registerHelper("tsTypeForAppender", (column) => {
+			if (column.type instanceof ListType) {
+				const baseType$1 = "readonly DuckDBValue[]";
+				return column.nullable ? `${baseType$1} | null` : baseType$1;
+			}
 			const typeStr = column.type?.toString().toUpperCase() || "";
 			let baseType;
 			if (typeStr === "INTEGER" || typeStr === "INT" || typeStr === "INT4" || typeStr === "SMALLINT" || typeStr === "INT2" || typeStr === "TINYINT" || typeStr === "INT1" || typeStr === "UINTEGER" || typeStr === "UINT4" || typeStr === "USMALLINT" || typeStr === "UINT2" || typeStr === "UTINYINT" || typeStr === "UINT1" || typeStr === "DOUBLE" || typeStr === "FLOAT8" || typeStr === "FLOAT" || typeStr === "FLOAT4" || typeStr === "REAL") baseType = "number";
@@ -2145,6 +2184,21 @@ var TsDuckDBGenerator = class extends TsGenerator {
 	}
 	async beforeGenerate(projectDir, gen, queries, tables) {
 		await super.beforeGenerate(projectDir, gen, queries, tables);
+		Handlebars.registerHelper("hasListParams", (queryHelper) => {
+			const paramTypes = queryHelper.query.parameterTypes;
+			if (!paramTypes) return false;
+			for (const [, colType] of paramTypes) if (colType instanceof ListType) return true;
+			return false;
+		});
+		Handlebars.registerHelper("bindStatements", (queryHelper) => {
+			const paramNames = queryHelper.parameterNames;
+			const paramTypes = queryHelper.query.parameterTypes;
+			return paramNames.map((name, i) => {
+				const colType = paramTypes?.get(name);
+				if (colType instanceof ListType) return `stmt.bindList(${i + 1}, ${name}.items, new DuckDBListType(${resolveElementType(colType.baseType)}));`;
+				return `stmt.bindValue(${i + 1}, ${name});`;
+			}).join("\n        ");
+		});
 		Handlebars.registerHelper("tsType", (column) => {
 			const inlineType = (col) => {
 				const t = col.type;
@@ -2304,7 +2358,7 @@ var SqlQueryHelper = class {
 	get variables() {
 		return Array.from(this.query.variables.entries()).map(([name, value]) => ({
 			name,
-			type: this.generator.mapParameterType(detectParameterType(value), false)
+			type: this.generator.mapParameterType(this.query.parameterTypes?.get(name) ?? detectParameterType(value), false)
 		}));
 	}
 	get sqlQuery() {
@@ -2514,7 +2568,10 @@ async function writeGeneratedFile(projectDir, gen, generator, file, queries, tab
 	await generator.beforeGenerate(projectDir, gen, queries, tables);
 	const templatePath = join(dirname(new URL(import.meta.url).pathname), gen.template ?? generator.template);
 	const name = gen.name ?? basename(file, extname(file));
-	const sourceFile = generateSourceFile(name, queries, tables, templatePath, generator, engine, gen.projectName ?? name, gen.config);
+	const sourceFile = generateSourceFile(name, queries, tables, templatePath, generator, engine, gen.projectName ?? name, {
+		migrations: true,
+		...gen.config
+	});
 	if (writeToStdout) {
 		process.stdout.write(sourceFile);
 		if (!sourceFile.endsWith("\n")) process.stdout.write("\n");
@@ -2668,7 +2725,7 @@ async function processProject(projectPath) {
 //#region src/mcp-server.ts
 const server = new Server({
 	name: "sqg-mcp",
-	version: process.env.npm_package_version ?? "0.8.0"
+	version: process.env.npm_package_version ?? "0.10.0"
 }, { capabilities: {
 	tools: {},
 	resources: {}
@@ -2997,7 +3054,7 @@ async function startMcpServer() {
 
 //#endregion
 //#region src/sqg.ts
-const version = process.env.npm_package_version ?? "0.8.0";
+const version = process.env.npm_package_version ?? "0.10.0";
 const description = process.env.npm_package_description ?? "SQG - SQL Query Generator - Type-safe code generation from SQL (https://sqg.dev)";
 consola.level = LogLevels.info;
 const program = new Command().name("sqg").description(`${description}

package/dist/templates/java-jdbc.hbs

@@ -17,11 +17,17 @@ import java.time.OffsetDateTime;
 import java.time.OffsetTime;
 import java.util.ArrayList;
 import java.util.Arrays;
+import java.util.Iterator;
 import java.util.List;
 import java.util.HashMap;
 import java.util.Collections;
+import java.util.NoSuchElementException;
+import java.util.Spliterator;
+import java.util.Spliterators;
 import java.util.UUID;
 import java.util.function.Function;
+import java.util.stream.Stream;
+import java.util.stream.StreamSupport;
 {{#if tables.length}}
 import org.duckdb.DuckDBAppender;
 import org.duckdb.DuckDBConnection;
@@ -203,6 +209,36 @@ public class {{className}} {
             {{> execute}}
         }
     }
+    {{#if isQuery}}{{#unless isOne}}
+    public Stream<{{rowType}}> {{functionName}}Stream({{#each variables}}{{{type}}} {{name}}{{#unless @last}}, {{/unless}}{{/each}}) throws SQLException {
+        var stmt = connection.prepareStatement({{{partsToString sqlQueryParts}}});
+        {{#each parameterNames}}stmt.setObject({{plusOne @index}}, {{this}});
+        {{/each}}
+        var rs = stmt.executeQuery();
+        var iter = new Iterator<{{rowType}}>() {
+            private Boolean hasNext = null;
+            public boolean hasNext() {
+                if (hasNext == null) {
+                    try { hasNext = rs.next(); }
+                    catch (SQLException e) { throw new RuntimeException(e); }
+                }
+                return hasNext;
+            }
+            public {{rowType}} next() {
+                if (!hasNext()) throw new NoSuchElementException();
+                hasNext = null;
+                try { return {{> readRow}}; }
+                catch (SQLException e) { throw new RuntimeException(e); }
+            }
+        };
+        return StreamSupport.stream(
+            Spliterators.spliteratorUnknownSize(iter, Spliterator.ORDERED), false
+        ).onClose(() -> {
+            try { rs.close(); stmt.close(); }
+            catch (SQLException e) { throw new RuntimeException(e); }
+        });
+    }
+    {{/unless}}{{/if}}
     {{/unless}}
     {{/each}}
 
@@ -220,7 +256,7 @@ public class {{className}} {
 
 {{#each tables}}
 /** Row type for {{tableName}} appender */
-public record {{rowTypeName}}({{#each columns}}{{mapType this}} {{name}}{{#unless @last}}, {{/unless}}{{/each}}) {}
+public record {{rowTypeName}}({{#each columns}}{{{mapType this}}} {{name}}{{#unless @last}}, {{/unless}}{{/each}}) {}
 
 /** Appender for bulk inserts into {{tableName}} */
 public static class {{className}} implements AutoCloseable {
@@ -246,7 +282,7 @@ public static class {{className}} implements AutoCloseable {
     }
 
     /** Append a single row with individual values */
-    public {{className}} append({{#each columns}}{{mapType this}} {{name}}{{#unless @last}}, {{/unless}}{{/each}}) throws SQLException {
+    public {{className}} append({{#each columns}}{{{mapType this}}} {{name}}{{#unless @last}}, {{/unless}}{{/each}}) throws SQLException {
         appender.beginRow();
         {{#each columns}}
         appender.append({{name}});

package/dist/templates/turso.hbs

@@ -33,36 +33,32 @@ export class {{className}} {
 
 {{#if config.migrations}}
     static async applyMigrations(db: Database, projectName = '{{projectName}}'): Promise<void> {
-        const createStmt = await db.prepare(`CREATE TABLE IF NOT EXISTS _sqg_migrations (
+        const createStmt = db.prepare(`CREATE TABLE IF NOT EXISTS _sqg_migrations (
             project TEXT NOT NULL,
             migration_id TEXT NOT NULL,
             applied_at TEXT NOT NULL DEFAULT (datetime('now')),
             PRIMARY KEY (project, migration_id)
         )`);
         await createStmt.run();
-        const tx = await db.transaction('write');
-        try {
-            const selectStmt = await tx.prepare('SELECT migration_id FROM _sqg_migrations WHERE project = ?');
-            const rows = await selectStmt.all(projectName) as { migration_id: string }[];
-            const applied = new Set(rows.map(r => r.migration_id));
-            const migrations: [string, string][] = [
-                {{#each migrations}}
-                ['{{{id}}}', {{{quote sqlQuery}}}],
-                {{/each}}
-            ];
+        const selectStmt = db.prepare('SELECT migration_id FROM _sqg_migrations WHERE project = ?');
+        const rows = await selectStmt.all(projectName) as { migration_id: string }[];
+        const applied = new Set(rows.map(r => r.migration_id));
+        const migrations: [string, string][] = [
+            {{#each migrations}}
+            ['{{{id}}}', {{{quote sqlQuery}}}],
+            {{/each}}
+        ];
+        const applyFn = db.transaction(async () => {
             for (const [id, sql] of migrations) {
                 if (!applied.has(id)) {
-                    const execStmt = await tx.prepare(sql);
+                    const execStmt = db.prepare(sql);
                     await execStmt.run();
-                    const insertStmt = await tx.prepare('INSERT INTO _sqg_migrations (project, migration_id) VALUES (?, ?)');
+                    const insertStmt = db.prepare('INSERT INTO _sqg_migrations (project, migration_id) VALUES (?, ?)');
                     await insertStmt.run(projectName, id);
                 }
             }
-            await tx.commit();
-        } catch (e) {
-            await tx.rollback();
-            throw e;
-        }
+        });
+        await applyFn();
     }
 {{/if}}
 

package/dist/templates/typescript-duckdb.hbs

@@ -1,5 +1,5 @@
 // {{generatedComment}}
-import type { DuckDBConnection, DuckDBMaterializedResult, DuckDBAppender, DuckDBDateValue, DuckDBTimeValue, DuckDBTimestampValue, DuckDBBlobValue } from "@duckdb/node-api";
+import { DuckDBListType, listValue, VARCHAR, INTEGER, BIGINT, DOUBLE, FLOAT, BOOLEAN, SMALLINT, TINYINT, type DuckDBConnection, type DuckDBMaterializedResult, type DuckDBAppender, type DuckDBDateValue, type DuckDBTimeValue, type DuckDBTimestampValue, type DuckDBBlobValue, type DuckDBValue } from "@duckdb/node-api";
 
 export class {{className}} {
     
@@ -60,6 +60,28 @@ export class {{className}} {
     {{#unless skipGenerateFunction}}
     async {{functionName}}({{#each variables}}{{name}}: {{type}}{{#unless @last}}, {{/unless}}{{/each}}): Promise<{{> returnType }}> {
         const sql = {{{quote sqlQuery}}};
+        {{#if (hasListParams this)}}
+        const stmt = await this.conn.prepare(sql);
+        {{{bindStatements this}}}
+        {{#if isQuery}}
+        const reader = await stmt.runAndReadAll();
+        {{#if isPluck}}
+        {{#if isOne}}
+        return reader.getRows()[0]?.[0] as {{> rowType}} | undefined;
+        {{else}}
+        return reader.getRows().map((row) => row[0] as {{> rowType}});
+        {{/if}}
+        {{else}}
+        {{#if isOne}}
+        return reader.getRowObjects()[0] as {{> rowType}} | undefined;
+        {{else}}
+        return reader.getRowObjects() as {{> rowType}}[];
+        {{/if}}
+        {{/if}}
+        {{else}}
+        return await stmt.run();
+        {{/if}}
+        {{else}}
         {{#if isQuery}}
         const reader = await this.conn.runAndReadAll(sql,[{{> params}}]);
         {{#if isPluck}}
@@ -78,6 +100,7 @@ export class {{className}} {
         {{else}}
         return await this.conn.run(sql,[{{> params}}]);
         {{/if}}
+        {{/if}}
     }
     {{/unless}}
 
@@ -113,10 +136,10 @@ export class {{className}} {
         if (row.{{name}} === null || row.{{name}} === undefined) {
             this.appender.appendNull();
         } else {
-            this.appender.append{{appendMethod this}}(row.{{name}});
+            this.appender.append{{appendMethod this}}(row.{{name}}{{{appendListTypeArg this}}});
         }
         {{else}}
-        this.appender.append{{appendMethod this}}(row.{{name}});
+        this.appender.append{{appendMethod this}}(row.{{name}}{{{appendListTypeArg this}}});
         {{/if}}
         {{/each}}
         this.appender.endRow();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sqg/sqg",
-  "version": "0.8.0",
+  "version": "0.10.0",
   "description": "SQG - SQL Query Generator - Type-safe code generation from SQL (https://sqg.dev)",
   "type": "module",
   "bin": {