feat: continuous task distillation to capture learnings during execution #243
Conversation
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
This adds a mechanism to persist project knowledge so that Claude doesn't have to re-explore the codebase every time a new task starts. - Add GenerateMemoriesMD() function that creates/updates .claude/memories.md from accumulated project memories - Use .claude/memories.md instead of CLAUDE.md to avoid clobbering any existing project documentation - Call GenerateMemoriesMD() after each successful memory extraction - Organize memories by category (patterns, context, decisions, gotchas) - Add comprehensive tests including verification that existing CLAUDE.md files are not modified Claude Code reads files in the .claude/ directory, so this provides context without interfering with manually-maintained CLAUDE.md files. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
bborn
force-pushed
the
task/531-how-can-we-avoid-the-executor-having-to
branch
from
18d823b to
bfb1163
Compare
Instead of only extracting memories when tasks complete, this change implements continuous distillation that triggers: - When new compaction content exists (compaction is newer than last distillation) - When enough time has passed (10+ min) since last distillation Key changes: - Add last_distilled_at field to Task struct and DB - Add shouldDistill() function with time/content gating - Add MaybeDistillTask() that runs distillation asynchronously - Call MaybeDistillTask after every ExecResult (executor-agnostic) - Refactor distillation logic into task_distillation.go This ensures project memories are captured even for long-running tasks that never reach completion, and knowledge is extracted continuously rather than only at task end. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
bborn
changed the title
Merged main which added claude_pane_id and shell_pane_id columns to the tasks table. Resolved conflicts by including both sets of columns (pane IDs from main + summary/last_distilled_at from this branch) in all SELECT queries and Scan statements. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Merged main which added pinned column to the tasks table. Resolved conflicts by including both pinned and summary columns in all SELECT queries and Scan statements. Kept both SaveTaskSummary and UpdateTaskPinned functions. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Merged main which deprecated the Recurrence field and updated related comments. Kept LastDistilledAt field for distillation tracking. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
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
.claude/memories.mdfrom project memories (persists across sessions)Changes
last_distilled_atfield to Task struct and DB migrationshouldDistill()function with time/content gatingMaybeDistillTask()that runs distillation asynchronously after every ExecResulttask_distillation.goGenerateMemoriesMD()creates.claude/memories.mdfile that Claude reads automaticallyTest plan
shouldDistilltrigger logic🤖 Generated with Claude Code