@servicetitan/startup 36.4.0 → 37.0.0

package/src/core/moment-locales.ts

@@ -0,0 +1,6 @@
+/*
+ * The moment locales the build supports. Under webpack, MomentLocalesPlugin narrows moment's
+ * built-in locales down to this set; under Vite, Rolldown bundles no locales, so the Vite plugin
+ * statically imports these to add them back. Shared so both bundlers agree on the set.
+ */
+export const MOMENT_LOCALES = ['en-au', 'en-ca', 'en-gb'];

package/src/core/resolve-output-path.ts

@@ -0,0 +1,10 @@
+import path from 'path';
+import { getBundleType } from './get-bundle-type';
+import { resolveOutputRoot } from './resolve-output-root';
+import { BuildContext } from './types';
+
+export function resolveOutputPath(context: BuildContext, outputPath: string | undefined): string {
+    const root = resolveOutputRoot(context, outputPath);
+    const subdir = getBundleType(context);
+    return subdir ? path.join(root, subdir) : root;
+}

package/src/core/resolve-output-root.ts

@@ -0,0 +1,8 @@
+import path from 'path';
+import { BuildContext } from './types';
+
+export function resolveOutputRoot(context: BuildContext, outputPath: string | undefined): string {
+    const { destination, isWebComponent } = context.package;
+    const defaultOutputPath = path.join(process.cwd(), destination, 'bundle');
+    return isWebComponent ? defaultOutputPath : (outputPath ?? defaultOutputPath);
+}

package/src/core/sort-shared-css.ts

@@ -0,0 +1,7 @@
+export function sortSharedCss(cssFiles: string[]): string[] {
+    // Put design-system CSS first to give anvil2 CSS higher precedence
+    const offset = cssFiles.findIndex(name => name.startsWith('design-system'));
+    return offset > 0
+        ? [cssFiles[offset], ...cssFiles.filter((_, index) => index !== offset)]
+        : cssFiles;
+}

package/src/core/types.ts

@@ -16,6 +16,7 @@ export interface PackageContext {
     isWebComponent: boolean;
     isLegacyRoot: boolean;
     isExposeSharedDependencies: boolean;
+    exposeProcessEnv?: string[];
     packageManager: ReturnType<typeof getPackageManager>;
 }
 
