sparkle-design-cli 2.0.7-beta.0 → 2.0.7-beta.10

package/lib/font-manager.js

@@ -5,7 +5,7 @@
 
 import fs from 'fs';
 import path from 'path';
-import { REGEX, COMMENTS, IMPORTS, MESSAGES } from './constants.js';
+import { REGEX, COMMENTS, IMPORTS, MESSAGES, GLOBALS_CSS_CANDIDATES } from './constants.js';
 
 /**
  * sparkle-design.css からフォントの@import文を抽出する
@@ -58,8 +58,8 @@ function resolveNodeModulesRelPath(globalsPath) {
  */
 function createSourceBlock(sourcePackages = [], nodeModulesRel = '../node_modules') {
   const defaultPackage = 'sparkle-design';
-  const allPackages = [defaultPackage, ...sourcePackages.filter(p => p !== defaultPackage)];
-  const sourceLines = allPackages.map(pkg => `@source "${nodeModulesRel}/${pkg}/dist";`);
+  const allPackages = [defaultPackage, ...sourcePackages.filter((p) => p !== defaultPackage)];
+  const sourceLines = allPackages.map((pkg) => `@source "${nodeModulesRel}/${pkg}/dist";`);
   return [COMMENTS.SOURCE_DIRECTIVE, ...sourceLines].join('\n');
 }
 
@@ -88,22 +88,64 @@ function createCustomCssImportBlock(customCssPath, globalsPath) {
   return [COMMENTS.CUSTOM_CSS, `@import "${relativePath}";`].join('\n');
 }
 
+/**
+ * entry CSS (globals.css / index.css) から sparkle-design.css への相対 path を
+ * 計算し、`@import` 文を組み立てる。同一ディレクトリなら `./sparkle-design.css`、
+ * 異なるなら `./app/sparkle-design.css` のような相対 path になる。Vite の
+ * `src/index.css` → `src/app/sparkle-design.css` レイアウトで正しい path が
+ * 出るようにするために追加された関数（beta.7 まではハードコード）。
+ * en: Compute the relative path from the entry CSS to sparkle-design.css.
+ * Handles layouts like Vite's `src/index.css` → `src/app/sparkle-design.css`,
+ * where the two files live in different directories.
+ */
+function buildSparkleImportStatement(globalsPath, sparkleDesignPath) {
+  if (!globalsPath || !sparkleDesignPath) {
+    return IMPORTS.SPARKLE_DESIGN;
+  }
+  const globalsDir = path.dirname(globalsPath);
+  const rel = path.relative(globalsDir, sparkleDesignPath).split(path.sep).join('/');
+  // path.relative が空文字を返すのは globalsPath === sparkleDesignPath のときのみ
+  // （想定外）。念のため fallback として同一 dir 前提の default import を返す。
+  // en: path.relative returns '' only when paths match; use the same-dir default.
+  if (!rel) return IMPORTS.SPARKLE_DESIGN;
+  const importPath = rel.startsWith('.') ? rel : `./${rel}`;
+  return `@import "${importPath}";`;
+}
+
 /**
  * sparkle-design.css importブロックを生成する
  * @param {Array<string>} sourcePackages 追加パッケージ名の配列
  * @param {string} globalsPath globals.css の絶対パス（@source パス計算用）
  * @param {string|null} customCssPath custom-css ファイルの相対パス
+ * @param {string|null} sparkleDesignPath sparkle-design.css の絶対パス（相対 import 計算用）
  * @returns {string} sparkle-design.css importブロック
  */
