@ywal123456/jskim 0.2.0 → 0.3.1

package/scripts/lib/resolve-project.js

@@ -3,6 +3,7 @@
 const fs = require('node:fs');
 const path = require('node:path');
 const { mergeConfig } = require('./merge-config');
+const { formatConfigValidationError } = require('./format-diagnostic');
 
 /**
  * 読み込んだ設定から名前付きプロジェクトを解決・検証します。
@@ -12,7 +13,7 @@ const { mergeConfig } = require('./merge-config');
  * @param {string} options.workspaceRoot
  * @param {string|undefined} options.projectName
  * @param {string} [options.commandName='build']
- * @param {string} [options.usageLine] プロジェクト名欠落時の使用方法行
+ * @param {string} [options.usageLine]
  * @returns {object} 解決済みプロジェクト
  */
 function resolveProject({
@@ -45,7 +46,8 @@ function resolveProject({
     );
   }
 
-  const merged = mergeConfig(config.defaults || {}, projectConfig);
+  const defaults = config.defaults || {};
+  const merged = mergeConfig(defaults, projectConfig);
 
   if (!merged.sourceDir || String(merged.sourceDir).trim() === '') {
     throw new Error(
@@ -79,39 +81,43 @@ function resolveProject({
     );
   }
 
-  if (!Array.isArray(merged.render) || merged.render.length === 0) {
+  validateData(merged.data, name);
+  validateNunjucks(merged.nunjucks, name);
+
+  const hasFiles = Array.isArray(merged.files) && merged.files.length > 0;
+  const hasRender = Array.isArray(merged.render) && merged.render.length > 0;
+  const hasCopy = Array.isArray(merged.copy) && merged.copy.length > 0;
+
+  if (hasFiles && (hasRender || hasCopy)) {
+    const conflict = [];
+    if (hasRender) {
+      conflict.push('render');
+    }
+    if (hasCopy) {
+      conflict.push('copy');
+    }
     throw new Error(
-      `[JSKim] プロジェクト "${name}" の render 設定が無い、または空です。\n` +
-        `jskim.config.js の defaults.render または projects.${name}.render を定義してください。`
+      `[JSKim] files と ${conflict.join(' / ')} を同時に設定できません。\n` +
+        `プロジェクト: ${name}\n` +
+        `衝突: files と ${conflict.join(' / ')}\n` +
+        `files mode を使う場合は render / copy を空にしてください。\n` +
+        `legacy mode を使う場合は files を設定しないでください。`
     );
   }
 
-  for (let i = 0; i < merged.render.length; i += 1) {
-    const rule = merged.render[i];
-    if (!rule || typeof rule !== 'object') {
-      throw new Error(
-        `[JSKim] プロジェクト "${name}" の render[${i}] が不正です。\n` +
-          `原因: 各 render ルールはオブジェクトである必要があります。`
-      );
-    }
-    if (!rule.from || String(rule.from).trim() === '') {
-      throw new Error(
-        `[JSKim] プロジェクト "${name}" の render[${i}].from が不正です。\n` +
-          `原因: from は必須です（sourceDir 基準）。`
-      );
-    }
-    if (!Array.isArray(rule.include) || rule.include.length === 0) {
-      throw new Error(
-        `[JSKim] プロジェクト "${name}" の render[${i}].include が不正です。\n` +
-          `原因: include は空でない glob パターン配列である必要があります。`
-      );
-    }
-    if (!rule.extension || String(rule.extension).trim() === '') {
+  const pipelineMode = hasFiles ? 'files' : 'legacy';
+
+  if (pipelineMode === 'files') {
+    validateFilesRules(merged.files, name);
+  } else {
+    if (!hasRender) {
       throw new Error(
-        `[JSKim] プロジェクト "${name}" の render[${i}].extension が不正です。\n` +
-          `原因: extension は必須です（例: ".html"）。`
+        `[JSKim] プロジェクト "${name}" の render 設定が無い、または空です。\n` +
+          `jskim.config.js の defaults.render または projects.${name}.render を定義してください。\n` +
+          `新しい files pipeline を使う場合は defaults.files を設定してください。`
       );
     }
+    validateRenderRules(merged.render, name);
   }
 
   validateWatchConfig(merged.watch, name);
@@ -124,9 +130,13 @@ function resolveProject({
     outputDir,
     sourceDirConfig: merged.sourceDir,
     outputDirConfig: merged.outputDir,
+    pipelineMode,
     render: merged.render,
     templates: merged.templates,
     copy: merged.copy,
+    files: merged.files || [],
+    data: merged.data,
+    nunjucks: merged.nunjucks,
     build: merged.build,
     watch: merged.watch,
     serve: merged.serve,
@@ -135,6 +145,137 @@ function resolveProject({
   };
 }
 
+function validateFilesRules(files, projectName) {
+  for (let i = 0; i < files.length; i += 1) {
+    const rule = files[i];
+    if (!rule || typeof rule !== 'object') {
+      throw new Error(
+        `[JSKim] プロジェクト "${projectName}" の files[${i}] が不正です。\n` +
+          `原因: 各 files ルールはオブジェクトである必要があります。`
+      );
+    }
+    if (!rule.from || String(rule.from).trim() === '') {
+      throw new Error(
+        `[JSKim] プロジェクト "${projectName}" の files[${i}].from が不正です。\n` +
+          `原因: from は必須です（sourceDir 基準）。`
+      );
+    }
+    if (rule.include !== undefined) {
+      if (!Array.isArray(rule.include) || rule.include.length === 0) {
+        throw new Error(
+          `[JSKim] プロジェクト "${projectName}" の files[${i}].include が不正です。\n` +
+            `原因: include は空でない glob 配列である必要があります。`
+        );
+      }
+    }
+    if (rule.exclude !== undefined && !Array.isArray(rule.exclude)) {
+      throw new Error(
+        `[JSKim] プロジェクト "${projectName}" の files[${i}].exclude が不正です。\n` +
+          `原因: exclude は配列である必要があります。`
+      );
+    }
+  }
+}
+
+function validateRenderRules(render, projectName) {
+  for (let i = 0; i < render.length; i += 1) {
+    const rule = render[i];
+    if (!rule || typeof rule !== 'object') {
+      throw new Error(
+        `[JSKim] プロジェクト "${projectName}" の render[${i}] が不正です。\n` +
+          `原因: 各 render ルールはオブジェクトである必要があります。`
+      );
+    }
+    if (!rule.from || String(rule.from).trim() === '') {
+      throw new Error(
+        `[JSKim] プロジェクト "${projectName}" の render[${i}].from が不正です。\n` +
+          `原因: from は必須です（sourceDir 基準）。`
+      );
+    }
+    if (!Array.isArray(rule.include) || rule.include.length === 0) {
+      throw new Error(
+        `[JSKim] プロジェクト "${projectName}" の render[${i}].include が不正です。\n` +
+          `原因: include は空でない glob パターン配列である必要があります。`
+      );
+    }
+    if (!rule.extension || String(rule.extension).trim() === '') {
+      throw new Error(
+        `[JSKim] プロジェクト "${projectName}" の render[${i}].extension が不正です。\n` +
+          `原因: extension は必須です（例: ".html"）。`
+      );
+    }
+  }
+}
+
+function validateData(data, projectName) {
+  if (data == null) {
+    return;
+  }
+  if (typeof data !== 'object' || Array.isArray(data)) {
+    throw new Error(
+      formatConfigValidationError({
+        projectName,
+        configKey: 'data',
+        detail:
+          'plain object を指定してください（null / array / primitive は不可）。',
+        received: Array.isArray(data) ? 'array' : typeof data,
+      })
+    );
+  }
+}
+
+function validateNunjucks(nunjucks, projectName) {
+  if (nunjucks == null) {
+    return;
+  }
+  if (typeof nunjucks !== 'object' || Array.isArray(nunjucks)) {
+    throw new Error(
+      formatConfigValidationError({
+        projectName,
+        configKey: 'nunjucks',
+        detail: 'plain object を指定してください。',
+        received: Array.isArray(nunjucks) ? 'array' : typeof nunjucks,
+      })
+    );
+  }
+
+  const filters = nunjucks.filters || {};
+  if (typeof filters !== 'object' || Array.isArray(filters)) {
+    throw new Error(
+      formatConfigValidationError({
+        projectName,
+        configKey: 'nunjucks.filters',
+        detail: 'plain object を指定してください。',
+        received: Array.isArray(filters) ? 'array' : typeof filters,
+      })
+    );
+  }
+  for (const [key, value] of Object.entries(filters)) {
+    if (typeof value !== 'function') {
+      throw new Error(
+        formatConfigValidationError({
+          projectName,
+          configKey: `nunjucks.filters.${key}`,
+          detail: 'filter は function である必要があります。',
+          received: typeof value,
+        })
+      );
+    }
+  }
+
+  const globals = nunjucks.globals || {};
+  if (typeof globals !== 'object' || Array.isArray(globals)) {
+    throw new Error(
+      formatConfigValidationError({
+        projectName,
+        configKey: 'nunjucks.globals',
+        detail: 'plain object を指定してください。',
+        received: Array.isArray(globals) ? 'array' : typeof globals,
+      })
+    );
+  }
+}
+
 function validateWatchConfig(watch, projectName) {
   const debounce = watch && watch.debounce;
 
@@ -144,10 +285,12 @@ function validateWatchConfig(watch, projectName) {
     debounce < 0
   ) {
     throw new Error(
-      `[JSKim] 設定値が不正です: watch.debounce\n` +
-        `プロジェクト: ${projectName}\n` +
-        `0以上の有限な数値を指定してください。\n` +
-        `受け取った値: ${String(debounce)}`
+      formatConfigValidationError({
+        projectName,
+        configKey: 'watch.debounce',
+        detail: '0以上の有限な数値を指定してください。',
+        received: String(debounce),
+      })
     );
   }
 }
@@ -158,10 +301,12 @@ function validateServeConfig(serve, projectName) {
 
   if (typeof host !== 'string' || host.trim() === '') {
     throw new Error(
-      `[JSKim] 設定値が不正です: serve.host\n` +
-        `プロジェクト: ${projectName}\n` +
-        `空でない文字列を指定してください。\n` +
-        `受け取った値: ${String(host)}`
+      formatConfigValidationError({
+        projectName,
+        configKey: 'serve.host',
+        detail: '空でない文字列を指定してください。',
+        received: String(host),
+      })
     );
   }
 
@@ -172,10 +317,12 @@ function validateServeConfig(serve, projectName) {
     port > 65535
   ) {
     throw new Error(
-      `[JSKim] 設定値が不正です: serve.port\n` +
-        `プロジェクト: ${projectName}\n` +
-        `1から65535までの整数を指定してください。\n` +
-        `受け取った値: ${String(port)}`
+      formatConfigValidationError({
+        projectName,
+        configKey: 'serve.port',
+        detail: '1から65535までの整数を指定してください。',
+        received: String(port),
+      })
     );
   }
 }
@@ -185,10 +332,12 @@ function validateDevConfig(dev, projectName) {
 
   if (typeof liveReload !== 'boolean') {
     throw new Error(
-      `[JSKim] 設定値が不正です: dev.liveReload\n` +
-        `プロジェクト: ${projectName}\n` +
-        `boolean を指定してください。\n` +
-        `受け取った値: ${String(liveReload)}`
+      formatConfigValidationError({
+        projectName,
+        configKey: 'dev.liveReload',
+        detail: 'boolean を指定してください。',
+        received: String(liveReload),
+      })
     );
   }
 }

package/scripts/lib/resolve-watch-paths.js

@@ -5,15 +5,25 @@ const path = require('node:path');
 
 /**
  * 解決済みプロジェクト設定から監視ディレクトリを計算します。
- * sourceDir 配下の render[].from / templates[] / copy[].from を監視します。
+ * files mode: files[].from / templates[]
+ * legacy mode: render[].from / templates[] / copy[].from
  * outputDir / dist / node_modules は監視しません。
  *
  * @param {object} project
  * @returns {{ absolutePaths: string[], displayPaths: string[] }}
  */
 function resolveWatchPaths(project) {
-  const { name, sourceDir, outputDir, workspaceRoot, render, templates, copy } =
-    project;
+  const {
+    name,
+    sourceDir,
+    outputDir,
+    workspaceRoot,
+    render,
+    templates,
+    copy,
+    files,
+    pipelineMode,
+  } = project;
 
   const absolutePaths = [];
   const seen = new Set();
@@ -39,9 +49,23 @@ function resolveWatchPaths(project) {
     absolutePaths.push(resolved);
   }
 
-  for (const rule of render) {
-    if (rule && rule.from) {
-      addCandidate(path.join(sourceDir, rule.from));
+  if (pipelineMode === 'files') {
+    for (const rule of Array.isArray(files) ? files : []) {
+      if (rule && rule.from) {
+        addCandidate(path.join(sourceDir, rule.from));
+      }
+    }
+  } else {
+    for (const rule of Array.isArray(render) ? render : []) {
+      if (rule && rule.from) {
+        addCandidate(path.join(sourceDir, rule.from));
+      }
+    }
+
+    for (const rule of Array.isArray(copy) ? copy : []) {
+      if (rule && rule.from) {
+        addCandidate(path.join(sourceDir, rule.from));
+      }
     }
   }
 
@@ -51,16 +75,10 @@ function resolveWatchPaths(project) {
     }
   }
 
-  for (const rule of Array.isArray(copy) ? copy : []) {
-    if (rule && rule.from) {
-      addCandidate(path.join(sourceDir, rule.from));
-    }
-  }
-
   if (absolutePaths.length === 0) {
     throw new Error(
       `[JSKim] プロジェクト "${name}" の監視パスがありません。\n` +
-        `原因: render[].from / templates[] / copy[].from のいずれも既存パスに解決できませんでした。\n` +
+        `原因: files[].from / render[].from / templates[] / copy[].from のいずれも既存パスに解決できませんでした。\n` +
         `sourceDir: ${sourceDir}`
     );
   }

package/scripts/lib/to-display-path.js

@@ -0,0 +1,28 @@
+'use strict';
+
+const path = require('node:path');
+
+/**
+ * ワークスペース相対の表示用パスへ変換します。
+ * @param {string} abs
+ * @param {string} workspaceRoot
+ * @returns {string}
+ */
+function toDisplayPath(abs, workspaceRoot) {
+  if (!abs) {
+    return '';
+  }
+  const resolved = path.resolve(abs);
+  if (!workspaceRoot) {
+    return resolved.split(path.sep).join('/');
+  }
+  const rel = path.relative(workspaceRoot, resolved);
+  if (!rel || rel.startsWith('..') || path.isAbsolute(rel)) {
+    return resolved.split(path.sep).join('/');
+  }
+  return rel.split(path.sep).join('/');
+}
+
+module.exports = {
+  toDisplayPath,
+};