eleventy-plugin-cloudcannon - npm Package Compare versions

Comparing version 1.1.0 to 1.1.1-0

package.json

 {
   "name": "eleventy-plugin-cloudcannon",
-  "version": "1.1.0",
+  "version": "1.1.1-0",
   "description": "Eleventy plugin to create CloudCannon editor details",
@@ -41,3 +41,3 @@ "main": ".eleventy.js",
   "devDependencies": {
-    "ava": "^3.15.0",
+    "ava": "^4.2.0",
     "c8": "^7.8.0",
@@ -44,0 +44,0 @@ "eslint": "^8.1.0"

src/generators/collections.js

 const { readdirSync } = require('fs');
 const { dirname, join } = require('path');
-const { bold, yellow } = require('chalk');
+const { bold } = require('chalk');
 const { log } = require('../util/logger.js');
@@ -9,21 +9,12 @@ const { stringifyJson } = require('../util/json.js');
 
-function cheapPlural(amount, str) {
-	const amountStr = amount === 0 ? 'no' : amount;
-	return `${amountStr} ${str}${amount === 1 ? '' : 's'}`;
-}
-
 // Tests stringify individually to avoid one item breaking it
-function processItems(items, tag, config) {
-	return items?.reduce?.((memo, item) => {
-		const processed = processItem(item, tag, config.source);
-		const stringified = stringifyJson(processed);
+function processCollectionItem(item, collectionKey, config) {
+	const processed = processItem(item, collectionKey, config.source);
+	const stringified = stringifyJson(processed);
 
-		if (stringified === undefined) {
-			log(`⚠️ ${bold(item?.inputPath || 'unknown file')} skipped due to JSON stringify failing`);
-		} else {
-			memo.push(JSON.parse(stringified));
-		}
+	if (stringified !== undefined) {
+		return JSON.parse(stringified);
+	}
 
-		return memo;
-	}, []) || [];
+	log(`⚠️ ${bold(item?.inputPath || 'unknown file')} skipped due to JSON stringify failing`);
 }
@@ -40,23 +31,38 @@
 
-function getCollections(collectionsConfig, context, config) {
-	const { all, ...otherCollections } = context.collections;
+function getCollectionKey(item, collectionsConfig, config) {
+	const sourcePath = getSourcePath(item?.inputPath, config.source) || '';
+	const parts = sourcePath.split('/');
 
-	if (!otherCollections.pages) {
-		otherCollections.pages = all.filter((item) => isPage(item) || isStaticPage(item));
+	// Find collection from config based on explicit path
+	for (let i = parts.length - 1; i >= 0; i--) {
+		const collectionPath = parts.slice(0, i).join('/');
+
+		const configKey = Object.keys(collectionsConfig || {}).find((key) => {
+			return collectionsConfig[key]?.path === collectionPath;
+		});
+
+		if (configKey) {
+			return configKey;
+		}
 	}
 
-	return Object.keys(collectionsConfig).reduce((memo, collectionKey) => {
-		const items = otherCollections[collectionKey];
+	if (isPage(item) || isStaticPage(item)) {
+		return 'pages';
+	}
+}
 
-		if (items?.length) {
-			memo[collectionKey] = processItems(items, collectionKey, config);
-			const filesCount = cheapPlural(memo[collectionKey].length, 'file');
-			log(`📁 Processed ${bold(collectionKey)} collection with ${filesCount}`);
-		} else if (collectionsConfig[collectionKey].auto_discovered) {
-			log(`📂 ${yellow('Ignored')} ${bold(collectionKey)} collection`);
-			delete collectionsConfig[collectionKey];
-		} else {
-			log(`📁 Processed ${bold(collectionKey)} collection`);
+function getCollections(collectionsConfig, context, config) {
+	return context.collections.all.reduce((memo, item) => {
+		const collectionKey = getCollectionKey(item, collectionsConfig, config);
+
+		if (!collectionKey) {
+			log(`⚠️ No collection for ${bold(item?.inputPath || 'unknown file')}`);
+			return memo;
 		}
 
+		const processed = processCollectionItem(item, collectionKey, config);
+
+		memo[collectionKey] = memo[collectionKey] || [];
+		memo[collectionKey].push(processed);
+
 		return memo;
@@ -67,5 +73,3 @@ }, {});
 function discoverCollectionsConfig(context, config) {
-	const { all, ...otherCollections } = context.collections;
-
-	const guessed = all.reduce((memo, item) => {
+	const discovered = context.collections.all.reduce((memo, item) => {
 		const tag = item.data?.tags?.[0];
@@ -76,5 +80,5 @@
 			// Map tags to basePaths, items with same tags can exist in separate folders
-			const inputPath = getSourcePath(item.inputPath, config.source);
-			memo[tag].basePaths.add(dirname(inputPath));
-			// Tracks how collection items are output
+			const sourcePath = getSourcePath(item.inputPath, config.source);
+			memo[tag].basePaths.add(dirname(sourcePath));
+			// Tracks how many collection items are output
 			memo[tag].outputOffset += item.url === false ? -1 : 1;
@@ -86,22 +90,27 @@ }
 
-	return Object.keys(guessed).reduce((memo, key) => {
-		if (!otherCollections[key]) {
-			return memo;
-		}
-
+	return Object.keys(discovered).reduce((memo, tag) => {
 		// Finds the top-most common basePaths to prevent sub-folders becoming separate entries
-		const topBasePaths = Array.from(guessed[key].basePaths).filter(isTopPath);
+		const topBasePaths = Array.from(discovered[tag].basePaths).filter(isTopPath);
 
 		// Consider a collection output unless more items are not output
-		const isOutput = guessed[key].outputOffset >= 0;
+		const isOutput = discovered[tag].outputOffset >= 0;
 
 		topBasePaths.forEach((basePath) => {
-			// Multiple collections can share this basePath, but this should cover common use-cases
-			const collectionKey = topBasePaths.length === 1 ? key : basePath;
-			const customPath = memo[collectionKey]?.path;
+			// Finds the existing key for this base path, or creates one
+			// If multiple discovered collections use the same base path, the first one processed is used
+			const collectionKey = Object.keys(memo).find((k) => memo[k].path === basePath)
+				// Use the tag as the collection key if the files are all in one base path
+				|| (topBasePaths.length === 1 ? tag : basePath);
 
+			const existingPath = memo[collectionKey]?.path;
+			const autoDiscovered = !existingPath && existingPath !== '';
+
+			if (autoDiscovered) {
+				log(`🔍 Discovered ${bold(collectionKey)} collection`);
+			}
+
 			memo[collectionKey] = {
 				path: basePath,
 				output: isOutput,
-				auto_discovered: !customPath && customPath !== '',
+				auto_discovered: autoDiscovered,
 				...memo[collectionKey]
@@ -108,0 +117,0 @@ };

src/generators/info.js

 const { getData } = require('./data.js');
 const { getGenerator } = require('./generator.js');
 const { getCollections, getCollectionsConfig } = require('./collections.js');
+const { bold, yellow } = require('chalk');
+const { log } = require('../util/logger.js');
 require('pkginfo')(module, 'name', 'version');
 
+function cheapPlural(amount, str) {
+	const amountStr = amount === 0 ? 'no' : amount;
+	return `${amountStr} ${str}${amount === 1 ? '' : 's'}`;
+}
+
 const cloudcannon = {
@@ -15,2 +22,14 @@ name: module.exports.name,
 
+	Object.keys(collectionsConfig).forEach((collectionKey) => {
+		if (collections[collectionKey]?.length) {
+			const filesCount = cheapPlural(collections[collectionKey].length, 'file');
+			log(`📁 Processed ${bold(collectionKey)} collection with ${filesCount}`);
+		} else if (collectionsConfig[collectionKey].auto_discovered) {
+			log(`📂 ${yellow('Ignored')} ${bold(collectionKey)} collection`);
+			delete collectionsConfig[collectionKey];
+		} else {
+			log(`📁 Processed ${bold(collectionKey)} collection`);
+		}
+	});
+
 	return {
@@ -17,0 +36,0 @@ ...config,

src/util/items.js

@@ -42,3 +42,3 @@ const { dirname, basename, extname } = require('path');
 
-function processItem(item, tag, source) {
+function processItem(item, collectionKey, source) {
 	if (!item.inputPath) {
@@ -73,4 +73,4 @@ return;
 
-	if (tag) {
-		processed.collection = tag;
+	if (collectionKey) {
+		processed.collection = collectionKey;
 	}
@@ -77,0 +77,0 @@

New alerts

No v1

Quality

Package is not semver >=1. This means it is not stable and does not support ^ ranges.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

26306

increased by4.18%

Lines of code

503

increased by4.57%

Worsened metrics

Number of low quality alerts

1

increased byInfinity%

No dependency changes