@@ -25,7 +26,6 @@ export interface BuildIdentity {
     light?: boolean;
     headless?: boolean;
     emitExposedDependencies?: boolean;
-    outputBasePath?: string;
 }
 
 export interface BuildOptions {

package/src/cypress/config/__tests__/vite-config.test.ts

@@ -0,0 +1,72 @@
+import type { InlineConfig } from 'vite';
+import { createCssConfig } from '../../../vite/config/create-css-config';
+import { istanbulPlugin, svgrPlugin } from '../../../vite/plugins';
+import { viteConfig } from '../vite-config';
+
+jest.mock('vite', () => ({
+    mergeConfig: (base: object, overrides: object) => ({ ...base, ...overrides }),
+}));
+jest.mock('../../../vite/config/create-css-config');
+jest.mock('../../../vite/plugins');
+
+describe(`[startup/cypress-config] ${viteConfig.name}`, () => {
+    const cssConfig: any = { modules: { localsConvention: 'camelCaseOnly' } };
+    const svgr: any = { name: 'svgr' };
+    const istanbul: any = { name: 'istanbul' };
+
+    let overrides: InlineConfig | undefined;
+
+    const subject = () => viteConfig(overrides);
+
+    beforeEach(() => {
+        jest.clearAllMocks();
+        jest.mocked(createCssConfig).mockReturnValue(cssConfig);
+        jest.mocked(svgrPlugin).mockReturnValue(svgr);
+        jest.mocked(istanbulPlugin).mockReturnValue(istanbul);
+        overrides = undefined;
+    });
+
+    test('builds the CSS config for development', () => {
+        subject();
+
+        expect(createCssConfig).toHaveBeenCalledWith(false);
+    });
+
+    test('wires the shared CSS config', () => {
+        expect(subject().css).toBe(cssConfig);
+    });
+
+    test('includes the SVG plugin', () => {
+        expect(subject().plugins).toContain(svgr);
+    });
+
+    test('enables code coverage', () => {
+        subject();
+
+        expect(istanbulPlugin).toHaveBeenCalledWith(
+            expect.objectContaining({ options: { codeCoverage: true } })
+        );
+    });
+
+    test('includes the coverage plugin', () => {
+        expect(subject().plugins).toContain(istanbul);
+    });
+
+    test('dedupes React', () => {
+        expect(subject().resolve?.dedupe).toEqual(['react', 'react-dom']);
+    });
+
+    test('pre-bundles React', () => {
+        expect(subject().optimizeDeps?.include).toContain('react/jsx-runtime');
+    });
+
+    describe('with overrides', () => {
+        beforeEach(() => {
+            overrides = { clearScreen: false };
+        });
+
+        test('merges overrides into the config', () => {
+            expect(subject().clearScreen).toBe(false);
+        });
+    });
+});

package/src/cypress/config/index.ts

@@ -1 +1,2 @@
+export * from './vite-config';
 export * from './webpack-config';

package/src/cypress/config/vite-config.ts

@@ -0,0 +1,37 @@
+import { type InlineConfig, mergeConfig } from 'vite';
+import { createCssConfig } from '../../vite/config/create-css-config';
+import { istanbulPlugin, svgrPlugin } from '../../vite/plugins';
+import type { ViteBuildContext } from '../../vite/types';
+
+/*
+ * Dedupe and pre-bundle React so component tests load a single React instance.
+ * Mirrors how viteFinal handles Storybook, where automatic discovery can miss
+ * React in monorepos.
+ */
+const REACT_DEDUPE = ['react', 'react-dom'];
+const REACT_OPTIMIZE = [...REACT_DEDUPE, 'react/jsx-runtime'];
+
+export function viteConfig(overrides?: InlineConfig): InlineConfig {
+    const context = getContext();
+
+    const config: InlineConfig = {
+        css: createCssConfig(false),
+        resolve: { dedupe: REACT_DEDUPE },
+        optimizeDeps: { include: REACT_OPTIMIZE },
+        plugins: [svgrPlugin(context), istanbulPlugin(context)],
+    };
+
+    return mergeConfig(config, overrides ?? {});
+}
+
+function getContext(): ViteBuildContext {
+    /*
+     * svgrPlugin reads bundlerConfig.pluginOverrides and istanbulPlugin reads
+     * options.codeCoverage, so a minimal context reuses both. Component tests
+     * always instrument for coverage, matching the webpack Cypress config.
+     */
+    return {
+        options: { codeCoverage: true },
+        bundlerConfig: { pluginOverrides: {} },
+    } as ViteBuildContext;
+}

package/src/storybook-config/__tests__/vite-final.test.ts

@@ -0,0 +1,122 @@
+import type { InlineConfig } from 'vite';
+import { createCssConfig } from '../../vite/config/create-css-config';
+import { svgrPlugin } from '../../vite/plugins/svgr-plugin';
+import { viteFinal } from '../vite-final';
+
+jest.mock('../../vite/plugins/svgr-plugin', () => ({ svgrPlugin: jest.fn() }));
+
+describe(`[startup/storybook-config] ${viteFinal.name}`, () => {
+    const mockSvgrPlugin = { name: 'vite-plugin-svgr' } as any;
+    let config: InlineConfig;
+    let configType: 'DEVELOPMENT' | 'PRODUCTION' | undefined;
+
+    beforeEach(() => {
+        jest.clearAllMocks();
+        jest.mocked(svgrPlugin).mockReturnValue(mockSvgrPlugin);
+        config = {};
+        configType = undefined;
+    });
+
+    const subject = () => viteFinal(config, { configType });
+
+    test('adds svgr plugin', () => {
+        expect(subject()).toEqual(expect.objectContaining({ plugins: [mockSvgrPlugin] }));
+    });
+
+    test('dedupes react', () => {
+        expect(subject()).toEqual(
+            expect.objectContaining({ resolve: { dedupe: ['react', 'react-dom'] } })
+        );
+    });
+
+    test('pre-bundles react', () => {
+        expect(subject()).toEqual(
+            expect.objectContaining({
+                optimizeDeps: { include: ['react', 'react-dom', 'react/jsx-runtime'] },
+            })
+        );
+    });
+
+    describe.each([
+        ['DEVELOPMENT', false],
+        ['PRODUCTION', true],
+    ] as const)('when configType is %s', (type, isProduction) => {
+        const cssConfig = createCssConfig(isProduction);
+
+        beforeEach(() => (configType = type));
+
+        test('adds the css modules', () => {
+            const { modules } = cssConfig;
+
+            expect(subject()).toEqual(
+                expect.objectContaining({ css: expect.objectContaining({ modules }) })
+            );
+        });
+
+        test('adds the preprocessor options', () => {
+            const { preprocessorOptions } = cssConfig;
+
+            expect(subject()).toEqual(
+                expect.objectContaining({ css: expect.objectContaining({ preprocessorOptions }) })
+            );
+        });
+    });
+
+    describe('when config already contains css preprocessor options', () => {
+        const scssOptions = { quietDeps: true };
+
+        beforeEach(() => (config = { css: { preprocessorOptions: { scss: scssOptions } } }));
+
+        test('preserves existing preprocessor options', () => {
+            const { preprocessorOptions } = createCssConfig(false);
+
+            expect(subject()).toEqual(
+                expect.objectContaining({
+                    css: expect.objectContaining({
+                        preprocessorOptions: {
+                            ...config.css?.preprocessorOptions,
+                            ...preprocessorOptions,
+                        },
+                    }),
+                })
+            );
+        });
+    });
+
+    describe('when config already contains plugins', () => {
+        const existingPlugin = { name: 'existing' };
+
+        beforeEach(() => (config = { plugins: [existingPlugin] }));
+
+        test('preserves existing plugins', () => {
+            expect(subject()).toEqual(
+                expect.objectContaining({ plugins: [existingPlugin, mockSvgrPlugin] })
+            );
+        });
+    });
+
+    describe('when config already contains resolve.dedupe', () => {
+        beforeEach(() => (config = { resolve: { dedupe: ['lodash', 'react'] } }));
+
+        test('merges react without duplicating existing entries', () => {
+            expect((subject().resolve as { dedupe: string[] }).dedupe).toEqual([
+                'lodash',
+                'react',
+                'react-dom',
+            ]);
+        });
+    });
+
+    describe('when config already contains optimizeDeps.include', () => {
+        beforeEach(() => (config = { optimizeDeps: { include: ['lodash', 'react'] } }));
+
+        test('merges react without duplicating existing entries', () => {
+            expect((subject().optimizeDeps as { include: string[] }).include).toEqual([
+                'lodash',
+                'react',
+                'react-dom',
+                'react/jsx-runtime',
+            ]);
+        });
+    });
+});

package/src/storybook-config/index.ts

@@ -1,2 +1,3 @@
 export * from './swc';
+export * from './vite-final';
 export * from './webpack-final';

package/src/storybook-config/vite-final.ts

@@ -0,0 +1,65 @@
+import { inspect } from 'node:util';
+import type { InlineConfig } from 'vite';
+import { log } from '../utils';
+import { createCssConfig } from '../vite/config/create-css-config';
+import { svgrPlugin } from '../vite/plugins/svgr-plugin';
+import type { ViteBuildContext } from '../vite/types';
+
+interface ViteFinalOptions {
+    /*
+     * Storybook signals the build type through `configType`, not Vite's
+     * `config.mode` (which Storybook's Vite builder leaves unset).
+     */
+    configType?: 'DEVELOPMENT' | 'PRODUCTION';
+}
+
+/*
+ * Pre-bundle and dedupe React so the Storybook preview loads a single React
+ * instance. Storybook leaves this to Vite's automatic discovery, which can miss
+ * React in monorepos.
+ */
+const REACT_DEDUPE = ['react', 'react-dom'];
+const REACT_OPTIMIZE = [...REACT_DEDUPE, 'react/jsx-runtime'];
+
+export function viteFinal(
+    config: InlineConfig,
+    { configType }: ViteFinalOptions = {}
+): InlineConfig {
+    const isProduction = configType === 'PRODUCTION';
+    const css = createCssConfig(isProduction);
+    const plugins = [svgrPlugin(getContext())];
+
+    const merged: InlineConfig = {
+        ...config,
+        css: {
+            ...config.css,
+            modules: css.modules,
+            preprocessorOptions: {
+                ...config.css?.preprocessorOptions,
+                ...css.preprocessorOptions,
+            },
+        },
+        resolve: {
+            ...config.resolve,
+            dedupe: [...new Set([...(config.resolve?.dedupe ?? []), ...REACT_DEDUPE])],
+        },
+        optimizeDeps: {
+            ...config.optimizeDeps,
+            include: [...new Set([...(config.optimizeDeps?.include ?? []), ...REACT_OPTIMIZE])],
+        },
+        plugins: [...(config.plugins ?? []), ...plugins],
+    };
+
+    /* istanbul ignore next: debug only */
+    log.debug('storybook-config:vite-final', () => inspect({ merged }, { depth: null }));
+
+    return merged;
+}
+
+function getContext(): ViteBuildContext {
+    /*
+     * svgrPlugin only reads bundlerConfig.pluginOverrides, so a minimal context
+     * is enough to reuse it with startup's default SVG include patterns.
+     */
+    return { bundlerConfig: { pluginOverrides: {} } } as ViteBuildContext;
+}

package/src/utils/__tests__/get-configuration.test.ts

@@ -4,6 +4,7 @@ import {
     getConfiguration,
     getConfigurationSafe,
     getESLintConfiguration,
+    getExposeProcessEnv,
     getJestConfiguration,
     getReviewConfiguration,
     getStylelintConfiguration,
@@ -18,6 +19,7 @@ import {
     isLegacy,
     isLegacyRoot,
     isStyleCheckDisabled,
+    isViteDevServerDisabled,
     isWebComponent,
 } from '../get-configuration';
 import { log } from '../log';
@@ -225,6 +227,22 @@ describe('[startup] Utils', () => {
         });
     });
 
