Worktree dream backfill#21
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…alContext Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…MCPToolOutput Complements claude-memory-60m's additionalContext path (which skips mcp__ tools). Gated MCP-only; falls through to the capture queue so event capture is unaffected. Best-effort: no hit / any error passes the output through unchanged. Reuses tj_core::recall::relevant_recall — no recall logic here. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
No version bump — workspace is already 0.12.0; sibling bullet under the existing push-recall (additionalContext) entry. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Add nullable alternatives column to decisions table (migration v7) and project a decision event's meta.alternatives JSON into it on index. The append-only log is untouched; existing decisions stay NULL. Pack cache is wiped so packs re-render once events carry alternatives. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
render_active_decisions now reads the alternatives column and renders a 'considered:' block under each decision, marking the chosen option and showing each rationale. Malformed/empty payloads are skipped so the decision itself always renders. Coexists with the Completeness and Subtasks sections added earlier. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
EventAddParams gains an optional alternatives JSON value. On a decision event it is stamped onto meta.alternatives; on any other event type the handler rejects it with a clear error. MCP-only for v1 — no CLI path. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…r prompt Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…ntexts Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…er files Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
- cargo fmt --all over the epic's new code (fmt check was failing CI) - task_pack_returns_rpc_error_when_state_dir_is_unusable now holds the handler_env() mutex; it mutates the process-global XDG_DATA_HOME which the task_create/task_close handler tests read, so under parallel CI it poisoned their env mid-run and task_close_reports_completeness_gaps failed with an unrelated path error.
- needless borrow (cli.rs test) + map_or->is_some_and (cli main) via clippy --fix - mcp tests: reword handler_env doc to drop the '+' markdown-list marker (doc_list_item_without_indentation), and #![allow(await_holding_lock)] on the tests module — the guard is intentionally held across .await to serialize handler tests on the current-thread runtime.
- export_memory_dry_run test: assert filename + 'memory' separately instead of the 'memory/<file>' literal — Windows prints a backslash separator. - ci: msrv job pins dtolnay/rust-toolchain@1.88 again (dependabot #17 bumped it to @1.100, which installs a non-existent rust 1.100.0 → 404).
cargo doc with RUSTDOCFLAGS=-D warnings rejected the bare <id> in the create --parent / task_create parent doc comments as an unclosed HTML tag. Reworded to 'the given id'.
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
Type of change
Test plan
behavior (feature)
cargo fmt --all -- --checkcargo clippy --workspace --all-targets -- -D warningscargo test --workspace --all-targetscargo doc --workspace --no-depswithRUSTDOCFLAGS=-D warningsCHANGELOG
## [Unreleased](Added / Changed / Removed/ Fixed / BREAKING) — or this PR doesn't affect users.
Related issues