技能（Skill）：审查 SQL

目的 (Purpose)

仅查看 SQL 和查询相关代码的 语言和查询约定。涵盖注入和参数化、索引和执行计划问题、事务和隔离、NULL 和唯一约束、方言可移植性、大表和分页模式以及敏感列和权限。以标准格式发出结果列表以进行聚合。不定义范围或执行全面的安全/架构审查；注入在这里是一个特定于 SQL 的问题，但更广泛的安全性是针对 review-security。

核心目标（Core Objective）

首要目标：生成以 SQL 为中心的 findings 列表，涵盖注入/参数化、索引、事务、NULL/约束、方言可移植性、分页模式及敏感列访问，针对给定代码范围。

成功标准（必须满足所有要求）：

✅ 仅限 SQL 范围：仅审查 SQL 和查询约定；未执行范围选择、完全安全性或架构分析
✅ 涵盖所有七个 SQL 维度：注入/参数化、索引/执行计划、事务/隔离、NULL/唯一约束、方言/可移植性、大表/分页模式以及相关的敏感列/权限
✅ 结果格式兼容：每个结果包括位置、类别（language-sql）、严重性、标题、描述和可选建议
✅ 已标记关键注入问题：SQL 注入模式（字符串连接、用户输入插值）被标记为“严重”严重性
✅ 位置精确引用：所有结果都引用特定文件：行或查询标识符位置

验收测试：输出是否包含涵盖所有相关查询维度的 SQL 结果列表，其中注入风险标记为“关键”以及每个结果的具体位置引用？

范围边界（范围边界）

本技能负责：

通过字符串连接或插值进行 SQL 注入 — 参数化查询和准备好的语句
WHERE/JOIN/ORDER BY 列的索引间隙
事务边界、隔离级别、死锁风险、长时间运行的事务
比较/聚合中的 NULL 处理、唯一约束、NOT NULL 正确性
数据库方言可移植性（LIMIT/OFFSET/FETCH、日期函数、供应商特定语法）
大表全扫描、分页策略（keyset vs OFFSET）
SELECT 中的敏感列暴露、最低权限角色使用

本技能不负责：

范围选择——范围由调用者提供
更广泛的安全分析（超越 SQL 注入）——使用“review-security”
架构分析——使用“review-architecture”
全面精心策划的审核——使用“审核代码”

转交点：当所有 SQL 结果发出后，将其交给 review-code 进行聚合。对于更广泛的安全问题（身份验证、加密、配置），请重定向到“审查安全”。

使用场景（用例）

精心安排的审阅：当 review-code 为包含 SQL（.sql 文件、嵌入式 SQL 或 ORM 生成的 SQL）的项目运行时用作语言步骤。
仅 SQL 审查：当用户只想检查查询的正确性、性能和安全性时。
迁移或可移植性：检查特定方言的构造和跨数据库的可移植性。

何时使用：当正在审查的代码包含 SQL（原始 .sql、嵌入代码或 ORM 生成）时。范围（差异与路径）由调用者或用户确定。

行为（行为）

该技能的范围

分析：给定范围（文件、片段或差异）中的 SQL 和查询逻辑。接受 .sql 文件、应用程序代码中的嵌入式 SQL 或 ORM 生成的 SQL（如果可见）。
不要：决定范围（差异与代码库）；不执行完整的应用程序安全或架构审查。关注 SQL/查询维度。

查看清单（仅限 SQL 维度）

注入和参数化：SQL中用户输入无需字符串串联或插值；使用参数化查询或准备好的语句；避免来自不可信输入的动态 SQL。
索引和执行计划：对未索引列进行过滤或联接的查询；在大表上选择*；缺少 WHERE/JOIN/ORDER BY 的索引。
事务和隔离：适当的事务边界；隔离级别和锁定；避免长时间运行的事务；僵局风险。
NULL和唯一约束：比较和聚合中NULL的处理；独特的约束和重复的处理；在适当的情况下 NOT NULL。
方言和可移植性：特定于数据库的语法（例如 LIMIT 与 OFFSET/FETCH、日期函数）和可移植性（如果需要多数据库支持）。
大表和分页：对大表进行全量扫描；分页（键集与 OFFSET）和可扩展性。
敏感列和权限：SELECT中的敏感数据； SQL 中的最低权限和角色使用（如果可见）。