+    describe(getExposeProcessEnv.name, () => {
+        const subject = () => getExposeProcessEnv();
+
+        itReturns(subject, []);
+
+        describe('with "cli.expose-process-env"', () => {
+            const content = { cli: { 'expose-process-env': ['API_URL', 'FEATURE_FLAG'] } };
+
+            beforeEach(() => mockPackageJson(content));
+
+            test('returns "cli.expose-process-env"', () => {
+                expect(subject()).toEqual(content.cli['expose-process-env']);
+            });
+        });
+    });
+
     describe(getViteConfiguration.name, () => {
         const subject = () => getViteConfiguration();
 
@@ -412,16 +430,16 @@ describe('[startup] Utils', () => {
     });
 
     describe.each([
-        {
-            fn: isBundle,
-            defaultValue: true,
-            trigger: { cli: { webpack: false } },
-        },
         {
             fn: isDevServerDisabled,
             defaultValue: false,
             trigger: { cli: { webpack: { devServer: false } } },
         },
+        {
+            fn: isViteDevServerDisabled,
+            defaultValue: false,
+            trigger: { cli: { vite: { devServer: false } } },
+        },
         {
             fn: isExposeSharedDependencies,
             defaultValue: false,
@@ -486,4 +504,63 @@ describe('[startup] Utils', () => {
             });
         });
     });
+
+    describe(isBundle.name, () => {
+        let json: Record<string, any>;
+
+        beforeEach(() => {
+            json = { name: '{packageName}', cli: {} };
+            mockPackageJson(json);
+        });
+
+        const subject = () => isBundle();
+
+        itReturns(subject, true);
+
+        describe('when cli.webpack is false', () => {
+            beforeEach(() => {
+                json.cli.webpack = false;
+                mockPackageJson(json);
+            });
+
+            itReturns(subject, false);
+
+            describe('when cli.vite is truthy', () => {
+                beforeEach(() => {
+                    json.cli.vite = {};
+                    mockPackageJson(json);
+                });
+
+                test('throws error', () => {
+                    expect(subject).toThrow(
+                        `package ${json.name} has conflicting webpack and vite configurations: ` +
+                            `cli.webpack is false but cli.vite is not`
+                    );
+                });
+            });
+        });
+
+        describe('when cli.vite is false', () => {
+            beforeEach(() => {
+                json.cli.vite = false;
+                mockPackageJson(json);
+            });
+
+            itReturns(subject, false);
+
+            describe('when cli.webpack is truthy', () => {
+                beforeEach(() => {
+                    json.cli.webpack = {};
+                    mockPackageJson(json);
+                });
+
+                test('throws error', () => {
+                    expect(subject).toThrow(
+                        `package ${json.name} has conflicting webpack and vite configurations: ` +
+                            `cli.vite is false but cli.webpack is not`
+                    );
+                });
+            });
+        });
+    });
 });

