Skip to content

fix: disable graph DB mmap before WAL#117

Merged
ScriptedAlchemy merged 1 commit into
masterfrom
codex/windows-mcp-handler-mmap
Jun 24, 2026
Merged

fix: disable graph DB mmap before WAL#117
ScriptedAlchemy merged 1 commit into
masterfrom
codex/windows-mcp-handler-mmap

Conversation

@ScriptedAlchemy

@ScriptedAlchemy ScriptedAlchemy commented Jun 24, 2026

Copy link
Copy Markdown
Owner

Summary

  • move graph DB PRAGMA mmap_size ahead of journal_mode = WAL
  • mirror the GlobalDb Windows mmap ordering used by fix: disable GlobalDb mmap on Windows #115
  • target the Windows 6/8 master failure from run 28077056988, where mcp_handler_test aborted with 0xc0000005/os error 998

Verification

  • cargo fmt --check
  • cargo test -p tracedecay --test mcp_handler_test path_containment_read_rejects_parent_traversal_before_serving_file -- --exact
  • cargo test -p tracedecay --test mcp_handler_test test_by_qualified_name_finds_indexed_node -- --exact
  • cargo test -p tracedecay mmap -- --nocapture

@changeset-bot

changeset-bot Bot commented Jun 24, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: aa508af

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ScriptedAlchemy ScriptedAlchemy merged commit a6ca211 into master Jun 24, 2026
16 checks passed
@ScriptedAlchemy ScriptedAlchemy deleted the codex/windows-mcp-handler-mmap branch June 24, 2026 05:43
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