sparkle-design-cli 2.0.7-beta.1 → 2.0.7-beta.10
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +135 -24
- package/bin/sparkle-design.js +12 -1
- package/lib/anti-pattern-rules.js +29 -15
- package/lib/check.js +93 -11
- package/lib/constants.js +19 -2
- package/lib/font-manager.js +188 -37
- package/lib/generate-css.js +151 -15
- package/lib/setup.js +379 -16
- package/package.json +15 -5
package/README.md
CHANGED
|
@@ -15,7 +15,7 @@ setup は次を自動で行います:
|
|
|
15
15
|
1. パッケージマネージャー(pnpm / npm / yarn / bun)を自動検出
|
|
16
16
|
2. `sparkle-design` を dependencies、`tailwindcss` + `@tailwindcss/postcss` を devDependencies に追加
|
|
17
17
|
3. 未作成の場合のみ初期ファイルを生成:
|
|
18
|
-
- `sparkle.config.json`(デフォルトは blue /
|
|
18
|
+
- `sparkle.config.json`(デフォルトは blue / BIZ UDPGothic / BIZ UDGothic / md — sparkle-design 本体のデフォルトに合わせて統一)
|
|
19
19
|
- `postcss.config.mjs`
|
|
20
20
|
- Tailwind エントリ CSS — Next.js App Router なら `src/app/globals.css`、Vite なら `src/index.css`、それ以外は `src/globals.css`(プロジェクト構成から自動判定)
|
|
21
21
|
4. AI 指示ファイル(`CLAUDE.md` / `AGENTS.md` / Cursor rules)に Sparkle Design Guard ブロックを追加
|
|
@@ -35,10 +35,10 @@ npm install -g sparkle-design-cli
|
|
|
35
35
|
|
|
36
36
|
npm の dist-tag でチャネルを分離しています。通常は latest を使い、品質保証中の変更を試したい場合のみ beta を指定してください。
|
|
37
37
|
|
|
38
|
-
| チャネル | dist-tag | 用途
|
|
39
|
-
|
|
40
|
-
| 安定版
|
|
41
|
-
| Beta
|
|
38
|
+
| チャネル | dist-tag | 用途 | 指定方法 |
|
|
39
|
+
| -------- | -------- | ------------------------------------------------------------- | ----------------------------------- |
|
|
40
|
+
| 安定版 | `latest` | 本番運用向け。`npx --yes sparkle-design-cli` は常にこれを取得 | (デフォルト) |
|
|
41
|
+
| Beta | `beta` | 品質保証中の検証用 | `npx --yes sparkle-design-cli@beta` |
|
|
42
42
|
|
|
43
43
|
```bash
|
|
44
44
|
# beta で setup を試す
|
|
@@ -103,6 +103,12 @@ sparkle-design-cli generate --output ./styles/design.css
|
|
|
103
103
|
|
|
104
104
|
# 設定ファイルと出力先を両方指定
|
|
105
105
|
sparkle-design-cli generate -c ./config/custom.json -o ./dist/styles.css
|
|
106
|
+
|
|
107
|
+
# Tailwind エントリ CSS を明示指定
|
|
108
|
+
sparkle-design-cli generate --globals-path src/styles/app.css
|
|
109
|
+
|
|
110
|
+
# CI 向け: @source 注入や Tailwind import 欠落などの失敗を exit 1 にする
|
|
111
|
+
sparkle-design-cli generate --strict
|
|
106
112
|
```
|
|
107
113
|
|
|
108
114
|
#### generate オプション一覧
|
|
@@ -110,6 +116,12 @@ sparkle-design-cli generate -c ./config/custom.json -o ./dist/styles.css
|
|
|
110
116
|
- `-h, --help`: ヘルプメッセージを表示
|
|
111
117
|
- `-c, --config <パス>`: 設定ファイルのパス(デフォルト: `./sparkle.config.json`)
|
|
112
118
|
- `-o, --output <パス>`: 出力ファイルのパス(デフォルト: `./src/app/sparkle-design.css`)
|
|
119
|
+
- `--globals-path <パス>`: Tailwind エントリポイント CSS のパス(デフォルト: 自動検出)。
|
|
120
|
+
**指定したパスが存在しない場合は `--strict` の有無に関わらず常に exit 1**(`sparkle.config.json` の `extend.globals-path` も同様)
|
|
121
|
+
- `--strict`: 以下を warn ではなく **exit 1** に昇格させる(CI 向け。既定は warn + 継続で後方互換を維持):
|
|
122
|
+
- デザインシステムパッケージが `package.json` に入っているのに Tailwind エントリ CSS が見つからない
|
|
123
|
+
- エントリ CSS はあるが `@import "tailwindcss";` が書かれていない(警告メッセージに追記すべき行まで actionable に表示)
|
|
124
|
+
- エントリ CSS への書き込みに失敗した
|
|
113
125
|
|
|
114
126
|
### check: アンチパターン検査
|
|
115
127
|
|
|
@@ -142,24 +154,15 @@ sparkle-design-cli check --help
|
|
|
142
154
|
- children なしの Button に prefixIcon / suffixIcon を使わない
|
|
143
155
|
- Material Symbols を className 直書きで使わない
|
|
144
156
|
- shadcn/ui 既定 token を Sparkle Design 内へ持ち込まない
|
|
145
|
-
- Tailwind デフォルト typography
|
|
146
|
-
- CardTitle に typography
|
|
157
|
+
- Tailwind デフォルト typography を Sparkle Design コンポーネント内で使わない
|
|
158
|
+
- CardTitle に typography 系クラスを付与しない
|
|
147
159
|
- CardControl に Button / IconButton 以外を入れない
|
|
148
|
-
-
|
|
149
|
-
- asChild
|
|
150
|
-
-
|
|
151
|
-
- Button の prefixIcon に JSX を渡さない
|
|
160
|
+
- Card 系コンポーネントのデフォルト padding を安易に上書きしない
|
|
161
|
+
- asChild 使用時に prefixIcon / suffixIcon / isLoading を使わない
|
|
162
|
+
- Sparkle Design コンポーネントでは isDisabled を使う
|
|
163
|
+
- Button の prefixIcon / suffixIcon に JSX を渡さない
|
|
152
164
|
- Icon の children にテキストを渡さない
|
|
153
|
-
- クリック可能な Card
|
|
154
|
-
|
|
155
|
-
#### Manual Review Reminders
|
|
156
|
-
|
|
157
|
-
`--format json` では、機械検出できない観点も `manualReviewReminders` として返します。現在は次を含みます。
|
|
158
|
-
|
|
159
|
-
- Badge と Tag の意味的な使い分け
|
|
160
|
-
- CardDescription の typography / color token 明示
|
|
161
|
-
- Dialog と Modal の UX 上の使い分け
|
|
162
|
-
- 手書き span が Tag で置き換え可能か
|
|
165
|
+
- クリック可能な Card は ClickableCard を使う(button / a / role="button" でラップしない)
|
|
163
166
|
|
|
164
167
|
#### 導入先での推奨設定
|
|
165
168
|
|
|
@@ -198,6 +201,7 @@ npx --yes sparkle-design-cli setup --assistant claude --skip-install --skip-scaf
|
|
|
198
201
|
```
|
|
199
202
|
|
|
200
203
|
既存ファイルは上書きされません:
|
|
204
|
+
|
|
201
205
|
- `sparkle.config.json` / `postcss.config.*` / Tailwind エントリ CSS が存在する場合はそのまま保持
|
|
202
206
|
- 既に依存関係に入っているパッケージはインストールをスキップ
|
|
203
207
|
- `package.json` の既存の `lint:sparkle*` 独自 script は保持(`--force-script-update` で上書き)
|
|
@@ -205,6 +209,7 @@ npx --yes sparkle-design-cli setup --assistant claude --skip-install --skip-scaf
|
|
|
205
209
|
`--target` を省略した場合、lint 対象は `src`, `src/components`, `src/features`, `app`, `components` の順で自動検出されます。
|
|
206
210
|
|
|
207
211
|
利用できる assistant:
|
|
212
|
+
|
|
208
213
|
- `claude`: `CLAUDE.md`
|
|
209
214
|
- `codex`: `AGENTS.md`
|
|
210
215
|
- `cursor`: `.cursor/rules/sparkle-design-guard.mdc`
|
|
@@ -220,9 +225,117 @@ npx --yes sparkle-design-cli setup --assistant claude --skip-install --skip-scaf
|
|
|
220
225
|
- `--skip-generate`: `generate` 実行をスキップ
|
|
221
226
|
- `--dry-run`: ファイルを変更せず結果だけ表示
|
|
222
227
|
- `--force-script-update`: 既存の `lint:sparkle` 系 script も上書き
|
|
228
|
+
- `--strict`: `generate` ステージの失敗を exit 1 に昇格(CI 向け)。
|
|
229
|
+
現状は `generate` の失敗のみ対象(install / scaffold / guard の失敗は従来どおり warn)。
|
|
230
|
+
`--skip-generate` / `--dry-run` と併用した場合は strict チェック対象が無くなるため警告を出します。
|
|
223
231
|
|
|
224
232
|
`setup` は通常実行時も JSON サマリーを stdout に表示します。`--dry-run` を付けると、その JSON を表示したままファイル変更だけを抑止します。
|
|
225
233
|
|
|
234
|
+
### 手動セットアップ(Next.js / Vite 以外のプロジェクト向け)
|
|
235
|
+
|
|
236
|
+
`setup` コマンドは **Next.js App Router** と **Vite** を前提に scaffold / entry CSS の配置を自動判定します。Remix / Astro / TanStack Start など他のフレームワークや、独自ディレクトリ構成のプロジェクトでは自動判定が外れるため、以下の手順で手動セットアップすることを推奨します。
|
|
237
|
+
|
|
238
|
+
#### 手順
|
|
239
|
+
|
|
240
|
+
**1. パッケージをインストール**
|
|
241
|
+
|
|
242
|
+
```bash
|
|
243
|
+
# 本体
|
|
244
|
+
pnpm add sparkle-design # or npm install / yarn add / bun add
|
|
245
|
+
|
|
246
|
+
# Tailwind v4
|
|
247
|
+
pnpm add -D tailwindcss @tailwindcss/postcss
|
|
248
|
+
```
|
|
249
|
+
|
|
250
|
+
**2. `sparkle.config.json` をプロジェクトルートに作成**
|
|
251
|
+
|
|
252
|
+
```json
|
|
253
|
+
{
|
|
254
|
+
"primary": "blue",
|
|
255
|
+
"font-pro": "Inter",
|
|
256
|
+
"font-mono": "JetBrains Mono",
|
|
257
|
+
"radius": "md"
|
|
258
|
+
}
|
|
259
|
+
```
|
|
260
|
+
|
|
261
|
+
選択肢の詳細は本 README の「[設定オプション](#設定オプション)」を参照してください。
|
|
262
|
+
|
|
263
|
+
**3. `postcss.config.mjs` をプロジェクトルートに作成**
|
|
264
|
+
|
|
265
|
+
```js
|
|
266
|
+
export default {
|
|
267
|
+
plugins: {
|
|
268
|
+
'@tailwindcss/postcss': {},
|
|
269
|
+
},
|
|
270
|
+
};
|
|
271
|
+
```
|
|
272
|
+
|
|
273
|
+
**4. Tailwind エントリ CSS を自前で用意**
|
|
274
|
+
|
|
275
|
+
既存プロジェクトに組み込む場合、`@import "tailwindcss";` を含む CSS ファイル(例: `src/styles/app.css`)があればそれを使います。無ければ作成してアプリケーションのエントリから import します。
|
|
276
|
+
|
|
277
|
+
**5. `generate` を実行**
|
|
278
|
+
|
|
279
|
+
```bash
|
|
280
|
+
npx --yes sparkle-design-cli generate
|
|
281
|
+
```
|
|
282
|
+
|
|
283
|
+
これで `sparkle-design.css` が `src/app/sparkle-design.css` に生成され、`SparkleHead.tsx` も同じ場所に出ます。エントリ CSS の検出に失敗する場合は `sparkle.config.json` の `extend.globals-path` に明示指定してください。
|
|
284
|
+
|
|
285
|
+
```json
|
|
286
|
+
{
|
|
287
|
+
"primary": "blue",
|
|
288
|
+
"extend": {
|
|
289
|
+
"globals-path": "src/styles/app.css"
|
|
290
|
+
}
|
|
291
|
+
}
|
|
292
|
+
```
|
|
293
|
+
|
|
294
|
+
**6. フォントの `<link>` タグを手動で配置**
|
|
295
|
+
|
|
296
|
+
Next.js / Vite 以外ではアプリケーションフレームワークごとに「`<head>` 相当」の配置方法が異なります。自動生成される `SparkleHead.tsx` はそのまま使えるはずですが、使えない場合は中身(`preconnect` + Google Fonts の `<link>` タグ群)を自前のレイアウトにコピーしてください。
|
|
297
|
+
|
|
298
|
+
たとえば Astro なら `src/layouts/BaseLayout.astro` の `<head>` に直接書きます:
|
|
299
|
+
|
|
300
|
+
```html
|
|
301
|
+
<link rel="preconnect" href="https://fonts.googleapis.com" />
|
|
302
|
+
<link rel="preconnect" href="https://fonts.gstatic.com" crossorigin />
|
|
303
|
+
<link rel="stylesheet" href="https://fonts.googleapis.com/css2?family=Material+Symbols+Rounded:FILL,wght@0..1,500&display=block" />
|
|
304
|
+
<!-- sparkle.config.json の font-pro / font-mono に合わせた Google Fonts URL -->
|
|
305
|
+
<link rel="stylesheet" href="https://fonts.googleapis.com/css2?family=Inter:wght@400;700&display=swap" />
|
|
306
|
+
<link rel="stylesheet" href="https://fonts.googleapis.com/css2?family=JetBrains+Mono:wght@400;700&display=swap" />
|
|
307
|
+
```
|
|
308
|
+
|
|
309
|
+
**7. アンチパターン検査を package.json に追加(任意)**
|
|
310
|
+
|
|
311
|
+
```json
|
|
312
|
+
{
|
|
313
|
+
"scripts": {
|
|
314
|
+
"lint:sparkle": "npx --yes sparkle-design-cli check src --strict",
|
|
315
|
+
"lint:sparkle:json": "npx --yes sparkle-design-cli check src --format json"
|
|
316
|
+
}
|
|
317
|
+
}
|
|
318
|
+
```
|
|
319
|
+
|
|
320
|
+
**8. AI エージェントを使うなら Guard ブロックと hook を手動で配置**
|
|
321
|
+
|
|
322
|
+
AI ガード(`CLAUDE.md` / `AGENTS.md` / `.cursor/rules/*.mdc`)と hook 設定(`.claude/settings.json` / `.cursor/hooks.json` / `.codex/hooks.json`)は `setup` に任せるのが最も楽です。
|
|
323
|
+
|
|
324
|
+
```bash
|
|
325
|
+
# ガード追加・hook 設定のみ(パッケージインストールや scaffold はスキップ)
|
|
326
|
+
npx --yes sparkle-design-cli setup --assistant claude --skip-install --skip-scaffold --skip-generate
|
|
327
|
+
```
|
|
328
|
+
|
|
329
|
+
この `--skip-*` 組み合わせは手動セットアップ済みのプロジェクトに対して Guard + hook だけ差し込むのに使えます。
|
|
330
|
+
|
|
331
|
+
#### 既知の未対応ケース
|
|
332
|
+
|
|
333
|
+
- **モノレポ(workspaces)**: `generate` は実行した cwd からパッケージマネージャーを遡及検出しないため、サブパッケージで直接実行するのが確実です。
|
|
334
|
+
- **CSS 以外のエントリ(CSS-in-JS / vanilla-extract 等)**: Sparkle Design は Tailwind v4 の `@source` + CSS variable 前提で作られているため、ビルドパイプラインの外で CSS を扱うソリューションは対象外です。
|
|
335
|
+
- **Tailwind v3**: `@source` ディレクティブ依存のため v4 必須です。
|
|
336
|
+
|
|
337
|
+
Next.js / Vite 以外で導入したい方で困ったときは Issue で教えていただけると助かります。
|
|
338
|
+
|
|
226
339
|
## 設定ファイル (sparkle.config.json)
|
|
227
340
|
|
|
228
341
|
### 設定ファイルの作成
|
|
@@ -259,9 +372,7 @@ Figma プラグインが出力する基本4項目に加えて、プロジェク
|
|
|
259
372
|
{ "family": "Montserrat", "weights": [500, 600, 700] },
|
|
260
373
|
{ "family": "Noto Sans JP", "weights": [400, 500, 600, 700] }
|
|
261
374
|
],
|
|
262
|
-
"mono": [
|
|
263
|
-
{ "family": "Roboto Mono", "weights": [400, 700] }
|
|
264
|
-
]
|
|
375
|
+
"mono": [{ "family": "Roboto Mono", "weights": [400, 700] }]
|
|
265
376
|
},
|
|
266
377
|
"source-packages": ["@goodpatch/sparkle-design-internal"],
|
|
267
378
|
"custom-css": "./src/app/custom-tokens.css"
|
package/bin/sparkle-design.js
CHANGED
|
@@ -19,6 +19,7 @@ function parseGenerateOptions(args) {
|
|
|
19
19
|
configPath: null,
|
|
20
20
|
outputPath: null,
|
|
21
21
|
globalsPath: null,
|
|
22
|
+
strict: false,
|
|
22
23
|
help: false,
|
|
23
24
|
};
|
|
24
25
|
|
|
@@ -36,6 +37,8 @@ function parseGenerateOptions(args) {
|
|
|
36
37
|
} else if (arg === '--globals-path') {
|
|
37
38
|
options.globalsPath = requireOptionValue(args, i, '--globals-path');
|
|
38
39
|
i += 1;
|
|
40
|
+
} else if (arg === '--strict') {
|
|
41
|
+
options.strict = true;
|
|
39
42
|
} else {
|
|
40
43
|
throw new Error(`Unknown option for generate: ${arg}`);
|
|
41
44
|
}
|
|
@@ -86,6 +89,7 @@ function parseSetupOptions(args) {
|
|
|
86
89
|
skipInstall: false,
|
|
87
90
|
skipScaffold: false,
|
|
88
91
|
skipGenerate: false,
|
|
92
|
+
strict: false,
|
|
89
93
|
help: false,
|
|
90
94
|
};
|
|
91
95
|
|
|
@@ -113,6 +117,8 @@ function parseSetupOptions(args) {
|
|
|
113
117
|
options.skipScaffold = true;
|
|
114
118
|
} else if (arg === '--skip-generate') {
|
|
115
119
|
options.skipGenerate = true;
|
|
120
|
+
} else if (arg === '--strict') {
|
|
121
|
+
options.strict = true;
|
|
116
122
|
} else {
|
|
117
123
|
throw new Error(`Unknown option for setup: ${arg}`);
|
|
118
124
|
}
|
|
@@ -161,6 +167,8 @@ Generate options:
|
|
|
161
167
|
-c, --config <path> 設定ファイルのパス (default: ./sparkle.config.json)
|
|
162
168
|
-o, --output <path> 出力ファイルのパス (default: ./src/app/sparkle-design.css)
|
|
163
169
|
--globals-path <path> Tailwind エントリポイント CSS のパス (default: 自動検出)
|
|
170
|
+
--strict globals.css への @source 注入等が失敗したら exit 1
|
|
171
|
+
(CI 向け。既定は warn + 継続で後方互換を維持)
|
|
164
172
|
|
|
165
173
|
sparkle.config.json の設定フィールド:
|
|
166
174
|
|
|
@@ -204,6 +212,7 @@ Setup options:
|
|
|
204
212
|
--skip-install パッケージインストール(sparkle-design, tailwindcss)をスキップ
|
|
205
213
|
--skip-scaffold 初期ファイル(sparkle.config.json, postcss.config, エントリ CSS)生成をスキップ
|
|
206
214
|
--skip-generate generate 実行をスキップ
|
|
215
|
+
--strict generate の失敗を exit 1 に昇格(CI 向け)
|
|
207
216
|
|
|
208
217
|
Setup の動作:
|
|
209
218
|
1. パッケージマネージャー検出(pnpm / npm / yarn / bun)
|
|
@@ -257,7 +266,9 @@ function main() {
|
|
|
257
266
|
process.exit(0);
|
|
258
267
|
}
|
|
259
268
|
|
|
260
|
-
generateCSS(options.configPath, options.outputPath, options.globalsPath
|
|
269
|
+
generateCSS(options.configPath, options.outputPath, options.globalsPath, {
|
|
270
|
+
strict: options.strict,
|
|
271
|
+
});
|
|
261
272
|
return;
|
|
262
273
|
}
|
|
263
274
|
|
|
@@ -971,21 +971,35 @@ const ANTI_PATTERN_GROUPS = [
|
|
|
971
971
|
description: 'CardControl に Button / IconButton 以外を入れない',
|
|
972
972
|
recommendation:
|
|
973
973
|
'CardControl はアクションボタン用です。ステータス表示には CardDescription を使ってください。',
|
|
974
|
-
//
|
|
975
|
-
//
|
|
976
|
-
//
|
|
977
|
-
//
|
|
978
|
-
//
|
|
979
|
-
//
|
|
980
|
-
//
|
|
981
|
-
// en:
|
|
982
|
-
//
|
|
983
|
-
//
|
|
984
|
-
//
|
|
985
|
-
//
|
|
986
|
-
//
|
|
987
|
-
|
|
988
|
-
/<CardControl\b
|
|
974
|
+
// 2-pass matcher。1 本の regex に入れ子 negative lookahead を押し込む代わりに、
|
|
975
|
+
// まず <CardControl>…</CardControl> ブロックを抽出し、次にブロック内部だけを
|
|
976
|
+
// スキャンして Button / IconButton / 入れ子 CardControl 以外の開始タグを見つける。
|
|
977
|
+
// 1. 入れ子 negative lookahead が無くなり読みやすい
|
|
978
|
+
// 2. 自己閉じ `<CardControl />` は 1st pass でそもそも拾わない(special case 不要)
|
|
979
|
+
// 3. O(総コンテンツ長) で評価が済むため、大規模 JSX でも backtracking で
|
|
980
|
+
// 処理時間が跳ねない
|
|
981
|
+
// en: Two-pass matcher. First pass extracts `<CardControl>…</CardControl>`
|
|
982
|
+
// blocks (self-closing variants are skipped because the opening-tag regex
|
|
983
|
+
// requires the previous char not to be `/`). Second pass scans the inner
|
|
984
|
+
// content for any element that is not Button / IconButton / nested
|
|
985
|
+
// CardControl. This replaces the nested tempered-lazy regex with a shape
|
|
986
|
+
// that is easier to reason about and stays linear in total content length.
|
|
987
|
+
match: (content) => {
|
|
988
|
+
const BLOCK_PATTERN = /<CardControl\b[^>]*(?<!\/)>([\s\S]*?)<\/CardControl\b[^>]*>/g;
|
|
989
|
+
const NON_BUTTON_CHILD = /<(?!Button\b|IconButton\b|CardControl\b)[A-Za-z][\w.]*/g;
|
|
990
|
+
const results = [];
|
|
991
|
+
for (const blockMatch of content.matchAll(BLOCK_PATTERN)) {
|
|
992
|
+
const openingEnd = blockMatch.index + blockMatch[0].indexOf('>') + 1;
|
|
993
|
+
const inner = blockMatch[1];
|
|
994
|
+
for (const childMatch of inner.matchAll(NON_BUTTON_CHILD)) {
|
|
995
|
+
results.push({
|
|
996
|
+
index: openingEnd + childMatch.index,
|
|
997
|
+
text: childMatch[0],
|
|
998
|
+
});
|
|
999
|
+
}
|
|
1000
|
+
}
|
|
1001
|
+
return results;
|
|
1002
|
+
},
|
|
989
1003
|
},
|
|
990
1004
|
featureSection: lines([
|
|
991
1005
|
'### CardControl にはアクションボタンのみを入れる',
|
package/lib/check.js
CHANGED
|
@@ -12,7 +12,12 @@ const BASE_MANUAL_REVIEW_REMINDERS = getManualReviewReminders();
|
|
|
12
12
|
|
|
13
13
|
const SPARKLE_HEAD_JSX_PATTERN = /<SparkleHead\s*\/?\s*>/;
|
|
14
14
|
const CSP_PATTERN = /content-security-policy|contentSecurityPolicy|Content-Security-Policy/i;
|
|
15
|
-
const NEXT_CONFIG_CANDIDATES = [
|
|
15
|
+
const NEXT_CONFIG_CANDIDATES = [
|
|
16
|
+
'next.config.js',
|
|
17
|
+
'next.config.ts',
|
|
18
|
+
'next.config.mjs',
|
|
19
|
+
'next.config.cjs',
|
|
20
|
+
];
|
|
16
21
|
|
|
17
22
|
function toRelativeReportPath(filePath) {
|
|
18
23
|
return path.relative(process.cwd(), filePath).split(path.sep).join('/');
|
|
@@ -65,10 +70,30 @@ function formatMatch(match) {
|
|
|
65
70
|
return match[0].replace(/\s+/g, ' ').trim().slice(0, 120);
|
|
66
71
|
}
|
|
67
72
|
|
|
73
|
+
function formatSnippet(text) {
|
|
74
|
+
return text.replace(/\s+/g, ' ').trim().slice(0, 120);
|
|
75
|
+
}
|
|
76
|
+
|
|
68
77
|
function collectFindings(filePath, content) {
|
|
69
78
|
const findings = [];
|
|
70
79
|
|
|
71
80
|
for (const rule of RULES) {
|
|
81
|
+
if (typeof rule.match === 'function') {
|
|
82
|
+
// rule.match(content) -> Array<{ index: number, text: string }>
|
|
83
|
+
// 複雑な照合が必要なルール向けの opt-in API。2-pass 方式などで regex 単体の
|
|
84
|
+
// backtracking リスクを回避したいときに使う。index は content 内の絶対オフセット。
|
|
85
|
+
for (const hit of rule.match(content)) {
|
|
86
|
+
findings.push({
|
|
87
|
+
filePath,
|
|
88
|
+
id: rule.id,
|
|
89
|
+
description: rule.description,
|
|
90
|
+
recommendation: rule.recommendation,
|
|
91
|
+
line: getLineNumber(content, hit.index ?? 0),
|
|
92
|
+
snippet: formatSnippet(hit.text ?? ''),
|
|
93
|
+
});
|
|
94
|
+
}
|
|
95
|
+
continue;
|
|
96
|
+
}
|
|
72
97
|
for (const match of content.matchAll(rule.pattern)) {
|
|
73
98
|
findings.push({
|
|
74
99
|
filePath,
|
|
@@ -96,8 +121,10 @@ function collectFontImportFindings(cssFiles) {
|
|
|
96
121
|
findings.push({
|
|
97
122
|
filePath,
|
|
98
123
|
id: 'font-import-in-css',
|
|
99
|
-
description:
|
|
100
|
-
|
|
124
|
+
description:
|
|
125
|
+
'CSS にフォント @import が残っています。SparkleHead コンポーネントに移行してください。',
|
|
126
|
+
recommendation:
|
|
127
|
+
'sparkle-design-cli generate を再実行すると、フォント読み込みが SparkleHead.tsx に移行され、globals.css から @import が除去されます。',
|
|
101
128
|
line: getLineNumber(content, match.index ?? 0),
|
|
102
129
|
snippet: match[0].slice(0, 120),
|
|
103
130
|
});
|
|
@@ -140,12 +167,20 @@ function collectNextjsCspFindings(cwd) {
|
|
|
140
167
|
if (!CSP_PATTERN.test(content)) continue;
|
|
141
168
|
|
|
142
169
|
const missingDomains = [
|
|
143
|
-
{
|
|
144
|
-
|
|
170
|
+
{
|
|
171
|
+
domain: 'fonts.googleapis.com',
|
|
172
|
+
directive: 'style-src',
|
|
173
|
+
pattern: new RegExp(FONT_DOMAINS.GOOGLEAPIS.replace('https://', '')),
|
|
174
|
+
},
|
|
175
|
+
{
|
|
176
|
+
domain: 'fonts.gstatic.com',
|
|
177
|
+
directive: 'font-src',
|
|
178
|
+
pattern: new RegExp(FONT_DOMAINS.GSTATIC.replace('https://', '')),
|
|
179
|
+
},
|
|
145
180
|
].filter(({ pattern }) => !pattern.test(content));
|
|
146
181
|
|
|
147
182
|
if (missingDomains.length > 0) {
|
|
148
|
-
const missing = missingDomains.map(d => `${d.domain} (${d.directive})`).join(', ');
|
|
183
|
+
const missing = missingDomains.map((d) => `${d.domain} (${d.directive})`).join(', ');
|
|
149
184
|
findings.push({
|
|
150
185
|
filePath: configPath,
|
|
151
186
|
id: 'csp-font-block',
|
|
@@ -206,7 +241,8 @@ function createCheckReport(targets = []) {
|
|
|
206
241
|
if (!hasSparkleHeadUsage(fileContents)) {
|
|
207
242
|
manualReviewReminders.push({
|
|
208
243
|
id: 'sparkle-head-missing',
|
|
209
|
-
message:
|
|
244
|
+
message:
|
|
245
|
+
'SparkleHead がソースコード内に見つかりませんでした。ルートレイアウトの <head> 内に <SparkleHead /> を配置してフォントの早期読み込みを有効にしてください。',
|
|
210
246
|
});
|
|
211
247
|
}
|
|
212
248
|
|
|
@@ -232,9 +268,37 @@ function printTextReport(report, options = {}) {
|
|
|
232
268
|
}
|
|
233
269
|
}
|
|
234
270
|
|
|
235
|
-
|
|
236
|
-
|
|
237
|
-
|
|
271
|
+
const reminders = report.manualReviewReminders ?? [];
|
|
272
|
+
if (reminders.length === 0) {
|
|
273
|
+
// 0 件のときは従来通り静かに済ます。
|
|
274
|
+
// en: Nothing to review — keep quiet.
|
|
275
|
+
} else {
|
|
276
|
+
// Guard block から参照される acknowledgment フォーマットと合わせるため、
|
|
277
|
+
// AI 向けに「各 ID を response に echo して確認したことを示してほしい」
|
|
278
|
+
// と明示的に要求する。hook による exit blocking はしない(判断事項のため)。
|
|
279
|
+
// en: Ask the AI to echo each reminder ID in its final response, so humans
|
|
280
|
+
// can see which reminders were considered. This is not enforced by hook
|
|
281
|
+
// exit code — reminders are judgment calls, not hard failures.
|
|
282
|
+
console.log('');
|
|
283
|
+
console.log('=== Manual review reminders (must acknowledge each ID in your response) ===');
|
|
284
|
+
console.log(
|
|
285
|
+
'These are judgment calls the linter cannot detect. For every item below,'
|
|
286
|
+
);
|
|
287
|
+
console.log(
|
|
288
|
+
'explicitly state the reminder ID in your reply together with whether the'
|
|
289
|
+
);
|
|
290
|
+
console.log(
|
|
291
|
+
'current code already satisfies it, or what change is needed. Silence = skipped.'
|
|
292
|
+
);
|
|
293
|
+
console.log('');
|
|
294
|
+
for (const reminder of reminders) {
|
|
295
|
+
console.log(`- [${reminder.id}] ${reminder.message}`);
|
|
296
|
+
}
|
|
297
|
+
console.log('');
|
|
298
|
+
console.log(
|
|
299
|
+
'Example acknowledgment: "[badge-tag-semantics] reviewed — Badge used only for counts (OK). [dialog-modal-ux] reviewed — changed Dialog to Modal for form case."'
|
|
300
|
+
);
|
|
301
|
+
console.log('=========================================================================');
|
|
238
302
|
}
|
|
239
303
|
|
|
240
304
|
if (options.strict && report.findings.length > 0) {
|
|
@@ -245,6 +309,7 @@ function printTextReport(report, options = {}) {
|
|
|
245
309
|
function printJsonReport(report, options = {}) {
|
|
246
310
|
const strictMode = Boolean(options.strict);
|
|
247
311
|
const passed = !(strictMode && report.findings.length > 0);
|
|
312
|
+
const reminders = report.manualReviewReminders ?? [];
|
|
248
313
|
|
|
249
314
|
console.log(
|
|
250
315
|
JSON.stringify(
|
|
@@ -253,6 +318,18 @@ function printJsonReport(report, options = {}) {
|
|
|
253
318
|
targetCount: report.targets.length,
|
|
254
319
|
checkedFileCount: report.checkedFiles.length,
|
|
255
320
|
findingCount: report.findings.length,
|
|
321
|
+
reminderCount: reminders.length,
|
|
322
|
+
// AI 向けの attention フラグ。reminder が 1 件でもあれば true になり、
|
|
323
|
+
// AI は最終 response で各 reminder ID を echo する必要がある。
|
|
324
|
+
// exit code には寄与しない(判断事項を hook で block するのは過剰)。
|
|
325
|
+
// en: AI attention flag. When reminders exist, the AI MUST echo each
|
|
326
|
+
// reminder ID in its final reply. Not tied to exit code — reminders
|
|
327
|
+
// are judgment calls, not hard errors.
|
|
328
|
+
manualReviewRequired: reminders.length > 0,
|
|
329
|
+
reminderAcknowledgmentFormat:
|
|
330
|
+
reminders.length > 0
|
|
331
|
+
? 'Respond with each reminder ID followed by your review, e.g. `[badge-tag-semantics] reviewed — Badge is used only for counts (OK)`.'
|
|
332
|
+
: null,
|
|
256
333
|
strictMode,
|
|
257
334
|
passed,
|
|
258
335
|
exitCode: passed ? 0 : 1,
|
|
@@ -277,4 +354,9 @@ export function checkProject(targets = [], options = {}) {
|
|
|
277
354
|
return report.findings.length > 0;
|
|
278
355
|
}
|
|
279
356
|
|
|
280
|
-
export {
|
|
357
|
+
export {
|
|
358
|
+
RULES,
|
|
359
|
+
collectFindings,
|
|
360
|
+
createCheckReport,
|
|
361
|
+
BASE_MANUAL_REVIEW_REMINDERS as MANUAL_REVIEW_REMINDERS,
|
|
362
|
+
};
|
package/lib/constants.js
CHANGED
|
@@ -44,8 +44,16 @@ export const REGEX = {
|
|
|
44
44
|
/\/\*\s*フォントのインポート[^*]*\*\/\s*\n?(?:@import\s+(?:url\([^)]+fonts\.googleapis\.com[^)]+\)|['"][^"']*fonts\.googleapis\.com[^"']*['"]);?\s*\n?)*\n?/g,
|
|
45
45
|
|
|
46
46
|
// Sparkle Design関連
|
|
47
|
+
// `@import "sparkle-design.css"` / `@import "./sparkle-design.css"` だけでなく
|
|
48
|
+
// `@import "./app/sparkle-design.css"` 等、任意の相対パスを含む variant も
|
|
49
|
+
// 除去対象にする。Vite のように entry CSS (src/index.css) と sparkle-design.css
|
|
50
|
+
// (src/app/sparkle-design.css) が別ディレクトリに置かれるレイアウトでは
|
|
51
|
+
// 後者のような path になるので、同一 dir 前提の regex だと AI が手で直したもの
|
|
52
|
+
// を除去できず二重 import が残ってしまう。
|
|
53
|
+
// en: Match any relative path ending in `sparkle-design.css`, including layouts
|
|
54
|
+
// like Vite where the entry CSS and sparkle-design.css live in different dirs.
|
|
47
55
|
SPARKLE_IMPORT:
|
|
48
|
-
/\/\*\s*Sparkle Design[^*]*\*\/\s*\n?|@import\s+['"]
|
|
56
|
+
/\/\*\s*Sparkle Design[^*]*\*\/\s*\n?|@import\s+['"][^"']*sparkle-design\.css['"];?\s*\n?/g,
|
|
49
57
|
TAILWIND_IMPORT: /@import\s+['"]tailwindcss['"];?/,
|
|
50
58
|
|
|
51
59
|
// @source ディレクティブ関連
|
|
@@ -137,6 +145,11 @@ export const COMMENTS = {
|
|
|
137
145
|
};
|
|
138
146
|
|
|
139
147
|
// インポート文のテンプレート
|
|
148
|
+
// SPARKLE_DESIGN は entry CSS と sparkle-design.css が同じディレクトリにある
|
|
149
|
+
// 場合のデフォルト。異なる場合は buildSparkleImportStatement() で相対 path を
|
|
150
|
+
// 計算して差し替える(Vite の src/index.css → src/app/sparkle-design.css 等)。
|
|
151
|
+
// en: Default when entry CSS and sparkle-design.css share a directory. Use
|
|
152
|
+
// buildSparkleImportStatement() for layouts that place them in different dirs.
|
|
140
153
|
export const IMPORTS = {
|
|
141
154
|
SPARKLE_DESIGN: '@import "./sparkle-design.css";',
|
|
142
155
|
TAILWIND: '@import "tailwindcss";',
|
|
@@ -170,7 +183,11 @@ export const MESSAGES = {
|
|
|
170
183
|
FONT_REMOVE_SKIPPED: 'ℹ️ globals.css の更新に失敗したため、フォントimport削除をスキップします。',
|
|
171
184
|
|
|
172
185
|
// 警告メッセージ
|
|
173
|
-
|
|
186
|
+
// 指定された globals.css に Tailwind import が無いと @source / sparkle-design.css
|
|
187
|
+
// の挿入ができないため、どのファイルに何を書けばよいかを示す actionable なメッセージにする。
|
|
188
|
+
// en: Make the warning actionable — tell the user which file to edit and what to add.
|
|
189
|
+
TAILWIND_NOT_FOUND: (path) =>
|
|
190
|
+
`⚠️ ${path} に Tailwind import が見つかりません。ファイル先頭に \`@import "tailwindcss";\` を追記してください。`,
|
|
174
191
|
GLOBALS_UPDATE_FAILED: (error) => `⚠️ globals.css の更新に失敗しました: ${error}`,
|
|
175
192
|
FONT_MANAGEMENT_ERROR: (error) => `⚠️ フォント管理処理でエラーが発生しました: ${error}`,
|
|
176
193
|
COLOR_CONVERSION_FAILED: (hex) => `⚠️ 色の変換に失敗しました: ${hex}`,
|