jskelet 0.3.5 → 0.4.1

package/src/views/components/loader.js

@@ -1,7 +1,6 @@
 /**
- * `views/components/**` altındaki tüm modülleri yükler ve named export'larını
- * tek bir nesnede birleştirir. Bu nesne EJS şablonlarına local olarak geçer,
- * böylece `<%- card({ … }) %>` gibi çağrılar import gerektirmeden çalışır.
+ * `views/components/**` (ve feature/shared köklerindeki eşleri) altındaki
+ * tüm modülleri yükler ve named export'larını tek bir nesnede birleştirir.
  *
  * Elle bakılan bir barrel dosyası yok: yeni bir bileşen eklemek için dosyayı
  * oluşturmak yeterli.
@@ -44,17 +43,15 @@ function collect(dir, out = []) {
 }
 
 /**
- * @param {string} dir `views/components` dizininin mutlak yolu. Dizin yoksa
- *   boş nesne döner: bileşen kullanmayan bir proje de çalışmalı.
- * @returns {Promise<Record<string, unknown>>}
+ * Tek bir `components/` dizinini yükler.
+ *
+ * @param {string} dir
+ * @param {Record<string, unknown>} components
+ * @param {Map<string, string>} origin
+ * @returns {Promise<void>}
  */
-export async function loadComponents(dir) {
-  if (!dir || !fs.existsSync(dir)) return {};
-
-  /** @type {Record<string, unknown>} */
-  const components = {};
-  /** @type {Map<string, string>} */
-  const origin = new Map();
+async function loadDir(dir, components, origin) {
+  if (!dir || !fs.existsSync(dir)) return;
 
   const barrel = path.join(dir, BARREL);
   const files = [
@@ -63,23 +60,42 @@ export async function loadComponents(dir) {
   ];
 
   for (const file of files) {
-    const relative = path.relative(dir, file).split(path.sep).join("/");
+    const isBarrel = path.basename(file) === BARREL;
+    // Kimlik mutlak yol — çoklu components kökünde göreli ad çakışmasın.
+    const fileId = isBarrel ? BARREL : file.split(path.sep).join("/");
     const module = await import(pathToFileURL(file).href);
 
     for (const [name, value] of Object.entries(module)) {
       if (name === "default") continue;
 
       const previous = origin.get(name);
-      if (previous && previous !== BARREL && previous !== relative) {
-        console.warn(
-          `[components] '${name}' is defined twice: ${previous} and ${relative} — the second one wins.`,
+      // Barrel üzerine yazmak bilinçli; iki gerçek bileşen dosyası çakışması hata.
+      if (previous && previous !== BARREL && previous !== fileId) {
+        throw new Error(
+          `[components] '${name}' is defined twice: ${previous} and ${fileId}`,
         );
       }
 
       components[name] = value;
-      origin.set(name, relative);
+      origin.set(name, fileId);
     }
   }
+}
+
+/**
+ * @param {string | string[]} dirs Tek dizin veya çoklu kök (feature/shared).
+ * @returns {Promise<Record<string, unknown>>}
+ */
+export async function loadComponents(dirs) {
+  /** @type {Record<string, unknown>} */
+  const components = {};
+  /** @type {Map<string, string>} */
+  const origin = new Map();
+
+  const list = Array.isArray(dirs) ? dirs : [dirs];
+  for (const dir of list) {
+    await loadDir(dir, components, origin);
+  }
 
   return components;
 }