@netlify/build 36.2.4 → 36.3.0

package/lib/plugins/child/diff.d.ts

@@ -1,2 +1,8 @@
-export function cloneNetlifyConfig(netlifyConfig: any): any;
-export function getConfigMutations(netlifyConfig: any, netlifyConfigCopy: any, event: any): any;
+export type ConfigMutation = {
+    keys: string[];
+    keysString: string;
+    value: unknown;
+    event: string;
+};
+export declare function cloneNetlifyConfig<T>(netlifyConfig: T): T;
+export declare function getConfigMutations(netlifyConfig: object, netlifyConfigCopy: object, event: string): ConfigMutation[];

package/lib/plugins/child/diff.js

@@ -3,9 +3,9 @@ import isPlainObj from 'is-plain-obj';
 import rfdc from 'rfdc';
 const clone = rfdc();
 // Copy `netlifyConfig` so we can compare before/after mutating it
-export const cloneNetlifyConfig = function (netlifyConfig) {
+export function cloneNetlifyConfig(netlifyConfig) {
     return clone(netlifyConfig);
-};
+}
 // Diff `netlifyConfig` before and after mutating it to retrieve an array of
 // `configMutations` objects.
 // We need to keep track of the changes on `netlifyConfig` so they can be
@@ -14,13 +14,13 @@ export const cloneNetlifyConfig = function (netlifyConfig) {
 //  - Apply the change to `netlifyConfig` in the parent process so it can
 //    run `@netlify/config` to normalize and validate the new values
 // `configMutations` is passed to parent process as JSON
-export const getConfigMutations = function (netlifyConfig, netlifyConfigCopy, event) {
+export function getConfigMutations(netlifyConfig, netlifyConfigCopy, event) {
     const configMutations = diffObjects(netlifyConfig, netlifyConfigCopy, []);
     return configMutations.map((configMutation) => getConfigMutation(configMutation, event));
-};
+}
 // We only recurse over plain objects, not arrays. Which means array properties
 // can only be modified all at once.
-const diffObjects = function (objA, objB, parentKeys) {
+function diffObjects(objA, objB, parentKeys) {
     const allKeys = [...new Set([...Object.keys(objA), ...Object.keys(objB)])];
     return allKeys.flatMap((key) => {
         const valueA = objA[key];
@@ -34,8 +34,8 @@ const diffObjects = function (objA, objB, parentKeys) {
         }
         return [{ keys, value: valueB }];
     });
-};
-const getConfigMutation = function ({ keys, value }, event) {
+}
+function getConfigMutation({ keys, value }, event) {
     const serializedKeys = keys.map(String);
     return {
         keys: serializedKeys,
@@ -43,4 +43,4 @@ const getConfigMutation = function ({ keys, value }, event) {
         value,
         event,
     };
-};
+}

package/lib/plugins/child/run.d.ts

@@ -15,7 +15,7 @@ export function run({ event, error, constants, envChanges, featureFlags, netlify
 }): Promise<{
     deployEnvVars: any[];
     newEnvChanges: any;
-    configMutations: any;
+    configMutations: import("./diff.js").ConfigMutation[];
     returnValue: {
         generatedFunctions: any[];
     } | undefined;

package/lib/plugins_core/frameworks_api/index.js

@@ -17,6 +17,7 @@ const ALLOWED_PROPERTIES = [
     ['headers'],
     ['images', 'remote_images'],
     ['redirects'],
+    ['spa_fallback'],
 ];
 // For array properties, any values set in this API will be merged with the
 // main configuration file in such a way that user-defined values always take

package/lib/plugins_core/spa_fallback/index.d.ts

@@ -0,0 +1,2 @@
+import { CoreStep } from '../types.js';
+export declare const applySpaFallback: CoreStep;

package/lib/plugins_core/spa_fallback/index.js

@@ -0,0 +1,51 @@
+import { logWarning } from '../../log/logger.js';
+import { getConfigMutations } from '../../plugins/child/diff.js';
+// The catch-all redirect that makes a single-page application's client-side
+// router handle every path.
+const SPA_FALLBACK_REDIRECT = {
+    from: '/*',
+    status: 200,
+    to: '/index.html',
+};
+function findCatchAllRedirect(redirects) {
+    return redirects.find((r) => r.from === '/*');
+}
+// A catch-all redirect only serves the single-page application the way ours
+// would if it rewrites (rather than redirects) to the same destination.
+function matchesSpaFallback(redirect) {
+    return redirect.to === SPA_FALLBACK_REDIRECT.to && redirect.status === SPA_FALLBACK_REDIRECT.status;
+}
+function coreStep(coreStepFunctionArgs) {
+    const { netlifyConfig, logs } = coreStepFunctionArgs;
+    if (!netlifyConfig.spa_fallback) {
+        return Promise.resolve({});
+    }
+    const existingCatchAll = findCatchAllRedirect(netlifyConfig.redirects);
+    if (existingCatchAll) {
+        // `spa_fallback` asked us to add a catch-all redirect, but the site
+        // already has one. If it doesn't already do what ours would, warn the
+        // user instead of silently overriding their explicit configuration.
+        if (!matchesSpaFallback(existingCatchAll)) {
+            logWarning(logs, `
+Warning: "spa_fallback" is enabled, but a catch-all redirect ("/*") already exists that does not rewrite to "${SPA_FALLBACK_REDIRECT.to}" with a "${String(SPA_FALLBACK_REDIRECT.status)}" status, so Netlify did not add its own.
+Please make sure your existing catch-all redirect correctly serves your single-page application.`);
+        }
+        return Promise.resolve({});
+    }
+    const newConfig = {
+        redirects: [...netlifyConfig.redirects, SPA_FALLBACK_REDIRECT],
+    };
+    const configMutations = getConfigMutations(netlifyConfig, {
+        ...netlifyConfig,
+        ...newConfig,
+    }, applySpaFallback.event);
+    return Promise.resolve({ configMutations });
+}
+export const applySpaFallback = {
+    coreStep,
+    coreStepDescription: () => '',
+    coreStepId: 'spa_fallback',
+    coreStepName: 'Applying SPA fallback redirect',
+    event: 'onPostBuild',
+    quiet: true,
+};

package/lib/steps/get.js

@@ -13,6 +13,7 @@ import { preCleanup } from '../plugins_core/pre_cleanup/index.js';
 import { preDevCleanup } from '../plugins_core/pre_dev_cleanup/index.js';
 import { saveArtifacts } from '../plugins_core/save_artifacts/index.js';
 import { scanForSecrets } from '../plugins_core/secrets_scanning/index.js';
+import { applySpaFallback } from '../plugins_core/spa_fallback/index.js';
 // Get all build steps
 export const getSteps = function (steps, eventHandlers) {
     const stepsA = addCoreSteps(steps);
@@ -72,6 +73,7 @@ const addCoreSteps = function (steps) {
         bundleFunctions,
         bundleEdgeFunctions,
         copyDbMigrations,
+        applySpaFallback,
         scanForSecrets,
         uploadBlobs,
         deploySite,

package/lib/types/config/netlify_config.d.ts

@@ -61,5 +61,9 @@ export interface NetlifyConfig {
      * object with options for database configuration
      */
     database?: DatabaseConfig;
+    /**
+     * Whether the site is a single-page application (SPA). Defaults to `false`.
+     */
+    spa_fallback?: boolean;
 }
 export {};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@netlify/build",
-  "version": "36.2.4",
+  "version": "36.3.0",
   "description": "Netlify build module",
   "type": "module",
   "exports": "./lib/index.js",
@@ -24,7 +24,7 @@
   "scripts": {
     "prebuild": "rm -rf lib",
     "postbuild": "npx cpy \"src/**/*.yml\" \"lib/\"",
-    "build": "tsc",
+    "build": "tsc --project tsconfig.build.json",
     "test:types": "tsd",
     "test": "ava && tsd && vitest run",
     "test:dev": "ava -w",
@@ -69,14 +69,14 @@
     "@bugsnag/js": "^8.0.0",
     "@netlify/blobs": "^10.4.4",
     "@netlify/cache-utils": "^7.1.1",
-    "@netlify/config": "^25.1.1",
+    "@netlify/config": "^25.2.0",
     "@netlify/edge-bundler": "16.0.1",
-    "@netlify/functions-utils": "^7.1.1",
+    "@netlify/functions-utils": "^7.1.2",
     "@netlify/git-utils": "^7.1.0",
     "@netlify/opentelemetry-utils": "^3.1.0",
     "@netlify/plugins-list": "^6.81.6",
     "@netlify/run-utils": "^7.1.0",
-    "@netlify/zip-it-and-ship-it": "15.3.1",
+    "@netlify/zip-it-and-ship-it": "15.3.2",
     "@sindresorhus/slugify": "^2.0.0",
     "ansi-escapes": "^7.0.0",
     "ansis": "^4.1.0",
@@ -152,5 +152,5 @@
   "engines": {
     "node": ">=22.12.0"
   },
-  "gitHead": "20f14cb73414fc393175094e88278bea96b15f71"
+  "gitHead": "f34dd81fc3bc8925aabe5cf9c1cdce4e8eebc4e0"
 }

package/lib/plugins/compatibility.test.d.ts

@@ -1 +0,0 @@
-export {};

package/lib/plugins/compatibility.test.js

@@ -1,298 +0,0 @@
-import { describe, expect, test } from 'vitest';
-import { getExpectedVersion } from './compatibility.js';
-const noopSystemLog = () => {
-    // no-op
-};
-describe(`getExpectedVersion`, () => {
-    test('should ignore the new major version if the version is pinned', async () => {
-        const versions = [
-            { version: '5.0.0', conditions: [] },
-            { version: '4.41.2', conditions: [] },
-        ];
-        const { version } = await getExpectedVersion({
-            versions,
-            nodeVersion: '18.19.0',
-            packageJson: {},
-            packageName: '@netlify/cool-plugin',
-            buildDir: '/some/path',
-            pinnedVersion: '4',
-            systemLog: noopSystemLog,
-        });
-        expect(version).toBe('4.41.2');
-    });
-    test('matches prerelease versions', async () => {
-        const versions = [
-            { version: '5.0.0', conditions: [] },
-            { version: '4.42.0-alpha.1', conditions: [] },
-            { version: '4.41.2', conditions: [] },
-        ];
-        const { version: version1 } = await getExpectedVersion({
-            versions,
-            nodeVersion: '18.19.0',
-            packageJson: {},
-            packageName: '@netlify/cool-plugin',
-            buildDir: '/some/path',
-            systemLog: noopSystemLog,
-        });
-        const { version: version2 } = await getExpectedVersion({
-            versions,
-            nodeVersion: '18.19.0',
-            packageJson: {},
-            packageName: '@netlify/cool-plugin',
-            buildDir: '/some/path',
-            pinnedVersion: '4',
-            systemLog: noopSystemLog,
-        });
-        expect(version1).toBe('5.0.0');
-        expect(version2).toBe('4.42.0-alpha.1');
-    });
-    test('should retrieve a new major version if the overridePinnedVersion is specified', async () => {
-        const versions = [
-            { version: '5.0.0', conditions: [], overridePinnedVersion: '>=4.0.0' },
-            { version: '4.41.2', conditions: [] },
-        ];
-        const { version } = await getExpectedVersion({
-            versions,
-            nodeVersion: '18.19.0',
-            packageJson: {},
-            packageName: '@netlify/cool-plugin',
-            buildDir: '/some/path',
-            pinnedVersion: '4',
-            systemLog: noopSystemLog,
-        });
-        expect(version).toBe('5.0.0');
-    });
-    test('should retrieve the plugin based on the condition of a nodeVersion', async () => {
-        const versions = [
-            {
-                version: '4.42.0',
-                conditions: [{ type: 'nodeVersion', condition: '>=18.0.0' }],
-            },
-            { version: '4.41.2', conditions: [] },
-        ];
-        const { version } = await getExpectedVersion({
-            versions,
-            nodeVersion: '17.19.0',
-            packageJson: {},
-            packageName: '@netlify/cool-plugin',
-            buildDir: '/some/path',
-            pinnedVersion: '4',
-            systemLog: noopSystemLog,
-        });
-        expect(version).toBe('4.41.2');
-    });
-    test('should retrieve the plugin based on conditions and feature flag due to pinned version', async () => {
-        const versions = [
-            {
-                version: '5.0.0-beta.1',
-                conditions: [
-                    { type: 'nodeVersion', condition: '>= 18.0.0' },
-                    { type: 'siteDependencies', condition: { next: '>=13.5.0' } },
-                ],
-                overridePinnedVersion: '>=4.0.0',
-            },
-            {
-                version: '4.42.0',
-                conditions: [{ type: 'siteDependencies', condition: { next: '>=10.0.9' } }],
-            },
-            { version: '4.41.2', conditions: [] },
-            {
-                version: '3.9.2',
-                conditions: [{ type: 'siteDependencies', condition: { next: '<10.0.9' } }],
-            },
-        ];
-        const { version: version1 } = await getExpectedVersion({
-            versions,
-            nodeVersion: '17.19.0',
-            packageJson: { dependencies: { next: '10.0.8' } },
-            packageName: '@netlify/cool-plugin',
-            buildDir: '/some/path',
-            pinnedVersion: '3',
-            systemLog: noopSystemLog,
-        });
-        expect(version1).toBe('3.9.2');
-        const { version: version2 } = await getExpectedVersion({
-            versions,
-            nodeVersion: '17.19.0',
-            packageJson: { dependencies: { next: '11.0.0' } },
-            packageName: '@netlify/cool-plugin',
-            buildDir: '/some/path',
-            pinnedVersion: '4',
-            systemLog: noopSystemLog,
-        });
-        expect(version2).toBe('4.42.0');
-        const { version: version3 } = await getExpectedVersion({
-            versions,
-            nodeVersion: '18.19.0',
-            packageJson: { dependencies: { next: '13.5.0' } },
-            packageName: '@netlify/cool-plugin',
-            buildDir: '/some/path',
-            pinnedVersion: '4',
-            systemLog: noopSystemLog,
-        });
-        expect(version3).toBe('5.0.0-beta.1');
-    });
-    test('should work with rc versions inside the siteDependencies constraints', async () => {
-        const versions = [
-            {
-                version: '5.0.0-beta.1',
-                conditions: [
-                    { type: 'nodeVersion', condition: '>= 18.0.0' },
-                    { type: 'siteDependencies', condition: { next: '>=13.5.0' } },
-                ],
-                overridePinnedVersion: '>=4.0.0',
-            },
-            {
-                version: '4.42.0',
-                conditions: [{ type: 'siteDependencies', condition: { next: '>=10.0.9' } }],
-            },
-            { version: '4.41.2', conditions: [] },
-            {
-                version: '3.9.2',
-                conditions: [{ type: 'siteDependencies', condition: { next: '<10.0.9' } }],
-            },
-        ];
-        const { version } = await getExpectedVersion({
-            versions,
-            nodeVersion: '18.19.0',
-            packageJson: { dependencies: { next: '14.1.1-canary.36' } },
-            packageName: '@netlify/cool-plugin',
-            buildDir: '/some/path',
-            pinnedVersion: '4',
-            systemLog: noopSystemLog,
-        });
-        expect(version).toBe('5.0.0-beta.1');
-    });
-    test('should retrieve the plugin based on conditions and feature flag due to pinned version', async () => {
-        const versions = [
-            {
-                version: '5.0.0-beta.1',
-                conditions: [
-                    { type: 'nodeVersion', condition: '>= 18.0.0' },
-                    { type: 'siteDependencies', condition: { next: '>=13.5.0' } },
-                ],
-                overridePinnedVersion: '>=4.0.0',
-            },
-            { version: '4.41.2', conditions: [] },
-            {
-                version: '3.9.2',
-                conditions: [{ type: 'siteDependencies', condition: { next: '<10.0.9' } }],
-            },
-        ];
-        const { version: version1 } = await getExpectedVersion({
-            versions,
-            nodeVersion: '20.0.0',
-            packageJson: { dependencies: { next: '14.0.0' } },
-            packageName: '@netlify/cool-plugin',
-            buildDir: '/some/path',
-            pinnedVersion: '4',
-            systemLog: noopSystemLog,
-        });
-        expect(version1).toBe('5.0.0-beta.1');
-        // out of range
-        const { version: version2 } = await getExpectedVersion({
-            versions,
-            nodeVersion: '20.0.0',
-            packageJson: { dependencies: { next: '13.0.0' } },
-            packageName: '@netlify/cool-plugin',
-            buildDir: '/some/path',
-            pinnedVersion: '4',
-            systemLog: noopSystemLog,
-        });
-        expect(version2).toBe('4.41.2');
-    });
-    test('matches the first entry that satisfies the constraints, even if it also matches another entry further down with more specific constraints', async () => {
-        const versions = [
-            { version: '4.41.2', conditions: [] },
-            {
-                version: '5.0.0-beta.1',
-                conditions: [
-                    { type: 'nodeVersion', condition: '>= 18.0.0' },
-                    { type: 'siteDependencies', condition: { next: '>=13.5.0' } },
-                ],
-                overridePinnedVersion: '>=4.0.0',
-            },
-            {
-                version: '3.9.2',
-                conditions: [{ type: 'siteDependencies', condition: { next: '<10.0.9' } }],
-            },
-        ];
-        const { version } = await getExpectedVersion({
-            versions,
-            nodeVersion: '20.0.0',
-            packageJson: { dependencies: { next: '14.0.0' } },
-            packageName: '@netlify/cool-plugin',
-            buildDir: '/some/path',
-            pinnedVersion: '4',
-            systemLog: noopSystemLog,
-        });
-        expect(version).toBe('4.41.2');
-    });
-    test('if no pinned version is set, it matches the first version regardless of whether its requirements match the conditions (legacy behavior)', async () => {
-        const versions = [
-            {
-                version: '5.0.0-beta.1',
-                conditions: [
-                    { type: 'nodeVersion', condition: '>= 18.0.0' },
-                    { type: 'siteDependencies', condition: { next: '>=13.5.0' } },
-                ],
-                overridePinnedVersion: '>=4.0.0',
-            },
-            { version: '4.41.2', conditions: [] },
-            {
-                version: '3.9.2',
-                conditions: [{ type: 'siteDependencies', condition: { next: '<10.0.9' } }],
-            },
-        ];
-        const logMessages = [];
-        const { version } = await getExpectedVersion({
-            versions,
-            nodeVersion: '20.0.0',
-            packageJson: { dependencies: { next: '12.0.0' } },
-            packageName: '@netlify/cool-plugin',
-            buildDir: '/some/path',
-            systemLog: (message) => {
-                logMessages.push(message);
-            },
-            authoritative: true,
-        });
-        expect(logMessages.length).toBe(1);
-        expect(logMessages[0]).toBe(`Detected mismatch in selected version for plugin '@netlify/cool-plugin': used legacy version '5.0.0-beta.1' over new version '4.41.2'`);
-        expect(version).toBe('5.0.0-beta.1');
-    });
-    test('if no pinned version is set, it matches the first version whose requirements match the conditions', async () => {
-        const versions = [
-            {
-                version: '5.0.0-beta.1',
-                conditions: [
-                    { type: 'nodeVersion', condition: '>= 18.0.0' },
-                    { type: 'siteDependencies', condition: { next: '>=13.5.0' } },
-                ],
-                overridePinnedVersion: '>=4.0.0',
-            },
-            { version: '4.41.2', conditions: [] },
-            {
-                version: '3.9.2',
-                conditions: [{ type: 'siteDependencies', condition: { next: '<10.0.9' } }],
-            },
-        ];
-        const logMessages = [];
-        const { version } = await getExpectedVersion({
-            versions,
-            nodeVersion: '20.0.0',
-            packageJson: { dependencies: { next: '12.0.0' } },
-            packageName: '@netlify/cool-plugin',
-            buildDir: '/some/path',
-            systemLog: (message) => {
-                logMessages.push(message);
-            },
-            featureFlags: {
-                netlify_build_updated_plugin_compatibility: true,
-            },
-            authoritative: true,
-        });
-        expect(logMessages.length).toBe(1);
-        expect(logMessages[0]).toBe(`Detected mismatch in selected version for plugin '@netlify/cool-plugin': used new version of '4.41.2' over legacy version '5.0.0-beta.1'`);
-        expect(version).toBe('4.41.2');
-    });
-});

package/lib/plugins_core/db_setup/validation.test.d.ts

@@ -1 +0,0 @@
-export {};

package/lib/plugins_core/db_setup/validation.test.js

@@ -1,226 +0,0 @@
-import { describe, expect, test } from 'vitest';
-import { MIGRATION_DIR_PATTERN, MIGRATION_FILE_PATTERN, trackMigrationNumber, validateMigrations, formatValidationErrors, } from './validation.js';
-describe('MIGRATION_DIR_PATTERN', () => {
-    const validNames = [
-        '1700000000_create-users',
-        '1700000001_add-posts',
-        '0000000000_init',
-        '9999999999_z',
-        '1700000000_a',
-        '1700000000_abc-def-123',
-        '001_create-users',
-        '1_init',
-        '0001_add-posts',
-        '42_z',
-        '1700000000_under_score',
-        '001_create_users_table',
-        '1_a_b_c',
-    ];
-    test.each(validNames)('matches valid name: %s', (name) => {
-        expect(MIGRATION_DIR_PATTERN.test(name)).toBe(true);
-    });
-    const invalidNames = [
-        { name: '1700000000_CAPS', reason: 'uppercase letters' },
-        { name: 'no-timestamp', reason: 'no numeric prefix' },
-        { name: '1700000000_', reason: 'empty slug' },
-        { name: '1700000000', reason: 'missing underscore and slug' },
-        { name: '_create-users', reason: 'missing number prefix' },
-        { name: '1700000000_hello world', reason: 'spaces in slug' },
-        { name: '1700000000_special!char', reason: 'special characters in slug' },
-    ];
-    test.each(invalidNames)('rejects invalid name: $name ($reason)', ({ name }) => {
-        expect(MIGRATION_DIR_PATTERN.test(name)).toBe(false);
-    });
-});
-describe('MIGRATION_FILE_PATTERN', () => {
-    const validNames = [
-        '1700000000_create-users.sql',
-        '001_create_users_table.sql',
-        '1_init.sql',
-        '42_a-b-c.sql',
-        '1700000000_under_score.sql',
-    ];
-    test.each(validNames)('matches valid name: %s', (name) => {
-        expect(MIGRATION_FILE_PATTERN.test(name)).toBe(true);
-    });
-    const invalidNames = [
-        { name: '1700000000_CAPS.sql', reason: 'uppercase letters' },
-        { name: 'no-timestamp.sql', reason: 'no numeric prefix' },
-        { name: '1700000000_.sql', reason: 'empty slug' },
-        { name: '1700000000_create-users.txt', reason: 'wrong extension' },
-        { name: '1700000000_create-users', reason: 'no extension' },
-        { name: '1700000000_hello world.sql', reason: 'spaces in slug' },
-    ];
-    test.each(invalidNames)('rejects invalid name: $name ($reason)', ({ name }) => {
-        expect(MIGRATION_FILE_PATTERN.test(name)).toBe(false);
-    });
-});
-describe('trackMigrationNumber', () => {
-    const cases = [
-        { name: '001_create-users', expected: '1' },
-        { name: '1_init', expected: '1' },
-        { name: '0001_add-posts', expected: '1' },
-        { name: '42_z', expected: '42' },
-        { name: '1700000000_create-users', expected: '1700000000' },
-        { name: '1700000000_create-users.sql', expected: '1700000000' },
-        { name: '001_create-users.sql', expected: '1' },
-        { name: '0000000000_init', expected: '0' },
-    ];
-    test.each(cases)('tracks $name under key $expected', ({ name, expected }) => {
-        const map = new Map();
-        trackMigrationNumber(map, name);
-        expect(map.get(expected)).toEqual([name]);
-    });
-    test('groups names with the same migration number', () => {
-        const map = new Map();
-        trackMigrationNumber(map, '001_create-users');
-        trackMigrationNumber(map, '1_init');
-        expect(map.get('1')).toEqual(['001_create-users', '1_init']);
-    });
-});
-describe('validateMigrations', () => {
-    test('returns valid result with sorted dirs when all dirs are valid', () => {
-        const dirNames = ['1700000001_add-posts', '1700000000_create-users'];
-        const existingSqlFiles = new Set(['1700000000_create-users', '1700000001_add-posts']);
-        const result = validateMigrations(dirNames, [], existingSqlFiles);
-        expect(result).toEqual({
-            valid: true,
-            dirs: ['1700000000_create-users', '1700000001_add-posts'],
-            files: [],
-        });
-    });
-    test('silently skips non-matching directory names', () => {
-        const dirNames = ['bad-name', '1700000000_create-users'];
-        const existingSqlFiles = new Set(['1700000000_create-users']);
-        const result = validateMigrations(dirNames, [], existingSqlFiles);
-        expect(result).toEqual({
-            valid: true,
-            dirs: ['1700000000_create-users'],
-            files: [],
-        });
-    });
-    test('returns error for missing migration.sql files', () => {
-        const dirNames = ['1700000000_create-users', '1700000001_add-posts'];
-        const existingSqlFiles = new Set(['1700000000_create-users']);
-        const result = validateMigrations(dirNames, [], existingSqlFiles);
-        expect(result).toEqual({
-            valid: false,
-            errors: [{ type: 'missing_sql_file', dirName: '1700000001_add-posts' }],
-        });
-    });
-    test('returns valid result with sorted files for loose .sql files', () => {
-        const fileNames = ['002_add-posts.sql', '001_create-users.sql'];
-        const result = validateMigrations([], fileNames, new Set());
-        expect(result).toEqual({
-            valid: true,
-            dirs: [],
-            files: ['001_create-users.sql', '002_add-posts.sql'],
-        });
-    });
-    test('silently skips non-matching file names', () => {
-        const fileNames = ['README.sql', '001_create-users.sql', 'bad-name.sql'];
-        const result = validateMigrations([], fileNames, new Set());
-        expect(result).toEqual({
-            valid: true,
-            dirs: [],
-            files: ['001_create-users.sql'],
-        });
-    });
-    test('handles mixed dirs and files', () => {
-        const dirNames = ['1700000000_create-users'];
-        const fileNames = ['1700000001_add-posts.sql'];
-        const existingSqlFiles = new Set(['1700000000_create-users']);
-        const result = validateMigrations(dirNames, fileNames, existingSqlFiles);
-        expect(result).toEqual({
-            valid: true,
-            dirs: ['1700000000_create-users'],
-            files: ['1700000001_add-posts.sql'],
-        });
-    });
-    test('returns valid result for empty input', () => {
-        const result = validateMigrations([], [], new Set());
-        expect(result).toEqual({
-            valid: true,
-            dirs: [],
-            files: [],
-        });
-    });
-    test('allows underscores in dir names', () => {
-        const dirNames = ['001_create_users_table'];
-        const existingSqlFiles = new Set(['001_create_users_table']);
-        const result = validateMigrations(dirNames, [], existingSqlFiles);
-        expect(result).toEqual({
-            valid: true,
-            dirs: ['001_create_users_table'],
-            files: [],
-        });
-    });
-    test('returns error for duplicate migration numbers in dirs', () => {
-        const dirNames = ['001_create-users', '001_add-posts'];
-        const existingSqlFiles = new Set(['001_create-users', '001_add-posts']);
-        const result = validateMigrations(dirNames, [], existingSqlFiles);
-        expect(result).toEqual({
-            valid: false,
-            errors: [
-                { type: 'duplicate_migration_number', migrationNumber: '1', names: ['001_add-posts', '001_create-users'] },
-            ],
-        });
-    });
-    test('returns error for duplicate migration numbers in files', () => {
-        const fileNames = ['001_create-users.sql', '001_add-posts.sql'];
-        const result = validateMigrations([], fileNames, new Set());
-        expect(result).toEqual({
-            valid: false,
-            errors: [
-                {
-                    type: 'duplicate_migration_number',
-                    migrationNumber: '1',
-                    names: ['001_add-posts.sql', '001_create-users.sql'],
-                },
-            ],
-        });
-    });
-    test('returns error for duplicate migration numbers across dirs and files', () => {
-        const dirNames = ['001_create-users'];
-        const fileNames = ['001_add-posts.sql'];
-        const existingSqlFiles = new Set(['001_create-users']);
-        const result = validateMigrations(dirNames, fileNames, existingSqlFiles);
-        expect(result).toEqual({
-            valid: false,
-            errors: [
-                { type: 'duplicate_migration_number', migrationNumber: '1', names: ['001_add-posts.sql', '001_create-users'] },
-            ],
-        });
-    });
-    test('treats different zero-padded prefixes as duplicates', () => {
-        const dirNames = ['1_init', '001_setup'];
-        const existingSqlFiles = new Set(['1_init', '001_setup']);
-        const result = validateMigrations(dirNames, [], existingSqlFiles);
-        expect(result).toEqual({
-            valid: false,
-            errors: [{ type: 'duplicate_migration_number', migrationNumber: '1', names: ['001_setup', '1_init'] }],
-        });
-    });
-});
-describe('formatValidationErrors', () => {
-    test('formats missing_sql_file errors', () => {
-        const message = formatValidationErrors([{ type: 'missing_sql_file', dirName: '1700000000_create-users' }]);
-        expect(message).toContain('Database migration validation failed');
-        expect(message).toContain('"1700000000_create-users/migration.sql" is missing');
-    });
-    test('formats multiple errors', () => {
-        const message = formatValidationErrors([
-            { type: 'missing_sql_file', dirName: '1700000000_create-users' },
-            { type: 'missing_sql_file', dirName: '1700000001_add-posts' },
-        ]);
-        expect(message).toContain('"1700000000_create-users/migration.sql" is missing');
-        expect(message).toContain('"1700000001_add-posts/migration.sql" is missing');
-    });
-    test('formats duplicate_migration_number errors', () => {
-        const message = formatValidationErrors([
-            { type: 'duplicate_migration_number', migrationNumber: '1', names: ['001_add-posts', '001_create-users'] },
-        ]);
-        expect(message).toContain('Database migration validation failed');
-        expect(message).toContain('Duplicate migration number 1: "001_add-posts", "001_create-users"');
-    });
-});