Skip to content

feat(aidd-refine): rename improve into review-conversation and read whole session trees - #1068

Open
alexsoyes wants to merge 5 commits into
nextfrom
feat/review-conversation-skill
Open

alexsoyes wants to merge 5 commits into
nextfrom
feat/review-conversation-skill

Conversation

@alexsoyes

@alexsoyes alexsoyes commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

🎯 What & why

05-improve read one conversation. A delegated run spreads over a tree of sessions, and most of the waste sits in the children. The skill becomes 05-review-conversation: it reads the whole tree and says who owns each cost.

🛠️ How it works

  • 01-read-conversation.md: reads every child session; when the tree exceeds one context, one delegated reader per child with session-review-brief.md (eight categories, evidence by message sequence). Per-session measurements: errors by tool, replayed commands, files reread, polling, compactions, user messages after the first.
  • 02-recommend.md: every finding gets one owner: framework text, delegation brief, host or environment, project. Only the first two take an edit.
  • The HTML report keeps its template; only its title changes. The test moved with the assets.

🧪 How to verify

  • node --test scripts/__tests__/review-conversation-report.test.js
  • Run aidd-refine:05-review-conversation <session> on a session that spawned sub-agents: one report per child session in the run directory, one owner per finding in the HTML report.

✅ I certify

  • I DO CERTIFY I READ EACH LINE OF THE PULL REQUEST BECAUSE I AM A SOFTWARE ENGINEER, NOT A AI PUPPY.

@alexsoyes
alexsoyes force-pushed the feat/review-conversation-skill branch from 3c53241 to 56c5cec Compare October 21, 2026 02:01
@alexsoyes
alexsoyes marked this pull request as ready for review October 21, 2026 02:34
@alexsoyes
alexsoyes requested a review from a team as a code owner October 21, 2026 02:34
Comment thread plugins/aidd-refine/skills/05-review-conversation/actions/01-read-conversation.md Outdated
Comment thread plugins/aidd-refine/skills/05-review-conversation/assets/session-review-brief.md Outdated
@blafourcade blafourcade assigned alexsoyes and unassigned blafourcade Oct 21, 2026
Comment thread plugins/aidd-refine/skills/05-review-conversation/actions/01-read-conversation.md Outdated
@blafourcade blafourcade added this to the This week milestone Oct 22, 2026 — with Claude
@alexsoyes
alexsoyes requested a review from blafourcade October 22, 2026 08:58
Comment thread plugins/aidd-refine/skills/05-review-conversation/actions/01-read-conversation.md Outdated
…hole session trees

The skill reads every session the conversation spawned, delegates each
one with a fixed brief when the tree exceeds one context, measures per
session, and attributes each finding to its owner: the framework text,
the delegation brief, the host, or the project. Only the first two take
an edit.
…ring contract

- actions follow Input, Output, Process, Test; rules become step sub-items
- conversation-sources moves to references; the brief states how it is filled
- one sub-agent per child session, with a check that each report exists
@alexsoyes
alexsoyes force-pushed the feat/review-conversation-skill branch from ce0aa8b to 287a013 Compare October 23, 2026 09:06
…sions

- the orchestrator lists child sessions and reads only the root
- readers report every measurement the root records
- sources name where each host keeps child sessions; OpenCode and Codex paths corrected
- the report shows each finding's owner and the findings without edit
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants