Skip to content

fix: serialize Viewer config writes - #2513

Draft
Memtensor-AI wants to merge 7 commits into
mainfrom
codex/memos-config-concurrency-20261017
Draft

Memtensor-AI wants to merge 7 commits into
mainfrom
codex/memos-config-concurrency-20261017

Conversation

@Memtensor-AI

@Memtensor-AI Memtensor-AI commented Oct 17, 2026 •

Copy link
Copy Markdown
Collaborator

Problem

The Viewer /api/config read-modify-write path can race when multiple PATCH requests arrive together. In the disposable MemOS evaluation this surfaced as HTTP 500s from competing config writes, which can invalidate evaluation setup and user configuration.

Change

  • serialize Viewer config writes with an in-process write lock
  • add a 10-request concurrent PATCH regression test
  • preserve existing merge and restart behavior

Validation

  • npm test -- --run tests/viewer-config.test.ts — 2/2 passed
  • Project Harness Full revision 0 — verificationPassed=true, readyForHandoff=true
  • exact HEAD: 3726e91d4ed4dd7670598aaa4adc90659a86d522
  • Harness evidence uses the full Ruff + pytest + coverage command with HF_HUB_DISABLE_XET=1 to avoid an execution-host-only Xet TLS retry loop.

Formal release approval remains separate from this Draft PR.

@Memtensor-AI Memtensor-AI added area:plugin OpenClaw & Hermes status:in-progress Someone or AI is working on it | 人工或 AI 正在处理 labels Oct 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:plugin OpenClaw & Hermes status:in-progress Someone or AI is working on it | 人工或 AI 正在处理

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants