Skip to content

test(memory): harden evidence schema compatibility harness - #3669

Merged
Dallas98 merged 1 commit into
developfrom
agent/fix-memory-evidence-orm-schema
Aug 14, 2026
Merged

test(memory): harden evidence schema compatibility harness#3669
Dallas98 merged 1 commit into
developfrom
agent/fix-memory-evidence-orm-schema

Conversation

@JasonW404

@JasonW404 JasonW404 commented Aug 14, 2026

Copy link
Copy Markdown
Member

变更内容

  • 加固记忆梦境模块的数据库 Schema 兼容性测试,覆盖两种证据字段的存储约定
  • 验证长期记忆版本中的证据 ID 能够以 JSONB 格式通过 SQLAlchemy 正确读写
  • 验证梦境决策中的证据 ID 继续使用 PostgreSQL VARCHAR[] 类型

背景

记忆梦境数据模型中的两个证据字段有意采用不同的 PostgreSQL 存储类型。本次变更使长期记忆版本的 ORM 映射与其 JSONB Schema 保持一致,并扩展数据库测试工具,以端到端验证两种字段约定。

验证

  • backend/.venv/bin/pytest -q test/backend/database/test_memory_dreaming_schema.py test/backend/database/test_memory_dreaming_postgres_integration.py
  • 结果:2 个测试通过,4 个测试跳过(当前未配置真实 PostgreSQL 集成测试环境)

@JasonW404 JasonW404 changed the title fix(memory): align long-term evidence ORM schema test(memory): harden evidence schema compatibility harness Aug 14, 2026
@codecov

codecov Bot commented Aug 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@JasonW404
JasonW404 marked this pull request as ready for review August 14, 2026 07:56
Copilot AI lite review requested due to automatic review settings August 14, 2026 07:56

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Dallas98
Dallas98 merged commit 1ec01f1 into develop Aug 14, 2026
13 checks passed
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.

3 participants