# 迁移安全检查清单

## 描述
在开启或更新拉取请求（PR）之前，检查当前 PR 中的每个数据库迁移是否存在安全问题。

## 使用时机
当 PR 差异中包含 `db/migrate/`（Rails）、`migrations/`（Django）或 `prisma/migrations/`（Prisma）中新增或修改的文件时，调用此技能。

## 检查清单

### 1. 检测破坏性操作
扫描每个迁移文件，查找以下内容：
- `DROP TABLE` 或 `drop_table`
- `DROP COLUMN`、`remove_column` 或列删除操作
- `TRUNCATE`
- 导致精度损失的数据类型变更（例如 `text` → `varchar(50)`）

如发现上述任何情况，在 PR 中添加评论，标记该操作并确认迁移中是否包含数据备份步骤。

### 2. 验证外键索引
对于每个 `add_reference`、`add_foreign_key` 或以 `_id` 结尾的新列，确认存在对应的索引。如果不存在，请在提交前将其添加到迁移中。

### 3. 检查回滚安全性
- 在测试数据库上运行 `bin/rails db:migrate:rollback STEP=<n>`（其中 n 为新迁移的数量）。
- 如果回滚失败，添加 `down` 方法或可逆块后重试。
- 在 PR 描述中报告所有不可逆的迁移。

### 4. 验证 schema 文件是否为最新版本
运行迁移后，将 `db/schema.rb`（或 `structure.sql`）与 PR 中的版本进行差异比较。如果存在差异，请重新生成 schema 文件并将其包含在提交中。

### 5. 运行模型测试
执行 `bin/rails test test/models/`，捕获因 schema 变更导致验证或关联失效的问题。所有测试必须通过后方可开启 PR。

### 6. Enforce migration naming conventions
Reject migrations that don't follow the pattern
`YYYYMMDDHHMMSS_verb_noun.rb` (e.g. `add_index_to_invoices`).
Rename the file if needed.

### 7. Check for long-running locks
Flag any `add_column` on tables with >1M rows that doesn't use
`disable_ddl_transaction!` (Postgres) or `ALGORITHM=INPLACE`
(MySQL). Large tables need non-blocking migrations.

### 8. Require a migration test
Ensure a corresponding test file exists at
`test/migrations/YYYYMMDDHHMMSS_migration_name_test.rb`.
If missing, generate a skeleton test that runs the migration
up and down.