Skip to content

fix: preserve disabled Prompt files and filter invalid daily memories - #184

Draft
NongHua123 wants to merge 6 commits into
mainfrom
codex/issue-141-prompt-memory-closeout
Draft

fix: preserve disabled Prompt files and filter invalid daily memories#184
NongHua123 wants to merge 6 commits into
mainfrom
codex/issue-141-prompt-memory-closeout

Conversation

@NongHua123

@NongHua123 NongHua123 commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

Scope and rationale / 范围与原因

修复 #141 中两个已在 0.4.4 原生复现的 P1:

  • 保存新建或已停用的 Prompt,以及从文件导入时,只更新提示词库,保留生效文件和恢复记录。只有真正 enabled→disabled 的转换才执行原有停用逻辑。显式导入使用已有 UUID 能力,避免同秒 ID 碰撞覆盖已启用条目。
  • Daily Memory 列表、搜索与直接读写删除共用严格 ASCII/真实日历日期校验,过滤 README、非法日期和同名目录,使合法日记仍能读取、搜索、保存。

改动收敛到既有两个后端模块和相邻回归测试;没有新增 IPC、数据库迁移或依赖。同步维护文档和行为合同。

Related work / 关联工作

Refs #141。只覆盖两个具体 P1,不关闭汇总票或其余 UI/Windows findings。基线为 v0.4.4/main 2f264d2f,修复提交 c6303266

规划、审阅及原生验收记录

Evidence / 验证证据

  • 聚焦回归:旧实现 10 项中 6 失败;独立审阅补充导入碰撞用例并先复现失败;最终 mise run rust:test issue_141_ 为 11 项通过。
  • 完整本机门禁:mise exec -- uv run --locked --no-sync mise run check:prearchive --exclude-active-task .trellis/tasks/09-08-issue-141-prompt-memory-closeout 通过(该路径为执行时的活动任务目录)。前端 1,582 通过,Rust 主库 3,197 通过,合同 616 通过,其余 suites/格式/类型/lint/Clippy/锁文件检查通过;保留既有 skipped/ignored,不将其记为通过。
  • mise run build:debug 原入口重试成功生成 macOS app/DMG。首次 DMG 失败记录保留;产物仅用于本机调试复验。
  • macOS 原生 CUA:在全新合成资料下验证 Prompt 新建、编辑、导入、启用后再次导入、启用编辑与显式停用;验证混合目录、中文搜索、闰日读取、合法日记保存。文件/数据库回读及恢复副本比对通过,12 项快照断言通过。
  • 退出测试构建后恢复原安装版与正常数据库;真实 Prompt/Memory 的内容摘要、大小和修改时间与测试前一致。
  • 未验证:Windows 原生 GUI、正式签名/发布以及 [UAT] 汇总 macOS 0.4.2 / Windows 0.4.0 NO-GO 验收发现与复验计划 #141 其他 findings。完整日志、用户路径和原始截图不进入 Git/Issue;截图仅保留在 CUA 操作历史。

Risk and rollback / 风险与回退

保留显式启用、已启用内容保存、显式停用和合法日记的原有语义;旧 Prompt ID 不迁移。新导入 ID 仍保留 imported- 前缀。实现不重写既有事务/文件写入机制。需要回退时,反向应用 c6303266 中两个源文件及三份现行合同/维护文档的差异,保留后续归档资料。五文件反向补丁已通过 git apply --reverse --check,未实际回滚;无数据库降级步骤。原安装版未替换,运行环境恢复已核验。

Contract and provenance impact / 契约与来源影响

  • 可执行回归测试与现行维护文档已同步。
  • 上游同步不适用;本分支基于当前 main,不引入重复实现。
  • CI/Release 配置变更不适用;不修改版本或发布流程。

Checklist / 检查清单

  • 当前宿主完整门禁通过(使用上述归档前入口)。
  • 测试覆盖旧失败行为与正常成功路径。
  • 没有用户可见文案变更,locale/无障碍更新不适用。
  • 未包含 secret、证书、个人配置、虚拟环境或用户数据。
  • 明确区分本地检查、原生操作与正式发布证据。

@github-actions github-actions Bot added documentation Improvements or additions to documentation backend labels Sep 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant