π Release v2.1.6 (Force Replace) - #2
Merged
Merged
Conversation
Synced from ukkit/memcord-dev @ 0956e87869afddeb98bc27350529cd305892ff69
β οΈ FORCE REPLACE - All files replaced regardless of modification status
π Summary: 9 files changed
Release notes: https://github.com/ukkit/memcord-dev/releases/tag/v2.1.6
Co-authored-by: Release Bot <release-bot@memcord.dev>
This was referenced Nov 9, 2025
ukkit
pushed a commit
that referenced
this pull request
Nov 10, 2025
**Bug #1: Acronym Tokenization (CRITICAL)** - Problem: Search tokenizer filtered words with len <= 2 - Impact: CI, CD, UI, UX, DB, API, AWS and other acronyms were unsearchable - Fix: Changed tokenizer from 'len > 2' to 'len > 1' (src/memcord/search.py:178) - Result: All 2-letter acronyms now searchable **Bug #2: Search Index Staleness (CRITICAL)** - Problem: Search index never refreshed across MCP instances - Impact: Content saved in one session was not searchable in another session - Root Cause: Per-instance index initialized once, never checked for disk changes - Fix: Added modification time (mtime) tracking and staleness detection - Implementation: - Added _index_mtime_snapshot dict to track file mtimes - Added _is_search_index_stale() to detect changes - Updated search_memory() to check staleness before searching - Updated _save_slot() to maintain mtime snapshot on saves - Added cache invalidation when index refreshes - Result: Cross-session search now works correctly **Testing** - Added comprehensive test suite: tests/test_search_index_staleness.py - 10 tests covering same-instance, cross-instance, external modifications - All 260 tests passing (250 existing + 10 new) - Performance validated: staleness check adds < 50ms overhead **Impact** Fixes core search functionality that was completely broken for: - Cross-session workflows - Multi-instance collaboration - External modifications - Common acronym searches (CI/CD, API, UI, DB, AWS, etc.) Authored by: Aaron Lippold <lippold@gmail.com>
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
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.
Release Sync: v2.1.6
This PR contains the synchronized files from the development repository for release v2.1.6.
π Sync Details
π Changes
Please review the changes carefully before merging.
β Pre-merge Checklist
π Links
This PR was created automatically by the Release Sync workflow.