start declaring modules

Author: stevensJourney

package.json

@@ -20,7 +20,7 @@
     "format": "prettier --write .",
     "lint": "eslint .",
     "release": "pnpm build:packages:prod && pnpm changeset publish",
-    "test": "pnpm run -r --workspace-concurrency=0 test"
+    "test": "pnpm run -r --workspace-concurrency=1 test --run"
   },
   "keywords": [],
   "type": "module",

packages/attachments/package.json

@@ -16,15 +16,24 @@
     "url": "https://github.com/powersync-ja/powersync-js/issues"
   },
   "description": "An PowerSync library to manage attachments for TypeScript and React Native apps",
+  "type": "module",
   "main": "./lib/index.js",
+  "module": "./lib/index.js",
   "types": "./lib/index.d.ts",
+  "exports": {
+    ".": {
+      "types": "./lib/index.d.ts",
+      "import": "./lib/index.js",
+      "default": "./lib/index.js"
+    }
+  },
   "files": [
     "lib"
   ],
   "scripts": {
     "build": "tsc -b",
     "build:prod": "tsc -b --sourceMap false",
-    "clean": "rm -rf lib tsconfig.tsbuildinfo node_modules",
+    "clean": "rm -rf lib",
     "watch": "tsc -b -w",
     "test": "pnpm build && vitest"
   },

packages/attachments/src/AbstractAttachmentQueue.ts

@@ -1,6 +1,6 @@
 import { AbstractPowerSyncDatabase, Transaction } from '@powersync/common';
