@ywal123456/jskim 0.4.0 → 0.5.1

package/scripts/commands/watch-command.js

@@ -14,7 +14,7 @@ const { registerShutdown } = require('./register-shutdown');
 async function runWatchCommand(options = {}) {
   const workspaceRoot = options.workspaceRoot || process.cwd();
   const usageLine =
-    options.usageLine || 'npm run watch -- <project-name>';
+    options.usageLine || 'jskim watch [<project>]';
 
   const runtime = createWatchRuntime({
     mode: 'watch',

package/scripts/dev.js

@@ -1,11 +1,25 @@
 'use strict';
 
+const { parseCommandArgv } = require('./lib/parse-cli-args');
 const { runDevCommand } = require('./commands/dev-command');
 
+let parsed;
+try {
+  parsed = parseCommandArgv('dev', process.argv.slice(2));
+} catch (err) {
+  const message = err && err.message ? err.message : String(err);
+  console.error(message);
+  process.exitCode = 1;
+  return;
+}
+
 runDevCommand({
-  projectName: process.argv[2],
+  projectName: parsed.projectName,
   workspaceRoot: process.cwd(),
-  usageLine: 'npm run dev -- <project-name>',
+  usageLine: 'jskim dev [<project>] [--host <host>] [--port <port>] [--open]',
+  host: parsed.options.host,
+  port: parsed.options.port,
+  open: parsed.options.open,
 }).catch((err) => {
   const message = err && err.message ? err.message : String(err);
   console.error(message);

package/scripts/lib/apply-serve-cli-overrides.js

@@ -0,0 +1,96 @@
+'use strict';
+
+const { formatConfigValidationError } = require('./format-diagnostic');
+
+/**
+ * CLI --host / --port を resolved project に適用した新しいオブジェクトを返します。
+ * 元の project / config は変更しません。
+ *
+ * @param {object} project
+ * @param {{ host?: string, port?: string|number }} [overrides]
+ * @returns {object}
+ */
+function applyServeCliOverrides(project, overrides = {}) {
+  if (!project || typeof project !== 'object') {
+    throw new Error('[JSKim] projectが不正です。');
+  }
+
+  const hasHost = overrides.host !== undefined;
+  const hasPort = overrides.port !== undefined;
+  if (!hasHost && !hasPort) {
+    return project;
+  }
+
+  const nextServe = {
+    ...(project.serve || {}),
+  };
+
+  if (hasHost) {
+    nextServe.host = overrides.host;
+  }
+
+  if (hasPort) {
+    nextServe.port = coercePort(overrides.port, project.name);
+  }
+
+  validateServeOverride(nextServe, project.name);
+  return {
+    ...project,
+    serve: nextServe,
+  };
+}
+
+function coercePort(value, projectName) {
+  if (typeof value === 'number') {
+    return value;
+  }
+  const text = String(value).trim();
+  if (text === '' || !/^-?\d+$/.test(text)) {
+    throw new Error(
+      formatConfigValidationError({
+        projectName,
+        configKey: 'CLI --port',
+        detail: '1から65535までの整数を指定してください。',
+        received: String(value),
+      })
+    );
+  }
+  return Number(text);
+}
+
+function validateServeOverride(serve, projectName) {
+  const host = serve && serve.host;
+  const port = serve && serve.port;
+
+  if (typeof host !== 'string' || host.trim() === '') {
+    throw new Error(
+      formatConfigValidationError({
+        projectName,
+        configKey: 'CLI --host / serve.host',
+        detail: '空でない文字列を指定してください。',
+        received: String(host),
+      })
+    );
+  }
+
+  if (
+    typeof port !== 'number' ||
+    !Number.isInteger(port) ||
+    port < 1 ||
+    port > 65535
+  ) {
+    throw new Error(
+      formatConfigValidationError({
+        projectName,
+        configKey: 'CLI --port / serve.port',
+        detail: '1から65535までの整数を指定してください。',
+        received: String(port),
+      })
+    );
+  }
+}
+
+module.exports = {
+  applyServeCliOverrides,
+  coercePort,
+};

package/scripts/lib/assert-output-dirs-compatible.js

@@ -0,0 +1,88 @@
+'use strict';
+
+const path = require('node:path');
+
+/**
+ * --all 用に、resolve済みprojectのoutputDir衝突を検査します。
+ *
+ * @param {Array<{ name: string, outputDir: string }>} projects
+ */
+function assertCompatibleOutputDirs(projects) {
+  const items = Array.isArray(projects) ? projects : [];
+
+  for (let i = 0; i < items.length; i += 1) {
+    for (let j = i + 1; j < items.length; j += 1) {
+      const a = items[i];
+      const b = items[j];
+      const relation = classifyOutputDirRelation(a.outputDir, b.outputDir);
+      if (!relation) {
+        continue;
+      }
+
+      const label =
+        relation === 'same'
+          ? '同じoutputDir'
+          : '入れ子（祖先/子孫）のoutputDir';
+
+      throw new Error(
+        `[JSKim] --all でbuildできません: project間のoutputDirが衝突しています。\n` +
+          `理由: ${label} のため、clean付きbuildで他projectの成果物を消す恐れがあります。\n\n` +
+          `project: ${a.name}\n` +
+          `outputDir: ${toPosix(a.outputDir)}\n\n` +
+          `project: ${b.name}\n` +
+          `outputDir: ${toPosix(b.outputDir)}\n\n` +
+          `各projectのoutputDirを重複・入れ子にならないよう変更してください。`
+      );
+    }
+  }
+}
+
+/**
+ * @param {string} left
+ * @param {string} right
+ * @returns {'same'|'nested'|null}
+ */
+function classifyOutputDirRelation(left, right) {
+  const a = path.resolve(left);
+  const b = path.resolve(right);
+  if (samePath(a, b)) {
+    return 'same';
+  }
+  if (isStrictAncestor(a, b) || isStrictAncestor(b, a)) {
+    return 'nested';
+  }
+  return null;
+}
+
+function samePath(a, b) {
+  const na = path.resolve(a);
+  const nb = path.resolve(b);
+  if (process.platform === 'win32') {
+    return na.toLowerCase() === nb.toLowerCase();
+  }
+  return na === nb;
+}
+
+function isStrictAncestor(ancestor, descendant) {
+  let a = path.resolve(ancestor);
+  let d = path.resolve(descendant);
+  if (process.platform === 'win32') {
+    a = a.toLowerCase();
+    d = d.toLowerCase();
+  }
+  if (samePath(a, d)) {
+    return false;
+  }
+  const rel = path.relative(a, d);
+  return rel !== '' && !rel.startsWith('..') && !path.isAbsolute(rel);
+}
+
+function toPosix(abs) {
+  return path.resolve(abs).split(path.sep).join('/');
+}
+
+module.exports = {
+  assertCompatibleOutputDirs,
+  classifyOutputDirRelation,
+  samePath,
+};

package/scripts/lib/create-watch-runtime.js

@@ -5,11 +5,17 @@ const path = require('node:path');
 const chokidar = require('chokidar');
 const { loadConfig, CONFIG_FILENAME } = require('./load-config');
 const { resolveProject } = require('./resolve-project');
+const { selectProjectName } = require('./select-project-name');
+const { applyServeCliOverrides } = require('./apply-serve-cli-overrides');
 const { createProjectWatcher } = require('./create-project-watcher');
 const { createStaticServer } = require('./create-static-server');
 const { createLiveReload } = require('./create-live-reload');
 const { formatListenError } = require('../commands/serve-errors');
 const { classifyReload } = require('./classify-reload');
+const {
+  buildBrowserOpenUrl,
+  openBrowser: defaultOpenBrowser,
+} = require('./open-browser');
 
 const DEV_RESTART_KEYS = [
   'outputDir',
@@ -28,16 +34,24 @@ const DEV_RESTART_KEYS = [
  * @param {string|undefined} options.projectName
  * @param {string} [options.commandName]
  * @param {string} [options.usageLine]
+ * @param {{ host?: string, port?: string|number, open?: boolean }} [options.cliOverrides]
  * @returns {{ start: Function, close: Function }}
  */
 function createWatchRuntime(options) {
   const mode = options.mode;
   const workspaceRoot = options.workspaceRoot || process.cwd();
-  const projectName = options.projectName;
   const commandName = options.commandName || mode;
   const usageLine =
-    options.usageLine || `npm run ${commandName} -- <project-name>`;
+    options.usageLine || `jskim ${commandName} [<project>]`;
+  const openBrowserFn = options.openBrowserFn || defaultOpenBrowser;
+  const cliOverrides = {
+    host: options.cliOverrides && options.cliOverrides.host,
+    port: options.cliOverrides && options.cliOverrides.port,
+    open: Boolean(options.cliOverrides && options.cliOverrides.open),
+  };
 
+  /** @type {string|undefined} */
+  let selectedProjectName = options.projectName;
   let currentProject = null;
   let configPath = null;
   let projectWatcher = null;
@@ -45,6 +59,7 @@ function createWatchRuntime(options) {
   let liveReload = null;
   let staticServer = null;
   let liveReloadEnabled = false;
+  let browserOpened = false;
 
   let stopping = false;
   let started = false;
@@ -53,15 +68,31 @@ function createWatchRuntime(options) {
   let pendingConfigReload = false;
   let sourceBuilding = false;
 
+  function withCliOverrides(project) {
+    return applyServeCliOverrides(project, {
+      host: cliOverrides.host,
+      port: cliOverrides.port,
+    });
+  }
+
   function resolveCurrentProject() {
     const loaded = loadConfig(workspaceRoot);
-    const project = resolveProject({
+    const name = selectProjectName({
       config: loaded.config,
-      workspaceRoot,
-      projectName,
+      projectName: selectedProjectName,
       commandName,
       usageLine,
     });
+    selectedProjectName = name;
+    const project = withCliOverrides(
+      resolveProject({
+        config: loaded.config,
+        workspaceRoot,
+        projectName: name,
+        commandName,
+        usageLine,
+      })
+    );
     return {
       project,
       configPath: loaded.configPath,
@@ -118,6 +149,7 @@ function createWatchRuntime(options) {
     const host = project.serve.host.trim();
     const port = project.serve.port;
     const outputDisplay = toDisplayPath(project.outputDir, workspaceRoot);
+    const browserUrl = buildBrowserOpenUrl({ host, port });
 
     liveReload = createLiveReload({
       projectName: project.name,
@@ -154,6 +186,7 @@ function createWatchRuntime(options) {
         host,
         port,
         kind: '開発',
+        commandName: 'dev',
       });
       await cleanupDevComponents();
       throw formatted;
@@ -165,11 +198,30 @@ function createWatchRuntime(options) {
       console.log('[JSKim] 開発サーバーを起動しました。');
       console.log(`プロジェクト: ${project.name}`);
       console.log(`ルート: ${outputDisplay}`);
-      console.log(`URL: ${staticServer.url}`);
+      console.log(`URL: ${browserUrl}`);
       console.log(
         `ライブリロード: ${liveReloadEnabled ? '有効' : '無効'}`
       );
       console.log('終了するには Ctrl+C を押してください。');
+      maybeOpenBrowser(browserUrl);
+    }
+  }
+
+  function maybeOpenBrowser(url) {
+    if (!cliOverrides.open || browserOpened || stopping) {
+      return;
+    }
+    browserOpened = true;
+    const result = openBrowserFn(url);
+    if (!result.ok) {
+      const message =
+        result.error && result.error.message
+          ? result.error.message
+          : String(result.error || 'unknown');
+      console.warn(
+        `[JSKim] browserを開けませんでした。手動で次のURLを開いてください: ${url}\n` +
+          `原因: ${message}`
+      );
     }
   }
 
@@ -314,13 +366,22 @@ function createWatchRuntime(options) {
     let candidate;
     try {
       const loaded = loadConfig(workspaceRoot);
-      candidate = resolveProject({
+      const name = selectProjectName({
         config: loaded.config,
-        workspaceRoot,
-        projectName,
+        projectName: selectedProjectName,
         commandName,
         usageLine,
       });
+      selectedProjectName = name;
+      candidate = withCliOverrides(
+        resolveProject({
+          config: loaded.config,
+          workspaceRoot,
+          projectName: name,
+          commandName,
+          usageLine,
+        })
+      );
     } catch (err) {
       const message = err && err.message ? err.message : String(err);
       console.error('[JSKim] 設定ファイルの再読み込みに失敗しました。');

package/scripts/lib/open-browser.js

@@ -0,0 +1,102 @@
+'use strict';
+
+const { spawn } = require('node:child_process');
+
+/**
+ * browser で開く URL を組み立てます（listen host と分離）。
+ *
+ * @param {{ host: string, port: number }} options
+ * @returns {string}
+ */
+function buildBrowserOpenUrl({ host, port }) {
+  let browserHost = String(host || '').trim();
+  if (browserHost === '0.0.0.0') {
+    browserHost = '127.0.0.1';
+  } else if (browserHost === '::') {
+    browserHost = 'localhost';
+  }
+
+  const portText = String(port);
+  if (browserHost.includes(':') && !browserHost.startsWith('[')) {
+    return `http://[${browserHost}]:${portText}/`;
+  }
+  return `http://${browserHost}:${portText}/`;
+}
+
+/**
+ * OS別の open 用 executable / args を返します（shell 不使用）。
+ *
+ * @param {string} url
+ * @param {string} [platform=process.platform]
+ * @returns {{ command: string, args: string[] }}
+ */
+function buildOpenBrowserCommand(url, platform = process.platform) {
+  if (typeof url !== 'string' || url.trim() === '') {
+    throw new Error('[JSKim] browser URLが空です。');
+  }
+
+  if (platform === 'win32') {
+    return {
+      command: 'rundll32.exe',
+      args: ['url.dll,FileProtocolHandler', url],
+    };
+  }
+
+  if (platform === 'darwin') {
+    return {
+      command: 'open',
+      args: [url],
+    };
+  }
+
+  return {
+    command: 'xdg-open',
+    args: [url],
+  };
+}
+
+/**
+ * browser を開きます。失敗しても throw せず結果を返します。
+ *
+ * @param {string} url
+ * @param {object} [options]
+ * @param {typeof spawn} [options.spawnFn]
+ * @param {string} [options.platform]
+ * @returns {{ ok: boolean, error?: Error, command: string, args: string[] }}
+ */
+function openBrowser(url, options = {}) {
+  const spawnFn = options.spawnFn || spawn;
+  const platform = options.platform || process.platform;
+  const { command, args } = buildOpenBrowserCommand(url, platform);
+
+  try {
+    const child = spawnFn(command, args, {
+      shell: false,
+      stdio: 'ignore',
+      detached: true,
+      windowsHide: true,
+    });
+    if (child && typeof child.unref === 'function') {
+      child.unref();
+    }
+    if (child && typeof child.on === 'function') {
+      child.on('error', () => {
+        // warning は呼び出し側で扱う
+      });
+    }
+    return { ok: true, command, args };
+  } catch (err) {
+    return {
+      ok: false,
+      error: err instanceof Error ? err : new Error(String(err)),
+      command,
+      args,
+    };
+  }
+}
+
+module.exports = {
+  buildBrowserOpenUrl,
+  buildOpenBrowserCommand,
+  openBrowser,
+};

package/scripts/lib/parse-cli-args.js

@@ -0,0 +1,216 @@
+'use strict';
+
+const COMMANDS = new Set(['build', 'watch', 'serve', 'dev']);
+
+const COMMAND_OPTIONS = {
+  build: new Set(['--all']),
+  watch: new Set([]),
+  serve: new Set(['--host', '--port']),
+  dev: new Set(['--host', '--port', '--open']),
+};
+
+const VALUE_OPTIONS = new Set(['--host', '--port']);
+const BOOLEAN_OPTIONS = new Set(['--all', '--open']);
+
+/**
+ * jskim の argv（node / script を除く）を解析します。
+ *
+ * @param {string[]} argv
+ * @returns {{
+ *   kind: 'help'|'version'|'command',
+ *   command?: string,
+ *   projectName?: string,
+ *   options: { all: boolean, open: boolean, host?: string, port?: string }
+ * }}
+ */
+function parseJskimArgv(argv) {
+  const args = Array.isArray(argv) ? argv.slice() : [];
+
+  if (args.length === 0) {
+    return { kind: 'help', options: emptyOptions() };
+  }
+
+  const first = args[0];
+  if (first === '--help' || first === '-h' || first === 'help') {
+    return { kind: 'help', options: emptyOptions() };
+  }
+  if (first === '--version' || first === '-v') {
+    return { kind: 'version', options: emptyOptions() };
+  }
+
+  if (!COMMANDS.has(first)) {
+    throw new Error(formatUnknownCommand(first));
+  }
+
+  const parsed = parseCommandArgv(first, args.slice(1));
+  return {
+    kind: 'command',
+    command: first,
+    projectName: parsed.projectName,
+    options: parsed.options,
+  };
+}
+
+/**
+ * 特定 command の残 argv を解析します（scripts/*.js 用）。
+ *
+ * @param {string} command
+ * @param {string[]} argv command 名を除いた引数、または scripts の process.argv.slice(2)
+ * @returns {{ projectName?: string, options: object }}
+ */
+function parseCommandArgv(command, argv) {
+  if (!COMMANDS.has(command)) {
+    throw new Error(formatUnknownCommand(command));
+  }
+
+  const allowed = COMMAND_OPTIONS[command];
+  const args = Array.isArray(argv) ? argv.slice() : [];
+  const options = emptyOptions();
+  /** @type {string[]} */
+  const positionals = [];
+  /** @type {Set<string>} */
+  const seen = new Set();
+
+  for (let i = 0; i < args.length; i += 1) {
+    const token = args[i];
+
+    if (token === '--') {
+      throw new Error(
+        `[JSKim] サポートされていない引数です: --\n` +
+          `使用方法: jskim ${command} ...`
+      );
+    }
+
+    if (token.startsWith('-')) {
+      if (token.includes('=') && token.startsWith('--')) {
+        throw new Error(
+          `[JSKim] この書き方のoptionはサポートしていません: ${token}\n` +
+            `例: --port 4000（= は使えません）`
+        );
+      }
+
+      if (!BOOLEAN_OPTIONS.has(token) && !VALUE_OPTIONS.has(token)) {
+        throw new Error(
+          `[JSKim] 不明なoptionです: ${token}\n` +
+            formatAllowedOptions(command)
+        );
+      }
+
+      if (!allowed.has(token)) {
+        throw new Error(
+          `[JSKim] コマンド "${command}" ではoption ${token} を使えません。\n` +
+            formatAllowedOptions(command)
+        );
+      }
+
+      if (seen.has(token)) {
+        throw new Error(`[JSKim] optionが重複しています: ${token}`);
+      }
+      seen.add(token);
+
+      if (BOOLEAN_OPTIONS.has(token)) {
+        if (token === '--all') {
+          options.all = true;
+        } else if (token === '--open') {
+          options.open = true;
+        }
+        continue;
+      }
+
+      // value option
+      const value = args[i + 1];
+      if (
+        value === undefined ||
+        (value.startsWith('-') && isKnownOptionToken(value))
+      ) {
+        throw new Error(
+          `[JSKim] option ${token} の値がありません。\n` +
+            `使用方法: ${token} <value>`
+        );
+      }
+      i += 1;
+      if (token === '--host') {
+        options.host = value;
+      } else if (token === '--port') {
+        options.port = value;
+      }
+      continue;
+    }
+
+    positionals.push(token);
+  }
+
+  if (positionals.length > 1) {
+    throw new Error(
+      `[JSKim] project名は1つだけ指定してください。\n` +
+        `受け取った値: ${positionals.join(', ')}\n` +
+        `使用方法: jskim ${command} [<project>]`
+    );
+  }
+
+  if (options.all && positionals.length > 0) {
+    throw new Error(
+      `[JSKim] --all と project名は同時に指定できません。\n` +
+        `使用方法: jskim build --all\n` +
+        `または: jskim build <project>`
+    );
+  }
+
+  if (options.all && command !== 'build') {
+    throw new Error(
+      `[JSKim] コマンド "${command}" ではoption --all を使えません。\n` +
+        formatAllowedOptions(command)
+    );
+  }
+
+  return {
+    projectName: positionals[0],
+    options,
+  };
+}
+
+function emptyOptions() {
+  return {
+    all: false,
+    open: false,
+  };
+}
+
+function isKnownOptionToken(token) {
+  return (
+    BOOLEAN_OPTIONS.has(token) ||
+    VALUE_OPTIONS.has(token) ||
+    token === '--help' ||
+    token === '-h' ||
+    token === '--version' ||
+    token === '-v'
+  );
+}
+
+function formatAllowedOptions(command) {
+  const allowed = [...COMMAND_OPTIONS[command]];
+  if (allowed.length === 0) {
+    return `コマンド "${command}" で使える追加optionはありません。`;
+  }
+  return `使えるoption: ${allowed.join(', ')}`;
+}
+
+function formatUnknownCommand(command) {
+  return [
+    `[JSKim] 不明なコマンドです: ${command}`,
+    '',
+    '使用できるコマンド:',
+    '  build [<project>]',
+    '  build --all',
+    '  watch [<project>]',
+    '  serve [<project>] [--host <host>] [--port <port>]',
+    '  dev [<project>] [--host <host>] [--port <port>] [--open]',
+  ].join('\n');
+}
+
+module.exports = {
+  parseJskimArgv,
+  parseCommandArgv,
+  COMMANDS,
+  COMMAND_OPTIONS,
+};