v0.1.0 to v0.1.0

---

id: 'config-json'

name: Configuration File Review

description: Review JSON/YAML configuration files for common issues and best practices.

version: 0.1.0

category: midstream

phase: midstream

applyTo:

- '**/*.json'

- '**/*.yml'

- '**/*.yaml'

tags:

- config

- json

- yaml

- midstream

severity: minor

inputContext:

- diff

outputKind:

- findings

modelHint: cheap

---

## Pattern declaration

Primary pattern: Reviewer

Secondary patterns: Inversion

Why: 設定ファイルの構文・セキュリティ・妥当性をチェックリスト型で評価するが、JSON/YAML以外の変更では実行不要

## Goal

設定ファイル（JSON/YAML）の変更をレビューし、一般的な問題やベストプラクティス違反を検出する。

## Guidance

- スキーマ違反や構文エラーの可能性を指摘する

- 機密情報（API キー、パスワード、トークン）の直書きを検出する

- 廃止された設定項目や非推奨オプションの使用を指摘する

- 設定値の妥当性（範囲外の数値、無効な組み合わせ）を確認する

- 環境固有の設定が誤ってコミットされていないか確認する

## Non-goals

- `package-lock.json` や自動生成ファイルはレビュー対象外（ノイズ削減のため）

- `node_modules/` 内のファイルは対象外

- テスト用フィクスチャファイルの内容は厳密にチェックしない

## Pre-execution Gate / 実行前ゲート

このスキルは以下の条件がすべて満たされない限り`NO_REVIEW`を返す。

- [ ] 差分にJSON/YAMLファイル（`.json`, `.yml`, `.yaml`）の変更が含まれている

- [ ] 対象ファイルが`package-lock.json`や自動生成ファイルではない

- [ ] inputContextにdiffが含まれている

## False-positive guards

- `.example` や `.sample` 接尾辞のファイルはダミー値を許容する

- テストディレクトリ内のファイルは緩和する

- 明示的にコメントで意図が説明されている場合は抑制する

## Rule / ルール

- 機密情報（APIキー、パスワード、トークン）の直書きは blocker として報告する

- JSON/YAML の構文エラーや不正なスキーマは major として報告する

- 非推奨オプションや廃止された設定項目は minor として報告する

- 環境固有の設定（localhost、開発用URL等）の混入は warning として報告する

## Output

標準フォーマット: `<file>:<line>: Finding: ... Evidence: ... Impact: ... Fix: ... Severity: ... Confidence: ...`