package/src/utils/__tests__/get-package-name.test.ts

@@ -0,0 +1,57 @@
+import { fs, vol } from 'memfs';
+import { getPackageName } from '../get-package-name';
+import { LocationOrJson } from '../types';
+
+jest.mock('fs', () => fs);
+
+describe(getPackageName.name, () => {
+    let locationOrJson: LocationOrJson | undefined;
+
+    beforeEach(() => {
+        vol.reset();
+        locationOrJson = undefined;
+    });
+
+    const subject = () => getPackageName(locationOrJson);
+
+    describe('with a JSON object', () => {
+        beforeEach(() => (locationOrJson = { name: 'foo' }));
+
+        test('returns "name" property', () => {
+            expect(subject()).toBe((locationOrJson as Record<string, any>).name);
+        });
+    });
+
+    describe('when name is not present', () => {
+        beforeEach(() => (locationOrJson = {}));
+
+        test('returns empty string', () => {
+            expect(subject()).toBe('');
+        });
+    });
+
+    describe('with a location', () => {
+        const packageJson = { name: 'bar' };
+
+        beforeEach(() => {
+            locationOrJson = 'packages/foo';
+            vol.fromJSON({
+                [`${locationOrJson as string}/package.json`]: JSON.stringify(packageJson),
+            });
+        });
+
+        test('returns "name" from location\'s package.json', () => {
+            expect(subject()).toBe(packageJson.name);
+        });
+    });
+
+    describe('with no argument', () => {
+        const packageJson = { name: 'baz' };
+
+        beforeEach(() => vol.fromJSON({ 'package.json': JSON.stringify(packageJson) }));
+
+        test('returns "name" from current package.json', () => {
+            expect(subject()).toBe(packageJson.name);
+        });
+    });
+});

