@mintlify/previewing 4.0.1554 → 4.0.1555

package/dist/__tests__/{validate-build.test.js → check-build.test.js}

@@ -1,11 +1,10 @@
-import { getWarnings, prebuild } from '@mintlify/prebuild';
+import { prebuild, StrictModeError } from '@mintlify/prebuild';
+import { checkBuild } from '../local-preview/check-build.js';
 import { isOnline } from '../local-preview/is-online.js';
-import validateBuild, { checkBuild } from '../local-preview/validate-build.js';
 import { addLog } from '../logging-state.js';
-import { WarningLog } from '../logs.js';
 vi.mock('@mintlify/prebuild', async (importOriginal) => {
     const mod = await importOriginal();
-    return { ...mod, prebuild: vi.fn(), getWarnings: vi.fn(() => []) };
+    return { ...mod, prebuild: vi.fn() };
 });
 vi.mock('../local-preview/is-online.js', () => ({ isOnline: vi.fn() }));
 vi.mock('fs-extra', () => {
@@ -32,11 +31,12 @@ describe('checkBuild', () => {
         vi.spyOn(process, 'chdir').mockImplementation(() => undefined);
     });
     it('logs nothing', async () => {
-        vi.mocked(prebuild).mockRejectedValue(new Error('Build validation failed with 1 warning(s). See above for details.'));
+        vi.mocked(prebuild).mockRejectedValue(new StrictModeError('Build validation failed with 1 warning(s). See above for details.'));
         const result = await checkBuild(argv);
         expect(result).toMatchObject({
             passed: false,
             error: expect.stringContaining('Build validation failed'),
+            strict: true,
         });
         expect(addLog).not.toHaveBeenCalled();
     });
@@ -46,7 +46,12 @@ describe('checkBuild', () => {
     });
     it('returns a build failure instead of exiting', async () => {
         vi.mocked(prebuild).mockRejectedValue(new Error('boom'));
-        expect(await checkBuild(argv)).toEqual({ passed: false, error: 'boom', warnings: [] });
+        expect(await checkBuild(argv)).toEqual({
+            passed: false,
+            error: 'boom',
+            strict: false,
+            warnings: [],
+        });
     });
     it.each([
         ['passes', () => vi.mocked(prebuild).mockResolvedValue(undefined)],
@@ -66,19 +71,3 @@ describe('checkBuild', () => {
         expect(prebuild).not.toHaveBeenCalled();
     });
 });
-describe('validateBuild', () => {
-    beforeEach(() => {
-        vi.clearAllMocks();
-        vi.mocked(isOnline).mockResolvedValue(true);
-        vi.spyOn(process, 'chdir').mockImplementation(() => undefined);
-        vi.spyOn(process, 'exit').mockImplementation(() => undefined);
-    });
-    it('prints the collected warnings before the failure', async () => {
-        vi.mocked(prebuild).mockRejectedValue(new Error('Build validation failed with 1 warning(s).'));
-        vi.mocked(getWarnings).mockReturnValue([{ type: 'missing-file', message: 'missing a.mdx' }]);
-        await validateBuild(argv);
-        const logged = vi.mocked(addLog).mock.calls.map(([node]) => node);
-        expect(logged).toContainEqual(expect.objectContaining({ type: WarningLog, props: { message: 'missing a.mdx' } }));
-        expect(process.exit).toHaveBeenCalledWith(1);
-    });
-});

package/dist/index.d.ts

@@ -1,11 +1,11 @@
 import { LOCAL_LINKED_CLI_VERSION } from './constants.js';
+import { checkBuild } from './local-preview/check-build.js';
 import { getLatestClientVersion, downloadTargetMint, getVersionMap } from './local-preview/client.js';
 import { exportSite } from './local-preview/export.js';
 import dev from './local-preview/index.js';
-import validateBuild, { checkBuild } from './local-preview/validate-build.js';
-export type { BuildValidationResult } from './local-preview/validate-build.js';
+export type { BuildValidationResult } from './local-preview/check-build.js';
 import { getClientVersion } from './util.js';
 export * from './logs.js';
 export * from './logging-state.js';
 export * from './packages/index.js';
-export { dev, exportSite, validateBuild, checkBuild, getClientVersion, getLatestClientVersion, downloadTargetMint, getVersionMap, LOCAL_LINKED_CLI_VERSION, };
+export { dev, exportSite, checkBuild, getClientVersion, getLatestClientVersion, downloadTargetMint, getVersionMap, LOCAL_LINKED_CLI_VERSION, };

package/dist/index.js

@@ -1,10 +1,10 @@
 import { LOCAL_LINKED_CLI_VERSION } from './constants.js';
+import { checkBuild } from './local-preview/check-build.js';
 import { getLatestClientVersion, downloadTargetMint, getVersionMap, } from './local-preview/client.js';
 import { exportSite } from './local-preview/export.js';
 import dev from './local-preview/index.js';
-import validateBuild, { checkBuild } from './local-preview/validate-build.js';
 import { getClientVersion } from './util.js';
 export * from './logs.js';
 export * from './logging-state.js';
 export * from './packages/index.js';
-export { dev, exportSite, validateBuild, checkBuild, getClientVersion, getLatestClientVersion, downloadTargetMint, getVersionMap, LOCAL_LINKED_CLI_VERSION, };
+export { dev, exportSite, checkBuild, getClientVersion, getLatestClientVersion, downloadTargetMint, getVersionMap, LOCAL_LINKED_CLI_VERSION, };

package/dist/local-preview/{validate-build.d.ts → check-build.d.ts}

@@ -5,6 +5,7 @@ export type BuildValidationResult = ({
 } | {
     passed: false;
     error: string;
+    strict: boolean;
 }) & {
     warnings: Warning[];
 };
@@ -13,5 +14,3 @@ export type BuildValidationResult = ({
  * instead of printed.
  */
 export declare const checkBuild: (argv: ArgumentsCamelCase) => Promise<BuildValidationResult>;
-declare const validateBuild: (argv: ArgumentsCamelCase, onSuccess?: () => Promise<void>) => Promise<never>;
-export default validateBuild;

package/dist/local-preview/{validate-build.js → check-build.js}

@@ -1,9 +1,6 @@
-import { jsx as _jsx } from "react/jsx-runtime";
-import { getWarnings, prebuild, setWarningsSilent } from '@mintlify/prebuild';
+import { getWarnings, prebuild, setWarningsSilent, StrictModeError, } from '@mintlify/prebuild';
 import fse, { pathExists } from 'fs-extra';
 import { CLIENT_PATH, CMD_EXEC_PATH, VERSION_PATH, NEXT_PUBLIC_PATH, NEXT_PROPS_PATH, PREVIEW_PATH, configureSharedPreviewPath, } from '../constants.js';
-import { addLog, clearLogs } from '../logging-state.js';
-import { ErrorLog, SpinnerLog, SuccessLog, WarningLog } from '../logs.js';
 import { timeDev, timeDevSync } from './dev-timing.js';
 import { isOnline } from './is-online.js';
 import { silentUpdateClient } from './update.js';
@@ -59,36 +56,15 @@ export const checkBuild = async (argv) => {
     }
     catch (err) {
         const errorText = err instanceof Error && err.message ? err.message : 'build validation failed';
-        return { passed: false, error: errorText, warnings: getWarnings() };
+        return {
+            passed: false,
+            error: errorText,
+            strict: err instanceof StrictModeError,
+            warnings: getWarnings(),
+        };
     }
     finally {
         setWarningsSilent(false);
         process.chdir(cwd);
     }
 };
-const validateBuild = async (argv, onSuccess) => {
-    addLog(_jsx(SpinnerLog, { message: "validating build..." }));
-    let code;
-    try {
-        const result = await checkBuild(argv);
-        clearLogs();
-        result.warnings.forEach(({ message }) => addLog(_jsx(WarningLog, { message: message })));
-        if (result.passed) {
-            addLog(_jsx(SuccessLog, { message: "build validation passed" }));
-            await onSuccess?.();
-            code = 0;
-        }
-        else {
-            addLog(_jsx(ErrorLog, { message: result.error }));
-            code = 1;
-        }
-    }
-    catch (err) {
-        clearLogs();
-        addLog(_jsx(ErrorLog, { message: err instanceof Error ? err.message : String(err) }));
-        code = 2;
-    }
-    await new Promise((resolve) => setTimeout(resolve, 50));
-    process.exit(code);
-};
-export default validateBuild;