Skip to content

fix(memory): repair signed-package service restart on Windows and macOS - #360

Merged
syzsunshine219 merged 1 commit into
MemTensor:v1.1.3from
Memtensor-AI:codex/fix-memory-service-restart-v1-1-3
Sep 5, 2026
Merged

fix(memory): repair signed-package service restart on Windows and macOS#360
syzsunshine219 merged 1 commit into
MemTensor:v1.1.3from
Memtensor-AI:codex/fix-memory-service-restart-v1-1-3

Conversation

@Memtensor-AI

@Memtensor-AI Memtensor-AI commented Sep 5, 2026

Copy link
Copy Markdown
Collaborator

Bug

After installing the signed 1.1.2 package, restarting the Memory service fails. Windows reports access denied from schtasks, while macOS reports a memory layer network error.

Root cause

The packaged Desktop startup path still asked the Memory installer to register an operating-system service. On Windows, a non-elevated signed package could not reliably overwrite an existing scheduled task. On macOS, launchd KeepAlive could race with restart and health checks, bringing the old process back and causing port contention.

Fix

  • Skip OS service registration and installer health checks for packaged Desktop startup.
  • Let Desktop directly own the bundled Memory child process.
  • Add runtime-pointer loading, SQLite-lock-aware waiting, protocol-compatible health waiting, graceful shutdown, and legacy-service takeover.
  • Preserve standalone OS service registration for the regular Memory CLI.
  • Add regression coverage that starts the materialized bundled runtime as a Desktop child.

Validation

  • Project Harness Full evidence passed.
  • Root lint and typecheck passed.
  • Desktop, Memory, and App Agent test suites passed.
  • Memory-layer smoke tests passed 3/3.

Fix commit: 080c4ce

@Memtensor-AI
Memtensor-AI marked this pull request as ready for review September 5, 2026 02:50
@syzsunshine219
syzsunshine219 merged commit 19cd99d into MemTensor:v1.1.3 Sep 5, 2026
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.

2 participants