mdv-live 0.5.10 → 0.5.11

package/CHANGELOG.md

@@ -5,6 +5,23 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.5.11] - 2026-05-09
+
+### Fixed
+
+- **Marp PDF export ENOENT in fresh install** (実は 0.5.5 から潜在):
+  - `src/api/pdf.js` が marp 実行ファイルを
+    `node_modules/mdv-live/node_modules/.bin/marp` で解決していたが、npm
+    hoisting により実体は top-level の `node_modules/.bin/marp` にある
+  - dev 環境 (mdv-live リポ内) では nested の方が存在するため気づかず、
+    `npm install mdv-live` した fresh install では ENOENT
+  - `require.resolve('@marp-team/marp-cli/package.json')` から bin スクリプト
+    を解決し `node` で実行する方式に変更 (hoist/nest 両対応)
+
+### Tests
+
+- 242 → **243 件 (+1)**: marp-cli bin entry の実在チェック regression test
+
 ## [0.5.10] - 2026-05-09
 
 ### Fixed (UX revert)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mdv-live",
-  "version": "0.5.10",
+  "version": "0.5.11",
   "description": "Markdown Viewer - File tree + Live preview + Marp support + Hot reload",
   "main": "src/server.js",
   "bin": {

package/src/api/pdf.js

@@ -1,19 +1,25 @@
 import { spawn } from 'child_process';
 import fs from 'fs/promises';
 import path from 'path';
-import { fileURLToPath } from 'url';
 import os from 'os';
+import { createRequire } from 'module';
 import { isMarp } from '../rendering/markdown.js';
 import { validatePath } from '../utils/path.js';
 
-const marpBin = path.join(
-  path.dirname(fileURLToPath(import.meta.url)),
-  '..',
-  '..',
-  'node_modules',
-  '.bin',
-  'marp'
-);
+const require = createRequire(import.meta.url);
+// marp-cli は npm hoisting によりインストール先が変わる (top-level / nested)。
+// require.resolve でパッケージの実体を特定し、その bin スクリプトを node で
+// 直接実行する。`node_modules/.bin/marp` 直叩きは fresh install で nested
+// パスが無い時に ENOENT する罠。
+const marpEntry = (() => {
+  const pkgPath = require.resolve('@marp-team/marp-cli/package.json');
+  const pkg = require('@marp-team/marp-cli/package.json');
+  const binRel = typeof pkg.bin === 'string' ? pkg.bin : pkg.bin?.marp;
+  if (!binRel) {
+    throw new Error('@marp-team/marp-cli does not declare a "marp" bin entry');
+  }
+  return path.join(path.dirname(pkgPath), binRel);
+})();
 const PDF_EXPORT_TIMEOUT_MS = 180000;
 
 /**
@@ -23,7 +29,9 @@ const PDF_EXPORT_TIMEOUT_MS = 180000;
  */
 function runMarp(args) {
   return new Promise((resolve, reject) => {
-    const child = spawn(marpBin, args, { stdio: ['ignore', 'pipe', 'pipe'] });
+    const child = spawn(process.execPath, [marpEntry, ...args], {
+      stdio: ['ignore', 'pipe', 'pipe'],
+    });
     let stderr = '';
     child.stderr.on('data', (chunk) => { stderr += chunk.toString(); });