work/: triage against shipped main (#168, #155, #165, #132, #158)#171
Conversation
`work/` had accumulated a few plan docs whose contents have since landed. Triage per main #3198, executed for the three shipped-cleanly cases (the plugin-system RFC + the still-unshipped docs stay untouched in this PR — they belong to a separate phase). - **DELETE** `work/async-agents.md` — fully shipped in PR #132 (`294ed3b`) with the cross-link follow-up in PR #158 (`1ab9f13`). Content reflected in `dev-docs/agents.md`. - **SHRINK** `work/rendering-next.md` — drop the §2 ✅ COMPLETED signature-unification subsection (ancient history) and §3 "Additional Tool Output Parsers" (`GlobOutput` / `GrepOutput` / `WebSearchOutput` / `WebFetchOutput` all live in `models.py` now). Keep §1 (recursive template), §2 (visitor), and §3 (performance) — still speculative. Renumbered the survivors. - **SHRINK** `work/obsidian-friendly-output.md` — the main implementation shipped in PR #155 (`8de5b89`); per-message Markdown timestamps shipped in PR #165 (`68e5bfb`); the `--combined yes/no/only` flag and the `--expand-paths` bullet-list Markdown index both shipped too. Drop all of that; retain the seven open follow-ups (cache-freshness destination- awareness, archived projects with `--output`, the absolute- `--filter-path`-without-`--expand-paths` silent-exclude footgun, `--filter-path` auto-implying `--all-projects`, `--expand-paths` for single-session, `--dry-run` mode, plus Obsidian frontmatter / wikilinks / peek-debug-logging / symlinks). Untouched (still unshipped or shipped-but-pending follow-up): `work/refactor-reindex-with-ghosting.md`, `work/session-state-propagation.md`, `work/tool-renderer-plugins.md` (the RFC graduates to `dev-docs/plugins.md` only once PR #169 merges — phase 2). No code changes; existing inbound links from `dev-docs/{dag,messages}.md` to `rendering-next.md` remain valid (the file still exists, just shrunk). Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
💤 Files with no reviewable changes (1)
📝 WalkthroughWalkthroughThis PR updates project planning documentation to reflect shipped implementations. The obsidian-friendly-output document transitions from feature planning to post-implementation follow-ups, recording PR deliveries and remaining tasks. Completed work is cleaned up from rendering-next, and async-agents planning is archived. ChangesDocumentation Cleanup and Follow-up Tracking
🎯 1 (Trivial) | ⏱️ ~5 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
work/had accumulated a few plan docs whose contents have since landed. Triage per monk's #3198 mail-thread (federation-internal), executed for the three shipped-cleanly cases. The plugin-system RFC (work/tool-renderer-plugins.md) and still-unshipped plans stay untouched in this PR; they belong to a separate phase.work/async-agents.md— fully shipped in PR Support async agents (#90) #132 (294ed3b) with the cross-link follow-up in PR Cross-link TaskOutput / TaskUpdate headers back to their spawn (#154) #158 (1ab9f13). Content reflected indev-docs/agents.md.work/rendering-next.md— drop the §2 ✅ COMPLETED signature-unification subsection (ancient history) and §3 "Additional Tool Output Parsers" (GlobOutput/GrepOutput/WebSearchOutput/WebFetchOutputall live inmodels.pynow). Keep the recursive-template / visitor-pattern / performance speculative sections. Renumbered survivors.work/obsidian-friendly-output.md— the main implementation shipped in PR Obsidian-friendly output: --output dir + --expand-paths + --filter-path (#151) #155 (8de5b89); per-message Markdown timestamps in PR Per-message timestamps in Markdown output (#160) #165 (68e5bfb);--combined yes/no/only+--expand-pathsbullet-list Markdown index also shipped. Drop all that. Retain the seven open follow-ups (cache-freshness destination-awareness, archived projects with--output, the absolute---filter-path-without---expand-pathssilent-exclude footgun,--filter-pathauto-implying--all-projects,--expand-pathsfor single-session,--dry-runmode, plus Obsidian frontmatter / wikilinks / peek-debug-logging / symlinks).Net diff: 3 files changed, 29 insertions, 728 deletions.
Untouched (still unshipped or shipped-but-pending follow-up):
work/refactor-reindex-with-ghosting.md,work/session-state-propagation.md,work/tool-renderer-plugins.md(the RFC will graduate todev-docs/plugins.mdonly once PR #169 merges — phase 2).Test plan
dev-docs/{dag,messages}.mdtorendering-next.mdremain valid (the file still exists, just shrunk).just ciclean (format / test-all / ruff / pyright / ty).async-agents.mdorobsidian-friendly-output.mdfrom anywhere else in the repo (verified viarg).🤖 Generated with Claude Code
Summary by CodeRabbit