语气和参考

专业和技术：参考特定位置（文件：行或查询标识符）。发出包含位置、类别、严重性、标题、描述、建议的结果。

输入与输出 (Input & Output)

输入（输入）

代码范围：包含 SQL 的文件或片段（例如 .sql 文件、带有嵌入式 SQL 的代码或 ORM 生成的 SQL（如果可用））。由用户或范围技能提供。

输出（输出）

以附录：输出合同中定义的格式发出零个或多个结果。
此技能的类别是语言-sql。

限制（限制）

硬边界（Hard Boundaries）

不要执行范围选择或全面的安全/架构审查。遵守 SQL 和查询约定。
不要在没有具体地点或可行建议的情况下给出结论。
不要假设特定的数据库供应商，除非另有说明；相关时请注意方言。

技能边界 (Skill Boundaries)

不要做这些（其他技能可以处理它们）：

不要选择或定义代码范围 - 范围由调用者或“审查代码”确定
不要执行 SQL 注入之外的广泛安全分析 - 使用“review-security”
不要执行架构分析——使用“review-architecture”
不要审查非 SQL 代码的 SQL 约定（应用程序代码中的 SQL 注入应由“审查安全”标记）

何时停止并交接：

当所有 SQL 结果发出后，将其交给“review-code”进行聚合
当用户需要更广泛的安全分析（身份验证、加密、配置）时，重定向到“审查安全”
当用户需要全面审查（范围+语言+cognitive）时，重定向到“审查代码”

自检（Self-Check）

核心成功标准

仅 SQL 范围：仅审查 SQL 和查询约定；未执行范围选择、完全安全性或架构分析
涵盖所有七个 SQL 维度：注入/参数化、索引/执行计划、事务/隔离、NULL/唯一约束、方言/可移植性、大表/分页模式以及相关的敏感列/权限
结果格式兼容：每个结果包括位置、类别（语言-sql）、严重性、标题、描述和可选建议
已标记关键注入问题：SQL 注入模式（字符串连接、用户输入插值）被标记为“严重”严重性
位置精确引用：所有结果都引用特定文件：行或查询标识符位置

流程质量检查

是否仅审查了 SQL/查询维度（没有超出查询设计的范围/架构）？
是否涵盖相关的参数化、索引、事务、NULL/约束和可移植性？
每个发现是否都包含位置、类别=语言-sql、严重性、标题、描述和可选建议？
问题是否通过 file:line 或查询标识符引用？

验收测试

输出是否包含涵盖所有相关查询维度的 SQL 结果列表，其中注入风险标记为“关键”以及每个结果的具体位置引用？

示例（示例）

示例 1：查询中的字符串连接

输入：使用字符串连接（包括用户输入）构建的查询。
预期：发出 SQL 注入的关键发现；建议参数化查询或准备好的语句。类别=语言-sql。

示例 2：没有分页的大表

输入：SELECT * FROM large_table ORDER BY id，没有 LIMIT 或分页。
预期：发出性能和可扩展性的结果；建议分页（例如键集或 OFFSET/FETCH），并在不需要时避免使用 SELECT *。类别=语言-sql。

边缘情况：ORM 生成的 SQL

输入：仅使用 ORM 的应用程序代码；生成的 SQL 不可见。
预期：检查代码中的任何原始 SQL 或查询构建器；如果没有可见的 SQL，请说明并跳过或报告“范围内没有要审查的 SQL”。不要发明 SQL。

附录：输出合约

每项调查结果必须遵循标准调查结果格式：

元素	要求
位置	`path/to/file.ext`（可选行或范围）或查询标识符。
类别	`语言-sql`。
严重性	`关键` \| `主要` \| `次要` \| `建议`。
标题	简短的一行摘要。
描述	1-3 句话。
建议	具体修复或改进（可选）。

示例：

- **Location**: `scripts/orders.sql:12`
- **Category**: language-sql
- **Severity**: critical
- **Title**: Query built with string concatenation; injection risk
- **Description**: User-controlled input is concatenated into the WHERE clause.
- **Suggestion**: Use parameterized query or prepared statement with placeholders.

review-sql