Skip to content

Fix AgentMemory slot MCP errors and Windows portability - #1

Merged
arkhipowiktor-dev merged 15 commits into
mainfrom
codex/fix-agentmemory-errors
Aug 3, 2026
Merged

Fix AgentMemory slot MCP errors and Windows portability#1
arkhipowiktor-dev merged 15 commits into
mainfrom
codex/fix-agentmemory-errors

Conversation

@arkhipowiktor-dev

@arkhipowiktor-dev arkhipowiktor-dev commented Aug 3, 2026

Copy link
Copy Markdown
Owner

Результат

Исправлены сбои AgentMemory, обнаруженные в локальной установке Виктора, и восстановлена полноценная кроссплатформенная проверка проекта.

Функциональные исправления

  • MCP-вызовы memory_slot_* при отключённых optional slots больше не вызывают HTTP 500: возвращается контролируемый MCP-ответ с инструкцией включить AGENTMEMORY_SLOTS=true и перезапустить сервис;
  • диагностика различает намеренно глобальные sessionless memories и действительно мигрируемые session-linked записи без project scope;
  • Windows-пути одинаково обрабатываются в hooks, CLI tests, compression tests и Obsidian export tests;
  • filesystem watcher канонизирует Windows 8.3 short paths перед fs.watch, устраняя подтверждённый native crash Node.js/libuv;
  • shutdown filesystem watcher теперь ожидает закрытия native watcher handles;
  • сборка копирует assets кроссплатформенным Node-скриптом вместо POSIX cp / mkdir -p;
  • generated skills drift проверяется одинаково при LF и CRLF;
  • отображаемые счётчики актуализированы: 130 total REST/MCP surfaces, 119 endpoints в generated REST reference.

CI

Windows возвращён в полную CI-матрицу:

  • Ubuntu: Node 20, 22, 24, 26;
  • macOS: Node 20, 22, 24, 26;
  • Windows: Node 20, 22, 24, 26.

Exact-head проверка:

  • head: ca8ba72785602e2ca124ddf61fb8c4c13010bac6;
  • workflow run: CI #14 / run 30805422866;
  • результат: 12 / 12 jobs passed;
  • в каждом job успешно выполнены dependency install, TypeScript build, generated skills validation и полный npm test.

Подтверждённая внешняя причина Windows crash

Сбой Assertion failed: !_wcsnicmp(filename, dir, dirlen), file src\win\fs-event.c, line 72 является подтверждённой регрессией Node.js/libuv на Windows Server 2025:

  • Node.js issue: nodejs/node#63638 (confirmed-bug, windows, libuv);
  • upstream fix: libuv/libuv#5152 — merged, но ещё отсутствует в используемых Node 24.18 и Node 26 builds.

AgentMemory больше не зависит от ожидания обновления Node: watcher передаёт libuv канонический длинный путь.

Изменённые области

21 файл:

  • CI/build: .github/workflows/ci.yml, package.json, scripts/copy-build-assets.mjs;
  • runtime: src/mcp/server.ts, src/functions/diagnostics.ts, src/hooks/_project.ts, integrations/filesystem-watcher/*, src/index.ts;
  • generated docs: README.md, AGENTS.md, plugin/skills/agentmemory-rest-api/REFERENCE.md, scripts/skills/generate.ts;
  • regression/portability tests: test/cli-remove.test.ts, test/compress-file.test.ts, test/copilot-plugin.test.ts, test/diagnostics.test.ts, test/fs-watcher.test.ts, test/mcp-slots-disabled.test.ts, test/obsidian-export.test.ts, test/slots-flag-gate.test.ts.

Аудит

  • fork main и upstream rohitg00/agentmemory:main подтверждённо находятся на общей базе 5023cf3ccbbecc4e89c345bb56e1a3de756ac144;
  • полный diff просмотрен: случайных файлов и реальных credentials не обнаружено;
  • новые строки, похожие на секреты, присутствуют только как синтетические regression fixtures для проверки redaction;
  • интеграция с backend VakhtovikPlus не выполнялась;
  • sessionless memories остаются глобальными по текущему контракту AgentMemory и не должны использоваться как многопользовательская product memory VakhtovikPlus без отдельной tenant/user isolation.

Статус

DRAFT / CI GREEN / READY FOR OWNER DECISION

PR намеренно остаётся Draft. Ready и merge выполнять только после отдельного подтверждения владельца.

@arkhipowiktor-dev arkhipowiktor-dev changed the title Fix AgentMemory slot MCP errors and Windows test portability Fix AgentMemory slot MCP errors and Windows portability Aug 3, 2026
@arkhipowiktor-dev
arkhipowiktor-dev marked this pull request as ready for review August 3, 2026 10:30
@arkhipowiktor-dev
arkhipowiktor-dev merged commit 7e04d6a into main Aug 3, 2026
12 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.

1 participant