Skip to content

🐛 fix(metadata): 正确识别 SQLite 自增主键 - #106

Merged
ZhaoXingPeng merged 1 commit into
mainfrom
fix/105-sqlite-autoincrement
Sep 7, 2026
Merged

ZhaoXingPeng merged 1 commit into
mainfrom
fix/105-sqlite-autoincrement

Conversation

@ZhaoXingPeng

@ZhaoXingPeng ZhaoXingPeng commented Sep 7, 2026 •

Copy link
Copy Markdown
Owner

关联 Issue

Closes #105

背景(Situation)

SQLite 的 PRAGMA table_info 对 INTEGER PRIMARY KEY AUTOINCREMENT 只返回 INTEGER,不会把 AUTOINCREMENT 写入 type 字段。原元数据读取只检查 type,代码生成因此丢失自增语义。

任务(Task)

从 SQLite catalog 读取表 DDL,准确区分带 AUTOINCREMENT 的 INTEGER 主键和普通主键;保持表名参数化和其他数据库查询路径不变。

行动(Action)

  • 查询 sqlite_master 的表定义并检测 AUTOINCREMENT 关键字。
  • 仅对 INTEGER 主键的真实自增表标记 auto_increment=true。
  • 使用绑定参数读取表名,保留 SQLite 标识符安全边界。
  • 增加真实 SQLite 表、普通主键和特殊表名回归测试。
  • 没有做什么:不修改 MySQL/PostgreSQL SQL、DDL、模板或数据库数据。

验证(Verification)

环境:Windows 11,Python 3.12.12 / SQLite 3.50.4。

  • SQLite 元数据与跨数据库契约定向测试:14 passed。
  • PYTHONPATH=src python -m pytest tests/unit/ -q:650 passed。
  • Ruff 检查、格式检查和 git diff --check:通过。

实验与证据(Evidence)

创建 generated_ids (id INTEGER PRIMARY KEY AUTOINCREMENT, value TEXT) 后,修复前 id 的 auto_increment 为 false,修复后为 true;value 和普通 INTEGER PRIMARY KEY 保持 false。表名通过 ? 参数传递,未拼接进 catalog SQL。

兼容性、风险与回滚

  • 公共 API / 数据格式 / 迁移:补全现有 normalized metadata 字段,不改变其他方言。
  • 风险与已知限制:读取一次 sqlite_master 使 SQLite 描述查询往返增加;未测性能收益,后续基准需以新次数为基线。
  • 回滚:revert 对应提交即可恢复旧检测逻辑。
  • 后续 Issue:如需覆盖 SQLite WITHOUT ROWID、生成列等特殊 DDL,应另建 metadata Issue。

@ZhaoXingPeng

Copy link
Copy Markdown
Owner Author

问题复现:对 id INTEGER PRIMARY KEY AUTOINCREMENT 执行 PRAGMA table_info 只返回 type=INTEGER,原实现因此将 auto_increment 错误标记为 false。

@ZhaoXingPeng

Copy link
Copy Markdown
Owner Author

实现记录:新增 SQLite 表 DDL 查询,从 sqlite_master 读取并识别 AUTOINCREMENT;表名通过参数绑定,未引入 SQL 标识符拼接。

@ZhaoXingPeng

Copy link
Copy Markdown
Owner Author

行为对比:修复后自增 INTEGER 主键返回 auto_increment=true,普通 INTEGER 主键和非主键列仍返回 false。

@ZhaoXingPeng

Copy link
Copy Markdown
Owner Author

测试记录:新增真实 SQLite fixture;元数据和跨数据库契约测试 14 项通过,完整单元测试 650 项通过,Ruff 和 diff 检查通过。

@ZhaoXingPeng

Copy link
Copy Markdown
Owner Author

边界说明:SQLite 只在存在主键列时查询表 DDL,减少无关查询;本次未建立可比性能基准,性能数据未测量。

@ZhaoXingPeng
ZhaoXingPeng merged commit 9fa3ca6 into main Sep 7, 2026
6 of 10 checks passed
@ZhaoXingPeng
ZhaoXingPeng deleted the fix/105-sqlite-autoincrement branch September 9, 2026 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🐛 fix(metadata): 正确识别 SQLite 自增主键

1 participant