-function createSparkleImportBlock(sourcePackages = [], globalsPath = null, customCssPath = null) {
+function createSparkleImportBlock(
+  sourcePackages = [],
+  globalsPath = null,
+  customCssPath = null,
+  sparkleDesignPath = null
+) {
   const parts = [''];
 
-  if (sourcePackages !== null && sourcePackages !== undefined) {
-    const nodeModulesRel = globalsPath ? resolveNodeModulesRelPath(globalsPath) : '../node_modules';
-    parts.push(createSourceBlock(sourcePackages, nodeModulesRel));
-  }
-
-  parts.push(COMMENTS.SPARKLE_IMPORT, IMPORTS.SPARKLE_DESIGN);
+  // CSS 仕様上 `@import` は `@charset` / `@layer` 以外の at-rule より前に
+  // 書く必要がある。`@source` を `@import` より前に置くと、PostCSS 処理系に
+  // よっては後続の `@import "./sparkle-design.css"` が silent に drop され、
+  // sparkle tokens が最終 CSS に出ない回帰になる（beta.6 以前で再現）。
+  // そのため順序は:
+  //   1. @import "./sparkle-design.css"   （Sparkle 本体のトークン）
+  //   2. @import "./custom-tokens.css"    （ユーザーのカスタムトークン、任意）
+  //   3. @source "../node_modules/..."    （Tailwind スキャン対象、任意）
+  // の順で出力する。
+  // en: Per CSS spec, all `@import` rules must precede any other at-rule
+  // (except `@charset` / `@layer`). Placing `@source` between imports makes
+  // downstream `@import "./sparkle-design.css"` silently drop in some
+  // PostCSS toolchains, which removes Sparkle tokens from the output.
+  // We now emit @import blocks first, then any @source directives.
+  parts.push(
+    COMMENTS.SPARKLE_IMPORT,
+    buildSparkleImportStatement(globalsPath, sparkleDesignPath)
+  );
 
   // カスタムCSS import を sparkle-design.css の後に配置
   const customBlock = createCustomCssImportBlock(customCssPath, globalsPath);
@@ -111,6 +153,11 @@ function createSparkleImportBlock(sourcePackages = [], globalsPath = null, custo
     parts.push(customBlock);
   }
 
+  if (sourcePackages !== null && sourcePackages !== undefined) {
+    const nodeModulesRel = globalsPath ? resolveNodeModulesRelPath(globalsPath) : '../node_modules';
+    parts.push(createSourceBlock(sourcePackages, nodeModulesRel));
+  }
+
   parts.push('');
   return parts.join('\n');
 }
@@ -126,8 +173,14 @@ function removeExistingImports(globalsContent) {
   // 既存のフォントimportを削除
   cleaned = cleaned.replace(REGEX.EXISTING_FONT_IMPORT_BLOCK, '');
 
-  // 既存の @source ディレクティブを削除
-  cleaned = cleaned.replace(REGEX.SOURCE_DIRECTIVE, '');
+  // CLI が過去に書き込んだ「コメント + @source 連続行」ブロックのみを削除する。
+  // ユーザーが手書きした `@source "..."` は保持する（以前は SOURCE_DIRECTIVE
+  // を単体で全削除していたため、手書き @source が消える退行があった）。
+  // en: Remove only the managed comment + @source block that the CLI itself
+  // wrote in the past. User-authored @source lines must survive this step.
+  cleaned = cleaned.replace(REGEX.MANAGED_SOURCE_BLOCK, '');
+  // 念のため CLI コメント単独で孤立している残骸も掃除。
+  // en: Sweep up any orphan comment lines left over from historical writes.
   cleaned = cleaned.replace(REGEX.SOURCE_COMMENT, '');
 
   // 既存のsparkle-design.css importを削除
@@ -174,28 +227,47 @@ function reconstructGlobalsCss(globalsContent, sparkleImportBlock, tailwindInfo)
   const afterTailwind = globalsContent.substring(tailwindInfo.afterIndex);
 
   // Tailwind + sparkle-design.css + 残りのコンテンツ（フォント @import なし）
-  return (
-    beforeTailwind +
-    tailwindInfo.match +
-    sparkleImportBlock +
-    afterTailwind.trimStart()
-  );
+  return beforeTailwind + tailwindInfo.match + sparkleImportBlock + afterTailwind.trimStart();
 }
 
 /**
  * globals.css を構造化して管理する
- * - フォントimportを先頭に配置
+ * - フォントimportを先頭に配置（v2.0.0 以降は SparkleHead.tsx に移行したため空配列が渡る）
  * - Tailwind importを確保
  * - sparkle-design.css importをTailwindの後に配置
+ * - 自動検出した sourcePackages から @source ディレクティブを挿入
+ *
+ * **v2.0.7-beta.2**: 戻り値を真偽値から `{ status, reason }` の状態付きに変更。
+ * 呼び出し側が「作業不要でスキップ」「失敗したがデフォルトは warn 継続」を
+ * 区別できるようにし、`--strict` モードで失敗を exit code に反映できるよう
+ * にした（#31）。`skipped` は後方互換のため `false` に、`updated` は `true`
+ * に toBoolean で扱える想定。
+ *
  * @param {Array<string>} fontImports フォントimport文の配列
  * @param {string} globalsPath globals.cssのパス
  * @param {Array<string>|null} sourcePackages 追加パッケージ名の配列（null の場合は @source を生成しない）
  * @param {string|null} customCssPath custom-css ファイルの相対パス
- * @returns {boolean} globals.css の更新に成功した場合は true
+ * @returns {{ status: 'skipped' | 'updated' | 'failed', reason?: string }}
+ *   skipped: 触る理由がない (fonts/sourcePackages/customCss すべて空)
+ *   updated: globals.css を正常に書き換えた
+ *   failed: 書き換え対象だったが失敗した (TAILWIND_IMPORT 欠落、write 失敗等)
  */
-export function updateGlobalsWithFonts(fontImports, globalsPath, sourcePackages = null, customCssPath = null) {
-  if (fontImports.length === 0) {
-    return false;
+export function updateGlobalsWithFonts(
+  fontImports,
+  globalsPath,
+  sourcePackages = null,
+  customCssPath = null,
+  sparkleDesignPath = null
+) {
+  const hasFonts = fontImports.length > 0;
+  const hasSourcePackages = sourcePackages !== null && sourcePackages !== undefined;
+  const hasCustomCss = Boolean(customCssPath);
+
+  // フォント import も @source も custom-css もないなら globals.css に触る
+  // 理由がないので skip。
+  // en: Nothing to inject, so leave globals.css alone.
+  if (!hasFonts && !hasSourcePackages && !hasCustomCss) {
+    return { status: 'skipped', reason: 'no-work' };
   }
 
   try {
@@ -205,15 +277,42 @@ export function updateGlobalsWithFonts(fontImports, globalsPath, sourcePackages
     // 2. 既存のimportを削除
     globalsContent = removeExistingImports(globalsContent);
 
-    // 3. Tailwind import の位置を見つける
-    const tailwindInfo = findTailwindImport(globalsContent);
+    // 3. Tailwind import の位置を見つける。無ければ先頭に自動で prepend する。
+    //    ここで silent に失敗すると `create-vite` 等が生成した既存 index.css に
+    //    `@import "tailwindcss"` が無いプロジェクトで setup が no-op に終わり、
+    //    AI が後から手書きで import を補うが `@source` を知らず落とす、という
+    //    連鎖的な崩れが起きていた（beta.8 試用で確認）。canonical な import を
+    //    自動投入することでその連鎖を根絶する。
+    // en: If `@import "tailwindcss"` is missing from an existing entry CSS
+    // (e.g. untouched `create-vite` default), auto-prepend it instead of
+    // silently warning. Without this recovery, the whole patch becomes a no-op
+    // and AI ends up hand-writing imports without `@source`, producing drifted
+    // output (observed in beta.8 user testing).
+    let tailwindInfo = findTailwindImport(globalsContent);
     if (!tailwindInfo) {
-      console.warn(MESSAGES.TAILWIND_NOT_FOUND);
-      return false;
+      const displayPath = path.relative(process.cwd(), globalsPath) || globalsPath;
+      console.warn(
+        `⚠️  ${displayPath} に \`@import "tailwindcss";\` が見つからなかったため先頭に追記します（canonical な Tailwind v4 entry point にするため）。`
+      );
+      globalsContent = `${IMPORTS.TAILWIND}\n${globalsContent.startsWith('\n') ? '' : ''}${globalsContent}`;
+      tailwindInfo = findTailwindImport(globalsContent);
+      if (!tailwindInfo) {
+        // prepend したのに正規表現で拾えないのは `REGEX.TAILWIND_IMPORT` が
+        // 想定と違う（= コードの不整合）。fail-fast する。
+        // en: We just wrote the canonical import but the regex doesn't see it —
+        // this is an internal inconsistency, not a user error. Fail loud.
+        console.error(MESSAGES.TAILWIND_NOT_FOUND(displayPath));
+        return { status: 'failed', reason: 'tailwind-import-missing-after-prepend' };
+      }
     }
 
     // 4. importブロックを生成（フォント @import は SparkleHead に移行したため生成しない）
-    const sparkleImportBlock = createSparkleImportBlock(sourcePackages, globalsPath, customCssPath);
+    const sparkleImportBlock = createSparkleImportBlock(
+      sourcePackages,
+      globalsPath,
+      customCssPath,
+      sparkleDesignPath
+    );
 
     // 5. globals.css を再構築（フォント @import なし）
     const reconstructedContent = reconstructGlobalsCss(
@@ -225,14 +324,45 @@ export function updateGlobalsWithFonts(fontImports, globalsPath, sourcePackages
     // 6. 更新したglobals.cssを書き込む
     fs.writeFileSync(globalsPath, reconstructedContent, 'utf8');
     console.log(MESSAGES.GLOBALS_UPDATED(globalsPath));
-    return true;
+    return { status: 'updated' };
   } catch (error) {
     console.error(MESSAGES.GLOBALS_UPDATE_FAILED(error.message));
-    // globals.cssの更新は必須ではないので、エラーでも処理を続行
-    return false;
+    return { status: 'failed', reason: `write-error: ${error.code ?? error.message}` };
   }
 }
 
+// Vite プロジェクト判定用の config 候補。setup.js 側の `VITE_CONFIG_FILES` と
+// 同じ対象を見る必要がある（判定ロジックが分かれると scaffold と generate で
+// 挙動が食い違う）。
+// en: Vite config candidates used for project detection. Must stay in sync with
+// setup.js's VITE_CONFIG_FILES so scaffold and generate agree on the layout.
+const VITE_CONFIG_FILES = [
+  'vite.config.ts',
+  'vite.config.js',
+  'vite.config.mjs',
+  'vite.config.cjs',
+  'vite.config.mts',
+  'vite.config.cts',
+];
+
+function isViteProject(cwd) {
+  return VITE_CONFIG_FILES.some((name) => fs.existsSync(path.join(cwd, name)));
+}
+
+// Vite プロジェクトで候補をソートするときの優先度。setup の
+// `defaultGlobalsCssTarget` に合わせて `src/index.css` を `src/globals.css`
+// より手前に引き上げる。それ以外は元の順序（数字昇順）を保つ。
+// en: When the project is Vite, promote `src/index.css` ahead of
+// `src/globals.css` so that `generate` patches the same file `setup` scaffolds.
+function vitePriority(candidate) {
+  if (candidate === 'src/index.css') return 0;
+  if (candidate === 'src/globals.css') return 1;
+  // それ以外の候補は元の GLOBALS_CSS_CANDIDATES 順序を保つため、index を
+  // オフセット付きで返す。
+  // en: Preserve the original order for everything else.
+  return 10 + GLOBALS_CSS_CANDIDATES.indexOf(candidate);
+}
+
 /**
  * sparkle-design.css と同じディレクトリで Tailwind のエントリポイント CSS を自動検出する
  * @param {string} dir 検索対象ディレクトリ
@@ -242,7 +372,16 @@ function detectTailwindEntrypoint(dir) {
   let entries;
   try {
     entries = fs.readdirSync(dir, { withFileTypes: true });
-  } catch {
+  } catch (err) {
+    // ENOENT / ENOTDIR はよくある（dir 自体が無い / ファイルを渡された）ので
+    // 静かにスキップ。権限 (EACCES) などは debugging 価値があるので表に出す。
+    // en: ENOENT / ENOTDIR are expected (dir missing / path is a file). Surface
+    // permission-style failures so the user can debug.
+    if (err.code !== 'ENOENT' && err.code !== 'ENOTDIR') {
+      console.warn(
+        `⚠️  ${dir} の読み込みに失敗したため同階層 detection をスキップします (${err.code ?? err.message})`
+      );
+    }
     return null;
   }
 
@@ -251,9 +390,20 @@ function detectTailwindEntrypoint(dir) {
     if (entry.name === 'sparkle-design.css') continue;
 
     const filePath = path.join(dir, entry.name);
-    const content = fs.readFileSync(filePath, 'utf8');
-    if (REGEX.TAILWIND_IMPORT.test(content)) {
-      return filePath;
+    try {
+      const content = fs.readFileSync(filePath, 'utf8');
+      if (REGEX.TAILWIND_IMPORT.test(content)) {
+        return filePath;
+      }
+    } catch (err) {
+      // 1 候補が読めなくても他の候補で続行できるようログを出してスキップする。
+      // 以前はここで throw させて外側の catch に落としていたため、同階層に
+      // 壊れた CSS が 1 つあると全部の detection が死んでいた。
+      // en: Keep scanning even if a single candidate file fails to read —
+      // previously a single unreadable CSS poisoned the whole detection.
+      console.warn(
+        `⚠️  ${entry.name} の読み込みに失敗したためスキップします (${err.code ?? err.message})`
+      );
     }
   }
 
@@ -262,10 +412,16 @@ function detectTailwindEntrypoint(dir) {
 
 /**
  * globals パスを解決する
- * 優先順位: 明示的指定 > 自動検出 > デフォルト（globals.css）
+ * 優先順位: 明示的指定 > sparkle-design.css 同階層で自動検出 > プロジェクト root の既知候補 > デフォルト（globals.css）
+ *
+ * **v2.0.7-beta.1**: Vite プロジェクトのように `sparkle-design.css` を `src/app/`
+ * に、entry CSS を `src/index.css` に置くレイアウトで `@source` が挿入されない
+ * 不具合があったため、同階層で見つからない場合はプロジェクト root からも
+ * 既知の候補を探すように拡張した（setup が scaffold に使う候補と同一）。
+ *
  * @param {string} sparkleDesignPath sparkle-design.css のパス
  * @param {string|null} explicitGlobalsPath 明示的に指定された globals パス
- * @returns {{ path: string, source: 'explicit' | 'detected' | 'default' } | null}
+ * @returns {{ path: string, source: 'explicit' | 'detected' | 'project' | 'default' } | null}
  */
 function resolveGlobalsPath(sparkleDesignPath, explicitGlobalsPath = null) {
   const dir = path.dirname(sparkleDesignPath);
@@ -275,11 +431,23 @@ function resolveGlobalsPath(sparkleDesignPath, explicitGlobalsPath = null) {
     if (fs.existsSync(resolved)) {
       return { path: resolved, source: 'explicit' };
     }
-    console.warn(`⚠️  指定された globals パスが見つかりません: ${explicitGlobalsPath}`);
-    return null;
+    // 明示指定は settings bug / typo なので silent warn ではなく throw に
+    // 昇格させる。--strict の有無に関わらず呼び出し元で伝播させたいので、
+    // error.code を付けて上流で識別できるようにする。
+    // en: An explicit --globals-path value that doesn't exist is almost
+    // always a typo. We tag the error so manageFontImports's catch knows
+    // to re-throw regardless of strict mode.
+    const err = new Error(
+      `指定された globals パスが見つかりません: ${explicitGlobalsPath} (cwd 基準で解決: ${resolved})`
+    );
+    err.code = 'E_EXPLICIT_GLOBALS_PATH_NOT_FOUND';
+    throw err;
   }
 
-  // 自動検出: @import "tailwindcss" を含む CSS ファイルを探す
+  // 1. 自動検出: sparkle-design.css と同じディレクトリで @import "tailwindcss" を含む
+  //    CSS ファイルを探す。Next.js App Router のように両者が同階層に居るケース用。
+  // en: Look next to sparkle-design.css for a Tailwind entry CSS. Covers the
+  // Next.js App Router layout where both live under `src/app/`.
   const detected = detectTailwindEntrypoint(dir);
   if (detected) {
     const basename = path.basename(detected);
@@ -289,7 +457,64 @@ function resolveGlobalsPath(sparkleDesignPath, explicitGlobalsPath = null) {
     return { path: detected, source: 'detected' };
   }
 
-  // デフォルト: globals.css
+  // 2. プロジェクト root から既知の候補を探す。Vite のように sparkle-design.css
+  //    が src/app/ に、entry CSS が src/index.css にあるレイアウトに対応。
+  //    Vite プロジェクトでは `setup` の scaffold が `src/index.css` を選ぶので、
+  //    `src/globals.css` と両方存在していても index.css を優先する（scaffold
+  //    した場所と generate が patch する場所が食い違わないように）。
+  // en: Fall back to project-wide candidates. For Vite projects, promote
+  // `src/index.css` above `src/globals.css` so that `generate` patches the same
+  // file `setup` would have scaffolded.
+  const cwd = process.cwd();
+  const candidates = isViteProject(cwd)
+    ? GLOBALS_CSS_CANDIDATES.slice().sort((a, b) => vitePriority(a) - vitePriority(b))
+    : GLOBALS_CSS_CANDIDATES;
+  // 2-a) まず既知候補の中で `@import "tailwindcss"` を持つファイルを探す。
+  //       持っているファイルが第一候補（canonical Tailwind entry CSS）。
+  // 2-b) 2-a で見つからなかった場合、`@import` 無しで既存の候補ファイルが
+  //       あればそれを拾う（Vite の `create-vite` 直後のように entry CSS は
+  //       あるが Tailwind 未設定のプロジェクト）。これで上流の
+  //       `updateGlobalsWithFonts` が auto-prepend で tailwind import を
+  //       補えるため、silent に no-op 終了する回帰を避けられる（beta.8 で再現）。
+  // en: First pick candidates that already contain `@import "tailwindcss"`.
+  // If none match, fall back to the first existing candidate anyway so the
+  // upstream patcher can auto-prepend the canonical tailwind import. Without
+  // this fallback, a Vite project created by `create-vite` with an untouched
+  // `src/index.css` would cause setup to end as a silent no-op.
+  let fallbackExistingPath = null;
+  for (const candidate of candidates) {
+    const absolute = path.resolve(cwd, candidate);
+    if (!fs.existsSync(absolute)) continue;
+    if (path.resolve(absolute) === path.resolve(sparkleDesignPath)) continue;
+    try {
+      const content = fs.readFileSync(absolute, 'utf8');
+      if (REGEX.TAILWIND_IMPORT.test(content)) {
+        console.log(`📝 Tailwind エントリポイントを検出しました（project root）: ${candidate}`);
+        return { path: absolute, source: 'project' };
+      }
+      if (!fallbackExistingPath) {
+        fallbackExistingPath = { path: absolute, candidate };
+      }
+    } catch (err) {
+      // ENOENT 系は existsSync で既に弾いているので、ここに来るのは権限不足や
+      // ディレクトリ衝突などデバッグ価値のあるケース。silent に消さずに理由を出す。
+      // en: ENOENT is already filtered by existsSync above, so surfacing the
+      // error code here catches permission / type issues worth debugging.
+      console.warn(
+        `⚠️  候補 ${candidate} の読み込みに失敗したためスキップします (${err.code ?? err.message})`
+      );
+    }
+  }
+
+  if (fallbackExistingPath) {
+    console.log(
+      `📝 Tailwind import 未設定の既存 entry CSS を検出しました（project root）: ${fallbackExistingPath.candidate}。後続処理で \`@import "tailwindcss";\` を自動追記します。`
+    );
+    return { path: fallbackExistingPath.path, source: 'existing-no-tailwind' };
+  }
+
+  // 3. デフォルト: sparkle-design.css と同じディレクトリの globals.css
+  // en: Last resort — a globals.css next to sparkle-design.css.
   const defaultPath = path.join(dir, 'globals.css');
   if (fs.existsSync(defaultPath)) {
     return { path: defaultPath, source: 'default' };
@@ -301,19 +526,54 @@ function resolveGlobalsPath(sparkleDesignPath, explicitGlobalsPath = null) {
 /**
  * フォント管理の自動処理を実行する
  * sparkle-design.css からフォントimportを抽出し、Tailwind エントリポイント CSS に @source 等を挿入する
+ *
+ * **v2.0.7-beta.2**: 戻り値を `{ status, reason? }` に変更し、呼び出し側が
+ * 「作業不要のスキップ」「作業すべき状態だが失敗」を区別できるようにした
+ * (#31)。`options.strict` が true のとき、`status === 'failed'` になる状態は
+ * throw に昇格させて `bin/sparkle-design.js` の outer catch で exit 1 に
+ * 繋げる。
+ *
  * @param {string} sparkleDesignPath sparkle-design.cssのパス
  * @param {Array<string>|null} sourcePackages 追加パッケージ名の配列（null の場合は @source を生成しない）
  * @param {string|null} customCssPath custom-css ファイルの相対パス
  * @param {string|null} globalsPathOverride 明示的に指定された globals パス
+ * @param {{ strict?: boolean }} [options] strict=true のとき失敗を throw する
+ * @returns {{ status: 'skipped' | 'updated' | 'failed', reason?: string }}
  */
-export function manageFontImports(sparkleDesignPath, sourcePackages = null, customCssPath = null, globalsPathOverride = null) {
+export function manageFontImports(
+  sparkleDesignPath,
+  sourcePackages = null,
+  customCssPath = null,
+  globalsPathOverride = null,
+  options = {}
+) {
+  const strict = Boolean(options.strict);
+  const hasWork =
+    (sourcePackages !== null && sourcePackages !== undefined) || Boolean(customCssPath);
+
+  const raiseOrReturn = (result) => {
+    if (strict && result.status === 'failed') {
+      throw new Error(
+        `globals.css の更新に失敗しました (${result.reason ?? 'unknown'})。--strict モードでは exit 1 で終了します。`
+      );
+    }
+    return result;
+  };
+
   try {
     // 1. globals パスを解決
     const resolved = resolveGlobalsPath(sparkleDesignPath, globalsPathOverride);
 
     if (!resolved) {
+      // 作業すべき状態（sourcePackages 等あり）で entry CSS が見つからないのは
+      // 事実上の不具合。strict なら throw、非 strict なら従来どおり情報ログ。
+      // en: With work queued, missing entry CSS is effectively a misconfiguration.
+      if (hasWork) {
+        console.warn(MESSAGES.GLOBALS_NOT_FOUND);
+        return raiseOrReturn({ status: 'failed', reason: 'entry-css-not-found' });
+      }
       console.log(MESSAGES.GLOBALS_NOT_FOUND);
-      return;
+      return { status: 'skipped', reason: 'entry-css-not-found-no-work' };
     }
 
     const globalsPath = resolved.path;
@@ -322,27 +582,57 @@ export function manageFontImports(sparkleDesignPath, sourcePackages = null, cust
     const sparkleContent = fs.readFileSync(sparkleDesignPath, 'utf8');
     const fontImports = extractFontImports(sparkleContent);
 
-    if (fontImports.length === 0) {
-      console.log(MESSAGES.NO_FONT_IMPORTS);
-      return;
+    // v2.0.0 以降はフォント @import が SparkleHead.tsx に移行したため、
+    // sparkle-design.css に font @import が残らないのが通常状態。それでも
+    // 既存 globals.css には `@source` / sparkle-design.css import /
+    // custom-css import を挿入する必要があるので、単に fonts が空だからと
+    // いって早期 return せず、下流の update 関数に判断を委ねる。
+    // en: Fonts moved to SparkleHead in v2.0.0, so "no font imports in
+    // sparkle-design.css" is now the normal case. Don't bail out here —
+    // `updateGlobalsWithFonts` still needs to inject `@source` and the
+    // sparkle-design.css import into existing globals.css.
+    if (fontImports.length > 0) {
+      console.log(MESSAGES.FONT_DETECTED(fontImports.length));
     }
 
-    console.log(MESSAGES.FONT_DETECTED(fontImports.length));
-
-    // 4. globals.css にフォントimportを追加
-    const globalsUpdated = updateGlobalsWithFonts(fontImports, globalsPath, sourcePackages, customCssPath);
+    // 4. globals.css に import / @source を追加
+    // sparkle-design.css の絶対パスを渡すことで、globalsPath と異なる
+    // ディレクトリに置かれている場合でも相対 import が正しく計算される
+    // （Vite の src/index.css → src/app/sparkle-design.css 等）。
+    // en: Pass the absolute sparkle-design.css path so the import can be
+    // computed relative to the entry CSS, even in Vite-style layouts where
+    // the two files live in different directories.
+    const result = updateGlobalsWithFonts(
+      fontImports,
+      globalsPath,
+      sourcePackages,
+      customCssPath,
+      sparkleDesignPath
+    );
 
-    if (!globalsUpdated) {
-      console.log(MESSAGES.FONT_REMOVE_SKIPPED);
-      return;
+    if (result.status === 'skipped') {
+      return result;
+    }
+    if (result.status === 'failed') {
+      return raiseOrReturn(result);
     }
 
-    // 5. sparkle-design.css からフォントimportを削除
-    const cleanedSparkleContent = removeFontImportsFromCSS(sparkleContent);
-    fs.writeFileSync(sparkleDesignPath, cleanedSparkleContent, 'utf8');
-    console.log(MESSAGES.FONT_REMOVED);
+    // 5. fonts が sparkle-design.css 側に残っていれば削除（globals 側に移動済みの前提）
+    // en: If there were fonts to move, strip them from sparkle-design.css.
+    if (fontImports.length > 0) {
+      const cleanedSparkleContent = removeFontImportsFromCSS(sparkleContent);
+      fs.writeFileSync(sparkleDesignPath, cleanedSparkleContent, 'utf8');
+      console.log(MESSAGES.FONT_REMOVED);
+    }
+    return result;
   } catch (error) {
     console.error(MESSAGES.FONT_MANAGEMENT_ERROR(error.message));
-    // フォント管理は必須ではないので、エラーでも処理を続行
+    // strict モード、または明示指定された globals path の not-found は
+    // ユーザーが必ず気付くべきなので非 strict でも再 throw する。
+    // en: Always re-throw explicit --globals-path typos; respect strict for the rest.
+    if (strict || error.code === 'E_EXPLICIT_GLOBALS_PATH_NOT_FOUND') {
+      throw error;
+    }
+    return { status: 'failed', reason: `manage-error: ${error.code ?? error.message}` };
   }
 }