-import { ATTACHMENT_TABLE, AttachmentRecord, AttachmentState } from './Schema';
-import { EncodingType, StorageAdapter } from './StorageAdapter';
+import { ATTACHMENT_TABLE, AttachmentRecord, AttachmentState } from './Schema.js';
+import { EncodingType, StorageAdapter } from './StorageAdapter.js';
 
 export interface AttachmentQueueOptions {
   powersync: AbstractPowerSyncDatabase;

packages/attachments/src/index.ts

@@ -1,4 +1,4 @@
-export * from './Schema';
-export * from './StorageAdapter';
+export * from './Schema.js';
+export * from './StorageAdapter.js';
 
-export * from './AbstractAttachmentQueue';
+export * from './AbstractAttachmentQueue.js';

packages/attachments/tests/attachments/AttachmentQueue.test.ts

@@ -1,26 +1,24 @@
-import * as commonSdk from '@powersync/common';
 import { beforeEach, describe, expect, it, vi } from 'vitest';
-import { AbstractAttachmentQueue } from '../../src/AbstractAttachmentQueue';
-import { AttachmentRecord, AttachmentState } from '../../src/Schema';
-import { AbstractPowerSyncDatabase } from '@powersync/common';
-import { StorageAdapter } from '../../src/StorageAdapter';
+import { AbstractAttachmentQueue } from '../../src/AbstractAttachmentQueue.js';
+import { AttachmentRecord, AttachmentState } from '../../src/Schema.js';
+import { StorageAdapter } from '../../src/StorageAdapter.js';
 
 const record = {
   id: 'test-1',
   filename: 'test.jpg',
   state: AttachmentState.QUEUED_DOWNLOAD
- }
+};
 
 const mockPowerSync = {
   currentStatus: { status: 'initial' },
   registerListener: vi.fn(() => {}),
   resolveTables: vi.fn(() => ['table1', 'table2']),
   onChangeWithCallback: vi.fn(),
-  getAll: vi.fn(() => Promise.resolve([{id: 'test-1'}, {id: 'test-2'}])),
+  getAll: vi.fn(() => Promise.resolve([{ id: 'test-1' }, { id: 'test-2' }])),
   execute: vi.fn(() => Promise.resolve()),
   getOptional: vi.fn((_query, params) => Promise.resolve(record)),
   watch: vi.fn((query, params, callbacks) => {
-    callbacks?.onResult?.({ rows: { _array: [{id: 'test-1'}, {id: 'test-2'}] } });
+    callbacks?.onResult?.({ rows: { _array: [{ id: 'test-1' }, { id: 'test-2' }] } });
   }),
   writeTransaction: vi.fn(async (callback) => {
     await callback({
@@ -57,9 +55,9 @@ describe('attachments', () => {
 
   it('should not download attachments when downloadRecord is called with downloadAttachments false', async () => {
     const queue = new TestAttachmentQueue({
-        powersync: mockPowerSync as any,
-        storage: mockStorage,
-        downloadAttachments: false
+      powersync: mockPowerSync as any,
+      storage: mockStorage,
+      downloadAttachments: false
     });
 
     await queue.downloadRecord(record);
@@ -69,9 +67,9 @@ describe('attachments', () => {
 
   it('should download attachments when downloadRecord is called with downloadAttachments true', async () => {
     const queue = new TestAttachmentQueue({
-        powersync: mockPowerSync as any,
-        storage: mockStorage,
-        downloadAttachments: true
+      powersync: mockPowerSync as any,
+      storage: mockStorage,
+      downloadAttachments: true
     });
 
     await queue.downloadRecord(record);
@@ -82,9 +80,9 @@ describe('attachments', () => {
   // Testing the inverse of this test, i.e. when downloadAttachments is false, is not required as you can't wait for something that does not happen
   it('should not download attachments with watchDownloads is called with downloadAttachments false', async () => {
     const queue = new TestAttachmentQueue({
-        powersync: mockPowerSync as any,
-        storage: mockStorage,
-        downloadAttachments: true
+      powersync: mockPowerSync as any,
+      storage: mockStorage,
+      downloadAttachments: true
     });
 
     queue.watchDownloads();

packages/attachments/tsconfig.json

@@ -2,9 +2,12 @@
   "extends": "../../tsconfig.base",
   "compilerOptions": {
     "baseUrl": "./",
-    "jsx": "react",
     "rootDir": "src",
-    "outDir": "./lib"
+    "module": "node16",
+    "moduleResolution": "node16",
+    "target": "es6",
+    "outDir": "./lib",
+    "tsBuildInfoFile": "./lib/tsconfig.tsbuildinfo"
   },
   "references": [
     {

packages/common/package.json

@@ -12,9 +12,9 @@
   "types": "lib/index.d.ts",
   "exports": {
     ".": {
+      "types": "./lib/index.d.ts",
       "import": "./dist/bundle.mjs",
       "require": "./dist/bundle.cjs",
-      "types": "./lib/index.d.ts",
       "default": "./dist/bundle.mjs"
     }
   },
@@ -35,7 +35,7 @@
   "scripts": {
     "build": "tsc -b && rollup -c rollup.config.mjs",
     "build:prod": "tsc -b --sourceMap false && rollup -c rollup.config.mjs --sourceMap false",
-    "clean": "rm -rf lib dist tsconfig.tsbuildinfo node_modules",
+    "clean": "rm -rf lib dist",
     "test": "vitest"
   },
   "dependencies": {

packages/common/tsconfig.json

@@ -13,7 +13,8 @@
     "preserveConstEnums": true,
     "esModuleInterop": false,
     "skipLibCheck": false,
-    "strictNullChecks": true
+    "strictNullChecks": true,
+    "tsBuildInfoFile": "./lib/tsconfig.tsbuildinfo"
   },
   "include": ["src/**/*"]
 }

packages/drizzle-driver/package.json

@@ -2,8 +2,17 @@
   "name": "@powersync/drizzle-driver",
   "version": "0.4.0",
   "description": "Drizzle driver for PowerSync",
+  "type": "module",
   "main": "lib/src/index.js",
+  "module": "lib/src/index.js",
   "types": "lib/src/index.d.ts",
+  "exports": {
+    ".": {
+      "types": "./lib/src/index.d.ts",
+      "import": "./lib/src/index.js",
+      "default": "./lib/src/index.js"
+    }
+  },
   "author": "JOURNEYAPPS",
   "license": "Apache-2.0",
   "files": [
@@ -21,7 +30,7 @@
   "scripts": {
     "build": "tsc --build",
     "build:prod": "tsc --build --sourceMap false",
-    "clean": "rm -rf lib tsconfig.tsbuildinfo",
+    "clean": "rm -rf lib",
     "watch": "tsc --build -w",
     "test": "vitest"
   },

packages/drizzle-driver/src/index.ts

@@ -2,8 +2,8 @@ import {
   wrapPowerSyncWithDrizzle,
   type DrizzleQuery,
   type PowerSyncSQLiteDatabase
-} from './sqlite/PowerSyncSQLiteDatabase';
-import { toCompilableQuery } from './utils/compilableQuery';
+} from './sqlite/PowerSyncSQLiteDatabase.js';
+import { toCompilableQuery } from './utils/compilableQuery.js';
 import {
   DrizzleAppSchema,
   toPowerSyncTable,
@@ -14,14 +14,14 @@ import {
   type ExtractPowerSyncColumns,
   type TableName,
   type TablesFromSchemaEntries
-} from './utils/schema';
+} from './utils/schema.js';
 
 export {
   DrizzleAppSchema,
   DrizzleAppSchemaOptions,
+  DrizzleQuery,
   DrizzleTablePowerSyncOptions,
   DrizzleTableWithPowerSyncOptions,
-  DrizzleQuery,
   Expand,
   ExtractPowerSyncColumns,
   PowerSyncSQLiteDatabase,