fix: distiller MCP-tool pin + capture status; yvh diagnosis (0.25.1)#46
Merged
Merged
Conversation
The task-journal-distiller subagent listed the journal MCP tools under one marketplace-specific prefix (mcp__plugin_task-journal_task-journal__*) which would leave it unable to write events on installs with a different prefix. Drop the explicit tools list so it inherits the session tools and can always reach the journal MCP. Also add `capture status` to report the realtime-capture marker state without changing it. claude-memory-yvh Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.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.
yvh diagnosis: the worker is NOT under-recording
Investigated the claim that the classify-worker under-records (claude-memory-yvh). Evidence says it's healthy:
pending/queue: 0 files — fully drained.So a task like
tj-aknbh66f8nlooking "interrupted" was genuinely incomplete (investigation without conclusion) + compaction markers —completeleft it open correctly. Not a capture bug. No worker fix shipped (don't fix what isn't broken).The real find: the global
.capture-disabledmarker was set (from an earlier debugging step this session), silencing auto-capture across all projects — re-enabled at runtime withcapture on.Code changes
task-journal-distillerlisted the journal MCP tools under one marketplace-specific prefix (mcp__plugin_task-journal_task-journal__*). On any install with a different prefix the agent would silently lackevent_addand be useless. Dropped the explicittools:list so it inherits the session tools and can always reach the journal MCP. (Found in review of feat: in-session compaction distiller subagent + advisory (0.25.0) #44.)task-journal capture status— reports ON/OFF of the realtime-capture marker without changing it (the barecapturehad onlyon/off).Tests
capture_status_reports_current_state(ON fresh, OFF afteroff, status doesn't flip). Full local gate green.🤖 Generated with Claude Code