feat: isolate developer memorize runs - #700
Open
MrXnneHang wants to merge 3 commits into
Open
MrXnneHang wants to merge 3 commits into
MrXnneHang wants to merge 3 commits into
Conversation
Allocate a private directory for each prepare batch and return its run id. Target commit, resource verification, and explicit discard by that id. Keep failed backend commits retryable and remove successful runs completely. Document the CLI migration, executor ownership, and backend conflict boundary. BREAKING CHANGE: memorize commit and verify-resources require a run id. Co-authored-by: Codex <codex@openai.com> AI-Model: gpt-6-astra
Co-authored-by: Codex <codex@openai.com> AI-Model: gpt-6-astra
Report successful backend submission when snapshot refresh or working-file cleanup fails, so callers can discard the stopped run instead of resubmitting. Cover snapshot, job, and directory cleanup failures in regression tests. Co-authored-by: Codex <codex@openai.com> AI-Model: gpt-6-astra
13 tasks
MrXnneHang
added this pull request to stack #702
September 14, 2026 08:30
This was referenced Sep 14, 2026
MrXnneHang
changed the title
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📝 Pull Request Summary
Developer memorize 的固定工作区会让独立调用相互覆盖。本改动为每次 prepare 分配独立 run，并提供明确的提交和丢弃生命周期。
这是供分层评审的 stack 底层，与 xnne-bot 的 #698 使用相同提交。最终合入由 #698 承接；本 PR 用于评审，请勿直接合入。
✅ What does this PR do?
🤔 Why is this change needed?
独立调用需要隔离临时文件，同时保留明确的后端写入边界。工作区隔离不解决同一后端 RecallFile 的并发覆盖，相关完整演化周期仍需协调串行执行。
实现与讨论：#698。上层 PR 添加 host 主动 memorize 指引；evolve 类型选择由 #697 跟踪。
🔍 Type of Change
✅ PR Quality Checklist
feat:,fix:,docs:,memory base:)沿用 #698 对相同提交的验证：699 passed、7 skipped；mypy / pre-commit 通过，Python 3.11 / 3.13 CI 通过。真实 CLI 并行 prepare 已确认工作区独立，commit / discard 清理互不干扰。
本次仅建立评审 stack，没有新增代码改动。
📌 Optional