CommonJS compatibility for Powersync drizzle-driver

[samxtu]

Hi! 👋

Firstly, thanks for your work on this project! 🙂

Today I used patch-package to patch @powersync/[email protected] for the project I'm working on.

Here is the diff that solved my problem:

diff --git a/node_modules/@powersync/drizzle-driver/lib/src/index.js b/node_modules/@powersync/drizzle-driver/lib/src/index.js
index 2605c54..b21eb5c 100644
--- a/node_modules/@powersync/drizzle-driver/lib/src/index.js
+++ b/node_modules/@powersync/drizzle-driver/lib/src/index.js
@@ -1,4 +1,4 @@
-import { wrapPowerSyncWithDrizzle } from './sqlite/PowerSyncSQLiteDatabase';
-import { toCompilableQuery } from './utils/compilableQuery';
-import { DrizzleAppSchema, toPowerSyncTable } from './utils/schema';
-export { DrizzleAppSchema, toCompilableQuery, toPowerSyncTable, wrapPowerSyncWithDrizzle };
+import { wrapPowerSyncWithDrizzle } from './sqlite/PowerSyncSQLiteDatabase.js';
+import { toCompilableQuery } from './utils/compilableQuery.js';
+import { DrizzleAppSchema, toPowerSyncTable } from './utils/schema.js';
+export { DrizzleAppSchema, toPowerSyncTable, wrapPowerSyncWithDrizzle, toCompilableQuery };
diff --git a/node_modules/@powersync/drizzle-driver/lib/src/sqlite/PowerSyncSQLiteBaseSession.js b/node_modules/@powersync/drizzle-driver/lib/src/sqlite/PowerSyncSQLiteBaseSession.js
index 17cfe75..1eb9375 100644
--- a/node_modules/@powersync/drizzle-driver/lib/src/sqlite/PowerSyncSQLiteBaseSession.js
+++ b/node_modules/@powersync/drizzle-driver/lib/src/sqlite/PowerSyncSQLiteBaseSession.js
@@ -2,7 +2,7 @@ var _a, _b;
 import { entityKind } from 'drizzle-orm/entity';
 import { NoopLogger } from 'drizzle-orm/logger';
 import { SQLiteSession, SQLiteTransaction } from 'drizzle-orm/sqlite-core/session';
-import { PowerSyncSQLitePreparedQuery } from './PowerSyncSQLitePreparedQuery';
+import { PowerSyncSQLitePreparedQuery } from './PowerSyncSQLitePreparedQuery.js';
 export class PowerSyncSQLiteTransaction extends SQLiteTransaction {
 }
 _a = entityKind;
diff --git a/node_modules/@powersync/drizzle-driver/lib/src/sqlite/PowerSyncSQLiteDatabase.js b/node_modules/@powersync/drizzle-driver/lib/src/sqlite/PowerSyncSQLiteDatabase.js
index e43abc3..b928974 100644
--- a/node_modules/@powersync/drizzle-driver/lib/src/sqlite/PowerSyncSQLiteDatabase.js
+++ b/node_modules/@powersync/drizzle-driver/lib/src/sqlite/PowerSyncSQLiteDatabase.js
@@ -3,8 +3,8 @@ import { DefaultLogger } from 'drizzle-orm/logger';
 import { createTableRelationsHelpers, extractTablesRelationalConfig } from 'drizzle-orm/relations';
 import { BaseSQLiteDatabase } from 'drizzle-orm/sqlite-core/db';
 import { SQLiteAsyncDialect } from 'drizzle-orm/sqlite-core/dialect';
-import { toCompilableQuery } from './../utils/compilableQuery';
-import { PowerSyncSQLiteSession } from './PowerSyncSQLiteSession';
+import { toCompilableQuery } from './../utils/compilableQuery.js';
+import { PowerSyncSQLiteSession } from './PowerSyncSQLiteSession.js';
 export class PowerSyncSQLiteDatabase extends BaseSQLiteDatabase {
     constructor(db, config = {}) {
         const dialect = new SQLiteAsyncDialect({ casing: config.casing });
diff --git a/node_modules/@powersync/drizzle-driver/lib/src/sqlite/PowerSyncSQLiteSession.js b/node_modules/@powersync/drizzle-driver/lib/src/sqlite/PowerSyncSQLiteSession.js
index 135e075..cbb75dd 100644
--- a/node_modules/@powersync/drizzle-driver/lib/src/sqlite/PowerSyncSQLiteSession.js
+++ b/node_modules/@powersync/drizzle-driver/lib/src/sqlite/PowerSyncSQLiteSession.js
@@ -9,7 +9,7 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
 };
 var _a;
 import { entityKind } from 'drizzle-orm/entity';
-import { PowerSyncSQLiteTransaction, PowerSyncSQLiteBaseSession } from './PowerSyncSQLiteBaseSession';
+import { PowerSyncSQLiteTransaction, PowerSyncSQLiteBaseSession } from './PowerSyncSQLiteBaseSession.js';
 export class PowerSyncSQLiteSession extends PowerSyncSQLiteBaseSession {
     constructor(db, dialect, schema, options = {}) {
         super(db, dialect, schema, options);

This issue body was partially generated by patch-package.

[simolus3]

Is this for running on Node with CJS or a specific bundler? We can look into making those changes, but it would be useful to understand why your tools need that format so that we can test them as well.

[ryanleecode]

the package is also busted. it doesnt have "type module" import specifiers think its cjs.

https://arethetypeswrong.github.io/?p=%40powersync%2Fdrizzle-driver%400.4.0