feat(gdu): add Overdrive barrel-splitting Vite plugin (#408)

* feat(gdu): add Overdrive barrel-splitting Vite plugin

Rewrites `import { X } from '@autoguru/overdrive'` into deep component
imports (e.g. `@autoguru/overdrive/components/Box`) during Vite builds,
enabling granular tree-shaking and eliminating Vanilla Extract CSS
side-effect leakage from unused components.

Includes a 136-entry export manifest covering all components, hooks,
styles, themes, and utils — with a generator script for regeneration
when Overdrive updates.

Validated: fmo-booking builds identically (227 JS chunks, 82 CSS files,
zero barrel imports in output, build-manifest.json unchanged).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix(gdu): fix dev mode env token replacement for Vite 8/OXC

Vite 8 with OXC does not replace custom process.env.X dotted member
expressions via the define config. In production the mfeEnvTokens
plugin handles this, but dev mode had no equivalent — causing
process.env.mfeBasePath etc. to evaluate to undefined and breaking
Router basename resolution.

- Add devEnvReplace plugin to runSPA-vite.ts that inlines env values
  via pre-transform string replacement in dev mode
- Spread envDefines (not remainingDefines) into base define so the
  dev plugin can extract process.env.X entries
- Add apply: 'build' to overdriveBarrelSplit to skip it in dev mode

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix(gdu): address lint errors and review feedback

- Extract OverdriveCategory type alias (sonarjs/use-type-alias)
- Reduce processTopLevelBarrel cognitive complexity by extracting
  category detection helper (sonarjs/cognitive-complexity)
- Use String.raw for regex escape string (unicorn/prefer-string-raw)
- Flip negated ternary condition (unicorn/no-negated-condition)
- Add empty line between import groups (import/order)
- Use JSON.stringify for manifest output values to handle special chars
- Use full path in regeneration command doc comment
- Use proper JS identifier regex pattern for $ in export names
- Remove non-portable shebang from manifest generator

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Author: amir-zahedi

packages/gdu/commands/start/runSPA-vite.ts

@@ -81,6 +81,46 @@ function relayCjsToEsmPlugin(): VitePlugin {
 	};
 }
 
+/**
+ * Vite 8 with OXC does not replace custom `process.env.X` dotted member
+ * expressions via the `define` config — only simple identifiers and the
+ * special-cased `NODE_ENV` work.  In production the `mfeEnvTokens` plugin
+ * (apply: 'build') handles this; for dev mode we need an equivalent
+ * pre-transform that inlines the literal values.
+ */
+function devEnvReplace(define: Record<string, any>): VitePlugin {
+	const envMap: Record<string, string> = {};
+	for (const [key, value] of Object.entries(define)) {
+		const match = /^process\.env\.(.+)$/.exec(key);
+		if (match) {
+			envMap[match[1]] = String(value);
+		}
+	}
+
+	const keys = Object.keys(envMap);
+	if (keys.length === 0) {
+		return { name: 'gdu-dev-env-replace' };
+	}
+
+	const escapedKeys = keys.map((k) =>
+		k.replace(/[.*+?^${}()|[\]\\]/g, String.raw`\$&`),
+	);
+	const pattern = new RegExp(
+		`\\bprocess\\.env\\.(${escapedKeys.join('|')})\\b`,
+		'g',
+	);
+
+	return {
+		name: 'gdu-dev-env-replace',
+		enforce: 'pre',
+		transform(code) {
+			if (!code.includes('process.env.')) return null;
+			const result = code.replace(pattern, (_, key) => envMap[key]);
+			return result === code ? null : { code: result, map: null };
+		},
+	};
+}
+
 function spaHtmlPlugin(guruConfig: GuruConfig): VitePlugin {
 	const entryPath = join(gduEntryPath, 'spa', 'client.js');
 	const templatePath = getConsumerHtmlTemplate(guruConfig);
@@ -215,17 +255,21 @@ export const runSPAVite = async (guruConfig: GuruConfig, isDebug: boolean) => {
 		}
 	}
 
+	// Build the full define map so devEnvReplace sees the dev overrides
+	// (e.g. NODE_ENV = 'development') rather than the production defaults.
+	const devDefine = {
+		...base.define,
+		'process.env.NODE_ENV': JSON.stringify('development'),
+		__DEV__: JSON.stringify(true),
+		__DEBUG__: JSON.stringify(!!isDebug),
+	};
+
 	const server = await createServer({
 		root: PROJECT_ROOT,
 
 		resolve: base.resolve,
 
-		define: {
-			...base.define,
-			'process.env.NODE_ENV': JSON.stringify('development'),
-			__DEV__: JSON.stringify(true),
-			__DEBUG__: JSON.stringify(!!isDebug),
-		},
+		define: devDefine,
 
 		css: {
 			devSourcemap: true,
@@ -293,6 +337,7 @@ export const runSPAVite = async (guruConfig: GuruConfig, isDebug: boolean) => {
 		appType: 'custom',
 
 		plugins: [
+			devEnvReplace(devDefine),
 			...(vanillaExtractPlugin ? [vanillaExtractPlugin()] : []),
 			...(relayTransformPlugin ? [relayTransformPlugin] : []),
 			relayCjsToEsmPlugin(),