refactor(memory): remove memorizer and file-backed memory backends#233
Merged
Conversation
Deletes the Memorizer-MCP and file-backed tool implementations, their tests, and all conditional branches that routed to them. SQLite is the only supported memory backend. Changes: - Delete 11 source files: Memorizer*Tool, MemorizerMemoryExtractor, File*Tool, FileMemoryStore, FileMemoryExtractor, StoreMemoryTool - Delete 12 test files covering the removed tools - Remove FileBacked/MemorizerConnected/MemorizerDisconnected from MemoryContextState enum and MemoryIndexContextLayer switch - Remove MemoryConfig.Provider property and schema enum (sqlite only) - Simplify ToolIndexUpdater: drop _memoryConfig field, hardcode ResolveMemoryState() to SqlitePrimary - Simplify DaemonRuntimeStatusService: drop fileMemoryStore param and memorizer/file branches from BuildMemoryStatusAsync - Remove memorizer-usage system skill and embedded copy - Remove memorizer references from capability-reference skill - Extract FakeNetclawTool to shared test helper (was in deleted file) - Update ConfigSchemaDoctorCheckTests to not reference removed Provider
Follow-up cleanup addressing review findings from the initial deletion. Init wizard: - Remove memory backend selection step (Memorizer/files choices) - Remove MemorizerUrl, StartMemorizerProbe, ProbeMemorizerAsync - Remove Memory.Provider from generated config - Remove memorizer MCP server generation from config output - Update TOOLING.md memory triage table to reference memory tools - Delete 3 dead wizard tests, fix sqlite assertion Production code: - Remove dead memorizer/files switch arms from CLI status display - Remove dead MemoryCount/IndexPath/Endpoint/ToolCount from DaemonRuntimeStatus.Memory wire type - Remove memorizer/search_memories from verified-tool-finding filter in LlmSessionActor - Simplify MemoryCheckpointHealthDoctorCheck: always assume SQLite - Fix IMemoryExtractor doc comment - Fix NetclawPaths legacy comment Skills & docs: - Remove memorizer-usage row from CLAUDE.md/AGENTS.md skill sync table - Remove memorizer cross-reference from capability-reference 1.0.0/1.1.0 - Update FakeNetclawTool default grant category to "builtin" - Delete dead Passes_WhenMemoryProviderIsNotSqlite test
This was referenced Mar 14, 2026
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
1 participant
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
FileBacked,MemorizerConnected,MemorizerDisconnectedfromMemoryContextStateenum and their switch cases inMemoryIndexContextLayerMemoryConfig.Providerproperty and thememorizer/filesenum values from the JSON schema — SQLite is the only backendToolIndexUpdater: drops_memoryConfigfield, hardcodesResolveMemoryState()toSqlitePrimaryDaemonRuntimeStatusService: removesfileMemoryStoreoptional param and the memorizer/file branches fromBuildMemoryStatusAsyncmemorizer-usagesystem skill and embedded copy; removes memorizer cross-reference fromcapability-referenceFakeNetclawToolto a standalone shared test helper (it was embedded in a deleted file but referenced by SubAgent tests)Test plan
dotnet build Netclaw.slnx— 0 errorsdotnet test Netclaw.slnx— all 896 tests pass (0 failures)dotnet slopwatch analyze— 0 new violations