@neovici/cfg 2.7.5 → 2.8.0

package/check-duplicate-components.mjs

@@ -0,0 +1,102 @@
+#!/usr/bin/env node
+/* eslint-disable no-console */
+/**
+ * Check for duplicate @neovici/cosmoz-* web components in package-lock.json
+ *
+ * Web components register themselves globally via customElements.define().
+ * Having multiple versions of the same component causes runtime errors
+ * because the second version fails to register with the same tag name.
+ */
+
+import { readFileSync, existsSync } from 'fs';
+import { join } from 'path';
+
+const LOCKFILE = 'package-lock.json',
+	PACKAGE_PATTERN = /^@neovici\/cosmoz-/u,
+	PACKAGE_NAME_PATTERN = /node_modules\/(@[^/]+\/[^/]+|[^/]+)$/u;
+
+const readLockfile = (lockfilePath) => {
+	if (!existsSync(lockfilePath)) {
+		console.error(`ERROR: ${lockfilePath} not found`);
+		process.exit(1);
+	}
+
+	const lockfile = JSON.parse(readFileSync(lockfilePath, 'utf8'));
+
+	if (lockfile.lockfileVersion < 2) {
+		console.error('ERROR: This script requires lockfileVersion >= 2');
+		process.exit(1);
+	}
+
+	return lockfile;
+};
+
+const extractPackageName = (path) => {
+	const match = path.match(PACKAGE_NAME_PATTERN);
+	return match?.[1];
+};
+
+const collectComponentVersions = (packages) => {
+	const componentVersions = new Map();
+
+	for (const [path, info] of Object.entries(packages)) {
+		if (!path.startsWith('node_modules/')) continue;
+
+		const packageName = extractPackageName(path);
+		if (!packageName || !PACKAGE_PATTERN.test(packageName)) continue;
+
+		const { version } = info;
+		if (!version) continue;
+
+		if (!componentVersions.has(packageName)) {
+			componentVersions.set(packageName, []);
+		}
+		componentVersions.get(packageName).push({ version, path });
+	}
+
+	return componentVersions;
+};
+
+const findDuplicates = (componentVersions) =>
+	[...componentVersions.entries()]
+		.filter(([, instances]) => {
+			const uniqueVersions = new Set(instances.map((i) => i.version));
+			return uniqueVersions.size > 1;
+		})
+		.map(([packageName, instances]) => ({ packageName, instances }));
+
+const printDuplicates = (duplicates) => {
+	console.error('ERROR: Duplicate web components detected!\n');
+	console.error('Web components register themselves globally. Having multiple');
+	console.error('versions will cause runtime conflicts and errors.\n');
+
+	for (const { packageName, instances } of duplicates) {
+		console.error(`${packageName}:`);
+		for (const { version, path } of instances) {
+			console.error(`  - ${version} at ${path}`);
+		}
+		console.error('');
+	}
+
+	console.error('Fix: Update your dependencies to ensure all packages use');
+	console.error('compatible version ranges of web components.\n');
+	console.error('Run `npm ls <package-name>` to see the dependency tree');
+	console.error('and identify which packages need updating.');
+};
+
+const main = () => {
+	const lockfilePath = join(process.cwd(), LOCKFILE),
+		lockfile = readLockfile(lockfilePath),
+		componentVersions = collectComponentVersions(lockfile.packages || {}),
+		duplicates = findDuplicates(componentVersions);
+
+	if (duplicates.length === 0) {
+		console.log('✓ No duplicate web components found');
+		process.exit(0);
+	}
+
+	printDuplicates(duplicates);
+	process.exit(1);
+};
+
+main();

package/eslint/typescript.mjs

@@ -1,8 +1,8 @@
-import tseslint from 'typescript-eslint';
+import tseslint, { configs } from 'typescript-eslint';
 
 export default tseslint.config({
 	files: ['**/*.+(ts|tsx)'],
-	extends: [tseslint.configs.recommended],
+	extends: [configs.recommended],
 	rules: {
 		'import/named': 'off',
 		'no-unused-vars': 'off',

package/eslint.config.mjs

@@ -0,0 +1,11 @@
+import cfg from './eslint/index.mjs';
+import globals from 'globals';
+
+export default [
+	...cfg,
+	{
+		languageOptions: {
+			globals: { ...globals.node },
+		},
+	},
+];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@neovici/cfg",
-  "version": "2.7.5",
+  "version": "2.8.0",
   "description": "Configuration for Neovici packages",
   "homepage": "https://github.com/Neovici/cfg#readme",
   "bugs": {
@@ -13,6 +13,9 @@
   "license": "Apache-2.0",
   "author": "Neovici dev",
   "main": "index.js",
+  "bin": {
+    "check-duplicate-components": "./check-duplicate-components.mjs"
+  },
   "files": [
     "eslint/*.js",
     "eslint/*.mjs",
@@ -20,7 +23,8 @@
     "prettier/*.mjs",
     "web/*.js",
     "web/*.mjs",
-    "*.js"
+    "*.js",
+    "*.mjs"
   ],
   "scripts": {
     "lint": "eslint --cache .",
@@ -45,26 +49,26 @@
     ]
   },
   "dependencies": {
-    "@eslint/eslintrc": "^3.2.0",
-    "@eslint/js": "^9.18.0",
+    "@eslint/eslintrc": "^3.3.3",
+    "@eslint/js": "^9.39.2",
     "@playwright/test": "^1.40.1",
     "@web/dev-server": "^0.4.0",
     "@web/dev-server-esbuild": "^1.0.0",
     "@web/test-runner": "^0.20.0",
     "@web/test-runner-commands": "^0.9.0",
     "@web/test-runner-playwright": "^0.11.0",
-    "eslint": "^9.0.0",
-    "eslint-config-prettier": "^10.0.0",
-    "eslint-import-resolver-alias": "^1.0.0",
-    "eslint-import-resolver-typescript": "^4.0.0",
-    "eslint-plugin-html": "^8.0.0",
-    "eslint-plugin-import": "^2.0.0",
-    "eslint-plugin-mocha": "^11.0.0",
-    "globals": "^16.0.0",
+    "eslint": "^9.39.2",
+    "eslint-config-prettier": "^10.1.8",
+    "eslint-import-resolver-alias": "^1.1.2",
+    "eslint-import-resolver-typescript": "^4.4.4",
+    "eslint-plugin-html": "^8.1.3",
+    "eslint-plugin-import": "^2.32.0",
+    "eslint-plugin-mocha": "^11.2.0",
+    "globals": "^17.0.0",
     "prettier": "^3.0.0",
     "prettier-plugin-organize-imports": "^4.3.0",
     "typescript": "^5.1.0",
-    "typescript-eslint": "^8.20.0"
+    "typescript-eslint": "^8.53.0"
   },
   "devDependencies": {
     "@commitlint/cli": "^20.0.0",

package/web/performer.mjs

@@ -13,7 +13,6 @@ const perform = async (payload, context) => {
 
 export const performer = () => ({
 	name: 'cz-performer',
-	// eslint-disable-next-line max-statements
 	async executeCommand({ command, payload, session }) {
 		if (command !== 'cz-perform') return;
 		if (session.browser.type !== 'playwright') return;