Add SQL parameter validation to lint-scan

[tomoemon]

Summary

• lint-scan に SQL パラメータ (@param) と Params マップキーの整合性チェックを追加
• SQL にあるが Params にないパラメータ、Params にあるが SQL にないキーの両方をエラーとして報告
• Params が変数参照の場合はマップリテラルを要求するエラーを報告（//nolint:spantool で抑制可能）
• コールバックがない場合でもパラメータチェックは動作
• README に型互換性チェックは行わない旨を明記

設計の背景

既存の ToStruct チェックと同様に双方向のエラー報告を採用した。ToStruct では struct にあるが SELECT にないフィールドもエラーとして報告しており、Params でも未使用キーを警告ではなくエラーとすることで一貫性を保っている。

Params が変数参照の場合はエラーを報告し、マップリテラルの使用を促す。静的解析不可能なケースでも //nolint:spantool コメントで明示的に抑制できる。

検討した代替案

• SQL の二重パース回避: extractSQLParams と extractSelectInfo が同じ SQL を個別にパースしているが、lint ツールの実行速度への影響は軽微なため、関心の分離を優先して現状のまま据え置いた
• extractMapLiteralKeys と lint.go の extractMapKeys の共通化: 戻り値の型（[]string vs []mapKey）やエラー処理方針が異なるため、無理に統合せず個別に実装した

Test plan

• go test ./... 全テスト通過
• パラメータ一致（正常系）
• SQL にあるが Params にないパラメータ（エラー報告）
• Params にあるが SQL にないキー（エラー報告）
• Params が変数参照（エラー報告）
• Params が変数参照 + //nolint:spantool（抑制）
• Params フィールド未指定 + SQL にパラメータあり（スキップ）
• パラメータなし + Params なし（正常）
• 空マップ + SQL にパラメータあり（エラー報告）
• コールバックなしでもパラメータチェック動作

🤖 Generated with Claude Code