fix: get_indexing_status lastUpdated remains stale after incremental sync#404
Open
linze0721 wants to merge 1 commit into
Open
Conversation
…illiztech#395) Signed-off-by: Berserk Agent <berserk@agent.local>
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.
Summary
Fixes #395: incremental sync (
handleSyncIndex→reindexByChange) now updates persisted MCP snapshot metadata soget_indexing_statusreflects recent sync activity, not only the last full index.Root cause
SyncManager.handleSyncIndex()completed vector/Merkle updates but never calledSnapshotManagerto bump freshness or save~/.contextsnapshot JSON.get_indexing_statusreadindexedInfo.lastUpdated, which was only set on full index / recovery paths.Changes
config.tsCodebaseInfoIndexed: optionallastFullIndexAt,lastIncrementalSyncAt,lastSyncStatssnapshot.tsrecordIncrementalSyncSuccess()updateslastUpdated, incremental fields, optional file/chunk counts;setCodebaseIndexed()setslastFullIndexAtsync.tsreindexByChangeper codebase, record stats andsaveCodebaseSnapshot()handlers.tslastUpdated)snapshot.incremental-sync.test.ts,sync.incremental-metadata.test.ts; extendedhandlers.get-indexing-status.test.tsVerification
7/7 tests passed.
Commit
563063bberserk/e3b41593-0b22-40ec-a8a8-9181c1d52479(pushed toorigin)Client impact
Agents can distinguish: stale full-index-only timestamp vs successful incremental sync (including zero-change runs) without forcing
index_codebase force=true.Commit
563063b
Branch
berserk/e3b41593-0b22-40ec-a8a8-9181c1d52479
Fixes #395