@griddo/ax 12.2.3-rc.1 → 12.3.0

package/config/__tests__/webpack5-migration.test.js

@@ -100,10 +100,14 @@ describe("webpack.config.js — wp5 patterns", () => {
 		expect(webpackConfig).toContain("namedExport: true");
 	});
 
-	test("CSS modules use exportLocalsConvention dashesOnly (not camelCaseOnly)", () => {
-		// camelCaseOnly converts @keyframes _in → export var in (reserved word!)
-		// dashesOnly only converts hyphens, keeping _in as _in (valid identifier)
-		expect(webpackConfig).toContain('exportLocalsConvention: "dashesOnly"');
+	test("CSS modules use exportLocalsConvention asIs (webpack 4 class names)", () => {
+		// asIs keeps the names as written in the CSS, so client projects keep
+		// reading `styles["heading-sm"]` exactly like they did under webpack 4.
+		// css-loader derives `useExportsAs` from this value (its own option is not
+		// user-settable) and emits `export { _1 as "heading-sm" }`, so hyphenated
+		// classes are valid named exports. The convention to avoid is camelCaseOnly:
+		// it turns @keyframes _in into `export var in` (reserved word).
+		expect(webpackConfig).toContain('exportLocalsConvention: "asIs"');
 	});
 
 	test("postcss-loader uses postcssOptions (not ident + plugins fn)", () => {
@@ -370,8 +374,10 @@ describe("package.json — wp5 dependencies", () => {
 		expect(allDeps["react-dev-utils"]).toMatch(/^\^?12/);
 	});
 
-	test("css-loader is ^6.x", () => {
-		expect(allDeps["css-loader"]).toMatch(/^\^?6/);
+	test("css-loader is ^6.11.x (asIs + namedExport needs useExportsAs)", () => {
+		// Earlier 6.x rejected `exportLocalsConvention: "asIs"` alongside
+		// `namedExport: true`, so the floor is not just ^6.
+		expect(allDeps["css-loader"]).toMatch(/^\^?6\.(1[1-9]|[2-9]\d)/);
 	});
 
 	test("babel-loader is ^9.x", () => {

package/config/webpack.config.js

@@ -470,10 +470,13 @@ module.exports = function (webpackEnv) {
 									// only sees locals as named exports when `namedExport` is on.
 									// griddo-components relies on `import * as styles` heavily.
 									namedExport: true,
-									// `dashesOnly` converts only hyphenated names (foo-bar -> fooBar) so
-									// `composes: heading-sm` works, while leaving JS-reserved names such as
-									// `@keyframes _in` untouched (camelCaseOnly would break those).
-									exportLocalsConvention: "dashesOnly",
+									// `asIs` keeps the webpack 4 class names, so `styles["heading-sm"]`
+									// keeps working in client projects. css-loader derives
+									// `useExportsAs` from this value and emits
+									// `export { _1 as "heading-sm" }` instead of the invalid
+									// `export var heading-sm`. camelCaseOnly is what we must avoid:
+									// it would turn `@keyframes _in` into `export var in` (reserved word).
+									exportLocalsConvention: "asIs",
 									getLocalIdent: getCSSModuleLocalIdent,
 								},
 							}),
@@ -507,7 +510,7 @@ module.exports = function (webpackEnv) {
 									sourceMap: isEnvProduction ? shouldUseSourceMap : isEnvDevelopment,
 									modules: {
 										namedExport: true,
-										exportLocalsConvention: "dashesOnly",
+										exportLocalsConvention: "asIs",
 										getLocalIdent: getCSSModuleLocalIdent,
 									},
 								},

package/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "@griddo/ax",
 	"description": "Griddo Author Experience",
-	"version": "12.2.3-rc.1",
+	"version": "12.3.0",
 	"authors": [
 		"Álvaro Sánchez' <alvaro.sanches@secuoyas.com>",
 		"Diego M. Béjar <diego.bejar@secuoyas.com>",
@@ -77,7 +77,7 @@
 		"case-sensitive-paths-webpack-plugin": "2.4.0",
 		"compress.js": "1.2.2",
 		"connected-react-router": "6.9.3",
-		"css-loader": "^6.8.0",
+		"css-loader": "^6.11.0",
 		"css-minimizer-webpack-plugin": "^5.0.0",
 		"date-fns": "2.30.0",
 		"dotenv": "6.2.0",
@@ -213,5 +213,5 @@
 	"publishConfig": {
 		"access": "public"
 	},
-	"gitHead": "f328137a3b2bb2527d7d849e0ca9a2adec6f03cb"
+	"gitHead": "3702a3da08742efbc21a68aaae6c4617bde86e11"
 }

package/src/__tests__/gainup-trace.test.ts

@@ -0,0 +1,67 @@
+import { execFileSync } from "node:child_process";
+import { existsSync, readFileSync } from "node:fs";
+import { homedir } from "node:os";
+import { join } from "node:path";
+
+// GAINUP: la trazabilidad spec↔test es parte del contrato del paquete.
+// El tooling GAINUP no vive en el repo: se instala aparte, en ~/.claude/gainup, y el repo
+// solo lleva los artefactos (specs, config). Por eso este guardián se SALTA cuando el
+// validador no está instalado: un clon sin el tooling no debe comerse rojos de una capa
+// que no puede operar. Con el tooling presente corre igual que siempre.
+// Corremos en --strict, como cx y render: errores (link-rot, UC active sin test,
+// trace roto) y avisos (AC sin test, UC sin indexar en el README) ponen la suite
+// en rojo. Sin --strict la red de este paquete sería más fina que la de sus
+// vecinos sin que se viera.
+const PKG_ROOT = join(__dirname, "..", ".."); // src/__tests__ -> griddo-ax
+const TRACE = join(homedir(), ".claude", "gainup", "scripts", "gainup-trace.mjs");
+
+// GAINUP se instala como plugin de Claude Code, y su ruta de instalación es versionada
+// (…/plugins/cache/<marketplace>/gainup/<versión>/): el alias de arriba lo mantiene el hook
+// SessionStart del plugin, porque este arnés corre desde `yarn` y no tiene ${CLAUDE_PLUGIN_ROOT}.
+//
+// «Sin instalar» es un estado legítimo y el guardián se salta. «Instalado pero sin resolver»
+// no lo es: sería un verde por no haber validado nada — el verde falso contra el que está
+// montada esta capa. Ese caso revienta nombrando el motivo en vez de saltarse en silencio.
+const PLUGINS_DB = join(homedir(), ".claude", "plugins", "installed_plugins.json");
+function gainupPluginInstalled(): boolean {
+	try {
+		const db = JSON.parse(readFileSync(PLUGINS_DB, "utf8")) as { plugins?: Record<string, unknown> };
+		return Object.keys(db.plugins ?? {}).some((key) => key.startsWith("gainup@"));
+	} catch {
+		return false; // sin fichero o ilegible: no hay plugin del que hablar
+	}
+}
+
+if (gainupPluginInstalled() && !existsSync(TRACE)) {
+	throw new Error(
+		`GAINUP consta instalado como plugin pero el validador no está en ${TRACE}. ` +
+			"Sin él este guardián se saltaría y la suite daría verde sin haber validado nada. " +
+			"Abre una sesión de Claude Code (el hook del plugin recrea el alias) o enlázalo a mano.",
+	);
+}
+
+// El validador deduce la raíz del monorepo subiendo desde el cwd; Jest corre desde el
+// directorio del paquete, así que se la pasamos explícita.
+const ENV = { ...process.env, GAINUP_REPO_ROOT: join(PKG_ROOT, "..", "..") };
+
+// Jest no tiene `describe.skipIf` (es de Vitest): el ternario es su equivalente.
+const describeIfInstalled = existsSync(TRACE) ? describe : describe.skip;
+
+describeIfInstalled("GAINUP traceability", () => {
+	it("sin link-rot, UC sin test ni avisos (--strict)", () => {
+		let output = "";
+		let ok = true;
+		try {
+			output = execFileSync(process.execPath, [TRACE, "griddo-ax", "--strict"], {
+				encoding: "utf8",
+				env: ENV,
+			});
+		} catch (error) {
+			ok = false;
+			const e = error as { stdout?: string; stderr?: string };
+			output = `${e.stdout ?? ""}${e.stderr ?? ""}`;
+		}
+		if (!ok) console.error(output);
+		expect(ok).toBe(true);
+	});
+});

package/src/__tests__/aiup-trace.test.ts

@@ -1,23 +0,0 @@
-import { execFileSync } from "node:child_process";
-import { join } from "node:path";
-
-const PKG_ROOT = join(__dirname, "..", ".."); // src/__tests__ -> griddo-ax
-const TRACE = join(PKG_ROOT, "..", "..", "scripts", "aiup-trace.mjs");
-
-describe("Griddo-AIUP traceability", () => {
-	it("sin link-rot, traza rota ni UC active sin test", () => {
-		let output = "";
-		let ok = true;
-		try {
-			output = execFileSync(process.execPath, [TRACE, "griddo-ax"], {
-				encoding: "utf8",
-			});
-		} catch (error) {
-			ok = false;
-			const e = error as { stdout?: string; stderr?: string };
-			output = `${e.stdout ?? ""}${e.stderr ?? ""}`;
-		}
-		if (!ok) console.error(output);
-		expect(ok).toBe(true);
-	});
-});