Split migrations into separte bundle

README.md

@@ -113,7 +113,8 @@ Pass the migration source to all migrate commands as follows:
 
 ```ts
 import {join} from 'node:path'
-import {KyselyFsMigrationSource, PGColdDialect} from 'kysely-knex'
+import {PGColdDialect} from 'kysely-knex'
+import {KyselyFsMigrationSource} from 'kysely-knex/migrations'
 import {knex} from '../src/knex'
 import {kysely} from '../src/kysely'
 
@@ -130,7 +131,8 @@ or pass it to Knex CLI globally:
 `knexfile.ts`:
 
 ```ts
-const {KyselyFsMigrationSource, PGColdDialect} = require('kysely-knex')
+const {PGColdDialect} = require('kysely-knex')
+const {KyselyFsMigrationSource} = require('kysely-knex/migrations')
 
 module.exports = {
   // ...

knexfile.ts

@@ -1,4 +1,5 @@
-const {KyselyFsMigrationSource, PGColdDialect} = require('./dist/cjs')
+const {PGColdDialect} = require('./dist')
+const {KyselyFsMigrationSource} = require('./dist/migrations')
 
 module.exports = {
   client: 'pg',

package.json

@@ -6,12 +6,16 @@
   "homepage": "https://github.com/kysely-org/kysely-knex",
   "author": "Igal Klebanov <[email protected]>",
   "license": "MIT",
-  "main": "./dist/cjs/index.js",
-  "module": "./dist/esm/index.js",
+  "main": "./dist/index.js",
+  "module": "./dist/index.mjs",
   "exports": {
     ".": {
-      "import": "./dist/esm/index.js",
-      "require": "./dist/cjs/index.js"
+      "import": "./dist/index.mjs",
+      "require": "./dist/index.js"
+    },
+    "./migrations": {
+      "import": "./dist/migrations.mjs",
+      "require": "./dist/migrations.js"
     }
   },
   "files": [

scripts/dist-fix.js

@@ -1,70 +1,26 @@
-const {
-  mkdir,
-  readdir,
-  rename,
-  rm,
-  writeFile,
-  copyFile,
-  readFile,
-  unlink,
-} = require('node:fs/promises')
+// @ts-check
+const {readdir, writeFile, readFile} = require('node:fs/promises')
 const path = require('node:path')
 
 ;(async () => {
   const distPath = path.join(__dirname, '../dist')
-  const distCjsPath = path.join(distPath, 'cjs')
-  const distEsmPath = path.join(distPath, 'esm')
-
-  const [dist, distEsm] = await Promise.all([
-    readdir(distPath),
-    readdir(distEsmPath),
-    rm(distCjsPath, {force: true, recursive: true}),
-  ])
+  const dist = await readdir(distPath)
 
+  // Inject type declarations for deno.
+  // See https://docs.deno.com/runtime/manual/advanced/typescript/types
   await Promise.all([
-    mkdir(distCjsPath),
-    writeFile(
-      path.join(distEsmPath, 'package.json'),
-      JSON.stringify({type: 'module', sideEffects: false}),
-    ),
     ...dist
-      .filter((distFilePath) => distFilePath.match(/\.d\.ts$/))
-      .map((distFilePath) =>
-        copyFile(
-          path.join(distPath, distFilePath),
-          path.join(distEsmPath, distFilePath),
-        ),
-      ),
-    ...distEsm
-      .filter((esmFilePath) => esmFilePath.match(/\.js$/))
-      .map(async (esmFilePath) => {
-        const distEsmFilePath = path.join(distEsmPath, esmFilePath)
-
-        const esmFile = await readFile(distEsmFilePath)
-        const esmFileContents = esmFile.toString()
-
-        const dtsFilePath = `./${esmFilePath.replace('.js', '.d.ts')}`
-
-        const denoFriendlyEsmFileContents = [
-          `/// <reference types="${dtsFilePath}" />`,
-          esmFileContents,
-        ].join('\n')
+      .filter((filePath) => filePath.match(/\.m?js$/))
+      .filter((filePath) => !filePath.startsWith('chunk-'))
+      .map(async (filename) => {
+        const distFilePath = path.join(distPath, filename)
 
-        await unlink(distEsmFilePath)
+        const dtsFilePath = filename.replace(/\.(m?)js$/, '.d.$1ts')
+        const denoFriendlyJsFileContents =
+          `/// <reference types="${dtsFilePath}" />\n` +
+          (await readFile(distFilePath)).toString()
 
-        await writeFile(distEsmFilePath, denoFriendlyEsmFileContents)
+        await writeFile(distFilePath, denoFriendlyJsFileContents)
       }),
   ])
-
-  await Promise.all([
-    writeFile(
-      path.join(distCjsPath, 'package.json'),
-      JSON.stringify({type: 'commonjs', sideEffects: false}),
-    ),
-    ...dist
-      .filter((filePath) => filePath.match(/\.[t|j]s(\.map)?$/))
-      .map((filePath) =>
-        rename(path.join(distPath, filePath), path.join(distCjsPath, filePath)),
-      ),
-  ])
 })()

src/index.ts

@@ -27,4 +27,3 @@ export * from './cold-dialect/pg/cold-dialect.js'
 export * from './cold-dialect/pg/query-compiler.js'
 export * from './cold-dialect/pg/results-parser.js'
 export * from './cold-dialect/results-parser.js'
-export * from './fs-migration-source.js'

src/fs-migration-source.ts → src/migrations/fs-migration-source.ts

@@ -2,8 +2,8 @@ import type {Knex} from 'knex'
 // @ts-ignore
 import {FsMigrations} from 'knex/lib/migrations/migrate/sources/fs-migrations.js'
 import {Kysely} from 'kysely'
-import type {ColdDialect} from './cold-dialect/cold-dialect.js'
-import {KyselyKnexDialect} from './dialect.js'
+import type {ColdDialect} from '../cold-dialect/cold-dialect.js'
+import {KyselyKnexDialect} from '../dialect.js'
 
 export interface KyselyFsMigrationSourceProps {
   kyselySubDialect: ColdDialect

src/migrations/index.ts

@@ -0,0 +1 @@
+export * from './fs-migration-source.js'

tests/nodejs/fs-migration-source.test.ts

@@ -1,5 +1,5 @@
 import {join} from 'node:path'
-import {KyselyFsMigrationSource} from '../..'
+import {KyselyFsMigrationSource} from '../../dist/migrations'
 import {
   CONFIGS,
   SUPPORTED_DIALECTS,
@@ -12,7 +12,7 @@ import {
 for (const dialect of SUPPORTED_DIALECTS) {
   describe(`KyselyFsMigrationSource: ${dialect}`, () => {
     let ctx: TestContext
-    let migrationSource: KyselyFsMigrationSource
+    let migrationSource: InstanceType<typeof KyselyFsMigrationSource>
 
     before(async function () {
       ctx = await initTest(this, dialect)

tsup.config.ts

@@ -3,9 +3,11 @@ import {defineConfig} from 'tsup'
 export default defineConfig({
   clean: true,
   dts: true,
-  entry: ['./src/index.ts'],
+  entry: {
+    index: 'src/index.ts',
+    migrations: 'src/migrations/index.ts',
+  },
   format: ['cjs', 'esm'],
-  legacyOutput: true,
   outDir: 'dist',
   sourcemap: true,
 })