sparkle-design-cli 2.2.2 → 2.2.3

package/lib/constants.js

@@ -71,8 +71,19 @@ export const REGEX = {
   // prefix and a path separator just before the filename. This preserves user-
   // authored files like `custom-sparkle-design.css` that would otherwise be
   // deleted by a loose suffix match.
+  //
+  // ※ path prefix は `(?:\.{1,2}\/(?:[^"']*\/)?)?` で「`./` または `../` +
+  // 任意の中間ディレクトリ（省略可）」を表す。以前は中間ディレクトリ部分が
+  // 必須（`(?:\.{1,2}\/)[^"']*\/`）だったため、CLI 自身が書く正規形
+  // `@import "./sparkle-design.css"`（中間ディレクトリなし）にマッチせず、
+  // generate を再実行するたびに import が重複挿入されていた（issue #59）。
+  // en: The path prefix `(?:\.{1,2}\/(?:[^"']*\/)?)?` means an optional
+  // `./` / `../` followed by optional intermediate directories. The previous
+  // pattern required a second slash, so the CLI's own canonical output
+  // `@import "./sparkle-design.css"` never matched and every re-run of
+  // `generate` inserted a duplicate import (issue #59).
   SPARKLE_IMPORT:
-    /\/\*\s*Sparkle Design[^*]*\*\/\s*\n?|@import\s+['"](?:(?:\.{1,2}\/)[^"']*\/)?sparkle-design\.css['"];?\s*\n?/g,
+    /\/\*\s*Sparkle Design[^*]*\*\/\s*\n?|@import\s+['"](?:\.{1,2}\/(?:[^"']*\/)?)?sparkle-design\.css['"];?\s*\n?/g,
   TAILWIND_IMPORT: /@import\s+['"]tailwindcss['"];?/,
 
   // @source ディレクティブ関連

package/lib/font-manager.js

@@ -154,10 +154,7 @@ function createSparkleImportBlock(
   // 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)
-  );
+  parts.push(COMMENTS.SPARKLE_IMPORT, buildSparkleImportStatement(globalsPath, sparkleDesignPath));
 
   // カスタムCSS import を sparkle-design.css の後に配置
   const customBlock = createCustomCssImportBlock(customCssPath, globalsPath);
@@ -285,7 +282,13 @@ export function updateGlobalsWithFonts(
 
   try {
     // 1. globals.css を読み込む
-    let globalsContent = fs.readFileSync(globalsPath, 'utf8');
+    // 再構築結果との比較用に元の内容を保持する（issue #59: 変更がない場合は
+    // 書き込まず no-op にして、生成物をコミット管理しているリポジトリで
+    // 毎回ノイズ diff（mtime 更新・formatter との競合）が出ないようにする）。
+    // en: Keep the original content so we can skip the write when the
+    // reconstruction is byte-identical (issue #59 idempotency).
+    const originalContent = fs.readFileSync(globalsPath, 'utf8');
+    let globalsContent = originalContent;
 
     // 2. 既存のimportを削除
     globalsContent = removeExistingImports(globalsContent);
@@ -340,7 +343,13 @@ export function updateGlobalsWithFonts(
       tailwindInfo
     );
 
-    // 6. 更新したglobals.cssを書き込む
+    // 6. 再構築結果が元の内容と同一なら書き込まない（冪等 no-op、issue #59）
+    // en: Skip the write when nothing would change (idempotent no-op, #59).
+    if (reconstructedContent === originalContent) {
+      return { status: 'skipped', reason: 'already-up-to-date', globalsPath, sourcePackages };
+    }
+
+    // 7. 更新したglobals.cssを書き込む
     fs.writeFileSync(globalsPath, reconstructedContent, 'utf8');
     console.log(MESSAGES.GLOBALS_UPDATED(globalsPath));
     // `globalsPath` / `sourcePackages` は generate のサマリ出力（issue #56）が
@@ -643,7 +652,12 @@ export function manageFontImports(
       sparkleDesignPath
     );
 
-    if (result.status === 'skipped') {
+    // `already-up-to-date`（globals.css は正規形のまま変更不要、issue #59）の
+    // 場合も、後続の「sparkle-design.css からの legacy フォント import 除去」は
+    // 実行する必要があるため early return しない。
+    // en: `already-up-to-date` (no-op write, issue #59) must still fall through
+    // to the legacy font-import stripping below.
+    if (result.status === 'skipped' && result.reason !== 'already-up-to-date') {
       return result;
     }
     if (result.status === 'failed') {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sparkle-design-cli",
-  "version": "2.2.2",
+  "version": "2.2.3",
   "description": "Sparkle Design CLI — プロジェクトセットアップ、CSS・フォント生成、アンチパターン検査、AI エージェント（Claude Code / Cursor / Codex）向けのガードと hook 設定まで一括で行う sparkle-design 公式 CLI。",
   "publishConfig": {
     "registry": "https://registry.npmjs.org",