package/src/utils/__tests__/validate-shared-dependencies.test.ts

@@ -3,53 +3,68 @@ import { validateSharedDependencies } from '../validate-shared-dependencies';
 
 jest.mock('fs', () => fs);
 
-describe('validateSharedDependencies', () => {
-    function mockPackageJson(content: Record<string, any> = {}) {
-        vol.fromJSON({ 'package.json': JSON.stringify(content) });
-    }
+describe(`[startup] ${validateSharedDependencies.name}`, () => {
+    let cli: Record<string, any>;
 
+    beforeEach(() => (cli = {}));
     afterEach(() => vol.reset());
 
-    test('passes when neither is set', () => {
-        mockPackageJson();
-        expect(() => validateSharedDependencies()).not.toThrow();
-    });
+    const subject = () => {
+        vol.fromJSON({ 'package.json': JSON.stringify({ cli }) });
+        return validateSharedDependencies();
+    };
 
-    test('passes when only cli top level is set', () => {
-        mockPackageJson({ cli: { 'shared-dependencies': { defaults: '' } } });
-        expect(() => validateSharedDependencies()).not.toThrow();
-    });
+    function itDoesNotThrow() {
+        test('does not throw', () => {
+            expect(subject).not.toThrow();
+        });
+    }
 
-    test('passes when only cli.webpack is set', () => {
-        mockPackageJson({ cli: { webpack: { 'shared-dependencies': { defaults: '' } } } });
-        expect(() => validateSharedDependencies()).not.toThrow();
-    });
+    function itThrows(message: RegExp) {
+        test('throws', () => {
+            expect(subject).toThrow(message);
+        });
+    }
 
-    test('passes when both are set and identical', () => {
-        const deps = { defaults: '', lodash: 'SharedDependencies.Lodash' };
-        mockPackageJson({
-            cli: { 'shared-dependencies': deps, 'webpack': { 'shared-dependencies': deps } },
+    itDoesNotThrow();
+
+    describe('with conflicting {cli, cli.webpack}.shared-dependencies', () => {
+        beforeEach(() => {
+            cli['shared-dependencies'] = { defaults: '' };
+            cli.webpack = { 'shared-dependencies': { defaults: '', extra: 'foo' } };
         });
-        expect(() => validateSharedDependencies()).not.toThrow();
+
+        itThrows(/both set but differ/);
     });
 
-    test('throws when shared-dependencies both exist and differ', () => {
-        mockPackageJson({
-            cli: {
-                'shared-dependencies': { defaults: '' },
-                'webpack': { 'shared-dependencies': { defaults: '', extra: 'foo' } },
-            },
+    describe('with conflicting {cli, cli.webpack}.expose-shared-dependencies', () => {
+        beforeEach(() => {
+            cli['expose-shared-dependencies'] = true;
+            cli.webpack = { 'expose-shared-dependencies': false };
         });
-        expect(() => validateSharedDependencies()).toThrow(/both set but differ/);
+
+        itThrows(/both set but differ/);
     });
 
-    test('throws when expose-shared-dependencies both exist and differ', () => {
-        mockPackageJson({
-            cli: {
-                'expose-shared-dependencies': true,
-                'webpack': { 'expose-shared-dependencies': false },
-            },
+    describe('when configured as a web-component', () => {
+        beforeEach(() => (cli['web-component'] = true));
+
+        describe.each([
+            { cli: { 'expose-shared-dependencies': true } },
+            { cli: { webpack: { 'expose-shared-dependencies': true } } },
+        ])('with %o', config => {
+            beforeEach(() => Object.assign(cli, config.cli));
+
+            itThrows(/only applies to host applications/);
+        });
+
+        describe.each([
+            { cli: { 'expose-shared-dependencies': false } },
+            { cli: { webpack: { 'expose-shared-dependencies': false } } },
+        ])('with %o', config => {
+            beforeEach(() => Object.assign(cli, config.cli));
+
+            itDoesNotThrow();
         });
-        expect(() => validateSharedDependencies()).toThrow(/both set but differ/);
     });
 });