@hiscovega/vundle 0.0.13 → 0.0.15

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # Changelog
 
+## [0.0.15](https://github.com/griddo/griddo-bundler/compare/v0.0.14...v0.0.15) (2026-01-21)
+
+
+### Features
+
+* add initial client configuration and test suite ([3c51477](https://github.com/griddo/griddo-bundler/commit/3c514775e98cba6384028db9787b596632369fe1))
+* **vite:** enhance external handling and improve build resilience ([2acb47a](https://github.com/griddo/griddo-bundler/commit/2acb47a19d845f19c971242f7396642f02c75fba))
+
+## [0.0.14](https://github.com/griddo/griddo-bundler/compare/v0.0.13...v0.0.14) (2026-01-21)
+
+
+### Bug Fixes
+
+* **vite:** remove build warnings and harden externals ([1b95862](https://github.com/griddo/griddo-bundler/commit/1b9586219354627837e8fd8165151ce62960ebfe))
+
 ## [0.0.13](https://github.com/griddo/griddo-bundler/compare/v0.0.12...v0.0.13) (2025-11-10)
 
 ## [0.0.12](https://github.com/griddo/griddo-bundler/compare/v0.0.11...v0.0.12) (2025-10-16)

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@hiscovega/vundle",
-	"version": "0.0.13",
+	"version": "0.0.15",
 	"description": "Master bundler configuration for Griddo Instance projects",
 	"main": "vite.config.mjs",
 	"module": "vite.config.mjs",
@@ -8,11 +8,11 @@
 	"type": "module",
 	"exports": {
 		".": {
-			"types": "./vite.config.mjs",
+			"types": "./types/client-config.d.ts",
 			"import": "./vite.config.mjs"
 		},
 		"./vite.config": {
-			"types": "./vite.config.mjs",
+			"types": "./types/client-config.d.ts",
 			"import": "./vite.config.mjs"
 		},
 		"./plugins": {
@@ -35,11 +35,14 @@
 	],
 	"scripts": {
 		"remote:build": "vite build --config ./vite.config.mjs",
+		"test": "vitest run --config ./vitest.config.mjs",
+		"test:watch": "vitest --config ./vitest.config.mjs",
+		"fixture:build": "cd fixtures/client && ../../node_modules/.bin/vite build --config ../../vite.config.mjs",
 		"lint": "biome check .",
 		"lint:fix": "biome check --write .",
 		"format": "biome format --write .",
 		"format:check": "biome format .",
-		"prepublishOnly": "npm run lint && npm run format:check",
+		"prepublishOnly": "npm run lint && npm run format:check && npm test",
 		"prepare": "npm run lint",
 		"release": "release-it",
 		"release:dry": "release-it --dry-run",
@@ -96,6 +99,7 @@
 	"devDependencies": {
 		"@biomejs/biome": "2.2.4",
 		"@release-it/conventional-changelog": "^10.0.1",
-		"release-it": "^19.0.4"
+		"release-it": "^19.0.4",
+		"vitest": "^4.0.17"
 	}
 }

package/plugins/index.mjs

@@ -18,6 +18,9 @@ export const plugins = [
 	// }),
 
 	viteStaticCopy({
+		// `static/` y `thumbnails/` no son obligatorios en todos los clientes.
+		// Si no existen/no hay archivos, no queremos que el build falle.
+		silent: true,
 		targets: [
 			{ src: "static", dest: "" },
 			{ src: "thumbnails", dest: "" },

package/vite.config.mjs

@@ -5,6 +5,30 @@ import { plugins as masterPlugins } from "./plugins/index.mjs";
 export default defineConfig(async ({ command, mode }) => {
 	let clientConfig = {};
 
+	const sharedAssetFileNames = (assetInfo) => {
+		if (assetInfo.name?.endsWith(".css")) {
+			return "builder.css";
+		}
+		if (assetInfo.name?.match(/\.(png|jpe?g|gif|webp)$/)) {
+			return "assets/[name]-[hash][extname]";
+		}
+		return "assets/[name]-[hash][extname]";
+	};
+
+	const toExternalFn = (external) => {
+		if (!external) return () => false;
+		if (typeof external === "function") return external;
+
+		const externals = Array.isArray(external) ? external : [external];
+		return (id) => {
+			for (const ext of externals) {
+				if (typeof ext === "string" && (id === ext || id.startsWith(`${ext}/`))) return true;
+				if (ext instanceof RegExp && ext.test(id)) return true;
+			}
+			return false;
+		};
+	};
+
 	// Usamos la función nativa de Vite para encontrar, cargar y transpilar
 	// la configuración del cliente, soportando .ts, .js, .mjs, etc.
 	try {
@@ -35,7 +59,10 @@ export default defineConfig(async ({ command, mode }) => {
 	// --- Configuración Base del Master ---
 	const masterConfig = {
 		logLevel: "warn",
-		publicDir: "static",
+		// El copiado de assets lo gestiona `vite-plugin-static-copy` y además
+		// `copyPublicDir` está desactivado; evitamos confusiones deshabilitando
+		// `publicDir` explícitamente.
+		publicDir: false,
 		define: { global: "globalThis" },
 		plugins: masterPlugins,
 		build: {
@@ -46,14 +73,20 @@ export default defineConfig(async ({ command, mode }) => {
 					builder: resolve(process.cwd(), "src/builder.jsx"),
 					"griddo.config": resolve(process.cwd(), "src/griddo.config.ts"),
 				},
-				formats: ["es", "cjs"],
 				name: "GriddoInstance",
 			},
 			rollupOptions: {
 				preserveEntrySignatures: "strict",
 				external: (id) => {
 					// 1. Reglas 'external' del master (no se pueden sobreescribir)
-					const masterRules = ["@griddo/core", "react", "react-dom", "react/jsx-runtime", "prop-types"];
+					const masterRules = [
+						"@griddo/core",
+						"react",
+						"react-dom",
+						"react/jsx-runtime",
+						"react/jsx-dev-runtime",
+						"prop-types",
+					];
 					if (masterRules.some((pkg) => id === pkg || id.startsWith(`${pkg}/`))) {
 						return true;
 					}
@@ -79,30 +112,14 @@ export default defineConfig(async ({ command, mode }) => {
 						dir: "dist",
 						format: "es",
 						entryFileNames: "[name].js",
-						assetFileNames: (assetInfo) => {
-							if (assetInfo.name?.endsWith(".css")) {
-								return "builder.css";
-							}
-							if (assetInfo.name?.match(/\.(png|jpe?g|gif|webp)$/)) {
-								return "assets/[name]-[hash][extname]";
-							}
-							return "assets/[name]-[hash][extname]";
-						},
+						assetFileNames: sharedAssetFileNames,
 					},
 					{
 						dir: "dist",
 						format: "cjs",
 						entryFileNames: "[name].cjs",
 						exports: "named",
-						assetFileNames: (assetInfo) => {
-							if (assetInfo.name?.endsWith(".css")) {
-								return "builder.css";
-							}
-							if (assetInfo.name?.match(/\.(png|jpe?g|gif|webp)$/)) {
-								return "assets/[name]-[hash][extname]";
-							}
-							return "assets/[name]-[hash][extname]";
-						},
+						assetFileNames: sharedAssetFileNames,
 					},
 				],
 			},
@@ -113,15 +130,18 @@ export default defineConfig(async ({ command, mode }) => {
 	};
 
 	// --- Fusión Final ---
-	// Antes de fusionar, nos aseguramos de que la configuración `external` del
-	// master nunca sea sobreescrita por el cliente. Esta es la clave para
-	// evitar la duplicación de React.
-	if (clientConfig.build?.rollupOptions?.external) {
-		console.log(
-			"WARN: The client-defined 'build.rollupOptions.external' property will be ignored. Use 'griddoOptions.customExternals' instead.",
-		);
-		delete clientConfig.build.rollupOptions.external;
-	}
+	const merged = mergeConfig(masterConfig, clientConfig);
+
+	// Componemos externals: reglas del master (React/@griddo/core, etc.) + API declarativa
+	// `griddoOptions.customExternals` + `build.rollupOptions.external` estándar del cliente.
+	// Esto mantiene compatibilidad con Vite/Rollup sin permitir que el cliente "rebundee" React.
+	const clientExternal = toExternalFn(clientConfig.build?.rollupOptions?.external);
+	const masterExternal = toExternalFn(masterConfig.build?.rollupOptions?.external);
+
+	if (!merged.build) merged.build = {};
+	if (!merged.build.rollupOptions) merged.build.rollupOptions = {};
+
+	merged.build.rollupOptions.external = (id, ...args) => masterExternal(id, ...args) || clientExternal(id, ...args);
 
-	return mergeConfig(masterConfig, clientConfig);
+	return merged;
 });