ns-best-practices

ベストプラクティス

品質とセキュリティ衛生に焦点を当てたレビューです。シニアによるコードレビューやビジュアルデザインの方針に代わるものではありません。

セッションの開始

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

使用する場面

リクエスト	このスキル	代わりに使用
セキュリティヘッダー／CSP／HSTS	はい	—
依存関係の監査／既知のCVEの一斉確認	はい	—
ブラウザー互換性のベースライン	はい	—
アクセシビリティの簡易チェック（ランドマーク、代替テキスト、ラベル）	はい	—
「UIをレビューして」／Web Interface Guidelines／UXとアクセシビリティの監査	はい（references/ui-guidelines-review.md）	—
個性的なビジュアルへの再設計／凡庸さを避ける美学	いいえ	ns-frontend-design
「このMRをレビューして」／SOLIDの詳細な検討	いいえ	ns-reviewer
新機能の実装	いいえ	ns-coder／ns-spec-driven

ワークフロー

1. 対象範囲 — Webアプリの範囲、APIゲートウェイ、静的アセット、またはprototype/。
2. ベースライン — references/checklist.mdを読み、該当するセクションだけにチェックを付けます（オフライン時のセキュリティ／互換性／アクセシビリティ）。
3. ガイドライン — UI／アクセシビリティ／UXの監査を行う場合、または呼び出し元がns-proto-creatorでクローズアウトする場合は、references/ui-guidelines-review.mdを最後まで読みます（ルールは同梱済みで、外部から取得する必要はありません）。
4. スキャン — 設定（nginx、Vite、Next、Laravelのミドルウェアなど）を確認します。スタックを前提にしないでください。
5. 報告 — 検出事項を表にまとめます。深刻度、場所（可能であればfile:line）、推奨事項、作業量を記載します。
6. 修正 — ユーザーが実装を依頼した場合は、安全最小限の差分で修正します。それ以外は報告のみとします。

他のスキルとの重複

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

デリバリーでは順番に使用する場合があります。スキルの振り分けは明示的に行います。

禁止事項

• 根拠なしに本番環境が安全だと宣言する
• リスクを指摘せずに既存の連携を壊す
• 高リスク領域における脅威モデリングの代わりにする（人間にエスカレーションする）
• UI監査／プロトタイプのクローズアウトでreferences/ui-guidelines-review.mdを省略しながら、UI全体を確認したと主張する