ns-code-best-practices

ベストプラクティス

重点を絞った品質とセキュリティの衛生管理を確認するものであり、シニアによるコードレビューやビジュアルデザインの指針に代わるものではありません。

ハーネスの確認

../ns-harness/references/harness-discovery.mdを参照してください。architecture-rules.mdとセキュリティ関連のハーネスルールを読んでください。

使用する場面

リクエスト	このスキル	代わりに使用
セキュリティヘッダー／CSP／HSTS	はい	—
依存関係の監査／既知のCVEの一斉チェック	はい	—
ブラウザー互換性のベースライン	はい	—
アクセシビリティの簡易チェック（ランドマーク、代替テキスト、ラベル）	はい	—
「UIをレビューして」／Web Interface Guidelines／UXアクセシビリティ監査	はい（ガイドラインを取得）	—
個性的なビジュアルへの刷新／ありがちなデザインの回避	いいえ	ns-code-frontend-design
「このMRをレビューして」／SOLIDの詳細な検討	いいえ	ns-code-reviewer
新機能の実装	いいえ	ns-code-coder／ns-spec-driven

ワークフロー

1. 範囲 — ウェブアプリの対象範囲、APIゲートウェイ、静的アセット、または{product_root}/prototype/を確認し、{product_root}を特定します。
2. ベースライン — references/checklist.mdを読み、該当するセクション（オフラインでのセキュリティ／互換性／アクセシビリティ）だけにチェックを付けます。
3. ガイドライン — UI／アクセシビリティ／UXの監査、または呼び出し元がns-proto-creatorでクローズアウトを行う場合は、references/ui-guidelines-review.mdに従います（WebFetchが必要です）。
4. スキャン — 設定（nginx、Vite、Next、Laravelミドルウェアなど）を確認します。スタックを決めつけないでください。
5. 報告 — 重大度、場所（可能であればfile:line）、推奨事項、工数を含む指摘事項の表を作成します。
6. 修正 — ユーザーが実装を依頼した場合は、安全性を最小限の差分で確保します。それ以外は報告のみ行います。

他のスキルとの重複

• このスキル： 横断的な衛生管理とWeb Interface Guidelinesへの準拠。
• ns-code-frontend-design： 個性的な外観、タイポグラフィ、モーション、ありがちなAI UIの回避。
• ns-code-reviewer： 差分を対象にした変更固有のSOLID、テスト、保守性の確認。

納品にあたっては、順番に使用してもかまいません。スキル間の振り分けでは、明示的な指定を優先してください。

禁止事項

• 根拠なしに、本番環境が安全だと断言する
• リスクを明示せずに既存の連携を壊す
• 高リスク領域で脅威モデリングの代わりに用いる（人間にエスカレーションする）
• UI監査／プロトタイプのクローズアウトでガイドラインを取得せずに、UI全体のチェックを完了したと主張する